From 0a38ed22e6b217f8b0a1596a3272a4987efabf33 Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Tue, 12 Dec 2023 13:58:07 +0100
Subject: [PATCH] =?UTF-8?q?Assiduit=C3=A9:=20Modifie=20couleurs=20calendri?=
 =?UTF-8?q?er?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/static/css/assiduites.css                | 10 ++++-
 app/templates/assiduites/pages/bilan_etud.j2 |  6 +++
 app/templates/assiduites/pages/calendrier.j2 | 44 ++++++++++++++++----
 app/views/assiduites.py                      |  4 +-
 4 files changed, 55 insertions(+), 9 deletions(-)

diff --git a/app/static/css/assiduites.css b/app/static/css/assiduites.css
index 45efe6bcf..2df17893a 100644
--- a/app/static/css/assiduites.css
+++ b/app/static/css/assiduites.css
@@ -3,8 +3,10 @@
   --color-absent: #e62a11;
   --color-absent-clair: rgb(252, 151, 50);
   --color-retard: #f0c865;
-  --color-justi: #29b1b9;
+  --color-justi: #29b990;
   --color-justi-clair: #48f6ff;
+  --color-justi-attente: yellow;
+  --color-justi-attente-stripe: #29b990; /* pink #fa25cb; */ /* #789dbb;*/
   --color-justi-invalide: #a84476;
   --color-nonwork: #badfff;
 
@@ -689,3 +691,9 @@ tr.row-assiduite.present td.assi-type {
 tr.row-justificatif.valide td.assi-type {
   background-color: var(--color-justi);
 }
+tr.row-justificatif.attente td.assi-type {
+  background-color: var(--color-justi-attente);
+}
+tr.row-justificatif.non_valide td.assi-type {
+  background-color: var(--color-justi-invalide);
+}
diff --git a/app/templates/assiduites/pages/bilan_etud.j2 b/app/templates/assiduites/pages/bilan_etud.j2
index 0ed1e57fa..02ba9591b 100644
--- a/app/templates/assiduites/pages/bilan_etud.j2
+++ b/app/templates/assiduites/pages/bilan_etud.j2
@@ -27,6 +27,12 @@
     <section class="nonvalide">
         <!-- Tableaux des assiduités (retard/abs) non justifiées -->
         <h4>Absences et retards non justifiés</h4>
+
+        {# XXX XXX XXX #}
+        <div class="ue_warning">Attention, cette page utilise des couleurs et conventions différentes
+        de celles des autres pages ScoDoc: elle sera prochainement modifée, merci de votre patience.
+        </div>
+
         <span class="iconline">
             <a class="icon filter" onclick="filterAssi()"></a>
             <a class="icon download" onclick="downloadAssi()"></a>
diff --git a/app/templates/assiduites/pages/calendrier.j2 b/app/templates/assiduites/pages/calendrier.j2
index 0fa311817..6058bfa5d 100644
--- a/app/templates/assiduites/pages/calendrier.j2
+++ b/app/templates/assiduites/pages/calendrier.j2
@@ -40,10 +40,13 @@
             <li><span title="Jaune clair" class="demo color retard est_just"></span> &rightarrow; retard justifié
             </li>
 
-            <li><span title="Quart Bleu" class="est_just demo"></span> &rightarrow; la période est justifiée par un
+            <li><span title="Quart Bleu" class="est_just demo"></span> &rightarrow; la période est couverte par un
                 justificatif valide</li>
-            <li><span title="Quart Violet" class="invalide demo"></span> &rightarrow; la période est
-                justifiée par un justificatif non valide / en attente de validation
+            <li><span title="Justif. non valide" class="invalide demo"></span> &rightarrow; la période est
+                couverte par un justificatif non valide
+            </li>
+            <li><span title="Justif. en attente" class="attente demo"></span> &rightarrow; la période
+                a un justificatif en attente de validation
             </li>
         </ul>
 
@@ -102,7 +105,6 @@
     .color.absent.est_just {
         background-color: var(--color-absent-justi) !important;
     }
-
     .color.retard {
         background-color: var(--color-retard) !important;
     }
@@ -167,10 +169,32 @@
         right: 0;
         background-color: var(--color-justi-invalide) !important;
     }
-
+    .color.attente::before {
+        content: "";
+        position: absolute;
+        width: 25%;
+        height: 100%;
+        right: 0;
+        background: repeating-linear-gradient(
+            to bottom,
+            var(--color-justi-attente-stripe) 0px,
+            var(--color-justi-attente-stripe) 4px,
+            var(--color-justi-attente) 4px,
+            var(--color-justi-attente) 7px
+        )!important;
+    }
     .demo.invalide {
         background-color: var(--color-justi-invalide) !important;
     }
+    .demo.attente {
+        background: repeating-linear-gradient(
+            to bottom,
+            var(--color-justi-attente-stripe) 0px,
+            var(--color-justi-attente-stripe) 4px,
+            var(--color-justi-attente) 4px,
+            var(--color-justi-attente) 7px
+        )!important;
+    }
 
     .demo.est_just {
         background-color: var(--color-justi) !important;
@@ -509,7 +533,10 @@
 
                     if (justificatifsMatin.some((j) => j.etat.toLowerCase() === "valide")) {
                         est_just = ["est_just"];
-                    } else if (justificatifsMatin.some((j) => j.etat.toLowerCase() !== "valide")) {
+                    } else if (justificatifsMatin.some((j) => j.etat.toLowerCase() === "attente")) {
+                        est_just = ["attente"];
+                    }
+                    else if (justificatifsMatin.some((j) => j.etat.toLowerCase() !== "valide")) {
                         est_just = ["invalide"];
                     }
 
@@ -550,7 +577,10 @@
 
                     if (justificatifsAprem.some((j) => j.etat.toLowerCase() === "valide")) {
                         est_just = ["est_just"];
-                    } else if (justificatifsAprem.some((j) => j.etat.toLowerCase() !== "valide")) {
+                    } else if (justificatifsAprem.some((j) => j.etat.toLowerCase() === "attente")) {
+                        est_just = ["attente"];
+                    }
+                    else if (justificatifsAprem.some((j) => j.etat.toLowerCase() !== "valide")) {
                         est_just = ["invalide"];
                     }
 
diff --git a/app/views/assiduites.py b/app/views/assiduites.py
index 17e0ff009..6160a478d 100644
--- a/app/views/assiduites.py
+++ b/app/views/assiduites.py
@@ -712,7 +712,9 @@ def _upload_justificatif_files(
 
         return True
     except ScoValueError as exc:
-        log(f"_upload_justificatif_files: error on {file.filename} for etud {etud.id}")
+        log(
+            f"_upload_justificatif_files: error on {file.filename} for etud {just.etudid}"
+        )
         form.error_message = f"Erreur sur fichier justificatif: {exc.args[0]}"
         return False