From b87e938cd5405ddb0c47f5d613116ae0a5491be6 Mon Sep 17 00:00:00 2001
From: Marc Rufener <marc.rufener@gmail.com>
Date: Wed, 19 Jul 2023 16:28:10 +0200
Subject: [PATCH 1/7] fix info_gen client moral creation redirection

---
 .../components/Affaires/InfosGenerales/InfosGenerales.vue | 8 ++++++++
 .../Affaires/InfosGenerales/infosGenerales.html           | 4 ++--
 2 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/front/src/components/Affaires/InfosGenerales/InfosGenerales.vue b/front/src/components/Affaires/InfosGenerales/InfosGenerales.vue
index 81cd8f00..838bf215 100644
--- a/front/src/components/Affaires/InfosGenerales/InfosGenerales.vue
+++ b/front/src/components/Affaires/InfosGenerales/InfosGenerales.vue
@@ -456,6 +456,14 @@ export default {
       });
     },
 
+    /**
+     * open create contact
+     */
+     openCreateContact(client_id) {
+      let routeData = this.$router.resolve({name: "ClientsEdit", params: {id: client_id}});
+      window.open(routeData.href, "_blank");
+    },
+
   },
 
   mounted: function() {
diff --git a/front/src/components/Affaires/InfosGenerales/infosGenerales.html b/front/src/components/Affaires/InfosGenerales/infosGenerales.html
index 53679cc7..e21b5ff3 100644
--- a/front/src/components/Affaires/InfosGenerales/infosGenerales.html
+++ b/front/src/components/Affaires/InfosGenerales/infosGenerales.html
@@ -163,7 +163,7 @@
                                 md-dense>
                                 <label>Personne de contact</label>
                                 <template slot="md-autocomplete-empty" slot-scope="{ term }">
-                                    Le contact "{{ term }}" n'existe pas dans la base. <a @click="openCreateContact(form.client_commande.id)">Le créer !</a>
+                                    Le contact "{{ term }}" n'existe pas dans la base. <a @click="openCreateContact(form.client_commande_id)">Le créer !</a>
                                 </template>
                             </md-autocomplete>
                         </div>
@@ -192,7 +192,7 @@
                                 md-dense>
                                 <label>À l'attention de...</label>
                                 <template slot="md-autocomplete-empty" slot-scope="{ term }">
-                                    Le contact "{{ term }}" n'existe pas dans la base. <a @click="openCreateContact(form.client_envoi.id)">Le créer !</a>
+                                    Le contact "{{ term }}" n'existe pas dans la base. <a @click="openCreateContact(form.client_envoi_id)">Le créer !</a>
                                 </template>
                             </md-autocomplete>
                         </div>

From 856153dfb5385c46c34d53db5a5fbca8c0f40532 Mon Sep 17 00:00:00 2001
From: Marc Rufener <marc.rufener@gmail.com>
Date: Fri, 28 Jul 2023 17:08:42 +0200
Subject: [PATCH 2/7] update client-search for affaire_creation

---
 .../Affaires/NewAffaire/NewAffaire.vue        | 15 +++---
 .../Affaires/NewAffaire/newAffaire.html       |  7 +--
 .../Utils/ClientSearch/ClientSearch.vue       | 47 +++++++++++--------
 3 files changed, 38 insertions(+), 31 deletions(-)

diff --git a/front/src/components/Affaires/NewAffaire/NewAffaire.vue b/front/src/components/Affaires/NewAffaire/NewAffaire.vue
index fcae1836..5bf65e15 100644
--- a/front/src/components/Affaires/NewAffaire/NewAffaire.vue
+++ b/front/src/components/Affaires/NewAffaire/NewAffaire.vue
@@ -907,22 +907,23 @@ export default {
       }
 
       // is facture required?
-      await this.getAffaireTypeRequirements(selectedType.id)
+      await this.getAffaireTypeRequirements(selectedType.id);
+      let client_cadastration_id = Number(process.env.VUE_APP_CLIENT_CADASTRATION_ID);
 
       if (this.form.type.id === this.typesAffaires_conf.cadastration) {
         this.showClientsForm = false;
-        this.form.client_commande_id = process.env.VUE_APP_CLIENT_CADASTRATION_ID;
-        this.form.client_envoi_id = process.env.VUE_APP_CLIENT_CADASTRATION_ID;
+        this.form.client_commande_id = client_cadastration_id;
+        this.form.client_envoi_id = client_cadastration_id;
         this.form.client_envoi_complement = null;
-        this.client_facture_id = process.env.VUE_APP_CLIENT_CADASTRATION_ID;
+        this.client_facture_id = client_cadastration_id;
         this.client_facture_premiere_ligne = null;
         this.form.nom = "Cadastration sur ";
       } else if (this.form.type.id === this.typesAffaires_conf.mpd) {
-        this.form.client_commande_id = process.env.VUE_APP_CLIENT_CADASTRATION_ID;
-        this.form.client_envoi_id = process.env.VUE_APP_CLIENT_CADASTRATION_ID;
+        this.form.client_commande_id = client_cadastration_id;
+        this.form.client_envoi_id = client_cadastration_id;
         this.form.client_envoi_complement = null;
         this.client_facture_premiere_ligne = null;
-        this.client_facture_id = process.env.VUE_APP_CLIENT_CADASTRATION_ID;
+        this.client_facture_id = client_cadastration_id;
         this.client_facture_premiere_ligne = null;
         this.form.nom = "Mise à jour périodique";
       } else {
diff --git a/front/src/components/Affaires/NewAffaire/newAffaire.html b/front/src/components/Affaires/NewAffaire/newAffaire.html
index 9a2fed09..3b1f6f0a 100644
--- a/front/src/components/Affaires/NewAffaire/newAffaire.html
+++ b/front/src/components/Affaires/NewAffaire/newAffaire.html
@@ -234,7 +234,6 @@ <h1 style="flex: 1;">Nouvelle affaire</h1>
                         <ClientSearch id="client_commande"
                             :client_id="form.client_commande_id"
                             @update:client_id="client_id => selectedClient(client_id, 'client_commande')"
-                            :initial_client_id="form.client_commande_id"
                             :permission_createClient="permission.editClientAllowed"
                             :old_clients="false"
                             validation_error_msg="Le client qui a commandé l'affaire doit être renseigné ici"
@@ -264,7 +263,6 @@ <h1 style="flex: 1;">Nouvelle affaire</h1>
                         <ClientSearch id="client_envoi"
                             :client_id="form.client_envoi_id"
                             @update:client_id="client_id => selectedClient(client_id, 'client_envoi')"
-                            :initial_client_id="form.client_envoi_id"
                             :permission_createClient="permission.editClientAllowed"
                             :old_clients="false"
                             validation_error_msg="Le client à qui l'on doit envoyer les documents doit être renseigné ici"
@@ -299,10 +297,9 @@ <h1 style="flex: 1;">Nouvelle affaire</h1>
                         </div>
     
                         <div class="md-layout-item md-small-size-100 md-size-100">
-                            <ClientSearch id="client_envoi"
-                                :client_id="form.client_facture_id"
+                            <ClientSearch id="client_facture"
+                                :client_id="client_facture_id"
                                 @update:client_id="client_id => selectedClient(client_id, 'client_facture')"
-                                :initial_client_id="form.client_facture_id"
                                 :permission_createClient="permission.editClientAllowed"
                                 :old_clients="false"
                                 validation_error_msg="Le client à qui sera adressée la facture de l'affaire doit être renseigné ici"
diff --git a/front/src/components/Utils/ClientSearch/ClientSearch.vue b/front/src/components/Utils/ClientSearch/ClientSearch.vue
index 0538922d..0388b894 100644
--- a/front/src/components/Utils/ClientSearch/ClientSearch.vue
+++ b/front/src/components/Utils/ClientSearch/ClientSearch.vue
@@ -21,6 +21,10 @@ export default {
       type: Number,
       default: null
     },
+    client_id: {
+      type: Number,
+      default: null
+    },
     label: {
       type: String,
       default: 'Client'
@@ -51,7 +55,6 @@ export default {
     return {
       client: null,
       liste_clients: [],
-      searchTerm: null
     };
   },
   
@@ -107,6 +110,11 @@ export default {
 
 
     async getClientById(client_id) {
+      if (client_id === null) {
+        this.client = '';
+        return;
+      }
+
       this.$http.get(
         process.env.VUE_APP_API_URL + process.env.VUE_APP_CLIENT_AGGREGATED_BY_ID_ENDPOINT + '/' + client_id,
         {
@@ -154,24 +162,6 @@ export default {
 
   },
 
-  computed: {
-    value: {
-      get() {
-        return this.client_id;
-      },
-      set(value) {
-        if (value && value.id) {
-          this.client_id = value.id;
-          this.$emit('update:client_id', value.id);
-        }
-        if (value === null || value === '') {
-          this.client_id = value;
-          this.$emit('update:client_id', value);
-        }
-      }
-    }
-  },
-
   mounted: function() {
     if (this.initial_client_id !== null) {
       this.getClientById(this.initial_client_id);
@@ -179,6 +169,25 @@ export default {
 
 
     this.$root.$on('resetSearchClientTerm', () => { document.querySelector('#clientSearchAutocomplete > button').click(); });
+  },
+
+  watch: {
+    client_id: function() {
+      if (this.client_id !== null) {
+        this.getClientById(this.client_id);
+      } else {
+        this.client = '';
+      }
+    },
+    
+    client: function() {
+      if (this.client && this.client.id) {
+        this.$emit('update:client_id', this.client.id);
+      } else {
+        this.client_id = null;
+        this.$emit('update:client_id', null);
+      }
+    }
   }
 };
 

From 8b8c89ebd74dba57543c25f1bc2458fba95ef189 Mon Sep 17 00:00:00 2001
From: Marc Rufener <marc.rufener@gmail.com>
Date: Mon, 14 Aug 2023 10:32:54 +0200
Subject: [PATCH 3/7] fix error message first connexion into the app

---
 front/src/components/Login/Login.vue    | 6 +++---
 front/src/services/exceptionsHandler.js | 8 +-------
 front/src/services/helper.js            | 5 ++++-
 3 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/front/src/components/Login/Login.vue b/front/src/components/Login/Login.vue
index 0f7a1443..01b65a10 100644
--- a/front/src/components/Login/Login.vue
+++ b/front/src/components/Login/Login.vue
@@ -40,7 +40,7 @@ export default {
             if (process.env.VUE_APP_SERVICE_MO === response.data.service) {
               this.$router.push(this.$route.query.redirect || { name: "Cockpit"});
             } else {
-                this.$router.push(this.$route.query.redirect || { name: "Preavis"});
+              this.$router.push(this.$route.query.redirect || { name: "Preavis"});
             }
           } else {
             this.showProgess = false;
@@ -48,9 +48,9 @@ export default {
           }
         })
         //Error 
-        .catch(err => {
+        .catch(() => {
           this.showProgess = false;
-          handleException(err, this);
+          this.$root.$emit('ShowError', "Le nom d'utilisateur ou le mot de passe est incorrect")
         });
       },
       
diff --git a/front/src/services/exceptionsHandler.js b/front/src/services/exceptionsHandler.js
index 220fe08a..87f943b0 100644
--- a/front/src/services/exceptionsHandler.js
+++ b/front/src/services/exceptionsHandler.js
@@ -12,12 +12,7 @@ export const handleException = function (error, component) {
     }
     //Not authorized
     else if(code === 403){
-        if(component.$router && component.$router.currentRoute && component.$router.currentRoute.name != 'Login') {
-            component.$router.push({name: "Login", query: { redirect: component.$router.currentRoute.path }});
-            component.$root.$emit("ShowError", "Veuillez vous connecter pour continuer"); 
-        } else {
-            component.$root.$emit("ShowError", "Le nom d'utilisateur ou le mot de passe est incorrect");
-        }
+       component.$root.$emit("ShowError", "Veuillez vous connecter pour continuer"); 
     }
     //Custom error
     else if(error && error.msg){
@@ -38,4 +33,3 @@ export const handleException = function (error, component) {
        component.$root.$emit("ShowError", "Une erreur est survenue, contacter l'administrateur.");   
     }
 };
-
diff --git a/front/src/services/helper.js b/front/src/services/helper.js
index 26376d67..830d69db 100644
--- a/front/src/services/helper.js
+++ b/front/src/services/helper.js
@@ -8,8 +8,11 @@ export const checkLogged = function () {
     var session_user = JSON.parse(localStorage.getItem('infolica_user')) || null;
     
     //Set current user functions
-    if(session_user)
+    if(session_user){
         setCurrentUserFunctions();
+    } else {
+        alert("Veuillez vous connecter pour continuer")
+    }
     
     return session_user !== null;
 };

From e3dff2699c08aadaba5c10c847be0b649e7e388d Mon Sep 17 00:00:00 2001
From: Marc Rufener <marc.rufener@gmail.com>
Date: Wed, 16 Aug 2023 09:10:01 +0200
Subject: [PATCH 4/7] fix error not-responding "click all"

---
 .../components/Affaires/SuiviMandat/SuiviMandat.vue  |  2 +-
 .../components/Affaires/SuiviMandat/suiviMandat.html | 12 ++++++------
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/front/src/components/Affaires/SuiviMandat/SuiviMandat.vue b/front/src/components/Affaires/SuiviMandat/SuiviMandat.vue
index 7794fdcb..486b4b2e 100644
--- a/front/src/components/Affaires/SuiviMandat/SuiviMandat.vue
+++ b/front/src/components/Affaires/SuiviMandat/SuiviMandat.vue
@@ -175,7 +175,7 @@ export default {
           for (const elem of fields[x].children) {
             if (elem.className === 'md-checkbox-container') {
               elem.click();
-              break
+              break;
             }
           }
         }
diff --git a/front/src/components/Affaires/SuiviMandat/suiviMandat.html b/front/src/components/Affaires/SuiviMandat/suiviMandat.html
index 55be1f2d..08374839 100644
--- a/front/src/components/Affaires/SuiviMandat/suiviMandat.html
+++ b/front/src/components/Affaires/SuiviMandat/suiviMandat.html
@@ -360,8 +360,8 @@
                                             <div class="md-layout md-gutter">
                                                 <div class="md-layout-item md-size-30">
                                                     <div class="md-subhead">Général</div>
-                                                    <md-button style="margin-left: 30px; background-color: lightgrey;" class="md-raised md-dense" @click="selectAll('gen_')">Tout cocher</md-button>
-                                                    <div id="gen_">
+                                                    <md-button style="margin-left: 30px; background-color: lightgrey;" class="md-raised md-dense" @click="selectAll('ctrl_coord-mpd-gen_')">Tout cocher</md-button>
+                                                    <div id="ctrl_coord-mpd-gen_">
                                                         <md-checkbox class="md-primary" v-model="suiviMandat.ap_21">Répertoire de travail (bon endroit, bonne structure)</md-checkbox>
                                                         <md-checkbox class="md-primary" v-model="suiviMandat.gen_pdf_travaux">Fichier PDF après MPD figurant les travaux en couleur</md-checkbox>
                                                         <md-checkbox class="md-primary" v-model="suiviMandat.gen_scan_croquis">Scan du croquis terrain</md-checkbox>
@@ -372,8 +372,8 @@
           
                                                 <div class="md-layout-item md-size-30">
                                                     <div class="md-subhead">Base de données</div>
-                                                    <md-button style="margin-left: 30px; background-color: lightgrey;" class="md-raised md-dense" @click="selectAll('geos_')">Tout cocher</md-button>
-                                                    <div id="geos_">
+                                                    <md-button style="margin-left: 30px; background-color: lightgrey;" class="md-raised md-dense" @click="selectAll('ctrl_coord-mpd-geos_')">Tout cocher</md-button>
+                                                    <div id="ctrl_coord-mpd-geos_">
                                                         <md-checkbox class="md-primary" v-model="suiviMandat.geos_couleur_pastilles">Toutes les pastilles sont vertes ou grises</md-checkbox>
                                                         <md-checkbox class="md-primary" v-model="suiviMandat.geos_couvsol_numerotation_pt_particuliers">Réservation des numéros de points particuliers (660)</md-checkbox>
                                                         <md-checkbox class="md-primary" v-model="suiviMandat.geos_couvsol_numerotation_bat">Réservation des numéros de bâtiments</md-checkbox>
@@ -385,8 +385,8 @@
                                                 
                                                 <div class="md-layout-item md-size-30">
                                                     <div class="md-subhead">Normes</div>
-                                                    <md-button style="margin-left: 30px; background-color: lightgrey;" class="md-raised md-dense" @click="selectAll('normes_')">Tout cocher</md-button>
-                                                    <div id="normes_">
+                                                    <md-button style="margin-left: 30px; background-color: lightgrey;" class="md-raised md-dense" @click="selectAll('ctrl_coord-mpd-normes_')">Tout cocher</md-button>
+                                                    <div id="ctrl_coord-mpd-normes_">
                                                         <md-checkbox class="md-primary" v-model="suiviMandat.normes_respect_petits_objects">Les objets du type : balcons, petits bâtiments, petites surfaces de nature respectent les normes (ne pas en saisir trop)</md-checkbox>
                                                         <md-checkbox class="md-primary" v-model="suiviMandat.normes_bat_couv_digitalises">Pas de bâtiments ni de couverts digitalisés</md-checkbox>
                                                     </div>

From 8a7c01125771ef115d10d71a52056a7dc398f284 Mon Sep 17 00:00:00 2001
From: Marc Rufener <marc.rufener@gmail.com>
Date: Wed, 16 Aug 2023 10:44:18 +0200
Subject: [PATCH 5/7] update config

---
 back/development.ini.template  | 3 ++-
 back/infolica/scripts/utils.py | 4 ++--
 back/production.ini.template   | 3 ++-
 3 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/back/development.ini.template b/back/development.ini.template
index 5eddcc65..3fca472b 100644
--- a/back/development.ini.template
+++ b/back/development.ini.template
@@ -78,11 +78,12 @@ affaire_type_cadastration_id = 2
 affaire_type_ppe_id = 3
 affaire_type_revision_abornement_id = 4
 affaire_type_mpd_id = 8
-affaire_remaniement_parcellaire_id = 9
+affaire_type_remaniement_parcellaire_id = 9
 affaire_type_retablissement_pfp3_id = 12
 affaire_type_autre_id = 7
 affaire_type_part_copropriete_id = 10
 affaire_type_modification_ppe_id = 20
+affaire_type_retour_etat_juridique_id = 21
 affaire_type_modification_ordre_separation = 100
 
 # ID Numéros-types biens-fonds
diff --git a/back/infolica/scripts/utils.py b/back/infolica/scripts/utils.py
index 2c14adbe..5e33e280 100644
--- a/back/infolica/scripts/utils.py
+++ b/back/infolica/scripts/utils.py
@@ -414,8 +414,8 @@ def affaireUpdatePermission(cls, request, affaire_type):
         # Affaire autre
         elif affaire_type == request.registry.settings['affaire_type_autre_id']:
             permission = request.registry.settings['affaire_autre_edition']
-        # Affaire affaire_remaniement_parcellaire_id
-        elif affaire_type == request.registry.settings['affaire_remaniement_parcellaire_id']:
+        # Affaire affaire_type_remaniement_parcellaire_id
+        elif affaire_type == request.registry.settings['affaire_type_remaniement_parcellaire_id']:
             permission = request.registry.settings['affaire_remaniement_parcellaire_edition']
 
         return permission
diff --git a/back/production.ini.template b/back/production.ini.template
index be175ddf..919287ca 100644
--- a/back/production.ini.template
+++ b/back/production.ini.template
@@ -77,11 +77,12 @@ affaire_type_cadastration_id = 2
 affaire_type_ppe_id = 3
 affaire_type_revision_abornement_id = 4
 affaire_type_mpd_id = 8
-affaire_remaniement_parcellaire_id = 9
+affaire_type_remaniement_parcellaire_id = 9
 affaire_type_retablissement_pfp3_id = 12
 affaire_type_autre_id = 7
 affaire_type_part_copropriete_id = 10
 affaire_type_modification_ppe_id = 20
+affaire_type_retour_etat_juridique_id = 21
 affaire_type_modification_ordre_separation = 100
 
 # ID Numéros-types biens-fonds

From f14d581212276b165579be63f33e5ee1b17582ff Mon Sep 17 00:00:00 2001
From: Marc Rufener <marc.rufener@gmail.com>
Date: Fri, 18 Aug 2023 08:16:24 +0200
Subject: [PATCH 6/7] fix bug that makes the cockpit crash if no affaire_nom is
 given

---
 back/infolica/views/affaire.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/back/infolica/views/affaire.py b/back/infolica/views/affaire.py
index a930907b..0847bbf9 100644
--- a/back/infolica/views/affaire.py
+++ b/back/infolica/views/affaire.py
@@ -135,7 +135,7 @@ def affaire_cockpit_view(request):
         etape_datetime = datetime.strftime(affaire.etape_datetime, '%Y-%m-%d %H:%M:%S')
         etape_days_elapsed = (datetime.now().date() - affaire.etape_datetime.date()).days
         etape_days_elapsed_text = "aujourd'hui" if etape_days_elapsed == 0 else ("hier" if etape_days_elapsed == 1 else str(etape_days_elapsed) + " jours")
-        title = affaire.technicien_initiales + " — Affaire " + str(affaire.id) + " — " + affaire.cadastre + " — " + affaire.nom + " — Dans cette étape depuis " + etape_days_elapsed_text
+        title = affaire.technicien_initiales + " — Affaire " + str(affaire.id) + " — " + affaire.cadastre + " — " + (affaire.nom if affaire.nom is not None else "~ Aucune description ~") + " — Dans cette étape depuis " + etape_days_elapsed_text
         
         nb_preavis = request.dbsession.query(func.count(Preavis.affaire_id)).filter(Preavis.affaire_id == affaire.id).scalar()
         nb_closed_preavis = request.dbsession.query(func.count(Preavis.affaire_id)).filter(Preavis.affaire_id == affaire.id, Preavis.date_reponse != None).scalar()

From 35e8f012cb69bdf221d3f4c5f7abf0a81ba5c5f2 Mon Sep 17 00:00:00 2001
From: Marc Rufener <marc.rufener@gmail.com>
Date: Fri, 18 Aug 2023 09:30:15 +0200
Subject: [PATCH 7/7] fix redirection that was not working anymore

---
 front/src/services/exceptionsHandler.js | 3 ++-
 front/src/services/helper.js            | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/front/src/services/exceptionsHandler.js b/front/src/services/exceptionsHandler.js
index 87f943b0..6e06e797 100644
--- a/front/src/services/exceptionsHandler.js
+++ b/front/src/services/exceptionsHandler.js
@@ -12,7 +12,8 @@ export const handleException = function (error, component) {
     }
     //Not authorized
     else if(code === 403){
-       component.$root.$emit("ShowError", "Veuillez vous connecter pour continuer"); 
+        let a = component.$router.push({name: "Login", query: { redirect: component.$router.currentRoute.path }});
+        a.then(() => { component.$root.$emit("ShowError", "Veuillez vous connecter pour continuer") });
     }
     //Custom error
     else if(error && error.msg){
diff --git a/front/src/services/helper.js b/front/src/services/helper.js
index 830d69db..4498c17b 100644
--- a/front/src/services/helper.js
+++ b/front/src/services/helper.js
@@ -11,7 +11,7 @@ export const checkLogged = function () {
     if(session_user){
         setCurrentUserFunctions();
     } else {
-        alert("Veuillez vous connecter pour continuer")
+        alert("Veuillez vous connecter pour continuer");
     }
     
     return session_user !== null;