chore: cleanup some erro handling and gui state (avoids #10)

Author: TheBossMagnus

common/src/main/java/io/github/thebossmagnus/mods/config_manager/common/AddFlagsUtil.java

@@ -10,20 +10,26 @@ public class AddFlagsUtil {
     private static final AtomicBoolean OVERWRITE_FLAG = new AtomicBoolean(false);
 
 
-    public static void setUpdateFlag(boolean updateFlag) {
+    public static void setUpdateFlag(boolean updateFlag) throws RuntimeException {
         UPDATE_FLAG.set(updateFlag);
         addFlags();
     }
 
 
-    public static void setOverwriteFlag(boolean overwriteFlag) {
+    public static void setOverwriteFlag(boolean overwriteFlag) throws RuntimeException {
         OVERWRITE_FLAG.set(overwriteFlag);
         addFlags();
     }
 
     private static void addFlags() throws RuntimeException {
-        Path gameDir = Services.PLATFORM.getGameDir();
-        Path configDir = gameDir.resolve("config");
+        Path configDir;
+        try {
+            Path gameDir = Services.PLATFORM.getGameDir();
+            configDir = gameDir.resolve("config");
+        } catch (Exception e) {
+            Constants.LOGGER.error("Error while resolving path", e);
+            throw new RuntimeException("Error while resolving path: " + e.getMessage(), e);
+        }
         try {
             if (!Files.exists(configDir)) {
                 Files.createDirectories(configDir);

common/src/main/java/io/github/thebossmagnus/mods/config_manager/common/screen/Gui.java

@@ -16,11 +16,13 @@ public class Gui extends Screen {
     private static final int COLOR_RED = 0xFF0000;
     private static final int COLOR_WHITE = 0xFFFFFF;
     private final Screen parent;
-    private final Component updateWarnings = Component.translatable("* %s\n* %s",
+    private final Component updateWarnings = Component.translatable("* %s\n" +
+                    "* %s",
             Component.translatable("config_manager.warning.lose_some_config"),
             Component.translatable("config_manager.warning.game_restart")
     );
-    private final Component resetWarnings = Component.translatable("* %s\n* %s",
+    private final Component resetWarnings = Component.translatable("* %s\n" +
+                    "* %s",
             Component.translatable("config_manager.warning.lose_all_config"),
             Component.translatable("config_manager.warning.game_restart")
     );
@@ -35,23 +37,21 @@ public Gui(Screen screen) {
     }
 
     private Button createConfigButton(Component label, Runnable flagSetter) {
-        // Use a local mutable state for each button
         final boolean[] isFirstClick = {true};
         return Button.builder(label, (btn) -> {
                     if (isFirstClick[0]) {
                         btn.setMessage(Component.translatable("config_manager.confirmation").withStyle(style -> style.withColor(COLOR_RED)));
                         isFirstClick[0] = false;
                     } else {
                         try {
-                            btn.active = false;
                             flagSetter.run();
+                            Constants.LOGGER.info("Flag Added, it will be processed on the next game boot");
                             btn.setMessage(Component.translatable("config_manager.success").withStyle(style -> style.withColor(COLOR_WHITE)));
-                            Constants.LOGGER.info("Flag Added");
-                        } catch (Exception e) {
+                        } catch (Throwable e) {
                             Constants.LOGGER.error("Failed to add flag", e);
                             btn.setMessage(Component.translatable("config_manager.error").withStyle(style -> style.withColor(COLOR_RED)));
-                            btn.active = false;
                         }
+                        btn.active = false;
                     }
                 })
                 .size(buttonWidth, buttonHeight)