nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-09-20 03:47:11 +00:00
Code Issues Packages Projects Releases Wiki Activity

perf: skip legacy batch ledger lookups when no legacy entry exists (#59110)

Browse Source 
* perf: skip legacy batch ledger lookups when no legacy entry exists
This commit is contained in:
rohitwaghchaure
2026-09-16 17:15:49 +05:30
committed by GitHub
parent 30cfb2094f
commit 0130d287f6
1 changed files with 38 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

39
erpnext/stock/deprecated_serial_batch.py
View File

@@ -11,6 +11,38 @@ from pypika.functions import Coalesce
from pypika.terms import ExistsCriterion
@frappe.request_cache
@deprecated
def has_legacy_batch_ledgers(item_code: str, warehouse: str) -> bool:
"""`False` when no Stock Ledger Entry of the item and warehouse uses the
denormalized `batch_no` field.
Batches are tracked through the Serial and Batch Bundle since v15, so this is
`False` for most of the item and warehouse combinations and the expensive
aggregates (`FOR UPDATE`) below can be skipped without reading the ledger. The
probe is an index only scan on the `batch_no, item_code, warehouse` index,
`is_cancelled` is intentionally left out of it to keep it so, an item with only
cancelled legacy ledgers simply falls back to the aggregate.
Cached for the request, nothing creates a legacy ledger midway.
"""
sle = frappe.qb.DocType("Stock Ledger Entry")
return bool(
frappe.qb.from_(sle)
.select(sle.batch_no)
.where(
sle.batch_no.isnotnull()
& (sle.batch_no != "")
& (sle.item_code == item_code)
& (sle.warehouse == warehouse)
)
.limit(1)
.run()
)
class DeprecatedSerialNoValuation:
@deprecated
def calculate_stock_value_from_deprecarated_ledgers(self):
@@ -96,7 +128,9 @@ class DeprecatedBatchNoValuation:
def get_sle_for_batches(self):
from erpnext.stock.utils import get_combine_datetime
if not self.batchwise_valuation_batches:
if not self.batchwise_valuation_batches or not has_legacy_batch_ledgers(
self.sle.item_code, self.sle.warehouse
):
return []
sle = frappe.qb.DocType("Stock Ledger Entry")
@@ -242,6 +276,9 @@ class DeprecatedBatchNoValuation:
def set_balance_value_from_sl_entries(self) -> None:
from erpnext.stock.utils import get_combine_datetime
if not has_legacy_batch_ledgers(self.sle.item_code, self.sle.warehouse):
return
sle = frappe.qb.DocType("Stock Ledger Entry")
batch = frappe.qb.DocType("Batch")