[SOLVED][1.7.10] On block placed method is executed twice?

[DonKresenko]

I'm trying to make whenever I place my block, the item pops out. The problem is, that the method is somehow executed twice. Here's my code

 

@SideOnly(Side.CLIENT)
public void onBlockAdded(World world, int i, int j, int k) {
   	super.onBlockAdded(world, i, j, k);
   	if(!world.isRemote) {
   		float f = 0.7F;
   		double d = world.rand.nextFloat() * f + (1.0F - f) * 0.5D;
   		double d1 = world.rand.nextFloat() * f + (1.0F - f) * 0.2D + 0.6D;
   		double d2 = world.rand.nextFloat() * f + (1.0F - f) * 0.5D;
   		EntityItem pan = new EntityItem(world, i + d, j + d1, k + d2, new ItemStack(Items.blaze_rod, 1));
   		pan.delayBeforeCanPickup = 5;
   		world.spawnEntityInWorld(pan);
   	}
   		System.out.println("executed");
}

[Nephroid]

I'm pretty sure it's because your print statement is both client and server side. If you put it inside the if block, it should only be executed once.

[larsgerrits]

Yes, the method is executed twice: on the server and on the client.

[DonKresenko]

No luck. Even when i put this...

 

if(world.getBlock(i, j, k) == Test.testBlock){...}

 

...it isn't working

[pickaxe_engineer]

Hi

 

I assume your issue is that you marked the method to clientside only.

 

Read this: http://www.minecraftforge.net/forum/index.php/topic,22764.0.html

 

Sincerely -pick

[Nephroid]

No luck. Even when i put this...

 

if(world.getBlock(i, j, k) == Test.testBlock){...}

 

...it isn't working

How does this new piece of code relate to the original code?

 

Anyways, like I said before, if you move the print statement inside the if block, it should only be printed once.

if(!world.isRemote) {
  ...
  print("Hello World");
}

 

Also remove the

SideOnly

annotation, that physically removes code from the client/server (think of it as commenting code out on one side).

[DonKresenko]

Thank you for replaying, but removing the clientside only doesn't fix this. The item is spawned twice again

[DonKresenko]

Any ideas?

[Ernio]

Could you post updated code (or update 1st post), there is just no way this spawns twice if you did as suggested.

[DonKresenko]

This is what i have currently in my block class

 

//@SideOnly(Side.CLIENT)
   	public void onBlockAdded(World world, int i, int j, int k) {
   		
       	super.onBlockAdded(world, i, j, k);
       	if (!world.isRemote && world.getBlock(i,j,k) == Test.testBlock) {
       	float f = 0.7F;
        double d = world.rand.nextFloat() * f + (1.0F - f) * 0.5D;
        double d1 = world.rand.nextFloat() * f + (1.0F - f) * 0.2D + 0.6D;
        double d2 = world.rand.nextFloat() * f + (1.0F - f) * 0.5D;
        EntityItem entityitem = new EntityItem(world, i + d, j + d1, k + d2, new ItemStack(Items.blaze_powder, 1));
        entityitem.delayBeforeCanPickup = 5;
        world.spawnEntityInWorld(entityitem);
   	}
}

 

And, as I said, the items is spawned twice

[larsgerrits]

Is there somewhere you call the

onBlockAdded

method yourself? If so,you shouldn't.

[DonKresenko]

I call this method in my block class, nowhere else

[DonKresenko]

In other word, what should i do in order to my item to dorp whenever the block is placed?

[DonKresenko]

Sorry, I don't quite get it

[DonKresenko]

Oh, wait. Nevermind. I fixed it