ForestryMC / Binnie

Minecraft addons for the Forestry mod. Extra Bees, Extra Trees, Genetics, and more.
https://minecraft.curseforge.com/projects/binnies-mods
Other
69 stars 77 forks source link

Binnie's Drinks & Spice Of Life - Interaction Issue #613

Open Danekjovax opened 4 years ago

Danekjovax commented 4 years ago

Version Info:

modpack-Dungeons Dragons & Space Shuttles v.6.3 forge-1.12.2-14.23.5.2807-universal binnie-mods-1.12.2-2.5.1.188 SpiceOfLife-mc1.12-1.3.12 solcarrot-1.12.2-1.8.3

I'm not sure where the root cause of this issue resides, so I'll put the issue to both mods' and let you both discuss.

I recently brewed up some Binnie's Drinks, and looking at the tooltip for one of them I noticed some strange OreDict info with it (I am using extended JEI view): 2019-10-28_21 45 13-sm

Not sure if that's an issue, but it may relate to what I saw next. In Spice of Life, foods/drinks are recorded if you've had them or not. At the time, I haven't had any of the Binnie's Drinks, so they all reported as "Not Yet Eaten! ..." in the tooltip info. I drank one of the Apple Juices I just brewed (from the Fruit Press), and SoL registered I drank it (sparkle effect for eating/drinking new food happened).

When I looked in my Food Book (SoL), I saw that ALL the drinks were recorded as being "Tried"! 2019-10-28_21 45 11-sm 2019-10-28_21 45 12-sm 2019-10-28_21 45 13-sm2 2019-10-28_21 45 16-sm

Why did this happen? This also seems to have caused a desynch in the foods "Tried" in the food book as a result: 2019-10-28_21 44 45-sm 2019-10-28_21 44 52-sm

Let me know if you need more info. Thanks for your great mods, and I pray we can find a solution to this conundrum. ;2)

nstearns96 commented 4 years ago

Looks to be an issue with how forge registers universal bucket items, which the Binnie Core bottle is, and with how Spice of Life distinguishes foods. The bottle items all share an item number and meta number which is what SoL uses to store it in a list of foods. This could be fixed on the mod's side by not using the forge bucket for the bottle and that's likely what will have to be done as I doubt that SoL will want to change it's storage method.

juliand665 commented 4 years ago

Ah yes, the actual list used for the player's health counts all items without subtypes as one (based on getHasSubtypes(), while the food book simply collects all items (flattened to their getSubItems()) and marks them as either eaten or not eaten depending on whether them (or an equivalent) has been eaten. I guess one of two things is happening here:

nstearns96 commented 4 years ago

getHasSubtypes() returns true and and the bottled fluids are only distinguished by an nbt tag, "Fluid".

juliand665 commented 4 years ago

Ah yes, so there's no conceivable way we could fix this from our side, since items could also have subtypes and different tags despite being conceptually the same item (e.g. a renamed golden apple).

Yourname942 commented 4 years ago

Is there a way to disable the drinks from Binnie's Mods?