From 0e400bdb1bd99443946da87063b3087565ad2b39 Mon Sep 17 00:00:00 2001
From: Halyna Baklanova <novabalka@gmail.com>
Date: Mon, 16 Sep 2024 11:14:32 +0300
Subject: [PATCH] Solution

---
 taxi/views.py                          | 18 +++++++++++------
 taxi_service/settings.py               |  4 ++++
 taxi_service/urls.py                   |  4 +++-
 templates/includes/sidebar.html        | 27 ++++++++++++++++++++++----
 templates/registration/logged_out.html |  6 ++++++
 templates/registration/login.html      | 13 +++++++++++++
 templates/taxi/car_list.html           |  2 +-
 templates/taxi/driver_list.html        |  4 ++--
 templates/taxi/index.html              |  1 +
 9 files changed, 65 insertions(+), 14 deletions(-)
 create mode 100644 templates/registration/logged_out.html
 create mode 100644 templates/registration/login.html

diff --git a/taxi/views.py b/taxi/views.py
index 82ad312f..e3f47da9 100644
--- a/taxi/views.py
+++ b/taxi/views.py
@@ -1,47 +1,53 @@
+from django.contrib.auth import logout
+from django.contrib.auth.decorators import login_required
+from django.contrib.auth.mixins import LoginRequiredMixin
 from django.shortcuts import render
 from django.views import generic
 
 from .models import Driver, Car, Manufacturer
 
 
+@login_required
 def index(request):
     """View function for the home page of the site."""
-
     num_drivers = Driver.objects.count()
     num_cars = Car.objects.count()
     num_manufacturers = Manufacturer.objects.count()
+    num_visits = request.session.get("num_visits", 0)
+    request.session["num_visits"] = num_visits + 1
 
     context = {
         "num_drivers": num_drivers,
         "num_cars": num_cars,
         "num_manufacturers": num_manufacturers,
+        "num_visits": num_visits + 1,
     }
 
     return render(request, "taxi/index.html", context=context)
 
 
-class ManufacturerListView(generic.ListView):
+class ManufacturerListView(LoginRequiredMixin, generic.ListView):
     model = Manufacturer
     context_object_name = "manufacturer_list"
     template_name = "taxi/manufacturer_list.html"
     paginate_by = 5
 
 
-class CarListView(generic.ListView):
+class CarListView(LoginRequiredMixin, generic.ListView):
     model = Car
     paginate_by = 5
     queryset = Car.objects.select_related("manufacturer")
 
 
-class CarDetailView(generic.DetailView):
+class CarDetailView(LoginRequiredMixin, generic.DetailView):
     model = Car
 
 
-class DriverListView(generic.ListView):
+class DriverListView(LoginRequiredMixin, generic.ListView):
     model = Driver
     paginate_by = 5
 
 
-class DriverDetailView(generic.DetailView):
+class DriverDetailView(LoginRequiredMixin, generic.DetailView):
     model = Driver
     queryset = Driver.objects.prefetch_related("cars__manufacturer")
diff --git a/taxi_service/settings.py b/taxi_service/settings.py
index b6c0cf19..9884baec 100644
--- a/taxi_service/settings.py
+++ b/taxi_service/settings.py
@@ -133,3 +133,7 @@
 # https://docs.djangoproject.com/en/4.0/ref/settings/#default-auto-field
 
 DEFAULT_AUTO_FIELD = "django.db.models.BigAutoField"
+
+AUTH_USER_MODEL = "taxi.Driver"
+
+LOGIN_REDIRECT_URL = "/taxi/"
diff --git a/taxi_service/urls.py b/taxi_service/urls.py
index 8b94449f..7978b82f 100644
--- a/taxi_service/urls.py
+++ b/taxi_service/urls.py
@@ -20,6 +20,8 @@
 
 
 urlpatterns = [
-    path("admin/", admin.site.urls),
     path("", include("taxi.urls", namespace="taxi")),
+    path("admin/", admin.site.urls),
+    path("taxi/", include("taxi.urls", namespace="taxi")),
+    path("accounts/", include("django.contrib.auth.urls")),
 ] + static(settings.STATIC_URL, document_root=settings.STATIC_ROOT)
diff --git a/templates/includes/sidebar.html b/templates/includes/sidebar.html
index b7cd72dc..143dc7b1 100644
--- a/templates/includes/sidebar.html
+++ b/templates/includes/sidebar.html
@@ -1,6 +1,25 @@
 <ul class="sidebar-nav">
-  <li><a href="{% url "taxi:index" %}">Home page</a></li>
-  <li><a href="{% url "taxi:manufacturer-list" %}">Manufacturers</a></li>
-  <li><a href="{% url "taxi:car-list" %}">Cars</a></li>
-  <li><a href="{% url "taxi:driver-list" %}">Drivers</a></li>
+{% if user.is_authenticated %}
+  
+      <a href="{% url "taxi:driver-detail" pk=user.id %}">
+        {{ user.username }} - {{ user.first_name }} {{ user.last_name }}
+      </a>
+      <br>
+      <br>
+      <a href="{% url "logout" %}">Logout</a>
+ 
+  {% else %}
+    <li>
+      <a href="{% url "login" %}">Login</a>
+    </li>
+  {% endif %}
+  {% if user.is_authenticated %}
+
+    <br>
+  {% endif %}
+  <br>
+  <a href="{% url "taxi:index" %}">Home page</a><br>
+  <a href="{% url "taxi:manufacturer-list" %}">Manufacturers</a><br>
+  <a href="{% url "taxi:car-list" %}">Cars</a><br>
+  <a href="{% url "taxi:driver-list" %}">Drivers</a>
 </ul>
diff --git a/templates/registration/logged_out.html b/templates/registration/logged_out.html
new file mode 100644
index 00000000..5fc1872a
--- /dev/null
+++ b/templates/registration/logged_out.html
@@ -0,0 +1,6 @@
+{% extends "base.html" %}
+
+{% block content %}
+  <p>Logged out</p>
+  <a href="{% url "login" %}">Click here to log in again</a>
+{% endblock %}
diff --git a/templates/registration/login.html b/templates/registration/login.html
new file mode 100644
index 00000000..50d20c5a
--- /dev/null
+++ b/templates/registration/login.html
@@ -0,0 +1,13 @@
+{% extends "base.html" %}
+
+{% block content %}
+  <h1>Login</h1>
+  {% if form.errors %}
+    <p style="color: red">Invalid credentials</p>
+  {% endif %}
+  <form action="{% url "login" %}" method="post">
+    {% csrf_token %}
+    {{ form.as_p }}
+    <input type="submit" value="Submit">
+  </form>
+{% endblock  %}
diff --git a/templates/taxi/car_list.html b/templates/taxi/car_list.html
index e107b579..c32beb01 100644
--- a/templates/taxi/car_list.html
+++ b/templates/taxi/car_list.html
@@ -6,7 +6,7 @@ <h1>Car list</h1>
     <ul>
       {% for car in car_list %}
         <li>
-          <a href="{% url "taxi:car-detail" pk=car.id %} ">{{ car.id }}</a>
+          <a href="{% url "taxi:car-detail" pk=car.id %}">{{ car.id }}</a>
           {{ car.model }} ({{ car.manufacturer.name }})
         </li>
       {% endfor %}
diff --git a/templates/taxi/driver_list.html b/templates/taxi/driver_list.html
index c6c3e823..a190220f 100644
--- a/templates/taxi/driver_list.html
+++ b/templates/taxi/driver_list.html
@@ -6,8 +6,8 @@ <h1>Driver list</h1>
     <ul>
       {% for driver in driver_list%}
         <li>
-          <a href="{% url "taxi:driver-detail" pk=driver.id %}">{{ driver.username }}</a>
-          ({{ driver.first_name }} {{ driver.last_name }})
+          <a href="{% url "taxi:driver-detail" driver.pk %}">{{ driver.username }}</a>
+          ({{ driver.first_name }} {{ driver.last_name }}) {% if driver.id == user.id %} (Me) {% endif %}
         </li>
       {% endfor %}
     </ul>
diff --git a/templates/taxi/index.html b/templates/taxi/index.html
index 13c59aa8..4d30ced2 100644
--- a/templates/taxi/index.html
+++ b/templates/taxi/index.html
@@ -10,4 +10,5 @@ <h2>Dynamic content</h2>
     <li><strong>Drivers:</strong> {{ num_drivers }}</li>
     <li><strong>Manufacturers:</strong> {{ num_manufacturers }}</li>
   </ul>
+  <p>You have visited this page {{ num_visits }} time{{ num_visits|pluralize }}.</p>
 {% endblock %}