From 7e3f0815a09f97da4d2bc56a000e293ce03566ae Mon Sep 17 00:00:00 2001 From: "ashutosh.nehete" Date: Mon, 5 May 2025 19:05:16 +0530 Subject: [PATCH] Fixed issue of returing email already existed even if entered new email --- Marco.Pms.Services/Controllers/EmployeeController.cs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Marco.Pms.Services/Controllers/EmployeeController.cs b/Marco.Pms.Services/Controllers/EmployeeController.cs index 8b0eba5..7aa4e3b 100644 --- a/Marco.Pms.Services/Controllers/EmployeeController.cs +++ b/Marco.Pms.Services/Controllers/EmployeeController.cs @@ -207,9 +207,9 @@ namespace MarcoBMS.Services.Controllers if (model.Email != null) { // 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 demo = existingUser != new IdentityUser(); if (existingUser != null) { /* Identity user Exists - Create/update employee Employee */ @@ -229,7 +229,7 @@ namespace MarcoBMS.Services.Controllers // Create Employee record if missing //Employee newEmployee = GetNewEmployeeModel(model, TenantId, existingUser.Id); //_context.Employees.Add(newEmployee); - return BadRequest(ApiResponse.ErrorResponse("Email already exist", "Email already exist", 400)); + return Conflict(ApiResponse.ErrorResponse("Email already exist", "Email already exist", 409)); }