SpongeAPI 8

[lucko]

No description provided.

[lucko]

Test jar LuckPerms-Sponge-5.2.43.jar compiled from this branch: https://we.tl/t-8AYRD2N0kp

[dualspiral]

One thing, when you update someone's permissions, you will want to call Sponge.getCommandManager().updateCommandTreeForPlayer(ServerPlayer); for any player that has had their permissions updated so Sponge will send the updated command tree to the player.

[lucko]

Sure will do - do you know if it's safe to call that async?

[Zidane]

@dualspiral @lucko That would not be safe to call async.

[lucko]

Thanks 👍

Added that, here's another jar: https://we.tl/t-7EKaJk4gGE

[funniray]

Sponge's API 8 seems to have changed all calls such as getPlayer() to just player() and it appears that this branch no longer works.

Are there any plans currently to update this branch to the latest version of API 8?

[lucko]

I'm waiting until SpongeAPI 8 satablises before rebasing and updating this.

[lucko]

Updated to latest Sponge API.. new test build:

LuckPerms-Sponge-5.3.78.jar.zip

[lucko]

LuckPerms-Sponge-5.3.80.jar.zip

[Lunoro]

There's an error with a scheduler in 5.3.80. (latest API changes)

[johnfriedrich]

https://pastebin.com/CSK6cNYN error on load

[dualspiral]

Just as a note, I notice that stack trace is on SpongeForge which is somewhat behind SpongeVanilla in implementation, particularly during loading. I would suggest testing against SpongeVanilla - if it works there, it more likely needs sorting in SF, not LP.

[lucko]

LuckPerms-Sponge-5.3.82.jar.zip

[tyhdefu]

Sponge 8 has stablised version 8.0.0 of the API - https://forums.spongepowered.org/t/status-update-8th-january-2022-spongeapi-8-released/39861 (Although no stable release of the implementation yet)
So this should be changed to be using the non-snapshot version of the api and fixed for any changes.
Thanks for all your work on LuckPerms.