[1.7.2] Persist TileEntity data when Block is broken

[Shinaka]

Hey guys. I've been trying to suss through most of this without asking any questions on the forums - but I've hit an issue I can't seem to find an answer to. I have a TileEntity that when placed, assigns the placing player as its owner (just a string on the tileentity that gets saved in the nbt). However, I want that data to persist if the player breaks the block (and thus its in their inventory) and later places it. Sort of like how the Tesseracts in TE remember their owner (and other settings), etc. Any ideas?

[TheSparkst3r]

You'll need to use the ItemStack's NBTTagCompound.

 

In your Block's getDrops method you should get the TileEntity. TileYours being your TileEntity

 

TileYours tile = (TileYours)world.getTileEntity(x, y, z)

You should probably also perform a check on the tile to ensure it is instanceof TileYours

Then you need to get the value from the tile. You need to create the getPlayerName method in your tile entity class and insert it into a new stack's nbttag

	@Override
public ArrayList<ItemStack> getDrops(World world, int x, int y, int z, int metadata, int fortune) {
	ArrayList<ItemStack> items = new ArrayList<ItemStack>();

	TileEntity t = world.getTileEntity(x, y, z);

	if (t instanceof TileYours) {
		TileYours tile = (TileYours)t;
		String name = tile.getPlayerName();

		ItemStack stack = new ItemStack(world.getBlock(x, y, z), 1, metadata);
		if (!stack.hasTagCompound()) {
			stack.setTagCompound(new NBTTagCompound());
		}
		stack.getTagCompound().setString("playerName", name);
		items.add(stack);
	}

	return items;

}

 

And in your onBlockPlaced you will need do the opposite and set the name in the tileentity

[Shinaka]

That makes perfect sense! Thanks a ton!

[Shinaka]

Hmm. It seems as though the TileEntity has already been destroyed by the time getDrops is called. Any ideas?

 

You'll need to use the ItemStack's NBTTagCompound.

 

In your Block's getDrops method you should get the TileEntity. TileYours being your TileEntity

 

TileYours tile = (TileYours)world.getTileEntity(x, y, z)

You should probably also perform a check on the tile to ensure it is instanceof TileYours

Then you need to get the value from the tile. You need to create the getPlayerName method in your tile entity class and insert it into a new stack's nbttag

	@Override
public ArrayList<ItemStack> getDrops(World world, int x, int y, int z, int metadata, int fortune) {
	ArrayList<ItemStack> items = new ArrayList<ItemStack>();

	TileEntity t = world.getTileEntity(x, y, z);

	if (t instanceof TileYours) {
		TileYours tile = (TileYours)t;
		String name = tile.getPlayerName();

		ItemStack stack = new ItemStack(world.getBlock(x, y, z), 1, metadata);
		if (!stack.hasTagCompound()) {
			stack.setTagCompound(new NBTTagCompound());
		}
		stack.getTagCompound().setString("playerName", name);
		items.add(stack);
	}

	return items;

}

 

And in your onBlockPlaced you will need do the opposite and set the name in the tileentity

[TheSparkst3r]

Hmm, I was afraid of that. TE uses a tool to dismantle the blocks safely by purposely calling the methods to break the block and saving the tile data in the correct order. You'll have to look around for a method in your block that calls before the tile is deleted so long as you have access to the world and coordinates and able to return the stack to drop.

[DiabolusNeil]

Override the breakBlock() method and put your code in that.

[jabelar]

Yeah, I was just going to post the same thing.  The TileEntity is removed in the block's breakBlock() method.  So you just need to Override that in your custom block where you save the data somehow before the tile entity is removed.

 

Here's the vanilla method:

 

public void breakBlock(World p_149749_1_, int p_149749_2_, int p_149749_3_, int p_149749_4_, Block p_149749_5_, int p_149749_6_)

    {

        if (hasTileEntity(p_149749_6_) && !(this instanceof BlockContainer))

        {

            p_149749_1_.removeTileEntity(p_149749_2_, p_149749_3_, p_149749_4_);

        }

    }

 

 

[TheSparkst3r]

Yeah, that'd work. But you'll also need to make sure the normal drop is disabled or you'll have 2 items dropping.

You'll also need to create an entityitem in the breakBlock method and spawn that in the world manually for your drop.

[Shinaka]

Is there a good example to look at for spawning the entityItem?

 

Yeah, that'd work. But you'll also need to make sure the normal drop is disabled or you'll have 2 items dropping.

You'll also need to create an entityitem in the breakBlock method and spawn that in the world manually for your drop.

[sequituri]

spawnEntityInWorld()

[MrArcane111]

If you look at the BlockContainer class, inside the breakBlock method, there is a removeTileEntity method. My guess is, you could pop your NBT-adding code into there. Of course, there is always events, but if you don't have to use them, then don't.

[delpi]

I'll give you an alternative idea that assumes one bloke per player.

 

On the savenbt method in the tile, check for the owner to be online and save its nbt to the player.  Then you never need to mess with the rest.

[apuppy120]

You'll need to use the ItemStack's NBTTagCompound.

 

In your Block's getDrops method you should get the TileEntity. TileYours being your TileEntity

 

TileYours tile = (TileYours)world.getTileEntity(x, y, z)

You should probably also perform a check on the tile to ensure it is instanceof TileYours

Then you need to get the value from the tile. You need to create the getPlayerName method in your tile entity class and insert it into a new stack's nbttag

	@Override
public ArrayList<ItemStack> getDrops(World world, int x, int y, int z, int metadata, int fortune) {
	ArrayList<ItemStack> items = new ArrayList<ItemStack>();

	TileEntity t = world.getTileEntity(x, y, z);

	if (t instanceof TileYours) {
		TileYours tile = (TileYours)t;
		String name = tile.getPlayerName();

		ItemStack stack = new ItemStack(world.getBlock(x, y, z), 1, metadata);
		if (!stack.hasTagCompound()) {
			stack.setTagCompound(new NBTTagCompound());
		}
		stack.getTagCompound().setString("playerName", name);
		items.add(stack);
	}

	return items;

}

 

And in your onBlockPlaced you will need do the opposite and set the name in the tileentity

 

I have the part that you put in breakBlock(), but what do you put in onBlockPlaced? You said "the opposite" and I don't quite get what the code for that would be.

[apuppy120]

It is pretty much exactly the same as what I currently have.

 

This is the code that I have in the PotentialBlock class in the breakBlock() method.

 

@Override
public void breakBlock(World world, int x, int y, int z, Block block, int p_149749_6_)
{
	TilePotBlock t = (TilePotBlock)world.getTileEntity(x, y, z);
	ArrayList<ItemStack> items = new ArrayList<ItemStack>();
	if (t instanceof TilePotBlock) {
		TilePotBlock tile = (TilePotBlock)t;

		ItemStack stack = new ItemStack(world.getBlock(x, y, z), 1);
		if (!stack.hasTagCompound()) {
			stack.setTagCompound(new NBTTagCompound());
		}
		stack.getTagCompound().setBoolean("potBlockR", tile.red);
		stack.getTagCompound().setBoolean("potBlockG", tile.green);
		stack.getTagCompound().setBoolean("potBlockB", tile.blue);
		stack.getTagCompound().setBoolean("potBlockIR", tile.ir);


		items.add(stack);
		world.spawnEntityInWorld(new EntityItem(world, x, y, z, items.get(0)));
		world.setBlock(x, y, z, Blocks.air);

	}


}

It is deleting the information when I'm placing it back down.