nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-09-17 02:26:33 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(stock): fetch item stock UOM in stock reconciliation (#58284)

Browse Source
This commit is contained in:
Sudharsanan Ashok
2026-08-19 10:27:07 +05:30
committed by GitHub
parent d070a1daa1
commit 3a6e17a03d
2 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
erpnext/stock/doctype/stock_reconciliation/stock_reconciliation.py
View File

@@ -1317,12 +1317,15 @@ def get_item_and_warehouses(item_code, warehouse):
from frappe.utils.nestedset import get_descendants_of from frappe.utils.nestedset import get_descendants_of
items = [] items = []
stock_uom = frappe.get_cached_value("Item", item_code, "stock_uom")
if frappe.get_cached_value("Warehouse", warehouse, "is_group"): if frappe.get_cached_value("Warehouse", warehouse, "is_group"):
childrens = get_descendants_of("Warehouse", warehouse, ignore_permissions=True, order_by="lft") childrens = get_descendants_of("Warehouse", warehouse, ignore_permissions=True, order_by="lft")
for ch_warehouse in childrens: for ch_warehouse in childrens:
items.append(frappe._dict({"item_code": item_code, "warehouse": ch_warehouse})) items.append(
frappe._dict({"item_code": item_code, "warehouse": ch_warehouse, "stock_uom": stock_uom})
)
else: else:
items = [frappe._dict({"item_code": item_code, "warehouse": warehouse})] items = [frappe._dict({"item_code": item_code, "warehouse": warehouse, "stock_uom": stock_uom})]
return items return items
@@ -1349,6 +1352,7 @@ def get_items_for_stock_reco(warehouse, company):
bin_dt.warehouse.as_("warehouse"), bin_dt.warehouse.as_("warehouse"),
item.has_serial_no, item.has_serial_no,
item.has_batch_no, item.has_batch_no,
item.stock_uom,
) )
.where( .where(
((item.disabled == 0) | item.disabled.isnull()) ((item.disabled == 0) | item.disabled.isnull())
@@ -1373,6 +1377,7 @@ def get_items_for_stock_reco(warehouse, company):
item_default.default_warehouse.as_("warehouse"), item_default.default_warehouse.as_("warehouse"),
item.has_serial_no, item.has_serial_no,
item.has_batch_no, item.has_batch_no,
item.stock_uom,
) )
.where( .where(
item_default.default_warehouse.isin(warehouses_in_tree) item_default.default_warehouse.isin(warehouses_in_tree)
@@ -1412,6 +1417,7 @@ def get_item_data(row, qty, valuation_rate, serial_no=None):
"current_serial_no": serial_no, "current_serial_no": serial_no,
"serial_no": serial_no, "serial_no": serial_no,
"batch_no": row.get("batch_no"), "batch_no": row.get("batch_no"),
"stock_uom": row.get("stock_uom"),
} }
@@ -1439,6 +1445,7 @@ def get_itemwise_batch(warehouse, posting_date, company, item_code=None):
"valuation_rate": row[9], "valuation_rate": row[9],
"item_name": row[1], "item_name": row[1],
"batch_no": row[4], "batch_no": row[4],
"stock_uom": row[11],
} }
) )
) )

5
erpnext/stock/doctype/stock_reconciliation/test_stock_reconciliation.py
View File

@@ -140,6 +140,7 @@ class TestStockReconciliation(ERPNextTestSuite, StockTestMixin):
"_Test Stock Reco Item", "_Test Stock Reco Item",
is_stock_item=1, is_stock_item=1,
valuation_rate=100, valuation_rate=100,
stock_uom="_Test UOM",
warehouse="_Test Warehouse Ledger 1 - _TC", warehouse="_Test Warehouse Ledger 1 - _TC",
opening_stock=100, opening_stock=100,
) )
@@ -147,8 +148,8 @@ class TestStockReconciliation(ERPNextTestSuite, StockTestMixin):
items = get_items("_Test Warehouse Group 1 - _TC", nowdate(), nowtime(), "_Test Company") items = get_items("_Test Warehouse Group 1 - _TC", nowdate(), nowtime(), "_Test Company")
self.assertEqual( self.assertEqual(
["_Test Stock Reco Item", "_Test Warehouse Ledger 1 - _TC", 100], ["_Test Stock Reco Item", "_Test Warehouse Ledger 1 - _TC", 100, "_Test UOM"],
[items[0]["item_code"], items[0]["warehouse"], items[0]["qty"]], [items[0]["item_code"], items[0]["warehouse"], items[0]["qty"], items[0]["stock_uom"]],
) )
def test_stock_reco_for_serialized_item(self): def test_stock_reco_for_serialized_item(self):