Harvest with ease

Harvest with ease

15M Downloads

2 high plants in Simple Farming are harvested incorrectly

SimonMeskens opened this issue ยท 8 comments

commented

Describe the bug
Simple Farming has several two-high plants which are broken.

To Reproduce
Harvesting the top doesn't do anything, and harvesting the bottom will result in the top staying where it is, while the bottom gets harvested. Punching the top will then destroy the bottom.

Expected behavior
Probably the same as with two-high crops from other mods.

Screenshots
image

System information:

  • Mod loader: Forge
  • Mod version: 6.0.1.4
  • Minecraft version: 1.19.2
commented

Since v9.0.0.3-beta it's possible to add either a configuration option or a datapack to blacklist certain crops from interacting with this mod.
This can "solve" any incompatibility by blacklisting the crops that do not work.
Still, to add proper compatibility, the other mod would need to implement HWE events.

Closing this for now.

commented

To speed up my work, could you please provide a list of all the two blocks tall crops?
Thank you in advance, I'd be doing that myself but I'm quite busy lately.

commented

Sure! These seem to be the ones:

I also tested the grapes and they seem to work fine.

simplefarming:corn
simplefarming:corn_top
simplefarming:kenaf
simplefarming:kenaf_top
simplefarming:sorghum
simplefarming:sorghum_top
commented

Ah, I found one more (the grapes work differently than everything else):

simplefarming:grape_trunk
simplefarming:grape_trunk_top

These ones you can just disable, since they don't grow crops, the actual crops grow on simplefarming:grapevine, but that block harvests fine.

commented

Thank you for your help!
I will most likely tackle this issue by the end of September!

commented

No problem! I'm not blocked by this issue, thanks for the quick response.

commented

I haven't forgotten about this, I'm just more busy then I thought I'd be.
I will soon start working on it!

commented

So, I looked more into this issue and it seems there's not much I can do.
Simple Farming would need to change how its tall crops are coded to resemble more other mods like Supplementaries and Immersive Engineering, otherwise I would need to create hard-coded features just for Simple Farming, but that would go against this mod being general.

I am opening another issue on Simple Farming GitHub on your and my behalf.