complements_import_users_from_9.0.52

This commit is contained in:
Jean-Marie Place 2021-10-13 15:03:41 +02:00
parent dc26d1edea
commit 071c15af79
3 changed files with 10 additions and 7 deletions

View File

@ -179,11 +179,12 @@ def import_users(users):
user_ok, msg = sco_users.check_modif_user( user_ok, msg = sco_users.check_modif_user(
0, 0,
ignore_optionals=False, ignore_optionals=False,
ajout_en_masse=True,
user_name=u["user_name"], user_name=u["user_name"],
nom=u["nom"], nom=u["nom"],
prenom=u["prenom"], prenom=u["prenom"],
email=u["email"], email=u["email"],
roles=u["roles"].split(","), roles=[r for r in u["roles"].split(",") if r],
dept=u["dept"], dept=u["dept"],
) )
if not user_ok: if not user_ok:

View File

@ -388,6 +388,7 @@ def user_info_page(user_name=None):
def check_modif_user( def check_modif_user(
edit, edit,
ignore_optionals=False, ignore_optionals=False,
ajout_en_masse=False,
user_name="", user_name="",
nom="", nom="",
prenom="", prenom="",
@ -402,9 +403,10 @@ def check_modif_user(
(si ok est faux, l'utilisateur peut quand même forcer la creation) (si ok est faux, l'utilisateur peut quand même forcer la creation)
- msg: message warning à presenter à l'utilisateur - msg: message warning à presenter à l'utilisateur
""" """
MSG_OPT = """ if ajout_en_masse:
(vous pouvez forcer l'opération en cochant "<em>Ignorer les avertissements</em>" en bas de page) MSG_OPT = """<br/>Impossible de forcer l'opération lors d'une importation en masse"""
""" else:
MSG_OPT = """<br/>Attention: (vous pouvez forcer l'opération en cochant "<em>Ignorer les avertissements</em>" en bas de page)"""
# ce login existe ? # ce login existe ?
user = _user_list(user_name) user = _user_list(user_name)
if edit and not user: # safety net, le user_name ne devrait pas changer if edit and not user: # safety net, le user_name ne devrait pas changer
@ -438,7 +440,7 @@ def check_modif_user(
and dept != "" and dept != ""
and Departement.query.filter_by(acronym=dept).first() is None and Departement.query.filter_by(acronym=dept).first() is None
): ):
return False, "département '%s' inexistant" % u["dept"] + MSG_OPT return False, "département '%s' inexistant" % dept + MSG_OPT
if ignore_optionals and not roles: if ignore_optionals and not roles:
return False, "aucun rôle sélectionné, êtes vous sûr ?" + MSG_OPT return False, "aucun rôle sélectionné, êtes vous sûr ?" + MSG_OPT
# ok # ok

View File

@ -380,14 +380,14 @@ def create_user_form(user_name=None, edit=0, all_roles=1):
if err: if err:
H.append(tf_error_message("""Erreur: %s""" % err)) H.append(tf_error_message("""Erreur: %s""" % err))
return "\n".join(H) + "\n" + tf[1] + F return "\n".join(H) + "\n" + tf[1] + F
ok, msg = sco_users.check_modif_user( ok, msg = sco_users.check_modif_user(
edit, edit,
ignore_optionals=force, ignore_optionals=not force,
user_name=user_name, user_name=user_name,
nom=vals["nom"], nom=vals["nom"],
prenom=vals["prenom"], prenom=vals["prenom"],
email=vals["email"], email=vals["email"],
dept=vals["dept"],
roles=vals["roles"], roles=vals["roles"],
) )
if not ok: if not ok: