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");
}

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.

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

No luck. Even when i put this...

 

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

 

...it isn't working

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

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).

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

Any ideas?

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

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

Is there somewhere you call the

onBlockAdded

method yourself? If so,you shouldn't.

I call this method in my block class, nowhere else

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

Sorry, I don't quite get it

Oh, wait. Nevermind. I fixed it