From 72144a49525e4361fc4b7b87126ede8f30b919ea Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Wed, 29 Jun 2022 09:08:31 +0200
Subject: [PATCH] Minor bugs (edit_ue)

---
 app/scodoc/sco_bulletins_standard.py |  2 +-
 app/scodoc/sco_edit_ue.py            | 33 ++++++++++++++++++----------
 2 files changed, 22 insertions(+), 13 deletions(-)

diff --git a/app/scodoc/sco_bulletins_standard.py b/app/scodoc/sco_bulletins_standard.py
index e7c92ad7..0277ee8c 100644
--- a/app/scodoc/sco_bulletins_standard.py
+++ b/app/scodoc/sco_bulletins_standard.py
@@ -439,7 +439,7 @@ class BulletinGeneratorStandard(sco_bulletins_generator.BulletinGenerator):
                     ects_txt = "-"
 
                 t = {
-                    "titre": ue["acronyme"] + " " + ue["titre"],
+                    "titre": ue["acronyme"] + " " + (ue["titre"] or ""),
                     "_titre_html": plusminus
                     + ue["acronyme"]
                     + " "
diff --git a/app/scodoc/sco_edit_ue.py b/app/scodoc/sco_edit_ue.py
index 87991769..016d8880 100644
--- a/app/scodoc/sco_edit_ue.py
+++ b/app/scodoc/sco_edit_ue.py
@@ -312,8 +312,7 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No
                     "type": "int",
                     "allow_null": False,
                     "title": parcours.SESSION_NAME.capitalize(),
-                    "explanation": "%s de l'UE dans la formation"
-                    % parcours.SESSION_NAME,
+                    "explanation": f"{parcours.SESSION_NAME} de l'UE dans la formation",
                     "labels": ["non spécifié"] + [str(x) for x in semestres_indices],
                     "allowed_values": [""] + semestres_indices,
                 },
@@ -464,9 +463,7 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No
             + ue_div
             + html_sco_header.sco_footer()
         )
-    else:
-        if not tf[2]:  # ceci ne devrait pas arriver
-            raise ScoValueError("Merci de reprendre l'opération")
+    elif tf[2]:
         if create:
             if not tf[2]["ue_code"]:
                 del tf[2]["ue_code"]
@@ -499,14 +496,26 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No
         else:
             do_ue_edit(tf[2])
             flash("UE modifiée")
-        return flask.redirect(
-            url_for(
-                "notes.ue_table",
-                scodoc_dept=g.scodoc_dept,
-                formation_id=formation_id,
-                semestre_idx=tf[2]["semestre_idx"],
-            )
+
+    if tf[2]:
+        dest_semestre_idx = tf[2]["semestre_idx"]
+    elif ue:
+        dest_semestre_idx = ue.semestre_idx
+    elif default_semestre_idx:
+        dest_semestre_idx = default_semestre_idx
+    elif "semestre_idx" in request.form:
+        dest_semestre_idx = request.form["semestre_idx"]
+    else:
+        dest_semestre_idx = 1
+
+    return flask.redirect(
+        url_for(
+            "notes.ue_table",
+            scodoc_dept=g.scodoc_dept,
+            formation_id=formation_id,
+            semestre_idx=dest_semestre_idx,
         )
+    )
 
 
 def _add_ue_semestre_id(ues: list[dict], is_apc):