EssentialsDiscord PlayerAdvancementDoneEvent error
PetyXbron opened this issue ยท 9 comments
Type of bug
Error in console
/ess dump all
output
Version 2.19.3-dev+1-05ce11f
UPDATE BRANCH 2.x
BUKKIT VERSION 1.18.1-R0.1-SNAPSHOT
SERVER VERSION git-Paper-99 (MC: 1.18.1)
JAVA VERSION 17.0.1
Error log (if applicable)
https://paste.gg/p/anonymous/c29c6446e28e438d9f61da28e35124ee
Bug description
Got this error when a player got a Minecraft [Hot Tourist Destinations]
advancement.
Message (announcement) isn't sent to primary (or staff) channel when set.
Steps to reproduce
- Use EssentialsDiscord when server running and set in config.yml
advancement: primary
# Achievement/advancement messages sent when a player is awarded an achievement/advancement.
advancement: primary
- Get [Hot Tourist Destinations] advancement (Nether)
- View the console
Expected behaviour
Get a new message in Discord channel (primary or staff one) without any errors in console
Actual behaviour
Error received to console when player received achievement.
[20:00:10 INFO]: vojtas456 has completed the challenge [Hot Tourist Destinations]
This is caused by a recent Paper change which breaks the method we use to get plain text versions of the advancement messages.
So it will be fixed when Paper releases Stable 1.18.1 update?
I doubt that. We will likely need to add a second Paper advancement provider for 1.18.1+, unless Paper decides to revert and preserve the legacy serializer method.
Okay. This seems to be a problem for a long time. Thank you for your quick response, and I hope it will be fixed soon.
This is caused by a recent Paper change which breaks the method we use to get plain text versions of the advancement messages.
Same thing happened. Player received achievement and I got an error to console.
[20:16:10 INFO]: xilyx11 has made the advancement [Caves & Cliffs]
Error: https://paste.gg/p/anonymous/3e1fb141a7964830b3a4d8aa7ae47793
So this doesn't happens only with [Hot Tourist Destinations] advancement
This is caused by a recent Paper change which breaks the method we use to get plain text versions of the advancement messages.
So it will be fixed when Paper releases Stable 1.18.1 update?
I have fixed this in Paper, PaperMC/Paper@1ce4281 (build 100).
Since the method we use is deprecated, I will also add a mitigation in Essentials to assure this doesn't become an issue when Paper removes it.
I appreciate the hard work. I just came poking around GitHub after seeing the same issue on build 99 and I see you got a fix out just over an hour ago. Thanks!