Merge pull request #56250 from mihir-kandoi/pg-56249-review-followup

fix(stock): make get_incoming_value_for_serial_nos a staticmethod
This commit is contained in:
Mihir Kandoi
2026-06-21 15:17:59 +05:30
committed by GitHub
2 changed files with 3 additions and 2 deletions

View File

@@ -77,7 +77,7 @@ class TestStockLedgerEntry(ERPNextTestSuite, StockTestMixin):
mk_sle("MAT-SLE-TIE-B", 200) # later/larger name -> deterministic winner
value = update_entries_after.get_incoming_value_for_serial_nos(
None, frappe._dict(company=company_a), [serial]
frappe._dict(company=company_a), [serial]
)
# the latest (creation/name desc) same-date SLE wins -> 200 on both engines
self.assertEqual(value, 200.0)

View File

@@ -1473,7 +1473,8 @@ class update_entries_after:
for item in sr.items:
item.db_update()
def get_incoming_value_for_serial_nos(self, sle, serial_nos):
@staticmethod
def get_incoming_value_for_serial_nos(sle, serial_nos):
# get rate from serial nos within same company
all_serial_nos = frappe.get_all(
"Serial No", fields=["purchase_rate", "name", "company"], filters={"name": ("in", serial_nos)}