Merge pull request #723 from rufener/bugfix

Bug fix
sitn · Mar 10, 2023 · 5461aff · 5461aff
2 parents a5c6c04 + 65c6cb3
commit 5461aff
Show file tree

Hide file tree

Showing 10 changed files with 66 additions and 94 deletions.
diff --git a/back/infolica/views/numero.py b/back/infolica/views/numero.py
@@ -126,7 +126,7 @@ def numeros_search_view(request):
     # filter by conditions
     query = request.dbsession.query(VNumeros).order_by(
         VNumeros.cadastre,
-        VNumeros.numero.desc()
+        VNumeros.numero.asc()
     ).filter(*conditions)
 
     query = query.filter(VNumeros.type_numero_id <= 4)

diff --git a/front/.env b/front/.env
@@ -5,6 +5,31 @@ PUBLIC_PATH = "{PUBLIC_PATH}"
 VUE_APP_STATUS = "development"
 VUE_APP_VERSION = "1.0"
 
+
+#==================================
+#========    MAP CONFIG    ========
+#==================================
+#Map infos
+VUE_APP_MAP_PROJECTION = "EPSG:2056"
+VUE_APP_MAP_DEFAULT_CENTER = {"x": 2555484, "y": 1204017}
+VUE_APP_MAP_DEFAULT_ZOOM = 4
+VUE_APP_MAP_DEFAULT_AFFAIRE_ZOOM = 13
+VUE_APP_WMS_URL = "https://sitn.ne.ch/mapproxy95/service"
+VUE_APP_WMS_LAYERS = "plan_cadastral_couleur"
+VUE_APP_WMTS_LAYER = "plan_cadastral_couleur"
+VUE_APP_WMTS_GETCAPABILITIES_URL = "https://sitn.ne.ch/web_getcapabilities/WMTSGetCapabilities95.xml"
+
+
+#SITN search service
+VUE_APP_SITN_SEARCH_SERVICE_URL = "https://sitn.ne.ch/search?limit=20&query="
+VUE_APP_SITN_SEARCH_CATEGORIES_ALIASES = {"ImmeublesCanton":"Biens-fonds","adresses_sitn":"Adresses","localite":"Localité"}
+
+
+#SITN couches environnement et aménagement du territoire
+VUE_APP_SITN_AMENAGEMENT_TERRITOIRE_URL = overwriteme
+VUE_APP_SITN_CADASTRE_URL = overwriteme
+VUE_APP_SITN_SITES_POLLUES_URL = overwriteme
+
 #=====================================
 #========    URL ENDPOINTS    ========
 #=====================================
@@ -201,31 +226,6 @@ VUE_APP_DATEFORMAT_CLIENT = "DD.MM.YYYY"
 VUE_APP_DATETIMEFORMAT_WS = "YYYY-MM-DD HH:mm:ss"
 
 
-#==================================
-#========    MAP CONFIG    ========
-#==================================
-#Map infos
-VUE_APP_MAP_PROJECTION = "EPSG:2056"
-VUE_APP_MAP_DEFAULT_CENTER = {"x": 2555484, "y": 1204017}
-VUE_APP_MAP_DEFAULT_ZOOM = 4
-VUE_APP_MAP_DEFAULT_AFFAIRE_ZOOM = 13
-VUE_APP_WMS_URL = "https://sitn.ne.ch/mapproxy95/service"
-VUE_APP_WMS_LAYERS = "plan_cadastral_couleur"
-VUE_APP_WMTS_LAYER = "plan_cadastral_couleur"
-VUE_APP_WMTS_GETCAPABILITIES_URL = "https://sitn.ne.ch/web_getcapabilities/WMTSGetCapabilities95.xml"
-
-
-#SITN search service
-VUE_APP_SITN_SEARCH_SERVICE_URL = "https://sitn.ne.ch/search?limit=20&query="
-VUE_APP_SITN_SEARCH_CATEGORIES_ALIASES = {"ImmeublesCanton":"Biens-fonds","adresses_sitn":"Adresses","localite":"Localité"}
-
-
-#SITN couches environnement et aménagement du territoire
-VUE_APP_SITN_AMENAGEMENT_TERRITOIRE_URL = "https://sitnintra.ne.ch/theme/amenagement_territoire?map_zoom=11&map_crosshair=true"
-VUE_APP_SITN_CADASTRE_URL = "https://sitnintra.ne.ch/theme/cadastre"
-VUE_APP_SITN_SITES_POLLUES_URL = "https://sitnintra.ne.ch/theme/canepo"
-
-
 #======================================
 #========    VALEURS CONFIG    ========
 #======================================

diff --git a/front/src/components/Affaires/ActivationAffaire/activationAffaire.html b/front/src/components/Affaires/ActivationAffaire/activationAffaire.html
@@ -3,6 +3,7 @@
     <md-dialog-title>Réactivation de l'affaire</md-dialog-title>
 
     <md-dialog-content>
+      <md-progress-bar md-mode="indeterminate" v-if="showProgressBar"></md-progress-bar>
       <p>
         En confirmant, les opérations suivantes seront réalisées:
         <ul>
@@ -18,7 +19,6 @@
 
 
     <md-dialog-actions>
-      <md-progress-bar md-mode="indeterminate" v-if="showProgressBar"></md-progress-bar>
       <md-button class="md-accent" @click="showActivationDialog=false">Annuler</md-button>
       <md-button class="md-primary" @click="activateAffaire" :disabled="disabledConfirmBtn">Confirmer</md-button>
     </md-dialog-actions>

diff --git a/front/src/components/Affaires/Etape/Etape.vue b/front/src/components/Affaires/Etape/Etape.vue
@@ -1,14 +1,15 @@
-<style src="./etape.css" scoped></style>
+<style src="./etape.css" ></style>
 <style lang="css">.md-menu-content { z-index: 9000 !important; }</style>
 <template src="./etape.html"></template>
 
 
 <script>
 import ClotureAffaire from "@/components/Affaires/ClotureAffaire/ClotureAffaire.vue";
 import DateRangePicker from "@/components/Utils/DateRangePicker/DateRangePicker.vue";
+import NewStepSetter from "@/components/Utils/NewStepSetter/NewStepSetter.vue";
 
 import { handleException } from "@/services/exceptionsHandler";
-import { getTypesAffaires, stringifyAutocomplete2, logAffaireEtape, checkPermission } from '@/services/helper';
+import { logAffaireEtape, checkPermission } from '@/services/helper';
 
 
 const moment = require('moment');
@@ -17,7 +18,8 @@ export default {
   name: "Etape",
   components: {
     ClotureAffaire,
-    DateRangePicker
+    DateRangePicker,
+    NewStepSetter
   },
   props: {
     affaire: Object,
@@ -34,15 +36,14 @@ export default {
     dateperiod_start: null,
     etapeAffaire: {
       nb_jours_etape: 0,
-      prochaine: null,
+      prochaine_id: null,
       remarque: null,
       showDialog: false,
     },
     final_decision: false,
     isAdmin: false,
     numerosReserves: [],
     periodClientStatus: false,
-    suiviAffaireTheorique: [],
     updateAffaireDate: {
       text: "",
       value: false,
@@ -58,39 +59,12 @@ export default {
   }),
 
   methods: {
-    /**
-     * Search affaire etapes
-     */
-    async searchAffaireEtapes() {
-      this.$http.get(
-        process.env.VUE_APP_API_URL + process.env.VUE_APP_ETAPES_INDEX_ENDPOINT,
-        {
-          withCredentials: true,
-          headers: { Accept: "application/json" }
-        }
-      ).then(response => {
-        this.affaireEtapes = stringifyAutocomplete2(response.data.filter(x => x.ordre !== null));
-        // get suivi d'affaire théorique
-        this.typesAffaires = getTypesAffaires().then(response => {
-          if (response && response.data) {
-            try {
-              this.suiviAffaireTheorique = response.data.filter(x => x.id === this.affaire.type_id)[0].logique_processus;
-            }
-            catch {
-              this.suiviAffaireTheorique = [];
-            }
-
-          }
-        })
-      }).catch(err => handleException(err, this));
-    },
-
     /**
      * open New state dialog
      */
     async openNewStateDialog(){
       // set next step prediction
-      this.etapeAffaire.prochaine = null;
+      this.etapeAffaire.prochaine_id = null;
       this.etapeAffaire.chef_equipe_id = this.affaire.technicien_id || null;
       this.etapeAffaire.remarque = null;
 
@@ -99,13 +73,10 @@ export default {
       
       this.etapeAffaire.nb_jours_etape = Math.floor((now_datetime - etape_datetime)/3600000/24) + 1;
       
-      if (this.suiviAffaireTheorique.includes(this.affaire.etape_id)) {
-        this.etapeAffaire.prochaine = this.affaireEtapes.filter(x => x.id === this.suiviAffaireTheorique[this.suiviAffaireTheorique.indexOf(this.affaire.etape_id)+1])[0];
-      }
 
       // if step "chez le client" next step is "operateur_travail"
       if (this.affaire.etape_id === this.etapes_affaire_conf.chez_client) {
-        this.etapeAffaire.prochaine = this.affaireEtapes.filter(x => x.id === this.etapes_affaire_conf.travaux_chef_equipe)[0];
+        this.etapeAffaire.prochaine_id = this.etapes_affaire_conf.travaux_chef_equipe;
       } 
       
       const etapes_jours_clients = [
@@ -161,7 +132,7 @@ export default {
         this.joursHorsSGRF.date_to = null;
       }
 
-      if (!this.etapeAffaire.prochaine || !this.etapeAffaire.prochaine.id) {
+      if (!this.etapeAffaire.prochaine_id) {
         alert("Il faut renseigner le champ 'prochaine étape'.")
         return
       }
@@ -185,9 +156,9 @@ export default {
       }
 
       // fix value of this.etapeAffaire.chef_equipe_id to null if another step is selected
-      this.etapeAffaire.chef_equipe_id = this.etapeAffaire.prochaine.id && this.etapeAffaire.prochaine.id === this.etapes_affaire_conf.travaux_chef_equipe? this.etapeAffaire.chef_equipe_id: null;
+      this.etapeAffaire.chef_equipe_id = this.etapeAffaire.prochaine_id === this.etapes_affaire_conf.travaux_chef_equipe? this.etapeAffaire.chef_equipe_id: null;
       
-      logAffaireEtape(this.affaire.id, this.etapeAffaire.prochaine.id, this.etapeAffaire.remarque, this.etapeAffaire.chef_equipe_id, this.joursHorsSGRF.date_from, this.joursHorsSGRF.date_to)
+      logAffaireEtape(this.affaire.id, this.etapeAffaire.prochaine_id, this.etapeAffaire.remarque, this.etapeAffaire.chef_equipe_id, this.joursHorsSGRF.date_from, this.joursHorsSGRF.date_to)
       .then(() => {
         this.$root.$emit("ShowMessage", "L'étape a bien été mise à jour");
         this.etapeAffaire.showDialog = false;
@@ -280,7 +251,7 @@ export default {
         }
 
         // Si aucun numéro n'est réservé dans l'affaire, clôre l'affaire
-        if ((this.etapeAffaire.prochaine && this.etapeAffaire.prochaine.id && this.etapeAffaire.prochaine.id === this.etapes_affaire_conf.fin_processus) &&
+        if ((this.etapeAffaire.prochaine_id === this.etapes_affaire_conf.fin_processus) &&
             (![this.typesAffaires_conf.mutation, this.typesAffaires_conf.modification, this.typesAffaires_conf.remaniement_parcellaire,
                this.typesAffaires_conf.modification_visa, this.typesAffaires_conf.modification_mutation].includes(this.affaire.type_id)) &&
             (this.numerosReserves.length === 0)) {
@@ -306,9 +277,9 @@ export default {
       this.updateAffaireDate.show = (
         [this.etapes_affaire_conf.envoi, this.etapes_affaire_conf.validation].includes(this.affaire.etape_id) && this.affaire.type_id !== this.typesAffaires_conf.pcop) ||
         (this.etapes_affaire_conf.envoi_pcop === this.affaire.etape_id && this.affaire.type_id === this.typesAffaires_conf.pcop) || 
-        this.etapeAffaire.prochaine && this.etapeAffaire.prochaine.id === this.etapes_affaire_conf.fin_processus && this.affaire.type_id === this.typesAffaires_conf.cadastration;
+        this.etapeAffaire.prochaine_id === this.etapes_affaire_conf.fin_processus && this.affaire.type_id === this.typesAffaires_conf.cadastration;
       
-      if (this.etapeAffaire.prochaine && this.etapeAffaire.prochaine.id) {
+      if (this.etapeAffaire.prochaine_id) {
         this.setNewEtapeParameters();
       }
     },
@@ -325,12 +296,12 @@ export default {
       };
 
       if (
-        (this.etapeAffaire.prochaine.id && this.etapeAffaire.prochaine.id === this.etapes_affaire_conf.fin_processus) &&
+        (this.etapeAffaire.prochaine_id === this.etapes_affaire_conf.fin_processus) &&
         ![this.typesAffaires_conf.ppe, this.typesAffaires_conf.modification_ppe, this.typesAffaires_conf.pcop].includes(this.affaire.type_id)
       ) {
         // update date_validation or date_cloture if next step is "fin de processus"
         if (
-          (this.etapeAffaire.prochaine && this.etapeAffaire.prochaine.id && this.etapeAffaire.prochaine.id === this.etapes_affaire_conf.fin_processus) &&
+          (this.etapeAffaire.prochaine_id === this.etapes_affaire_conf.fin_processus) &&
           (![this.typesAffaires_conf.mutation, this.typesAffaires_conf.modification, this.typesAffaires_conf.remaniement_parcellaire,
           this.typesAffaires_conf.modification_visa, this.typesAffaires_conf.modification_mutation].includes(this.affaire.type_id)) && (this.numerosReserves.length === 0)
         ) {
@@ -364,7 +335,7 @@ export default {
         ((this.affaire.etape_id === this.etapes_affaire_conf.envoi && ![this.typesAffaires_conf.pcop, this.typesAffaires_conf.cadastration].includes(this.affaire.type_id)) ||
         (this.affaire.etape_id === this.etapes_affaire_conf.envoi_pcop && this.affaire.type_id === this.typesAffaires_conf.pcop) ||
         (this.affaire.etape_id === this.etapes_affaire_conf.envoi_cadastration && this.affaire.type_id === this.typesAffaires_conf.cadastration)) &&
-        ([this.etapes_affaire_conf.validation, this.etapes_affaire_conf.signature_art35].includes(this.etapeAffaire.prochaine.id) ||
+        ([this.etapes_affaire_conf.validation, this.etapes_affaire_conf.signature_art35].includes(this.etapeAffaire.prochaine_id) ||
         [this.typesAffaires_conf.ppe, this.typesAffaires_conf.modification_ppe, this.typesAffaires_conf.pcop].includes(this.affaire.type_id))
       ) {
         this.updateAffaireDate = {
@@ -421,10 +392,14 @@ export default {
       }).catch(err => handleException(err, this));
     },
 
+    /** get selected value for new step */
+    setNewStepId(value) {
+      this.etapeAffaire.prochaine_id = value;
+    }
+
   },
 
   mounted: function() {
-    this.searchAffaireEtapes();
     this.$root.$on( "setEtapeNouveauxNumeros", (data) => this.setNumerosReserves(data) );
 
     // operateur is admin?

diff --git a/front/src/components/Affaires/Etape/etape.css b/front/src/components/Affaires/Etape/etape.css
@@ -1,3 +1,8 @@
+.etapeDialog .md-dialog-container {
+    min-width: 750px !important;
+    width: 750px !important;
+}
+
 .etape {
     width: 100%;
     float: left;

diff --git a/front/src/components/Affaires/Etape/etape.html b/front/src/components/Affaires/Etape/etape.html
@@ -18,29 +18,16 @@
 
 
     <!-- Dialog pour la création de nouvelle étape dans l'affaire -->
-    <md-dialog :md-active.sync="etapeAffaire.showDialog" style="margin: auto; width: 900px">
+    <md-dialog :md-active.sync="etapeAffaire.showDialog" class="etapeDialog">
         <md-dialog-title>Nouvelle étape</md-dialog-title>
 
         <md-dialog-content>
             <div class="md-layout md-gutter md-alignment-top-space-between">
-                <div class="md-layout-item md-size-45">
-                    <!-- Nom de l'étape en cours -->
-                    <md-field>
-                        <label>Etape en cours</label>
-                        <md-input v-model="affaire.etape" readonly></md-input>
-                    </md-field>
-                </div>
-
-                <h3 style="margin-top: 24px">=></h3>
+                <div class="md-layout-item md-size-100">
 
-                <div class="md-layout-item md-size-45">
-                    <!-- Nom de la prochaine étape -->
-                    <md-autocomplete v-model="etapeAffaire.prochaine" :md-options="affaireEtapes" md-dense required @md-selected="onSelectNextStep" style="z-index: 9000 !important">
-                        <label>Prochaine étape</label>
-                    </md-autocomplete>
+                    <NewStepSetter :affaire_id="affaire.id" @new-step-selected="setNewStepId" />
                 </div>
-
-                <div class="md-layout-item md-size-45" v-if="etapeAffaire.prochaine && etapeAffaire.prochaine.id && etapeAffaire.prochaine.id === etapes_affaire_conf.travaux_chef_equipe">
+                <div class="md-layout-item md-size-45" v-if="etapeAffaire.prochaine_id === etapes_affaire_conf.travaux_chef_equipe">
                     <!-- Sélection de l'opérateur -->
                     <md-field>
                         <label>Opérateur·rice</label>

diff --git a/front/src/components/Affaires/NumerosAffaire/ReferenceNumeros/ReferenceNumeros.vue b/front/src/components/Affaires/NumerosAffaire/ReferenceNumeros/ReferenceNumeros.vue
@@ -1,4 +1,4 @@
-<style src="./referenceNumeros.css" scoped></style>
+<style src="./referenceNumeros.css"></style>
 <style lang="css">.md-menu-content { z-index: 9000 !important; }</style>
 <template src="./referenceNumeros.html"></template>
 

diff --git a/front/src/components/Affaires/NumerosAffaire/ReferenceNumeros/referenceNumeros.css b/front/src/components/Affaires/NumerosAffaire/ReferenceNumeros/referenceNumeros.css
@@ -9,3 +9,8 @@
     vertical-align: top;
     margin: 0px;
 }
+
+.referenceNumerosDialog .md-dialog-container {
+    min-width: 900px;
+    width: 900px;
+}
diff --git a/front/src/components/Affaires/NumerosAffaire/ReferenceNumeros/referenceNumeros.html b/front/src/components/Affaires/NumerosAffaire/ReferenceNumeros/referenceNumeros.html
@@ -1,7 +1,7 @@
 <div class="referenceNumeros">
 
     <md-dialog :md-active.sync="showReferenceDialog" md-confirm-text="Enregistrer" md-cancel-text="Annuler"
-        md-title="Référencer des numéros" :md-close-on-esc="true" :md-click-outside-to-close="true">
+        md-title="Référencer des numéros" :md-close-on-esc="true" :md-click-outside-to-close="true" class="referenceNumerosDialog">
 
         <md-dialog-title>{{ dialogTitle }}</md-dialog-title>
 

diff --git a/front/src/components/Utils/NewStepSetter/NewStepSetter.vue b/front/src/components/Utils/NewStepSetter/NewStepSetter.vue
@@ -32,7 +32,7 @@ export default {
         }
       ).then(response => {
         this.actual_step = response.data.etape;
-        this.next_step_id = response.data.predicted_next_step_id;
+        this.new_step_id = response.data.predicted_next_step_id;
       }).catch(err => handleException(err, this));
     },