Ajout de la suppression de la reservation de salle

fonction_utilitaire/fonctions_utilitaire.py

@@ -15,6 +15,21 @@ try:
 except:
     pass
 
+try:
+    EMAIL_EQUIPE_INFORMATIQUE = list(
+        Employe.objects.filter(departement__nom="Systeme informatique").values_list('user__email', flat=True)
+    ) or ["support@cerfig.org"]
+except:
+    EMAIL_EQUIPE_INFORMATIQUE = ["support@cerfig.org"]
+
+def est_equipe_informatique(employe):
+    """Vérifie si un employé appartient au département informatique"""
+    return employe.departement and employe.departement.nom == "Systeme informatique"
+
+def est_assistante_direction(employe):
+    """Vérifie si un employé est l'assistante de direction"""
+    return employe.fonction == "assistant_direction"
+
 def solde_conge(employe):
     """Fonction de calcul du solde de congé restant l'employé"""
     contrat = Contrat.objects.filter(employe=employe, statut='actif').order_by('-date_debut').first()

gestion_employe/views.py

@@ -3,8 +3,8 @@ from datetime import datetime, timedelta
 
 import requests
 from dateutil.relativedelta import relativedelta
+from decouple import config
 from django.contrib import messages
-from django.contrib.auth import logout
 from django.contrib.auth.decorators import login_required
 from django.contrib.auth.models import User
 from django.contrib.auth.password_validation import validate_password
@@ -27,6 +27,18 @@ def index(request):
     if not request.user.groups.filter(name="ressource_humaine").exists():
         return redirect('gestion_employe:mon-profil')
 
+    response = requests.get(f"{config('SERVEUR_AUTHENTIFICATION')}recuperer-list-departement/")
+
+    if response.status_code == 200:
+        try:
+            departements_data = response.json()
+            for dept in departements_data:
+                Departement.objects.update_or_create(
+                    nom = dept['nom']  
+                )
+        except json.JSONDecodeError as e:
+            print(f"[ERROR] Erreur lors du décodage JSON : {e}")
+
     employes = Employe.objects.all().order_by('-user__date_joined')
     nombre_employes = Employe.objects.count()
     nombre_cps = Contrat.objects.filter(type_contrat='contrat_prestation').count()
@@ -521,7 +533,7 @@ def modifier_mot_passe(request):
         }
 
         response = requests.post(
-            "http://localhost:8000/modifier-mot-de-passe/",
+            f"{config('SERVEUR_AUTHENTIFICATION')}modifier-mot-de-passe/",
             json=donnees,
             headers=headers
         )

gestion_salle/admin.py

@@ -1,3 +1 @@
 from django.contrib import admin
-
-# Register your models here.

gestion_salle/static/gestion_salle/js/index.js

@@ -2,7 +2,71 @@ const $ = (element) => document.getElementById(element);
 const { Schedule } = calendarjs;
 
 let dateAUtiliser = new Date().toISOString().split('T')[0];
-let currentReservationId = null;
+
+function afficherDetailsReservation(data) {
+    const reservationId = data.id_reservation || data.id;
+    $("id_reservation_detail").value = reservationId;
+    $("id_reservation_refus").value = reservationId;
+    $("id_reservation_zoom").value = reservationId;
+    $("employe").value = data.employe;
+    $("salle").value = data.salle;
+    $("statut-reservation").textContent = data.statut;
+    $("date_evenement").value = data.date_evenement || data.date_debut;
+    $("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_ordinateur ?? data.besoin_ordi;
+    $("lien_zoom").value = data.lien_zoom || "";
+    $("lien_zoom_container").classList.toggle("d-none", !data.besoin_zoom);
+
+    const currentUserId = $("current-user-id").dataset.userId;
+    const isOwner = data.est_proprietaire === true
+        || String(data.employe_id) === currentUserId;
+    const isPending = data.est_en_attente === true
+        || data.statut === "En attente"
+        || data.statut === "en_attente";
+
+    $("bouton-supprimer-reservation").classList.toggle(
+        "d-none",
+        !(isOwner && isPending)
+    );
+
+    for (const id of ["refuserReservation", "refuserReservationInfo", "bouton-valider", "bouton-valider-info"]) {
+        const button = $(id);
+        if (button) {
+            button.style.display = isPending && !isOwner ? "inline-block" : "none";
+        }
+    }
+}
+
+function supprimerReservation(reservationId) {
+    if (!window.confirm("Voulez-vous vraiment supprimer cette demande de réservation en attente ?")) {
+        return;
+    }
+
+    const form = $("formulaire-details");
+    const csrf = form.querySelector("[name=csrfmiddlewaretoken]").value;
+    const url = $("bouton-supprimer-reservation").dataset.url.replace(
+        "/0/",
+        `/${reservationId}/`
+    );
+
+    fetch(url, {
+        method: "POST",
+        headers: {"X-CSRFToken": csrf},
+    })
+        .then(response => response.json().then(data => ({response, data})))
+        .then(({response, data}) => {
+            window.alert(data.message);
+            if (response.ok && data.success) {
+                window.location.reload();
+            }
+        })
+        .catch(() => {
+            window.alert("La suppression a échoué. Veuillez réessayer.");
+        });
+}
 
 const calendrier = Schedule(document.getElementById('planning-reservation'), {
     type: 'weekdays',
@@ -19,31 +83,7 @@ const calendrier = Schedule(document.getElementById('planning-reservation'), {
             return response.json();
         })
         .then(data => {
-            currentReservationId = data.id_reservation;
-            $("id_reservation_detail").value = data.id_reservation;
-            $("id_reservation_refus").value = data.id_reservation;
-            $("id_reservation_zoom").value = data.id_reservation;
-            $("employe").value=data.employe;
-            $("salle").value=data.salle;
-            $("statut-reservation").innerHTML=data.statut;
-            $("date_evenement").value=data.date_evenement;
-            $("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_ordinateur;
-            $("lien_zoom").value=data.lien_zoom;    
-            
-            const zoomContainer = $("lien_zoom_container");
-                if (!data.besoin_zoom) {
-                    zoomContainer.classList.add("d-none");
-                } else {
-                    zoomContainer.classList.remove("d-none");
-                }
-            if (data.besoin_zoom === false){
-                $("lien_zoom_container").className = "d-none";
-            }
-
+            afficherDetailsReservation(data);
         })
         .catch(error => console.error("Impossible de charger les détails de la réservation :", error));
     }
@@ -58,6 +98,10 @@ $('semaineDate').addEventListener('change', () => {
     calendrier.render();
 })
 
+$("bouton-supprimer-reservation").addEventListener("click", () => {
+    supprimerReservation($("id_reservation_detail").value);
+});
+
 evenement_defini = null
 
 $("liste-salle").addEventListener("change", (e) => {
@@ -76,42 +120,22 @@ $("liste-salle").addEventListener("change", (e) => {
 function chargement_evenement(){
     const url = $("planning-reservation").dataset.url;
     fetch (url)
-    .then(response => response.json())
+    .then(response => {
+        if (!response.ok) {
+            throw new Error(`Erreur HTTP ${response.status}`);
+        }
+        return response.json();
+    })
     .then(data => {
         calendrier.setData(data);
     })
+    .catch(error => console.error("Impossible de charger les réservations :", error));
 }
 
 document.addEventListener("DOMContentLoaded", function () {
     chargement_evenement()
 })
 
-$("bouton-annuler").addEventListener("click", (e) => {
-    e.preventDefault();
-    
-    const csrf = document.querySelector("[name=csrfmiddlewaretoken]").value;
-    const url_annuler = $("formulaire-details").dataset.urlannuler;
-
-    console.log("URL d'annulation :");
-    console.log($("formulaire-details"))
-
-    e.preventDefault();
-
-    fetch(
-        url_annuler,
-        {
-            method: "POST",
-            headers: {
-                "X-Requested-With": "XMTHttpRequest",
-                "X-CSRFToken": csrf
-            },
-            body: new FormData($("formulaire-details"))
-        }
-    )
-    .then(response => response.json())
-    .then(data => console.log(data))
-})
-
 if($("bouton-valider")){
     $("bouton-valider").addEventListener("click", (e) => {
         const csrf = document.querySelector("[name=csrfmiddlewaretoken]").value;
@@ -183,42 +207,7 @@ const tableau_reservation_attente = new Tabulator("#tableau-reservation-attente"
 tableau_reservation_attente.on("rowClick", (row, rowData) => {
     const data = rowData.getData();
 
-    console.log(data);
-
-    if(data.besoin_zoom === false){
-        $("lien_zoom_container").className = 'd-none';
-    }
-
-    const id_user = $("current-user-id").dataset.userid;
-
-    $("id_reservation_detail").value = data.id;
-    $("id_reservation_refus").value = data.id;
-    $("id_reservation_zoom").value = data.id;
-
-    $("employe").value=data.employe;
-    $("salle").value=data.salle;
-    $("statut-reservation").innerHTML=data.statut;
-    $("date_evenement").value = data.date_evenement;
-    $("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;
-
-    console.log("Verification de data :", data);
-
-    const refuserReservation = document.querySelector("#refuserReservation");
-    const bouton_valider = document.querySelector("#bouton-valider");
-    
-    if((refuserReservation | bouton_valider) && data.statut === "En attente"){
-        refuserReservation.style.display = "inline-block";
-        bouton_valider.style.display = "inline-block";
-    }
-
-    if (id_user != data.employe_id){
-        $("bouton-annuler").className = "d-none";
-    }
+    afficherDetailsReservation(data);
 
     const modal = new bootstrap.Modal($("modalDetailReservation"));
     bootstrap.Modal.getOrCreateInstance($("modalReservationAttente")).hide();

gestion_salle/templates/gestion_salle/index.html

@@ -55,5 +55,5 @@
 {% block 'js' %}
 <script src="https://cdn.jsdelivr.net/npm/lemonadejs/dist/lemonade.min.js"></script>
 <script src="https://cdn.jsdelivr.net/npm/@calendarjs/ce/dist/index.min.js"></script>
-<script src="{% static 'gestion_salle/js/index.js' %}?v=3"></script>
+<script src="{% static 'gestion_salle/js/index.js' %}?v=5"></script>
 {% endblock %}

gestion_salle/templates/gestion_salle/parts/modalDetailResevation.html

@@ -10,7 +10,7 @@
                 <button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="Close"></button>
             </div>
             <div class="modal-body">
-                <form method="POST" data-urlannuler="{% url 'gestion_salle:annuler-reservation' %}" data-urlvalider="{% url 'gestion_salle:valider-reservation' %}" id="formulaire-details">
+                <form method="POST" data-urlvalider="{% url 'gestion_salle:valider-reservation' %}" id="formulaire-details">
                      {% csrf_token %} 
                     <input type='' class="form-control" id="id_reservation_detail" name='id_reservation' hidden>
                     <div class="form-group mb-2">
@@ -50,16 +50,31 @@
                         <input type="checkbox" class="form-check-input" id="besoin_ordinateur" readonly >
                     </div>
                     <hr />
-                    <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>
+                    <div class='d-flex justify-content-center mt-2'>
+                        {% if est_equipe_informatique %}
+                            <button class="btn btn-primary me-1" id="ajoutZoom">Ajout du lien zoom</button>
                         {% endif %}
-                        {% if appartient_direction %}
-                            <button class="btn btn-danger" id="refuserReservation" data-lienrefus="{% url 'gestion_salle:refuser-reservation' %}">Refuser</button>
-                            <button class="btn btn-success" id="bouton-valider">Valider</button>
+                        {% if est_assistante_direction %}
+                            <button class="btn btn-danger me-1" id="refuserReservation" data-lienrefus="{% url 'gestion_salle:refuser-reservation' %}">Refuser</button>
+                            <button class="btn btn-success me-1" 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>
+                        {% if est_equipe_informatique %}
+                            <button class="btn btn-danger me-1" id="refuserReservationInfo" data-lienrefus="{% url 'gestion_salle:refuser-reservation' %}">Refuser</button>
+                            <button class="btn btn-success me-1" id="bouton-valider-info">Valider</button>
+                        {% endif %}
+                        <span id="current-user-id" data-user-id="{{ user.id }}"></span>
+                        <button
+                            type="button"
+                            class="btn btn-outline-danger d-none"
+                            id="bouton-supprimer-reservation"
+                            data-url="{% url 'gestion_salle:supprimer-reservation' 0 %}"
+                        >
+                            Supprimer ma demande
+                        </button>
+
+                         <button type="button" class="btn btn-secondary" id="bouton-annuler" data-bs-dismiss="modal">
+                            Fermer
+                        </button>
                     </div>
                 </form>
             </div>

gestion_salle/urls.py

@@ -45,5 +45,10 @@ urlpatterns = [
         'liste-reservation-attente',
         views.liste_reservation_attente,
         name="liste-reservation-attente"
+    ),
+    path(
+        'reservation/<int:reservation_id>/supprimer',
+        views.supprimer_reservation,
+        name="supprimer-reservation",
     )
 ]

gestion_salle/views.py

@@ -3,6 +3,7 @@ from datetime import timedelta
 
 from django.contrib import messages
 from django.contrib.auth.decorators import login_required
+from django.db.models import Q
 from django.forms import model_to_dict
 from django.http import HttpRequest, JsonResponse
 from django.shortcuts import redirect, render
@@ -16,13 +17,16 @@ from .models import Reservation
 
 @login_required
 def index(request: HttpRequest):
-
+    """Vue d'affichage de la page d'accueil de la gestion des salles"""
     try:
         employe = Employe.objects.get(user=request.user)
     except Employe.DoesNotExist:
         messages.error(request, "Profil employé introuvable.")
         return redirect('gestion_conges:conge')
 
+    est_equipe_info = fonctions_utilitaire.est_equipe_informatique(employe)
+    est_assistante = fonctions_utilitaire.est_assistante_direction(employe)
+
     if request.method == "POST":
         form = ReservationForm(request.POST)
 
@@ -37,11 +41,11 @@ def index(request: HttpRequest):
             besoin_ordi = form.cleaned_data['besoin_ordi']
 
             if date_fin < date_debut:
-                messages.error(request, "Date fin invalide.")
+                messages.error(request, "La date de fin est antérieure à la date de début.")
                 return redirect('gestion_salle:reservation-salle')
 
             if heure_fin <= heure_debut:
-                messages.error(request, "Heure invalide.")
+                messages.error(request, "L'heure de fin est antérieure à l'heure de début.")
                 return redirect('gestion_salle:reservation-salle')
             
             if not request.user.first_name.strip() or not request.user.last_name.strip():
@@ -54,8 +58,9 @@ def index(request: HttpRequest):
             created = []
             current_date = date_debut
 
-            while current_date <= date_fin:
+            print(f"Création de réservations pour {employe}")
 
+            while current_date <= date_fin:
                 reservation = Reservation.objects.create(
                     employe=employe,
                     date_debut=current_date,
@@ -74,18 +79,35 @@ def index(request: HttpRequest):
 
             messages.success(request, "Réservation(s) créée(s) avec succès.")
             
-            if fonctions_utilitaire.EMAIL_ASSISTANTE_DE_DIRECTION:
+            # Nouvelle logique d'envoi de mails à la création :
+            # 1. Une demande de salle (formation/réunion) est envoyée à l'assistante de direction pour validation.
+            #    - Si elle nécessite un lien Zoom ou un ordinateur, l'équipe informatique sera notifiée
+            #      seulement après la validation par l'assistante.
+            # 2. Une demande de lien Zoom directement est envoyée à l'équipe informatique.
+            if salle != "lien_zoom" and fonctions_utilitaire.EMAIL_ASSISTANTE_DE_DIRECTION:
                 fonctions_utilitaire.envoyer_mail(
-                    sujet = "Reservation de salle",
-                    message = f"""
+                    sujet="Demande de réservation de salle",
+                    message=f"""Bonjour,
 Une nouvelle demande de réservation de la {dict(Reservation.TYPE_CHOICES).get(salle)} a été effectuée par {employe.user.first_name} {employe.user.last_name} du {form.cleaned_data.get('date_debut').strftime('%d/%m/%Y')} au {form.cleaned_data.get('date_fin').strftime('%d/%m/%Y')} pour motif "{motif}". 
+{"Cette demande nécessite un lien Zoom et/ou un ordinateur et sera traitée par l'équipe informatique après votre validation." if besoin_zoom or besoin_ordi else ""}
 Veuillez vous connecter à la plateforme pour plus de détails.""",
-                    destinataires = fonctions_utilitaire.EMAIL_ASSISTANTE_DE_DIRECTION
+                    destinataires=fonctions_utilitaire.EMAIL_ASSISTANTE_DE_DIRECTION
+                )
+
+            if salle == "lien_zoom" and fonctions_utilitaire.EMAIL_EQUIPE_INFORMATIQUE:
+                fonctions_utilitaire.envoyer_mail(
+                    sujet="Demande de lien Zoom",
+                    message=f"""Bonjour,
+Une nouvelle demande de lien Zoom a été effectuée par {employe.user.first_name} {employe.user.last_name} du {date_debut.strftime('%d/%m/%Y')} au {date_fin.strftime('%d/%m/%Y')} de {heure_debut.strftime('%H:%M')} à {heure_fin.strftime('%H:%M')} pour le motif {motif}.
+
+Veuillez vous connecter à la plateforme pour traiter cette demande.""",
+                    destinataires=fonctions_utilitaire.EMAIL_EQUIPE_INFORMATIQUE
                 )
             return redirect('gestion_salle:reservation-salle')
 
+
     formulaire_reservation = ReservationForm()
-    departement = employe.departement
+    # departement = employe.departement
 
     liste_demande_reservation = Reservation.objects.filter(
         employe=employe,
@@ -95,29 +117,33 @@ Veuillez vous connecter à la plateforme pour plus de détails.""",
     context = {
         'formulaire_reservation': formulaire_reservation,
         'nb_reservation_attente': Reservation.objects.filter(statut='en_attente').count(),
-        'appartient_au_departement_informatique': departement and departement.nom == "Systeme informatique",
-        'appartient_direction':  request.user.groups.filter(name='direction').exists(),
+        'appartient_au_departement_informatique': est_equipe_info,
+        'est_assistante_direction': est_assistante,
+        'est_equipe_informatique': est_equipe_info,
         'liste_demande_reservation': list(liste_demande_reservation),
     }
 
     return render(request, "gestion_salle/index.html", context)
  
+@login_required
 def liste_reservation(request:HttpRequest):
     """Vue d'affichage des creneaux disponibles"""
-    reservations = Reservation.objects.filter(statut = "validee")
+    try:
+        employe = Employe.objects.get(user=request.user)
+        reservations = Reservation.objects.filter(
+            Q(statut="validee")
+            | Q(statut="en_attente", employe=employe)
+        )
+    except Employe.DoesNotExist:
+        reservations = Reservation.objects.filter(statut="validee")
+
     liste_reservation = []
 
     for reservation in reservations:
-        print(reservation.statut)
-        color = None
         if reservation.statut == "en_attente":
             color = "#ffc107"
-        elif reservation.statut == "validee":
-            color = "#198754"
         else:
-            color = "#dc3545"
-
-        print(dict(Reservation.TYPE_CHOICES).get(reservation.salle))
+            color = "#198754"
 
         liste_reservation.append({ 
             "guid": reservation.pk, 
@@ -126,15 +152,45 @@ def liste_reservation(request:HttpRequest):
             "start": reservation.heure_debut, 
             "end": reservation.heure_fin, 
             "color": color,
+            "statut": dict(Reservation.STATUT).get(reservation.statut),
         })
     return JsonResponse(liste_reservation, safe=False)
 
 @login_required
 def liste_reservation_attente(request):
+    """Liste des réservations en attente, filtrée selon le profil :
+    - Assistante de direction : voit les salles à valider (pas les liens choisis).
+    - Équipe informatique : voit les liens choisis et les salles demandant zoom/ordi non traités.
+    """
+    try:
+        employe = Employe.objects.get(user=request.user)
+    except Employe.DoesNotExist:
+        return JsonResponse({"success": False, "message": "Profil employé introuvable"})
+
+    est_informatique = fonctions_utilitaire.est_equipe_informatique(employe)
+    est_assistante = fonctions_utilitaire.est_assistante_direction(employe)
+
     reservations = Reservation.objects.filter(statut="en_attente")
+
+    if est_assistante:
+        # L'assistante voit les demandes de salle (formation / réunion)
+        # Elle ne reçoit pas les demandes de simple lien (lien_zoom) qui vont à l'équipe info.
+        reservations = reservations.exclude(salle="lien_zoom")
+    elif est_informatique:
+        # L'équipe informatique voit :
+        #  - les demandes de lien zoom directs
+        #  - les demandes de salle avec besoin_zoom ou besoin_ordi (déjà validés par l'assistance)
+        #    pour lesquelles le lien_zoom n'a pas encore était fourni (lien_zoom vide)
+        reservations = reservations.filter(
+            Q(salle="lien_zoom") | Q(besoin_zoom=True) | Q(besoin_ordi=True)
+        )
+
+    print("Test :", [reservation.employe for reservation in reservations])
+
     liste_reservation = [
         {
             **model_to_dict(reservation),
+            "id": reservation.pk,
             "employe": f"{reservation.employe.user.first_name} {reservation.employe.user.last_name}",
             "employe_id": reservation.employe.user.id,
             "salle": dict(Reservation.TYPE_CHOICES).get(reservation.salle),
@@ -145,6 +201,49 @@ def liste_reservation_attente(request):
 
     return JsonResponse(liste_reservation, safe=False)
 
+
+@login_required
+def supprimer_reservation(request: HttpRequest, reservation_id: int):
+    if request.method != "POST":
+        return JsonResponse(
+            {"success": False, "message": "Méthode non autorisée."},
+            status=405,
+        )
+
+    try:
+        employe = Employe.objects.get(user=request.user)
+    except Employe.DoesNotExist:
+        return JsonResponse(
+            {"success": False, "message": "Profil employé introuvable."},
+            status=403,
+        )
+
+    try:
+        reservation = Reservation.objects.get(
+            pk=reservation_id,
+            employe=employe,
+        )
+    except Reservation.DoesNotExist:
+        return JsonResponse(
+            {"success": False, "message": "Réservation introuvable ou accès non autorisé."},
+            status=404,
+        )
+
+    if reservation.statut != "en_attente":
+        return JsonResponse(
+            {
+                "success": False,
+                "message": "Seules les réservations en attente peuvent être supprimées.",
+            },
+            status=400,
+        )
+
+    reservation.delete()
+    return JsonResponse(
+        {"success": True, "message": "Votre réservation en attente a été supprimée."}
+    )
+
+
 def detail_reservation(request:HttpRequest, reservation_id:int):
     reservation = Reservation.objects.get(id=reservation_id)
     employe = reservation.employe.user
@@ -161,55 +260,107 @@ def detail_reservation(request:HttpRequest, reservation_id:int):
         'besoin_zoom': reservation.besoin_zoom,
         'besoin_ordinateur': reservation.besoin_ordi,
         'lien_zoom': reservation.lien_zoom or '',
+        'est_proprietaire': request.user.is_authenticated and reservation.employe.user_id == request.user.id,
+        'est_en_attente': reservation.statut == "en_attente",
     }
 
     return JsonResponse(reservation_json, safe=True)
 
 @login_required
 def ajouter_lien_zoom(request:HttpRequest):
-    """Vue de gestion de l'ajout du lien zoom"""
+    """Traitement de la réservation par l'équipe informatique :
+    - Ajoute lien zoom si demandé.
+    - Valide une réservation qui exigeait un traitement par l'équipe informatique.
+    - Envoie un mail à toutes les équipes et au demandeur après validation.
+    """
+    reservation = None
+
     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.")
-            return redirect("salle")
-        
-    if not (reservation.besoin_zoom or reservation.salle == "lien_zoom"):
-        messages.error(request, "Cette réservation ne nécessite pas de lien Zoom.")
-        return redirect('gestion_salle:reservation-salle')
-    elif reservation.statut in ["annulee", "refusee"]:
-        messages.error(request, "Le lien Zoom ne peut être ajouté pour les réservations annulée ou refusée.")
+        except (Reservation.DoesNotExist, ValueError):
+            messages.error(request, "La réservation sélectionnée n'existe pas.")
             return redirect('gestion_salle:reservation-salle')
 
-    if request.method == "POST":
+        try:
+            employe = Employe.objects.get(user=request.user)
+        except Employe.DoesNotExist:
+            messages.error(request, "Profil employé introuvable.")
+            return redirect('gestion_salle:reservation-salle')
+
+        # Vérification : seule l'équipe informatique peut traiter ces demandes
+        if not fonctions_utilitaire.est_equipe_informatique(employe):
+            messages.error(request, "Vous n'êtes pas autorisé à traiter cette demande.")
+            return redirect('gestion_salle:reservation-salle')
+
+        if reservation.salle == "lien_zoom" or reservation.besoin_zoom:
             lien = request.POST.get("lien_zoom")
+            if not lien:
+                messages.error(request, "Veuillez fournir un lien Zoom.")
+                return redirect('gestion_salle:reservation-salle')
             reservation.lien_zoom = lien
+
         reservation.statut = "validee"
         reservation.save()
-        messages.success(request, "Le lien Zoom a été ajouté et la réservation a été validée. ")
+        messages.success(request, "La demande a été traitée et validée. ")
+
+        # Mail au demandeur
+        fonctions_utilitaire.envoyer_mail(
+            sujet="Réservation traitée",
+            message=f"""Bonjour {reservation.employe.user.first_name} {reservation.employe.user.last_name}, votre demande de {dict(Reservation.TYPE_CHOICES).get(reservation.salle)}{' avec lien Zoom' if reservation.lien_zoom else ''} a été traitée. Veuillez vous connecter à la plateforme pour plus de détails.""",
+            destinataires=[reservation.employe.user.email]
+        )
+
+        # Mail aux emails de l'équipe informatique (hors utilisateur courant)
+        autres_membres = Employe.objects.filter(departement__nom="Systeme informatique").exclude(id=employe.id).values_list('user__email', flat=True)
+        if autres_membres:
+            fonctions_utilitaire.envoyer_mail(
+                sujet="Réservation validée par l'équipe informatique",
+                message=f"""Bonjour,
+La réservation {dict(Reservation.TYPE_CHOICES).get(reservation.salle)} du {reservation.date_debut.strftime('%d/%m/%Y')} pour {reservation.employe.user.first_name} {reservation.employe.user.last_name} a été traitée et validée par {employe.user.first_name} {employe.user.last_name}.\n\nVeuillez vous connecter à la plateforme pour plus de détails.""",
+                destinataires=list(autres_membres)
+            )
+
     return redirect('gestion_salle:reservation-salle')
 
 @login_required
 def annuler_reservation(request:HttpRequest):
-    """Vue de gestion de l'annulation de la reservation"""
+    """Vue de gestion de l'annulation de la reservation, réservée à l'employé demandeur ou aux validateurs."""
     if request.method == 'POST':
         reservation_id= request.POST['id_reservation']
         try:
             reservation =  Reservation.objects.get(id=reservation_id)
         except Reservation.DoesNotExist:
-            messages.error(request, "La resevertion selectionné n'existe pas.")
-            return redirect("salle")
+            messages.error(request, "La reservation selectionné n'existe pas.")
+            return redirect("gestion_salle:reservation-salle")
+
+        try:
+            employe = Employe.objects.get(user=request.user)
+        except Employe.DoesNotExist:
+            messages.error(request, "Profil employé introuvable.")
+            return redirect("gestion_salle:reservation-salle")
+
+        # Seul l'employé demandeur ou un validateur (assistante / équipe info) peut annuler
+        est_demandeur = reservation.employe == employe
+        est_validateur = fonctions_utilitaire.est_assistante_direction(employe) or fonctions_utilitaire.est_equipe_informatique(employe)
+
+        if not est_demandeur and not est_validateur:
+            messages.error(request, "Vous n'êtes pas autorisé à annuler cette réservation.")
+            return redirect('gestion_salle:reservation-salle')
 
         reservation.statut = 'annulee'
         reservation.save()
-        messages.success(request, "Votre réservation a été annulée.")
+        messages.success(request, "La réservation a été annulée.")
     return redirect('gestion_salle:reservation-salle')
 
 @login_required
 def valider_reservation(request: HttpRequest):
-    """Validation d'une réservation"""
+    """Validation d'une réservation selon le profil :
+    - Assistante de direction : valide les demandes de salles (sauf lien_zoom) ;
+    - Équipe informatique : valide les demandes de lien_zoom et les 
+    demandes de salle avec besoin zoom/ordi restées en attente après la première correction.
+    """
     if request.method == 'POST':
         reservation_id = request.POST.get('id_reservation')
 
@@ -217,36 +368,121 @@ def valider_reservation(request: HttpRequest):
             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")
+            return redirect('gestion_salle:reservation-salle')
 
-        reservation.statut = 'validee'
-        reservation.save()
+        try:
+            employe = Employe.objects.get(user=request.user)
+        except Employe.DoesNotExist:
+            messages.error(request, "Profil employé introuvable.")
+            return redirect('gestion_salle:reservation-salle')
 
+        est_assistante = fonctions_utilitaire.est_assistante_direction(employe)
+        est_equipe_info = fonctions_utilitaire.est_equipe_informatique(employe)
+
+        if est_assistante:
+            # L'assistante ne doit pas traiter les simple lien_zoom
+            if reservation.salle == "lien_zoom":
+                messages.error(request, "Les demandes de lien Zoom sont traitées par l'équipe informatique.")
+                return redirect('gestion_salle:reservation-salle')
+
+            if reservation.besoin_zoom or reservation.besoin_ordi:
+                # La demande reste en attente, l'équipe informatique doit être notifiée
+                # L'assistante a donné sa validation "salle", on garde le statut "en_attente"
+                # pour que l'équipe informatique puisse traiter, et on envoie le mail à l'équipe.
+                if fonctions_utilitaire.EMAIL_EQUIPE_INFORMATIQUE:
                     fonctions_utilitaire.envoyer_mail(
-            sujet = "Reservation de salle",
-            message = f"""Bonjour {reservation.employe.user.first_name} {reservation.employe.user.last_name}, votre reservation de la salle {dict(Reservation.TYPE_CHOICES).get(reservation.salle)} du {reservation.date_debut.strftime('%d/%m/%Y')} au {reservation.date_fin.strftime('%d/%m/%Y')} pour motif "{reservation.motif_reservation}" a été validée. Veuillez vous connecter à la plateforme pour plus de détails.""",
-            destinataires = [reservation.employe.user.email]
+                        sujet="Réservation de salle à traiter par l'équipe informatique",
+                        message=f"""Bonjour,
+L'assistante de direction a validé la réservation de la salle {dict(Reservation.TYPE_CHOICES).get(reservation.salle)} pour {reservation.employe.user.first_name} {reservation.employe.user.last_name} le {reservation.date_debut.strftime('%d/%m/%Y')} (de {reservation.heure_debut.strftime('%H:%M')} à {reservation.heure_fin.strftime('%H:%M')}).
+Cette demande nécessite {"un lien zoom" if reservation.besoin_zoom else ""}{" - un ordinateur" if reservation.besoin_ordi else ""}.
+
+Veuillez vous connecter à la plateforme pour traiter cette demande.""",
+                        destinataires=fonctions_utilitaire.EMAIL_EQUIPE_INFORMATIQUE
                     )
 
-    messages.success(request, f"La réservation de {request.user.first_name} {request.user.last_name} validée avec succès.")
+                messages.success(request, "Réservation validée, une demande a été envoyée à l'équipe informatique.")
+                reservation.statut = "en_attente"  # reste en attente pour l'équipe info
+                reservation.save()
+            else:
+                # Validation directe sans traitement informatique
+                reservation.statut = "validee"
+                reservation.save()
+                fonctions_utilitaire.envoyer_mail(
+                    sujet="Réservation de salle validée",
+                    message=f"""Bonjour {reservation.employe.user.first_name} {reservation.employe.user.last_name}, votre réservation de la {dict(Reservation.TYPE_CHOICES).get(reservation.salle)} du {reservation.date_debut.strftime('%d/%m/%Y')} au {reservation.date_fin.strftime('%d/%m/%Y')} pour motif "{reservation.motif_reservation}" a été validée. Veuillez vous connecter à la plateforme pour plus de détails.""",
+                    destinataires=[reservation.employe.user.email]
+                )
+
+        elif est_equipe_info:
+            # L'équipe informatique traite le reste : lien_zoom et besoins info
+            if reservation.salle == "lien_zoom" or reservation.besoin_zoom or reservation.besoin_ordi:
+                if reservation.besoin_zoom and not reservation.lien_zoom:
+                    # La demande exige un lien zoom qui n'a pas été encore rempli,
+                    # il est préférable de passer par ajouter_lien_zoom. On laisse traiter.
+                    messages.success(request, "Veuillez ajouter le lien Zoom via le champs dédié.")
+                    return redirect('gestion_salle:reservation-salle')
+
+                reservation.statut = "validee"
+                reservation.save()
+
+                # Mail au demandeur + autres membres de l'équipe
+                fonctions_utilitaire.envoyer_mail(
+                    sujet="Réservation validée",
+                    message=f"""Bonjour {reservation.employe.user.first_name} {reservation.employe.user.last_name}, votre demande de {dict(Reservation.TYPE_CHOICES).get(reservation.salle)} a été traitée et validée. Veuillez vous connecter pour plus de détails.""",
+                    destinataires=[reservation.employe.user.email]
+                )
+
+                autres = Employe.objects.filter(departement__nom="Systeme informatique").exclude(id=employe.id).values_list('user__email', flat=True)
+                if autres:
+                    fonctions_utilitaire.envoyer_mail(
+                        sujet="Réservation traitée par un collègue",
+                        message=f"""Bonjour {request.user.first_name} {request.user.last_name}, la demande de {dict(Reservation.TYPE_CHOICES).get(reservation.salle)} du {reservation.date_debut.strftime('%d/%m/%Y')} pour {reservation.employe.user.first_name} {reservation.employe.user.last_name} a été traitée et validée par {employe.user.first_name} {employe.user.last_name}. Veuillez vous connecter pour plus de détails.""",
+                        destinataires=list(autres)
+                    )
+            else:
+                messages.error(request, "Cette réservation ne relève pas de l'équipe informatique.")
+                return redirect('gestion_salle:reservation-salle')
+        else:
+            messages.error(request, "Vous n'êtes pas autorisé à effectuer cette validation.")
+            return redirect('gestion_salle:reservation-salle')
+
+    messages.success(request, f"La réservation de {request.user.first_name} {request.user.last_name} a été 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"""
+    """Vue de gestion de refus de la reservation, autorisée à l'assistant de direction 
+    et l'équipe informatique selon le type de demande."""
     data = json.loads(request.body)
     reservation_id = data.get("id_reservation")
     try:
         reservation =  Reservation.objects.get(id=reservation_id)
     except Reservation.DoesNotExist:
-        return JsonResponse({"message": "La resevertion selectionné n'existe pas."})
-    else:
+        return JsonResponse({"message": "La reservation selectionné n'existe pas."})
+
+    try:
+        employe = Employe.objects.get(user=request.user)
+    except Employe.DoesNotExist:
+        return JsonResponse({"message": "Profil employé introuvable."})
+
+    est_assistante = fonctions_utilitaire.est_assistante_direction(employe)
+    est_equipe_info = fonctions_utilitaire.est_equipe_informatique(employe)
+
+    peut_refuser = False
+    if est_assistante and reservation.salle != "lien_zoom":
+        peut_refuser = True
+    elif est_equipe_info and (reservation.salle == "lien_zoom" or reservation.besoin_zoom or reservation.besoin_ordi):
+        peut_refuser = True and (not reservation.besoin_zoom or reservation.lien_zoom)
+
+    if not peut_refuser:
+        return JsonResponse({"message": "Vous n'êtes pas autorisé à refuser cette réservation."})
+
     reservation.statut = "refusee"
     reservation.save()
 
     fonctions_utilitaire.envoyer_mail(
-            sujet = "Reservation de salle",
-            message = f"""Bonjour {request.user.first_name} {request.user.last_name}, votre reservation de la salle {dict(Reservation.TYPE_CHOICES).get(reservation.salle)} du {reservation.date_debut.strftime('%d/%m/%Y')} au {reservation.date_fin.strftime('%d/%m/%Y')} pour motif "{reservation.motif_reservation}" a été refusée. Veuillez vous connecter à l'Assistante de Direction pour plus de détails.""",
+        sujet = "Réservation refusée",
+        message = f"""Bonjour {reservation.employe.user.first_name} {reservation.employe.user.last_name}, votre demande de réservation du {reservation.date_debut.strftime('%d/%m/%Y')} de {reservation.date_debut.strftime('%d/%m/%Y')} de {reservation.heure_debut.strftime('%H:%M')} à {reservation.heure_fin.strftime('%H:%M')} a été refusée.""",
         destinataires = [reservation.employe.user.email]
     )