Fix account list subcommand

KingRainbow44 · Jul 31, 2023 · 665263d · 665263d
1 parent a4ad781
commit 665263d
Showing 1 changed file with 23 additions and 12 deletions.
diff --git a/src/main/java/emu/grasscutter/command/commands/AccountCommand.java b/src/main/java/emu/grasscutter/command/commands/AccountCommand.java
@@ -1,19 +1,18 @@
 package emu.grasscutter.command.commands;
 
-import static emu.grasscutter.utils.lang.Language.translate;
-
 import at.favre.lib.crypto.bcrypt.BCrypt;
 import emu.grasscutter.Grasscutter;
-import emu.grasscutter.command.Command;
-import emu.grasscutter.command.CommandHandler;
+import emu.grasscutter.command.*;
 import emu.grasscutter.config.Configuration;
-import emu.grasscutter.database.DatabaseHelper;
-import emu.grasscutter.database.DatabaseManager;
+import emu.grasscutter.database.*;
 import emu.grasscutter.game.Account;
 import emu.grasscutter.game.player.Player;
+
 import java.util.List;
 import java.util.stream.Collectors;
 
+import static emu.grasscutter.utils.lang.Language.translate;
+
 @Command(
         label = "account",
         usage = {
@@ -31,17 +30,17 @@ public void execute(Player sender, Player targetPlayer, List<String> args) {
             return;
         }
 
-        if (args.size() < 2) {
-            sendUsageMessage(sender);
-            return;
-        }
-
         String action = args.get(0);
-        String username = args.get(1);
 
         switch (action) {
             default -> this.sendUsageMessage(sender);
             case "create" -> {
+                if (args.size() < 2) {
+                    this.sendUsageMessage(sender);
+                    return;
+                }
+                var username = args.get(1);
+
                 int uid = 0;
                 String password = "";
                 if (Configuration.ACCOUNT.EXPERIMENTAL_RealPassword) {
@@ -94,6 +93,12 @@ public void execute(Player sender, Player targetPlayer, List<String> args) {
                 }
             }
             case "delete" -> {
+                if (args.size() < 2) {
+                    this.sendUsageMessage(sender);
+                    return;
+                }
+                var username = args.get(1);
+
                 // Get the account we want to delete.
                 Account toDelete = DatabaseHelper.getAccountByName(username);
                 if (toDelete == null) {
@@ -104,6 +109,12 @@ public void execute(Player sender, Player targetPlayer, List<String> args) {
                 CommandHandler.sendMessage(sender, translate(sender, "commands.account.delete"));
             }
             case "resetpass" -> {
+                if (args.size() < 2) {
+                    this.sendUsageMessage(sender);
+                    return;
+                }
+                var username = args.get(1);
+
                 if (!Configuration.ACCOUNT.EXPERIMENTAL_RealPassword) {
                     CommandHandler.sendMessage(
                             sender, "resetpass requires EXPERIMENTAL_RealPassword to be true.");