Fixed NBT being checked when it shouldn't

[TBiscuit1]

Not the most code efficient (still a noob to Forge codding), but it gets the job done

Feel free to upgrade it!

[noeppi-noeppi]

I still don't understand why this is necessary. If the ItemStack from the task has no NBT, Util.matchesNBT won't even check the actual nbt. And if the required ItemStack has a tag that does not contain the key Damage, it is not checked in the actual NBT as well.
At least that's how it is supposed to work...

[TBiscuit1]

And if the required ItemStack has a tag that does not contain the key Damage

The game automaticly gives the Damage key to any ItemStack with durability, which is the problem, so the compare ItemStack gets Damage:0 causing the nbt match to fail

Oh and it's without forgetting that EventListener has "tagMatches" which want the EXACT same nbt so your matchesNBT is meaningless at the end

[TBiscuit1]

Ok so now it has a better code optimisation and also fixes both the durability issue AND the nbt issue in general

But now there is a new problem, it's now impossible to have tools without durability lost thanks to that stupid "Damage:0" that the game insert inside the ItemStack for no reason