Jump to content

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


[NoOneButNo]

Recommended Posts

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).

Link to comment
Share on other sites

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.

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

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;
    }
    
        
    
        
    

}

Link to comment
Share on other sites

59 minutes ago, [NoOneButNo] said:

This is an example of an enchantment class I have:

Override canApplyAtEnchantingTable as well.

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

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.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.