perf: ensure cache works for non_standard_user_types when empty (#1…

…8665) (#18669)

(cherry picked from commit 8f057ef)

Co-authored-by: Sagar Vora <sagar@resilient.tech>

frappe/core/doctype/user_type/user_type.py

@@ -14,6 +14,12 @@ def validate(self):
 		self.set_modules()
 		self.add_select_perm_doctypes()
 
+	def clear_cache(self):
+		super().clear_cache()
+
+		if not self.is_standard:
+			frappe.cache().delete_value("non_standard_user_types")
+
 	def on_update(self):
 		if self.is_standard:
 			return
@@ -24,7 +30,6 @@ def on_update(self):
 		self.add_role_permissions_for_select_doctypes()
 		self.add_role_permissions_for_file()
 		self.update_users()
-		get_non_standard_user_type_details()
 		self.remove_permission_for_deleted_doctypes()
 
 	def on_trash(self):
@@ -184,19 +189,14 @@ def add_role_permissions(doctype, role):
 	return name
 
 
-def get_non_standard_user_type_details():
+def get_non_standard_user_types():
 	user_types = frappe.get_all(
 		"User Type",
 		fields=["apply_user_permission_on", "name", "user_id_field"],
 		filters={"is_standard": 0},
 	)
 
-	if user_types:
-		user_type_details = {d.name: [d.apply_user_permission_on, d.user_id_field] for d in user_types}
-
-		frappe.cache().set_value("non_standard_user_types", user_type_details)
-
-		return user_type_details
+	return {d.name: [d.apply_user_permission_on, d.user_id_field] for d in user_types}
 
 
 @frappe.whitelist()
@@ -290,10 +290,10 @@ def apply_permissions_for_non_standard_user_type(doc, method=None):
 	if not frappe.db.table_exists("User Type"):
 		return
 
-	user_types = frappe.cache().get_value("non_standard_user_types")
-
-	if not user_types:
-		user_types = get_non_standard_user_type_details()
+	user_types = frappe.cache().get_value(
+		"non_standard_user_types",
+		get_non_standard_user_types,
+	)
 
 	if not user_types:
 		return