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

"(loose) knit scarves" are MUCH more difficult to repair than "knit scarves" #36580

Closed
MLamor opened this issue Dec 31, 2019 · 2 comments · Fixed by #36700
Closed

"(loose) knit scarves" are MUCH more difficult to repair than "knit scarves" #36580

MLamor opened this issue Dec 31, 2019 · 2 comments · Fixed by #36700
Labels
<Bug> This needs to be fixed Items / Item Actions / Item Qualities Items and how they work and interact

Comments

@MLamor
Copy link

MLamor commented Dec 31, 2019

Game version: 0.D-10214-ge667aa0

Operating system: Windows 10

Tiles or curses: Tiles

Mods active: DDA, Disable NPC Needs, Icecoons Arsenal, Deadleavs Fictional Guns, Makeshift Items, More Survival Tools, Modular Turrets, Salvaged Robots, Mutant NPC, Beta National Guard Camp,
More Locations, Fuji's More Buildings, Mapgen Demo, No Rail Stations, Urban Development, Folding Parts, Vehicle Additions, Tanks & Other Vehicles, Roadheader & Mining Vehicles

Expected behavior

A scarf should be just as simple/difficult to repair whether it is being worn tight, or being worn loosely, because it is the same item either way.

Actual behavior

(loose) knit scarves are harder (or impossible) to repair, while a regular knit scarf is easy to repair.

Steps to reproduce the behavior

With 5 Tailoring skill, using a Sewing Kit, attempting to reinforce a (loose) knit scarf will have 0% chance for success, and 4% chance for failure, (a)ctivating the (loose) knit scarf to tighten it first changes it to 10% chance for success, and 0% chance for failure.

@Night-Pryanik Night-Pryanik added <Bug> This needs to be fixed Items / Item Actions / Item Qualities Items and how they work and interact labels Dec 31, 2019
@k4gi
Copy link
Contributor

k4gi commented Jan 2, 2020

I was investigating this yesterday (with the help of the Discord). The cause could be that repairing an item gets harder if the player doesn't know its recipe, or if a recipe doesn't exist. I couldn't decipher the code enough to find out if this is the case 😝

@Night-Pryanik
Copy link
Contributor

Here's comments from the repair_item_actor::repair_recipe_difficulty:

If the recipe is not known by the player, +1 to difficulty
If player doesn't meet the requirements of the recipe, +1 to difficulty
If the recipe doesn't exist, difficulty is 10

So, repair difficulty of crafting the knit scarf is 1, but because there is no recipe for crafting the loose knit scarf, its repair difficulty is 5 (and not 10, if I'm reading the code right). That's why repairing loose knit scarf is harder.

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 Items / Item Actions / Item Qualities Items and how they work and interact
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants