diff --git a/VERSION b/VERSION index 0bee604..3f684d2 100644 --- a/VERSION +++ b/VERSION @@ -1 +1 @@ -2.3.3 +2.3.4 diff --git a/app/api/route_register.py b/app/api/route_register.py index 4a732db..f72e3d1 100644 --- a/app/api/route_register.py +++ b/app/api/route_register.py @@ -122,7 +122,12 @@ def valid_temp_user(): # recherche de l'utilisateur temporaire correspondant au token temp_user = db.session.query(TempUser).filter(token == TempUser.token_role).first() if not temp_user: - return {"msg": "pas d'utilisateur trouvé avec le token user demandé"}, 422 + return { + "msg": f""" + Il n'y a pas d'utilisateur temporaire correspondant au token fourni {token}.
+ Il se peut que la demande de création de compte ai déjà été validée, ou bien que l'adresse de validation soit erronée.
+ """ + }, 422 req_data = temp_user.as_dict() # Récupération du groupe par défaut diff --git a/app/genericRepository.py b/app/genericRepository.py index 947edee..ab59e60 100644 --- a/app/genericRepository.py +++ b/app/genericRepository.py @@ -11,18 +11,20 @@ class GenericRepository(db.Model): @classmethod def get_one(cls, id, as_model=False): - """ Methode qui retourne un dictionnaire d'un élément d'un Model Avec pour paramètres l'id de l'élément Si as_model != False alors au lieu de retourner un dictionnaire on retourne l'object du modèle """ + if not id: + return None + + data = db.session.query(cls).get(id) - if as_model == False: - data = db.session.query(cls).get(id) + if not as_model: return data.as_dict(True) else: - return db.session.query(cls).get(id) + return data @classmethod def get_all( diff --git a/app/models.py b/app/models.py index cf04e52..e867b2f 100644 --- a/app/models.py +++ b/app/models.py @@ -194,12 +194,7 @@ def get_full_name(self): Methode qui concatène le nom et prénom du role retourne un nom complet """ - - if self.prenom_role == None: - full_name = self.nom_role - else: - full_name = self.nom_role + " " + self.prenom_role - return full_name + return ' '.join([(self.nom_role or ''), (self.prenom_role or '')]) def as_dict_full_name(self): """ diff --git a/app/t_roles/route.py b/app/t_roles/route.py index 8dc27dc..4f22d50 100644 --- a/app/t_roles/route.py +++ b/app/t_roles/route.py @@ -219,6 +219,9 @@ def updatepass(id_role=None): """ form = t_rolesforms.UserPass() myuser = TRoles.get_one(id_role) + # Build title + role_fullname= buildUserFullName(myuser) + title=f"Changer le mot de passe de l'utilisateur '{role_fullname}'" if request.method == "POST": if form.validate_on_submit() and form.validate(): @@ -241,11 +244,7 @@ def updatepass(id_role=None): return render_template( "user_pass.html", form=form, - title="Changer le mot de passe de l'utilisateur '" - + myuser["nom_role"] - + " " - + myuser["prenom_role"] - + "'", + title=title, id_role=id_role, ) form_user["id_role"] = id_role @@ -257,11 +256,7 @@ def updatepass(id_role=None): return render_template( "user_pass.html", form=form, - title="Changer le mot de passe de l'utilisateur '" - + myuser["nom_role"] - + " " - + myuser["prenom_role"] - + "'", + title=title, id_role=id_role, ) @@ -300,6 +295,14 @@ def info(id_role): pathU=URL_APPLICATION + "/user/update/", ) +def buildUserFullName(user): + fullname = [] + if user["nom_role"]: + fullname.append(user["nom_role"].upper()) + if user["prenom_role"]: + fullname.append(user["prenom_role"].title()) + return ' '.join(fullname) + def pops(form, with_group=True): """ diff --git a/app/templates/head-appli.html b/app/templates/head-appli.html index 90e498a..1375cf5 100644 --- a/app/templates/head-appli.html +++ b/app/templates/head-appli.html @@ -31,7 +31,7 @@