From 959a98d0a21e534b1c01e5c9728cf653df282dbc Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Sun, 10 Mar 2024 04:44:42 +0100
Subject: [PATCH] Fix bug: get_assiduites_count / feuille_preparation_jury

---
 app/scodoc/sco_assiduites.py  | 12 ++++++------
 app/scodoc/sco_prepajury.py   |  6 +++---
 sco_version.py                |  2 +-
 tests/unit/test_assiduites.py |  2 +-
 4 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/app/scodoc/sco_assiduites.py b/app/scodoc/sco_assiduites.py
index b9b650ea..57faa89b 100644
--- a/app/scodoc/sco_assiduites.py
+++ b/app/scodoc/sco_assiduites.py
@@ -111,9 +111,9 @@ class CountCalculator:
             evening if evening else ScoDocSiteConfig.get("assi_afternoon_time", "18:00")
         )
 
-        self.non_work_days: list[
-            scu.NonWorkDays
-        ] = scu.NonWorkDays.get_all_non_work_days(dept_id=g.scodoc_dept_id)
+        self.non_work_days: list[scu.NonWorkDays] = (
+            scu.NonWorkDays.get_all_non_work_days(dept_id=g.scodoc_dept_id)
+        )
 
         # Sera utilisé pour les assiduités longues (> 1 journée)
         self.nb_heures_par_jour = (
@@ -669,7 +669,7 @@ def create_absence_billet(
 
 
 # Gestion du cache
-def get_assiduites_count(etudid: int, sem: dict) -> tuple[int, int]:
+def get_assiduites_count(etudid: int, sem: dict) -> tuple[int, int, int]:
     """Les comptes d'absences de cet étudiant dans ce semestre:
     tuple (nb abs non justifiées, nb abs justifiées, nb abs total)
     Utilise un cache.
@@ -685,7 +685,7 @@ def get_assiduites_count(etudid: int, sem: dict) -> tuple[int, int]:
 
 def formsemestre_get_assiduites_count(
     etudid: int, formsemestre: FormSemestre, moduleimpl_id: int = None
-) -> tuple[int, int]:
+) -> tuple[int, int, int]:
     """Les comptes d'absences de cet étudiant dans ce semestre:
     tuple (nb abs non justifiées, nb abs justifiées, nb abs total)
     Utilise un cache.
@@ -712,7 +712,7 @@ def get_assiduites_count_in_interval(
     date_debut: datetime = None,
     date_fin: datetime = None,
     moduleimpl_id: int = None,
-):
+) -> tuple[int, int, int]:
     """Les comptes d'absences de cet étudiant entre ces deux dates, incluses:
     tuple (nb abs non justifiées, nb abs justifiées, nb abs total)
     On peut spécifier les dates comme datetime ou iso.
diff --git a/app/scodoc/sco_prepajury.py b/app/scodoc/sco_prepajury.py
index 4806a068..db04bc04 100644
--- a/app/scodoc/sco_prepajury.py
+++ b/app/scodoc/sco_prepajury.py
@@ -139,9 +139,9 @@ def feuille_preparation_jury(formsemestre_id):
                     main_partition_id, ""
                 )
         # absences:
-        e_nbabs, e_nbabsjust = sco_assiduites.get_assiduites_count(etud.id, sem)
-        nbabs[etud.id] = e_nbabs
-        nbabsjust[etud.id] = e_nbabs - e_nbabsjust
+        _, nbabsjust[etud.id], nbabs[etud.id] = sco_assiduites.get_assiduites_count(
+            etud.id, sem
+        )
 
     # Codes des UE "semestre précédent":
     ue_prev_codes = list(prev_moy_ue.keys())
diff --git a/sco_version.py b/sco_version.py
index 960e79a5..7cc08b37 100644
--- a/sco_version.py
+++ b/sco_version.py
@@ -1,7 +1,7 @@
 # -*- mode: python -*-
 # -*- coding: utf-8 -*-
 
-SCOVERSION = "9.6.950"
+SCOVERSION = "9.6.951"
 
 SCONAME = "ScoDoc"
 
diff --git a/tests/unit/test_assiduites.py b/tests/unit/test_assiduites.py
index 3063c973..11260746 100644
--- a/tests/unit/test_assiduites.py
+++ b/tests/unit/test_assiduites.py
@@ -1127,7 +1127,7 @@ def test_calcul_assiduites(test_client):
     etud: Identite = data["etuds"][0]
 
     """
-    Exemple tuple: 
+    Exemple tuple:
     (
         "12-04T08:00", # Date de début
         "12-04T09:00", # Date de fin