Added more debug messages to replacing players (#1513)

BG-Software-LLC · Dec 23, 2022 · b56a787 · b56a787
1 parent abff691
commit b56a787
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/main/java/com/bgsoftware/superiorskyblock/island/SIsland.java b/src/main/java/com/bgsoftware/superiorskyblock/island/SIsland.java
@@ -1425,7 +1425,7 @@ public void replacePlayers(SuperiorPlayer originalPlayer, SuperiorPlayer newPlay
         Preconditions.checkNotNull(originalPlayer, "originalPlayer parameter cannot be null.");
         Preconditions.checkNotNull(newPlayer, "newPlayer parameter cannot be null.");
 
-        Log.debug(Debug.REPLACE_PLAYER, owner.getName(), originalPlayer.getName(), newPlayer.getName());
+        Log.debug(Debug.REPLACE_PLAYER, owner, originalPlayer, newPlayer);
 
         if (owner.equals(originalPlayer)) {
             Log.debugResult(Debug.REPLACE_PLAYER, "Action", "Replace Owner");

diff --git a/src/main/java/com/bgsoftware/superiorskyblock/player/PlayersManagerImpl.java b/src/main/java/com/bgsoftware/superiorskyblock/player/PlayersManagerImpl.java
@@ -136,14 +136,17 @@ public List<SuperiorPlayer> matchAllPlayers(Predicate<? super SuperiorPlayer> pr
     }
 
     public void replacePlayers(SuperiorPlayer originPlayer, SuperiorPlayer newPlayer) {
-        Log.debug(Debug.REPLACE_PLAYER, originPlayer.getName(), newPlayer.getName());
+        Log.debug(Debug.REPLACE_PLAYER, originPlayer, newPlayer);
 
         newPlayer.merge(originPlayer);
 
         // We first want to replace the player for his own island
         Island playerIsland = originPlayer.getIsland();
-        if (playerIsland != null)
+        if (playerIsland != null) {
             playerIsland.replacePlayers(originPlayer, newPlayer);
+            if (playerIsland.getOwner() != newPlayer)
+                Log.debugResult(Debug.REPLACE_PLAYER, "Owner was not replaced", "Curr owner: " + playerIsland.getOwner().getUniqueId());
+        }
 
         for (Island island : plugin.getGrid().getIslands()) {
             if (island != playerIsland)