nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-08-14 23:18:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: Ignore permission while deleting user permission

Browse Source 
(cherry picked from commit 3b10ff7df7)
This commit is contained in:
Deepesh Garg
2026-07-20 17:05:21 +05:30
committed by Mergify
parent 6153202231
commit dede4a95cb
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -139,7 +139,7 @@ class Employee(NestedSet):
user = frappe.get_doc("User", existing_user_id)
validate_employee_role(user, ignore_emp_check=True)
user.save(ignore_permissions=True)
remove_user_permission("Employee", self.name, existing_user_id)
remove_user_permission("Employee", self.name, existing_user_id, ignore_permissions=True)
def after_rename(self, old, new, merge):
self.db_set("employee", new)
@@ -203,11 +203,11 @@ class Employee(NestedSet):
)
if employee_user_permission_exists and not self.create_user_permission:
remove_user_permission("Employee", self.name, self.user_id)
remove_user_permission("Company", self.company, self.user_id)
remove_user_permission("Employee", self.name, self.user_id, ignore_permissions=True)
remove_user_permission("Company", self.company, self.user_id, ignore_permissions=True)
elif not employee_user_permission_exists and self.create_user_permission:
add_user_permission("Employee", self.name, self.user_id)
add_user_permission("Company", self.company, self.user_id)
add_user_permission("Employee", self.name, self.user_id, ignore_permissions=True)
add_user_permission("Company", self.company, self.user_id, ignore_permissions=True)
def update_user(self):
# add employee role if missing