Crystal-Nest / harvest-with-ease

Don't break crops, right click them!
https://modrinth.com/mod/harvest-with-ease
GNU General Public License v3.0
5 stars 1 forks source link

Harvesting Ropped Tomatoes from Farmer's Delight deletes the block entirely #23

Closed PrettyMuchTara closed 1 year ago

PrettyMuchTara commented 1 year ago

Describe the bug When you rightclick the bottom or middle block of a ropelogged tomato vine, the middle block gets harvested and the vines deleted.

To Reproduce Grow a 2 or 3 tall tomato plant with rope from Farmer's Delight on top (suggest to set the randomTickSpeed to something high), then rightclick any of the vines and the middle or top gets deleted.

Expected behavior All vine blocks stay intake like the bottom one.

Screenshots

Relevant block tag I think (set false for bottom vine without a rope) Inked2023-06-13_23 24 37

Deleted middle section 2023-06-13_23 26 39

Top one stays floating 2023-06-13_23 24 50

System information:

Crystal-Spider commented 1 year ago

Hello! Thank you for the support!
Did you find this issue on any other plant?
Also, is this the only mod causing problems with HWE so far?

Let me know

PrettyMuchTara commented 1 year ago

Hi there, a bigger thank you for making awesome mods and maintaining them :)

I tried out other plants if they can be "ropelogged", but tomatoes are the only ones I could find, so that's Farmer's Delight and Cultural Delight covered. (Cultural Delight has 2 tall corn, but they behave sensibly) Supplementaries flax tho is also 2 blocks tall and items drop when you rightclick the bottom block and it keeps the plant (expected behaviour), but if you rightclick the top block it does the same but it drops nothing (might be a seperate issue tho).

Tested versions: Cultural Delight 0.14.9; Supplementaries 2.3.17

Other than these I haven't found any other issues.

Crystal-Spider commented 1 year ago

I managed to fix the problem with tomato vines.
I asked for some insights for Supplementaries, so that hopefully I can release a new version with both fixes.

Crystal-Spider commented 1 year ago

Both issues are fixed with the latest version (7.0.0.1 for ≥1.20 and 6.0.1.3 for <1.20).