feature: Affichage du message sur l'tat du contrat

gestion_conge/views.py

@@ -129,66 +129,69 @@ def demander_conge(request):
 
 @login_required
 def liste_demande_conges(request):
-    """Vue de liste des demandes de congés en attente de validation selon le statut de l'utilisateur actuel"""
+
     try:
-        employe = Employe.objects.get(user__username = request.user)
+        employe = Employe.objects.get(user=request.user)
     except Employe.DoesNotExist:
         return JsonResponse({
             "success": False,
-            "message": "Votre profil Utilisateur n'est lié à aucun profil Employé. Veuillez contacter l'administrateur."
+            "message": "Profil employé introuvable"
         })
 
-    try:
-        affectation = Affectation.objects.get(
-            employe=employe,
-            date_fin_daffectation__gte=timezone.now().date()
-        )
-    except Affectation.DoesNotExist:
-        affectation = None
+    affectation = Affectation.objects.filter(
+        employe=employe,
+        date_fin_daffectation__gte=timezone.now().date()
+    ).first()
+
+    is_direction = employe.user.groups.filter(name='direction').exists()
 
     if employe.chef:
-        print("chef")
-        conges_en_attente = Conge.objects.filter(
-            employe__departement = employe.departement,
-            validation_hierarchique = None
+
+        conges = Conge.objects.filter(
+            Q(employe__departement=employe.departement) |
+            Q(employe=employe)
         ).order_by('-date_demande')
 
     elif affectation and affectation.role == "chef_projet":
+
         employes_du_projet = Affectation.objects.filter(
-            projet = affectation.projet,
-            date_fin_daffectation__gte = timezone.now().date()
-        ).values('employe')
+            projet=affectation.projet,
+            date_fin_daffectation__gte=timezone.now().date()
+        ).values_list('employe', flat=True)
 
-        conges_en_attente = Conge.objects.filter(
-            employe__in = employes_du_projet,
-            validation_hierarchique = None
+        conges = Conge.objects.filter(
+            Q(employe__in=employes_du_projet) |
+            Q(employe=employe)
         ).order_by('-date_demande')
 
-    elif 'direction' in employe.user.groups.values_list('name', flat=True):
-        conges_en_attente = Conge.objects.filter(
-            validation_hierarchique = True,
-            validation_direction = None
-        ).order_by('-date_demande')
+   
+    elif is_direction:
+
+        conges = Conge.objects.filter(
+            Q(validation_hierarchique=True) |
+            Q(employe__user__groups__name='direction')
+        ).distinct().order_by('-date_demande')
 
     else:
-        conges_en_attente = Conge.objects.filter(
-            employe__user__username = request.user
+
+        conges = Conge.objects.filter(
+            employe=employe
         ).order_by('-date_demande')
 
     return JsonResponse({
         "success": True,
-        "data":[
-        {
-            **model_to_dict(conge), 
-            "prenom_nom": f"{conge.employe.user.first_name} {conge.employe.user.last_name}",
-            "date_demande": conge.date_demande,
-            "nombre_jours": conge.nombre_jours,
-            "type": dict(conge.TYPE_CHOICES).get(conge.type),
-            "solde_conge": fonctions_utilitaire.solde_conge(conge.employe)["quota_annuel"]
-        }
-        for conge in conges_en_attente]}, 
-        safe=False
-    )
+        "data": [
+            {
+                **model_to_dict(conge),
+                "prenom_nom": f"{conge.employe.user.first_name} {conge.employe.user.last_name}",
+                "date_demande": conge.date_demande,
+                "nombre_jours": conge.nombre_jours,
+                "type": dict(conge.TYPE_CHOICES).get(conge.type),
+                "solde_conge": fonctions_utilitaire.solde_conge(conge.employe)["quota_annuel"]
+            }
+            for conge in conges
+        ]
+    })
 
 @login_required
 def validation_de_conge(request):

gestion_employe/templates/gestion_employe/monprofil.html

@@ -19,12 +19,13 @@
     {% endfor %}
 {% endif %}
 
-{% if not expiration_contrat %}
-    <div class="alert alert-danger fade show alert-dismissible mt-2">
+{% if not has_contrat %}
+    <div class="alert alert-danger mt-2">
         <strong>Important :</strong> Les informations sur votre contrat n'ont pas été renseignées, veuillez contacter les ressources humaines.
     </div>
-{% elif contrat_nb_jours_restant %}
-    <div class="alert alert-danger fade show alert-dismissible mt-2">
+
+{% elif expiration_contrat %}
+    <div class="alert alert-warning mt-2">
         <strong>Important :</strong> Votre contrat de travail expire dans {{ contrat_nb_jours_restant }} jours, veuillez contacter les ressources humaines.
     </div>
 {% endif %}

gestion_employe/views.py

@@ -1,5 +1,5 @@
 import json
-from datetime import timedelta, datetime
+from datetime import date, timedelta, datetime
 from dateutil.relativedelta import relativedelta
 
 from django.utils import timezone
@@ -122,26 +122,41 @@ def mon_profil(request):
     try:
         employe = Employe.objects.get(user__username=request.user)
     except Employe.DoesNotExist:
-        messages.error(request, "Impossible d'acceder au menu 'Mon profil' car votre profil Utilisateur n'est lié à aucun profil Employé. Veuillez contacter l'Administrateur.")
+        messages.error(
+            request,
+            "Impossible d'acceder au menu 'Mon profil' car votre profil Utilisateur n'est lié à aucun profil Employé. Veuillez contacter l'Administrateur."
+        )
         return redirect("gestion_conges:conge")
-    
+
     contrats = Contrat.objects.filter(employe=employe, statut='actif').first()
+
     projets = Affectation.objects.filter(
-        employe = employe, 
-        date_fin_daffectation__gte = timezone.now().date()
+        employe=employe,
+        date_fin_daffectation__gte=timezone.now().date()
     ).select_related('projet')
 
+    has_contrat = contrats is not None
+    expiration_contrat = False
+    contrat_nb_jours_restant = None
+
+    if contrats:
+        nb_jours = contrats.nombre_jours_restant 
+        contrat_nb_jours_restant = nb_jours
+        expiration_contrat = nb_jours <= fonctions_utilitaire.DUREE_FIN_CONTRAT
+
     return render(
-        request, 
-        'gestion_employe/monprofil.html', 
+        request,
+        'gestion_employe/monprofil.html',
         {
             'employe': employe,
+
             'contrats': [{
                 **model_to_dict(contrats),
                 "type_contrat": dict(Contrat.TYPE_CONTRAT).get(contrats.type_contrat),
                 "statut": dict(Contrat.STATUT_CONTRAT).get(contrats.statut),
                 "fichier_contrat": contrats.fichier_contrat.url if contrats.fichier_contrat else "",
-            } if contrats else []],
+            }] if contrats else [],
+
             'projets': [
                 {
                     **model_to_dict(a.projet),
@@ -151,12 +166,13 @@ def mon_profil(request):
                     "pourcentage_temps_affectation": a.pourcentage_temps_affectation
                 } for a in projets
             ],
+
             "formation_form": FormationForm(),
-            "expiration_contrat": contrats.nombre_jours_restant <= fonctions_utilitaire.DUREE_FIN_CONTRAT if contrats else False,
-            "contrat_nb_jours_restant": contrats.nombre_jours_restant if contrats else None
+            "has_contrat": has_contrat,
+            "expiration_contrat": expiration_contrat,
+            "contrat_nb_jours_restant": contrat_nb_jours_restant
         }
     )
-
 @login_required
 def modifier_mot_passe(request):
     """Vue pour permettre à un utilisateur de modifier son mot de passe et ses informations de profil"""
@@ -240,14 +256,25 @@ def suppression_affectation(request):
     return JsonResponse({"message": "Affectation supprimée avec succès."})
 
 def creation_contrat(request):
-    """Vue pour permettre à un utilisateur de créer un contrat pour un employé"""
+    """Créer un contrat pour un employé (avec contrôle d'existence de contrat actif)"""
+
     try:
         employe = Employe.objects.get(id=request.POST.get('employe_id'))
     except Employe.DoesNotExist:
         messages.error(request, "Employé non trouvé.")
         return redirect('employe-index')
-    
+    contrat_actif = Contrat.objects.filter(
+        employe=employe,
+        date_fin__gte=date.today()
+    ).exists()
+
     if request.method == "POST":
+        if contrat_actif:
+            messages.error(
+                request,
+                "Impossible de créer un contrat : cet employé a déjà un contrat actif."
+            )
+            return redirect('gestion_employe:index')
         form = ContratForm(request.POST, request.FILES)
         if form.is_valid():
             contrat = form.save(commit=False)
@@ -256,9 +283,13 @@ def creation_contrat(request):
             messages.success(request, "Contrat créé avec succès.")
             return redirect('gestion_employe:index')
         messages.error(request, "Formulaire non valide")
+
     else:
         form = ContratForm(initial={'employe': employe})
-    return render(request, 'gestion_employe/index.html', {'contrat_form': form})
+
+    return render(request, 'gestion_employe/index.html', {
+        'contrat_form': form
+    })
 
 @login_required
 def enregistrer_detail_employe(request):

gestion_projet/static/gestion_projet/js/enregistrement_bailleur.js

@@ -1,4 +1,5 @@
 const btnEnregistrerBailleur = document.getElementById('btnEnregistrerBailleur');
+let table;
 
 btnEnregistrerBailleur.addEventListener('click', function() {
     const form = document.getElementById('formBailleur');
@@ -20,4 +21,31 @@ btnEnregistrerBailleur.addEventListener('click', function() {
             alert('Ce bailleur existe déjà dans la base de données.');
         }
     });
-});
+});
+
+document.addEventListener("DOMContentLoaded", function () {
+
+    table = new Tabulator("#table-bailleurs", {
+        ajaxURL: "/gestion-projet/bailleurs/",
+        layout: "fitColumns",
+        pagination: "local",
+        paginationSize: 5,
+
+        columns: [
+            {title: "#", formatter: "rownum", width: 60},
+            {title: "Organisme", field: "nom_organisme"},
+            {title: "Contact", field: "contact"},
+            {title: "Email", field: "email"},
+            {title: "Pays", field: "pays"},
+        ],
+        rowDblClick: function(e, row) {
+            const data = row.getData();
+
+            if (confirm(`Voulez-vous vraiment supprimer ${data.nom_organisme} ?`)) {
+                supprimerBailleur(data.id);
+            }
+        }
+    });
+
+});
+

gestion_projet/templates/gestion_projet/parts/creation_bailleur.html

@@ -1,20 +1,51 @@
 <div class="modal fade" id="modalBailleur" tabindex="-1" aria-labelledby="modalBailleurLabel" aria-hidden="true">
-    <div class="modal-dialog modal-dialog-centered">
+    <div class="modal-dialog ">
         <div class="modal-content">
             <div class="modal-header">
-                <h5 class="modal-title" id="modalBailleurLabel">Ajouter un Bailleur</h5>
+                <h5 class="modal-title" id="modalBailleurLabel">Gestion des Bailleurs</h5>
                 <button type="button" class="btn-close" data-bs-dismiss="modal"></button>
-            </div>
-            <div class="modal-body p-4">
-                <form id="formBailleur" method="POST" action="{% url 'gestion_projet:creation-bailleur' %}">
-                    {% csrf_token %}
-                    {{ form_ajout_bailleur.as_p }}
-                </form>
-            </div>
-            <div class="modal-footer">
-                <button type="submit" id="btnEnregistrerBailleur" class="btn btn-success"><i class="bi bi-save me-1"></i> Enregistrer</button>
-                <button type="button" class="btn btn-secondary" data-bs-dismiss="modal">Annuler</button>
+            </div>       
+            <ul class="nav nav-tabs px-3 pt-2" id="bailleurTab" role="tablist">
+                <li class="nav-item" role="presentation">
+                    <button class="nav-link active" id="ajouter-tab" data-bs-toggle="tab"
+                            data-bs-target="#ajouter" type="button" role="tab">
+                        Ajouter
+                    </button>
+                </li>
+                <li class="nav-item" role="presentation">
+                    <button class="nav-link" id="liste-tab" data-bs-toggle="tab"
+                            data-bs-target="#liste" type="button" role="tab">
+                        Liste
+                    </button>
+                </li>
+            </ul>
+            <div class="modal-body">
+                <div class="tab-content pt-3">
+                    <div class="tab-pane fade show active" id="ajouter" role="tabpanel">
+                        <form id="formBailleur" method="POST" action="{% url 'gestion_projet:creation-bailleur' %}">
+                            {% csrf_token %}
+                            {{ form_ajout_bailleur.as_p }}
+                            <button type="submit" class="btn btn-success mt-3" id="btnEnregistrerBailleur">
+                                <i class="bi bi-save me-1"></i> Enregistrer
+                            </button>
+                        </form>
+                    </div>
+                    <div class="tab-pane fade" id="liste" role="tabpanel">
+
+                        <div class="d-flex justify-content-between align-items-center mb-3">
+                            <h6 class="mb-0">Liste des bailleurs</h6>
+                        </div>
+                        <div class="row">
+                            <div class="col">
+                                <div id="table-bailleurs"
+                                    data-url="{% url 'gestion_projet:liste-bailleurs' %}">
+                                </div>                          
+                            </div>
+                        </div>
+                    </div>
+                </div>
             </div>
         </div>
     </div>
 </div>
+

gestion_projet/urls.py

@@ -19,6 +19,12 @@ urlpatterns = [
         views.creation_projet, 
         name='creation-projet'
     ),
+    path(
+        'bailleurs/',
+          views.liste_bailleur, 
+         name='liste-bailleurs'
+    ),
+
     path(
         'projet/modifier/<int:projet_id>/', 
         views.modification_projet, 
@@ -84,6 +90,7 @@ urlpatterns = [
         views.liste_activites_projet, 
         name='liste-activites-projet'
     ),
+    
     # path(
     #     'projet/ajout-de-document/', 
     #     views.ajouter_document_projet, 
@@ -119,4 +126,6 @@ urlpatterns = [
         views.mises_a_jour_projet,
         name='mises-a-jour-projet'
     )
-]
+    
+]
+

gestion_projet/views.py

@@ -2,6 +2,7 @@ from datetime import date
 from decimal import Decimal, InvalidOperation
 from django.http import JsonResponse
 from django.shortcuts import redirect, render
+from django.urls import reverse
 from django.utils import timezone
 from django.contrib import messages
 from django.contrib.auth.decorators import login_required
@@ -143,6 +144,22 @@ def creation_bailleur(request):
             return JsonResponse({'success': True})
     return JsonResponse({'success': False})
 
+@login_required
+def liste_bailleur(request):
+    """ Vue pour retourner la liste de tous les bailleurs """
+    bailleurs = Bailleur.objects.all().order_by('-id')
+    data = []
+    for b in bailleurs:
+        data.append({
+        "id": b.id,
+        "nom_organisme": b.nom_organisme,
+        "contact": b.contact,
+        "email": b.email,
+        "pays": b.pays,
+        })
+    return JsonResponse(data, safe=False)
+
+
 @login_required
 def ajouter_financement_projet(request):
     """Ajoute un financement à un projet en vérifiant que le total ne dépasse pas 100%"""
@@ -318,6 +335,7 @@ def activites_projet(request):
         }
     return render(request, 'gestion_projet/suivi_activite.html', context)
 
+
 @login_required
 def ajouter_activite_projet(request):
     """Vue pour ajouter une activité à un projet spécifique via un formulaire"""

gestion_salle/static/gestion_salle/js/index.js

@@ -167,9 +167,9 @@ tableau_reservation_attente.on("rowClick", (row, rowData) => {
         $("lien_zoom_container").className = 'd-none';
     }
 
-    if(data.statut !== "refusee"){
-        $("motif_refus_container").className = 'd-none';
-    }
+    // if(data.statut !== "refusee"){
+    //     $("motif_refus_container").className = 'd-none';
+    // }
 
     $("id_reservation_detail").value = data.id;
     $("id_reservation_refus").value = data.id;
@@ -178,14 +178,15 @@ tableau_reservation_attente.on("rowClick", (row, rowData) => {
     $("employe").value=data.employe;
     $("salle").value=data.salle;
     $("statut-reservation").innerHTML=data.statut;
-    $("date_evenement").value=data.date_debut;
+    $("date_debut").value = data.date_debut;
+    $("date_fin").value = data.date_fin;
     $("heure_debut").value=data.heure_debut;
     $("heure_fin").value=data.heure_fin;
     $("motif_reservation").value=data.motif_reservation;
     $("besoin_zoom").checked=data.besoin_zoom;
     $("besoin_ordinateur").checked=data.besoin_ordi;
     $("lien_zoom").value=data.lien_zoom;
-    $("motif_refus").value=data.motif_refus;
+    // $("motif_refus").value=data.motif_refus;
 
     const modal = new bootstrap.Modal($("modalDetailReservation"));
     bootstrap.Modal.getOrCreateInstance($("modalReservationAttente")).hide();

gestion_salle/templates/gestion_salle/parts/modalDetailResevation.html

@@ -21,8 +21,12 @@
                         <input class="form-control" id="salle"  readonly>
                     </div>
                     <div class="form-group mb-2">
-                        <label>Date de l'évènement :</label>
-                        <input type='date' class="form-control" id="date_evenement" readonly >
+                        <label>Date de debut :</label>
+                        <input type='date' class="form-control" id="date_debut" readonly >
+                    </div>
+                    <div class="form-group mb-2">
+                        <label>Date de fin :</label>
+                        <input type='date' class="form-control" id="date_fin" readonly >
                     </div>
                     <div class="form-group mb-2">
                         <label>Heure de début :</label>
@@ -48,24 +52,19 @@
                         <label label="form-check-label">Besoin d'un ordinateur</label>
                         <input type="checkbox" class="form-check-input" id="besoin_ordinateur" readonly >
                     </div>
-                    <div class="form-group mb-2" id='motif_refus_container'>
-                        <label>Motif de refus de la reservation :</label>
-                        <textarea class="form-control"  id="motif_refus" readonly></textarea>
-                    </div>
                     <div class='d-flex justify-content-around mt-2'>
                         {% if appartient_au_departement_informatique %}
                         <button class="btn btn-primary" id="ajoutZoom">Ajout du lien zoom</button>
                         {% endif %}
                         {% if appartient_direction  and  reservation.statut == "en_attente" %}
                         <button class="btn btn-danger" id="refuserReservation" data-lienrefus="{% url 'gestion_salle:refuser-reservation' %}">Refuser</button>
-                        {% endif %}
-                            <button class="btn btn-danger" id="bouton-annuler">Annuler</button>
-                        {% if appartient_direction %}
                         <button class="btn btn-success" id="bouton-valider">Valider</button>
                         {% endif %}
+                        <span id="current-user-id" data-user-id="{{ request.user.id }}"></span>
+                            <button class="btn btn-danger" id="bouton-annuler">Annuler</button>
                     </div>
                 </form>
             </div>
         </div>
     </div>
-</div>
+</div> 

gestion_salle/views.py

@@ -8,63 +8,93 @@ from django.forms import model_to_dict
 from gestion_employe.models import Employe
 from gestion_salle.forms import ReservationForm
 from .models import Reservation
+from datetime import timedelta
+from django.contrib.auth.models import User
+
 
 @login_required
-def index(request:HttpRequest):
-    """Vue de gestion de la reservation de la salle"""
+def index(request: HttpRequest):
+
     try:
         employe = Employe.objects.get(user=request.user)
     except Employe.DoesNotExist:
-        messages.error(request, "Impossible d'accéder au menu 'Reservation de salle' car votre profil Utilisateur n'est lié à aucun profil Employe. Veuillez contacter l'administrateur.")
+        messages.error(request, "Profil employé introuvable.")
         return redirect('gestion_conges:conge')
-    
+
+
+
     if request.method == "POST":
         form = ReservationForm(request.POST)
-        if form.is_valid():
-            date_debut = form.cleaned_data.get('date_debut')
-            date_fin = form.cleaned_data.get('date_fin')
-            salle = form.cleaned_data.get('salle')
-            heure_debut = form.cleaned_data.get('heure_debut')
-            heure_fin = form.cleaned_data.get('heure_fin')
-            motif_reservation = form.cleaned_data.get('motif_reservation')
-            besoin_zoom = form.cleaned_data.get('besoin_zoom')
-            besoin_ordi = form.cleaned_data.get('besoin_ordi')
-            
-            while date_debut <= date_fin :
-                reservation = Reservation(
-                    employe = employe,
-                    date_debut = date_debut,
-                    date_fin = date_debut,
-                    salle = salle,
-                    heure_debut = heure_debut,
-                    heure_fin = heure_fin,
-                    besoin_zoom = besoin_zoom,
-                    besoin_ordi = besoin_ordi,
-                    motif_reservation=motif_reservation,
-                )
-                reservation.save()
-                date_debut = date_debut + timedelta(days=1)
 
-            messages.success(request, "Réservation(s) créées avec succès.")
+        if form.is_valid():
+
+            date_debut = form.cleaned_data['date_debut']
+            date_fin = form.cleaned_data['date_fin']
+            salle = form.cleaned_data['salle']
+            heure_debut = form.cleaned_data['heure_debut']
+            heure_fin = form.cleaned_data['heure_fin']
+            motif = form.cleaned_data['motif_reservation']
+            besoin_zoom = form.cleaned_data['besoin_zoom']
+            besoin_ordi = form.cleaned_data['besoin_ordi']
+
+            if date_fin < date_debut:
+                messages.error(request, "Date fin invalide.")
+                return redirect('gestion_salle:reservation-salle')
+
+            if heure_fin <= heure_debut:
+                messages.error(request, "Heure invalide.")
+                return redirect('gestion_salle:reservation-salle')
+            
+            if not request.user.first_name.strip() or not request.user.last_name.strip():
+                messages.error(
+                    request,
+                    "Veuillez renseigner votre nom et prénom pour pouvoir faire une réservation."
+                )
+                return redirect('gestion_salle:reservation-salle')
+
+            created = []
+            current_date = date_debut
+
+            while current_date <= date_fin:
+
+                reservation = Reservation.objects.create(
+                    employe=employe,
+                    date_debut=current_date,
+                    date_fin=current_date,
+                    salle=salle,
+                    heure_debut=heure_debut,
+                    heure_fin=heure_fin,
+                    besoin_zoom=besoin_zoom,
+                    besoin_ordi=besoin_ordi,
+                    motif_reservation=motif,
+                    statut="en_attente"
+                )
+
+                created.append(reservation)
+                current_date += timedelta(days=1)
+
+            messages.success(request, "Réservation(s) créée(s) avec succès.")
             return redirect('gestion_salle:reservation-salle')
 
     formulaire_reservation = ReservationForm()
-    departement = Employe.objects.get(user__username=request.user).departement
-    appartient_direction = 'direction' in request.user.groups.values_list('name', flat=True)
-    liste_demande_reservation = [
-        reservation.id for reservation in
-        Reservation.objects.filter(employe=employe, statut='en_attente')
-    ]
-    
+    departement = employe.departement
+
+    appartient_direction = request.user.groups.filter(name='direction').exists()
+
+    liste_demande_reservation = Reservation.objects.filter(
+        employe=employe,
+        statut='en_attente'
+    ).values_list('id', flat=True)
+
     context = {
         'formulaire_reservation': formulaire_reservation,
         'nb_reservation_attente': Reservation.objects.filter(statut='en_attente').count(),
-        'appartient_au_departement_informatique': 'Informatique' == departement.nom  if departement else False,
+        'appartient_au_departement_informatique': departement and departement.nom == "Systeme informatique",
         'appartient_direction': appartient_direction,
-        'liste_demande_reservation': liste_demande_reservation
+        'liste_demande_reservation': list(liste_demande_reservation),
     }
-    return render(request, "gestion_salle/index.html", context)
 
+    return render(request, "gestion_salle/index.html", context)
 def liste_reservation(request:HttpRequest):
     """Vue d'affichage des creneaux disponibles"""
     reservations = Reservation.objects.filter(statut = "validee")
@@ -86,7 +116,6 @@ def liste_reservation(request:HttpRequest):
             "end": reservation.heure_fin, 
             "color": color,
         })
-
     return JsonResponse(liste_reservation, safe=False)
 
 @login_required
@@ -103,23 +132,25 @@ def liste_reservation_attente(request):
     return JsonResponse(liste_reservation, safe=False)
 
 def detail_reservation(request:HttpRequest, reservation_id:int):
+    
     reservation = Reservation.objects.get(id=reservation_id)
     employe = reservation.employe.user
+    
     reservation_json = {
+        
         'id_reservation': reservation_id,
         'employe': f"{employe.first_name} {employe.last_name}",
         'salle': reservation.salle,
         'statut': reservation.statut,
-        'date_evenement': reservation.date_debut.strftime('%Y-%m-%d'),
+        'date_debut': reservation.date_debut.strftime('%Y-%m-%d'),
+        'date_fin': reservation.date_fin.strftime('%Y-%m-%d'),
         'heure_debut': reservation.heure_debut.strftime('%H:%M'),
         'heure_fin': reservation.heure_fin.strftime('%H:%M'),
         'motif_reservation': reservation.motif_reservation,
         'besoin_zoom': reservation.besoin_zoom,
         'besoin_ordinateur': reservation.besoin_ordi,
         'lien_zoom': reservation.lien_zoom or '',
-        'motif_refus': reservation.motif_refus or '',
     }
-
     return JsonResponse(reservation_json, safe=True)
 
 @login_required
@@ -165,31 +196,35 @@ def annuler_reservation(request:HttpRequest):
     return redirect('gestion_salle:reservation-salle')
 
 @login_required
-def valider_reservation(request:HttpRequest):
-    """Vue de gestion de l'annulation de la reservation"""
+def valider_reservation(request: HttpRequest):
+    """Validation d'une réservation"""
     if request.method == 'POST':
-        reservation_id= request.POST['id_reservation']
+        reservation_id = request.POST.get('id_reservation')
+
         try:
-            reservation =  Reservation.objects.get(id=reservation_id)
-        except reservation.DoesNotExist:
-            messages.error(request, "La resevertion selectionné n'existe pas.")
+            reservation = Reservation.objects.get(id=reservation_id)
+        except Reservation.DoesNotExist:
+            messages.error(request, "La réservation sélectionnée n'existe pas.")
             return redirect("salle")
-        
+
         reservation.statut = 'validee'
         reservation.save()
-    messages.success(request, f"Réservation de {reservation.employe.get_full_name()} validée avec succès.")
+
     return redirect('gestion_salle:reservation-salle')
 
 @login_required
-def refuser_reservation(request:HttpRequest):
-    """Vue de gestion de refus de la reservation"""
-    data = json.loads(request.body)
-    reservation_id = data.get("id_reservation")
-    try:
-        reservation =  Reservation.objects.get(id=reservation_id)
-    except Reservation.DoesNotExist and ValueError:
-        return JsonResponse({"message": "La resevertion selectionné n'existe pas."})
-    else:
-        reservation.statut = "refusee"
+def refuser_reservation(request: HttpRequest):
+    """Refuser une réservation"""
+    if request.method == 'POST':
+        reservation_id = request.POST.get('id_reservation')
+
+        try:
+            reservation = Reservation.objects.get(id=reservation_id)
+        except Reservation.DoesNotExist:
+            messages.error(request, "La réservation n'existe pas.")
+            return redirect("salle")
+
+        reservation.statut = 'refusee'
         reservation.save()
-        return JsonResponse({"message": "Réservation refusée avec succès."})
+
+    return redirect('gestion_salle:reservation-salle')