Make mod overrides higher priority than user configuration

Prevents players inadvertently enabling incompatible options

Author: embeddedt

common/src/main/java/org/embeddedt/modernfix/core/config/ModernFixEarlyConfig.java

@@ -285,7 +285,10 @@ private void readProperties(Properties props) {
                 continue;
             }
 
-            option.setEnabled(enabled, true);
+            if(!option.isModDefined())
+                option.setEnabled(enabled, true);
+            else
+                LOGGER.warn("Option '{}' already disabled by a mod. Ignoring user configuration", key);
         }
     }
 

common/src/main/java/org/embeddedt/modernfix/screen/OptionList.java

@@ -128,7 +128,20 @@ public OptionEntry(String optionName, Option option) {
                     this.option.setEnabled(!this.option.isEnabled(), !this.option.isUserDefined());
                     ModernFix.LOGGER.error("Unable to save config", e);
                 }
+            }, (btn, gfx, x, y) -> {
+                if(this.option.isModDefined()) {
+                    String disablingMods = String.join(", ", this.option.getDefiningMods());
+                    OptionList.this.mainScreen.renderTooltip(
+                            gfx,
+                            new TranslatableComponent("modernfix.option." + (this.option.isEnabled() ? "enabled" : "disabled"))
+                                    .append(new TranslatableComponent("modernfix.option.mod_override", disablingMods)),
+                            x,
+                            y
+                    );
+                }
             });
+            if(this.option.isModDefined())
+                this.toggleButton.active = false;
             this.helpButton = new Button(75, 0, 20, 20, new TextComponent("?"), (arg) -> {
                 Minecraft.getInstance().setScreen(new ModernFixOptionInfoScreen(mainScreen, optionName));
             });

common/src/main/resources/assets/modernfix/lang/en_us.json

@@ -9,6 +9,9 @@
   "modernfix.config.done_restart": "Done (restart required)",
   "modernfix.option.on": "on",
   "modernfix.option.off": "off",
+  "modernfix.option.disabled": "disabled",
+  "modernfix.option.enabled": "enabled",
+  "modernfix.option.mod_override": " by mods [%s]",
   "modernfix.config.not_default": " (modified)",
   "asynclocator.map.locating": "Map (Locating...)",
   "asynclocator.map.none": "Map (No nearby feature found)",