[1.7] change item texture problem

[MrRiegel]

hey,

I made a sword with an inventory with one slot.

the texture of the sword depends on the containing item.

@Override
public IIcon getIconFromDamage(int p_77617_1_) {
	if (sword
			.getTagCompound()
			.getTagList(InventoryNevTool.tagName,
					sword.getTagCompound().getId()).getCompoundTagAt(0)
			.getShort("Damage") == 6) {
		return icon_f;
	} else {
		return itemIcon;
	}
}

 

but while the gui is open and changing the item, the sword icon is flickering.

https://drive.google.com/file/d/0Bxk8bebSTRk-LVN5YWhIWDF0RWc/view?usp=sharing

 

how can I stop this?

 

container:

public class ContainerNevTool extends Container {

InventoryNevTool inv;

public ContainerNevTool(EntityPlayer player, InventoryPlayer invPlayer,
		InventoryNevTool inv) {
	this.inv = inv;

	for (int i = 0; i < 1; i++) {
		for (int j = 0; j < 1; j++) {
			addSlotToContainer(new MySlot(inv, j + i * 1, 80 + j * 18,
					48 + i * 18));
		}
	}

	for (int i = 0; i < 3; i++) {
		for (int j = 0; j < 9; j++) {
			addSlotToContainer(new Slot(invPlayer, j + i * 9 + 9,
					8 + j * 18, 84 + i * 18));
		}
	}

	for (int i = 0; i < 9; i++) {
		if (i == invPlayer.currentItem)
			addSlotToContainer(new EvilSlot(invPlayer, i, 8 + i * 18, 142));
		addSlotToContainer(new Slot(invPlayer, i, 8 + i * 18, 142));
	}
}

@Override
public boolean canInteractWith(EntityPlayer player) {
	return inv.isUseableByPlayer(player);
}

@Override
public ItemStack transferStackInSlot(EntityPlayer player, int slot) {
	ItemStack stack = null;
	Slot slotObject = (Slot) inventorySlots.get(slot);

	// null checks and checks if the item can be stacked (maxStackSize > 1)
	if (slotObject != null && slotObject.getHasStack()) {
		ItemStack stackInSlot = slotObject.getStack();
		if (!stackInSlot.getItem().equals(ModItems.cry))
			return null;
		stack = stackInSlot.copy();

		// merges the item into player inventory since its in the tileEntity
		if (slot < inv.getSizeInventory()) {
			if (!this.mergeItemStack(stackInSlot, inv.getSizeInventory(),
					36 + inv.getSizeInventory(), true)) {
				return null;
			}
		}
		// places it into the tileEntity is possible since its in the player
		// inventory
		else if (!this.mergeItemStack(stackInSlot, 0,
				inv.getSizeInventory(), false)) {
			return null;
		}

		if (stackInSlot.stackSize == 0) {
			slotObject.putStack(null);
		} else {
			slotObject.onSlotChanged();
		}

		if (stackInSlot.stackSize == stack.stackSize) {
			return null;
		}
		slotObject.onPickupFromSlot(player, stackInSlot);
	}
	return stack;
}

}

 

inventory:

public class InventoryNevTool implements IInventory {

private ItemStack[] inv;

public static final int INV_SIZE = 1;
public static String tagName = "NevTool";

ItemStack storedInv = null;

public InventoryNevTool(ItemStack stack) {
	inv = new ItemStack[iNV_SIZE];
	this.storedInv = stack;
	if (!storedInv.hasTagCompound()) {
		storedInv.setTagCompound(new NBTTagCompound());
	}
	readFromNBT(storedInv.getTagCompound());
}

public void readFromNBT(NBTTagCompound compound) {
	String key = tagName;
	if (key == null || key.equals("")) {
		return;
	}
	NBTTagList items = compound.getTagList(key, compound.getId());
	for (int i = 0; i < items.tagCount(); ++i) {
		NBTTagCompound item = items.getCompoundTagAt(i);
		byte slot = item.getByte("Slot");
		if (slot >= 0 && slot < getSizeInventory()) {
			inv[slot] = ItemStack.loadItemStackFromNBT(item);
		}
	}

}

public void writeToNBT(NBTTagCompound compound) {
	String key = tagName;
	if (key == null || key.equals("")) {
		return;
	}
	NBTTagList items = new NBTTagList();
	for (int i = 0; i < getSizeInventory(); ++i) {
		if (getStackInSlot(i) != null) {
			NBTTagCompound item = new NBTTagCompound();
			item.setByte("Slot", (byte) i);
			getStackInSlot(i).writeToNBT(item);
			items.appendTag(item);
		}
	}
	compound.setTag(key, items);
}

public static boolean isNevPick(ItemStack stack) {
	return stack != null && stack.getItem() == ModItems.nevpick;
}

@Override
public int getSizeInventory() {
	return inv.length;
}

@Override
public ItemStack getStackInSlot(int slot) {
	return inv[slot];
}

@Override
public ItemStack decrStackSize(int slot, int amt) {
	ItemStack stack = getStackInSlot(slot);
	if (stack != null) {
		if (stack.stackSize <= amt) {
			setInventorySlotContents(slot, null);
		} else {
			stack = stack.splitStack(amt);
			if (stack.stackSize == 0) {
				setInventorySlotContents(slot, null);
			}
		}
	}
	return stack;
}

@Override
public ItemStack getStackInSlotOnClosing(int slot) {
	ItemStack stack = getStackInSlot(slot);
	if (stack != null) {
		setInventorySlotContents(slot, null);
	}
	this.markDirty();
	return stack;
}

@Override
public void setInventorySlotContents(int slot, ItemStack stack) {
	inv[slot] = stack;
	if (stack != null && stack.stackSize > getInventoryStackLimit()) {
		stack.stackSize = getInventoryStackLimit();
	}
	this.markDirty();

}

@Override
public String getInventoryName() {
	return storedInv.getDisplayName();
}

@Override
public boolean hasCustomInventoryName() {
	return false;
}

@Override
public int getInventoryStackLimit() {
	return 1;
}

@Override
public void markDirty() {
	for (int i = 0; i < getSizeInventory(); ++i) {
		if (getStackInSlot(i) != null && getStackInSlot(i).stackSize == 0)
			inv[i] = null;
	}
	writeToNBT(storedInv.getTagCompound());
}

@Override
public boolean isUseableByPlayer(EntityPlayer player) {
	ItemStack stack = player.getCurrentEquippedItem();
	return stack != null
			&& (stack.getItem() == ModItems.nevpick
					|| stack.getItem() == ModItems.nevshovel
					|| stack.getItem() == ModItems.nevaxe || stack
					.getItem() == ModItems.nevsword);
}

@Override
public void openInventory() {
}

@Override
public void closeInventory() {
}

@Override
public boolean isItemValidForSlot(int slot, ItemStack stack) {
	return false;
}

}

[MrRiegel]

the problem is that the nbt changes only on server side while the inventory is open.

only if the inventory is closed the nbt on client side changes too.

I tried to send packets in onUpdate() but it doesn't work.