diff --git a/DigitalLearningSolutions.Web/Controllers/SuperAdmin/Centres/CentresController.cs b/DigitalLearningSolutions.Web/Controllers/SuperAdmin/Centres/CentresController.cs index 60a377b0f8..3d40ddfd2d 100644 --- a/DigitalLearningSolutions.Web/Controllers/SuperAdmin/Centres/CentresController.cs +++ b/DigitalLearningSolutions.Web/Controllers/SuperAdmin/Centres/CentresController.cs @@ -233,18 +233,20 @@ public IActionResult EditCentreDetails(int centreId = 0) [Route("SuperAdmin/Centres/{centreId=0:int}/EditCentreDetails")] public IActionResult EditCentreDetails(EditCentreDetailsSuperAdminViewModel model) { - var centres = centresService.GetAllCentres().ToList(); - bool isExistingCentreName = centres.Where(center => center.Item1 == model.CentreId) - .Select(center => center.Item2) - .FirstOrDefault() - .Equals(model.CentreName.Trim()); - bool isCentreNamePresent = centres.Any(center => string.Equals(center.Item2.Trim(), model.CentreName?.Trim(), StringComparison.OrdinalIgnoreCase)); - - if (isCentreNamePresent && !isExistingCentreName) + if (!string.IsNullOrEmpty(model.CentreName)) { - ModelState.AddModelError("CentreName", CommonValidationErrorMessages.CentreNameAlreadyExist); - } + var centres = centresService.GetAllCentres().ToList(); + bool isExistingCentreName = centres.Where(center => center.Item1 == model.CentreId) + .Select(center => center.Item2) + .FirstOrDefault() + .Equals(model.CentreName.Trim()); + bool isCentreNamePresent = centres.Any(center => string.Equals(center.Item2.Trim(), model.CentreName?.Trim(), StringComparison.OrdinalIgnoreCase)); + if (isCentreNamePresent && !isExistingCentreName) + { + ModelState.AddModelError("CentreName", CommonValidationErrorMessages.CentreNameAlreadyExist); + } + } if (!ModelState.IsValid) { var regions = regionService.GetRegionsAlphabetical().ToList();