Merge pull request #3306 from Multiverse/chore/update-game-version

Update supported game version for 1.21.7 and 1.21.8

.github/workflows/generic.platform_uploads.yml

@@ -107,7 +107,7 @@ jobs:
         files: '["${{ github.workspace }}/${{ inputs.plugin_name }}-${{ steps.release-info.outputs.tag_name }}.jar"]'
         name: ${{ steps.release-info.outputs.tag_name }}
         changelog: ${{ steps.release-artifact.outputs.body }}
-        game_versions: 1.21.6, 1.21.5, 1.21.4, 1.21.3, 1.21.2, 1.21.1, 1.21, 1.20.6, 1.20.5, 1.20.4, 1.20.3, 1.20.2, 1.20.1, 1.20, 1.19.4, 1.19.3, 1.19.2, 1.19.1, 1.19, 1.18.2
+        game_versions: 1.21.8, 1.21.7, 1.21.6, 1.21.5, 1.21.4, 1.21.3, 1.21.2, 1.21.1, 1.21, 1.20.6, 1.20.5, 1.20.4, 1.20.3, 1.20.2, 1.20.1, 1.20, 1.19.4, 1.19.3, 1.19.2, 1.19.1, 1.19, 1.18.2
         version_type: ${{ steps.parse-release-type.outputs.release_type }}
         loaders: bukkit, spigot, paper
         dependencies: ${{ inputs.modrinth_dependencies }}
@@ -121,7 +121,7 @@ jobs:
         changelog: ${{ steps.release-artifact.outputs.body }}
         changelog_type: markdown
         display_name: ${{ steps.release-info.outputs.tag_name }}
-        game_versions: 1.21.6, 1.21.5, 1.21.4, 1.21.3, 1.21.2, 1.21.1, 1.21, 1.20.6, 1.20.5, 1.20.4, 1.20.3, 1.20.2, 1.20.1, 1.20, 1.19.4, 1.19.3, 1.19.2, 1.19.1, 1.19, 1.18.2
+        game_versions: 1.21.8, 1.21.7, 1.21.6, 1.21.5, 1.21.4, 1.21.3, 1.21.2, 1.21.1, 1.21, 1.20.6, 1.20.5, 1.20.4, 1.20.3, 1.20.2, 1.20.1, 1.20, 1.19.4, 1.19.3, 1.19.2, 1.19.1, 1.19, 1.18.2
         release_type: ${{ steps.parse-release-type.outputs.release_type }}
         project_relations: ${{ inputs.dbo_project_relations }}
         file_path: ${{ github.workspace }}/${{ inputs.plugin_name }}-${{ steps.release-info.outputs.tag_name }}.jar
@@ -136,5 +136,5 @@ jobs:
         channel: ${{ steps.parse-release-type.outputs.release_type }}
         files: '[{"path": "${{ github.workspace }}/${{ inputs.plugin_name }}-${{ steps.release-info.outputs.tag_name }}.jar", "platforms": ["PAPER"]}]'
         description: ${{ steps.release-artifact.outputs.body }}
-        platform_dependencies: '{"PAPER": ["1.18.2-1.21.6"]}'
+        platform_dependencies: '{"PAPER": ["1.18.2-1.21.8"]}'
         plugin_dependencies: ${{ inputs.hangar_plugin_dependencies }}

src/main/java/org/mvplugins/multiverse/core/commands/DeleteCommand.java

@@ -57,7 +57,7 @@ class DeleteCommand extends CoreCommand {
 
     @Subcommand("delete")
     @CommandPermission("multiverse.core.delete")
-    @CommandCompletion("@mvworlds:scope=loaded @flags:groupName=" + RemovePlayerFlags.NAME)
+    @CommandCompletion("@mvworlds:scope=both @flags:groupName=" + RemovePlayerFlags.NAME)
     @Syntax("<world>")
     @Description("{@@mv-core.delete.description}")
     void onDeleteCommand(

src/main/java/org/mvplugins/multiverse/core/commands/GameruleCommand.java

@@ -42,7 +42,6 @@ import org.mvplugins.multiverse.core.world.LoadedMultiverseWorld;
 import static org.mvplugins.multiverse.core.locale.message.MessageReplacement.replace;
 
 @Service
-@Subcommand("gamerule|rule")
 class GameruleCommand extends CoreCommand {
 
     private final PageFilterFlags flags;
@@ -52,7 +51,7 @@ class GameruleCommand extends CoreCommand {
         this.flags = flags;
     }
 
-    @Subcommand("set")
+    @Subcommand("gamerule|rule set")
     @CommandPermission("multiverse.core.gamerule.set")
     @CommandCompletion("@gamerules @gamerulesvalues @mvworlds:multiple|*")
     @Syntax("<Gamerule> <Gamerule value> [World or *]")
@@ -103,7 +102,7 @@ class GameruleCommand extends CoreCommand {
     }
 
     @SuppressWarnings("rawtypes,unchecked")
-    @Subcommand("reset")
+    @Subcommand("gamerule|rule reset")
     @CommandPermission("multiverse.core.gamerule.set")
     @CommandCompletion("@gamerules @mvworlds:multiple|*")
     @Syntax("<Gamerule> [World or *]")
@@ -145,9 +144,9 @@ class GameruleCommand extends CoreCommand {
         }
     }
 
-    @Subcommand("list")
+    @Subcommand("gamerule|rule list")
     @CommandPermission("multiverse.core.gamerule.list")
-    @CommandCompletion("@mvworlds|@flags:groupName=mvgamerulecommand @flags:groupName=" + PageFilterFlags.NAME)
+    @CommandCompletion("@mvworlds|@flags:groupName=" + PageFilterFlags.NAME + " @flags:groupName=" + PageFilterFlags.NAME)
     @Syntax("[world] [--page <page>] [--filter <filter>]")
     @Description("{@@mv-core.gamerule.list.description}")
     void onGameruleListCommand(
@@ -220,6 +219,11 @@ class GameruleCommand extends CoreCommand {
             super.onGameruleSetCommand(issuer, gamerule, gameRuleValue, worlds);
         }
 
+        @Override
+        void onGameruleResetCommand(MVCommandIssuer issuer, GameRule gamerule, LoadedMultiverseWorld[] worlds) {
+            super.onGameruleResetCommand(issuer, gamerule, worlds);
+        }
+
         @Override
         @CommandAlias("mvrules|mvgamerules")
         void onGameruleListCommand(MVCommandIssuer issuer, LoadedMultiverseWorld world, String[] flags) {

src/main/java/org/mvplugins/multiverse/core/commands/ModifyCommand.java

@@ -23,12 +23,19 @@ import org.mvplugins.multiverse.core.locale.MVCorei18n;
 import org.mvplugins.multiverse.core.world.MultiverseWorld;
 import org.mvplugins.multiverse.core.world.WorldManager;
 
+import java.util.Map;
+
 import static org.mvplugins.multiverse.core.locale.message.MessageReplacement.*;
 import static org.mvplugins.multiverse.core.locale.message.MessageReplacement.replace;
 
 @Service
 class ModifyCommand extends CoreCommand {
 
+    private static final Map<String, String> REMOVED_PROPERTIES = Map.of(
+            "animals", "/mv entity-spawn-config modify [world] animal set spawn <true|false>",
+            "monsters", "/mv entity-spawn-config modify [world] monster set spawn <true|false>"
+        );
+
     private final WorldManager worldManager;
 
     @Inject
@@ -65,6 +72,13 @@ class ModifyCommand extends CoreCommand {
             @Syntax("[value]")
             @Description("{@@mv-core.modify.value.description}")
             @Nullable String propertyValue) {
+        if (REMOVED_PROPERTIES.containsKey(propertyName)) {
+            issuer.sendMessage(MVCorei18n.MODIFY_PROPERTYREMOVED,
+                    replace("{property}").with(propertyName),
+                    replace("{replacement}").with(REMOVED_PROPERTIES.get(propertyName)));
+            return;
+        }
+
         MultiverseWorld world = worldValue.value();
 
         if (action.isRequireValue() && propertyValue == null) {

src/main/java/org/mvplugins/multiverse/core/listeners/MVEntityListener.java

@@ -92,7 +92,8 @@ final class MVEntityListener implements CoreListener {
         // Always allow custom command and plugins to spawn creatures
         if (event.getSpawnReason() == SpawnReason.CUSTOM
                 || event.getSpawnReason() == SpawnReason.COMMAND
-                || event.getSpawnReason() == SpawnReason.BREEDING) {
+                || event.getSpawnReason() == SpawnReason.BREEDING
+                || event.getSpawnReason() == SpawnReason.SPAWNER_EGG) {
             return;
         }
 

src/main/java/org/mvplugins/multiverse/core/listeners/MVPlayerListener.java

@@ -33,7 +33,9 @@ import org.mvplugins.multiverse.core.MultiverseCore;
 import org.mvplugins.multiverse.core.command.MVCommandManager;
 import org.mvplugins.multiverse.core.config.CoreConfig;
 import org.mvplugins.multiverse.core.destination.DestinationsProvider;
+import org.mvplugins.multiverse.core.dynamiclistener.EventRunnable;
 import org.mvplugins.multiverse.core.dynamiclistener.annotations.DefaultEventPriority;
+import org.mvplugins.multiverse.core.dynamiclistener.annotations.EventClass;
 import org.mvplugins.multiverse.core.dynamiclistener.annotations.EventMethod;
 import org.mvplugins.multiverse.core.dynamiclistener.annotations.EventPriorityKey;
 import org.mvplugins.multiverse.core.economy.MVEconomist;
@@ -195,59 +197,64 @@ final class MVPlayerListener implements CoreListener {
         return Option.of(mvWorld.getSpawnLocation());
     }
 
-    @EventMethod
+    @EventClass("org.spigotmc.event.player.PlayerSpawnLocationEvent")
     @EventPriorityKey("mvcore-player-spawn-location")
-    void playerSpawnLocation(PlayerSpawnLocationEvent event) {
-        Player player = event.getPlayer();
-        MultiverseWorld world = getWorldManager().getLoadedWorld(player.getWorld()).getOrNull();
-        if (world == null) {
-            Logging.finer("Player joined in a world that is not managed by Multiverse.");
-            return;
-        }
-        if (!player.hasPlayedBefore()) {
-            handleFirstSpawn(event);
-        } else {
-            handleJoinLocation(event);
-        }
-        this.handleGameModeAndFlight(player, event.getSpawnLocation().getWorld());
-    }
+    EventRunnable playerSpawnLocation() {
+        return new EventRunnable<PlayerSpawnLocationEvent>() {
+            @Override
+            public void onEvent(PlayerSpawnLocationEvent event) {
+                Player player = event.getPlayer();
+                MultiverseWorld world = getWorldManager().getLoadedWorld(player.getWorld()).getOrNull();
+                if (world == null) {
+                    Logging.finer("Player joined in a world that is not managed by Multiverse.");
+                    return;
+                }
+                if (!player.hasPlayedBefore()) {
+                    handleFirstSpawn(event);
+                } else {
+                    handleJoinLocation(event);
+                }
+                handleGameModeAndFlight(player, event.getSpawnLocation().getWorld());
+            }
 
-    private void handleFirstSpawn(PlayerSpawnLocationEvent event) {
-        if (!config.getFirstSpawnOverride()) {
-            Logging.finer("FirstSpawnOverride is disabled");
-            // User has disabled the feature in config
-            return;
-        }
-        Logging.fine("Moving NEW player to(firstspawnoverride): %s", config.getFirstSpawnLocation());
-        destinationsProvider.parseDestination(config.getFirstSpawnLocation())
-                .map(destination -> destination.getLocation(event.getPlayer())
-                        .peek(event::setSpawnLocation)
-                        .onEmpty(() -> Logging.warning("The destination in FirstSpawnLocation in config is invalid")))
-                .onFailure(failure -> {
-                    Logging.warning("Invalid destination in FirstSpawnLocation in config: %s");
-                    Logging.warning(failure.getFailureMessage().formatted(getLocales()));
-                });
-    }
+            private void handleFirstSpawn(PlayerSpawnLocationEvent event) {
+                if (!config.getFirstSpawnOverride()) {
+                    Logging.finer("FirstSpawnOverride is disabled");
+                    // User has disabled the feature in config
+                    return;
+                }
+                Logging.fine("Moving NEW player to(firstspawnoverride): %s", config.getFirstSpawnLocation());
+                destinationsProvider.parseDestination(config.getFirstSpawnLocation())
+                        .map(destination -> destination.getLocation(event.getPlayer())
+                                .peek(event::setSpawnLocation)
+                                .onEmpty(() -> Logging.warning("The destination in FirstSpawnLocation in config is invalid")))
+                        .onFailure(failure -> {
+                            Logging.warning("Invalid destination in FirstSpawnLocation in config: %s");
+                            Logging.warning(failure.getFailureMessage().formatted(getLocales()));
+                        });
+            }
 
-    private void handleJoinLocation(PlayerSpawnLocationEvent event) {
-        if (!config.getEnableJoinDestination()) {
-            Logging.finer("JoinDestination is disabled");
-            // User has disabled the feature in config
-            return;
-        }
-        if (config.getJoinDestination().isBlank()) {
-            Logging.warning("Joindestination is enabled but no destination has been specified in config!");
-            return;
-        }
-        Logging.finer("JoinDestination is " + config.getJoinDestination());
-        destinationsProvider.parseDestination(config.getJoinDestination())
-                .map(destination -> destination.getLocation(event.getPlayer())
-                    .peek(event::setSpawnLocation)
-                    .onEmpty(() -> Logging.warning("The destination in JoinDestination in config is invalid")))
-                .onFailure(failure -> {
-                    Logging.warning("Invalid destination in JoinDestination in config: %s");
-                    Logging.warning(failure.getFailureMessage().formatted(getLocales()));
-                });
+            private void handleJoinLocation(PlayerSpawnLocationEvent event) {
+                if (!config.getEnableJoinDestination()) {
+                    Logging.finer("JoinDestination is disabled");
+                    // User has disabled the feature in config
+                    return;
+                }
+                if (config.getJoinDestination().isBlank()) {
+                    Logging.warning("Joindestination is enabled but no destination has been specified in config!");
+                    return;
+                }
+                Logging.finer("JoinDestination is " + config.getJoinDestination());
+                destinationsProvider.parseDestination(config.getJoinDestination())
+                        .map(destination -> destination.getLocation(event.getPlayer())
+                                .peek(event::setSpawnLocation)
+                                .onEmpty(() -> Logging.warning("The destination in JoinDestination in config is invalid")))
+                        .onFailure(failure -> {
+                            Logging.warning("Invalid destination in JoinDestination in config: %s");
+                            Logging.warning(failure.getFailureMessage().formatted(getLocales()));
+                        });
+            }
+        };
     }
 
     /**

src/main/java/org/mvplugins/multiverse/core/locale/MVCorei18n.java

@@ -151,6 +151,7 @@ public enum MVCorei18n implements MessageKeyProvider {
     MODIFY_WORLD_DESCRIPTION,
     MODIFY_PROPERTY_DESCRIPTION,
     MODIFY_VALUE_DESCRIPTION,
+    MODIFY_PROPERTYREMOVED,
     MODIFY_CANNOTHAVEVALUE,
     MODIFY_SPECIFYVALUE,
     MODIFY_SUCCESS,

src/main/java/org/mvplugins/multiverse/core/world/WorldManager.java

@@ -765,6 +765,7 @@ public final class WorldManager {
         }).andFinally(() -> {
             this.loadTracker.remove(worldCreator.name());
         }).fold(throwable -> Attempt.failure(WorldCreatorFailureReason.BUKKIT_CREATION_FAILED,
+                        Replace.WORLD.with(worldCreator.name()),
                         Replace.ERROR.with(throwable)),
                 Attempt::success);
     }

src/main/resources/multiverse-core_en.properties

@@ -132,6 +132,7 @@ mv-core.modify.description=Modifies a world property of a given world.
 mv-core.modify.world.description=Name of world you want to modify.
 mv-core.modify.property.description=Property you want to modify.
 mv-core.modify.value.description=Value of property.
+mv-core.modify.propertyremoved=&cYou cannot modify '&3{property}&c' because it has been changed or removed. Please use '&a{replacement}&c' instead.
 mv-core.modify.cannothavevalue=You cannot specify a value to {action} '{property}'.
 mv-core.modify.specifyvalue=You must specify a value to {action} '{property}'.
 mv-core.modify.success=&aSuccessfully {action} '&9{property}&a' to '&9{value}&a' in world &9{world}&a.
@@ -259,12 +260,12 @@ mv-core.destination.shared.failurereason.worldnotfound=&cWorld '&6{world}&c' doe
 mv-core.destination.parse.failurereason.invaliddestinationid=Invalid destination id: {id}. Valid destination ids are: {ids}
 
 # teleport failure reason
-mv-core.teleportfailurereason.null.destination=Destination is invalid!
-mv-core.teleportfailurereason.null.location=Location is invalid!
-mv-core.teleportfailurereason.null.world=&cLocation's world is invalid!
-mv-core.teleportfailurereason.unsafe.location=Location is unsafe! Use `--unsafe` to override.
-mv-core.teleportfailurereason.teleport.failed=Something rejected the teleport.
-mv-core.teleportfailurereason.teleport.failed.exception=An error occurred during the teleport. See console for more details.
+mv-core.teleportfailurereason.null.destination=The destination is invalid!
+mv-core.teleportfailurereason.null.location=The location is invalid!
+mv-core.teleportfailurereason.null.world=&cThe target world is unloaded or does not exist!
+mv-core.teleportfailurereason.unsafe.location=The location is deemed unsafe! If this is intentional, use the `--unsafe` flag to override.
+mv-core.teleportfailurereason.teleport.failed=The teleport was rejected. Please ensure another plugin is not preventing the teleport and check server logs for any errors.
+mv-core.teleportfailurereason.teleport.failed.exception=An error occurred during the teleport. Check server logs for more details.
 mv-core.teleportfailurereason.event.cancelled=The teleport was cancelled by another plugin.
 
 # world manager result