CleverRaven
Several Food Items Spawn Without Vitamins
Describe the bug
What it says on the tin. Some of the ones I've personally encountered includes fruit jam, mayonnaise, BBQ sauce, honey mustard, human organs (all of them except the meat) and butter.
Attach save file
Steps to reproduce
- Spawn any of the affected items.
- No vitamins. :(
Expected behavior
Food spawning with vitamins. :)
Screenshots
No response
Versions and configuration
- OS: Windows
- OS Version: 10.0.19043.1466 (21H1)
- Game Version: 1b5a260 [64-bit]
- Graphics Version: Tiles
- Game Language: English [en]
- Mods loaded: [ Dark Days Ahead [dda], Bionic Professions [package_bionic_professions], Disable NPC Needs [no_npc_food], Portal Storms Ignore NPCs [personal_portal_storms], Slowdown Fungal Growth [no_fungal_growth], SpeedyDex [speedydex], Stats Through Kills [stats_through_kills], Stats Through Skills [StatsThroughSkills] ]
Additional context
I'm told this is a problem with the new vitamin code that affects a number of items, so while I listed off some examples I'm certain that I haven't seen every affected item. So for anyone passing by, feel free to add any other items you've seen without vitamins.
When it comes to human organs I suspect the copy-from logic is used incorrectly for vitamins as it seems the implementation assumes a vitamin entry extends the entry of the copied item. Unless vitamins are handled differently from other entries, you'd need to use the extend logic to extend, otherwise you replace. See #75540 for this.
#75426 is likely what caused this sort of stuff with the organs. I noticed this happening with a bunch of bird eggs as well, where they lost their egg allergen.
Its also worth pointing out not all food is supposed to spawn with vitamins. Fruit jam comes to mind - in JSON the only vitamin it has is the fruit allergen. Should it have some token vitamin content? Probably. But the bug here isn't the spawning.
Just to clarify, the food should have vitamin definition in json always - if food is crafted by player, vitamins in json are simply not used, and picked from ingredients (unless NUTRIENT_OVERRIDE flag is used), but if it's spawned naturally, vitamin definition in json is used
Human organs is a separate issue, posted by Patrik, and it's tad more difficult than this
So it looks like human organs have been fixed, which is great. But stuff like butter and fruit jam still spawn without vitamins, which sucks.
Do you mean that they spawn without any vitamins, or with only allergen vitamins? If it's the latter, that's not a bug at all.
Do you mean that they spawn without any vitamins, or with only allergen vitamins? If it's the latter, that's not a bug at all.
I mean the "actual" vitamins, not the allergens. Unless the butter in Cataclysm magically got rid of all of its calcium, for example.