From 157423edaffd2accb7ac2ef94f9c10f555dffb40 Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Mon, 18 Jul 2022 22:17:27 +0200
Subject: [PATCH] =?UTF-8?q?Fix:=20assure=20que=20le=20code=20enregistr?=
 =?UTF-8?q?=C3=A9=20est=20toujours=20dans=20le=20menu=20(utile=20pour=20le?=
 =?UTF-8?q?s=20ADM=20sur=20UE=20<=2010).?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/but/jury_but.py | 24 +++++++++++++++---------
 1 file changed, 15 insertions(+), 9 deletions(-)

diff --git a/app/but/jury_but.py b/app/but/jury_but.py
index 60cad7986f..2091c16af4 100644
--- a/app/but/jury_but.py
+++ b/app/but/jury_but.py
@@ -160,6 +160,9 @@ class DecisionsProposees:
         "Validation enregistrée"
         self.code_valide: str = code_valide
         "Code décision actuel enregistré"
+        # S'assure que le code enregistré est toujours présent dans le menu
+        if self.code_valide and self.code_valide not in self.codes:
+            self.codes.append(self.code_valide)
         self.explanation: str = explanation
         "Explication à afficher à côté de la décision"
         self.recorded = False
@@ -887,11 +890,21 @@ class DecisionsProposeesUE(DecisionsProposees):
         ue: UniteEns,
         inscription_etat: str = scu.INSCRIT,
     ):
-        super().__init__(etud=etud)
+        # Une UE peut être validée plusieurs fois en cas de redoublement (qu'elle soit capitalisée ou non)
+        # mais ici on a restreint au formsemestre donc une seule (prend la première)
+        self.validation = ScolarFormSemestreValidation.query.filter_by(
+            etudid=etud.id, formsemestre_id=formsemestre.id, ue_id=ue.id
+        ).first()
+
+        super().__init__(
+            etud=etud,
+            code_valide=self.validation.code if self.validation is not None else None,
+        )
+
         self.formsemestre = formsemestre
         self.ue: UniteEns = ue
         self.rcue: RegroupementCoherentUE = None
-        "Le rcu auquel est rattaché cette UE, ou None"
+        "Le rcue auquel est rattaché cette UE, ou None"
         self.inscription_etat = inscription_etat
         "inscription: I, DEM, DEF"
         if ue.type == sco_codes.UE_SPORT:
@@ -906,13 +919,6 @@ class DecisionsProposeesUE(DecisionsProposees):
             ]
             self.moy_ue = "-"
             return
-        # Une UE peut être validée plusieurs fois en cas de redoublement (qu'elle soit capitalisée ou non)
-        # mais ici on a restreint au formsemestre donc une seule (prend la première)
-        self.validation = ScolarFormSemestreValidation.query.filter_by(
-            etudid=self.etud.id, formsemestre_id=formsemestre.id, ue_id=ue.id
-        ).first()
-        if self.validation is not None:
-            self.code_valide = self.validation.code
 
         # Moyenne de l'UE ?
         res: ResultatsSemestreBUT = res_sem.load_formsemestre_results(formsemestre)