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

Add id_suffix to shadowed recipes (#56161) #56177

Merged
merged 1 commit into from
Mar 20, 2022

Conversation

remd
Copy link
Contributor

@remd remd commented Mar 17, 2022

Resolves #56161

  • Un-shadowed the following recipes, allowing them to be crafted:
    • armor_scrapsuit
    • armor_tiresuit
    • chem_hmtd
    • chem_nitric_acid
    • gambeson_vest
    • gambeson_vest_xs
    • k_gambeson_vest
    • k_gambeson_vest_xs
    • lc_chainmail_arms
    • pistol_stock
    • pot_copper
    • propane_cooker
    • thread (Spinning Wheel versions)
    • tshirt
    • xl_ch_chainmail_hood
    • xl_gambeson_vest
    • xl_hc_chainmail_hood
    • xl_k_gambeson_vest
    • xl_lc_chainmail_arms
    • xl_lc_chainmail_hood
    • xl_mc_chainmail_hood
    • xl_qt_chainmail_hood
    • xs_armor_lightplate

Summary

None

Purpose of change

Fixes #56161

Describe the solution

Added id_suffix to listed recipes so that their derived IDs will be unique, ensuring they do not conflict.

In three cases (lc_chainmail_arms, xs_lc_chainmail_arms, and xl_lc_chainmail_arms) the recipe was duplicated so I deleted the duplicate entries.

Describe alternatives you've considered

None

Testing

  1. Unlock all recipes via debug menu
  2. Confirm shadowed recipes are now visible in crafting menu

Additional context

Resolves CleverRaven#56161

* Un-shadowed the following recipes, allowing them to be crafted:
  * armor_scrapsuit
  * armor_tiresuit
  * chem_hmtd
  * chem_nitric_acid
  * gambeson_vest
  * gambeson_vest_xs
  * k_gambeson_vest
  * k_gambeson_vest_xs
  * lc_chainmail_arms
  * pistol_stock
  * pot_copper
  * propane_cooker
  * thread (Spinning Wheel versions)
  * tshirt
  * xl_ch_chainmail_hood
  * xl_gambeson_vest
  * xl_hc_chainmail_hood
  * xl_k_gambeson_vest
  * xl_lc_chainmail_arms
  * xl_lc_chainmail_hood
  * xl_mc_chainmail_hood
  * xl_qt_chainmail_hood
  * xs_armor_lightplate
@github-actions github-actions bot added the [JSON] Changes (can be) made in JSON label Mar 17, 2022
@remd
Copy link
Contributor Author

remd commented Mar 17, 2022

Some of the recipes added in #55455 vary their multiplier on the "chainmail_standard" requirements without changing the required amount of materials needed to craft the items. I gave those an id_suffix of less_charges, standard_charges, and more_charges.

It seems like recipes requiring a higher amount of charges to finish would also require a higher amount of materials (or lower quality base materials) but that's not how the recipes are defined. Likewise for the recipes requiring less charges, they require the same amount of materials. I'm honestly not sure what the intent is here.

Since I just wanted to fix the shadowing issue I didn't modify the requirements or recipes as I'm not sure what the original author's vision is.

@github-actions github-actions bot added astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Mar 18, 2022
@ZhilkinSerg ZhilkinSerg merged commit 4bae97b into CleverRaven:master Mar 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

recipe cover
2 participants