nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-02-21 18:36:30 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: do not check for permission if values are not changed in employee doctype (#47238)

Browse Source
This commit is contained in:
Lakshit Jain
2025-04-24 20:33:30 +05:30
committed by GitHub
parent 4a29a54804
commit 0caba9f70d
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
erpnext/setup/doctype/employee/employee.py
View File

@@ -85,9 +85,10 @@ class Employee(NestedSet):
self.reset_employee_emails_cache() self.reset_employee_emails_cache()
def update_user_permissions(self): def update_user_permissions(self):
if not has_permission("User Permission", ptype="write") or ( if not self.has_value_changed("user_id") and not self.has_value_changed("create_user_permission"):
not self.has_value_changed("user_id") and not self.has_value_changed("create_user_permission") return
):
if not has_permission("User Permission", ptype="write", raise_exception=False):
return return
employee_user_permission_exists = frappe.db.exists( employee_user_permission_exists = frappe.db.exists(