perf: send SLA doctypes in boot
This request is fired on every load, data rarely if ever changes though.
(cherry picked from commit bd9ef74ef7)
This commit is contained in:
@@ -612,3 +612,8 @@ global_search_doctypes = {
|
||||
additional_timeline_content = {
|
||||
"*": ["erpnext.telephony.doctype.call_log.call_log.get_linked_call_logs"]
|
||||
}
|
||||
|
||||
|
||||
extend_bootinfo = [
|
||||
"erpnext.support.doctype.service_level_agreement.service_level_agreement.add_sla_doctypes",
|
||||
]
|
||||
|
||||
@@ -822,13 +822,7 @@ $(document).on('app_ready', function() {
|
||||
|
||||
// Show SLA dashboard
|
||||
$(document).on('app_ready', function() {
|
||||
frappe.call({
|
||||
method: 'erpnext.support.doctype.service_level_agreement.service_level_agreement.get_sla_doctypes',
|
||||
callback: function(r) {
|
||||
if (!r.message)
|
||||
return;
|
||||
|
||||
$.each(r.message, function(_i, d) {
|
||||
$.each(frappe.boot.service_level_agreement_doctypes, function(_i, d) {
|
||||
frappe.ui.form.on(d, {
|
||||
onload: function(frm) {
|
||||
if (!frm.doc.service_level_agreement)
|
||||
@@ -910,8 +904,6 @@ $(document).on('app_ready', function() {
|
||||
},
|
||||
});
|
||||
});
|
||||
}
|
||||
});
|
||||
});
|
||||
|
||||
function set_time_to_resolve_and_response(frm, apply_sla_for_resolution) {
|
||||
|
||||
@@ -21,6 +21,7 @@ from frappe.utils import (
|
||||
time_diff_in_seconds,
|
||||
to_timedelta,
|
||||
)
|
||||
from frappe.utils.caching import redis_cache
|
||||
from frappe.utils.nestedset import get_ancestors_of
|
||||
from frappe.utils.safe_exec import get_safe_globals
|
||||
|
||||
@@ -209,6 +210,10 @@ class ServiceLevelAgreement(Document):
|
||||
def on_update(self):
|
||||
set_documents_with_active_service_level_agreement()
|
||||
|
||||
def clear_cache(self):
|
||||
get_sla_doctypes.clear_cache()
|
||||
return super().clear_cache()
|
||||
|
||||
def create_docfields(self, meta, service_level_agreement_fields):
|
||||
last_index = len(meta.fields)
|
||||
|
||||
@@ -990,6 +995,7 @@ def get_user_time(user, to_string=False):
|
||||
|
||||
|
||||
@frappe.whitelist()
|
||||
@redis_cache()
|
||||
def get_sla_doctypes():
|
||||
doctypes = []
|
||||
data = frappe.get_all("Service Level Agreement", {"enabled": 1}, ["document_type"], distinct=1)
|
||||
@@ -998,3 +1004,7 @@ def get_sla_doctypes():
|
||||
doctypes.append(entry.document_type)
|
||||
|
||||
return doctypes
|
||||
|
||||
|
||||
def add_sla_doctypes(bootinfo):
|
||||
bootinfo.service_level_agreement_doctypes = get_sla_doctypes()
|
||||
|
||||
Reference in New Issue
Block a user