Fix issues from AI

taxi/forms.py

@@ -27,7 +27,7 @@ def clean_license_number(self):
         ):
             raise forms.ValidationError(
                 "Ensure that license_number start with 3 UPPER"
-                "symbols and last 5 characters are digits"
+                " symbols and last 5 characters are digits"
             )
 
         return license_number

taxi/urls.py

@@ -53,7 +53,7 @@
         "drivers/<int:pk>/", DriverDetailView.as_view(), name="driver-detail"
     ),
     path(
-        "drivers/create", DriverCreateView.as_view(), name="driver-create"
+        "drivers/create/", DriverCreateView.as_view(), name="driver-create"
     ),
     path("drivers/<int:pk>/delete/",
          DriverDeleteView.as_view(), name="driver-delete"),

taxi/views.py

@@ -113,7 +113,7 @@ class DriverCreateView(LoginRequiredMixin, generic.CreateView):
     success_url = reverse_lazy("taxi:driver-list")
 
 
-class DriverDeleteView(generic.DeleteView):
+class DriverDeleteView(LoginRequiredMixin, generic.DeleteView):
     model = Driver
     success_url = reverse_lazy("taxi:driver-list")
 

templates/taxi/car_confirm_delete.html

@@ -7,4 +7,5 @@ <h1>Delete car?</h1>
 
     <input type="submit" value="Yes" class="btn btn-danger">
   </form>
+
 {% endblock %}

templates/taxi/car_detail.html

@@ -27,4 +27,5 @@ <h1>Drivers</h1>
       <li>{{ driver.username }} ({{ driver.first_name }} {{ driver.last_name }})</li>
     {% endfor %}
   </ul>
+
 {% endblock %}

templates/taxi/car_form.html

@@ -9,4 +9,5 @@ <h1>{{ object|yesno:"Update,Create" }} car</h1>
 
     <input type="submit" value="Submit" class="btn btn-primary">
   </form>
+
 {% endblock %}

templates/taxi/car_list.html

@@ -19,4 +19,5 @@ <h1>
   {% else %}
     <p>There are no cars in taxi</p>
   {% endif %}
+
 {% endblock %}

templates/taxi/driver_confirm_delete.html

@@ -7,4 +7,5 @@ <h1>Delete driver?</h1>
 
     <input type="submit" value="Yes" class="btn btn-danger">
   </form>
+
 {% endblock %}

templates/taxi/driver_detail.html

@@ -10,11 +10,11 @@ <h1>
   <p><strong>License number:</strong> {{ driver.license_number }}</p>
   <p><strong>Is staff:</strong> {{ driver.is_staff }}</p>
 
-  <a href="{% url 'taxi:driver-delete' pk=driver.id %}" class="btn btn-danger">
+  <a href="{% url 'taxi:driver-delete' pk=driver.pk %}" class="btn btn-danger">
     Delete
   </a>
 
-  <a href="{% url 'taxi:driver-update' pk=driver.id %}" class="btn btn-secondary">
+  <a href="{% url 'taxi:driver-update' pk=driver.pk %}" class="btn btn-secondary">
     Update
   </a>
 

templates/taxi/driver_form.html

@@ -9,4 +9,5 @@ <h1>{{ object|yesno:"Update,Create" }} driver</h1>
 
     <input type="submit" value="Submit" class="btn btn-primary">
   </form>
+
 {% endblock %}

templates/taxi/driver_list.html

@@ -1,7 +1,7 @@
 {% extends "base.html" %}
 
 {% block content %}
-    <h1>Driver List <a style="float:rigt" href="{% url 'taxi:driver-create' %}">+</a></h1>
+    <h1>Driver List <a style="float:right" href="{% url 'taxi:driver-create' %}">+</a></h1>
 
     {% if driver_list %}
     <table class="table">
@@ -26,4 +26,5 @@ <h1>Driver List <a style="float:rigt" href="{% url 'taxi:driver-create' %}">+</a
     {% else %}
       <p>There are no drivers in the service.</p>
     {% endif %}
+
 {% endblock %}

templates/taxi/driver_update_license.html

@@ -9,4 +9,5 @@ <h1>Update License</h1>
 
     <input type="submit" value="Submit" class="btn btn-primary">
   </form>
+
 {% endblock %}