When I try to use this code, it only runs half of the time. I'm using this to try to create an item that toggles between two states.

 

@Override
public void onPlayerStoppedUsing(ItemStack par1Stack, World p_77974_1_, EntityPlayer p_77974_2_, int p_77974_3_)
{
if(par1Stack.getItemDamage() == 1 || par1Stack.getItemDamage() == 3)
{
	par1Stack.setItemDamage(par1Stack.getItemDamage() - 1);
}
getIconFromDamage(par1Stack.getItemDamage());
System.out.println("stopped using");
System.out.println(par1Stack.getItemDamage());
}

 

There is also a line in onPlayerRightClick that sets the item in use.

Any help?