Merge 1.4.7

PnX-SI · Jul 22, 2021 · c1be175 · c1be175
2 parents 0d4eb3d + 2d21e87
commit c1be175
Show file tree

Hide file tree

Showing 5 changed files with 44 additions and 25 deletions.
diff --git a/VERSION b/VERSION
@@ -1 +1 @@
-1.4.6
+1.4.7
diff --git a/docs/changelog.rst b/docs/changelog.rst
@@ -1,7 +1,19 @@
 CHANGELOG
 *********
 
-1.4.6 (2020-06-03)
+1.4.7 (2021-07-22)
+------------------
+
+**🚀 Nouveautés**
+
+* Amélioration des messages et e-mails utilisateurs
+
+**🐛 Corrections**
+
+* Corrections de 2 bugs mineurs
+
+
+1.4.6 (2021-06-03)
 ------------------
 
 **🚀 Nouveautés**
@@ -17,7 +29,7 @@ CHANGELOG
 * Correction du nom d’un n° de séquence
 
 
-1.4.5 (2020-02-24)
+1.4.5 (2021-02-24)
 ------------------
 
 **🚀 Nouveautés**

diff --git a/src/pypnusershub/db/models.py b/src/pypnusershub/db/models.py
@@ -155,8 +155,10 @@ def __str__(self):
         return self.identifiant or ''
 
     def as_dict(self, data):
-        data["nom_role"] = data["nom_role"] or ""
-        data["prenom_role"] = data["prenom_role"] or ""
+        if 'nom_role' in data:
+            data["nom_role"] = data["nom_role"] or ""
+        if 'prenom_role' in data:
+            data["prenom_role"] = data["prenom_role"] or ""
         return data
 
 

diff --git a/src/pypnusershub/db/models_register.py b/src/pypnusershub/db/models_register.py
@@ -9,9 +9,6 @@
 from .models import User, db as DB
 
 
-config = current_app.config
-
-
 class TempUser(DB.Model):
     __tablename__ = "temp_users"
     __table_args__ = {"schema": "utilisateurs", "extend_existing": True}
@@ -61,10 +58,15 @@ def is_valid(self):
         if role:
             is_valid = False
             if role.email == self.email:
-                msg += f"Un compte avec l'email {self.email} existe déjà. "
+                msg += (
+                    f"Un compte avec l'email {self.email} existe déjà. " +
+                    "S'il s'agit de votre email, vous pouvez faire une demande de renouvellement " +
+                    "de mot de passe via la page de login de GeoNature."
+                )
             else:
                 msg += (
-                    f"Un compte avec l'identifiant {self.identifiant} existe déjà. "
+                    f"Un compte avec l'identifiant {self.identifiant} existe déjà. " +
+                    "Veuillez choisir un identifiant différent."
                 )
 
         temp_role = (
@@ -75,10 +77,16 @@ def is_valid(self):
         if temp_role:
             is_valid = False
             if temp_role.email == self.email:
-                msg += f"Un compte avec l'email {self.email} existe déjà. "
+                msg += (
+                    f"Un compte en attente de validation avec l'email {self.email} existe déjà. "+
+                    "Merci de patienter le temps que votre demande soit traitée."
+
+                )
             else:
                 msg += (
-                    f"Un compte avec l'identifiant {self.identifiant} existe déjà. "
+                    "Un compte en attente de validation avec l'identifiant " +
+                    f"{self.identifiant} existe déjà. " +
+                    "Veuillez choisir un identifiant différent."
                 )
 
         return (is_valid, msg)
@@ -104,8 +112,7 @@ def as_dict(self, recursif=False, columns=(), depth=None):
             "email": self.email,
             "id_organisme": self.id_organisme,
             "remarques": self.remarques,
-            "champs_addi": self.champs_addi
-
+            "champs_addi": self.champs_addi,
         }
 
 

diff --git a/src/pypnusershub/routes_register.py b/src/pypnusershub/routes_register.py
@@ -34,8 +34,6 @@
 # from .routes import check_auth
 
 
-config = current_app.config
-
 s = requests.Session()
 
 bp = Blueprint('register', __name__)
@@ -103,7 +101,7 @@ def __connect_admin(*args, **kwargs):
 
             # test si on est déjà connecté
             try:
-                r = s.post(config['URL_USERSHUB'] +
+                r = s.post(current_app.config['URL_USERSHUB'] +
                            "/api_register/test_connexion")
                 b_connexion = (r.status_code == 200)
             except requests.ConnectionError:
@@ -113,10 +111,10 @@ def __connect_admin(*args, **kwargs):
             if not b_connexion:
                 # connexion à usershub
                 r = s.post(
-                    config['URL_USERSHUB'] + "/" + "pypn/auth/login",
+                    current_app.config['URL_USERSHUB'] + "/" + "pypn/auth/login",
                     json={
-                        'login': config['ADMIN_APPLICATION_LOGIN'],
-                        'password': config['ADMIN_APPLICATION_PASSWORD'],
+                        'login': current_app.config['ADMIN_APPLICATION_LOGIN'],
+                        'password': current_app.config['ADMIN_APPLICATION_PASSWORD'],
                         'id_application': id_app_usershub
                     }
                 )
@@ -142,7 +140,7 @@ def test():
             - config['ADMIN_APPLICATION_PASSWORD']
     '''
 
-    r = s.post(config['URL_USERSHUB'] + "/api_register/test_connexion")
+    r = s.post(current_app.config['URL_USERSHUB'] + "/api_register/test_connexion")
 
     return req_json_or_text(r, "Test pypn")
 
@@ -176,7 +174,7 @@ def post_usershub(type_action):
 
         q = (db.session.query(AppUser.id_droit_max)
              .filter(AppUser.id_role == id_role)
-             .filter(AppUser.id_application == config['ID_APP']))
+             .filter(AppUser.id_application == current_app.config['ID_APP']))
         id_droit = q.one()[0]
 
     # si pas de droit definis pour cet action, alors les droits requis sont à 7 => action impossible
@@ -186,7 +184,7 @@ def post_usershub(type_action):
 
     # les test de paramètres seront faits dans UsersHub
     data = request.get_json()
-    url = config['URL_USERSHUB'] + "/" + "api_register/" + type_action
+    url = current_app.config['URL_USERSHUB'] + "/" + "api_register/" + type_action
     r_usershub = s.post(url, json=data)
 
     # after request definir route dans app
@@ -195,7 +193,7 @@ def post_usershub(type_action):
     if r_usershub.status_code == 200 and data.get('enable_post_action', True):
         out_after = after_request(type_action, get_json_request(r_usershub))
 
-        # 0 = pas d'action definie dans config['after_USERSHUB_request'][type_action]
+        # 0 = pas d'action definie dans current_app.config['after_USERSHUB_request'][type_action]
         if out_after != 0:
 
             if out_after['msg'] != "ok":
@@ -210,7 +208,7 @@ def after_request(type_action, data, *args, **kwargs):
         lorsqu'une fonction est definie dans config['after_USERSHUB_request'][type_action]
         elle est executée avec les données fournies en retour de la requete USERSHUB
     '''
-    after_request_dict = config.get('after_USERSHUB_request', None)
+    after_request_dict = current_app.config.get('after_USERSHUB_request', None)
 
     if not after_request_dict:
         return 0