Paradise icon indicating copy to clipboard operation
Paradise copied to clipboard

Fix issue with cooking recipes requiring reagents

Open warriorstar-orion opened this issue 2 months ago • 6 comments

What Does This PR Do

This PR fixes issues with cooking where multiple recipes with varying amounts of the same reagent ID at the same step may cause one recipe to no longer be considered valid. The issue stems from not properly handling cases where one single action may be a valid step in two recipes, but where only one step is completed (e.g. one recipe requires 5u milk, one requires 10u, and the player adds 5u to the container). This cleans up a bit of the tracker code, pulling step attempt info into a datum that we can still toss away at the end of the proc.

Why It's Good For The Game

Bugfix, more reliable cooking behavior, hopefully.

Testing

In progress.

  • [x] Banana bread / Banarnar bread recipe issue
  • [x] Autochef interactions

Declaration

  • [X] I confirm that I either do not require pre-approval for this PR, or I have obtained such approval and have included a screenshot to demonstrate this below.

Changelog

:cl: fix: Banana-nut bread's recipe can now be followed properly. /:cl:

warriorstar-orion avatar Oct 05 '25 22:10 warriorstar-orion

This pull request seems to be stale as there have been no changes in 14 days, please make changes within 7 days or the PR will be closed. If you believe this is a mistake, please inform a development team member on Discord.

github-actions[bot] avatar Oct 21 '25 01:10 github-actions[bot]

This pull request has not received any updates since being marked stale, and as such is now being automatically closed. Please feel free to re-open this pull request or open a new one once you have new updates.

github-actions[bot] avatar Oct 29 '25 01:10 github-actions[bot]

fuck i keep forgetting about this

warriorstar-orion avatar Oct 29 '25 19:10 warriorstar-orion

Kitchen machinery when upgraded does not produce more than one food, here is an example of an oven that was upgraded but producing 1 bread 509158377-b064bbdd-cf38-4875-951a-8f38a27063b6

zeufu avatar Nov 03 '25 20:11 zeufu

DEBUG: More than one valid recipe completion at the same step, this shouldn't happen. Valid recipes: /datum/cooking/recipe/carrotfries, /datum/cooking/recipe/deep_fried_anything

Edit: Unrelated issue, not relevant

DGamerL avatar Nov 10 '25 22:11 DGamerL

This pull request seems to be stale as there have been no changes in 14 days, please make changes within 7 days or the PR will be closed. If you believe this is a mistake, please inform a development team member on Discord.

github-actions[bot] avatar Nov 25 '25 01:11 github-actions[bot]