fix: deduplicate fog options by dimension effects

Sanitize fog distance overrides, skip empty dimension option groups, include known custom dimensions, and key fog overrides by dimension effects id so equivalent dimensions share one option.

(cherry picked from commit a749f2a)

Author: FlashyReese

common/src/main/java/me/flashyreese/mods/sodiumextra/client/config/SodiumExtraConfig.java

@@ -217,6 +217,36 @@ private static SodiumExtraGameOptions.AtmosphericFogSettings createDimensionFogS
         return settings;
     }
 
+    private static List<ResourceLocation> getDimensionFogEffectIds(SodiumExtraGameOptions.FogSettings fogSettings) {
+        Set<ResourceLocation> identifiers = new LinkedHashSet<>();
+        WorldDimensions.keysInOrder(Stream.empty())
+                .map(dim -> dim.location())
+                .forEach(identifiers::add);
+        addKnownWorldDimensionEffectIds(identifiers);
+        identifiers.addAll(fogSettings.dimensionOverrides.keySet());
+
+        identifiers.forEach(identifier -> fogSettings.dimensionOverrides.computeIfAbsent(identifier, ignored -> createDimensionFogSettings()));
+
+        return identifiers.stream()
+                .sorted(Comparator.comparing(ResourceLocation::toString))
+                .toList();
+    }
+
+    private static void addKnownWorldDimensionEffectIds(Set<ResourceLocation> identifiers) {
+        Minecraft minecraft = Minecraft.getInstance();
+        if (minecraft == null) {
+            return;
+        }
+
+        if (minecraft.level != null) {
+            identifiers.add(minecraft.level.dimensionType().effectsLocation());
+        }
+
+        if (minecraft.getSingleplayerServer() != null) {
+            minecraft.getSingleplayerServer().getAllLevels().forEach(level -> identifiers.add(level.dimensionType().effectsLocation()));
+        }
+    }
+
     private static void setAtmosphericFogStart(int value) {
         SodiumExtraGameOptions.FogSettings fogSettings = fogSettings();
         int clampedValue = Math.max(0, Math.min(value, 100));
@@ -515,10 +545,7 @@ private OptionPageBuilder createRenderPage(ConfigBuilder builder) {
                 .setName(Component.translatable("sodium-extra.option.render"));
 
         SodiumExtraGameOptions.FogSettings fogSettings = fogSettings();
-        WorldDimensions.keysInOrder(Stream.empty())
-                .map(dim -> dim.location())
-                .filter(identifier -> !fogSettings.dimensionOverrides.containsKey(identifier))
-                .forEach(identifier -> fogSettings.dimensionOverrides.put(identifier, createDimensionFogSettings()));
+        List<ResourceLocation> dimensionFogEffectIds = getDimensionFogEffectIds(fogSettings);
 
         page.addOptionGroup(builder.createOptionGroup()
                 .addOption(builder.createBooleanOption(ADVANCED_FOG_OPTION_ID)
@@ -606,9 +633,7 @@ private OptionPageBuilder createRenderPage(ConfigBuilder builder) {
         );
 
         OptionGroupBuilder dimensionFogGroup = builder.createOptionGroup();
-        fogSettings.dimensionOverrides.keySet().stream()
-                .sorted(Comparator.comparing(ResourceLocation::toString))
-                .forEach(identifier -> dimensionFogGroup.addOption(builder.createIntegerOption(id("fog." + identifier.toLanguageKey("options.dimensions")))
+        dimensionFogEffectIds.forEach(identifier -> dimensionFogGroup.addOption(builder.createIntegerOption(id("fog." + identifier.toLanguageKey("options.dimensions")))
                         .setEnabledProvider(
                                 SodiumExtraConfig::isDimensionFogOptionEnabled,
                                 ADVANCED_FOG_OPTION_ID,
@@ -626,7 +651,9 @@ private OptionPageBuilder createRenderPage(ConfigBuilder builder) {
                         )
                         .setDefaultValue(0)
                 ));
-        page.addOptionGroup(dimensionFogGroup);
+        if (!dimensionFogEffectIds.isEmpty()) {
+            page.addOptionGroup(dimensionFogGroup);
+        }
 
         page.addOptionGroup(builder.createOptionGroup()
                 .addOption(builder.createBooleanOption(PROTECTED_GAMEPLAY_FOG_OPTION_ID)

common/src/main/java/me/flashyreese/mods/sodiumextra/client/config/SodiumExtraGameOptions.java

@@ -351,14 +351,23 @@ public void sanitize() {
             if (this.dimensionOverrides == null) {
                 this.dimensionOverrides = new Object2ObjectArrayMap<>();
             }
-            this.dimensionOverrides.replaceAll((identifier, settings) -> {
+
+            Map<ResourceLocation, AtmosphericFogSettings> sanitizedDimensionOverrides = new Object2ObjectArrayMap<>(this.dimensionOverrides.size());
+            for (Map.Entry<ResourceLocation, AtmosphericFogSettings> entry : this.dimensionOverrides.entrySet()) {
+                ResourceLocation identifier = entry.getKey();
+                if (identifier == null) {
+                    continue;
+                }
+
+                AtmosphericFogSettings settings = entry.getValue();
                 if (settings == null) {
                     settings = new AtmosphericFogSettings();
                 }
 
                 settings.sanitize();
-                return settings;
-            });
+                sanitizedDimensionOverrides.put(identifier, settings);
+            }
+            this.dimensionOverrides = sanitizedDimensionOverrides;
 
             if (this.protectedGameplay == null) {
                 this.protectedGameplay = new ProtectedFogSettings();

common/src/main/java/me/flashyreese/mods/sodiumextra/client/fog/FogDistanceHelper.java

@@ -43,8 +43,8 @@ private FogDistanceHelper() {
 
     public static SodiumExtraGameOptions.AtmosphericFogSettings getAtmosphericSettings(ClientLevel level) {
         SodiumExtraGameOptions.FogSettings fogSettings = getFogSettings();
-        ResourceLocation dimensionId = level.dimension().location();
-        return fogSettings.getAtmospheric(dimensionId);
+        ResourceLocation dimensionEffectsId = level.dimensionType().effectsLocation();
+        return fogSettings.getAtmospheric(dimensionEffectsId);
     }
 
     public static int normalizeFogDistance(int fogDistance) {