From cefef8a89a531fefac19be0be9e73fbffc68fbaf Mon Sep 17 00:00:00 2001 From: Iziram Date: Wed, 5 Jun 2024 10:47:36 +0200 Subject: [PATCH 1/2] =?UTF-8?q?Assiduit=C3=A9=20:=20fix=20urgent=20bug?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../assiduites/pages/edit_assiduite_etud.j2 | 2 +- app/views/assiduites.py | 12 +++++++++--- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/app/templates/assiduites/pages/edit_assiduite_etud.j2 b/app/templates/assiduites/pages/edit_assiduite_etud.j2 index d80d58af..beee768b 100644 --- a/app/templates/assiduites/pages/edit_assiduite_etud.j2 +++ b/app/templates/assiduites/pages/edit_assiduite_etud.j2 @@ -56,7 +56,7 @@
Description: - {% if objet.description != "" and objet.description is not None %} + {% if objet.description != "" and objet.description != None %} {{objet.description}} {% else %} Pas de description diff --git a/app/views/assiduites.py b/app/views/assiduites.py index 8484e01d..97696fcd 100644 --- a/app/views/assiduites.py +++ b/app/views/assiduites.py @@ -234,9 +234,15 @@ def ajout_assiduite_etud() -> str | Response: if formsemestre_id: formsemestre = FormSemestre.get_formsemestre(formsemestre_id) formsemestre = formsemestre if formsemestre in sems_etud else None + if formsemestre is None: + raise ScoValueError("Etudiant non inscrit dans ce semestre") else: - formsemestre = [sem for sem in sems_etud if sem.est_courant()] + formsemestre = list( + sorted(sems_etud, key=lambda x: x.est_courant(), reverse=True) + ) # Mets le semestre courant en premier et les autres dans l'ordre formsemestre = formsemestre[0] if formsemestre else None + if formsemestre is None: + raise ScoValueError("Aucun semestre disponible pour cet étudiant") # Gestion évaluations (appel à la page depuis les évaluations) evaluation_id: int | None = request.args.get("evaluation_id") @@ -540,7 +546,7 @@ def _record_assiduite_etud( lien: str = url_for( "assiduites.edit_assiduite_etud", - assiuite_id=assi.assiduite_id, + assiduite_id=assi.assiduite_id, scodoc_dept=g.scodoc_dept, ) @@ -2245,7 +2251,7 @@ def edit_assiduite_etud(assiduite_id: int): etud=etud, sco=ScoData(etud, formsemestre=formsemestre), form=form, - readonly=True, + readonly=readonly, objet=_preparer_objet("assiduite", assi), title=f"Assiduité {etud.nom_short}", ) From 09f697941d66089301d25fbe6e02fd3ba41f1445 Mon Sep 17 00:00:00 2001 From: Iziram Date: Wed, 5 Jun 2024 11:21:53 +0200 Subject: [PATCH 2/2] =?UTF-8?q?Assiduit=C3=A9=20:=20ajout=5Fassiduite=5Fet?= =?UTF-8?q?ud:=20changement=20msg=20erreur=20semestre=20non=20dispo?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- app/views/assiduites.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/views/assiduites.py b/app/views/assiduites.py index 97696fcd..21f39f8c 100644 --- a/app/views/assiduites.py +++ b/app/views/assiduites.py @@ -242,7 +242,9 @@ def ajout_assiduite_etud() -> str | Response: ) # Mets le semestre courant en premier et les autres dans l'ordre formsemestre = formsemestre[0] if formsemestre else None if formsemestre is None: - raise ScoValueError("Aucun semestre disponible pour cet étudiant") + raise ScoValueError( + "L'étudiant n'est actuellement pas inscrit: on ne peut pas saisir son assiduité" + ) # Gestion évaluations (appel à la page depuis les évaluations) evaluation_id: int | None = request.args.get("evaluation_id")