[SOLVED] BlockActivated method being ran twice

[ThatBenderGuy]

public boolean onBlockActivated(World par1World, int x, int y, int z, EntityPlayer player, int par6, float par7, float par8, float par9)
    {
    	tentity.switchActivated(); //tentity is my tileentity. The method will switch a boolean to false or true
    	player.addChatMessage("[" + tentity.getActivated() + "]");
        return true;
    }

 

When I test this code in game and right click my block, it returns

"[true]

[false]"

 

For some reason it's being ran twice. Anyone know a work around?

 

EDIT:

Figure it out. Had to put my code inside an if statement like this

if (par1World.isRemote)
        {
            return true;
        }else{
            //Code goes here
        }

 

[Vogner]

Would anybody mind elaborating a little on why that works? I'm not sure why it would even be necessary, tbh. As of right now I have zero experience with server modding.

[Alex_hawks]

It's being run on both the server and the client...

That

if (par1World.isRemote)
        {
            return true;
        }else{
            //Code goes here
        }

is checking if it is on the client, and if it is, return true. If it isn't, it is actually doing stuff...

[Vogner]

I suppose my next question is... Why?

[Alex_hawks]

Because of the unified code that forge has implemented (Which IMHO is a very good thing). If you just used ModLoader you would have to maintain to separate code-bases, but wouldn't run into this issue.

[Vogner]

That makes a certain amount of sense. Thanks for the rundown, duder.