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

perf: use meta for virtual and single checks (#25620)

Browse source 
RQ workers spawn a new process for every job so site_cache does
nothing to prevent repeated queries.
This commit is contained in:
Ankush Menat 2024-03-26 13:10:15 +05:30 • committed by GitHub
parent 8961461bcb
commit 76096f5fb8
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 9 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
frappe/model/utils/__init__.py
View file

@ -128,9 +128,11 @@ def get_fetch_values(doctype, fieldname, value):
@site_cache()
def is_virtual_doctype(doctype: str):
if frappe.db.has_column("DocType", "is_virtual"):
return frappe.db.get_value("DocType", doctype, "is_virtual")
return False
if frappe.flags.in_install or frappe.flags.in_migrate:
if frappe.db.has_column("DocType", "is_virtual"):
return frappe.db.get_value("DocType", doctype, "is_virtual")
else:
return getattr(frappe.get_meta(doctype), "is_virtual", False)
@site_cache()
@ -140,4 +142,7 @@ def is_single_doctype(doctype: str) -> bool:
if doctype in DOCTYPES_FOR_DOCTYPE:
return False
return frappe.db.get_value("DocType", doctype, "issingle")
if frappe.flags.in_install or frappe.flags.in_migrate:
return frappe.db.get_value("DocType", doctype, "issingle")
else:
return getattr(frappe.get_meta(doctype), "issingle", False)