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

fix: patch QB before setup_module_map (#24981)

Browse source 
This function if cache isn't available CAN result in a DB query, this
will cause cold starts to fail because QB is not yet patched.
This commit is contained in:
Ankush Menat 2024-02-21 15:27:50 +05:30 • committed by GitHub
parent f027a03dcf
commit 3aa7fd77ed
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 2 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
frappe/__init__.py
View file

@ -300,12 +300,13 @@ def init(site: str, sites_path: str = ".", new_site: bool = False, force=False)
local.qb = get_query_builder(local.conf.db_type)
local.qb.get_query = get_query
setup_redis_cache_connection()
setup_module_map(include_all_apps=not (frappe.request or frappe.job or frappe.flags.in_migrate))
if not _qb_patched.get(local.conf.db_type):
patch_query_execute()
patch_query_aggregation()
setup_module_map(include_all_apps=not (frappe.request or frappe.job or frappe.flags.in_migrate))
local.initialised = True

2
frappe/query_builder/utils.py
View file

@ -138,7 +138,6 @@ def patch_query_execute():
# To support running union queries
_SetOperation.run = execute_query
_SetOperation.walk = prepare_query
frappe._qb_patched[frappe.conf.db_type] = True
def patch_query_aggregation():
@ -149,4 +148,3 @@ def patch_query_aggregation():
frappe.qb.min = _min
frappe.qb.avg = _avg
frappe.qb.sum = _sum
frappe._qb_patched[frappe.conf.db_type] = True