From 48376724f5978df42b1ecf4869622f30507c204f Mon Sep 17 00:00:00 2001
From: Iziram <matthias.hartmann@iziram.fr>
Date: Wed, 26 Jun 2024 10:32:15 +0200
Subject: [PATCH] =?UTF-8?q?Assiduit=C3=A9=20:=20Permission=20JustifValidat?=
 =?UTF-8?q?e=20#823?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/api/justificatifs.py                    |  9 ++-
 app/forms/assiduite/ajout_assiduite_etud.py |  8 +--
 app/scodoc/sco_formsemestre_status.py       |  4 +-
 app/scodoc/sco_permissions.py               |  5 ++
 app/views/assiduites.py                     | 69 ++++++++++++++++++++-
 5 files changed, 83 insertions(+), 12 deletions(-)

diff --git a/app/api/justificatifs.py b/app/api/justificatifs.py
index ad77f54cd..28063d059 100644
--- a/app/api/justificatifs.py
+++ b/app/api/justificatifs.py
@@ -374,6 +374,10 @@ def _create_one(
         errors.append("param 'etat': invalide")
 
     etat: scu.EtatJustificatif = scu.EtatJustificatif.get(etat)
+    if etat != scu.EtatJustificatif.ATTENTE and not current_user.has_permission(
+        Permission.JustifValidate
+    ):
+        errors.append("param 'etat': non autorisé (Permission.JustifValidate)")
 
     # cas 2 : date_debut
     date_debut: str = data.get("date_debut", None)
@@ -473,7 +477,10 @@ def justif_edit(justif_id: int):
         if etat is None:
             errors.append("param 'etat': invalide")
         else:
-            justificatif_unique.etat = etat
+            if current_user.has_permission(Permission.JustifValidate):
+                justificatif_unique.etat = etat
+            else:
+                errors.append("param 'etat': non autorisé (Permission.JustifValidate)")
 
     # Cas 2 : raison
     raison: str = data.get("raison", False)
diff --git a/app/forms/assiduite/ajout_assiduite_etud.py b/app/forms/assiduite/ajout_assiduite_etud.py
index 302f73776..5489ac350 100644
--- a/app/forms/assiduite/ajout_assiduite_etud.py
+++ b/app/forms/assiduite/ajout_assiduite_etud.py
@@ -170,13 +170,7 @@ class AjoutJustificatifEtudForm(AjoutAssiOrJustForm):
     )
     etat = SelectField(
         "État du justificatif",
-        choices=[
-            ("", "Choisir..."),  # Placeholder
-            (scu.EtatJustificatif.ATTENTE.value, "En attente de validation"),
-            (scu.EtatJustificatif.NON_VALIDE.value, "Non valide"),
-            (scu.EtatJustificatif.MODIFIE.value, "Modifié"),
-            (scu.EtatJustificatif.VALIDE.value, "Valide"),
-        ],
+        choices=[],  # sera rempli dynamiquement
         validators=[DataRequired(message="This field is required.")],
     )
     fichiers = MultipleFileField(label="Ajouter des fichiers")
diff --git a/app/scodoc/sco_formsemestre_status.py b/app/scodoc/sco_formsemestre_status.py
index 8c3ac414a..4053e694e 100755
--- a/app/scodoc/sco_formsemestre_status.py
+++ b/app/scodoc/sco_formsemestre_status.py
@@ -1136,7 +1136,9 @@ def formsemestre_status(formsemestre_id=None, check_parcours=True):
 
     # --- Lien Traitement Justificatifs:
 
-    if current_user.has_permission(Permission.AbsJustifView):
+    if current_user.has_permission(
+        Permission.AbsJustifView
+    ) and current_user.has_permission(Permission.JustifValidate):
         H.append(
             f"""<p>
             <a class="stdlink" href="{url_for('assiduites.traitement_justificatifs',
diff --git a/app/scodoc/sco_permissions.py b/app/scodoc/sco_permissions.py
index 6e5e53ee6..29dbc0a0b 100644
--- a/app/scodoc/sco_permissions.py
+++ b/app/scodoc/sco_permissions.py
@@ -68,6 +68,11 @@ _SCO_PERMISSIONS = (
         "AbsJustifView",
         "Visualisation du détail des justificatifs (motif, fichiers)",
     ),
+    (
+        1 << 51,
+        "JustifValidate",
+        "Définir la validité d'un justificatif (valide, invalide, modifié)",
+    ),
     # Attention: les permissions sont codées sur 64 bits.
 )
 
diff --git a/app/views/assiduites.py b/app/views/assiduites.py
index 00c4d3171..d5d23afd1 100644
--- a/app/views/assiduites.py
+++ b/app/views/assiduites.py
@@ -519,11 +519,22 @@ def _record_assiduite_etud(
 
         if assi_etat != scu.EtatAssiduite.PRESENT and form.est_just.data:
             # si la case "justifiée est cochée alors on créé un justificatif de même période"
+
+            # L'état est Valide si l'user à la permission JustifValidate
+            etat: scu.EtatJustificatif = scu.EtatJustificatif.ATTENTE
+            if current_user.has_permission(Permission.JustifValidate):
+                etat = scu.EtatJustificatif.VALIDE
+            else:
+                flash(
+                    "Vous ne pouvez pas créer de justificatif valide,"
+                    + " il est automatiquement passé 'EN ATTENTE'",
+                )
+
             justi: Justificatif = Justificatif.create_justificatif(
                 etudiant=etud,
                 date_debut=dt_debut_tz_server,
                 date_fin=dt_fin_tz_server,
-                etat=scu.EtatJustificatif.VALIDE,
+                etat=etat,
                 user_id=current_user.id,
             )
 
@@ -642,6 +653,21 @@ def edit_justificatif_etud(justif_id: int):
     readonly = not current_user.has_permission(Permission.AbsChange)
 
     form = AjoutJustificatifEtudForm(obj=justif)
+    # Limite les choix d'état si l'utilisateur n'a pas la permission de valider
+    choix_etat: list = [
+        (scu.EtatJustificatif.ATTENTE.value, "En attente de validation")
+    ]
+
+    if current_user.has_permission(Permission.JustifValidate):
+        choix_etat = [
+            ("", "Choisir..."),
+            (scu.EtatJustificatif.ATTENTE.value, "En attente de validation"),
+            (scu.EtatJustificatif.NON_VALIDE.value, "Non valide"),
+            (scu.EtatJustificatif.MODIFIE.value, "Modifié"),
+            (scu.EtatJustificatif.VALIDE.value, "Valide"),
+        ]
+
+    form.etat.choices = choix_etat
 
     if readonly:
         form.disable_all()
@@ -717,6 +743,21 @@ def ajout_justificatif_etud():
     )
 
     form = AjoutJustificatifEtudForm()
+    # Limite les choix d'état si l'utilisateur n'a pas la permission de valider
+    choix_etat: list = [
+        (scu.EtatJustificatif.ATTENTE.value, "En attente de validation")
+    ]
+
+    if current_user.has_permission(Permission.JustifValidate):
+        choix_etat = [
+            ("", "Choisir..."),
+            (scu.EtatJustificatif.ATTENTE.value, "En attente de validation"),
+            (scu.EtatJustificatif.NON_VALIDE.value, "Non valide"),
+            (scu.EtatJustificatif.MODIFIE.value, "Modifié"),
+            (scu.EtatJustificatif.VALIDE.value, "Valide"),
+        ]
+
+    form.etat.choices = choix_etat
     if form.validate_on_submit():
         if form.cancel.data:  # cancel button
             return redirect(redirect_url)
@@ -782,6 +823,16 @@ def _record_justificatif_etud(
         form.set_error("Erreur: état invalide")
         return False
 
+    if (
+        not current_user.has_permission(Permission.JustifValidate)
+        and etat != scu.EtatJustificatif.ATTENTE
+    ):
+        log("_record_justificatif_etud: pas la permission")
+        form.set_error(
+            "Erreur: vous n'avez pas la permission de définir la validité d'un justificatif"
+        )
+        return False
+
     try:
         message = ""
         if justif:
@@ -1521,12 +1572,22 @@ def tableau_assiduite_actions():
 
     # Justification d'une assiduité depuis le tableau
     if action == "justifier" and obj_type == "assiduite":
+        # L'état est Valide si l'user à la permission JustifValidate
+        etat: scu.EtatJustificatif = scu.EtatJustificatif.ATTENTE
+        if current_user.has_permission(Permission.JustifValidate):
+            etat = scu.EtatJustificatif.VALIDE
+        else:
+            flash(
+                "Vous ne pouvez pas créer de justificatif valide,"
+                + " il est automatiquement passé 'EN ATTENTE'",
+            )
+
         # Création du justificatif correspondant
         justificatif_correspondant: Justificatif = Justificatif.create_justificatif(
             etudiant=objet.etudiant,
             date_debut=objet.date_debut,
             date_fin=objet.date_fin,
-            etat=scu.EtatJustificatif.VALIDE,
+            etat=etat,
             user_id=current_user.id,
         )
 
@@ -1534,7 +1595,8 @@ def tableau_assiduite_actions():
         scass.simple_invalidate_cache(
             justificatif_correspondant.to_dict(), objet.etudiant.id
         )
-        flash(f"{objet_name} justifiée")
+        if etat == scu.EtatJustificatif.VALIDE:
+            flash(f"{objet_name} justifiée")
         return redirect(request.referrer)
 
     # Si on arrive ici, c'est que l'action n'est pas autorisée
@@ -1695,6 +1757,7 @@ def signale_evaluation_abs(etudid: int = None, evaluation_id: int = None):
 
 @bp.route("traitement_justificatifs")
 @scodoc
+@permission_required(Permission.JustifValidate)
 @permission_required(Permission.AbsJustifView)
 def traitement_justificatifs():
     """Page de traitement des justificatifs