fix: avoid setting unnecessary fields

(cherry picked from commit 97bb100010)
This commit is contained in:
Rucha Mahabal
2026-03-23 15:06:15 +05:30
committed by Mergify
parent e8ca394e8b
commit 3023302700

View File

@@ -429,8 +429,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"))
@@ -465,10 +463,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):