From b8cf116fc6e6422791855636f60d1b9619599d9c Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Wed, 31 Aug 2022 09:16:00 +0200
Subject: [PATCH] Code cosmetic: re-ecriture de ue_sharing_code, nom option
 show_in_list

---
 app/scodoc/sco_edit_ue.py | 23 ++++++++++++-----------
 app/scodoc/sco_groups.py  | 13 ++++++++-----
 2 files changed, 20 insertions(+), 16 deletions(-)

diff --git a/app/scodoc/sco_edit_ue.py b/app/scodoc/sco_edit_ue.py
index 767049a30..fe5326af0 100644
--- a/app/scodoc/sco_edit_ue.py
+++ b/app/scodoc/sco_edit_ue.py
@@ -1290,15 +1290,12 @@ def ue_sharing_code(ue_code=None, ue_id=None, hide_ue_id=None):
     Either ue_code or ue_id may be specified.
     hide_ue_id spécifie un id à retirer de la liste.
     """
-    from app.scodoc import sco_formations
-
     ue_code = str(ue_code)
     if ue_id:
-        ue = ue_list(args={"ue_id": ue_id})[0]
+        ue = UniteEns.query.get_or_404(ue_id)
         if not ue_code:
-            ue_code = ue["ue_code"]
-        F = sco_formations.formation_list(args={"formation_id": ue["formation_id"]})[0]
-        formation_code = F["formation_code"]
+            ue_code = ue.ue_code
+        formation_code = ue.formation.formation_code
         # UE du même code, code formation et departement:
         q_ues = (
             UniteEns.query.filter_by(ue_code=ue_code)
@@ -1319,18 +1316,22 @@ def ue_sharing_code(ue_code=None, ue_id=None, hide_ue_id=None):
     ues = q_ues.all()
     if not ues:
         if ue_id:
-            return """<span class="ue_share">Seule UE avec code %s</span>""" % ue_code
+            return (
+                f"""<span class="ue_share">Seule UE avec code {ue_code or '-'}</span>"""
+            )
         else:
-            return """<span class="ue_share">Aucune UE avec code %s</span>""" % ue_code
+            return f"""<span class="ue_share">Aucune UE avec code {ue_code or '-'}</span>"""
     H = []
     if ue_id:
-        H.append('<span class="ue_share">Autres UE avec le code %s:</span>' % ue_code)
+        H.append(
+            f"""<span class="ue_share">Autres UE avec le code {ue_code or '-'}:</span>"""
+        )
     else:
-        H.append('<span class="ue_share">UE avec le code %s:</span>' % ue_code)
+        H.append(f"""<span class="ue_share">UE avec le code {ue_code or '-'}:</span>""")
     H.append("<ul>")
     for ue in ues:
         H.append(
-            f"""<li>{ue.acronyme} ({ue.titre}) dans <a class="stdlink" 
+            f"""<li>{ue.acronyme} ({ue.titre}) dans <a class="stdlink"
             href="{url_for("notes.ue_table", scodoc_dept=g.scodoc_dept, formation_id=ue.formation.id)}"
             >{ue.formation.acronyme} ({ue.formation.titre})</a>, version {ue.formation.version}
             </li>
diff --git a/app/scodoc/sco_groups.py b/app/scodoc/sco_groups.py
index 1f759e2aa..6e008e9c1 100644
--- a/app/scodoc/sco_groups.py
+++ b/app/scodoc/sco_groups.py
@@ -453,8 +453,11 @@ def formsemestre_get_etud_groupnames(formsemestre_id, attr="group_name"):
 def get_etud_formsemestre_groups(
     etud: Identite, formsemestre: FormSemestre, only_to_show=True
 ) -> list[GroupDescr]:
-    """Liste les groupes auxquels est inscrit"""
-    # Note: je n'ai pas réussi à cosntruire une requete SQLAlechemy avec
+    """Liste les groupes auxquels est inscrit.
+    Si only_to_show (défaut vrai), ne donne que les groupes "visiables",
+    c'est à dire des partitions avec show_in_lists True.
+    """
+    # Note: je n'ai pas réussi à construire une requete SQLAlechemy avec
     # la Table d'association group_membership
     cursor = db.session.execute(
         text(
@@ -1061,8 +1064,8 @@ def edit_partition_form(formsemestre_id=None):
             else:
                 checked = ""
             H.append(
-                '<div><input type="checkbox" class="rkbox" data-partition_id="%s" %s onchange="update_show_in_list(this);"/>afficher sur noms groupes</div>'
-                % (p["partition_id"], checked)
+                f"""<div><input type="checkbox" class="rkbox" data-partition_id="{p['partition_id']
+                }" {checked} onchange="update_show_in_list(this);"/>Afficher ces groupes sur les tableaux et bulletins</div>"""
             )
             H.append("</td>")
             #
@@ -1108,7 +1111,7 @@ def edit_partition_form(formsemestre_id=None):
     (ainsi, on peut afficher le classement en groupes de TD mais pas en
     groupe de TP, si ce sont deux partitions).
     </li>
-    <li>Décocher "afficher sur noms groupes" pour ne pas que cette partition
+    <li>Décocher "Afficher ces groupes sur les tableaux et bulletins" pour ne pas que cette partition
     apparaisse dans les noms de groupes
     </li>
     </ul>