Fix lang issue with region not being replaced, fix landlord/owner not…

… being changed
md5sha256 · Nov 13, 2023 · 6f4524f · 6f4524f
1 parent 67dad42
commit 6f4524f
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/AreaShop/src/main/java/me/wiefferink/areashop/commands/TransferCommand.java b/AreaShop/src/main/java/me/wiefferink/areashop/commands/TransferCommand.java
@@ -4,6 +4,7 @@
 import jakarta.inject.Singleton;
 import me.wiefferink.areashop.MessageBridge;
 import me.wiefferink.areashop.managers.IFileManager;
+import me.wiefferink.areashop.regions.BuyRegion;
 import me.wiefferink.areashop.regions.GeneralRegion;
 import me.wiefferink.areashop.regions.RentRegion;
 import me.wiefferink.areashop.tools.Utils;
@@ -95,10 +96,11 @@ private void handlePlayer(Player player, String[] args) {
             this.messageBridge.message(player, "transfer-noPlayer", targetPlayerName);
             return;
         }
-        if (player.getUniqueId().equals(region.getLandlord())) {
+        if (region.isLandlord(targetPlayer.getUniqueId())) {
             // Transfer ownership if same as landlord
             region.setOwner(targetPlayer.getUniqueId());
-            this.messageBridge.message(player, "transfer-transferred-owner", targetPlayerName);
+            region.setLandlord(targetPlayer.getUniqueId(), targetPlayerName);
+            this.messageBridge.message(player, "transfer-transferred-owner", targetPlayerName, region);
             if (targetPlayer.isOnline()) {
                 this.messageBridge.message(targetPlayer.getPlayer(), "transfer-transferred-owner", targetPlayerName);
             }
@@ -134,11 +136,11 @@ public List<String> getTabCompleteList(int toComplete, String[] start, CommandSe
                 return this.fileManager.getRegionNames();
             }
             UUID uuid = player.getUniqueId();
-            return this.fileManager.getRegions()
+            return new ArrayList<>(this.fileManager.getRegions()
                     .stream()
                     .filter(region -> region.isOwner(uuid) || region.isLandlord(uuid))
                     .map(GeneralRegion::getName)
-                    .toList();
+                    .toList());
         }
         return Collections.emptyList();
     }