Bug: Meal plan grocery list doesn't consolidate ingredients #9

Open
opened 2026-04-21 14:44:07 +00:00 by frank · 0 comments
Owner

MealPlan::generateGroceryList() calls create() per ingredient, bypassing consolidation. Two recipes needing flour = two separate entries.\n\nFix: Add optional servingsMultiplier param to addFromRecipe(), rewrite generateGroceryList() to use it. Remove duplicate parseAmount/formatAmount from MealPlan.php (already extracted to UnitConverter).\n\nEstimate: 1.25 hr | Value: High\nFiles: api/models/MealPlan.php, api/models/GroceryItem.php\n\nRef: docs/roadmap.md

MealPlan::generateGroceryList() calls create() per ingredient, bypassing consolidation. Two recipes needing flour = two separate entries.\n\nFix: Add optional servingsMultiplier param to addFromRecipe(), rewrite generateGroceryList() to use it. Remove duplicate parseAmount/formatAmount from MealPlan.php (already extracted to UnitConverter).\n\nEstimate: 1.25 hr | Value: High\nFiles: api/models/MealPlan.php, api/models/GroceryItem.php\n\nRef: docs/roadmap.md
frank added this to the Quick Wins milestone 2026-04-21 14:44:07 +00:00
Sign in to join this conversation.
No description provided.