From 9c0ac1ab48e4e7224263ba2eebb7a11197bbcfe5 Mon Sep 17 00:00:00 2001 From: Emmanuel Viennet Date: Wed, 6 Dec 2023 12:59:30 +0100 Subject: [PATCH] =?UTF-8?q?Admin.=20comptes=20utilisateurs:=20permet=20au?= =?UTF-8?q?=20non=20super-admin=20de=20choisir=20le=20d=C3=A9partement=20d?= =?UTF-8?q?e=20rattachement=20si=20il=20a=20le=20droit=20d'administrer=20p?= =?UTF-8?q?lusieurs=20depts.?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- app/views/users.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/app/views/users.py b/app/views/users.py index 143a5e4c78..796e0f8526 100644 --- a/app/views/users.py +++ b/app/views/users.py @@ -488,7 +488,8 @@ def create_user_form(user_name=None, edit=0, all_roles=True): }, ), ] - # Si SuperAdmin, propose de choisir librement le dept du nouvel utilisateur + # Si on a le droit d'administrer les utilisateurs de plusieurs départements, + # propose le choix du dept du nouvel utilisateur selectable_dept_acronyms = set(administrable_dept_acronyms) if edit: if the_user.dept is not None: # ajoute dept actuel de l'utilisateur @@ -500,7 +501,7 @@ def create_user_form(user_name=None, edit=0, all_roles=True): if g.scodoc_dept in selectable_dept_acronyms else (auth_dept or "") ) - if is_super_admin and len(selectable_dept_acronyms) > 1: + if len(selectable_dept_acronyms) > 1: selectable_dept_acronyms = sorted(list(selectable_dept_acronyms)) descr.append( ( @@ -685,9 +686,8 @@ def create_user_form(user_name=None, edit=0, all_roles=True): if "status" in vals: vals["active"] = vals["status"] == "" # Département: - if auth_dept: # pas super-admin - if ("dept" in vals) and (vals["dept"] not in selectable_dept_acronyms): - del vals["dept"] # ne change pas de dept + if ("dept" in vals) and (vals["dept"] not in selectable_dept_acronyms): + del vals["dept"] # ne change pas de dept # Traitement des roles: ne doit pas affecter les rôles # que l'on en contrôle pas: for role in orig_roles_strings: # { "Ens_RT", "Secr_CJ", ... }