From 194d060f1398481f4debdd5c849b3a5955389222 Mon Sep 17 00:00:00 2001 From: Poovitha Palanivelu Date: Fri, 6 Mar 2026 12:27:24 +0530 Subject: [PATCH] fix: update user status depends on employee status --- erpnext/setup/doctype/employee/employee.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/erpnext/setup/doctype/employee/employee.py b/erpnext/setup/doctype/employee/employee.py index 9b4759b360a..1ecbb4b9ac7 100755 --- a/erpnext/setup/doctype/employee/employee.py +++ b/erpnext/setup/doctype/employee/employee.py @@ -183,13 +183,11 @@ class Employee(NestedSet): throw(_("Please enter relieving date.")) def validate_for_enabled_user_id(self, enabled): - if self.status != "Active": - return - if enabled is None: frappe.throw(_("User {0} does not exist").format(self.user_id)) - if enabled == 0: - frappe.throw(_("User {0} is disabled").format(self.user_id), EmployeeUserDisabledError) + + if self.status != "Active" and enabled or self.status == "Active" and enabled == 0: + frappe.set_value("User", self.user_id, "enabled", not enabled) def validate_duplicate_user_id(self): Employee = frappe.qb.DocType("Employee")