From 1a4461cfb054628f6c305d0c8a502a846f56b484 Mon Sep 17 00:00:00 2001 From: Matthias Hartmann Date: Fri, 17 Jun 2022 20:10:09 +0200 Subject: [PATCH] =?UTF-8?q?Fix=20Issue=20364=20:=20v=C3=A9rif=20group=5Fna?= =?UTF-8?q?me=20d=C3=A9j=C3=A0=20existant?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- app/scodoc/sco_groups.py | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/app/scodoc/sco_groups.py b/app/scodoc/sco_groups.py index 79ed91cad..6df63bef3 100644 --- a/app/scodoc/sco_groups.py +++ b/app/scodoc/sco_groups.py @@ -599,6 +599,11 @@ def XMLgetGroupsInPartition(partition_id): # was XMLgetGroupesTD response.headers["Content-Type"] = scu.XML_MIMETYPE return response +def check_group_name(group_name, partition): + """Raise ScoValueError if group_name exists in partition""" + exists = group_name in [g["group_name"] for g in get_partition_groups(partition)] + if exists: + raise ScoValueError("Le nom de groupe existe déjà dans la partition") def comp_origin(etud, cur_sem): """breve description de l'origine de l'étudiant (sem. precedent) @@ -1244,6 +1249,7 @@ def group_set_name(group_id, group_name, redirect=True): if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") redirect = int(redirect) + check_group_name(group_name, get_partition(group['partition_id'])) cnx = ndb.GetDBConnexion() groupEditor.edit(cnx, {"group_id": group_id, "group_name": group_name}) @@ -1372,13 +1378,13 @@ def groups_auto_repartition(partition_id=None): # Crée les nouveaux groupes group_ids = [] for group_name in group_names: - # try: - # checkGroupName(group_name) - # except: - # H.append('

Nom de groupe invalide: %s

'%group_name) - # return '\n'.join(H) + tf[1] + html_sco_header.sco_footer() + try: + check_group_name(group_name, partition) + except: + H.append('

Nom de groupe invalide: %s

'%group_name) + return '\n'.join(H) + tf[1] + html_sco_header.sco_footer() group_ids.append(create_group(partition_id, group_name)) - # + nt: NotesTableCompat = res_sem.load_formsemestre_results(formsemestre) identdict = nt.identdict # build: { civilite : liste etudids trie par niveau croissant }