Add player trade command (/wp trade <playerName>)

src/main/java/dansplugins/wildpets/WildPets.java

@@ -160,7 +160,7 @@ private void initializeCommandService() {
                 new InfoCommand(ephemeralData, configService, petRecordRepository), new ListCommand(petListRepository), new LocateCommand(ephemeralData),
                 new LockCommand(ephemeralData), new RenameCommand(ephemeralData, petListRepository, petRecordRepository, configService),
                 new SelectCommand(configService, ephemeralData, petListRepository), new SetFreeCommand(ephemeralData, petListRepository), new StatsCommand(petListRepository),
-                new TameCommand(ephemeralData), new UnlockCommand(ephemeralData), new WanderCommand(ephemeralData),
+                new TameCommand(ephemeralData), new TradeCommand(ephemeralData, petListRepository, petRecordRepository), new UnlockCommand(ephemeralData), new WanderCommand(ephemeralData),
                 new StayCommand(ephemeralData), new GatherCommand(petListRepository)
         ));
         commandService.initialize(commands, "That command wasn't found.");

src/main/java/dansplugins/wildpets/commands/HelpCommand.java

@@ -46,6 +46,7 @@ public boolean execute(CommandSender sender) {
         player.sendMessage(ChatColor.AQUA + "/wp lock - Lock your pet.");
         player.sendMessage(ChatColor.AQUA + "/wp unlock - Unlock your pet.");
         player.sendMessage(ChatColor.AQUA + "/wp setfree - Set your pet free.");
+        player.sendMessage(ChatColor.AQUA + "/wp trade (playerName) - Trade selected pet to another player.");
         player.sendMessage(ChatColor.AQUA + "/wp gather - Gather your pets in one place.");
         player.sendMessage(ChatColor.AQUA + "/wp config - View or set config options.");
         return true;

src/main/java/dansplugins/wildpets/commands/TradeCommand.java

@@ -0,0 +1,93 @@
+package dansplugins.wildpets.commands;
+
+import dansplugins.wildpets.data.EphemeralData;
+import dansplugins.wildpets.exceptions.PetRecordNotFoundException;
+import dansplugins.wildpets.pet.Pet;
+import dansplugins.wildpets.pet.list.PetListRepository;
+import dansplugins.wildpets.pet.record.PetRecord;
+import dansplugins.wildpets.pet.record.PetRecordRepository;
+
+import org.bukkit.Bukkit;
+import org.bukkit.ChatColor;
+import org.bukkit.command.CommandSender;
+import org.bukkit.entity.Player;
+
+import java.util.ArrayList;
+import java.util.Arrays;
+
+/**
+ * @author Daniel McCoy Stephenson
+ * Command to transfer a selected pet to another online player.
+ */
+public class TradeCommand extends AbstractPluginCommand {
+    private final EphemeralData ephemeralData;
+    private final PetListRepository petListRepository;
+    private final PetRecordRepository petRecordRepository;
+
+    public TradeCommand(EphemeralData ephemeralData, PetListRepository petListRepository, PetRecordRepository petRecordRepository) {
+        super(new ArrayList<>(Arrays.asList("trade")), new ArrayList<>(Arrays.asList("wp.trade")));
+        this.ephemeralData = ephemeralData;
+        this.petListRepository = petListRepository;
+        this.petRecordRepository = petRecordRepository;
+    }
+
+    @Override
+    public boolean execute(CommandSender sender) {
+        sender.sendMessage(ChatColor.RED + "Usage: /wp trade (playerName)");
+        return false;
+    }
+
+    @Override
+    public boolean execute(CommandSender sender, String[] args) {
+        if (!(sender instanceof Player)) {
+            return false;
+        }
+
+        Player player = (Player) sender;
+
+        Pet pet = ephemeralData.getPetSelectionForPlayer(player.getUniqueId());
+        if (pet == null) {
+            player.sendMessage(ChatColor.RED + "No pet selected. Use /wp select first.");
+            return false;
+        }
+
+        if (args.length < 1) {
+            return execute(sender);
+        }
+
+        String targetName = args[0];
+        Player targetPlayer = Bukkit.getPlayer(targetName);
+
+        if (targetPlayer == null) {
+            player.sendMessage(ChatColor.RED + "That player is not online.");
+            return false;
+        }
+
+        if (targetPlayer.getUniqueId().equals(player.getUniqueId())) {
+            player.sendMessage(ChatColor.RED + "You cannot trade a pet to yourself.");
+            return false;
+        }
+
+        String petName = pet.getName();
+
+        boolean success = petListRepository.transferPet(pet, targetPlayer.getUniqueId());
+        if (!success) {
+            player.sendMessage(ChatColor.RED + "Failed to transfer pet.");
+            return false;
+        }
+
+        // Update pet record
+        try {
+            PetRecord petRecord = petRecordRepository.getPetRecord(pet.getUniqueID());
+            petRecord.setOwnerUUID(targetPlayer.getUniqueId());
+        } catch (PetRecordNotFoundException e) {
+            petRecordRepository.addPetRecord(pet);
+        }
+
+        ephemeralData.clearPetSelectionForPlayer(player.getUniqueId());
+
+        player.sendMessage(ChatColor.GREEN + petName + " has been traded to " + targetPlayer.getName() + ".");
+        targetPlayer.sendMessage(ChatColor.GREEN + player.getName() + " has traded " + petName + " to you.");
+        return true;
+    }
+}

src/main/java/dansplugins/wildpets/pet/list/PetListRepository.java

@@ -50,6 +50,25 @@ public boolean removePet(Pet petToRemove) {
         return getPetList(petToRemove.getOwnerUUID()).removePet(petToRemove);
     }
 
+    public boolean transferPet(Pet pet, UUID newOwnerUUID) {
+        PetList oldList = getPetList(pet.getOwnerUUID());
+        if (oldList == null || !oldList.getPets().remove(pet)) {
+            return false;
+        }
+
+        PetList newList = getPetList(newOwnerUUID);
+        if (newList == null) {
+            createPetListForPlayer(newOwnerUUID);
+            newList = getPetList(newOwnerUUID);
+        }
+
+        pet.setOwnerUUID(newOwnerUUID);
+        pet.removeFromAccessList(oldList.getOwnerUUID());
+        pet.addToAccessList(newOwnerUUID);
+        newList.addPet(pet);
+        return true;
+    }
+
     public Pet getPet(Entity entity) {
         for (PetList petList : getPetLists()) {
             Pet pet = petList.getPet(entity.getUniqueId());

src/main/resources/plugin.yml

@@ -44,5 +44,7 @@ permissions:
     default: true
   wp.gather:
     default: true
+  wp.trade:
+    default: true
   wp.config:
     default: op