Hunger Overhaul with AgriCraft
InfinityRaider opened this issue ยท 6 comments
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
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.
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.
@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