From 8b2946ca6baecc5bd831300e2538057e5ad20f89 Mon Sep 17 00:00:00 2001 From: Mihir Kandoi Date: Fri, 7 Aug 2026 18:42:00 +0530 Subject: [PATCH] refactor: remove unused make_purchase_order_based_on_supplier Its only caller, Purchase Order's get_items_from_open_material_requests, was deleted in 91e9867fb1 (refactor: Cleanup buying module forms). The old dotted path was already broken by the move to mapper.py, so no external caller can be using it either. --- .../stock/doctype/material_request/mapper.py | 45 ------------------- 1 file changed, 45 deletions(-) diff --git a/erpnext/stock/doctype/material_request/mapper.py b/erpnext/stock/doctype/material_request/mapper.py index 26569f58cec..ea6fd98fdca 100644 --- a/erpnext/stock/doctype/material_request/mapper.py +++ b/erpnext/stock/doctype/material_request/mapper.py @@ -288,51 +288,6 @@ def get_items_based_on_default_supplier(supplier: str): return supplier_items -@frappe.whitelist() -def make_purchase_order_based_on_supplier( - source_name: str, target_doc: str | dict | Document | None = None, args: dict | None = None -): - mr = source_name - - supplier_items = get_items_based_on_default_supplier(args.get("supplier")) - - def postprocess(source, target_doc): - target_doc.supplier = args.get("supplier") - if getdate(target_doc.schedule_date) < getdate(nowdate()): - target_doc.schedule_date = None - target_doc.set( - "items", - [d for d in target_doc.get("items") if d.get("item_code") in supplier_items and d.get("qty") > 0], - ) - - set_missing_values(source, target_doc) - - target_doc = get_mapped_doc( - "Material Request", - mr, - { - "Material Request": { - "doctype": "Purchase Order", - }, - "Material Request Item": { - "doctype": "Purchase Order Item", - "field_map": [ - ["name", "material_request_item"], - ["parent", "material_request"], - ["uom", "stock_uom"], - ["uom", "uom"], - ], - "postprocess": update_item, - "condition": lambda doc: doc.ordered_qty < doc.qty, - }, - }, - target_doc, - postprocess, - ) - - return target_doc - - @frappe.whitelist() def make_supplier_quotation(source_name: str, target_doc: str | dict | Document | None = None): def postprocess(source, target_doc):