nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-08-14 15:11:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #57033 from aerele/perf-bom-raw-material-warehouse-lookup

Browse Source 
perf(stock): avoid n+1 queries for work order item source warehouse
This commit is contained in:
Mihir Kandoi
2026-07-10 16:37:44 +05:30
committed by GitHub
parent 4aabe3347f 3e4d5e6745
commit bf4e1a3ae0
1 changed files with 18 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
erpnext/stock/doctype/stock_entry/stock_entry.py
View File

@@ -1423,6 +1423,23 @@ class StockEntry(StockController, SubcontractingInwardController):
used_alternative_items = get_used_alternative_items(
subcontract_order_field=self.subcontract_data.order_field, work_order=self.work_order
)
skip_transfer, from_wip_warehouse = (
frappe.get_value("Work Order", self.work_order, ["skip_transfer", "from_wip_warehouse"])
if self.work_order
else [None, None]
)
wo_item_source_warehouses = {}
if skip_transfer and not from_wip_warehouse:
for d in frappe.get_all(
"Work Order Item",
filters={"parent": self.work_order},
fields=["item_code", "source_warehouse"],
):
# default ordering is creation desc; keep the first (most recent) row per
# item_code to match the limit-1 behaviour of the get_value call this replaces
wo_item_source_warehouses.setdefault(d.item_code, d.source_warehouse)
for item in item_dict.values():
# if source warehouse presents in BOM set from_warehouse as bom source_warehouse
if item["allow_alternative_item"]:
@@ -1430,18 +1447,8 @@ class StockEntry(StockController, SubcontractingInwardController):
"Work Order", self.work_order, "allow_alternative_item"
)
skip_transfer, from_wip_warehouse = (
frappe.get_value("Work Order", self.work_order, ["skip_transfer", "from_wip_warehouse"])
if self.work_order
else [None, None]
)
item.from_warehouse = (
frappe.get_value(
"Work Order Item",
{"parent": self.work_order, "item_code": item.item_code},
"source_warehouse",
)
wo_item_source_warehouses.get(item.item_code)
if skip_transfer and not from_wip_warehouse
else self.from_warehouse or item.source_warehouse or item.default_warehouse
)