Get NBT enchant levels instead of potentially event-modified levels

[TheWinABagel]

Currently, the disenchanting lens uses ItemStack#getAllEnchantments, which may not be accurate to the NBT level (due to potential modification from GetEnchantmentLevelEvent).

This PR swaps it over to ItemStack#getTagEnchantments instead, as it ignores the event.

[MichaelHillcox]

I don't know much about the event in question but if an event is pushing Enchantments onto an Item, would that not mean they should also be removed?

I'd guess what's happening here is that events are being used to push extra fake enchants onto items to provide temporary enhancements?

[Mrbysco]

Looks good to me. Has it been tested to work?

[Shadows-of-Fire]

I don't know much about the event in question but if an event is pushing Enchantments onto an Item, would that not mean they should also be removed?

No, enchantments from the event are transient and only for gameplay purposes. When removing the enchantments from the item, you should be using getTagEnchantments (or more generically, EnchantmentHelper.getEnchantmentsForCrafting).