Walkthrough

This pull request changes production plan material-request calculations to track consumed quantities per (item_code, warehouse) instead of a global per-item total. The get_material_request_items signature replaces total_qty with consumed_qty (a defaultdict keyed by (item_code, warehouse)) and call sites are updated to propagate and update this per-location consumption when projecting available quantity. A new test, test_projected_qty_cascading_across_multiple_sales_orders, is added to validate cascading projected requirements across multiple sales orders and related stock reconciliation behavior.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

• Mixed edits: function signature change, propagation of a new parameter through nested calls, and an added test.
• Areas needing extra attention:
  • erpnext/manufacturing/doctype/production_plan/production_plan.py — correctness of consumed_qty keying, initialization, update logic when projecting available qty, and all updated call sites.
  • erpnext/manufacturing/doctype/production_plan/test_production_plan.py — validity of the new test scenario and teardown (stock reconciliation cancel).
  • Search for any remaining call sites or external integrations still expecting the old total_qty parameter.

Possibly related PRs

• Fix/Support 50220 #49960 — Modifies production_plan quantity aggregation logic; likely related to reserved/projected quantity handling that interacts with the per-(item,warehouse) consumption changes.

Suggested reviewers

• rohitwaghchaure
• mihir-kandoi

• 📝 Generate docstrings

• X
• Mastodon
• Reddit
• LinkedIn