I'm trying to simply show a message when a custom event fires.

 

@ForgeSubscribe
public void onPlayerPlaceFurnace(PlayerInteractEvent event) {
	EntityPlayer p = event.entityPlayer;
	if(event.action == Action.RIGHT_CLICK_BLOCK) {
		ItemStack s = p.inventory.getCurrentItem();
		if(s.itemID == Block.furnaceIdle.blockID) {
			this.mc.getNetHandler().getPlayer().addChatMessage("Furnace Placed");
		}
	}
}

 

The problem I'm running into is whenever I actually place a furnace, I see "Furnace Placed" twice in chat. I can't figure out why this fires twice. Is there something else I should be doing?

You are calling it in the Client and the Server.

 

Use

if (!world.isRemote) {

}

I've added isRemote, however that did not help. I've noticed something different in the console. Whenever the event fires this happens:

 

2014-02-11 21:20:06 [iNFO] [Minecraft-Server] Saving and pausing game...
2014-02-11 21:20:06 [iNFO] [Minecraft-Server] Saving chunks for level 'TestCreative'/Overworld
2014-02-11 21:20:06 [iNFO] [Minecraft-Server] Saving chunks for level 'TestCreative'/Nether
2014-02-11 21:20:06 [iNFO] [Minecraft-Server] Saving chunks for level 'TestCreative'/The End
2014-02-11 21:20:11 [iNFO] [Minecraft-Client] [CHAT] Furnace Placed
2014-02-11 21:20:11 [iNFO] [Minecraft-Client] [CHAT] Furnace Placed
2014-02-11 21:20:12 [iNFO] [Minecraft-Server] Saving and pausing game...

 

So it looks like it's actually the client that fires twice.

Show me your code, with the !world.isRemote added.

@ForgeSubscribe
public void onPlayerPlaceFurnaceOld(PlayerInteractEvent event) {
	EntityPlayer p = event.entityPlayer;
	if(event.action == Action.RIGHT_CLICK_BLOCK) {
		ItemStack s = p.inventory.getCurrentItem();
		if(s.itemID == Block.furnaceIdle.blockID) {
			if(!this.mc.theWorld.isRemote) {
				this.mc.getNetHandler().getPlayer().addChatMessage("Furnace Placed");
			}
		}
	}
}

 

In this case, I don't get any chat messages. if I change it to:

 

if(this.mc.theWorld.isRemote) {
				this.mc.getNetHandler().getPlayer().addChatMessage("Furnace Placed");
			}

 

I get the double message again. I don't know if this is worth noting, but I have another event that checks if player breaks a furnace.

 

@ForgeSubscribe
public void onPlayerDestroyFurnace(BreakEvent event) {
	Block b = event.block;
	if(b == Block.furnaceIdle || b == Block.furnaceBurning) {
		this.mc.getNetHandler().getPlayer().addChatMessage("Furnace Destroyed");
	}
}

 

And this displays correctly, single message.

@ForgeSubscribe

public void onPlayerPlaceFurnace(PlayerInteractEvent event) {

EntityPlayer p = event.entityPlayer;

if(event.action == Action.RIGHT_CLICK_BLOCK) {

ItemStack s = p.inventory.getCurrentItem();

if(s.itemID == Block.furnaceIdle.blockID) {

this.mc.getNetHandler().getPlayer().addChatMessage("Furnace Placed");

}

}

}

TO

@ForgeSubscribe

public void onPlayerPlaceFurnace(PlayerInteractEvent event) {

EntityPlayer p = event.entityPlayer;

if(event.action == Action.RIGHT_CLICK_BLOCK) {

ItemStack s = p.inventory.getCurrentItem();

if(s.itemID == Block.furnaceIdle.blockID && !p.worldObj.isRemote) {

p.addChatMessage("Furnace Placed");

}

}

}

Thank you a lot for the help guys! This last piece works perfectly!

 

One more thing. Is there a way to restrict this, so it only works when user places the block? At this moment it also shows "Block Placed" when I use the furnace.

MovingObjectPosition m = new MovingObjectPosition(e.entityPlayer);

if(!e.entityPlayer.worldOb.isRemote && (e.entityPlayer.worldObj.getBlock(m.blockX, m.blockY, m.blockZ) == Block.furnaceIdle ||e.entityPlayer.worldObj.getBlock(m.blockX, m.blockY, m.blockZ) == Block.furnaceActivated)){

//do your stuff

}

 

Edit if you want a smp mod..., send the messages on the furnace destroy event, the same way as i told you to use on the interact event.....

That last piece of code doesn't seem to be working, "m" returns 0 on x, y and z.

My Bad

change the moving object position to

MovingObjectPosition m = Minecraft.getMinecraft().objectMouseOver;

Stop using the Minecraft client on server side, please.

My Bad

change the moving object position to

MovingObjectPosition m = Minecraft.getMinecraft().objectMouseOver;

 

Thank you for all the help. You are awesome!

 

Stop using the Minecraft client on server side, please.

 

I realize this is a simple mistake, but I just started to figure forge out a few days ago. I didn't know that calling addChatMessage would call it on both sides. How do I know when to check for isRemote?

I realize this is a simple mistake, but I just started to figure forge out a few days ago. I didn't know that calling addChatMessage would call it on both sides. How do I know when to check for isRemote?

 

Every time you need to do something that alters the world, or objects within it (and checking not-remote).

Every time you need to do something that is specific to the person-who-is-playing (rendering, input, etc).

 

Also, never ever use Minecraft.getMinecraft() for any reason.

Also, never ever use Minecraft.getMinecraft() for any reason.

Why not? I use it all the time

 

Because you know what it's for.  All these newbs go and do stuff like this:

 

 

function someArbitraryFoo(World world, int x, int y, int z) {
    if(!Minecraft.getMinecraft().getWorld().isRemote) {
        //do stuff
    }
}

 

Because they think that's the only way to get access to a world object.  It's like, "hello, it's passed to you" or "hello, that object contains a reference to its own world object" or "hello, you're coding inside an entity, it has one already."

2 of his classes were using client thing on server side, he was checking with a worldclient instance to see if was instancieded to worldserver

so 99% sure that he does other bad things so i just showed him the most easy way '-'