From b0f3d62dd0da7c06964a95cd61a5b115794b518d Mon Sep 17 00:00:00 2001 From: Patrick Eissler <77415730+PatrickDEissler@users.noreply.github.com> Date: Mon, 24 Feb 2025 08:47:17 +0100 Subject: [PATCH] fix: only update User Permissions if a relevant field has changed --- erpnext/setup/doctype/employee/employee.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/erpnext/setup/doctype/employee/employee.py b/erpnext/setup/doctype/employee/employee.py index 8941e4e984d..c7ecc3bc5fd 100755 --- a/erpnext/setup/doctype/employee/employee.py +++ b/erpnext/setup/doctype/employee/employee.py @@ -85,7 +85,10 @@ class Employee(NestedSet): self.reset_employee_emails_cache() def update_user_permissions(self): - if not has_permission("User Permission", ptype="write", print_logs=False): + if ( + not has_permission("User Permission", ptype="write", print_logs=False) + or (not self.has_value_changed("user_id") and not self.has_value_changed("create_user_permission")) + ): return employee_user_permission_exists = frappe.db.exists(