Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Faction base recipe from SD-card #33349

Closed
Melodeiro opened this issue Aug 18, 2019 · 3 comments · Fixed by #36566
Closed

Faction base recipe from SD-card #33349

Melodeiro opened this issue Aug 18, 2019 · 3 comments · Fixed by #36566
Labels
<Bug> This needs to be fixed Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Good First Issue This is a good first issue for a new contributor Player Faction Base / Camp All about the player faction base/camp/site (S2 - Confirmed) Bug that's been confirmed to exist
Milestone

Comments

@Melodeiro
Copy link

Describe the bug

I just got a weird recipe from Science SD-card

Steps To Reproduce

Probably it just randoms any recipes, including fake ones from faction bases

Screenshots

image

Versions and configuration

  • OS: Windows
    • OS Version: 10.0 1607
  • Game Version: 0.D-6568-gffec12f [64-bit]
  • Graphics Version: Tiles
  • Mods loaded: [
    Dark Days Ahead [dda],
    Aftershock [aftershock],
    DeadLeaves' Fictional Guns [FIC_Weapons],
    More Survival Tools [More_Survival_Tools],
    Makeshift Items Mod [makeshift],
    Garden Pots [growable-pots],
    Modular Turrets [modular_turrets],
    Salvaged Robots [Salvaged_Robots],
    Mining Mod [Mining_Mod],
    Mutant NPCs [mutant_npcs],
    Alternative Map Key [alt_map_key],
    Hydroponics [hydroponics],
    Beta National Guard Camp [national_guard_camp],
    More Locations [more_locations],
    Fuji's More Buildings [FujiStruct],
    Urban Development [Urban_Development],
    Vehicle Additions Pack [blazemod],
    Folding Parts pack [deoxymod],
    Tanks and Other Vehicles [Tanks],
    Necromancy [necromancy],
    Disable Religious Texts [no_religious_Texts],
    No Fungal Monsters [No_Fungi],
    Safe Autodoc [safeautodoc],
    Classes and Scenarios Mod [more_classes_scenarios],
    Disable NPC Needs [no_npc_food],
    Manual Bionic Installation [manualbionicinstall],
    Simplified Nutrition [novitamins],
    Stats Through Kills [stats_through_kills],
    StatsThroughSkills [StatsThroughSkills],
    SpeedyDex [speedydex]
    ]
@Night-Pryanik Night-Pryanik added <Bug> This needs to be fixed Crafting / Construction / Recipes Includes: Uncrafting / Disassembling labels Aug 19, 2019
@Leland
Copy link
Contributor

Leland commented Aug 20, 2019

Weirdly, that recipe is set to never_learn:

"result": "faction_base_kitchen_2",
"description": "We need to finish framing the walls for the cook-shack.",
"category": "CC_BUILDING",
"subcategory": "CSC_BUILDING_BASES",
"skill_used": "fabrication",
"difficulty": 4,
"autolearn": false,
"never_learn": true,

...which is supposed to prevent this from being learned at all, regardless of where it's been included:

Cataclysm-DDA/src/player.cpp

Lines 10453 to 10459 in 9f59a93

void player::learn_recipe( const recipe *const rec )
{
if( rec->never_learn ) {
return;
}
learned_recipes->include( rec );
}

Wonder how this happened.

@Leland Leland added (S1 - Need confirmation) Report waiting on confirmation of reproducibility Player Faction Base / Camp All about the player faction base/camp/site labels Aug 20, 2019
@Melodeiro
Copy link
Author

Where is the DEBUG word from then? Maybe that recipe is not from json, but hardcoded somewhere?

@Melodeiro
Copy link
Author

I guess, you need to add never_learn check here:

if( r.difficulty >= 3 && one_in( r.difficulty + 1 ) ) {

I don't see any checks when recipe is added to e-ink

@ZhilkinSerg ZhilkinSerg added (S2 - Confirmed) Bug that's been confirmed to exist and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels Dec 27, 2019
@ZhilkinSerg ZhilkinSerg added this to the 0.E milestone Dec 27, 2019
@ZhilkinSerg ZhilkinSerg added the Good First Issue This is a good first issue for a new contributor label Dec 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Good First Issue This is a good first issue for a new contributor Player Faction Base / Camp All about the player faction base/camp/site (S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants