I've been trying to change the onHarvestBlock event to change what sand and red sand drops when you mine it, normal sand works fine but i cant get red sand to drop the proper item.

 

Here is my code

@Mod.EventBusSubscriber
public class PocketSandEventHandler {
	
    
     
	@SubscribeEvent
	public void onHarvestBlock(BlockEvent.HarvestDropsEvent event)
	{
		final EntityPlayer PLAYER = event.getHarvester();
		if(null == PLAYER || null == PLAYER.getHeldItemMainhand()) return;

		if(ConfigHandler.overridesanddrops == true) {
		
		if(event.getState().getBlock() == Blocks.SAND)
		{
			event.getDrops().add(new ItemStack(ItemInit.sandpile, 4));
			event.getDrops().remove(0);
			
            
			
		}

		}
		
		if(ConfigHandler.overrideredsanddrops == true) {
			
			   if(event.getState().getBlock() == Blocks.SAND.getStateFromMeta(1)) {
					
					event.getDrops().add(new ItemStack(ItemInit.redsandpile, 4));
					event.getDrops().remove(1);
				}
		}
		
		
	}
	}
	

 

Edited October 19, 20186 yr by Trhod177

if(event.getState().getBlock() == Blocks.SAND.getStateFromMeta(1)) {

Block != IBlockState. ever

30 minutes ago, Trhod177 said:

event.getDrops().remove(1);

How do you know there are 2 items in the list. how do you even know that the list isnt empty?

 

31 minutes ago, Trhod177 said:

@Mod.EventBusSubscriber

Static subscriber annotation but 

 

32 minutes ago, Trhod177 said:

@SubscribeEvent public void onHarvestBlock(BlockEvent.HarvestDropsEvent event)

non-static subscriber methods?? why??

what do you mean by 

35 minutes ago, Cadiboo said:

Block != IBlockState. ever

 

Edited October 19, 20186 yr by Trhod177

2 hours ago, Trhod177 said:

event.getState().getBlock()

Returns an instance of the Block class.

2 hours ago, Trhod177 said:

Blocks.SAND.getStateFromMeta(1)

Returns an instance of the IBlockState class.

 

A block is never an IBlockState and an IBlockState is never a Block.

Ok then how do distinguish between sand and redsand

8 minutes ago, Trhod177 said:

Ok then how do distinguish between sand and redsand

You must compare the states. Or the properties of the IBlockState.

Something like if( state == Blocks.SAND.getDefaultState().withProperty( whateverPropertySandHasThatCausesItToBeRed, whateverValueMakesSandBeRed)) //I’m a red sand block!