Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CommandRegisteredEvent is deprecated #100

Open
xaanndeer opened this issue May 14, 2024 · 2 comments
Open

CommandRegisteredEvent is deprecated #100

xaanndeer opened this issue May 14, 2024 · 2 comments

Comments

@xaanndeer
Copy link

I just updated my server to Paper version git-Paper-70 (MC: 1.20.6) (Implementing API version 1.20.6-R0.1-SNAPSHOT) (Git: f041f48) and received this warning in console on startup. I suppose it has something to do with the recently merged Brigadier API, although I'm not sure. Entity selectors seem to be broken as well.

<plugin> has registered a listener for com.destroystokyo.paper.event.brigadier.CommandRegisteredEvent on method "public void revxrsal.commands.bukkit.brigadier.PaperCommodore$CommandRegisterListener.onCommandRegistered(com.destroystokyo.paper.event.brigadier.CommandRegisteredEvent<?>)", but the event is Deprecated. "Server performance will be affected"; please notify the authors <author>

I'm currently using Lamp common/bukkit version 3.2.1

@Revxrsal
Copy link
Owner

Seems Paper deprecated this event. I'll see it when I get a chance. Thank you for the notice.

@xaanndeer
Copy link
Author

Yes, this seems to be related to the recently merged Brigadier Command API. From my understanding this update has changed a lot with how commands are handled under the hood. Paper is now advising command frameworks to use this new API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants