From 24bb78bdfd38a9513f0f1a786f03b5613d780b3c Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Fri, 14 Jul 2023 07:16:33 +0200
Subject: [PATCH] =?UTF-8?q?lien=20groups=5Fauto=5Frepartition=20sur=20?=
 =?UTF-8?q?=C3=A9diteur=20de=20partitions?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/scodoc/sco_groups.py                 | 31 +++++++++++++++---------
 app/templates/scolar/partition_editor.j2 |  3 +++
 2 files changed, 23 insertions(+), 11 deletions(-)

diff --git a/app/scodoc/sco_groups.py b/app/scodoc/sco_groups.py
index d0123c5c..d32a12db 100644
--- a/app/scodoc/sco_groups.py
+++ b/app/scodoc/sco_groups.py
@@ -950,10 +950,20 @@ def edit_partition_form(formsemestre_id=None):
 }
           </script>
           """,
-        r"""<h2>Partitions du semestre</h2>
+        f"""<h2>Partitions du semestre</h2>
+          <p class="help">
+          👉💡 vous pourriez essayer <a href="{
+              url_for("scolar.partition_editor",
+                scodoc_dept=g.scodoc_dept, formsemestre_id=formsemestre_id)
+          }" class="stdlink">le nouvel éditeur</a>
+          </p>
+
           <form name="editpart" id="editpart" method="POST" action="partition_create">
           <div id="epmsg"></div>
-          <table><tr class="eptit"><th></th><th></th><th></th><th>Partition</th><th>Groupes</th><th></th><th></th><th></th></tr>
+          <table>
+          <tr class="eptit">
+          <th></th><th></th><th></th><th>Partition</th><th>Groupes</th><th></th><th></th><th></th>
+          </tr>
     """,
     ]
     i = 0
@@ -1379,9 +1389,11 @@ def groups_auto_repartition(partition_id=None):
         raise AccessDenied("Partition non éditable")
     formsemestre_id = partition.formsemestre_id
     formsemestre = partition.formsemestre
-    # renvoie sur page édition groupes
+    # renvoie sur page édition partitions et groupes
     dest_url = url_for(
-        "scolar.affect_groups", scodoc_dept=g.scodoc_dept, partition_id=partition_id
+        "scolar.partition_editor",
+        scodoc_dept=g.scodoc_dept,
+        formsemestre_id=formsemestre_id,
     )
     if not sco_permissions_check.can_change_groups(formsemestre_id):
         raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !")
@@ -1402,7 +1414,7 @@ def groups_auto_repartition(partition_id=None):
     H = [
         html_sco_header.sco_header(page_title="Répartition des groupes"),
         f"""<h2>Répartition des groupes de {partition.partition_name}</h2>
-        <p>Semestre {formsemestre.titre_annee()}</p>",
+        <p>Semestre {formsemestre.titre_annee()}</p>
         <p class="help">Les groupes existants seront <b>effacés</b> et remplacés par
           ceux créés ici. La répartition aléatoire tente d'uniformiser le niveau
           des groupes (en utilisant la dernière moyenne générale disponible pour
@@ -1426,12 +1438,9 @@ def groups_auto_repartition(partition_id=None):
         return flask.redirect(dest_url)
     else:
         # form submission
-        log(
-            "groups_auto_repartition( partition_id=%s partition_name=%s"
-            % (partition_id, partition.partition_name)
-        )
-        groupNames = tf[2]["groupNames"]
-        group_names = sorted({x.strip() for x in groupNames.split(",")})
+        log(f"groups_auto_repartition({partition})")
+        group_names = tf[2]["groupNames"]
+        group_names = sorted({x.strip() for x in group_names.split(",")})
         # Détruit les groupes existant de cette partition
         for group in partition.groups:
             db.session.delete(group)
diff --git a/app/templates/scolar/partition_editor.j2 b/app/templates/scolar/partition_editor.j2
index 36161631..98c8cf71 100644
--- a/app/templates/scolar/partition_editor.j2
+++ b/app/templates/scolar/partition_editor.j2
@@ -194,6 +194,9 @@
 				<label title="Doit-on afficher les groupes de cette partition dans les listes ?">
 					<input class=groupe type=checkbox ${partition.show_in_lists?"checked":""} data-attr=show_in_lists> Afficher sur bulletins et tableaux
 				</label>
+				<label>
+					<a class="stdlink" href="/ScoDoc/{{formsemestre.departement.acronym}}/Scolarite/groups_auto_repartition?partition_id=${partition.id}">Répartir les étudiants</a>
+				</label>
 			</div>
 			`;