From d5d66d79dc6ea9e1f6e4b5c50a0cf5660feffc0a Mon Sep 17 00:00:00 2001 From: JFronny Date: Thu, 21 Oct 2021 18:06:09 +0200 Subject: [PATCH] Extra logging to help debug #13 --- .../respackopts/data/entry/ConfigBooleanEntry.java | 6 +++++- .../respackopts/data/entry/ConfigEnumEntry.java | 5 ++++- .../respackopts/data/entry/ConfigNumericEntry.java | 11 +++++++++-- .../respackopts/integration/ModMenuCompat.java | 1 + .../gitlab/jfronny/respackopts/util/GuiFactory.java | 1 + 5 files changed, 20 insertions(+), 4 deletions(-) diff --git a/src/main/java/io/gitlab/jfronny/respackopts/data/entry/ConfigBooleanEntry.java b/src/main/java/io/gitlab/jfronny/respackopts/data/entry/ConfigBooleanEntry.java index f50d698..25b5b81 100644 --- a/src/main/java/io/gitlab/jfronny/respackopts/data/entry/ConfigBooleanEntry.java +++ b/src/main/java/io/gitlab/jfronny/respackopts/data/entry/ConfigBooleanEntry.java @@ -1,5 +1,6 @@ package io.gitlab.jfronny.respackopts.data.entry; +import io.gitlab.jfronny.respackopts.Respackopts; import me.shedaniel.clothconfig2.api.AbstractConfigListEntry; import meteordevelopment.starscript.value.Value; @@ -34,7 +35,10 @@ public class ConfigBooleanEntry extends ConfigEntry { return guiEntryBuilderParam.entryBuilder().startBooleanToggle(guiEntryBuilderParam.name(), getValue()) .setDefaultValue(getDefault()) .setSaveConsumer(value -> { - if (getValue() != value) guiEntryBuilderParam.saveCallback(); + if (getValue() != value) { + if (Respackopts.CONFIG.debugLogs) Respackopts.LOGGER.info("ConfigBooleanEntry SaveCallback"); + guiEntryBuilderParam.saveCallback(); + } setValue(value); }) .setTooltipSupplier(guiEntryBuilderParam.tooltipSupplier()) diff --git a/src/main/java/io/gitlab/jfronny/respackopts/data/entry/ConfigEnumEntry.java b/src/main/java/io/gitlab/jfronny/respackopts/data/entry/ConfigEnumEntry.java index eff4a41..d05127d 100644 --- a/src/main/java/io/gitlab/jfronny/respackopts/data/entry/ConfigEnumEntry.java +++ b/src/main/java/io/gitlab/jfronny/respackopts/data/entry/ConfigEnumEntry.java @@ -126,7 +126,10 @@ public class ConfigEnumEntry extends ConfigEntry { .setDefaultValue(getDefault()) .setSelections(() -> values.iterator()) .setSaveConsumer(value -> { - if (!Objects.equals(getValue(), value)) guiEntryBuilderParam.saveCallback(); + if (!Objects.equals(getValue(), value)) { + if (Respackopts.CONFIG.debugLogs) Respackopts.LOGGER.info("ConfigEnumEntry SaveCallback"); + guiEntryBuilderParam.saveCallback(); + } setValue(value); }) .setTooltipSupplier(guiEntryBuilderParam.tooltipSupplier()) diff --git a/src/main/java/io/gitlab/jfronny/respackopts/data/entry/ConfigNumericEntry.java b/src/main/java/io/gitlab/jfronny/respackopts/data/entry/ConfigNumericEntry.java index 7c86e8a..172bebd 100644 --- a/src/main/java/io/gitlab/jfronny/respackopts/data/entry/ConfigNumericEntry.java +++ b/src/main/java/io/gitlab/jfronny/respackopts/data/entry/ConfigNumericEntry.java @@ -1,5 +1,6 @@ package io.gitlab.jfronny.respackopts.data.entry; +import io.gitlab.jfronny.respackopts.Respackopts; import io.gitlab.jfronny.respackopts.data.enums.ConfigSyncMode; import io.gitlab.jfronny.respackopts.data.enums.NumericEntryType; import io.gitlab.jfronny.respackopts.gson.entry.NumericEntrySerializer; @@ -82,7 +83,10 @@ public class ConfigNumericEntry extends ConfigEntry { getValue().intValue(), min.intValue(), max.intValue()) .setDefaultValue(getDefault().intValue()) .setSaveConsumer(value -> { - if (!Objects.equals(getValue(), value.doubleValue())) guiEntryBuilderParam.saveCallback(); + if (!Objects.equals(getValue(), value.doubleValue())) { + if (Respackopts.CONFIG.debugLogs) Respackopts.LOGGER.info("ConfigNumericEntrySlider SaveCallback"); + guiEntryBuilderParam.saveCallback(); + } setValue(value.doubleValue()); }) .setTooltipSupplier(guiEntryBuilderParam.tooltipSupplier()) @@ -92,7 +96,10 @@ public class ConfigNumericEntry extends ConfigEntry { DoubleFieldBuilder builder = guiEntryBuilderParam.entryBuilder().startDoubleField(guiEntryBuilderParam.name(), getValue()) .setDefaultValue(getDefault()) .setSaveConsumer(value -> { - if (!Objects.equals(getValue(), value)) guiEntryBuilderParam.saveCallback(); + if (!Objects.equals(getValue(), value)) { + if (Respackopts.CONFIG.debugLogs) Respackopts.LOGGER.info("ConfigNumericEntryNormal SaveCallback"); + guiEntryBuilderParam.saveCallback(); + } setValue(value); }) .setTooltipSupplier(guiEntryBuilderParam.tooltipSupplier()); diff --git a/src/main/java/io/gitlab/jfronny/respackopts/integration/ModMenuCompat.java b/src/main/java/io/gitlab/jfronny/respackopts/integration/ModMenuCompat.java index 231db2c..6b2b172 100644 --- a/src/main/java/io/gitlab/jfronny/respackopts/integration/ModMenuCompat.java +++ b/src/main/java/io/gitlab/jfronny/respackopts/integration/ModMenuCompat.java @@ -23,6 +23,7 @@ public class ModMenuCompat implements ModMenuApi { ConfigEntryBuilder entryBuilder = builder.entryBuilder(); PackReloadType.Aggregator agg = new PackReloadType.Aggregator(); builder.setSavingRunnable(() -> { + if (Respackopts.CONFIG.debugLogs) Respackopts.LOGGER.info("ModMenuCompat SavingRunnable " + agg.get()); Respackopts.CONFIG.save(); Respackopts.save(); if (agg.get() == PackReloadType.Resource) { diff --git a/src/main/java/io/gitlab/jfronny/respackopts/util/GuiFactory.java b/src/main/java/io/gitlab/jfronny/respackopts/util/GuiFactory.java index 92a7bbe..3a91578 100644 --- a/src/main/java/io/gitlab/jfronny/respackopts/util/GuiFactory.java +++ b/src/main/java/io/gitlab/jfronny/respackopts/util/GuiFactory.java @@ -57,6 +57,7 @@ public class GuiFactory { PackReloadType.Aggregator agg = new PackReloadType.Aggregator(); builder.setSavingRunnable(() -> { Respackopts.save(); + if (Respackopts.CONFIG.debugLogs) Respackopts.LOGGER.info("GuiFactory SavingRunnable " + agg.get()); if (agg.get() == PackReloadType.Resource) { Respackopts.forcePackReload = true; DashLoaderCompat.requestForceReload();