From 2a4fdf8b84481614875b0a4febd255da13ef6242 Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Fri, 5 Jan 2024 23:20:32 +0100
Subject: [PATCH] Fix typos

---
 app/api/assiduites.py    |  4 ++--
 app/api/justificatifs.py |  2 +-
 app/models/assiduites.py | 11 ++++++-----
 3 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/app/api/assiduites.py b/app/api/assiduites.py
index 4ca6fcbc..c1efe25b 100644
--- a/app/api/assiduites.py
+++ b/app/api/assiduites.py
@@ -173,7 +173,7 @@ def count_assiduites(
             404,
             message="étudiant inconnu",
         )
-    g.scodoc_dept_id = etud.dept_id
+    set_sco_dept(etud.departement.acronym)
 
     # Les filtres qui seront appliqués au comptage (type, date, etudid...)
     filtered: dict[str, object] = {}
@@ -446,7 +446,7 @@ def count_assiduites_formsemestre(
     if formsemestre is None:
         return json_error(404, "le paramètre 'formsemestre_id' n'existe pas")
 
-    g.scodoc_dept_id = formsemestre.dept_id
+    set_sco_dept(formsemestre.departement.acronym)
 
     # Récupération des étudiants du formsemestre
     etuds = formsemestre.etuds.all()
diff --git a/app/api/justificatifs.py b/app/api/justificatifs.py
index fd9b8cc6..a22ca39a 100644
--- a/app/api/justificatifs.py
+++ b/app/api/justificatifs.py
@@ -156,7 +156,7 @@ def justificatifs_dept(dept_id: int = None, with_query: bool = False):
     # Récupération du département et des étudiants du département
     dept: Departement = Departement.query.get(dept_id)
     if dept is None:
-        json_error(404, "Assiduité non existante")
+        return json_error(404, "Assiduité non existante")
     etuds: list[int] = [etud.id for etud in dept.etudiants]
 
     # Récupération des justificatifs des étudiants du département
diff --git a/app/models/assiduites.py b/app/models/assiduites.py
index 5dc18a48..ccfdd81e 100644
--- a/app/models/assiduites.py
+++ b/app/models/assiduites.py
@@ -159,14 +159,14 @@ class Assiduite(ScoDocModel):
         # Vérification jours non travaillés
         # -> vérifie si la date de début ou la date de fin est sur un jour non travaillé
         # On récupère les formsemestres des dates de début et de fin
-        formsemetre_date_debut: FormSemestre = get_formsemestre_from_data(
+        formsemestre_date_debut: FormSemestre = get_formsemestre_from_data(
             {
                 "etudid": etud.id,
                 "date_debut": date_debut,
                 "date_fin": date_debut,
             }
         )
-        formsemetre_date_fin: FormSemestre = get_formsemestre_from_data(
+        formsemestre_date_fin: FormSemestre = get_formsemestre_from_data(
             {
                 "etudid": etud.id,
                 "date_debut": date_fin,
@@ -174,11 +174,11 @@ class Assiduite(ScoDocModel):
             }
         )
         if date_debut.weekday() in NonWorkDays.get_all_non_work_days(
-            formsemestre_id=formsemetre_date_debut
+            formsemestre_id=formsemestre_date_debut
         ):
             raise ScoValueError("La date de début n'est pas un jour travaillé")
         if date_fin.weekday() in NonWorkDays.get_all_non_work_days(
-            formsemestre_id=formsemetre_date_fin
+            formsemestre_id=formsemestre_date_fin
         ):
             raise ScoValueError("La date de fin n'est pas un jour travaillé")
 
@@ -186,7 +186,8 @@ class Assiduite(ScoDocModel):
         assiduites: Query = etud.assiduites
         if is_period_conflicting(date_debut, date_fin, assiduites, Assiduite):
             log(
-                f"create_assiduite: period_conflicting etudid={etud.id} date_debut={date_debut} date_fin={date_fin}"
+                f"""create_assiduite: period_conflicting etudid={etud.id} date_debut={
+                    date_debut} date_fin={date_fin}"""
             )
             raise ScoValueError(
                 "Duplication: la période rentre en conflit avec une plage enregistrée"