After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 777305 - Clicking 'Cook it later' resets the serve count.
Clicking 'Cook it later' resets the serve count.
Status: RESOLVED FIXED
Product: recipes
Classification: Other
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Recipes maintainer(s)
Recipes maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2017-01-16 06:38 UTC by Mohammed Sadiq
Modified: 2017-01-16 20:06 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Mohammed Sadiq 2017-01-16 06:38:00 UTC
Clicking the 'Cook it later' button resets the serve count, which also resets the purchase list amount to be bought.
Comment 1 Matthias Clasen 2017-01-16 16:21:31 UTC
The serve count that we set on the details page is just a transient thing, we don't store it with the recipe. Whats happening when you click 'cook it later' is that we store the favorites list and emit a recipe-changed signal that the details page catches, and reloads the recipe.

Two things that could be done here:

- be more granular about change notification from the recipe store, and add a eg ::favorites-changed signal. its a bit tricky to get this right

- teach the details page to preserve the serves value when the loaded recipe is the same as the previous one. This should not be hard


Making it possible to adjust servings on the shopping list should be a separate enhancement request.