From 963a326426d1b19041bcaab290d03f9692806883 Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Thu, 16 Mar 2023 07:54:11 +0100
Subject: [PATCH] =?UTF-8?q?Fix:=20d=C3=A9partement=20par=20d=C3=A9faut=20e?=
 =?UTF-8?q?dition=20utilisateur?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/views/users.py | 18 ++++++++++++------
 sco_version.py     |  2 +-
 2 files changed, 13 insertions(+), 7 deletions(-)

diff --git a/app/views/users.py b/app/views/users.py
index e180b9a3..e6400882 100644
--- a/app/views/users.py
+++ b/app/views/users.py
@@ -337,7 +337,7 @@ def create_user_form(user_name=None, edit=0, all_roles=True):
                     "title": "Pseudo (login)",
                     "size": 20,
                     "allow_null": False,
-                    "explanation": "nom utilisé pour la connexion. Doit être unique parmi tous les utilisateurs. "
+                    "explanation": "nom utilisé pour la connexion. Doit être unique parmi tous les utilisateurs."
                     "Lettres ou chiffres uniquement.",
                 },
             ),
@@ -459,8 +459,16 @@ def create_user_form(user_name=None, edit=0, all_roles=True):
         ]
     # Si SuperAdmin, propose de choisir librement le dept du nouvel utilisateur
     selectable_dept_acronyms = set(administrable_dept_acronyms)
-    if edit and the_user.dept is not None:  # ajoute dept actuel de l'utilisateur
-        selectable_dept_acronyms |= {the_user.dept}
+    if edit:
+        if the_user.dept is not None:  # ajoute dept actuel de l'utilisateur
+            selectable_dept_acronyms |= {the_user.dept}
+        default_dept = the_user.dept
+    else:
+        default_dept = (
+            g.scodoc_dept
+            if g.scodoc_dept in selectable_dept_acronyms
+            else (auth_dept or "")
+        )
     if is_super_admin and len(selectable_dept_acronyms) > 1:
         selectable_dept_acronyms = sorted(list(selectable_dept_acronyms))
         descr.append(
@@ -472,9 +480,7 @@ def create_user_form(user_name=None, edit=0, all_roles=True):
                     "explanation": """département de rattachement de l'utilisateur""",
                     "labels": selectable_dept_acronyms,
                     "allowed_values": selectable_dept_acronyms,
-                    "default": g.scodoc_dept
-                    if g.scodoc_dept in selectable_dept_acronyms
-                    else (auth_dept or ""),
+                    "default": default_dept,
                 },
             )
         )
diff --git a/sco_version.py b/sco_version.py
index caf26ca9..8a656e71 100644
--- a/sco_version.py
+++ b/sco_version.py
@@ -1,7 +1,7 @@
 # -*- mode: python -*-
 # -*- coding: utf-8 -*-
 
-SCOVERSION = "9.4.65"
+SCOVERSION = "9.4.66"
 
 SCONAME = "ScoDoc"