Fix blacklist

[democat3457]

This should fix #161 by not checking for similar NBT tags. This allows modded installs to function properly (since you can't specify NBT in the configuration file). CraftTweaker blacklist is still broken though.

[democat3457]

CraftTweaker blacklist fixed in the most recent commit.

[democat3457]

@Darkhax pls review

[kil0byt3]

It would be great if this could get merged in, helps out a lot with modpacks (specifically Rebirth of the Night)

[xJon]

@democat3457 It seems that with your commit tools & armor that have been used (with durability) can still be enchanted.

[democat3457]

This is good, no?

[xJon]

This is good, no?

This means it's still impossible to prevent Tinkers gear from being used in the Advanced Enchanting Table.
It'd be preferable to be able to prevent Tinkers gear from being enchanted regardless of durability :)

[democat3457]

oh i see - the blacklist currently doesn't apply to items with different durabilities. Do you know if Tinkers uses durabilities via NBT, metadata, or actual durability values?

[xJon]

Hmm, I don't but the mod is open source - https://github.com/SlimeKnights/TinkersConstruct/tree/1.12
Also that is the case for non tinkers items too, the blacklist only works if they haven't been used.

[democat3457]

@xJon have you tried putting a wildcard meta at the end of the item in the config? or try using crafttweaker with an itemstack that has a * meta.

in any case, the most recent commit should also fix that by defaulting items without meta in the config to a wildcard meta