diff --git a/app/api/assiduites.py b/app/api/assiduites.py index 4ca6fcbc7..c1efe25b8 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 fd9b8cc64..a22ca39ab 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 5dc18a489..ccfdd81e9 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"