[Transfer] Check limit when transferring shop and bump version

Author: sandtechnology

pom.xml

@@ -25,7 +25,7 @@
     <artifactId>QuickShop</artifactId>
 
     <properties>
-        <pluginver>5.1.2.1</pluginver>
+        <pluginver>5.1.2.2</pluginver>
         <package>org.maxgamer.quickshop</package>
         <developer>Ghost-chu</developer>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>

src/main/java/org/maxgamer/quickshop/command/subcommand/SubCommand_Transfer.java

@@ -19,6 +19,7 @@
 
 package org.maxgamer.quickshop.command.subcommand;
 
+import org.bukkit.command.CommandSender;
 import org.bukkit.entity.Player;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
@@ -55,6 +56,16 @@ public void onCommand(@NotNull Player sender, @NotNull String commandLabel, @Not
             }
             final UUID targetPlayerUUID = targetPlayer.getUuid();
             List<Shop> shopList = plugin.getShopManager().getPlayerAllShops(sender.getUniqueId());
+            if (plugin.isLimit()) {
+                final Player player = plugin.getServer().getPlayer(targetPlayerUUID);
+                if (player == null) {
+                    plugin.text().of(sender, "unknown-player").send();
+                    return;
+                }
+                if (!checkAndSendLimitMessage(player, sender, shopList.size())) {
+                    return;
+                }
+            }
             for (Shop shop : shopList) {
                 if (!shop.isBuying()) {
                     shop.setOwner(targetPlayerUUID);
@@ -88,6 +99,16 @@ public void onCommand(@NotNull Player sender, @NotNull String commandLabel, @Not
             }
             final UUID targetPlayerUUID = targetPlayer.getUuid();
             List<Shop> shopList = plugin.getShopManager().getPlayerAllShops(fromPlayer.getUuid());
+            if (plugin.isLimit()) {
+                final Player player = plugin.getServer().getPlayer(targetPlayerUUID);
+                if (player == null) {
+                    plugin.text().of(sender, "unknown-player").send();
+                    return;
+                }
+                if (!checkAndSendLimitMessage(player, sender, shopList.size())) {
+                    return;
+                }
+            }
             for (Shop shop : shopList) {
                 shop.setOwner(targetPlayerUUID);
             }
@@ -98,6 +119,27 @@ public void onCommand(@NotNull Player sender, @NotNull String commandLabel, @Not
         }
     }
 
+    private boolean checkAndSendLimitMessage(Player checkingPlayer, CommandSender commandSender, int increment) {
+        if (plugin.isLimit()) {
+            int owned = 0;
+            if (plugin.getConfig().getBoolean("limits.old-algorithm")) {
+                owned = plugin.getShopManager().getPlayerAllShops(checkingPlayer.getUniqueId()).size();
+            } else {
+                for (final Shop shop : plugin.getShopManager().getPlayerAllShops(checkingPlayer.getUniqueId())) {
+                    if (!shop.isUnlimited()) {
+                        owned++;
+                    }
+                }
+            }
+            int max = plugin.getShopLimit(checkingPlayer);
+            if (owned + increment <= max) {
+                plugin.text().of(commandSender, "reached-maximum-other-can-hold", String.valueOf(owned), String.valueOf(max)).send();
+                return false;
+            }
+        }
+        return true;
+    }
+
     @Override
     public @Nullable List<String> onTabComplete(@NotNull Player sender, @NotNull String commandLabel, @NotNull String[] cmdArg) {
         return cmdArg.length <= 2 ? Util.getPlayerList() : Collections.emptyList();

src/main/resources/lang/messages.json

@@ -70,6 +70,7 @@
   "unknown-owner": "Unknown",
   "owner-bypass-check": "&eBypassed all checks. Trade successful! (You are now the shop owner!)",
   "reached-maximum-can-create": "&cYou already created a maximum of {0}/{1} shops!",
+  "reached-maximum-other-can-hold": "&cTarget player has reached the maximum of {0}/{1} shops!",
   "restricted-prices": "&cRestricted price for {0}: Min {1}, max {2}",
   "no-enough-money-to-keep-shops": "&cYou didn't have enough money to keep your shops! All shops have been removed...",
   "nothing-to-flush": "&aYou have no new shop messages.",