Relecture du code v1

gestion_vehicule/models.py

@@ -3,7 +3,6 @@
 from django.db import models
 from authentification.models import Profile
 
-
 class Vehicule(models.Model):
     """Modèle représentant un véhicule."""
 
@@ -22,7 +21,6 @@ class Vehicule(models.Model):
         """Retourne une représentation en chaîne de caractères du véhicule."""
         return f"{self.immatriculation}"
 
-
 class DemandeVehicule(models.Model):
     """Modèle représentant une demande de véhicule."""
 
@@ -56,7 +54,6 @@ class DemandeVehicule(models.Model):
                f" le {self.date_heure}"
                )
 
-
 class CarnetBord(models.Model):
     """Modèle représentant le carnet de bord d'un véhicule."""
 

gestion_vehicule/templates/gestion_vehicule/detail_course.html

@@ -121,7 +121,6 @@
             </div>
         </div>
     </div>
-         <!-- --------------------------------- -->
 </div>
 
 {% endblock %}

gestion_vehicule/templates/gestion_vehicule/detail_vehicule.html

@@ -11,6 +11,14 @@
     </div>
     <div class="col">
         <h3 class="text-center">Carnet de bord du vehicule</h3>
+        {% if messages %}
+            {% for message in messages %}
+                <div class="alert alert-success alert-dismissible fade show" role="alert">
+                    {{ message }}
+                    <button type="button" class="btn-close" data-bs-dismiss="alert" aria-label="Close"></button>
+                </div>
+            {% endfor %}
+        {% endif %}
         <form action="{% url 'gestion_vehicule:enregistrer-carnetBord' vehicule.id %}" id="carnetBordForm" method="post">
             <div class="form-group">
                 <label for="">Immatriculation :</label>

gestion_vehicule/views.py

@@ -1,5 +1,6 @@
 """Views pour la gestion des véhicules."""
 
+from pyexpat.errors import messages
 from django.shortcuts import render, redirect
 from django.http import JsonResponse
 
@@ -12,7 +13,7 @@ from . import models
 
 @login_required(login_url="login")
 def index(request):
-    """Page d'accueil de la gestion des véhicules.
+    """Retournez la page d'accueil de la gestion des véhicules.
 
     Parameters
     ----------
@@ -43,7 +44,7 @@ def index(request):
 
 @login_required(login_url="login")
 def enregistrer_voiture(request):
-    """Enregistrer un nouveau véhicule dans le système.
+    """Enregistrez un nouveau véhicule.
 
     Parameters
     ----------
@@ -68,10 +69,9 @@ def enregistrer_voiture(request):
         return redirect("gestion_vehicule:index")
     return None
 
-
 @login_required(login_url="login")
 def enregistrer_demande(request):
-    """Enregistrer une nouvelle demande de véhicule.
+    """Enregistrez une nouvelle demande de véhicule.
 
     Parameters
     ----------
@@ -112,7 +112,7 @@ def enregistrer_demande(request):
 
 @login_required(login_url="login")
 def liste_demande(request):
-    """Afficher la liste des demandes de véhicules.
+    """Affichez la liste des demandes de véhicules.
 
     La liste des demandes affichées dépend du groupe de l'utilisateur connecté:
     - Si l'utilisateur appartient au groupe "Chauffeur", seules les demandes
@@ -184,7 +184,7 @@ def liste_demande(request):
 
     return JsonResponse(demandes_en_cours, safe=False)
 
-
+@login_required(login_url="login")
 def detail_course(request, id_demande):
     """Afficher les détails d'une demande de véhicule spécifique.
 
@@ -207,8 +207,6 @@ def detail_course(request, id_demande):
 
     details_course = models.CarnetBord.objects.get(course=la_demande)
 
-    print("Détails course:", la_demande.id)
-
     return render(
         request,
         "gestion_vehicule/detail_course.html",
@@ -300,7 +298,7 @@ def statut_demande(request):
 
 @login_required(login_url="login")
 def detail_voiture(request, vehicule_id):
-    """Afficher les détails d'un véhicule spécifique.
+    """Affichez les détails d'un véhicule spécifique.
 
     Parameters
     ----------
@@ -330,7 +328,7 @@ def detail_voiture(request, vehicule_id):
 
 @login_required(login_url="login")
 def enregistrer_prise_carburant(request, course_id):
-    """Enregistrer une prise de carburant pour une course spécifique.
+    """Enregistrez une prise de carburant pour une course spécifique.
 
     Parameters
     ----------
@@ -359,7 +357,7 @@ def enregistrer_prise_carburant(request, course_id):
 
 @login_required(login_url="login")
 def carnet_carburant(request, vehicule_id):
-    """Afficher le carnet de carburant d'un véhicule spécifique.
+    """Affichez le carnet de carburant d'un véhicule spécifique.
 
     Parameters
     ----------
@@ -385,7 +383,7 @@ def carnet_carburant(request, vehicule_id):
     )
 
 def carnet_sortie_vehicule(request, vehicule_id):
-    """Afficher le carnet de sortie d'un véhicule spécifique.
+    """Affichez le carnet de sortie d'un véhicule spécifique.
 
     Parameters
     ----------
@@ -408,10 +406,9 @@ def carnet_sortie_vehicule(request, vehicule_id):
         {"id_vehicule": vehicule_id},
     )
 
-
 @login_required(login_url="login")
 def filtre_carnet_carburant(request):
-    """Filtrer les prises de carburant d'un véhicule pour un mois spécifique.
+    """Filtrez les prises de carburant d'un véhicule pour un mois spécifique.
 
     Parameters
     ----------
@@ -459,7 +456,7 @@ def filtre_carnet_carburant(request):
     return JsonResponse(mon_carnet, safe=False)
 
 def filtre_sortie_vehicule(request):
-    """Filtrer les sorties d'un véhicule pour un mois spécifique.
+    """Filtrez les sorties d'un véhicule pour un mois spécifique.
 
     Parameters
     ----------
@@ -509,7 +506,7 @@ def filtre_sortie_vehicule(request):
 
 @login_required(login_url="login")
 def enregistrer_carnet_bord(request, vehicule_id):
-    """Enregistrer ou mettre à jour le carnet de bord d'un véhicule.
+    """Enregistrez ou mettez à jour le carnet de bord d'un véhicule.
 
     Parameters
     ----------
@@ -534,7 +531,7 @@ def enregistrer_carnet_bord(request, vehicule_id):
     """
     if request.method == "POST":
         annee, mois = request.POST["mois_concerne"].split("-")
-        carnet_bord, created = models.CarnetBord.objects.update_or_create(
+        _, created = models.CarnetBord.objects.update_or_create(
             mois_concerne="-".join([annee, mois, "01"]),
             defaults={
                 "km_debut_mois": request.POST["km_debut_mois"],
@@ -544,15 +541,15 @@ def enregistrer_carnet_bord(request, vehicule_id):
         )
 
     if created:
-        print("Création effectuée")
+        messages.success(request, "Création effectuée avec succès.")
     else:
-        print("Mise à jour effectué")
+        messages.success(request, "Mise à jour effectuée avec succès.")
 
     return redirect(detail_voiture, vehicule_id)
 
 @login_required(login_url="login")
-def carnet_vehicule_information(request, mois_concerne): # noqa: ARG001
-    """Récupérer les informations du carnet de bord pour un mois spécifique.
+def carnet_vehicule_information(_request, mois_concerne):
+    """Récupérez les informations du carnet de bord pour un mois spécifique.
 
     Parameters
     ----------
@@ -584,7 +581,7 @@ def carnet_vehicule_information(request, mois_concerne): # noqa: ARG001
     return JsonResponse(carnet_dict)
 
 def enregistrer_information_course_depart(request, id_demande):
-    """Enregistrer les informations de départ d'une course spécifique.
+    """Enregistrez les informations de départ d'une course spécifique.
 
     Parameters
     ----------
@@ -613,7 +610,7 @@ def enregistrer_information_course_depart(request, id_demande):
     return redirect("gestion_vehicule:detail-course", id_demande)
 
 def enregistrer_information_course_arrive(request, id_demande):
-    """Enregistrer les informations d'arrivée d'une course spécifique.
+    """Enregistrez les informations d'arrivée d'une course spécifique.
 
     Parameters
     ----------
@@ -632,17 +629,13 @@ def enregistrer_information_course_arrive(request, id_demande):
     demande = models.DemandeVehicule.objects.get(id=id_demande)
     vehicule = models.Vehicule.objects.get(id=demande.vehicule.id)
 
-    course_termier = False
-    if "course_terminer" in request.POST:
-        course_termier = True
-
     course = models.CarnetBord.objects.get(course=demande)
 
     course.vehicule = vehicule
     course.course = demande
     course.heure_arrive = request.POST["heure-effectif-arrivee"]
     course.km_arrive = request.POST["kilometrage-arrivee"]
-    course.course_termine = course_termier
+    course.course_termine = "course-terminer" in request.POST
     course.save()
 
     return redirect("gestion_vehicule:detail-course", id_demande)