fix: sync employee user status after save

This commit is contained in:
Mihir Kandoi
2026-06-11 20:58:35 +05:30
parent 559585fb7b
commit e40999c879

View File

@@ -64,6 +64,9 @@ class Employee(NestedSet):
)
def validate_user_details(self):
if not self.user_id:
return
self.validate_for_enabled_user_id()
self.validate_duplicate_user_id()
@@ -76,6 +79,7 @@ class Employee(NestedSet):
if self.user_id:
self.update_user()
self.update_user_permissions()
self.update_user_status()
self.reset_employee_emails_cache()
def update_user_permissions(self):
@@ -181,10 +185,15 @@ class Employee(NestedSet):
if not frappe.db.exists("User", self.user_id):
frappe.throw(_("User {0} does not exist").format(self.user_id))
def update_user_status(self):
if not self.user_id:
return
user = frappe.get_doc("User", self.user_id)
enabled = user.enabled
if self.status != "Active" and enabled or self.status == "Active" and enabled == 0:
user.enabled = not enabled
# Keep linked User status in sync from the Employee lifecycle and record the audit log.
user.save(ignore_permissions=True)
def validate_duplicate_user_id(self):