Hunger Overhaul

Hunger Overhaul

9M Downloads

Hunger Overhaul with AgriCraft

InfinityRaider opened this issue ยท 6 comments

commented

Hello,
I'm the author of a mod (still very beta) AgriCraft (https://github.com/InfinityRaider/AgriCraft), summarized it's basicly IC2 crops standalone, and for almost every seed/crop out there.

Now the problem is some people want to run Pam's Harvestcraft together with Hunger Overhaul and Spice of Life (which I personally do too) and then throw in AgriCraft to get the different crops trough crossbreeding (instead of random grass drops). I have my own harvesting mechanic and your BlockEvent.HarvestDropsEvent handler is overriding this, causing to drop nothing.

So I could either change my code or you could add in an exception like you did for Extra Utilities ender lilly seeds (the unlocalized name for my crop block is: AgriCraft:crops). I would like to hear your thoughts on this.

~InfinityRaider

commented

Why don't you use the Forge event?

commented

I've fixed it on my side

commented

Related: #49

It'd be good to have an IMC that would allow mods to blacklist their classes and whatnot. Forcing mods to use Forge events for their own blocks is pretty lame.

commented

I'm adding it in right now, but in my opinion using the event isn't optimal, everytime a player harvests a block, I'm checking if it's my block. So I just put it in my block class at first.
I'll just check if HO is installed and if it is I'll use the event and if it is not I'll use the method in my block class.

commented

@InfinityRaider, the next version of HO will allow you to use an IMC to blacklist your crops if you want to do that instead.

See: https://github.com/progwml6/HungerOverhaul/wiki#for-mod-developers

commented

That's probably way better then using the event, thanks.