Cannot place flowers on valid modded blocks
slowbr0 opened this issue ยท 4 comments
Issue Description:
Flowers from Biomes O' Plenty cannot be placed onto the mulch block from Inspirtations mod. This block is supposed to support valid vanilla and modded plant and flower blocks (excluding saplings) that can be placed onto dirt or grass blocks.
Here is a link to the issue that I first opened with the author of Inspirations:
KnightMiner/Inspirations#121
Steps to reproduce:
- create and place mulch block in world
- harvest (or create) and attempt to place flower (in most recent test I used a pink daffodil) onto mulch block
Additional Information:
- fml-client-latest.log or fml-server-latest.log: -
- Screenshots: -
Affected Versions (Exact numbers, do NOT use "latest"):
- Biomes O' Plenty: 7.0.1
- Minecraft: 1.12.2
- Forge: 14.23.5.2836
- Inspirations: 0.2.4
This is incredibly trivial to fix, so unless you are no longer releasing 1.12 updates I see no reason to not handle it. As I mentioned in the comment on my issue,
Biomes O Plenty bug, they need to return
EnumPlantType.Plains
fromgetPlantType
for their block and callcanSubstainPlant
in their placement checks
We're no longer releasing updates for 1.12 aside from PRs, and if it's so trivial to fix, why not make one yourself?
Well, I looked into how you handle plants, and it seems with how Biomes O Plenty is set up makes it a lot less trivial then I expected, so I stand corrected. Basically, you kinda wrote your own version of the Forge IPlantable in BlockQueries, so I'd essentially need to replace that with the Forge version in its usages to be fully compatible. Thats a bit more then I have the time to accomplish.
Still nothing I can do on my side to fix this as I am already implementing the proper methods for compatibility, so I guess I will just encourage you to use the Forge method in any 1.13 rewrites to ensure compatibility.