Making a vanilla block drop a second item

[TheCrudNation]

I'm trying to add an item to be dropped by grass. How would I go about this? 1.14

[Ugdhar]

Sounds like you probably want global loot modifiers

https://github.com/MinecraftForge/MinecraftForge/blob/1.14.x/src/test/java/net/minecraftforge/debug/gameplay/loot/GlobalLootModifiersTest.java

[TheCrudNation]

That seems overly complicated just to make an already existing block drop a second item.

[TheCrudNation]

I found something on another post from Forge 1.11.2 

@SubscribeEvent
	public void yourPlayerHarvestEvent(HarvestDropsEvent event) {
		if (event.getHarvester() != null) {
			
			if (event.getState() == Blocks.QUARTZ_ORE.getDefaultState()) {
				event.setDropChance(0.1f);
				event.getDrops().add(new ItemStack(ItemRegistry.herkimer_diamond, 1));
			}
			
		}
	}

Has it really changed that much in so little versions or is there a simpler way of doing it?

[Draco18s]

  

Just now, TheCrudNation said:

HarvestDropsEvent event

Doesn't exist any more. Has been replaced by Global Loot Modifiers

 

If you don't want to use them then do it this way:

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

https://github.com/Draco18s/ReasonableRealism/blob/1.14.4/src/main/resources/data/harderfarming/loot_tables/blocks/grass.json

Edited May 12, 2020 by Draco18s

[TheCrudNation]

I'm quite new to java, how does one make sure the event is fired? I created a class called EventHandler inside of my handlers package and it contains literally just this.

@EventBusSubscriber
public class EventHandler {

}

so anything I put in here will be fired?

[Ugdhar]

More or less.

Info on events here: https://mcforge.readthedocs.io/en/latest/events/intro/

[TheCrudNation]

So I don't need to reference my EventHandler class anywhere else?

[Ugdhar]

1 minute ago, TheCrudNation said:

So I don't need to reference my EventHandler class anywhere else?

As long as it's got the EventBusSubscriber annotation and the event handler methods are static, and have the SubscribeEvent annotation, you should be good to go.

[Draco18s]

6 hours ago, diesieben07 said:

Can you stop recommending this crappy way to people?

Sure can.