vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

perf: Avoid seprately fetching docstatus

Browse source
This commit is contained in:
Ankush Menat 2025-06-05 14:26:17 +05:30
parent c434f4ca14
commit 4dd21f175e
1 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
frappe/model/base_document.py
View file

@ -887,10 +887,13 @@ class BaseDocument:
if not meta.get("is_virtual"):
if not fields_to_fetch:
# cache a single value type
values = _dict(name=frappe.db.get_value(doctype, docname, "name", cache=True))
values = frappe.db.get_value(
doctype, docname, ("name", "docstatus"), cache=True, as_dict=True
)
else:
values_to_fetch = (
"name",
"docstatus",
*(_df.fetch_from.split(".")[-1] for _df in fields_to_fetch),
)
@ -926,7 +929,7 @@ class BaseDocument:
df.fieldname != "amended_from"
and (is_submittable or self.meta.is_submittable)
and frappe.get_meta(doctype).is_submittable
and DocStatus(frappe.db.get_value(doctype, docname, "docstatus") or 0).is_cancelled()
and DocStatus(values.docstatus or 0).is_cancelled()
):
cancelled_links.append((df.fieldname, docname, get_msg(df, docname)))