From d1d89cc42786615c11dfb64ceb51ad1d2a57431a Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Thu, 22 Feb 2024 17:39:18 +0100
Subject: [PATCH] =?UTF-8?q?Bulletin:=20d=C3=A9tection=20erreur=20rare=20?=
 =?UTF-8?q?=3F?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/scodoc/sco_bulletins.py  | 20 ++++++++++++--------
 app/scodoc/sco_exceptions.py |  8 ++++++--
 2 files changed, 18 insertions(+), 10 deletions(-)

diff --git a/app/scodoc/sco_bulletins.py b/app/scodoc/sco_bulletins.py
index 25b246b5..d7d91ba2 100644
--- a/app/scodoc/sco_bulletins.py
+++ b/app/scodoc/sco_bulletins.py
@@ -55,7 +55,7 @@ from app.models import (
     ScoDocSiteConfig,
 )
 from app.scodoc.sco_permissions import Permission
-from app.scodoc.sco_exceptions import AccessDenied, ScoValueError
+from app.scodoc.sco_exceptions import AccessDenied, ScoValueError, ScoTemporaryError
 from app.scodoc import html_sco_header
 from app.scodoc import htmlutils
 from app.scodoc import sco_assiduites
@@ -346,14 +346,14 @@ def formsemestre_bulletinetud_dict(formsemestre_id, etudid, version="long"):
         # auparavant on filtrait les modules sans notes
         #   si ue_status['cur_moy_ue'] != 'NA' alors u['modules'] = [] (pas de moyenne => pas de modules)
 
-        u[
-            "modules_capitalized"
-        ] = []  # modules de l'UE capitalisée (liste vide si pas capitalisée)
+        u["modules_capitalized"] = (
+            []
+        )  # modules de l'UE capitalisée (liste vide si pas capitalisée)
         if ue_status["is_capitalized"] and ue_status["formsemestre_id"] is not None:
             sem_origin = db.session.get(FormSemestre, ue_status["formsemestre_id"])
-            u[
-                "ue_descr_txt"
-            ] = f'capitalisée le {ndb.DateISOtoDMY(ue_status["event_date"])}'
+            u["ue_descr_txt"] = (
+                f'capitalisée le {ndb.DateISOtoDMY(ue_status["event_date"])}'
+            )
             u["ue_descr_html"] = (
                 f"""<a href="{ url_for( 'notes.formsemestre_bulletinetud',
             scodoc_dept=g.scodoc_dept, formsemestre_id=sem_origin.id, etudid=etudid)}"
@@ -731,7 +731,11 @@ def etud_descr_situation_semestre(
     infos["refcomp_specialite_long"] = ""
     if formsemestre.formation.is_apc():
         res: ResultatsSemestreBUT = res_sem.load_formsemestre_results(formsemestre)
-        parcour_id = res.etuds_parcour_id[etudid]
+        try:
+            parcour_id = res.etuds_parcour_id[etudid]
+        except KeyError as exc:
+            log("sco_bulletins: ScoTemporaryError 240222")
+            raise ScoTemporaryError() from exc
         parcour: ApcParcours = (
             db.session.get(ApcParcours, parcour_id) if parcour_id is not None else None
         )
diff --git a/app/scodoc/sco_exceptions.py b/app/scodoc/sco_exceptions.py
index 82339db9..23552c5c 100644
--- a/app/scodoc/sco_exceptions.py
+++ b/app/scodoc/sco_exceptions.py
@@ -28,6 +28,7 @@
 """Exception handling
 """
 from flask_login import current_user
+import app
 
 
 # --- Exceptions
@@ -237,8 +238,11 @@ class ScoTemporaryError(ScoValueError):
 
     def __init__(self, msg: str = ""):
         msg = """
-        <p>"Erreur temporaire</p>
-        <p>Veuillez ré-essayer. Si le problème persiste, merci de contacter l'assistance ScoDoc
+        <p>Erreur temporaire</p>
+        <p>Veuillez ré-essayer. Si le problème persiste (ou s'il venait
+        à se produire fréquemment), merci de contacter l'assistance ScoDoc
+        (voir <a href="https://scodoc.org/Contact/">les informations de contact</a>).
         </p>
         """
+        app.clear_scodoc_cache()
         super().__init__(msg)