Skip to content

Conversation

@WillFP
Copy link
Contributor

@WillFP WillFP commented Sep 13, 2020

For registering enchantment/alias for tabcompletion through essentials as a dependency.
Changes this:
((Map<String, Enchantment>) FieldUtils.readDeclaredStaticField(Enchantments.class, "ENCHANTMENTS", true)).put(name, enchantment);
To this:
Enchantments.registerEnchantment(name, enchantment)

@pop4959 pop4959 added the type: enhancement Features and feature requests. label Sep 13, 2020
pop4959
pop4959 previously approved these changes Sep 16, 2020
@pop4959 pop4959 requested a review from mdcfe September 16, 2020 05:04
@mdcfe mdcfe added the status: needs manual rebase Needs a manual local rebase due to base changes (regardless of what GH says) label Oct 5, 2020
Co-authored-by: MD <1917406+md678685@users.noreply.github.com>
Co-authored-by: MD <1917406+md678685@users.noreply.github.com>
@JRoy
Copy link
Member

JRoy commented Dec 21, 2020

Hello, can you please rebase this pull request so we can merge it! Thanks!

@JRoy JRoy changed the title Added support for registering enchantment names and aliases without reflection Add API to register enchantment names/aliases Jan 2, 2021
@JRoy JRoy merged commit 2f45d3e into EssentialsX:2.x Jan 2, 2021
@mdcfe mdcfe added this to the 2.19.0 milestone Feb 20, 2021
ressidell pushed a commit to ressidell/Essentials that referenced this pull request Apr 27, 2025
Co-authored-by: MD <1917406+md678685@users.noreply.github.com>
Co-authored-by: JRoy <joshroy126@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: needs manual rebase Needs a manual local rebase due to base changes (regardless of what GH says) type: enhancement Features and feature requests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants