Help with tile entities and onBlockActivated();

[AskHow1248]

I have a tile entity that holds a EMSLiquidStack (basically a LiquidStack but with my own liquid) and I'm trying to get the block for the tile entity to chat the liquid and the amount of liquid it contains.  The only problem is when I right click on the block it seems to call onBlockActivated() twice.  Any help would be appreciated.

 

Code:

 

 

Block class:

package QuantumMagic.Blocks;

import QuantumMagic.Core.CreativeTabQuantumMagic;
import QuantumMagic.ElectroMagneticSuspension.TileEntityWithEMSTank;
import QuantumMagic.TileEntities.TileEntityEMSPipe;
import QuantumMagic.TileEntities.TileEntityEMSTank;
import net.minecraft.block.BlockContainer;
import net.minecraft.block.material.Material;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.ChatMessageComponent;
import net.minecraft.world.IBlockAccess;
import net.minecraft.world.World;

public class BlockWithEMSTank extends BlockContainer {

Class <? extends TileEntityWithEMSTank> Tile;

public BlockWithEMSTank(int par1, Class <? extends TileEntityWithEMSTank> tile, String s) {
	super(par1, Material.iron);
	Tile = tile;
	setUnlocalizedName(s);
	setCreativeTab(CreativeTabQuantumMagic.tabQuantumMagic);
}

@Override
public TileEntity createNewTileEntity(World world) {
	try {
		return Tile.newInstance();
	} catch (InstantiationException e) {
		e.printStackTrace();
		return null;
	} catch (IllegalAccessException e) {
		e.printStackTrace();
		return null;
	}
}

@Override
public int isProvidingWeakPower(IBlockAccess world, int x, int y, int z, int side)
{		

	TileEntityEMSPipe tile = (TileEntityEMSPipe) world.getBlockTileEntity(x, y, z);

	if(tile == null)
		return 1;


	return tile.getAmount() > 0 ? 15 : 0;
}


public boolean onBlockActivated(World world, int x, int y, int z, EntityPlayer player, int par6, float pX, float pY, float pZ)
    {

	TileEntityWithEMSTank tank = (TileEntityWithEMSTank)world.getBlockTileEntity(x, y, z);

	if(tank != null && tank.getLiquidStack() != null && tank.getLiquid() != null)
		player.sendChatToPlayer(ChatMessageComponent.createFromTranslationKey("This EMS tank contains " + tank.getAmount() + "   " units of " + tank.getLiquid().getName()));

	System.out.println("This EMS tank contains " + tank.getAmount() + " units of " + tank.getLiquid().getName());



        return true;
    }

}

 

TileEntity:

package QuantumMagic.ElectroMagneticSuspension;

import QuantumMagic.ElectroMagneticSuspension.EMSLiquids.EMSLiquid;
import QuantumMagic.ElectroMagneticSuspension.EMSLiquids.EMSLiquids;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.tileentity.TileEntity;

public abstract class TileEntityWithEMSTank extends TileEntity implements IEMSTank {

protected EMSLiquidStack liquStack;

@Override
public boolean canFill(EMSLiquidStack stack) {
	return this.getLiquid().equals(stack.getLiquid());
}

@Override
public EMSLiquid getLiquid() {

	if(getLiquidStack() == null)
		return null;

	return getLiquidStack().getLiquid();
}

@Override
public int getAmount() {
	if(getLiquidStack() != null){
		if(getLiquidStack().getAmount() < 0)
			getLiquidStack().incremtAmount(-getLiquidStack().getAmount());

		return getLiquidStack().getAmount();
	}
	return 0;
}

@Override
public EMSLiquidStack getLiquidStack() {
	return liquStack;
}

@Override
public EMSLiquidStack fill(EMSLiquidStack stack) {
	if(stack != null && stack.getLiquid() != null){
		if(liquStack != null){
			if(!stack.getLiquid().equals(this.getLiquid()))
				return stack;


			while(getAmount() < getCapacity() && stack.getAmount() > 0){
				getLiquidStack().incremtAmount(1);
				stack.incremtAmount(-1);
			}

		} else if(liquStack == null || getLiquid() != null || this.getAmount() == 0){

			liquStack = new EMSLiquidStack(stack.getLiquid(), 0);

			while(getAmount() < getCapacity() && stack.getAmount() > 0){
				getLiquidStack().incremtAmount(1);
				stack.incremtAmount(-1);
			}

		}

	}
	return stack;

}

@Override
public EMSLiquidStack take(int wanted) {

	if(getLiquidStack() == null || getLiquid() == null){
		return null;
	}

	if(getAmount() == 0)
		return new EMSLiquidStack(getLiquid(), 0);

	EMSLiquidStack stack = new EMSLiquidStack(getLiquid(), 0);

	while(getAmount() > 0){
		stack.incremtAmount(1);
		getLiquidStack().incremtAmount(-1);
	}

	return stack;
}

public void fillFrom(IEMSTank tank) {

	if(tank != null && tank.getLiquidStack() != null && tank.getLiquid() != null){
		if(this.getLiquidStack() == null || this.getLiquid() == null || this.getAmount() == 0){

			liquStack = new EMSLiquidStack(tank.getLiquid(), 0);
			while(getAmount() < getCapacity() && tank.getAmount() > 0){
				liquStack.incremtAmount(1);
				tank.getLiquidStack().incremtAmount(-1);
			}

		}else{

			if(!this.getLiquid().equals(tank.getLiquid()))
				return;

			while(getAmount() < getCapacity() && tank.getAmount() > 0){
				liquStack.incremtAmount(1);
				tank.getLiquidStack().incremtAmount(-1);
			}

		}

	}

}

@Override
public void writeToNBT(NBTTagCompound compound){
	super.writeToNBT(compound);

	compound.setInteger("Amount", this.getAmount());
	compound.setInteger("LiquidId", this.getLiquid().getID());

}

public void readFromNBT(NBTTagCompound compound)
{
	super.readFromNBT(compound);

	liquStack = new EMSLiquidStack(EMSLiquids.getLiquidFromId(compound.getInteger("LiquidId")) , compound.getInteger("Amount"));


}



}

 

 

 

 

 

[CJLetsGame]

That's because it is called once on the client side and again on the server side.

[AskHow1248]

So I need to use world.isRemote, but which side do I want?

[CJLetsGame]

You dont want SideOnly, but you want to run the code client side only since you want to print the chat message.

 

if (world.isRemote()) //This is the client
{
//Code Here
}

[AskHow1248]

Okay.  Thank you!