HELLo

    public ItemStack onItemRightClick(ItemStack stack, World world, EntityPlayer player){
            System.out.println(world.isRemote); // true / false
        ++shot;
        if (shot>5){
            System.out.println(world.isRemote); // always true
            player.inventory.consumeInventoryItem(mod.item2.itemID);
            shot = 0;
        }
        System.out.println("shot="+shot);
        return stack;
}

Why world.isRemote is always true?

Looks to me like you made 'shot' both a class field in your Item class and probably 'static' on top of that. Oops on both counts. You should store the number of shots fired in the ItemStack's NBT and your problem with isRemote will probably fix itself.

 

Is that piece of code the only place you use the 'shot' variable?

few days ago similar code was working fine with static shot variable! It's confusing me so hard!!!

Looks to me like you made 'shot' both a class field in your Item class and probably 'static' on top of that. Oops on both counts. You should store the number of shots fired in the ItemStack's NBT and your problem with isRemote will probably fix itself.

 

Is that piece of code the only place you use the 'shot' variable?

 

I'm curious on how you set an itemStacks NBT (this is how you can keep certain information privy to each item stack right?) however I keep getting null exceptions and other types of errors. How does one use this properly? If there is a good tutorial on it please link me.

since no-one has actually answered your question i will

 

onItemRightClick is called by the client only when you right click with an item

the client will always returns true because it is connected to either a local-server task(started by the singleplayer client) or a server

 

 

 

 

 

А я тебе говорил, что хранить переменную надо в NBT

Верно.

thank you! I did it.

MultiMote

OMG! Ты тоже русский! До чего же world.isRemote тесен