AgriCraft

AgriCraft

30M Downloads

Village Crop Support

WolfieWaffle opened this issue ยท 9 comments

commented

This is minor, but right now, if you have vanilla farming disabled, the village farms still don't use crops. Once a plant has been broken in a village with vanilla farming disabled, the only way to re-plant it is using crops.

commented

And why is that a problem?

commented

Disabling farm generation could have interesting effects on hardcore
packs... I don't know If it would use that option, but I think you should
add both if possible, additional configuration can almost never go wrong :)

On Wed, Jun 24, 2015 at 4:26 PM, InfinityRaider [email protected]
wrote:

So since the greenhouses are fixed now, what about a config option to
disable the generation of vanilla farms in general instead of replacing
them with a custom field.
Or I could do both...

โ€”
Reply to this email directly or view it on GitHub
https://github.com/InfinityRaider/AgriCraft/issues/237#issuecomment-115030323
.

commented

It's not a major problem, I said it was minor. It's just a minor
enhancement that could be added if vanilla farming is disabled, since
village farms use vanilla farming.

On Wed, Jun 24, 2015 at 1:24 PM, InfinityRaider [email protected]
wrote:

And why is that a problem?

โ€”
Reply to this email directly or view it on GitHub
https://github.com/InfinityRaider/AgriCraft/issues/237#issuecomment-114986106
.

commented

But what exactly do you want me to add because I'm really not following?

commented

I was just saying maybe add a config option to set village farms to use
AgriCraft crops rather than vanilla, but now I'm thinking it might be
incompatible with harvestcraft+hunger overhaul, since those mods add other
plants to village farms.

On Wed, Jun 24, 2015 at 2:54 PM, InfinityRaider [email protected]
wrote:

But what exactly do you want me to add because I'm really not following?

โ€”
Reply to this email directly or view it on GitHub
https://github.com/InfinityRaider/AgriCraft/issues/237#issuecomment-115009570
.

commented

It should be possible though, I'll look into it

commented

So since the greenhouses are fixed now, what about a config option to disable the generation of vanilla farms in general instead of replacing them with a custom field.
Or I could do both...

commented

@InfinityRaider the ugliest response following: make it configurable :-)

commented

I've decided not to do it, replacing the Vanilla gardens won't be a problem, but there are other mods out there that add/replace the gardens as well (HarvestCraft, PMP, BoP, ...).
And I don't want to have different cases for all of them.