mirror of
https://github.com/frappe/erpnext.git
synced 2026-04-09 01:45:09 +00:00
fix: avoid setting unnecessary fields
This commit is contained in:
@@ -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):
|
||||
|
||||
Reference in New Issue
Block a user