Skip to content

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

Open
drappula wants to merge 3 commits into
Solara-Development:masterfrom
drappula:legacy-material-fix
Open

Remove the use of Material.values() which caused Bukkit Legacy Material Support to trigger#200
drappula wants to merge 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

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

2 participants