Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add API to register enchantment names/aliases #3669

Merged
merged 4 commits into from
Jan 2, 2021
Merged

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
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