From 75001025645d112abd9930b7a0ec8c4b45a25968 Mon Sep 17 00:00:00 2001
From: Dimitar Byalkov <CapitalRhino@users.noreply.github.com>
Date: Sat, 9 Apr 2022 05:56:40 +0300
Subject: [PATCH] Working version: Rents done

---
 .../WebApp/Controllers/RentsController.cs     | 59 ++++++++++++++-----
 RentACar/WebApp/Views/Rents/Create.cshtml     |  6 +-
 RentACar/WebApp/Views/Rents/Edit.cshtml       |  6 +-
 3 files changed, 50 insertions(+), 21 deletions(-)

diff --git a/RentACar/WebApp/Controllers/RentsController.cs b/RentACar/WebApp/Controllers/RentsController.cs
index 79b9f1d..6aff750 100644
--- a/RentACar/WebApp/Controllers/RentsController.cs
+++ b/RentACar/WebApp/Controllers/RentsController.cs
@@ -8,6 +8,7 @@ using Microsoft.EntityFrameworkCore;
 using Data;
 using Data.Entities;
 using Microsoft.AspNetCore.Authorization;
+using System.Text;
 
 namespace WebApp.Controllers
 {
@@ -61,15 +62,27 @@ namespace WebApp.Controllers
         [Authorize]
         public async Task<IActionResult> Create(Rents rents)
         {
-            
             if (ModelState.IsValid)
             {
-                var car = _context.Cars.FirstOrDefault(car => car.Id == 1);
-                rents.Car = car;
-                var user = _context.Users.FirstOrDefault(user => user.UserName == User.Identity.Name);
-                rents.User = user;
-                _context.Add(rents);
-                await _context.SaveChangesAsync();
+                try
+                {
+                    if (CarIsAvailable(rents.CarId, rents.StartDate, rents.EndDate))
+                    {
+                        var car = _context.Cars.FirstOrDefault(car => car.Id == rents.CarId);
+                        rents.Car = car;
+                        var user = _context.Users.FirstOrDefault(user => user.UserName == User.Identity.Name);
+                        rents.User = user;
+                        _context.Add(rents);
+                    }
+                    else throw new ArgumentOutOfRangeException("Selected car is unavailable at the chosen times.");
+                    await _context.SaveChangesAsync();
+                }
+                catch (ArgumentOutOfRangeException e)
+                {
+                    byte[] data = Encoding.UTF8.GetBytes(e.ParamName + " <a href=\"javascript:history.back()\">Back</a>");
+                    Response.ContentType = "text/html";
+                    await Response.Body.WriteAsync(data, 0, data.Length);
+                }
                 return RedirectToAction(nameof(Index));
             }
             return View(rents);
@@ -104,8 +117,12 @@ namespace WebApp.Controllers
             {
                 try
                 {
-                    _context.Update(rents);
-                    await _context.SaveChangesAsync();
+                    if (CarIsAvailable(rents.CarId, rents.StartDate, rents.EndDate))
+                    {
+                        _context.Update(rents);
+                        await _context.SaveChangesAsync();
+                    }
+                    else throw new ArgumentOutOfRangeException("Selected car is unavailable at the chosen times.");
                 }
                 catch (DbUpdateConcurrencyException)
                 {
@@ -118,6 +135,12 @@ namespace WebApp.Controllers
                         throw;
                     }
                 }
+                catch (ArgumentOutOfRangeException ex)
+                {
+                    byte[] data = Encoding.UTF8.GetBytes(ex.ParamName + " <a href=\"javascript:history.back()\">Back</a>");
+                    Response.ContentType = "text/html";
+                    await Response.Body.WriteAsync(data, 0, data.Length);
+                }
                 return RedirectToAction(nameof(Index));
             }
             return View(rents);
@@ -160,16 +183,22 @@ namespace WebApp.Controllers
         }
         public bool CarIsAvailable(int carId, DateTime startDate, DateTime endDate)
         {
-            var rents = _context.Rents.Where(x => x.Car.Id == carId);
-            if (rents != null)
+            var rents = _context.Rents.Where(x => x.CarId == carId);
+            foreach (var item in rents)
             {
-                foreach (var item in rents)
+                if (item.StartDate > item.EndDate)
+                    throw new ArgumentException("A start can not be after its end.");
+
+                if (startDate > endDate)
+                    throw new ArgumentException("B start can not be after its end.");
+
+                if (!((item.EndDate < startDate && item.StartDate < startDate) ||
+                            (endDate < item.StartDate && startDate < item.StartDate)))
                 {
-                    return !(item.StartDate < endDate && startDate < item.EndDate);
+                    return false;
                 }
-                return false;
             }
-            else return true;
+            return true;
         }
     }
 }
diff --git a/RentACar/WebApp/Views/Rents/Create.cshtml b/RentACar/WebApp/Views/Rents/Create.cshtml
index ea7883b..7bf5491 100644
--- a/RentACar/WebApp/Views/Rents/Create.cshtml
+++ b/RentACar/WebApp/Views/Rents/Create.cshtml
@@ -10,9 +10,9 @@
     <div class="col-md-4">
         <form asp-action="Create">
             <div class="form-group">
-                <label asp-for="Car.Id" class="control-label"></label>
-                <input asp-for="Car.Id" class="form-control" />
-                <span asp-validation-for="Car.Id" class="text-danger"></span>
+                <label asp-for="CarId" class="control-label"></label>
+                <input asp-for="CarId" class="form-control" />
+                <span asp-validation-for="CarId" class="text-danger"></span>
             </div>
             <div asp-validation-summary="ModelOnly" class="text-danger"></div>
             <div class="form-group">
diff --git a/RentACar/WebApp/Views/Rents/Edit.cshtml b/RentACar/WebApp/Views/Rents/Edit.cshtml
index 2b0d787..188d356 100644
--- a/RentACar/WebApp/Views/Rents/Edit.cshtml
+++ b/RentACar/WebApp/Views/Rents/Edit.cshtml
@@ -12,9 +12,9 @@
             <div asp-validation-summary="ModelOnly" class="text-danger"></div>
             <input type="hidden" asp-for="Id" />
             <div class="form-group">
-                <label asp-for="Car.Id" class="control-label"></label>
-                <input asp-for="Car.Id" class="form-control" />
-                <span asp-validation-for="Car.Id" class="text-danger"></span>
+                <label asp-for="CarId" class="control-label"></label>
+                <input asp-for="CarId" class="form-control" />
+                <span asp-validation-for="CarId" class="text-danger"></span>
             </div>
             <div class="form-group">
                 <label asp-for="StartDate" class="control-label"></label>