Cataclysm-DDA icon indicating copy to clipboard operation
Cataclysm-DDA copied to clipboard

Deconstructing damaged steel frames does not produce lumps.

Open strife26 opened this issue 7 years ago • 5 comments

Describe the bug
When attempting to deconstruct damaged steel frames, the game notifies the player that it's not guaranteed to recover components. However, it appears that the game treats lump of steel(10) as the results, not ten chances to recover a lump of steel. I've yet to successfully recover any material from a damaged frame. This results in damaged parts not being useful for recovering material.

To Reproduce
Steps to reproduce the behavior:

  1. Acquire damaged steel frames and required tools to deconstruct.
  2. Deconstruct frames.
  3. Note evaporation of any parts

Expected behavior
Some materials recovered from damaged frames, likely at a smaller but nonzero full recovery of 10 lumps. I'd imagine this problem might extent to other items that deconstruct into items with charges.

Versions and configuration(please complete the following information):

  • OS: Win10
  • Game Version 0.C-34729-g0051c81
  • Graphics version Tiles

strife26 avatar Dec 01 '18 05:12 strife26

Tried to confirm the bug. Deconstruction seems to work fine. At least with my current build: Build 8232 (Tiles - Win x64) - 0.C-34877-g7bae86c (tiles)

Damaged Frame yields: grafik

Pupsi-Mupsi avatar Dec 01 '18 07:12 Pupsi-Mupsi

@Pupsi-Mupsi did you uninstall the frame from a vehicle? That works correctly, but OP is most likely talking about disassembling an item (on the ground/in inventory) into its parts - the reverse of crafting it.

BevapDin avatar Dec 06 '18 10:12 BevapDin

@Pupsi-Mupsi did you uninstall the frame from a vehicle? That works correctly, but OP is most likely talking about disassembling an item (on the ground/in inventory) into its parts - the reverse of crafting it.

I couldn't remember so I repeated both. Removing form vehicle worked fine.

Disassembling frames: Undamaged frame - Yield = 10 lump of steel Damaged |. frame - Yield = 10 lump of steel Damaged . frame - Yield = nothing

Sorry, definitely my mistake.

Pupsi-Mupsi avatar Dec 06 '18 13:12 Pupsi-Mupsi

@strife26 and @Pupsi-Mupsi I have repeated your tests in the latest experimental, and while steel frames now give 20 lumps of steel on disassembly, the game still treats it as an all or nothing situation. I'm not sure how to fix this but maybe someone else can. I think the crafting.cpp file line 2808 is important disassembly code and the steel frame recipe is stored in recipe_vehicle.json. Best of luck!

DragonWizard23 avatar Dec 19 '23 14:12 DragonWizard23

/Invalid It's still an all or nothing situation but I think a separate feature request for damaged items since this bug is no longer applicable would be best.

TealcOneill avatar May 23 '24 21:05 TealcOneill