From 97bb10001002d5bceeab42c2e149d8ad484cf322 Mon Sep 17 00:00:00 2001 From: Rucha Mahabal Date: Mon, 23 Mar 2026 15:06:15 +0530 Subject: [PATCH] fix: avoid setting unnecessary fields --- erpnext/setup/doctype/employee/employee.py | 6 ------ 1 file changed, 6 deletions(-) diff --git a/erpnext/setup/doctype/employee/employee.py b/erpnext/setup/doctype/employee/employee.py index 880d7aed284..5d2c9b4ae61 100755 --- a/erpnext/setup/doctype/employee/employee.py +++ b/erpnext/setup/doctype/employee/employee.py @@ -431,8 +431,6 @@ def create_user(employee: str, email: str | None = None, create_user_permission: if emp.user_id: frappe.throw(_("Employee {0} already has a linked user").format(emp.name)) - if not email: - email = emp.company_email if not email: frappe.throw(_("Email is required to create a user")) @@ -467,10 +465,6 @@ def create_user(employee: str, email: str | None = None, create_user_permission: emp.user_id = user.name emp.create_user_permission = cint(create_user_permission) - if not emp.company_email: - emp.company_email = email - if not emp.prefered_contact_email: - emp.prefered_contact_email = "Company Email" emp.save() if cint(create_user_permission):