nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-09-24 05:47:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: resolve MRP purchase supplier with item group fallback, else throw

Browse Source
This commit is contained in:
Shllokkk
2026-09-24 00:27:24 +05:30
parent 79858d5af6
commit be283b70f2
1 changed files with 15 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
erpnext/manufacturing/report/material_requirements_planning_report/material_requirements_planning_report.py
View File

@@ -1224,6 +1224,13 @@ def get_item_details(item_code, company):
if default_data:
data.update(default_data)
if not data.get("default_supplier"):
# fall back to the item's Item Group default supplier (mirrors BuyingController)
item_group = frappe.db.get_value("Item", item_code, "item_group")
data.default_supplier = frappe.db.get_value(
"Item Default", {"parent": item_group, "company": company}, "default_supplier"
)
return data
@@ -1333,6 +1340,7 @@ def make_order(selected_rows: str | list, company: str, warehouse: str | None =
work_orders = []
covered_rows = 0
missing_bom = []
missing_supplier = []
for row in selected_rows:
row = frappe._dict(row)
# what is left to order once stock and the orders already placed are counted. rounding
@@ -1343,7 +1351,10 @@ def make_order(selected_rows: str | list, company: str, warehouse: str | None =
continue
if row.type_of_material == "Purchase":
purchase_orders.setdefault((row.default_supplier, row.release_date), []).append(row)
if row.default_supplier:
purchase_orders.setdefault((row.default_supplier, row.release_date), []).append(row)
elif row.item_code not in missing_supplier:
missing_supplier.append(row.item_code)
if row.type_of_material == "Manufacture":
if row.bom_no:
@@ -1354,6 +1365,9 @@ def make_order(selected_rows: str | list, company: str, warehouse: str | None =
if missing_bom:
frappe.throw(_("Default BOM for {0} not found").format(", ".join(missing_bom)))
if missing_supplier:
frappe.throw(_("Default Supplier for {0} not found").format(", ".join(missing_supplier)))
if not purchase_orders and not work_orders:
frappe.msgprint(
_("Nothing to order, the selected rows are already covered by stock or existing orders")