From d0159df66526e0d57e0e069a14fd023f9801c9b9 Mon Sep 17 00:00:00 2001
From: iziram <matthias.hartmann@iziram.fr>
Date: Mon, 11 Sep 2023 09:05:29 +0200
Subject: [PATCH] Assiduites : edit assi fix est_just #717

---
 app/api/assiduites.py                         | 22 +++++++++++++------
 app/models/assiduites.py                      | 20 ++++++++++++-----
 .../assiduites/widgets/tableau_assi.j2        |  2 +-
 3 files changed, 31 insertions(+), 13 deletions(-)

diff --git a/app/api/assiduites.py b/app/api/assiduites.py
index ab568ae6..72bbbeda 100644
--- a/app/api/assiduites.py
+++ b/app/api/assiduites.py
@@ -27,7 +27,7 @@ from app.models import (
     Justificatif,
 )
 from flask_sqlalchemy.query import Query
-from app.models.assiduites import get_assiduites_justif
+from app.models.assiduites import get_assiduites_justif, get_justifs_from_date
 from app.scodoc.sco_exceptions import ScoValueError
 from app.scodoc.sco_permissions import Permission
 from app.scodoc.sco_utils import json_error
@@ -851,12 +851,20 @@ def _edit_singular(assiduite_unique, data):
         assiduite_unique.desc = desc
 
     # Cas 4 : est_just
-    est_just = data.get("est_just")
-    if est_just is not None:
-        if not isinstance(est_just, bool):
-            errors.append("param 'est_just' : booléen non reconnu")
-        else:
-            assiduite_unique.est_just = est_just
+    if assiduite_unique.etat == scu.EtatAssiduite.PRESENT:
+        assiduite_unique.est_just = False
+    else:
+        assiduite_unique.est_just = (
+            len(
+                get_justifs_from_date(
+                    assiduite_unique.etudiant.id,
+                    assiduite_unique.date_debut,
+                    assiduite_unique.date_fin,
+                    valid=True,
+                )
+            )
+            > 0
+        )
 
     if errors:
         err: str = ", ".join(errors)
diff --git a/app/models/assiduites.py b/app/models/assiduites.py
index 0f8933e7..994a88e0 100644
--- a/app/models/assiduites.py
+++ b/app/models/assiduites.py
@@ -134,7 +134,10 @@ class Assiduite(db.Model):
 
         if not est_just:
             est_just = (
-                len(_get_assiduites_justif(etud.etudid, date_debut, date_fin)) > 0
+                len(
+                    get_justifs_from_date(etud.etudid, date_debut, date_fin, valid=True)
+                )
+                > 0
             )
 
         if moduleimpl is not None:
@@ -375,16 +378,23 @@ def compute_assiduites_justified(
 
 def get_assiduites_justif(assiduite_id: int, long: bool):
     assi: Assiduite = Assiduite.query.get_or_404(assiduite_id)
-    return _get_assiduites_justif(assi.etudid, assi.date_debut, assi.date_fin, long)
+    return get_justifs_from_date(assi.etudid, assi.date_debut, assi.date_fin, long)
 
 
-def _get_assiduites_justif(
-    etudid: int, date_debut: datetime, date_fin: datetime, long: bool = False
+def get_justifs_from_date(
+    etudid: int,
+    date_debut: datetime,
+    date_fin: datetime,
+    long: bool = False,
+    valid: bool = False,
 ):
-    justifs: Justificatif = Justificatif.query.filter(
+    justifs: Query = Justificatif.query.filter(
         Justificatif.etudid == etudid,
         Justificatif.date_debut <= date_debut,
         Justificatif.date_fin >= date_fin,
     )
 
+    if valid:
+        justifs = justifs.filter(Justificatif.etat == EtatJustificatif.VALIDE)
+
     return [j.justif_id if not long else j.to_dict(True) for j in justifs]
diff --git a/app/templates/assiduites/widgets/tableau_assi.j2 b/app/templates/assiduites/widgets/tableau_assi.j2
index 78c34cf0..8ef6848a 100644
--- a/app/templates/assiduites/widgets/tableau_assi.j2
+++ b/app/templates/assiduites/widgets/tableau_assi.j2
@@ -239,7 +239,7 @@
                     edit = setModuleImplId(edit, module);
 
                     fullEditAssiduites(data.assiduite_id, edit, () => {
-                        try { getAllAssiduitesFromEtud(etudid, assiduiteCallBack) } catch (_) { }
+                        loadAll();
                     })