I am subscribing the event BlockEvent.HarvestDropsEvent event and was wondering how make a block, like default stone, drop granite instead.  This is my code.

 

 

@SubscribeEvent

public void onDrop(BlockEvent.HarvestDropsEvent event){

if (event.state.getBlock() == Blocks.stone)

event.drops.set(0, new ItemStack((Block) Blocks.stone.getStateFromMeta(1).getValue(BlockStone.VARIANT)));

}

 

 

Thanks.

I wouldn't use "#set(...) but whatever (use #clear() and then #add(...)) (not necessarily)

 

As to problem - you are now only comparing Block - you also need to check if the meta of blockState is stone - meta=0;

 

Then indeed add new ItemStack, with one change:

Item.getItemFromBlock(the block you want) + the meta (like you did).

I tried it, but feel like I didn't do something right because I'm getting an exception.

 

 

@SubscribeEvent

public void onDrop(BlockEvent.HarvestDropsEvent event){

if (event.state.getBlock() == Blocks.stone){

event.drops.clear();

event.drops.add(new ItemStack(Item.getItemFromBlock((Block) Blocks.stone.getStateFromMeta(1).getValue(BlockStone.VARIANT))));

}

}

 

 

As for the error, I debugged it and it throws an exception on the "event.drops.add" method.

event.drops.add(new ItemStack(Item.getItemFromBlock(Blocks.stone), 1, 1)); // 3rd is meta

 

Don't make this harder than it should be. I don't get why people go with states - numbers are as nice as they get.

If you really want to extract meta its .getMetaFromState (other way around).

Common mistake:p  Overthinking thingsXD

 

Thank you for the help