accept ownership change sub-command

This command doesn't appear in the help or tab complete. It's intended
to only be used with `<click:run_command:...>` for better UX.

Author: zefir-git

src/main/java/pro/cloudnode/smp/bankaccounts/Account.java

@@ -13,7 +13,6 @@
 import org.bukkit.inventory.ItemStack;
 import org.bukkit.inventory.meta.ItemMeta;
 import org.bukkit.persistence.PersistentDataType;
-import org.bukkit.scheduler.BukkitTask;
 import org.jetbrains.annotations.NotNull;
 
 import javax.annotation.Nullable;
@@ -32,7 +31,6 @@
 import java.util.Objects;
 import java.util.Optional;
 import java.util.UUID;
-import java.util.function.Supplier;
 import java.util.logging.Level;
 
 /**
@@ -639,19 +637,19 @@ private static void deleteExpired() {
         /**
          * Get account ownership change request
          *
-         * @param account Account
+         * @param account Account ID
          * @param newOwner New owner
          */
-        public static @NotNull Optional<@NotNull ChangeOwnerRequest> get(final @NotNull Account account, @NotNull OfflinePlayer newOwner) {
+        public static @NotNull Optional<@NotNull ChangeOwnerRequest> get(final @NotNull String account, @NotNull OfflinePlayer newOwner) {
             try (final @NotNull Connection conn = BankAccounts.getInstance().getDb().getConnection();
                  final @NotNull PreparedStatement stmt = conn.prepareStatement("SELECT * FROM `change_owner_requests` WHERE `account` = ? AND `new_owner` = ? LIMIT 1")) {
-                stmt.setString(1, account.id);
+                stmt.setString(1, account);
                 stmt.setString(2, newOwner.getUniqueId().toString());
                 final @NotNull ResultSet rs = stmt.executeQuery();
                 return rs.next() ? Optional.of(new ChangeOwnerRequest(rs)) : Optional.empty();
             }
             catch (final @NotNull Exception e) {
-                BankAccounts.getInstance().getLogger().log(Level.SEVERE, "Could not get account ownership change request. account: " + account.id + ", newOwner: " + newOwner.getUniqueId(), e);
+                BankAccounts.getInstance().getLogger().log(Level.SEVERE, "Could not get account ownership change request. account: " + account + ", newOwner: " + newOwner.getUniqueId(), e);
                 return Optional.empty();
             }
         }

src/main/java/pro/cloudnode/smp/bankaccounts/BankConfig.java

@@ -521,6 +521,16 @@ public int interestInterval(final @NotNull Account.Type type) {
         return Objects.requireNonNull(config.getString("messages.errors.already-owns-account"));
     }
 
+    // messages.errors.change-owner-not-found
+    public @NotNull String messagesErrorsChangeOwnerNotFound() {
+        return Objects.requireNonNull(config.getString("messages.errors.change-owner-not-found"));
+    }
+
+    // messages.errors.change-owner-accept-failed
+    public @NotNull String messagesErrorsChangeOwnerAcceptFailed() {
+        return Objects.requireNonNull(config.getString("messages.errors.change-owner-accept-failed"));
+    }
+
     // messages.balance
     public @NotNull String messagesBalance() {
         return Objects.requireNonNull(config.getString("messages.balance"));

src/main/java/pro/cloudnode/smp/bankaccounts/Permissions.java

@@ -17,6 +17,7 @@ public final class Permissions {
     public static @NotNull String CHANGE_OWNER = "bank.change.owner";
     public static @NotNull String CHANGE_OWNER_OTHER = "bank.change.owner.other";
     public static @NotNull String CHANGE_OWNER_SKIP_CONFIRMATION = "bank.change.owner.skip-confirmation";
+    public static @NotNull String CHANGE_OWNER_ACCEPT = "bank.change.owner.accept";
     public static @NotNull String BALTOP = "bank.baltop";
     public static @NotNull String POS_CREATE = "bank.pos.create";
     public static @NotNull String POS_USE = "bank.pos.use";

src/main/java/pro/cloudnode/smp/bankaccounts/commands/BankCommand.java

@@ -180,6 +180,7 @@ public static boolean run(final @NotNull CommandSender sender, final @NotNull St
             case "unfreeze", "enable", "unblock" -> unfreeze(sender, argsSubset, label);
             case "delete" -> delete(sender, argsSubset, label);
             case "changeowner", "newowner", "newholder", "changeholder" -> changeOwner(sender, argsSubset, label);
+            case "acceptchangeowner" -> acceptChangeOwner(sender, argsSubset, label);
             case "transfer", "send", "pay" -> transfer(sender, argsSubset, label);
             case "transactions", "history" -> transactions(sender, argsSubset, label);
             case "instrument", "card" -> instrument(sender, argsSubset, label);
@@ -490,6 +491,33 @@ public static boolean changeOwner(final @NotNull CommandSender sender, final @No
         return true;
     }
 
+    /**
+     * Accept ownership change request
+     */
+    public static boolean acceptChangeOwner(final @NotNull CommandSender sender, final @NotNull String @NotNull [] args, final @NotNull String label) {
+        if (!sender.hasPermission(Permissions.CHANGE_OWNER_ACCEPT)) return sendMessage(sender, BankAccounts.getInstance().config().messagesErrorsNoPermission());
+        if (args.length < 1) return sendUsage(sender, label, "acceptchangeowner <account>");
+        final @NotNull Optional<Account.@NotNull ChangeOwnerRequest> request = Account.ChangeOwnerRequest.get(args[0], BankAccounts.getOfflinePlayer(sender));
+        if (request.isEmpty()) return sendMessage(sender, BankAccounts.getInstance().config().messagesErrorsChangeOwnerNotFound());
+        final @NotNull Optional<@NotNull Account> account = request.get().account();
+        if (account.isEmpty()) return sendMessage(sender, BankAccounts.getInstance().config().messagesErrorsAccountNotFound());
+
+        if (!sender.hasPermission(Permissions.ACCOUNT_CREATE_BYPASS)) {
+            final @NotNull Account @NotNull [] accounts = Account.get(BankAccounts.getOfflinePlayer(sender), account.get().type);
+            int limit = BankAccounts.getInstance().config().accountLimits(account.get().type);
+            if (limit != -1 && accounts.length >= limit)
+                return sendMessage(sender, BankAccounts.getInstance().config().messagesErrorsMaxAccounts(),
+                        Placeholder.unparsed("type", account.get().type.getName()),
+                        Placeholder.unparsed("limit", String.valueOf(limit))
+                );
+        }
+
+        final boolean success = request.get().confirm();
+        if (!success) return sendMessage(sender, BankAccounts.getInstance().config().messagesErrorsChangeOwnerAcceptFailed());
+        // TODO: send success message to new owner
+        return true;
+    }
+
     /**
      * Make a transfer to another account
      * <p>

src/main/resources/config.yml

@@ -353,6 +353,10 @@ messages:
         change-owner-no-history: "<red>(!) You must have a transaction history in order to change the owner.</red>"
         # The account is already owned by <player> (the new owner's username)
         already-owns-account: "<red>(!) This account is already owned by <gray><player></gray>.</red>"
+        # Change owner request not found
+        change-owner-not-found: "<red>(!) This account ownership change request was not found.</red>"
+        # Cannot accept change owner request
+        change-owner-accept-failed: "<red>(!) Failed to accept ownership change request.</red>"
 
     # Account balance
     # Available placeholders: