package com.averysumner.randomstuff;

import net.minecraft.client.renderer.block.model.ModelResourceLocation;
import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.projectile.EntitySnowball;
import net.minecraft.init.Items;
import net.minecraft.init.SoundEvents;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.stats.StatList;
import net.minecraft.util.ActionResult;
import net.minecraft.util.EnumActionResult;
import net.minecraft.util.EnumHand;
import net.minecraft.util.SoundCategory;
import net.minecraft.world.World;
import net.minecraftforge.client.model.ModelLoader;
import net.minecraftforge.fml.common.registry.GameRegistry;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;

public class ThrowableSlimeball extends Item {

    public ThrowableSlimeball(String registryName, String unlocalizedName) {
        setRegistryName(registryName);
        setUnlocalizedName(unlocalizedName);
        GameRegistry.register(this);
        
    }
    
    public ActionResult<ItemStack> onItemRightClick(ItemStack itemStackIn, World worldIn, EntityPlayer playerIn, EnumHand hand)
    {
    	
        if(itemStackIn == new ItemStack(Items.SLIME_BALL)){
        	System.out.println("SLIMEBALL CLICKED!");
        if (!playerIn.capabilities.isCreativeMode)
        {
            --itemStackIn.stackSize;
        }

        worldIn.playSound((EntityPlayer)null, playerIn.posX, playerIn.posY, playerIn.posZ, SoundEvents.ENTITY_SLIME_SQUISH, SoundCategory.NEUTRAL, 0.5F, 0.4F / (itemRand.nextFloat() * 0.4F + 0.8F));

        if (!worldIn.isRemote)
        {
            EntitySnowball entityslimeball = new EntitySnowball(worldIn, playerIn);
            entityslimeball.setHeadingFromThrower(playerIn, playerIn.rotationPitch, playerIn.rotationYaw, 0.0F, 1.5F, 1.0F);
            worldIn.spawnEntityInWorld(entityslimeball);
        }

        playerIn.addStat(StatList.getObjectUseStats(this));
        return new ActionResult(EnumActionResult.SUCCESS, itemStackIn);
}
        return new ActionResult(EnumActionResult.FAIL, itemStackIn);
}
}

 

I may just be being stupid here, but even if I'm holding a slimeball it doesn't go through. Can someone help me?

Im not creating and registering an item. Im using the vanilla slimeball.

The

ItemStack

argument of

Item#onItemRightClick

will only ever contain

this

. If it contains a different

Item

, something's gone wrong. This means your first if statement checking if the

ItemStack

contains

Items.SLIMEBALL

will never be

true

.

 

This also means that you don't need to check which

Item

the

ItemStack

contains at all, you already know it's

this

.

 

Im not creating and registering an item. Im using the vanilla slimeball.

 

You're creating a new item, not using the vanilla one. I explained how to use events to detect when the player right clicks with a slimeball in your previous thread.