[1.14.4] Event not getting registered

[cinsiian 1]

@EventBusSubscriber
public class Handler 
{
	@SubscribeEvent()
	public static void drop(HarvestDropsEvent event)
	{
		Block block = event.getState().getBlock();
		Random rand = new Random();
		
		if(block instanceof TallGrassBlock && !event.isSilkTouching())
		{
			int gen = 1 + rand.nextInt(100);
			int extra = 1 + rand.nextInt(50);
			if(gen < 51) {event.getDrops().add(new ItemStack(Items.PLANT_FIBER));}
			if(extra < 14) {event.getDrops().add(new ItemStack(Items.PLANT_FIBER));}
		}
		
		if((block == Blocks.GRASS || block == Blocks.DIRT || block == Blocks.COARSE_DIRT || block == Blocks.PODZOL || block == Blocks.MYCELIUM) && !event.isSilkTouching())
		{
			int gen = 1 + rand.nextInt(100);
			int extra = 1 + rand.nextInt(50);
			if(gen < 21) {event.getDrops().add(new ItemStack(Items.PLANT_FIBER));}
			if(extra < 14) {event.getDrops().add(new ItemStack(Items.PLANT_FIBER));}
		}
	}
}

Okay basically this event is not getting registered, whats the problem?

[cinsiian 1]

Do I have to register it via MinecraftForge.EVENT_BUS ?

[JimiIT92 19]

Look strange, the code for handling the event seems ok, it could be a bug with the latest mdk version

[cinsiian 1]

I am using the version 28.1.10

[Draco18s 2078]

5 hours ago, cinsiian said:

HarvestDropsEvent

That event does not exist on 1.14.

Well, the class does, but it isn't fired. 

https://github.com/MinecraftForge/MinecraftForge/pull/5871

[cinsiian 1]

20 hours ago, Draco18s said:

https://github.com/MinecraftForge/MinecraftForge/pull/5871

I cannot find any of these events in my forge

[Draco18s 2078]

8 minutes ago, cinsiian said:

I cannot find any of these events in my forge

No shit, Sherlock, that pull request is still open. The entire reason I linked it is because the problem you're having is because that event does not exist and there's why.

[cinsiian 1]

oh

[cinsiian 1]

So at the moment there is not an event similiar to HarvestDropsEvent?

[Draco18s 2078]

No, because it bypasses the loot table system.

That's why that PR exists and isn't merged, see Lex's comment.

[cinsiian 1]

Is it possible to replace the loottable file replacing it in data/minecraft/loot_tables?

[Draco18s 2078]

Yes, but keep in mind that replacement is "whichever mod loads last."

[cinsiian 1]

Okay, I will do that for now. In the future if they will add the events i will use them.

[cinsiian 1]

otherwise does a MinecraftForge.addGrassSeed void exists in 1.14.4?

[Draco18s 2078]

No. Use loot tables.

https://github.com/Draco18s/ReasonableRealism/blob/1.14.4/src/main/java/com/draco18s/harderfarming/EventHandlers.java#L52

Edited October 21 by Draco18s