[1.12.2] Syncing Energy Capability on ItemStacks

[Cadiboo]

I’ve got an Item that has an energy capability that writes its energy amount to not every time it updates (is this the way I should be doing it?) but when energy is added to it by other items in my inventory (Actually Additions Batteries) the energy only updates on the server, not the client. If I drop the item and pick it up again or put it into another inventory (chest, furnace, generator etc) the energy updates to the right amount. However moving it around in my own inventory or using the item doesn’t update the energy.

My code:

 

https://github.com/Cadiboo/WIPTech/blob/master/src/main/java/cadiboo/wiptech/item/ItemEnergy.java > https://github.com/Cadiboo/WIPTech/blob/master/src/main/java/cadiboo/wiptech/item/ItemGun.java > https://github.com/Cadiboo/WIPTech/blob/master/src/main/java/cadiboo/wiptech/item/ItemRailgun113.java

 

Provider

https://github.com/Cadiboo/WIPTech/tree/master/src/main/java/cadiboo/wiptech/provider

(ModularWeaponProvider & ModularWeaponProvider113) neither works

 

What am I doing wrong/ should I be doing differently?

Edited May 15, 2018 by Cadiboo
added code

[Cadiboo]

I am setting my NBT in my getShareTag correctly, and the NBT should just work (deserialise/read automagically) client side. Am I missing something?

[Cadiboo]

Update, I've got it to work, but I want the server to sync the ItemStacks NBT more often, does anyone know what method I should use to achieve this?

 

this is pseudo code of what I'm trying to achieve

 

public void onUpdate(ItemStack stack, World worldIn, Entity entityIn, int itemSlot, boolean isSelected) {

  if(energy.getEnergyStored() != prevEnergyStored)

  entityIn.connection.sendPacket(UPDATE_NBT);

}

[Cadiboo]

Yeah in the player inventory

 

What I’m trying to do:

Sync the energy to the client to display it inside the durability bar. I also don’t want this sync to trigger a re-equip animation OR to stop the ItemStack from being used (the way a bow is used).

 

What I assume I have to do:

Override getShareTag and shouldCauseReequipAnimation (and something else I think as these two aren’t enough to allow the time to continue being used).

Edited October 6, 2018 by Cadiboo