[1.15.2]Play animation and sound with custom item

[Mower]

I have a newly registered item, which is a water flask. I'm trying to play the drinking animation and the drinking sound while right clicking on it, but it doesn't work. Because of the sound playing try, my minecraft just crashes. And the animation just doesn't work.

Here's my whole Item class:

package com.venonat.venonatmod.objects.items;
import com.venonat.venonatmod.events.MyTimer;
import net.minecraft.client.Minecraft;
import net.minecraft.entity.player.PlayerEntity;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.item.UseAction;
import net.minecraft.util.ActionResult;
import net.minecraft.util.Hand;
import net.minecraft.util.SoundEvents;
import net.minecraft.world.World;

public class FlaskItem extends Item {
    MyTimer timer = MyTimer.getInstance();
    Minecraft mc;
    boolean alreadyExecuted;

    public FlaskItem(Properties props) {
        super(props);
    }


    @Override
    public UseAction getUseAction(ItemStack stack){
        getUseAction(stack);
        return UseAction.DRINK;
    }

    @Override
    public ActionResult<ItemStack> onItemRightClick(World worldIn, PlayerEntity playerIn, Hand handIn) {
        timer.setCountdown(timer.getCountdown() + 5);
        if (!alreadyExecuted) {
            mc.player.playSound(SoundEvents.ENTITY_GENERIC_DRINK, 0.5f, 1.0f);
            alreadyExecuted = true;
        }
        return super.onItemRightClick(worldIn, playerIn, handIn);
    }
}

and here is my item registration:

package com.venonat.venonatmod.init;

import com.venonat.venonatmod.VenonatMod;
import com.venonat.venonatmod.objects.items.FlaskItem;
import net.minecraft.item.Item;
import net.minecraft.item.ItemGroup;
import net.minecraftforge.event.RegistryEvent;
import net.minecraftforge.eventbus.api.SubscribeEvent;
import net.minecraftforge.fml.common.Mod;
import net.minecraftforge.fml.common.Mod.EventBusSubscriber.Bus;
import net.minecraftforge.registries.ObjectHolder;

@ObjectHolder(VenonatMod.MOD_ID + "flask")
@Mod.EventBusSubscriber(modid = VenonatMod.MOD_ID, bus = Bus.MOD)

public class ItemInit {

    @SubscribeEvent
    public static void registerItems(final RegistryEvent.Register<Item> event) {
        event.getRegistry().register(new FlaskItem(new Item.Properties().maxStackSize(1).group(ItemGroup.MISC)).setRegistryName("flask"));
    }
}

 

[poopoodice]

1 hour ago, Mower said:

I have a newly registered item, which is a water flask. I'm trying to play the drinking animation and the drinking sound while right clicking on it, but it doesn't work. Because of the sound playing try, my minecraft just crashes. And the animation just doesn't work.

Here's my whole Item class:

package com.venonat.venonatmod.objects.items;
import com.venonat.venonatmod.events.MyTimer;
import net.minecraft.client.Minecraft;
import net.minecraft.entity.player.PlayerEntity;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.item.UseAction;
import net.minecraft.util.ActionResult;
import net.minecraft.util.Hand;
import net.minecraft.util.SoundEvents;
import net.minecraft.world.World;

public class FlaskItem extends Item {
    MyTimer timer = MyTimer.getInstance();
    Minecraft mc;
    boolean alreadyExecuted;

    public FlaskItem(Properties props) {
        super(props);
    }


    @Override
    public UseAction getUseAction(ItemStack stack){
        getUseAction(stack);
        return UseAction.DRINK;
    }

    @Override
    public ActionResult<ItemStack> onItemRightClick(World worldIn, PlayerEntity playerIn, Hand handIn) {
        timer.setCountdown(timer.getCountdown() + 5);
        if (!alreadyExecuted) {
            mc.player.playSound(SoundEvents.ENTITY_GENERIC_DRINK, 0.5f, 1.0f);
            alreadyExecuted = true;
        }
        return super.onItemRightClick(worldIn, playerIn, handIn);
    }
}

and here is my item registration:

package com.venonat.venonatmod.init;

import com.venonat.venonatmod.VenonatMod;
import com.venonat.venonatmod.objects.items.FlaskItem;
import net.minecraft.item.Item;
import net.minecraft.item.ItemGroup;
import net.minecraftforge.event.RegistryEvent;
import net.minecraftforge.eventbus.api.SubscribeEvent;
import net.minecraftforge.fml.common.Mod;
import net.minecraftforge.fml.common.Mod.EventBusSubscriber.Bus;
import net.minecraftforge.registries.ObjectHolder;

@ObjectHolder(VenonatMod.MOD_ID + "flask")
@Mod.EventBusSubscriber(modid = VenonatMod.MOD_ID, bus = Bus.MOD)

public class ItemInit {

    @SubscribeEvent
    public static void registerItems(final RegistryEvent.Register<Item> event) {
        event.getRegistry().register(new FlaskItem(new Item.Properties().maxStackSize(1).group(ItemGroup.MISC)).setRegistryName("flask"));
    }
}

 

why use mc.player instead of the player already given? and you should probably store the cooldown in nbts

[Mower]

Could you show on my code what to do then?

[poopoodice]

1 hour ago, Mower said:

Could you show on my code what to do then?

You can use capabilities or nbt tags to store data, and you will probably want to just use the playerIn in onItemRightClick() instead