mirror of
https://github.com/frappe/erpnext.git
synced 2026-04-02 06:29:54 +00:00
fix: avoid setting unnecessary fields
(cherry picked from commit 97bb100010)
This commit is contained in:
@@ -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):
|
||||
|
||||
Reference in New Issue
Block a user