feat(tpa): when using tpa without specifying target, it will take the most recent one

Author: siansxint

Diff for: plugin/src/main/java/team/devblook/pepitocore/plugin/module/tpa/TPARequestManager.java

@@ -5,10 +5,12 @@
 import com.google.inject.Singleton;
 import net.kyori.adventure.text.Component;
 import net.kyori.adventure.text.format.TextColor;
+import org.bukkit.Bukkit;
 import org.bukkit.entity.Player;
 import team.devblook.pepitocore.plugin.module.tpa.model.TPARequest;
 
 import java.util.Collection;
+import java.util.Comparator;
 import java.util.UUID;
 
 @Singleton
@@ -37,6 +39,23 @@ public void send(Player from, Player to) {
         );
     }
 
+    public void accept(Player receiver) {
+        TPARequest last = requests.get(receiver.getUniqueId())
+                .stream()
+                .min(Comparator.comparingLong(TPARequest::elapsed))
+                .orElse(null);
+
+        if (last == null) {
+            receiver.sendMessage("No se ha encontrado ninguna solicitud de TPA pendiente.");
+            return;
+        }
+
+        accept(
+                receiver,
+                Bukkit.getPlayer(last.from())
+        );
+    }
+
     public void accept(Player receiver, Player from) {
         TPARequest request = new TPARequest(from, receiver);
 

Diff for: plugin/src/main/java/team/devblook/pepitocore/plugin/module/tpa/command/TPACommand.java

@@ -4,6 +4,7 @@
 import me.fixeddev.commandflow.annotated.CommandClass;
 import me.fixeddev.commandflow.annotated.annotation.ArgOrSub;
 import me.fixeddev.commandflow.annotated.annotation.Command;
+import me.fixeddev.commandflow.annotated.annotation.OptArg;
 import me.fixeddev.commandflow.bukkit.annotation.Sender;
 import net.kyori.adventure.text.Component;
 import net.kyori.adventure.text.format.TextColor;
@@ -56,17 +57,24 @@ public void cancel(@Sender Player player, OfflinePlayer target) {
     }
 
     @Command(names = "accept")
-    public void accept(@Sender Player player, OfflinePlayer target) {
+    public void accept(@Sender Player player, @OptArg OfflinePlayer target) {
+        if (target == null) {
+            requestManager.accept(player);
+            return;
+        }
+
         if (self(player, target)) {
             player.sendMessage(
                     Component.text("Emm, ¿Si no puedes solicitarte un TPA, por qué crees que esto es lógico?", TextColor.fromHexString("#E7783C"))
             );
             return;
         }
+
         Player from = online(target);
         if (from == null) {
             player.sendMessage(Component.text("El jugador" + target.getName() + " no está conectado.", TextColor.fromHexString("#E7783C")));
             return;
+
         }
 
         requestManager.accept(player, from);