From 54d3e5675f9d219729f38b0af8693066a734c088 Mon Sep 17 00:00:00 2001 From: Lewis Date: Wed, 13 Aug 2025 16:25:58 -0400 Subject: [PATCH] chore: improve code clarity per reviewer feedback - Rename stock_qty variable to reserved_qty for clarity - Update get_pos_reserved_qty_from_table to return float - Simplify aggregation logic in get_pos_reserved_qty - Ensure return values match docstring specifications --- erpnext/accounts/doctype/pos_invoice/pos_invoice.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/erpnext/accounts/doctype/pos_invoice/pos_invoice.py b/erpnext/accounts/doctype/pos_invoice/pos_invoice.py index 87b6a04b7f9..aac18100263 100644 --- a/erpnext/accounts/doctype/pos_invoice/pos_invoice.py +++ b/erpnext/accounts/doctype/pos_invoice/pos_invoice.py @@ -906,8 +906,7 @@ def get_pos_reserved_qty(item_code, warehouse): pinv_item_reserved_qty = get_pos_reserved_qty_from_table("POS Invoice Item", item_code, warehouse) packed_item_reserved_qty = get_pos_reserved_qty_from_table("Packed Item", item_code, warehouse) - reserved_qty = flt(pinv_item_reserved_qty[0].stock_qty) if pinv_item_reserved_qty else 0 - reserved_qty += flt(packed_item_reserved_qty[0].stock_qty) if packed_item_reserved_qty else 0 + reserved_qty = pinv_item_reserved_qty + packed_item_reserved_qty return reserved_qty @@ -932,7 +931,7 @@ def get_pos_reserved_qty_from_table(child_table, item_code, warehouse): qty_column = "qty" if child_table == "Packed Item" else "stock_qty" - stock_qty = ( + reserved_qty = ( frappe.qb.from_(p_inv) .from_(p_item) .select(Sum(p_item[qty_column]).as_("stock_qty")) @@ -945,7 +944,7 @@ def get_pos_reserved_qty_from_table(child_table, item_code, warehouse): ) ).run(as_dict=True) - return stock_qty + return flt(reserved_qty[0].stock_qty) if reserved_qty else 0 @frappe.whitelist()