(Stricter) How to apply an enchantment to a specific Item only.

[[NoOneButNo]]

I know that canApply method exists for Enchantment class, but the way it works isn't applicable for locking down to a specific Item. For example, I want an enchantment that can only be applicable to axes and not pickaxes but overriding this method does not work.

 

A code wise example:

@Override
    public boolean canApply(ItemStack stack)
        {
        return stack.getItem() instanceof ItemAxe ? true : false;
        }

 

This works if the item enchanted in question is a sword for example however does not work if EnumEnchantmentType is DIGGER (Checking for a pickaxe against axe returns false but the enchantment table still allows player to enchant it as if its true).

[Animefan8888]

14 minutes ago, [NoOneButNo] said:

I know that canApply method exists for Enchantment class, but the way it works isn't applicable for locking down to a specific Item. For example, I want an enchantment that can only be applicable to axes and not pickaxes but overriding this method does not work.

This should work.Can you post more code maybe its something else that is causing it to do that.

[[NoOneButNo]]

This is an example of an enchantment class I have:

 

public class EnchantmentPenetratingEdge extends Enchantment
{
    public EnchantmentPenetratingEdge()
    {
        super(Rarity.RARE, EnumEnchantmentType.DIGGER, new EntityEquipmentSlot[]{EntityEquipmentSlot.MAINHAND});
        this.setName("PenetratingEdge");
        this.setRegistryName("PenetratingEdge");
    }
        
    @Override
    public int getMaxLevel()
    {
        return 6;
    }
        
    @Override
    public int getMinEnchantability(int par1)
    {
        return 16 + 12 * (par1 - 1);
    }

    @Override
    public int getMaxEnchantability(int par1)
    {
        return super.getMinEnchantability(par1) + 40;
    }
        
    @Override
    public boolean canApply(ItemStack fTest)
    {

        //Used this to find out if it yields true or false; turns out its useless
        //System.out.println(fTest.getItem() instanceof ItemAxe);
        return fTest.getItem() instanceof ItemAxe ? true : false;
    }
    
        
    
        
    

}

[Animefan8888]

59 minutes ago, [NoOneButNo] said:

This is an example of an enchantment class I have:

Override canApplyAtEnchantingTable as well.

[[NoOneButNo]]

Ok, I'll try.

[Draco18s]

8 hours ago, [NoOneButNo] said:

 return fTest.getItem() instanceof ItemAxe ? true : false;

This can be simplified to return fTest.getItem() instanceof ItemAxe

There is no reason to do a boolean check on a boolean.

[[NoOneButNo]]

Well, got used to boolean check hehe.