From e4ab0d5885d6369480c98466874e0501d9f4acc9 Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Mon, 6 Feb 2023 10:24:41 +0100
Subject: [PATCH] code cosmetic

---
 app/scodoc/sco_edit_ue.py | 24 ++++++++++++++----------
 1 file changed, 14 insertions(+), 10 deletions(-)

diff --git a/app/scodoc/sco_edit_ue.py b/app/scodoc/sco_edit_ue.py
index 906493a2..30a84a00 100644
--- a/app/scodoc/sco_edit_ue.py
+++ b/app/scodoc/sco_edit_ue.py
@@ -135,7 +135,6 @@ def do_ue_create(args):
     formation: Formation = Formation.query.get(args["formation_id"])
     formation.invalidate_module_coefs()
     # news
-    ue = UniteEns.query.get(ue_id)
     formation = Formation.query.get(args["formation_id"])
     ScolarNews.add(
         typ=ScolarNews.NEWS_FORM,
@@ -468,7 +467,9 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No
             <ul>"""
             for m in ue.modules:
                 modules_div += f"""<li><a class="stdlink" href="{url_for(
-                    "notes.module_edit",scodoc_dept=g.scodoc_dept, module_id=m.id)}">{m.code} {m.titre or "sans titre"}</a></li>"""
+                    "notes.module_edit",
+                    scodoc_dept=g.scodoc_dept, module_id=m.id)
+                }">{m.code} {m.titre or "sans titre"}</a></li>"""
             modules_div += """</ul></div>"""
         else:
             modules_div = ""
@@ -515,7 +516,8 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No
                     {
                         "titre": tf[2]["titre"],
                         "code": tf[2]["acronyme"],
-                        "coefficient": 1.0,  # tous les modules auront coef 1, et on utilisera les ECTS
+                        # tous les modules auront coef 1, et on utilisera les ECTS:
+                        "coefficient": 1.0,
                         "ue_id": ue_id,
                         "matiere_id": matiere_id,
                         "formation_id": formation_id,
@@ -731,7 +733,7 @@ def ue_table(formation_id=None, semestre_idx=1, msg=""):  # was ue_list
     ]
     if locked:
         H.append(
-            f"""<p class="help">Cette formation est verrouillée car
+            """<p class="help">Cette formation est verrouillée car
 des semestres verrouillés s'y réferent.
 Si vous souhaitez modifier cette formation (par exemple pour y ajouter un module), 
 vous devez:
@@ -751,14 +753,15 @@ du programme" (menu "Semestre") si vous avez un semestre en cours);
 
     if ues_with_duplicated_code:
         H.append(
-            f"""<div class="ue_warning"><span>Attention: plusieurs UE de cette
+            f"""<div class="ue_warning"><span>Attention: plusieurs UEs de cette
             formation ont le même code : <tt>{
                 ', '.join([
-                '<a class="stdlink" href="' + url_for( "notes.ue_edit", scodoc_dept=g.scodoc_dept, ue_id=ue["ue_id"] )
+                '<a class="stdlink" href="' + url_for( "notes.ue_edit",
+                    scodoc_dept=g.scodoc_dept, ue_id=ue["ue_id"] )
                 + '">' + ue["acronyme"] + " (code " + ue["ue_code"] + ")</a>"
                 for ue in ues_with_duplicated_code ])
             }</tt>.
-            Il faut corriger cela, sinon les capitalisations et ECTS seront 
+            Il faut corriger cela, sinon les capitalisations et ECTS seront
             erronés !</span></div>"""
         )
 
@@ -786,7 +789,7 @@ du programme" (menu "Semestre") si vous avez un semestre en cours);
             descr_refcomp = ""
             msg_refcomp = "associer à un référentiel de compétences"
         else:
-            descr_refcomp = f"""Référentiel de compétences: 
+            descr_refcomp = f"""Référentiel de compétences:
             <a href="{url_for('notes.refcomp_show', 
             scodoc_dept=g.scodoc_dept, refcomp_id=formation.referentiel_competence.id)}"
             class="stdlink">
@@ -800,7 +803,7 @@ du programme" (menu "Semestre") si vous avez un semestre en cours);
                 or formation.formsemestres.count() == 0
             ):
                 H.append(
-                    f"""<a class="stdlink" href="{url_for('notes.refcomp_assoc_formation', 
+                    f"""<a class="stdlink" href="{url_for('notes.refcomp_assoc_formation',
             scodoc_dept=g.scodoc_dept, formation_id=formation_id)
             }">{msg_refcomp}</a>"""
                 )
@@ -810,7 +813,8 @@ du programme" (menu "Semestre") si vous avez un semestre en cours);
         H.append(
             f"""</li>
         <li> <a class="stdlink" href="{
-            url_for('notes.edit_modules_ue_coefs', scodoc_dept=g.scodoc_dept, formation_id=formation_id, semestre_idx=semestre_idx)
+            url_for('notes.edit_modules_ue_coefs', 
+                scodoc_dept=g.scodoc_dept, formation_id=formation_id, semestre_idx=semestre_idx)
         }">Éditer les coefficients des ressources et SAÉs</a>
         </li>
         </ul>