Fixed issue of returing email already existed even if entered new email
This commit is contained in:
parent
e22db781a1
commit
7e3f0815a0
@ -207,9 +207,9 @@ namespace MarcoBMS.Services.Controllers
|
|||||||
if (model.Email != null)
|
if (model.Email != null)
|
||||||
{
|
{
|
||||||
// Check if user already exists by email
|
// Check if user already exists by email
|
||||||
IdentityUser existingUser = await _userHelper.GetRegisteredUser(model.Email) ?? new IdentityUser();
|
IdentityUser? existingUser = await _userHelper.GetRegisteredUser(model.Email);
|
||||||
var existingEmployee = await _context.Employees.FirstOrDefaultAsync(e => e.Id == model.Id && e.IsActive == true);
|
var existingEmployee = await _context.Employees.FirstOrDefaultAsync(e => e.Id == model.Id && e.IsActive == true);
|
||||||
|
var demo = existingUser != new IdentityUser();
|
||||||
if (existingUser != null)
|
if (existingUser != null)
|
||||||
{
|
{
|
||||||
/* Identity user Exists - Create/update employee Employee */
|
/* Identity user Exists - Create/update employee Employee */
|
||||||
@ -229,7 +229,7 @@ namespace MarcoBMS.Services.Controllers
|
|||||||
// Create Employee record if missing
|
// Create Employee record if missing
|
||||||
//Employee newEmployee = GetNewEmployeeModel(model, TenantId, existingUser.Id);
|
//Employee newEmployee = GetNewEmployeeModel(model, TenantId, existingUser.Id);
|
||||||
//_context.Employees.Add(newEmployee);
|
//_context.Employees.Add(newEmployee);
|
||||||
return BadRequest(ApiResponse<object>.ErrorResponse("Email already exist", "Email already exist", 400));
|
return Conflict(ApiResponse<object>.ErrorResponse("Email already exist", "Email already exist", 409));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user