Fix `InventoryTag` properties handling - Githubissues

DenizenScript / Denizen

NPC and general Spigot scripting, using the Denizen Scripting Language!

https://denizenscript.com/

MIT License

216 stars 106 forks source link

Fix `InventoryTag` properties handling #2607

Closed tal5 closed 8 months ago

tal5 commented 8 months ago

Reported on Discord.

Changes (to `InventoryTag#internalApplyPropertySet`)

Moved everything but name to within the if check, as there's no point in creating them if they might not be used.
Removed descriptionElement in favor of using ObjectFetcher#pickObjectFor in the Mechanism constructor.
Now passes name to the mechanism constructor instead of getting it from data again (Mechanism lowercases it anyway).