Revxrsal / Lamp

A powerful, extendable, flexible yet simple to use commands annotation framework.
MIT License
171 stars 33 forks source link

CommandRegisteredEvent is deprecated #100

Open xaanndeer opened 1 month ago

xaanndeer commented 1 month ago

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 commented 1 month ago

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

xaanndeer commented 1 month ago

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.