add null check for modid when registering modmenu screens

2021-12-20 17:52:54 -06:00
1 changed files with 9 additions and 4 deletions
@@ -30,12 +30,17 @@ public class OmegaConfigGui {
     */
    public static <T extends Config> void registerConfigScreen(T config) {
        if(FabricLoader.getInstance().isModLoaded("modmenu")) {
-            OmegaScreenFactory<Screen> factory = OmegaConfigGui.getConfigScreenFactory(config);
+            // Ensure the config has a valid modid.
            if(config.getModid() != null) {
                OmegaScreenFactory<Screen> factory = OmegaConfigGui.getConfigScreenFactory(config);
-            if(modMenuInitialized) {
+                if(modMenuInitialized) {
-                OmegaModMenu.injectScreen(config, factory);
+                    OmegaModMenu.injectScreen(config, factory);
                } else {
                    REGISTERED_CONFIGURATIONS.put(config, factory);
                }
            } else {
-                REGISTERED_CONFIGURATIONS.put(config, factory);
+                OmegaConfig.LOGGER.warn(String.format("Skipping config screen registration for '%s' - you must implement getModid() in your config class!", config.getName()));
            }
        }
    }