[1.7.10] IInventory openInventory() and closeInventory() not working.

[Manslaughter777]

It seems that the methods openInventory() and closeInventory() are not working. I have made it print when i open the inventory but it doesnt print anything....

 

The IInventory class:

 

 

package com.manslaughter777.moondimension.inventory;

import com.manslaughter777.moondimension.Main;
import com.manslaughter777.moondimension.items.ItemToolUpgrade;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.inventory.IInventory;
import net.minecraft.item.ItemPickaxe;
import net.minecraft.item.ItemStack;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.nbt.NBTTagList;
import net.minecraftforge.common.util.Constants;

public class InventoryTool implements IInventory {
private String name;

public static boolean check;

/** Defining your inventory size this way is handy */
public static final int INV_SIZE = 4;

/**
 * Inventory's size must be same as number of slots you add to the Container
 * class
 */
private ItemStack[] slots = new ItemStack[iNV_SIZE];

/** Provides NBT Tag Compound to reference */
public final ItemStack invItem;

/**
 * @param itemstack
 *            - the ItemStack to which this inventory belongs
 */
public InventoryTool(ItemStack stack) {
	this.invItem = stack;

	this.name = stack.getDisplayName();

	if (!stack.hasTagCompound()) {
		stack.setTagCompound(new NBTTagCompound());
	}

	readFromNBT(stack.getTagCompound());
}

public void checkContents() {
	if(this.getStackInSlot(0) != null && this.getStackInSlot(0).getItem() == Main.autoRepair) {
		check = true;
	} else check = false;

	System.out.println("YO");
}

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

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

@Override
public ItemStack decrStackSize(int slot, int amount) {
	ItemStack stack = getStackInSlot(slot);
	if (stack != null) {
		if (stack.stackSize > amount) {
			stack = stack.splitStack(amount);

			markDirty();
		} else {
			setInventorySlotContents(slot, null);
		}
	}
	return stack;
}

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

@Override
public void setInventorySlotContents(int slot, ItemStack itemstack) {
	this.slots[slot] = itemstack;

	if (itemstack != null && itemstack.stackSize > this.getInventoryStackLimit()) {
		itemstack.stackSize = this.getInventoryStackLimit();
	}


	markDirty();
}

@Override
public String getInventoryName() {
	return name;
}

@Override
public boolean hasCustomInventoryName() {
	return name.length() > 0;
}

@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)
			slots[i] = null;
	}

	writeToNBT(invItem.getTagCompound());
}

@Override
public boolean isUseableByPlayer(EntityPlayer player) {

	return player.getHeldItem() == invItem;
}

@Override
public void openInventory() {
	checkContents();

	System.out.println("YEAH");
}

@Override
public void closeInventory() {
	checkContents();
}

/**
 * This method doesn't seem to do what it claims to do, as items can still
 * be left-clicked and placed in the inventory even when this returns false
 */
@Override
public boolean isItemValidForSlot(int slot, ItemStack itemstack) {

	return itemstack.getItem() instanceof ItemToolUpgrade;
}

/**
 * A custom method to read our inventory from an ItemStack's NBT compound
 */
public void readFromNBT(NBTTagCompound compound) {
	NBTTagList items = compound.getTagList("ItemInventory", Constants.NBT.TAG_COMPOUND);

	for (int i = 0; i < items.tagCount(); ++i) {
		NBTTagCompound item = (NBTTagCompound) items.getCompoundTagAt(i);
		byte slot = item.getByte("Slot");

		if (slot >= 0 && slot < getSizeInventory()) {
			slots[slot] = ItemStack.loadItemStackFromNBT(item);
		}
	}
}

/**
 * A custom method to write our inventory to an ItemStack's NBT compound
 */
public void writeToNBT(NBTTagCompound tagcompound) {
	NBTTagList items = new NBTTagList();

	for (int i = 0; i < getSizeInventory(); ++i) {

		if (getStackInSlot(i) != null) {

			NBTTagCompound item = new NBTTagCompound();
			item.setInteger("Slot", i);

			getStackInSlot(i).writeToNBT(item);

			items.appendTag(item);
		}
	}

	tagcompound.setTag("ItemInventory", items);
}
}

 

 

[Abastro]

It wouldn't be called automatically unless you call it yourself.

 

So.. What is the purpose of the InventoryTool class? If it is from some container,

you should call openInventory/closeInventory there.

[Manslaughter777]

It wouldn't be called automatically unless you call it yourself.

 

So.. What is the purpose of the InventoryTool class? If it is from some container,

you should call openInventory/closeInventory there.

Yes it is from a container. My mistake, thought is was called automatically

Thnx