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

perf: Skip one dump + load on boot.json (#29153)

Browse source 
Since we are json.dumping it, there's no point in sanitizing using SCRIPT_TAG_PATTERN.

This removes dumb json dump + load + dump.
This commit is contained in:
Ankush Menat 2025-01-14 15:22:20 +05:30 • committed by GitHub
parent b338bdd5f2
commit 9a0429ad1e
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 1 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
frappe/www/app.py
View file

@ -36,14 +36,6 @@ def get_context(context):
frappe.db.commit()
boot_json = frappe.as_json(boot, indent=None, separators=(",", ":"))
# remove script tags from boot
boot_json = SCRIPT_TAG_PATTERN.sub("", boot_json)
# TODO: Find better fix
boot_json = CLOSING_SCRIPT_TAG_PATTERN.sub("", boot_json)
hooks = frappe.get_hooks()
app_include_js = hooks.get("app_include_js", []) + frappe.conf.get("app_include_js", [])
app_include_css = hooks.get("app_include_css", []) + frappe.conf.get("app_include_css", [])
@ -62,7 +54,7 @@ def get_context(context):
"layout_direction": "rtl" if is_rtl() else "ltr",
"lang": frappe.local.lang,
"sounds": hooks["sounds"],
"boot": boot if context.get("for_mobile") else json.loads(boot_json),
"boot": boot,
"desk_theme": boot.get("desk_theme") or "Light",
"csrf_token": csrf_token,
"google_analytics_id": frappe.conf.get("google_analytics_id"),