schuetz3/schuetz3-erpnext
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #45924 from PatrickDEissler/develop

Browse Source 
fix(Employee): remove User Permissions if create_user_permission is unchecked
This commit is contained in:
Raffael Meyer
2025-03-14 13:57:42 +01:00
committed by GitHub
parent 3c7583495e 3876cf0c2b
commit 3be570b44a
1 changed files with 9 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -85,20 +85,21 @@ class Employee(NestedSet):
self.reset_employee_emails_cache()
def update_user_permissions(self):
if not self.create_user_permission:
return
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(
"User Permission", {"allow": "Employee", "for_value": self.name, "user": self.user_id}
)
if employee_user_permission_exists:
return
add_user_permission("Employee", self.name, self.user_id)
add_user_permission("Company", self.company, self.user_id)
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)
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)
def update_user(self):
# add employee role if missing