good

days

long ago i post this same question but let it behind unsolved for

things

 

create this 3d item and make it animated swiching json files,  when aim this drill at a block it begins to goin down and up like crazy, and thats anoing, worst is to see it  in  hand guns.

 

 

in this video if aim the drill to a block it begins to bounse but aiming to air works well, the shotgun goes down when shoot and its not suppouse to goin down when shoot , if aiming to a block it bounse mad as well. 

 

if i dont use playerIn.setItemInUse()  and EnumAction it swings even when aiming to air. 

Using the bow EnumAction,  i like the way it looks in thirth view but is shitti in first person, the animation stop until getMaxItemUseDuration() run out of time and restart

   

so help

how i stop the swing animations at item rigth click ?

the use i been using of playerIn.setItemInUse() is rigth? or Unnecessary

 

 

this is mi code i been geting the tics to switch the animations from the getModel() method to avoid mesing whith onUpdate() method and events

 

package mercenarymod.items.herraminetas;

import mercenarymod.materialesMercenarios;
import mercenarymod.Mercenary;
import net.minecraft.block.Block;
import net.minecraft.block.properties.PropertyDirection;
import net.minecraft.block.state.IBlockState;
import net.minecraft.client.resources.model.ModelResourceLocation;
import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.entity.EntityLivingBase;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.EnumAction;
import net.minecraft.item.ItemPickaxe;
import net.minecraft.item.ItemStack;
import net.minecraft.item.ItemSword;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.util.BlockPos;
import net.minecraft.util.ChatComponentText;
import net.minecraft.util.EnumFacing;
import net.minecraft.util.IChatComponent;
import net.minecraft.world.World;
import net.minecraftforge.fml.common.registry.GameRegistry;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;
import mercenarymod.blocks.MercenaryModBlocks;
import mercenarymod.items.MercenaryModItems;
import mercenarymod.utilidades.chat;
import mercenarymod.utilidades.nbtMercenaria;

import java.lang.Math;
import java.util.ArrayList;
import java.util.List;


import mercenarymod.utilidades.util;



//############################################################################3
public class drillMercenario extends ItemPickaxe{

public static String name = "drillMercenario";

boolean giro=false;

int munitionMax=0;
static int munitionMax0=64;

int tick=0;
int textura;

boolean continuar=false;


public static ArrayList<Integer> coor0 = new ArrayList<Integer>(); 
public static final PropertyDirection FACING = PropertyDirection.create("facing");

//############################################################################3
public drillMercenario(){
super(materialesMercenarios.carbono_carbono());

setUnlocalizedName(Mercenary.MODID + "_" + name);
GameRegistry.registerItem(this, name);
setCreativeTab(Mercenary.herramientas);
this.maxStackSize = 1;
}


//######################################################################################3	
 @Override
public ModelResourceLocation getModel(ItemStack drill, EntityPlayer playerIn, int useRemaining){

ModelResourceLocation modelresourcelocation = new ModelResourceLocation("modmercenario:herramientas/drill/drillMercenario_04", "inventory");	 
animation(drill, playerIn,tick);

if ((tick%10)==0){

boolean animation=nbtMercenaria.getBooleantag(drill, "animation");
//System.out.println("animation="+animation);

if(!animation){
//animation(drill, playerIn,tick);
//textura=nbtMercenaria.getInttag(drill, "textura");

giro=false;

}
}

System.out.println("textura="+textura);
switch(textura){
    default:modelresourcelocation = new ModelResourceLocation("modmercenario:herramientas/drill/drillMercenario", "inventory") ;break;
case 1:modelresourcelocation = new ModelResourceLocation("modmercenario:herramientas/drill/drillMercenario_00", "inventory") ;break;
case 2:modelresourcelocation = new ModelResourceLocation("modmercenario:herramientas/drill/drillMercenario_01", "inventory") ; break;	
case 3:modelresourcelocation = new ModelResourceLocation("modmercenario:herramientas/drill/drillMercenario_02", "inventory") ;break;
case 4:modelresourcelocation = new ModelResourceLocation("modmercenario:herramientas/drill/drillMercenario_03", "inventory") ;break;        
case 5:modelresourcelocation = new ModelResourceLocation("modmercenario:herramientas/drill/drillMercenario_04", "inventory") ;break;        
                }



tick++;
if (tick>100){tick=0;nbtMercenaria.setBooleantag(drill, "animation",false);}
  	return modelresourcelocation;
  	
	}

//####################################################################################3	
void animation(ItemStack drill, EntityPlayer playerIn,int tick){	 

 if (giro){
 System.out.println("Giro="+tick);	 
 switch (tick){
 case 0: case 12: case 49:  textura=1;break;
 case 3: case 15: case 46:  textura=2;break;
 case 6: case 18: case 43:  textura=3;break;
 case 9: case 21: case 40:  textura=4;break;

 case 23: textura=5;break;

 case 33: if(continuar){this.tick=22;continuar=false;}break;

 case 52: textura=0;this.tick=0;nbtMercenaria.setBooleantag(drill, "animation",false);giro=false;break;
          }   }




}

//####################################################################################3		 
void inicializar(ItemStack drill){  //this do nothing for now

int munition=0;
int metadata=drill.getMetadata();

switch(metadata){
case 1: munition=munitionMax0/2;break;
case 2: munition=munitionMax0;break;
default:munition=0;break;
            	}

nbtMercenaria.setInttag(drill, "munition", munition);
nbtMercenaria.setBooleantag(drill, "animation", false);
nbtMercenaria.setBooleantag(drill, "liberar", false);
nbtMercenaria.setInttag(drill, "bulletClip", 0);
nbtMercenaria.setInttag(drill, "textura", 0);			


}
//####################################################################################3	 


public ItemStack onItemRightClick(ItemStack drill, World worldIn, EntityPlayer playerIn){


System.out.println("onItemRightClick");
playerIn.setItemInUse(drill, this.getMaxItemUseDuration(drill));

boolean animation=nbtMercenaria.getBooleantag(drill, "animation");	
if (!animation){	

System.out.println("if (!animation)");	
playerIn.setItemInUse(drill, this.getMaxItemUseDuration(drill));
int munition=nbtMercenaria.getInttag(drill,"munition");	

//if (munition>0){}


giro=true;
tick=0;
nbtMercenaria.setBooleantag(drill, "animation", true);
System.out.println("\n\n acivar");
	            
                }     


if (animation & giro){
continuar=true;	
                   	 }	


return drill;}
//############################################################################3
public int getMaxItemUseDuration(ItemStack stack)
{
    return 1;
}

public EnumAction getItemUseAction(ItemStack stack)
{
    return EnumAction.NONE;
}

//######################################################################################3	
@SideOnly(Side.CLIENT)
  public void addInformation(ItemStack drill, EntityPlayer player, List list, boolean aBoolean)
  {
int munition=nbtMercenaria.getInttag(drill, "munition");	

if((munition>9998)||(munition<0)){  //inicializar
	int metadata=drill.getMetadata();

	switch(metadata){
	case 1: munition=munitionMax0/2;break;
	case 2: munition=munitionMax0;break;
	default:munition=0;break;
	            	}

                                 }

String lore="redstone="+munition;	
String lore0="requiere suspencion de redstone para funcionar";

if(lore != "")
     {
        list.add(lore);
        list.add(lore0);
     }
  }

}//fin de la classe 

 

 

Hello everyone!

I found a very simple solution to this problem that has bothered me too.

In the class "ItemRender.java" in method "updateEquippedItem()" is an assignment to a variable named flag

flag = this.equippedItemSlot == entityclientplayermp.inventory.currentItem && itemstack == this.itemToRender;

To solve this problem, just replace that line with this

boolean flag;
if(itemstack == null) {
flag = this.equippedItemSlot == entityclientplayermp.inventory.currentItem && itemstack == this.itemToRender;
} else {
flag = this.equippedItemSlot == entityclientplayermp.inventory.currentItem && itemstack.itemID == this.itemToRender.itemID;
}

It worked, I'm using version 1.5.2.

 

As said in the other post you posted on a old topic, your code is for 1.5. Now you can do it with Forge hooks. Nobody wants your code which they can't even use. It is for a MC version almost nobody uses anymore. And you posted on a topic made in February! Don't revive old threads like these anymore.

Well, if you are using the latest build of Forge (or close to it) for 1.8, you can utilize the fix LexManos implemented a little while ago. Just override the method shouldCauseReequipAnimation() in your classes of the Item type and simply return false.