Implement global profile clear as a BulkEditAction

Author: benwoo1110

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

@@ -16,34 +16,36 @@
 import org.mvplugins.multiverse.external.jakarta.inject.Inject;
 import org.mvplugins.multiverse.external.jetbrains.annotations.NotNull;
 import org.mvplugins.multiverse.inventories.commands.InventoriesCommand;
+import org.mvplugins.multiverse.inventories.commands.bulkedit.BulkEditCommand;
 import org.mvplugins.multiverse.inventories.profile.ProfileDataSource;
+import org.mvplugins.multiverse.inventories.profile.bulkedit.BulkEditAction;
+import org.mvplugins.multiverse.inventories.profile.bulkedit.BulkEditCreator;
 import org.mvplugins.multiverse.inventories.profile.key.GlobalProfileKey;
 
 import java.util.Arrays;
 import java.util.concurrent.CompletableFuture;
 
 @Service
-final class ClearCommand extends InventoriesCommand {
+final class ClearCommand extends BulkEditCommand {
 
     private final CommandQueueManager commandQueueManager;
-    private final ProfileDataSource profileDataSource;
     private final Flags flags;
 
     @Inject
     ClearCommand(
+            @NotNull BulkEditCreator bulkEditCreator,
             @NotNull CommandQueueManager commandQueueManager,
-            @NotNull ProfileDataSource profileDataSource,
             @NotNull Flags flags
     ) {
+        super(bulkEditCreator);
         this.commandQueueManager = commandQueueManager;
-        this.profileDataSource = profileDataSource;
         this.flags = flags;
     }
 
     @Subcommand("bulkedit globalprofile clear")
     @CommandPermission("multiverse.inventories.bulkedit")
     @CommandCompletion("@mvinvplayernames @flags:groupName=" + Flags.NAME)
-    @Syntax("<players> [--clear-all-playerprofiles]")
+    @Syntax("<players> [--clear-all-player-profiles]")
     void onCommand(
             MVCommandIssuer issuer,
 
@@ -55,19 +57,16 @@ void onCommand(
     ) {
         ParsedCommandFlags parsedFlags = flags.parse(flagArray);
 
-        commandQueueManager.addToQueue(CommandQueuePayload.issuer(issuer)
-                .prompt(Message.of("Are you sure you want to clear %d profiles?".formatted(globalProfileKeys.length)))
-                .action(() -> doClear(issuer, globalProfileKeys, parsedFlags.hasFlag(flags.clearAllPlayerprofiles))));
-    }
+        BulkEditAction<?> bulkEditAction = bulkEditCreator.globalProfileClear(
+                globalProfileKeys,
+                parsedFlags.hasFlag(flags.clearAllPlayerProfiles)
+        );
 
-    private void doClear(MVCommandIssuer issuer, GlobalProfileKey[] globalProfileKeys, boolean clearPlayerProfile) {
-        //TODO: Check lastWorld and online
-        CompletableFuture[] futures = Arrays.stream(globalProfileKeys)
-                .map(globalProfileKey -> profileDataSource.deleteGlobalProfile(globalProfileKey, clearPlayerProfile))
-                .toArray(CompletableFuture[]::new);
+        outputActionSummary(issuer, bulkEditAction);
 
-        CompletableFuture.allOf(futures)
-                .thenRun(() -> issuer.sendMessage("Successfully cleared %d profiles.".formatted(globalProfileKeys.length)));
+        commandQueueManager.addToQueue(CommandQueuePayload.issuer(issuer)
+                .prompt(Message.of("Are you sure you want to clear the selected global profiles?"))
+                .action(() -> runBulkEditAction(issuer, bulkEditAction)));
     }
 
     @Service
@@ -79,7 +78,7 @@ private Flags(@NotNull CommandFlagsManager flagsManager) {
             super(NAME, flagsManager);
         }
 
-        private final CommandFlag clearAllPlayerprofiles = flag(CommandFlag.builder("--clear-all-playerprofiles")
+        private final CommandFlag clearAllPlayerProfiles = flag(CommandFlag.builder("--clear-all-player-profiles")
                 .addAlias("-a")
                 .build());
     }

src/main/java/org/mvplugins/multiverse/inventories/commands/bulkedit/playerprofile/DeleteCommand.java

@@ -63,9 +63,7 @@ void onCommand(
                 sharable
         );
 
-        issuer.sendMessage("Summary of affected profiles:");
-        bulkEditAction.getActionSummary().forEach((key, value) ->
-                issuer.sendMessage("  %s: %s".formatted(key, value.size() > 10 ? value.size() : StringFormatter.join(value, ", "))));
+        outputActionSummary(issuer, bulkEditAction);
 
         commandQueueManager.addToQueue(CommandQueuePayload.issuer(issuer)
                 .prompt(Message.of("Are you sure you want to delete %s from the selected profiles?".formatted(sharable.getNames()[0])))

src/main/java/org/mvplugins/multiverse/inventories/profile/bulkedit/BulkEditAction.java

@@ -15,7 +15,7 @@
 import java.util.concurrent.CompletableFuture;
 
 @ApiStatus.Experimental
-public sealed abstract class BulkEditAction<K extends GlobalProfileKey> permits PlayerFileAction, PlayerProfileAction {
+public sealed abstract class BulkEditAction<K extends GlobalProfileKey> permits GlobalProfileClearAction, PlayerFileAction, PlayerProfileAction {
 
     protected final MultiverseInventories inventories;
     protected final ProfileDataSource profileDataSource;

src/main/java/org/mvplugins/multiverse/inventories/profile/bulkedit/BulkEditCreator.java

@@ -5,6 +5,7 @@
 import org.jvnet.hk2.annotations.Service;
 import org.mvplugins.multiverse.external.jakarta.inject.Inject;
 import org.mvplugins.multiverse.inventories.MultiverseInventories;
+import org.mvplugins.multiverse.inventories.profile.key.GlobalProfileKey;
 import org.mvplugins.multiverse.inventories.share.Sharable;
 
 @Service
@@ -18,6 +19,10 @@ public final class BulkEditCreator {
         this.inventories = inventories;
     }
 
+    public BulkEditAction<?> globalProfileClear(GlobalProfileKey[] globalProfileKeys, boolean clearPlayerProfiles) {
+        return new GlobalProfileClearAction(inventories, globalProfileKeys, clearPlayerProfiles);
+    }
+
     public BulkEditAction<?> playerProfileClear(PlayerProfilesPayload bulkProfilesPayload) {
         return new PlayerProfileClearAction(inventories, bulkProfilesPayload);
     }

src/main/java/org/mvplugins/multiverse/inventories/profile/bulkedit/GlobalProfileClearAction.java

@@ -0,0 +1,39 @@
+package org.mvplugins.multiverse.inventories.profile.bulkedit;
+
+import org.bukkit.entity.Player;
+import org.mvplugins.multiverse.inventories.MultiverseInventories;
+import org.mvplugins.multiverse.inventories.handleshare.ReadOnlyShareHandler;
+import org.mvplugins.multiverse.inventories.profile.key.GlobalProfileKey;
+
+import java.util.List;
+import java.util.concurrent.CompletableFuture;
+
+final class GlobalProfileClearAction extends BulkEditAction<GlobalProfileKey> {
+
+    private final boolean clearPlayerProfile;
+
+    GlobalProfileClearAction(MultiverseInventories inventories, GlobalProfileKey[] globalProfileKeys, boolean clearPlayerProfiles) {
+        super(inventories, globalProfileKeys);
+        this.clearPlayerProfile = clearPlayerProfiles;
+    }
+
+    @Override
+    protected List<GlobalProfileKey> aggregateKeys() {
+        return List.of(globalProfileKeys);
+    }
+
+    @Override
+    protected CompletableFuture<Void> performAction(GlobalProfileKey key) {
+        return profileDataSource.deleteGlobalProfile(key, clearPlayerProfile);
+    }
+
+    @Override
+    protected boolean isOnlinePlayerAffected(GlobalProfileKey key, Player player) {
+        return super.isOnlinePlayerAffected(key, player) && clearPlayerProfile;
+    }
+
+    @Override
+    protected void updateOnlinePlayerNow(Player player) {
+        new ReadOnlyShareHandler(inventories, player).handleSharing();
+    }
+}