Jump to content

[1.13.2] Tile Entity items not saved


MyRedAlien43

Recommended Posts

So, I made a tile entity that stores items, it works and all of that but the only problem is that it doesn't save the contents. My read and write methods:

@Override
public void read(NBTTagCompound compound) {
	super.read(compound);
  	this.handler = new ItemStackHandler(this.getSizeInventory());
  	this.handler.deserializeNBT(compound.getCompound("inventory"));
  	if(compound.contains("CustomName", 8)) {
    	this.customName = ITextComponent.Serializer.fromJson(compound.getString("CustomName"));
    }
}

@Override
public NBTTagCompound write(NBTTagCompound compound) {
	super.write(compound);
	compound.setTag("inventory", this.handler.serializeNBT());
	if(this.hasCustomName()) {
		compound.setString("CustomName", 	ITextComponent.Serializer.toJson(customName));
	}
    return compound;
}

(What the handler is:)

public ItemStackHandler handler = new ItemStackHandler(9);

And I get no error. After I rejoin the items I put in just disappear like they were never saved (or loaded).

Edited by MyRedAlien43
Link to comment
Share on other sites

4 minutes ago, diesieben07 said:

That means you are implementing IInventory. Don't.

 

Show where you open the GUI of your block.

Yea, I meant I now extend only TileEntity.

@Override
	public boolean onBlockActivated(IBlockState state, World worldIn, BlockPos pos, EntityPlayer player, EnumHand hand, EnumFacing side, float hitX, float hitY, float hitZ) {
		if (worldIn.isRemote) {
			return true;
		} else {
			TileEntity tileentity = worldIn.getTileEntity(pos);
			if (tileentity instanceof TileEntityBarrel && player instanceof EntityPlayerMP) {
				TileEntityBarrel barrel = (TileEntityBarrel)tileentity;
				NetworkHooks.openGui((EntityPlayerMP)player, barrel);
			}

			return true;
		}
	}

I don't know if this is correct. Also, my block does NOT extend BlockContainer

Edited by MyRedAlien43
Link to comment
Share on other sites

1 minute ago, diesieben07 said:

implementation of IInteractionObject (your tile entity, judging from your code above).

@Override
    public Container createContainer(InventoryPlayer playerInventory, EntityPlayer playerIn) {
        return new ContainerBarrel(playerInventory, this, playerIn);
    }

    @Override
    public String getGuiID() {
        return "moresimplestuff:barrel";
    }

 

1 minute ago, diesieben07 said:

Show where you registered your Gui factory

IGuiHandler?

Link to comment
Share on other sites

2 minutes ago, diesieben07 said:

IGuiHandler is not used in 1.13.2.

Your GUI factory is registered through ModLoadingContext#registerExtensionPoint with ExtensionPoint.GUIFACTORY.

Ohh that makes sense. Mind showing me how to do so? Sorry, 1.13.2 is new to me.

Edited by MyRedAlien43
Link to comment
Share on other sites

You should take a careful read of the 1.12 -> 1.13/1.14 update primer linked at the top of this forum.  In particular, this comment will be helpful to you: https://gist.github.com/williewillus/353c872bcf1a6ace9921189f6100d09a#gistcomment-2870154

 

Regarding your original problem (TE data not saving), are you marking the tile entity dirty when the inventory changes?  If not, your changes won't get saved. Best bet is to override the onSlotChanged() method of your ItemStackHandler object to call te.markDirty().

  • Thanks 1
Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.