Paper throws a warning in 1.21.9

[Novampr]

Paper throws a warning when using cloud-bukkit complaining about a deprecated event with undefined behaviour. Paper provides an alternative to this event which should be used when on a modern version of paper.

The warning can be seen as below: (For the purposes of this, Cloud was shaded into Geyser, this shouldn't affect this however)

[16:47:43] [Server thread/WARN]: [Geyser-Spigot] "Geyser-Spigot v2.9.0-SNAPSHOT" has registered a listener for org.spigotmc.event.player.PlayerSpawnLocationEvent on method "void org.geysermc.geyser.platform.spigot.shaded.org.incendo.cloud.bukkit.CloudBukkitListener.onPlayerLogin(org.spigotmc.event.player.PlayerSpawnLocationEvent)", but the event is Deprecated. "Listening to this event causes the player to be created early. Using the player from this event will result in undefined behavior. Prefer AsyncPlayerSpawnLocationEvent."; please notify the authors [GeyserMC].

[jpenilla]

Fixing this is not so simple as the Bukkit module cannot compile against Paper. It would probably need a second Paper module for Bukkit to depend on.

If avoiding issues like these on new versions is a priority I would recommend using the non-legacy Paper command manager on compatible versions. Especially with a multi-platform scaffolding already existing it should be quite straightforward.

[AlexProgrammerDE]

Could you let me manually lock brigadier, so that you don't have to add this listener at all?

[AlexProgrammerDE]

I can manually lock brigadier right after finishing registering all my commands

[bergerkiller]

Is affecting me also and was about to ask. Will need a fix as users are going to treat the warn as an error and bother me, as usual :)

Keep up the good work

EDIT

Fixing this is not so simple as the Bukkit module cannot compile against Paper. It would probably need a second Paper module for Bukkit to depend on.

Just want to add that bukkit has an event API where you can pass the class + a callback, without @eventlistener stuff. So technically you can make this work without ever having paper on the class path. Less proper than a separate module of course.

[bergerkiller]

Thanks for merging my simple fix, the 2.0.0-beta.13 release of cloud-minecraft now fixes the warning for me