[1.10.2] {Solved!!!} Right-Click Item for Chest-like GUI

EscapeMC · January 2, 2017

Hello once again Forge forums!

Today I am working on an item that on right click (in hand) it will bring up a GUI for a small-ish chest. I am thinking 9x2 inventory. (I have the GUI)

I know how to make a container, and I am making it while I wait for a response. I also know how to do it in my GuiHandler. I assume the gui itself will be VERY similar (if not the same) to that of a custom chest (I have one already).

My question(s) now,

1) Would the above information I stated be correct?

and 2) How would the tile entity differ? (My currect tile entity for a CHEST is

package com.github.escapemc.thingsmod.tileentity;

import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.EnumFacing;
import net.minecraftforge.common.capabilities.Capability;
import net.minecraftforge.items.CapabilityItemHandler;
import net.minecraftforge.items.ItemStackHandler;

public class TileEntityTestChest extends TileEntity {


private String customName;
protected ItemStackHandler inputSlot;

public TileEntityTestChest() {
	inputSlot = new ItemStackHandler(45);
}

@Override
public NBTTagCompound writeToNBT(NBTTagCompound compound) {
	super.writeToNBT(compound);
	compound.setTag("inputSlot", inputSlot.serializeNBT());

	return compound;
}


@Override
public void readFromNBT(NBTTagCompound compound) {
	inputSlot.deserializeNBT(compound.getCompoundTag("inputSlot"));
}

@Override
public <T> T getCapability(Capability<T> capability, EnumFacing facing) {
	if (capability == CapabilityItemHandler.ITEM_HANDLER_CAPABILITY) {
		this.markDirty();
		if(worldObj != null && worldObj.getBlockState(pos).getBlock() != getBlockType()) {
			return (T) inputSlot;
		}
		if(facing == null) {
			return (T) inputSlot;
		}
		if(facing == EnumFacing.UP) {
			return (T) inputSlot;
		}
		if(facing == EnumFacing.DOWN) {
			return (T) inputSlot;
		}
		if(facing == EnumFacing.NORTH) {
			return (T) inputSlot;
		}
		if(facing == EnumFacing.EAST) {
			return (T) inputSlot;
		}
		if(facing == EnumFacing.WEST) {
			return (T) inputSlot;
		}
		if(facing == EnumFacing.SOUTH) {
			return (T) inputSlot;
		}
	}
	return super.getCapability(capability, facing);
}	

@Override
public boolean hasCapability(Capability<?> capability, EnumFacing facing) {
	return this.getCapability(capability, facing) != null;
}

public void setCustomName(String customName) {
	this.customName = customName;
}



}

so how would it differ?)

EDIT and 3) how would I bind this gui to on a right click open the Gui? with onRightClick or something? What along with that would I need?

larsgerrits · January 2, 2017

1) It is very similar, it's just that you have to adapt your code to not take a position and a

TileEntity

, but an

ItemStack

(see below).

2) If you are making an

Item

with an inventory, you don't use a

TileEntity

. You have to save the inventory inside the

ItemStack

's NBT.

3) Probably

onItemRightClick

.

EscapeMC · January 2, 2017

1) It is very similar, it's just that you have to adapt your code to not take a position and a
TileEntity
, but an
ItemStack
(see below).

2) If you are making an
Item
with an inventory, you don't use a
TileEntity
. You have to save the inventory inside the
ItemStack
's NBT.

3) Probably
onItemRightClick
.

Oh wait duh! Why would I use a tile entity!!! *facepalm* -to myself, 'you idiot, its not a tile, and doesn't need an entity'-

Alright, how would I "save the inventory inside the ItemStack's NBT." Specifically, hwat are you saying with this and how can I do it?

Choonster · January 3, 2017

Use the capability system to attach an

IItemHandler

inventory to an

ItemStack

.

Specifically, override

Item#initCapabilities

to return an

ICapabilityProvider

that provides the

IItemHandler

capability. The

ICapabilityProvider

also needs to implement

INBTSerializable

to read the data from and write the data to NBT.

Forge provides the

ICapabilitySerializable

interface for this purpose: it's simply a shorter way to implement both

ICapabilityProvider

and

INBTSerializable

.

EscapeMC · January 3, 2017

Use the capability system to attach an
IItemHandler
inventory to an
ItemStack
.

Specifically, override
Item#initCapabilities
to return an
ICapabilityProvider
that provides the
IItemHandler
capability. The
ICapabilityProvider
also needs to implement
INBTSerializable
to read the data from and write the data to NBT.

Forge provides the
ICapabilitySerializable
interface for this purpose: it's simply a shorter way to implement both
ICapabilityProvider
and
INBTSerializable
.

(You have an example? I somewhat get it but am slightly confused)

Alright, so instead of making an item, my item needs to be an ItemStack?

This is quite confusing to me... (As stated just above)

larsgerrits · January 3, 2017

Specifically, override
Item#initCapabilities
to return an
ICapabilityProvider
that provides the
IItemHandler
capability. The
ICapabilityProvider
also needs to implement
INBTSerializable
to read the data from and write the data to NBT.

The

IItemHandler

can be the same as the one you would use in a

TileEntity

. The default implementation will do fine (

ItemStackHandler

).

In

Item#initCapabilities

you have a return a

ICapabilityProvider

, for which you want to use

CapabilityItemHandler.ITEM_HANDLER_CAPABILITY

for an inventory.

EscapeMC · January 3, 2017

Specifically, override
Item#initCapabilities
to return an
ICapabilityProvider
that provides the
IItemHandler
capability. The
ICapabilityProvider
also needs to implement
INBTSerializable
to read the data from and write the data to NBT.
The
IItemHandler
can be the same as the one you would use in a
TileEntity
. The default implementation will do fine (
ItemStackHandler
).

In
Item#initCapabilities
you have a return a
ICapabilityProvider
, for which you want to use
CapabilityItemHandler.ITEM_HANDLER_CAPABILITY
for an inventory.

(Again, do you have a reference I can look at to understand this all?)

I looked in the TileEntity Class and couldn't find either ItemStackHandler or IItemHandler. Now, am I being stupid? Wait a sec, *looks in my TileEntity for a chest, sees

	protected ItemStackHandler inputSlot;

public TileEntityTestChest() {
	inputSlot = new ItemStackHandler(45);
}

*

Ok, so is it something like this?

And then, for the NBT, would it be something similar to this?

	@Override
public NBTTagCompound writeToNBT(NBTTagCompound compound) {
	super.writeToNBT(compound);
	compound.setTag("inputSlot", inputSlot.serializeNBT());

	return compound;
}


@Override
public void readFromNBT(NBTTagCompound compound) {
	inputSlot.deserializeNBT(compound.getCompoundTag("inputSlot"));
}

Or am I very far off and using my TileEntity for a chest as a reference a bad idea?

larsgerrits · January 3, 2017

Specifically, override
Item#initCapabilities
to return an
ICapabilityProvider
that provides the
IItemHandler
capability. The
ICapabilityProvider
also needs to implement
INBTSerializable
to read the data from and write the data to NBT.
The
IItemHandler
can be the same as the one you would use in a
TileEntity
. The default implementation will do fine (
ItemStackHandler
).

In
Item#initCapabilities
you have a return a
ICapabilityProvider
, for which you want to use
CapabilityItemHandler.ITEM_HANDLER_CAPABILITY
for an inventory.
(Again, do you have a reference I can look at to understand this all?)

I currently do not. But why won't you just try it, and if you have a problem we can help?

And then, for the NBT, would it be something similar to this?
	@Override
public NBTTagCompound writeToNBT(NBTTagCompound compound) {
	super.writeToNBT(compound);
	compound.setTag("inputSlot", inputSlot.serializeNBT());

	return compound;
}


@Override
public void readFromNBT(NBTTagCompound compound) {
	inputSlot.deserializeNBT(compound.getCompoundTag("inputSlot"));
}
Or am I very far off and using my TileEntity for a chest as a reference a bad idea?

NBT will be saved for you. I was wrong in my first comment, as I didn't even think of

Item

capabilities...

EscapeMC · January 3, 2017

Ok ok ok, one last thing before I start:

Is my item I have created (test_bag) need to be an item, and I make a SEPARATE class of an item stack? Or do I make my item and ItemStack, and put all this tileentity-like code in the test_bag itself?

larsgerrits · January 3, 2017

You make your bag an

Item

.

You don't have to make a custom

ItemStack

class, as the vanilla one handles capabilities for you.

EscapeMC · January 3, 2017

Ok wait, my brain is twisting.

I so far have a very basic item.

package com.github.escapemc.thingsmod.items;

import com.github.escapemc.thingsmod.Reference;

import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;

public class test_bag extends Item {

public test_bag() {



	setUnlocalizedName(Reference.ModItems.TEST_BAG.getUnlocalizedName());
	setRegistryName(Reference.ModItems.TEST_BAG.getRegistryName());

}


}

Where do I start?

Choonster · January 3, 2017

Ok wait, my brain is twisting.

I so far have a very basic item.

package com.github.escapemc.thingsmod.items;

import com.github.escapemc.thingsmod.Reference;

import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;

public class test_bag extends Item {

public test_bag() {



	setUnlocalizedName(Reference.ModItems.TEST_BAG.getUnlocalizedName());
	setRegistryName(Reference.ModItems.TEST_BAG.getRegistryName());

}


}

Where do I start?

Create an implementation of

ICapabilitySerializable

that stores an

IItemHandler

instance (e.g. an

ItemStackHandler

), provides it from the

ICapabilityProvider

methods and reads it from/writes it to NBT in the

INBTSerializable

methods.

Override

Item#initCapabilities

to return a new instance of this class.

EscapeMC · January 3, 2017

Create an implementation of
ICapabilitySerializable
that stores an
IItemHandler
instance (e.g. an
ItemStackHandler
), provides it from the
ICapabilityProvider
methods and reads it from/writes it to NBT in the
INBTSerializable
methods.

Override
Item#initCapabilities
to return a new instance of this class.

Alright... So I implemented ICapabilitySerializable, and it made me add 4 unimplemented methods.

So far, I have edited one, but I am not sure 100% about any of them.

	@Override
public boolean hasCapability(Capability<?> capability, EnumFacing facing) {
	return this.getCapability(capability, facing) != null;
}

@Override
public <T> T getCapability(Capability<T> capability, EnumFacing facing) {
	return null;
}

@Override
public NBTBase serializeNBT() {
	return null;
}

@Override
public void deserializeNBT(NBTBase nbt) {

}

Draco18s · January 3, 2017

Do this:

1) store an
IItemHandler
instance (e.g. an
ItemStackHandler
)

2) provide it from the
ICapabilityProvider
methods and reads it from/writes it to NBT in the
INBTSerializable
methods.

3) Override
Item#initCapabilities
to return a new instance of this class.

EscapeMC · January 3, 2017

Do this:
1) store an
IItemHandler
instance (e.g. an
ItemStackHandler
)

2) provide it from the
ICapabilityProvider
methods and reads it from/writes it to NBT in the
INBTSerializable
methods.

3) Override
Item#initCapabilities
to return a new instance of this class.

1) Did I do this correctly with my inputSlot stuff? EXCEPT for the capabilities? (Pst, how would I do that? Looking back to my chest, I know there are no directions... so just ==null?)

2) Alright, I tried using the NBT stuff from my tileentity for my chest, did not work. I am not sure what to do with this serializeNBT and deserializeNBT.

3) Where and how exactly? I see none of this in my chest TE so I am confused

Animefan8888 · January 3, 2017

Deserialize and Serialize mean to read and to write. Did you use IInventory for your Chest?

EscapeMC · January 3, 2017

Deserialize and Serialize mean to read and to write. Did you use IInventory for your Chest?

I know what Deserialize and Serialize mean, I just didn't know what to do with them.

As to the IInventory part, no I do not believe I did. This was the code for read/write for my TE of my chest:

	@Override
public NBTTagCompound writeToNBT(NBTTagCompound compound) {
	super.writeToNBT(compound);
	compound.setTag("inputSlot", inputSlot.serializeNBT());

	return compound;
}


@Override
public void readFromNBT(NBTTagCompound compound) {
	inputSlot.deserializeNBT(compound.getCompoundTag("inputSlot"));
}

Animefan8888 · January 3, 2017

Then you should be good... could you post all of the code?

EscapeMC · January 3, 2017

Then you should be good... could you post all of the code?

For what? My bag? (and that was an accidental thank you, but have it anyway )

Animefan8888 · January 3, 2017

Then you should be good... could you post all of the code?

For what? My bag? (and that was an accidental thank you, but have it anyway )

OK, and yes.

EscapeMC · January 3, 2017

package com.github.escapemc.thingsmod.items;

import com.github.escapemc.thingsmod.Reference;

import net.minecraft.item.Item;
import net.minecraft.nbt.NBTBase;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.util.EnumFacing;
import net.minecraftforge.common.capabilities.Capability;
import net.minecraftforge.common.capabilities.ICapabilitySerializable;
import net.minecraftforge.items.ItemStackHandler;

13 public class test_bag extends Item implements ICapabilitySerializable {

protected ItemStackHandler inputSlot;

public test_bag() {

	inputSlot = new ItemStackHandler(18);





	setUnlocalizedName(Reference.ModItems.TEST_BAG.getUnlocalizedName());
	setRegistryName(Reference.ModItems.TEST_BAG.getRegistryName());

}

@Override
public boolean hasCapability(Capability<?> capability, EnumFacing facing) {
	return this.getCapability(capability, facing) != null;
}

@Override
public <T> T getCapability(Capability<T> capability, EnumFacing facing) {
	return null;
}

@Override
41	public NBTTagCompound writeToNBT(NBTTagCompound compound) {
42		super.writeToNBT(compound);
	compound.setTag("inputSlot", inputSlot.serializeNBT());

	return compound;
}


@Override
50	public void readFromNBT(NBTTagCompound compound) {
	inputSlot.deserializeNBT(compound.getCompoundTag("inputSlot"));
}

}

This gives me the following errors:

Line 13: The type test_bag must implement the inherited abstract method INBTSerializable.deserializeNBT(NBTBase)

Line 41: The method writeToNBT(NBTTagCompound) of type test_bag must override or implement a supertype method

Line 42: The method writeToNBT(NBTTagCompound) is undefined for the type Item

Line 50: The method readFromNBT(NBTTagCompound) of type test_bag must override or implement a supertype method

I mentioned the lines in the

[code] thing

Animefan8888 · January 3, 2017

package com.github.escapemc.thingsmod.items;

import com.github.escapemc.thingsmod.Reference;

import net.minecraft.item.Item;
import net.minecraft.nbt.NBTBase;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.util.EnumFacing;
import net.minecraftforge.common.capabilities.Capability;
import net.minecraftforge.common.capabilities.ICapabilitySerializable;
import net.minecraftforge.items.ItemStackHandler;

13 public class test_bag extends Item implements ICapabilitySerializable {

protected ItemStackHandler inputSlot;

public test_bag() {

	inputSlot = new ItemStackHandler(18);





	setUnlocalizedName(Reference.ModItems.TEST_BAG.getUnlocalizedName());
	setRegistryName(Reference.ModItems.TEST_BAG.getRegistryName());

}

@Override
public boolean hasCapability(Capability<?> capability, EnumFacing facing) {
	return this.getCapability(capability, facing) != null;
}

@Override
public <T> T getCapability(Capability<T> capability, EnumFacing facing) {
	return null;
}

@Override
41	public NBTTagCompound writeToNBT(NBTTagCompound compound) {
42		super.writeToNBT(compound);
	compound.setTag("inputSlot", inputSlot.serializeNBT());

	return compound;
}


@Override
50	public void readFromNBT(NBTTagCompound compound) {
	inputSlot.deserializeNBT(compound.getCompoundTag("inputSlot"));
}

}

This gives me the following errors:

Line 13: The type test_bag must implement the inherited abstract method INBTSerializable.deserializeNBT(NBTBase)

Line 41: The method writeToNBT(NBTTagCompound) of type test_bag must override or implement a supertype method

Line 42: The method writeToNBT(NBTTagCompound) is undefined for the type Item

Line 50: The method readFromNBT(NBTTagCompound) of type test_bag must override or implement a supertype method

I mentioned the lines in the

[code] thing

Change readFromNBT to deserializeNBT(NBTBase nbt)

Change writeToNBT to serializeNBT()

Don't make you Item implement ICapabilitySerializable make a new class that does, then override initCapabilities in your Item class.

EscapeMC · January 3, 2017

Change readFromNBT to deserializeNBT(NBTBase nbt)

Change writeToNBT to serializeNBT()

Wait can you write out exactly what each thing needs to change to because ti still does not work and I have a feeling that I am just doing 1-2 thing(s) wrong.

Don't make you Item implement ICapabilitySerializable make a new class that does, then override initCapabilities in your Item class.

How should I do this? What should I make this class? (What type?)

Animefan8888 · January 3, 2017

Change readFromNBT to deserializeNBT(NBTBase nbt)

Change writeToNBT to serializeNBT()

Wait can you write out exactly what each thing needs to change to because ti still does not work and I have a feeling that I am just doing 1-2 thing(s) wrong.

Don't make you Item implement ICapabilitySerializable make a new class that does, then override initCapabilities in your Item class.

How should I do this? What should I make this class? (What type?)

I don't think I can get anymore specific than what I said...

And just a class that implements ICapabilitySerializable

And in your ItemBag make a method called initCapabilities()

EscapeMC · January 3, 2017

Change readFromNBT to deserializeNBT(NBTBase nbt)

Change writeToNBT to serializeNBT()

Wait can you write out exactly what each thing needs to change to because ti still does not work and I have a feeling that I am just doing 1-2 thing(s) wrong.

Don't make you Item implement ICapabilitySerializable make a new class that does, then override initCapabilities in your Item class.

How should I do this? What should I make this class? (What type?)

I don't think I can get anymore specific than what I said...

And just a class that implements ICapabilitySerializable

And in your ItemBag make a method called initCapabilities()

Alright. So I've made this class. I called it "TestBagCapabilities"

Now I created initCapabilities in test_bag. Now what?

And the read/write NBT still isn't working..

Sign In

[1.10.2] {Solved!!!} Right-Click Item for Chest-like GUI

Recommended Posts

Top Posters In This Topic

Popular Days

Top Posters In This Topic

Popular Days

Join the conversation

Announcements

Recently Browsing

Posts

Topics

Who's Online (See full list)

Important Information