check that deadline is in the future, not in the past

miles-no · Apr 18, 2024 · 3ef9b70 · 3ef9b70
1 parent 91de8ee
commit 3ef9b70
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/kabinizer-back-end/kabinizer-api/Controllers/BookingRequestController.cs b/kabinizer-back-end/kabinizer-api/Controllers/BookingRequestController.cs
@@ -46,11 +46,11 @@ public void AddBookingRequests([Required] IEnumerable<CreateBookingRequestDto> r
         {
             throw new Exception("One or more periods are not part of a draw");
         }
-
-        var periodsWithCompletedDraw = periods.Where(p => p.Draw?.DeadlineEnd < DateTime.Now).ToList();
-        if (periodsWithCompletedDraw.Count != 0)
+        
+        var periodsWithEndedDraw = periods.Where(p => p.Draw?.DeadlineEnd > DateTime.Now).ToList();
+        if (periodsWithEndedDraw.Count != 0)
         {
-            throw new Exception("One or more draws are already completed");
+            throw new Exception("Cannot make a booking request for a draw that has ended");
         }
 
         var bookingRequestEntities = periodIds.Select(id => new BookingRequestEntity(currentUserId, id));