Farmer's Delight

Farmer's Delight

95M Downloads

[1.19.2]: Wild Cabbage crop feature checks for vanilla sand block, instead of sand tag

ManaStar opened this issue ยท 3 comments

commented

Description

As the title says, the wild cabbage crop checks for minecraft's sand block below it for placement instead of the sand block tag. This is inconsistent with the sandy shrub feature which checks for the sand tag.

Steps to reproduce

No response

Mod list

N/A

Logs

No response

Forge

  • I am using the Forge version of Farmer's Delight

Minimal instance

  • I have tested this on a minimal instance

Performance and shader mods

  • I have tested this without performance or shader mods
commented

I think that I originally did it like that because it's a beach plant, so it could never spawn on Red Sand. But some mods might introduce modded beches with custom sand, so this would make sense.

Will take a look later.

commented

Thinking a bit more, yeah, it doesn't make much sense to be so restrictive on which sands a beach crop can spawn in, since they check for the biome as well. I'll change the feature to use tags in a future patch to 1.19 and 1.20 soon. ๐Ÿ‘

commented

Updated on commit 7879cef, should be out for 1.2.2 later.