[Solved|1.18.2] Creating New EnchantmentCategory for Custom Enchantment

[RuthlessNail]

Quite a trivial question I reckon, but my brain just can't figure it out. 

I want to create a "sharpness-esque" enchantment for my custom hammer tool, I tried using EnchantmentCategory.create, but I seems to be only able to pass in one item at a time. The enchantment currently only work on the wooden hammer I passed in. I hope to make it work on all ItemHammer instance. (I also have a tag for all hammer item set up, but I can't figure out how to pass it in using either method)

    static EnchantmentCategory HAMMER_TYPE = EnchantmentCategory.create("hammer", item -> item == ModItems.WOODEN_HAMMER.get());
    protected HammerHurtEnchantment(Rarity pRarity, EnchantmentCategory pCategory, EquipmentSlot... pApplicableSlots) {
        super(pRarity, HAMMER_TYPE, pApplicableSlots);

Edited January 22, 2023 by RuthlessNail
editing the title to reflect that the problem have been solved

[warjort]

This how you would do it with forge's pickaxes item tag.

item -> ForgeRegistries.ITEMS.tags().getTag(Tags.Items.TOOLS_PICKAXES).contains(item);

Alternatively you could do it rhe old fashioned way with a marker interface (or abstract super class)

public class MyWoodenHammerItem implements Hammer {
...
}

public class MyMetalHammerItem implements Hammer {
...
}

item -> item instanceof Hammer

 

[RuthlessNail]

37 minutes ago, warjort said:

This how you would do it with forge's pickaxes item tag.

item -> ForgeRegistries.ITEMS.tags().getTag(Tags.Items.TOOLS_PICKAXES).contains(item);

Alternatively you could do it rhe old fashioned way with a marker interface (or abstract super class)

public class MyWoodenHammerItem implements Hammer {
...
}

public class MyMetalHammerItem implements Hammer {
...
}

item -> item instanceof Hammer

 

The first method work well with my current tag, I registered all my hammer items in the ModItems class using instance of custom HammerItem class so I probably would have to do it quite differently if I was using the second method (because I testing item -> item instanceof HammerItem prior to this post and it didn't work). Nonetheless, I will change the post topic to solved now. Greatly appreciate your help!