SCJ-244: Limit trial length to 40 days (#215)

bcgov · Jul 25, 2024 · 54e2c42 · 54e2c42
1 parent 2a79b74
commit 54e2c42
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 3 deletions.
diff --git a/app/Controllers/ScBookingController.cs b/app/Controllers/ScBookingController.cs
@@ -143,6 +143,14 @@ public async Task<IActionResult> BookingType(ScBookingTypeViewModel model)
                     );
                 }
 
+                if (model.EstimatedTrialLength > 40)
+                {
+                    ModelState.AddModelError(
+                        "EstimatedTrialLength",
+                        "Estimated trial length must be between 1 and 40 days."
+                    );
+                }
+
                 if (model.IsHomeRegistry == null)
                 {
                     ModelState.AddModelError(

diff --git a/app/Services/DataWriterService.cs b/app/Services/DataWriterService.cs
@@ -75,11 +75,11 @@ SessionUserInfo userInfo
             if (
                 !bookingInfo.EstimatedTrialLength.HasValue
                 || bookingInfo.EstimatedTrialLength.Value < 1
-                || bookingInfo.EstimatedTrialLength.Value > 99
+                || bookingInfo.EstimatedTrialLength.Value > 40
             )
             {
                 throw new InvalidOperationException(
-                    "HearingLength must be greater than zero and less than 100"
+                    "HearingLength must be greater than zero and less than 41"
                 );
             }
 

diff --git a/app/Views/ScBooking/BookingType.cshtml b/app/Views/ScBooking/BookingType.cshtml
@@ -76,7 +76,7 @@
 
                         <div>
                             <input class="mr-2 form-control d-inline-block EstimatedTrialLength" type="number" min="1"
-                                asp-for="EstimatedTrialLength" />
+                                max="40" asp-for="EstimatedTrialLength" />
                             <span>days</span>
 
                             <div class="mt-2">