Skip to content

Conversation

@iiAhmedYT
Copy link
Contributor

Information

This PR fixes an issue where clickable messages don't work on newer versions (1.21.5).

Details

Proposed fix:
Update Adventure

Environments tested:

OS: Windows, Linux

Java version: 21

  • Most recent Paper version (1.XX.Y, git-Paper-BUILD)
  • CraftBukkit/Spigot/Paper 1.12.2
  • CraftBukkit 1.8.8

Demonstration:

It's really not worth a demonstration lol

@JRoy
Copy link
Member

JRoy commented Jun 4, 2025

can you bump adventure platform to 4.4.0 as well? otherwise good

@iiAhmedYT
Copy link
Contributor Author

Yes sir captin sir 🫡

JRoy
JRoy previously approved these changes Jun 4, 2025
Copy link
Member

@JRoy JRoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

love to see it

@JRoy JRoy enabled auto-merge (squash) June 4, 2025 23:30
@JRoy JRoy disabled auto-merge June 4, 2025 23:30
@JRoy JRoy enabled auto-merge (squash) June 4, 2025 23:31
@JRoy JRoy disabled auto-merge June 4, 2025 23:31
@JRoy JRoy enabled auto-merge (squash) June 4, 2025 23:31
@iiAhmedYT
Copy link
Contributor Author

how does it even fail-

@iiAhmedYT
Copy link
Contributor Author

Apparently it failed to find Adventure Platform related classes.. (btw it is on Maven Central, so it not being found is actually insanity) the fix is not really on my mind

@JasonHorkles
Copy link
Contributor

Probs some maven cache or something silly

Real intention behind this is to rerun the build. lol
auto-merge was automatically disabled June 5, 2025 12:05

Head branch was pushed to by a user without write access

@iiAhmedYT
Copy link
Contributor Author

The error is about a class that isn't even used in the code, im not sure why it's happening but apparently it's loading and compiling just fine, the tests on the other hand aren't doing so fine themselves

@iiAhmedYT
Copy link
Contributor Author

Upon further investigation, this line:
https://github.com/KyoriPowered/adventure-platform/blob/main/platform-bukkit/src/main/java/net/kyori/adventure/platform/bukkit/BukkitComponentSerializer.java#L66

which uses Bukkit#getUnsafe#getDataVersion is the reason behind it all, the tests simply cant provide it

A solution would be to use proper testing for Bukkit like MockBukkit

or.. Just temporarily disable these tests

@JRoy
Copy link
Member

JRoy commented Jun 6, 2025

I'll look in a few

@JRoy JRoy merged commit 997f600 into EssentialsX:2.x Jun 6, 2025
1 check passed
Euphillya added a commit to Euphillya/Essentials-Folia that referenced this pull request Jun 27, 2025
EssentialsX@40540eb Expose IUser#getLastActivityTime to API (EssentialsX#6101)
EssentialsX@50b696b Adds social spy message type for in EssentialsDiscord (EssentialsX#5620)
EssentialsX@dabe687 Add SignTransactionEvent for buy and sell signs (EssentialsX#6071)
EssentialsX@0cb387f Fix NPE when unlinking offline players (EssentialsX#6111)
EssentialsX@f09541c Update to Minecraft 1.21.5 (EssentialsX#6109)
EssentialsX@c7ff994 Fire GlobalChatEvent even when local chat is disabled (EssentialsX#6113)
EssentialsX@06c886c Fix /baltop on 1.15.2 and below (EssentialsX#6115)
EssentialsX@2bb4438 Fix further trade sign validation mismatches (EssentialsX#6116)
EssentialsX@8455212 Fix /spawnmob on < 1.21.5 (EssentialsX#6121)
EssentialsX@be3e641 New Crowdin updates (EssentialsX#6089)
EssentialsX@3f5b122 Use legacy text for TranslatableException#getMessage (EssentialsX#6134)
EssentialsX@fe43017 Fix heer typo in messages_en.properties (EssentialsX#6135)
EssentialsX@a58db23 Update English source strings with fixes from Crowdin (EssentialsX#6140)
EssentialsX@d137031 Improve config comments & ordering (EssentialsX#6108)
EssentialsX@3b0c229 New Crowdin updates (EssentialsX#6131)
EssentialsX@3fe1495 Fix unexpected behavior with essentials.home.bed & essentials.sethome.bed permissions (EssentialsX#5991)
EssentialsX@aebe851 Fix double messages when deleting a non-existent home (EssentialsX#6150)
EssentialsX@3a5c7c4 Revert changes to /home and /sethome that break bed interactions (EssentialsX#6151)
EssentialsX@3d28d82 Release 2.21.1
EssentialsX@e836747 Prepare for 2.21.2 dev builds (EssentialsX#6128)
EssentialsX@d42347f Fix sending message during /editsign paste if SignChangeEvent is cancelled (EssentialsX#6136)
EssentialsX@997f600 Update adventure dependencies (EssentialsX#6163)
EssentialsX@4057326 Update to Minecraft 1.21.6
EssentialsX@2796399 Add 1.21.6 Mobs
EssentialsX@31ea903 Move tests to MockBukkit
EssentialsX@ff1b8b8 Add 1.21.6 Aliases
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

Successfully merging this pull request may close these issues.

3 participants