fix bug:unexpected exception when forceFakePlayerGameMode

Author: OptiJava

src/main/java/io/github/optijava/opt_carpet_addition/OptCarpetAddition.java

@@ -25,18 +25,35 @@ public class OptCarpetAddition implements CarpetExtension, ModInitializer {
     @Override
     public void onInitialize() {
         CarpetServer.manageExtension(additionInstance);
-        
+
         ServerEntityWorldChangeEvents.AFTER_PLAYER_CHANGE_WORLD.register(new FixExperienceBug());
     }
 
     @Override
     public void onGameStarted() {
         CarpetServer.settingsManager.parseSettingsClass(OptCarpetSettings.class);
         CarpetServer.settingsManager.addRuleObserver(((serverCommandSource, rule, s) -> {
-            if (Objects.equals(rule.name, "forceFakePlayerGameMode")) {
+            if (Objects.equals(rule.name, "forceFakePlayerGameMode") && !Objects.equals(OptCarpetSettings.forceFakePlayerGameMode, "false")) {
+                GameMode gameMode = GameMode.SURVIVAL;
+
+                switch (OptCarpetSettings.forceFakePlayerGameMode) {
+                    case "survival": {
+                        gameMode = GameMode.SURVIVAL;
+                        break;
+                    }
+                    case "creative": {
+                        gameMode = GameMode.CREATIVE;
+                        break;
+                    }
+                    case "adventure": {
+                        gameMode = GameMode.ADVENTURE;
+                        break;
+                    }
+                }
+
                 for (ServerPlayerEntity player : serverCommandSource.getMinecraftServer().getPlayerManager().getPlayerList()) {
                     if (player instanceof EntityPlayerMPFake) {
-                        player.setGameMode(GameMode.valueOf(OptCarpetSettings.forceFakePlayerGameMode));
+                        player.setGameMode(gameMode);
                     }
                 }
             }
@@ -52,7 +69,24 @@ public void registerCommands(CommandDispatcher<ServerCommandSource> dispatcher)
     @Override
     public void onPlayerLoggedIn(ServerPlayerEntity player) {
         if (player instanceof EntityPlayerMPFake && !(Objects.equals(OptCarpetSettings.forceFakePlayerGameMode, "false"))) {
-            player.setGameMode(GameMode.valueOf(OptCarpetSettings.forceFakePlayerGameMode));
+            GameMode gameMode = GameMode.SURVIVAL;
+
+            switch (OptCarpetSettings.forceFakePlayerGameMode) {
+                case "survival": {
+                    gameMode = GameMode.SURVIVAL;
+                    break;
+                }
+                case "creative": {
+                    gameMode = GameMode.CREATIVE;
+                    break;
+                }
+                case "adventure": {
+                    gameMode = GameMode.ADVENTURE;
+                    break;
+                }
+            }
+
+            player.setGameMode(gameMode);
         }
     }
 }

src/main/java/io/github/optijava/opt_carpet_addition/OptCarpetSettings.java

@@ -39,7 +39,7 @@ public class OptCarpetSettings {
     @Rule(
             desc = "Force fake player gamemode",
             category = RuleCategory.FEATURE,
-            options = {"survival", "creative", "adventure", "spectator", "false"}
+            options = {"survival", "creative", "adventure", "false"}
     )
     public static String forceFakePlayerGameMode = "false";
 
@@ -49,5 +49,5 @@ public class OptCarpetSettings {
             category = RuleCategory.BUGFIX
     )
     public static boolean fixXpLevelBug = false;
-    
+
 }