[1.12] Player not drinking the item [Solved]

[YellowMilk2]

I followed the ItemPotion class in Minecraft to see how to allow the player to drink a potion and wanted to implement it into my mod. However, when I call the onItemRightClick and getItemUseAction method like the class shows, my character just stand around doing nothing when I click and hold the right mouse button to drink. Can someone give me guidance to what I am missing?

My item class

public class ItemVodka extends Item {

    public ItemVodka(String name) {
        this.setUnlocalizedName(name);
        this.setRegistryName(name);
        this.setCreativeTab(CreativeTabs.FOOD);
    }

    @Override
    public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer playerIn, EnumHand handIn){
        playerIn.setActiveHand(handIn);
        return new ActionResult<ItemStack>(EnumActionResult.PASS, playerIn.getHeldItem(handIn));
    }

    @Override
    public EnumAction getItemUseAction(ItemStack stack) {
        return EnumAction.DRINK;
    }
}

 

Edited July 16, 2017 by YellowMilk2

[Animefan8888]

15 minutes ago, YellowMilk2 said:

I followed the ItemPotion class in Minecraft to see how to allow the player to drink a potion and wanted to implement it into my mod. However, when I call the onItemRightClick and getItemUseAction method like the class shows, my character just stand around doing nothing when I click and hold the right mouse button to drink. Can someone give me guidance to what I am missing?

My item class

public class ItemVodka extends Item {

    public ItemVodka(String name) {
        this.setUnlocalizedName(name);
        this.setRegistryName(name);
        this.setCreativeTab(CreativeTabs.FOOD);
    }

    @Override
    public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer playerIn, EnumHand handIn){
        playerIn.setActiveHand(handIn);
        return new ActionResult<ItemStack>(EnumActionResult.PASS, playerIn.getHeldItem(handIn));
    }

    @Override
    public EnumAction getItemUseAction(ItemStack stack) {
        return EnumAction.DRINK;
    }
}

 

You also need to override getMaxItemUseDuration