Merge pull request PnX-SI#179 from PnX-SI/develop

Develop > Master / 2.3.4

VERSION

@@ -1 +1 @@
-2.3.3
+2.3.4

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}.<br>
+                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.<br>
+                """
+        }, 422
 
     req_data = temp_user.as_dict()
     # Récupération du groupe par défaut

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(

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):
         """

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):
     """

app/templates/head-appli.html

@@ -31,7 +31,7 @@
           </ul>
           <ul class="navbar-nav ml-auto">
               <li class="nav-item">
-                <a class="nav-link"  title="Déconnexion" href="{{ url_for('auth.logout') }}?redirect={{config.URL_APPLICATION}}">
+                <a class="nav-link"  title="Déconnexion" href="{{ url_for('auth.logout') }}?redirect={{config.URL_REDIRECT}}">
                   {{user.identifiant}}
                   <i class="fa fa-sign-out font-medium" aria-hidden="true"></i>
                 </a>

app/templates/info_user.html

@@ -3,15 +3,15 @@
 
 {%set is_identifiant = user['identifiant'] is not none and user['identifiant'] != '' %}
 {%set is_uuid = user['uuid_role'] is not none and user['uuid_role'] != '' %}
-{%set is_organisme = organisme.nom_organisme is not none and organisme.nom_organisme != '' %}
+{%set is_organisme = organisme is not none and organisme['nom_organisme'] != '' %}
 {%set is_desc = user['desc_role'] is not none and user['desc_role'] != '' %}
 {%set is_remarques = user['remarques'] is not none and user['remarques'] != '' %}
 {%set is_mail = user['email'] is not none and user['email'] != '' %}
 {%set is_group = groups|length > 0 %}
 {%set is_list = lists|length > 0 %}
 {%set is_right = rights|length > 0 %}
 <div class="container main-zone ng-scope ">
-    <h3>Utilisateur "{{user['nom_role']}} {{user['prenom_role']}}"</h3>
+    <h3>Utilisateur "{{user['fullname']}}"</h3>
     <div class="ml-5">
         <small>
             {% if is_desc %}
@@ -21,7 +21,7 @@ <h3>Utilisateur "{{user['nom_role']}} {{user['prenom_role']}}"</h3>
                 <br /><strong>UUID :</strong> {{user['uuid_role']}}
             {% endif %}
             {% if is_identifiant %}
-                <br /><strong>Identifiant :</strong>{{user['identifiant']}}
+                <br /><strong>Identifiant :</strong> {{user['identifiant']}}
             {% endif %}
             {% if is_mail %}
                 <br /><strong>Email :</strong> <a href="mailto:{{user['email']}}">{{user['email']}}</a>

app/templates/login.html

@@ -45,7 +45,7 @@ <h2>UsersHub</h2>
         dataType:"json",
         success: function(response){
             console.log(response);
-            window.location.href = '{{url_application}}';
+            window.location.href = '{{url_application}}/';
         },
         error: function(error){
             $("#login-error" ).show();

docs/changelog.rst

@@ -2,6 +2,17 @@
 CHANGELOG
 =========
 
+2.3.4 (2023-06-27)
+------------------
+
+**🐛 Corrections**
+
+* Correction de la route de redirection après login ou un logout pour éviter les erreurs de redirection lorsque l'URL de l'application n'est pas située à la racine (#173 / #174 - par @joelclems)
+* Création de compte : affichage d'un message explicite lorsque le token d'un utlisateur temporaire n'est pas trouvé quand un administrateur cherche à le valider (#177 / https://github.com/PnX-SI/GeoNature/issues/862 - par @joelclems)
+* Correction de l'affichage de la fiche d'un utilisateur si celui-ci n'a pas d'organisme (#125 - par @jpm-cbna)
+* Correction de la modification du mot de passe d'un utilisateur qui n'a pas de prénom ou de nom (#107 - par @jpm-cbna)
+* Meilleure portabilité des scripts dans les différents systèmes Unix (#176 - par @MathRdt)
+
 2.3.3 (2023-02-16)
 ------------------
 

install_app.sh

@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/usr/bin/env bash
 
 . config/settings.ini || exit 1
 

install_db.sh

@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/usr/bin/env bash
 # Make sure root cannot run our script
 if [ "$(id -u)" == "0" ]; then
    echo "This script must NOT be run as root" 1>&2