From bb5e71c6515ec70a1bcd7fc310eaed9a05bb8212 Mon Sep 17 00:00:00 2001 From: embeddedt <42941056+embeddedt@users.noreply.github.com> Date: Sun, 23 Jul 2023 20:37:57 -0400 Subject: [PATCH 1/2] Remove second config warning, now redundant --- .../java/org/embeddedt/modernfix/forge/config/ConfigFixer.java | 2 -- 1 file changed, 2 deletions(-) diff --git a/forge/src/main/java/org/embeddedt/modernfix/forge/config/ConfigFixer.java b/forge/src/main/java/org/embeddedt/modernfix/forge/config/ConfigFixer.java index ecd00a5d..eaf9621b 100644 --- a/forge/src/main/java/org/embeddedt/modernfix/forge/config/ConfigFixer.java +++ b/forge/src/main/java/org/embeddedt/modernfix/forge/config/ConfigFixer.java @@ -50,8 +50,6 @@ public class ConfigFixer { this.actualHandler.accept(modConfigEvent); } } else { - if(modConfigEvent.getConfig().getConfigData() != null) - ModernFix.LOGGER.warn("Unable to sync on a {} config object", modConfigEvent.getConfig().getConfigData().getClass().getName()); this.actualHandler.accept(modConfigEvent); } } From 7f447016708e7fc77f211a7f1b40b9e34feff5ac Mon Sep 17 00:00:00 2001 From: embeddedt <42941056+embeddedt@users.noreply.github.com> Date: Mon, 24 Jul 2023 16:03:29 -0400 Subject: [PATCH 2/2] Display ModernFix version in early part of log --- .../embeddedt/modernfix/core/ModernFixMixinPlugin.java | 4 ++-- .../platform/forge/ModernFixPlatformHooksImpl.java | 8 +++++++- 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/common/src/main/java/org/embeddedt/modernfix/core/ModernFixMixinPlugin.java b/common/src/main/java/org/embeddedt/modernfix/core/ModernFixMixinPlugin.java index f68fb052..454afa95 100644 --- a/common/src/main/java/org/embeddedt/modernfix/core/ModernFixMixinPlugin.java +++ b/common/src/main/java/org/embeddedt/modernfix/core/ModernFixMixinPlugin.java @@ -32,8 +32,8 @@ public class ModernFixMixinPlugin implements IMixinConfigPlugin { throw new RuntimeException("Could not load configuration file for ModernFix", e); } - this.logger.info("Loaded configuration file for ModernFix: {} options available, {} override(s) found", - config.getOptionCount(), config.getOptionOverrideCount()); + this.logger.info("Loaded configuration file for ModernFix {}: {} options available, {} override(s) found", + ModernFixPlatformHooks.getVersionString(), config.getOptionCount(), config.getOptionOverrideCount()); config.getOptionMap().values().forEach(option -> { if (option.isOverridden()) { diff --git a/forge/src/main/java/org/embeddedt/modernfix/platform/forge/ModernFixPlatformHooksImpl.java b/forge/src/main/java/org/embeddedt/modernfix/platform/forge/ModernFixPlatformHooksImpl.java index e63bfbfa..0339c395 100644 --- a/forge/src/main/java/org/embeddedt/modernfix/platform/forge/ModernFixPlatformHooksImpl.java +++ b/forge/src/main/java/org/embeddedt/modernfix/platform/forge/ModernFixPlatformHooksImpl.java @@ -51,6 +51,7 @@ import java.net.URLClassLoader; import java.nio.file.Path; import java.util.Enumeration; import java.util.Map; +import java.util.Objects; import java.util.Optional; import java.util.function.Consumer; import java.util.function.Function; @@ -68,7 +69,12 @@ public class ModernFixPlatformHooksImpl { public static String getVersionString() { if(verString == null) { - verString = LoadingModList.get().getModFileById("modernfix").getMods().get(0).getVersion().toString(); + try { + verString = ModernFixMixinPlugin.class.getPackage().getImplementationVersion(); + Objects.requireNonNull(verString); + } catch(Throwable e) { + verString = "[unknown]"; + } } return verString; }