Merge pull request #42993 from shariquerik/apps-page-fix

fix: added app permission check for apps page
frappe · Aug 30, 2024 · 9d4ad98 · 9d4ad98
2 parents b01ac4d + 1d9ed27
commit 9d4ad98
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/erpnext/__init__.py b/erpnext/__init__.py
@@ -2,6 +2,7 @@
 import inspect
 
 import frappe
+from frappe.utils.user import is_website_user
 
 __version__ = "16.0.0-dev"
 
@@ -149,3 +150,13 @@ def caller(*args, **kwargs):
 		return frappe.get_attr(overrides[function_path][-1])(*args, **kwargs)
 
 	return caller
+
+
+def check_app_permission():
+	if frappe.session.user == "Administrator":
+		return True
+
+	if is_website_user():
+		return False
+
+	return True
diff --git a/erpnext/hooks.py b/erpnext/hooks.py
@@ -16,7 +16,7 @@
 		"logo": "/assets/erpnext/images/erpnext-logo-blue.png",
 		"title": "ERPNext",
 		"route": "/app/home",
-		# "has_permission": "erpnext.api.permission.has_app_permission"
+		"has_permission": "erpnext.check_app_permission",
 	}
 ]