Skip to content

Remove the use of Material.values() which caused Bukkit Legacy Material Support to trigger - #200

Merged
Devlrxxh merged 3 commits into
Solara-Development:masterfrom
drappula:legacy-material-fix
Sep 22, 2026
Merged

Devlrxxh merged 3 commits into
Solara-Development:masterfrom
drappula:legacy-material-fix

Conversation

@drappula

@drappula drappula commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Fixes #198

@drappula drappula changed the title Removes the use of Material.values() which caused Bukkit Legacy Material Support to trigger Remove the use of Material.values() which caused Bukkit Legacy Material Support to trigger Sep 4, 2026
@Biquaternions

Copy link
Copy Markdown

A small opt:
You can also get rid of the !m.isLegacy() check as the material registry already excludes legacy materials, see:
https://github.com/PaperMC/Paper/blob/main/paper-api/src/main/java/org/bukkit/Registry.java#L194

@Devlrxxh
Devlrxxh merged commit a00edbf into Solara-Development:master Sep 22, 2026
1 check passed
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.

Neptune using legacy material support

3 participants