You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Please move away from MinecraftServer.vanillaCommandDispatcher, instead, use MinecraftServer#getCommands().
This change supports the internal dispatcher being reloaded. We will need to figure out some api in order to properly support re-registering commands under some kind of lifecycle api.
Expected code
Replace the use of MINECRAFT_SERVER.vanillaCommandDispatcher in NMS implementations of NMS#getBrigadierDispatcher
I notice that comment is from a Paper PR. If this change was applied, would the CommandAPI still work on Spigot? If not, I suppose we would just need to migrate calls to getBrigadierDispatcher over to PaperImplementations.
Also, does this change apply to Paper right now, or only after that PR is merged?
I notice that comment is from a Paper PR. If this change was applied, would the CommandAPI still work on Spigot? If not, I suppose we would just need to migrate calls to getBrigadierDispatcher over to PaperImplementations.
Also, does this change apply to Paper right now, or only after that PR is merged?
Yes, this will still work on Spigot. The MinecraftServer#getCommands method exists in Spigot and has always existed since 1.13. There isn't any particular reason why the CommandAPI should not be using this method, other than the fact it hasn't been tested and testing will be required to ensure it's a viable solution (i.e. make sure everything works as it should!)
Description
As per this comment:
Expected code
Replace the use of
MINECRAFT_SERVER.vanillaCommandDispatcher
in NMS implementations ofNMS#getBrigadierDispatcher
CommandAPI/commandapi-nms/commandapi-1.19.3/src/main/java/dev/jorel/commandapi/nms/NMS_1_19_3_R2.java
Lines 395 to 398 in 2efd3a3
with
MINECRAFT_SERVER.getCommands()
:Extra details
No response
The text was updated successfully, but these errors were encountered: