From a35ce12d6000152feef4e87a15046d63a7b85d7e Mon Sep 17 00:00:00 2001 From: Shariq Ansari Date: Fri, 30 Aug 2024 16:51:26 +0530 Subject: [PATCH] fix: added app permission check for apps page (cherry picked from commit e8f8fb8a8f68f365a57f870697a5c81933d11c2c) --- erpnext/__init__.py | 11 +++++++++++ erpnext/hooks.py | 2 +- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/erpnext/__init__.py b/erpnext/__init__.py index 4a9236066cc..40fde01861b 100644 --- a/erpnext/__init__.py +++ b/erpnext/__init__.py @@ -2,6 +2,7 @@ import functools import inspect import frappe +from frappe.utils.user import is_website_user __version__ = "15.28.2" @@ -149,3 +150,13 @@ def allow_regional(fn): 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 \ No newline at end of file diff --git a/erpnext/hooks.py b/erpnext/hooks.py index 448d0d2a57b..038f58bfcb9 100644 --- a/erpnext/hooks.py +++ b/erpnext/hooks.py @@ -16,7 +16,7 @@ add_to_apps_screen = [ "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" } ]