don't merge configs in different subfolders (#59)

GTNewHorizons · Sep 1, 2024 · 0b01adb · 0b01adb
1 parent 071cb88
commit 0b01adb
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/src/main/java/com/gtnewhorizon/gtnhlib/config/ConfigurationManager.java b/src/main/java/com/gtnewhorizon/gtnhlib/config/ConfigurationManager.java
@@ -59,7 +59,7 @@ public static void registerConfig(Class<?> configClass) throws ConfigException {
         val modid = cfg.modid();
         val filename = Optional.of(cfg.filename().trim()).filter(s -> !s.isEmpty()).orElse(modid);
 
-        Configuration rawConfig = configs.computeIfAbsent(modid + "|" + filename, (ignored) -> {
+        Configuration rawConfig = configs.computeIfAbsent(getConfigKey(cfg), (ignored) -> {
             Path newConfigDir = configDir;
             if (!cfg.configSubDirectory().trim().isEmpty()) {
                 newConfigDir = newConfigDir.resolve(cfg.configSubDirectory().trim());
@@ -231,9 +231,7 @@ public static List<IConfigElement> getConfigElements(Class<?> configClass) throw
         init();
         val cfg = Optional.ofNullable(configClass.getAnnotation(Config.class)).orElseThrow(
                 () -> new ConfigException("Class " + configClass.getName() + " does not have a @Config annotation!"));
-        val modid = cfg.modid();
-        val filename = Optional.of(cfg.filename().trim()).filter(s -> !s.isEmpty()).orElse(modid);
-        val rawConfig = Optional.ofNullable(configs.get(modid + "|" + filename)).map(
+        val rawConfig = Optional.ofNullable(configs.get(getConfigKey(cfg))).map(
                 (conf) -> Optional.ofNullable(configToCategoryClassMap.get(conf))
                         .map((map) -> map.get(cfg.category().trim()).contains(configClass)).orElse(false) ? conf : null)
                 .orElseThrow(
@@ -267,6 +265,10 @@ public static List<IConfigElement> getConfigElementsMulti(Class<?>... configClas
         }
     }
 
+    private static String getConfigKey(Config cfg) {
+        return cfg.modid() + "|" + cfg.configSubDirectory() + "|" + cfg.filename();
+    }
+
     private static File minecraftHome() {
         return Launch.minecraftHome != null ? Launch.minecraftHome : new File(".");
     }