nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-05-25 07:54:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #53280 from frappe/mergify/bp/version-16-hotfix/pr-53203

Browse Source 
fix: update user status depends on employee status (backport #53203)
This commit is contained in:
Mihir Kandoi
2026-03-10 13:25:59 +05:30
committed by GitHub
parent 1a46a8fec0 c7da8e2471
commit 005ff0d6a8
1 changed files with 3 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -183,13 +183,11 @@ class Employee(NestedSet):
throw(_("Please enter relieving date.")) throw(_("Please enter relieving date."))
def validate_for_enabled_user_id(self, enabled): def validate_for_enabled_user_id(self, enabled):
if self.status != "Active":
return
if enabled is None: if enabled is None:
frappe.throw(_("User {0} does not exist").format(self.user_id)) 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): def validate_duplicate_user_id(self):
Employee = frappe.qb.DocType("Employee") Employee = frappe.qb.DocType("Employee")