From a7058fb86b37f68c30319b95f53b2371d32739cd Mon Sep 17 00:00:00 2001
From: iziram <matthias.hartmann@iziram.fr>
Date: Wed, 13 Sep 2023 08:59:54 +0200
Subject: [PATCH] Assiduites : fix #728

---
 app/scodoc/sco_groups_view.py |  2 +-
 app/static/js/assiduites.js   |  2 +-
 app/views/assiduites.py       | 12 ++++++++----
 3 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/app/scodoc/sco_groups_view.py b/app/scodoc/sco_groups_view.py
index 728ccb3635..0eefe05805 100644
--- a/app/scodoc/sco_groups_view.py
+++ b/app/scodoc/sco_groups_view.py
@@ -900,7 +900,7 @@ def form_choix_jour_saisie_hebdo(groups_infos, moduleimpl_id=None):
             formsemestre_id=groups_infos.formsemestre_id,
             moduleimpl_id="" if moduleimpl_id is None else moduleimpl_id
             )
-    }';">Saisie du jour</button>
+    }';">Saisie du jour ({datetime.date.today().strftime('%d/%m/%Y')})</button>
     """
 
 
diff --git a/app/static/js/assiduites.js b/app/static/js/assiduites.js
index 019cb894a2..dbfa3736e1 100644
--- a/app/static/js/assiduites.js
+++ b/app/static/js/assiduites.js
@@ -577,7 +577,7 @@ function updateDate() {
     return true;
   } else {
     const att = document.createTextNode(
-      "Le jour sélectionné n'est pas un jour travaillé."
+      `Le jour sélectionné (${formatDate(date)}) n'est pas un jour travaillé.`
     );
     openAlertModal("Erreur", att, "", "crimson");
     dateInput.value = dateInput.getAttribute("value");
diff --git a/app/views/assiduites.py b/app/views/assiduites.py
index a2e5a3b1d7..db3cb9005c 100644
--- a/app/views/assiduites.py
+++ b/app/views/assiduites.py
@@ -565,10 +565,14 @@ def signal_assiduites_group():
 
     real_date = scu.is_iso_formated(date, True).date()
 
-    if real_date < formsemestre.date_debut:
-        date = formsemestre.date_debut.isoformat()
-    elif real_date > formsemestre.date_fin:
-        date = formsemestre.date_fin.isoformat()
+    if real_date < formsemestre.date_debut or real_date > formsemestre.date_fin:
+        real_str = real_date.strftime("%d/%m/%Y")
+        form_deb = formsemestre.date_debut.strftime("%d/%m/%Y")
+        form_fin = formsemestre.date_fin.strftime("%d/%m/%Y")
+        raise ScoValueError(
+            f"Impossible de saisir les assiduités pour le {real_str}"
+            + f" : Jour en dehors du semestre ( {form_deb} → {form_fin}) "
+        )
 
     # --- Restriction en fonction du moduleimpl_id ---
     if moduleimpl_id: