Fixed validation issues.

Author: martinfoster-hee

DigitalLearningSolutions.Web/Controllers/SuperAdmin/Centres/CentresController.cs

@@ -360,31 +360,39 @@ public IActionResult CentreRoleLimits(int centreId = 0)
         [Route("SuperAdmin/Centres/{centreId=0:int}/CentreRoleLimits")]
         public IActionResult EditCentreRoleLimits(CentreRoleLimitsViewModel model)
         {
-            if (!ModelState.IsValid)
+            if (model.IsRoleLimitSetCmsAdministrators && model.RoleLimitCmsAdministrators == null)
             {
-                return View("CentreRoleLimits", model);
+                ModelState["RoleLimitCmsAdministrators.RoleLimitCmsAdministrators"]?.Errors.Clear();
             }
+            model.RoleLimitCmsAdministrators ??= -1;
 
-            if (!(model.IsRoleLimitSetCmsAdministrators))
+            if (model.IsRoleLimitSetCmsManagers && model.RoleLimitCmsManagers == null)
             {
-                model.RoleLimitCmsAdministrators = -1;
+                ModelState["RoleLimitCmsManagers.RoleLimitCmsManagers"]?.Errors.Clear();
             }
+            model.RoleLimitCmsManagers ??= -1;
 
-            if (!(model.IsRoleLimitSetCmsManagers))
+            if (model.IsRoleLimitSetContentCreatorLicences && model.RoleLimitContentCreatorLicences == null)
             {
-                model.RoleLimitCmsManagers = -1;
+                ModelState["RoleLimitContentCreatorLicences.RoleLimitContentCreatorLicences"]?.Errors.Clear();
             }
-            if (!(model.IsRoleLimitSetContentCreatorLicences))
+            model.RoleLimitContentCreatorLicences ??= -1;
+
+            if (model.IsRoleLimitSetCustomCourses && model.RoleLimitCustomCourses == null)
             {
-                model.RoleLimitContentCreatorLicences = -1;
+                ModelState["RoleLimitCustomCourses.RoleLimitCustomCourses"]?.Errors.Clear();
             }
-            if (!(model.IsRoleLimitSetCustomCourses))
+            model.RoleLimitCustomCourses ??= -1;
+
+            if (model.IsRoleLimitSetTrainers && model.RoleLimitTrainers == null)
             {
-                model.RoleLimitCustomCourses = -1;
+                ModelState["RoleLimitTrainers.RoleLimitTrainers"]?.Errors.Clear();
             }
-            if (!(model.IsRoleLimitSetTrainers))
+            model.RoleLimitTrainers ??= -1;
+
+            if (!ModelState.IsValid)
             {
-                model.RoleLimitTrainers = -1;
+                return View("CentreRoleLimits", model);
             }
 
             centresDataService.UpdateCentreRoleLimits(

DigitalLearningSolutions.Web/ViewModels/SuperAdmin/Centres/CentreRoleLimitsViewModel.cs

@@ -11,24 +11,19 @@ public class CentreRoleLimitsViewModel
         public bool IsRoleLimitSetCustomCourses { get; set; }
         public bool IsRoleLimitSetTrainers { get; set; }
 
-        [Required(ErrorMessage = "Please enter a number for the CMS Administrators role limit.")]
         [Range(-1, int.MaxValue, ErrorMessage = "The role limit must be a non-negative whole number.")]
-        public int RoleLimitCmsAdministrators { get; set; }
+        public int? RoleLimitCmsAdministrators { get; set; }
 
-        [Required(ErrorMessage = "Please enter a number for the CMS Managers role limit.")]
         [Range(-1, int.MaxValue, ErrorMessage = "The role limit must be a non-negative whole number.")]
-        public int RoleLimitCmsManagers { get; set; }
+        public int? RoleLimitCmsManagers { get; set; }
 
-        [Required(ErrorMessage = "Please enter a number for the Content Creator Licences role limit.")]
         [Range(-1, int.MaxValue, ErrorMessage = "The role limit must be a non-negative whole number.")]
-        public int RoleLimitContentCreatorLicences { get; set; }
+        public int? RoleLimitContentCreatorLicences { get; set; }
 
-        [Required(ErrorMessage = "Please enter a number for the Custom Courses role limit.")]
         [Range(-1, int.MaxValue, ErrorMessage = "The role limit must be a non-negative whole number.")]
-        public int RoleLimitCustomCourses { get; set; }
+        public int? RoleLimitCustomCourses { get; set; }
 
-        [Required(ErrorMessage = "Please enter a number for the Trainers role limit.")]
         [Range(-1, int.MaxValue, ErrorMessage = "The role limit must be a non-negative whole number.")]
-        public int RoleLimitTrainers { get; set; }
+        public int? RoleLimitTrainers { get; set; }
     }
 }