CleverRaven / Cataclysm-DDA

Cataclysm - Dark Days Ahead. A turn-based survival game set in a post-apocalyptic world.
http://cataclysmdda.org
Other
10.26k stars 4.12k forks source link

It is possible to have a recipe be both ``reversible: true`` and use ``result_mult`` #74845

Open Karol1223 opened 2 months ago

Karol1223 commented 2 months ago

Describe the bug

As mentioned in the title

Attach save file

N/A

Steps to reproduce

  1. Modify the JSON to have any result_mult recipe be also reversible (I chose gambeson batting)
  2. Disassemble a singular instance of the recipe's result item, see it give back the full ingredient range
  3. Be in awe at the infinite resource generation

Expected behavior

The game shouldn't allow this to happen, similar to how we don't allow this to happen for recipes using byproducts

Screenshots

No response

Versions and configuration

Additional context

No response

github-actions[bot] commented 1 month ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not bump or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered.