Use player names mapper data to tab complete and parse profile for bulkedit

Author: benwoo1110

src/main/java/org/mvplugins/multiverse/inventories/command/MVInvCommandCompletion.java

@@ -5,18 +5,23 @@
 import org.mvplugins.multiverse.core.command.MVCommandCompletions;
 import org.mvplugins.multiverse.core.command.MVCommandManager;
 import org.mvplugins.multiverse.core.config.handle.PropertyModifyAction;
+import org.mvplugins.multiverse.core.utils.StringFormatter;
 import org.mvplugins.multiverse.external.acf.commands.BukkitCommandCompletionContext;
 import org.mvplugins.multiverse.external.jakarta.inject.Inject;
 import org.mvplugins.multiverse.external.vavr.control.Try;
 import org.mvplugins.multiverse.inventories.config.InventoriesConfig;
 import org.mvplugins.multiverse.inventories.dataimport.DataImportManager;
+import org.mvplugins.multiverse.inventories.profile.PlayerNamesMapper;
 import org.mvplugins.multiverse.inventories.profile.group.WorldGroup;
 import org.mvplugins.multiverse.inventories.profile.group.WorldGroupManager;
+import org.mvplugins.multiverse.inventories.profile.key.GlobalProfileKey;
 import org.mvplugins.multiverse.inventories.share.Sharables;
 
 import java.util.Arrays;
 import java.util.Collection;
 import java.util.Collections;
+import java.util.List;
+import java.util.Objects;
 import java.util.Set;
 import java.util.stream.Collectors;
 
@@ -28,21 +33,26 @@ public final class MVInvCommandCompletion {
     private final InventoriesConfig inventoriesConfig;
     private final WorldGroupManager worldGroupManager;
     private final DataImportManager dataImportManager;
+    private final PlayerNamesMapper playerNamesMapper;
 
     @Inject
     private MVInvCommandCompletion(
             @NotNull InventoriesConfig inventoriesConfig,
             @NotNull WorldGroupManager worldGroupManager,
             @NotNull DataImportManager dataImportManager,
-            @NotNull MVCommandManager mvCommandManager) {
+            @NotNull MVCommandManager mvCommandManager,
+            @NotNull PlayerNamesMapper playerNamesMapper
+    ) {
         this.inventoriesConfig = inventoriesConfig;
         this.worldGroupManager = worldGroupManager;
         this.dataImportManager = dataImportManager;
+        this.playerNamesMapper = playerNamesMapper;
 
         MVCommandCompletions commandCompletions = mvCommandManager.getCommandCompletions();
         commandCompletions.registerAsyncCompletion("dataimporters", this::suggestDataImporters);
         commandCompletions.registerStaticCompletion("mvinvconfigs", inventoriesConfig.getStringPropertyHandle().getAllPropertyNames());
         commandCompletions.registerAsyncCompletion("mvinvconfigvalues", this::suggestConfigValues);
+        commandCompletions.registerAsyncCompletion("mvinvplayernames", this::suggestPlayerNames);
         commandCompletions.registerAsyncCompletion("sharables", this::suggestSharables);
         commandCompletions.registerAsyncCompletion("shares", this::suggestShares);
         commandCompletions.registerAsyncCompletion("worldGroups", this::suggestWorldGroups);
@@ -60,6 +70,25 @@ private Collection<String> suggestConfigValues(BukkitCommandCompletionContext co
                 .getOrElse(Collections.emptyList());
     }
 
+    private Collection<String> suggestPlayerNames(BukkitCommandCompletionContext context) {
+        if (Objects.equals(context.getInput(), "@all")) {
+            return Collections.emptyList();
+        }
+        List<String> playerNames = getPlayerNames();
+        if (context.getInput().indexOf(',') == -1) {
+            playerNames.add("@all");
+            return playerNames;
+        }
+        return StringFormatter.addonToCommaSeperated(context.getInput(), playerNames);
+    }
+
+    private List<String> getPlayerNames() {
+        return playerNamesMapper.getKeys()
+                .stream()
+                .map(GlobalProfileKey::getPlayerName)
+                .collect(Collectors.toList());
+    }
+
     private Collection<String> suggestSharables(BukkitCommandCompletionContext context) {
         String scope = context.getConfig("scope", "enabled");
 

src/main/java/org/mvplugins/multiverse/inventories/command/MVInvCommandContexts.java

@@ -4,12 +4,14 @@
 import org.bukkit.Bukkit;
 import org.jvnet.hk2.annotations.Service;
 import org.mvplugins.multiverse.core.command.MVCommandManager;
+import org.mvplugins.multiverse.core.utils.REPatterns;
 import org.mvplugins.multiverse.external.acf.commands.BukkitCommandExecutionContext;
 import org.mvplugins.multiverse.external.acf.commands.CommandContexts;
 import org.mvplugins.multiverse.external.acf.commands.InvalidCommandArgument;
 import org.mvplugins.multiverse.external.jakarta.inject.Inject;
 import org.mvplugins.multiverse.external.jetbrains.annotations.NotNull;
 import org.mvplugins.multiverse.external.vavr.control.Option;
+import org.mvplugins.multiverse.inventories.profile.PlayerNamesMapper;
 import org.mvplugins.multiverse.inventories.profile.group.WorldGroup;
 import org.mvplugins.multiverse.inventories.profile.group.WorldGroupManager;
 import org.mvplugins.multiverse.inventories.profile.key.GlobalProfileKey;
@@ -25,10 +27,16 @@
 public final class MVInvCommandContexts {
 
     private final WorldGroupManager worldGroupManager;
+    private final PlayerNamesMapper playerNamesMapper;
 
     @Inject
-    private MVInvCommandContexts(@NotNull MVCommandManager commandManager, @NotNull WorldGroupManager worldGroupManager) {
+    private MVInvCommandContexts(
+            @NotNull MVCommandManager commandManager,
+            @NotNull WorldGroupManager worldGroupManager,
+            @NotNull PlayerNamesMapper playerNamesMapper
+    ) {
         this.worldGroupManager = worldGroupManager;
+        this.playerNamesMapper = playerNamesMapper;
 
         CommandContexts<BukkitCommandExecutionContext> commandContexts = commandManager.getCommandContexts();
         commandContexts.registerContext(GlobalProfileKey[].class, this::parseGlobalProfileKeys);
@@ -40,15 +48,14 @@ private MVInvCommandContexts(@NotNull MVCommandManager commandManager, @NotNull
     private GlobalProfileKey[] parseGlobalProfileKeys(BukkitCommandExecutionContext context) {
         String profileStrings = context.popFirstArg();
         if (profileStrings.equals("@all")) {
-            return Arrays.stream(Bukkit.getOfflinePlayers())
-                    .map(GlobalProfileKey::create)
-                    .toArray(GlobalProfileKey[]::new);
+            return playerNamesMapper.getKeys().toArray(GlobalProfileKey[]::new);
         }
 
-        String[] profileNames = profileStrings.split(",");
+        String[] profileNames = REPatterns.COMMA.split(profileStrings);
         return Arrays.stream(profileNames)
-                .map(Bukkit::getOfflinePlayer)
-                .map(GlobalProfileKey::create)
+                .map(playerNamesMapper::getKey)
+                .filter(Option::isDefined)
+                .map(Option::get)
                 .toArray(GlobalProfileKey[]::new);
     }
 

src/main/java/org/mvplugins/multiverse/inventories/commands/bulkedit/globalprofile/ClearCommand.java

@@ -42,14 +42,15 @@ final class ClearCommand extends InventoriesCommand {
 
     @Subcommand("bulkedit globalprofile clear")
     @CommandPermission("multiverse.inventories.bulkedit")
-    @CommandCompletion("@players @flags:groupName=" + Flags.NAME)
-    @Syntax("<players>")
+    @CommandCompletion("@mvinvplayernames @flags:groupName=" + Flags.NAME)
+    @Syntax("<players> [--clear-all-playerprofiles]")
     void onCommand(
             MVCommandIssuer issuer,
 
             @Syntax("<players>")
             GlobalProfileKey[] globalProfileKeys,
 
+            @Syntax("[--clear-all-playerprofiles]")
             String[] flagArray
     ) {
         ParsedCommandFlags parsedFlags = flags.parse(flagArray);

src/main/java/org/mvplugins/multiverse/inventories/commands/bulkedit/globalprofile/ModifyCommand.java

@@ -6,6 +6,7 @@
 import org.mvplugins.multiverse.core.command.queue.CommandQueuePayload;
 import org.mvplugins.multiverse.core.config.handle.PropertyModifyAction;
 import org.mvplugins.multiverse.core.locale.message.Message;
+import org.mvplugins.multiverse.external.acf.commands.annotation.CommandCompletion;
 import org.mvplugins.multiverse.external.acf.commands.annotation.CommandPermission;
 import org.mvplugins.multiverse.external.acf.commands.annotation.Subcommand;
 import org.mvplugins.multiverse.external.acf.commands.annotation.Syntax;
@@ -32,6 +33,7 @@ final class ModifyCommand extends InventoriesCommand {
 
     @Subcommand("bulkedit globalprofile modify")
     @CommandPermission("multiverse.inventories.bulkedit")
+    @CommandCompletion("load-on-login|last-world @empty @mvinvplayernames")
     @Syntax("<property> <value> <players>")
     void onCommand(
             MVCommandIssuer issuer,