Fixed many issues with teamManager

src/main/java/xyz/twovb/sgm/commands/impl/JailCommand.java

@@ -8,12 +8,9 @@ import dev.rollczi.litecommands.annotations.command.Command;
 import dev.rollczi.litecommands.annotations.context.Context;
 import dev.rollczi.litecommands.annotations.execute.Execute;
 import dev.rollczi.litecommands.annotations.permission.Permission;
-import org.bukkit.command.CommandSender;
 import org.bukkit.entity.Player;
 import xyz.twovb.sgm.SGM;
 import xyz.twovb.sgm.games.impl.capturethebrick.CTB;
-import xyz.twovb.toolbox.managers.PlaceholderManager;
-import xyz.twovb.toolbox.utils.ChatUtils;
 
 @Command(name = "jail")
 @Permission("twovb.debug")

src/main/java/xyz/twovb/sgm/games/GameManager.java

@@ -69,7 +69,6 @@ public class GameManager {
             }
             game.addPlayer(player);
         } else {
-
             SGM.getInstance().getCLogger().log(game.getState().toString());
         }
     }

src/main/java/xyz/twovb/sgm/games/impl/capturethebrick/CTB.java

@@ -118,8 +118,6 @@ public class CTB implements Minigame {
                         return Component.text(time, NamedTextColor.GRAY);
                     })
                     .addBlankLine()
-//                    .addDynamicLine(() -> ChatUtils.translate("<red><bold>Red: " + getBrickCount(redTeam)))
-//                    .addDynamicLine(() -> ChatUtils.translate("<blue><bold>Blue: " + getBrickCount(blueTeam)))
                     .addDynamicLine(() -> ChatUtils.translate("<red>Red: <bold>" + getBrickCount(redTeam)))
                     .addDynamicLine(() -> ChatUtils.translate("<blue>Blue: <bold>" + getBrickCount(blueTeam)))
                     .addBlankLine()
@@ -169,23 +167,6 @@ public class CTB implements Minigame {
         for (Player player : players) {
             CtbData.Territory territory = getTerritory(player);
             player.sendActionBar(ChatUtils.translate(territory.getFancyText()));
-//            Location playerLoc = player.getLocation();
-//            Location blockCheck = new Location(gameWorld, playerLoc.getBlockX(), territoryLevel, playerLoc.getBlockZ());
-//            Block territoryCheck = gameWorld.getBlockAt(blockCheck);
-//            switch (territoryCheck.getType()) {
-//                case RED_CONCRETE:
-//                    player.sendActionBar(ChatUtils.translate("<red><bold>ʀᴇᴅ ᴛᴇʀʀɪᴛᴏʀʏ"));
-//                    break;
-//                case BLUE_CONCRETE:
-//                    player.sendActionBar(ChatUtils.translate("<blue><bold>ʙʟᴜᴇ ᴛᴇʀʀɪᴛᴏʀʏ"));
-//                    break;
-//                case WHITE_CONCRETE:
-//                    player.sendActionBar(ChatUtils.translate("<gray><bold>ɴᴇᴜᴛʀᴀʟ"));
-//                    break;
-//                default:
-//                    player.sendActionBar(ChatUtils.translate("<dark_gray><bold>???"));
-//                    break;
-//            }
         }
     }
 
@@ -200,7 +181,6 @@ public class CTB implements Minigame {
                 sb.addPlayer(player);
                 playerStateMap.put(player, CtbData.PlayerState.Free);
             }
-            // TODO: Fix incorrect amount of brick spawns.
             for (int i = 0; i <= this.bricks - 1; i++) {
                 placeBrick(redBrickSpawns.get(i), redTeam);
                 placeBrick(blueBrickSpawns.get(i), blueTeam);
@@ -393,7 +373,9 @@ public class CTB implements Minigame {
     }
 
     public void addPlayerToTeam(Player player, CtbData.Teams team) {
-        // TODO: VALIDATE IF ON TEAM ALREADY
+        if (teamManager.players().contains(player)) {
+            getTeam(player).defaultDisplay().removeEntry(player.getName());
+        }
         if (team.equals(CtbData.Teams.Blue)) {
             blueTeam.defaultDisplay().addEntry(player.getName());
         } else if (team.equals(CtbData.Teams.Red)) {
@@ -403,8 +385,7 @@ public class CTB implements Minigame {
     }
 
     public void removePlayerFromTeams(Player player) {
-        redTeam.defaultDisplay().removeEntry(player.getName());
+        getTeam(player).defaultDisplay().removeEntry(player.getName());
-        blueTeam.defaultDisplay().removeEntry(player.getName());
         this.teamManager.removePlayer(player);
     }
 

src/main/resources/games/capturethebrick/messages.yml

@@ -7,4 +7,4 @@ system:
     left: "&7%player% has left the game."
     jail:
       notify: "&7%player% has been jailed!"
-      info: "&7You have been jailed! You can not leave until you get freed by a player or Jailbreak."
+      info: "&7You have been jailed! You can not leave until you get freed by a player or Jailbreak™."