Merge pull request mviewer#1203 from jdev-org/issue-949

add bbox filter when needed in data.geopf calls

Author: lecault

apps/default.xml

@@ -12,8 +12,8 @@
             attribution="GéoBretagne. Données : les contributeurs d'<a href='https://www.openstreetmap.org/' target='_blank'>OpenStreetMap </a>,  <a href='https://www.openstreetmap.org/copyright' target='_blank'>ODbL </a>" />
     </baselayers>     
     <proxy url=""/>
-    <!-- <olscompletion url="https://api-adresse.data.gouv.fr/search/" type="search" attribution="La recherche d'adresse est un service de l'IGN via les données de la BAN"/> -->
-    <olscompletion url="https://data.geopf.fr/geocodage/completion" type="completion" attribution="Service d'autocomplétion de l'IGN"/>
+   <olscompletion url="https://data.geopf.fr/geocodage/search" type="search" attribution="La recherche d'adresse est un service de l'IGN via les données de la BAN"/>
+    <!-- <olscompletion url="https://data.geopf.fr/geocodage/completion" type="completion" attribution="Service d'autocomplétion de l'IGN"/> -->
 
     <searchparameters bbox="false" localities="true" features="false" />   
 

docs/doc_tech/config_search.rst

@@ -72,7 +72,7 @@ Options liées à la recherche d'adresse *(olscompletion)* et/ou à la recherche
 
 * ``localities`` *(optionnel)* : Utilisation du service d'adresse olscompletion : true ou false (defaut = true).
 * ``features`` *(optionnel)* : Utilisation du service de recherche d'entités elasticsearch ou fuse : true ou false (defaut = true).
-* ``bbox`` *(optionnel)* : Recherche d'adresse et/ou d'entitées limitée à l'emprise de la carte : true ou false (defaut = false).
+* ``bbox`` *(optionnel)* : Recherche d'adresse et/ou d'entitées limitée à l'emprise de la carte : true ou false (defaut = false). Avec un service olscompletion de type ``completion`` les résultats sont filtrés à l'intérieur de l'emprise ; pour les autres types (``search``/BAN) la bbox sert uniquement à ordonner la réponse.
 * ``querymaponclick`` *(optionnel)* : Interroge la carte après sélection d'un résultat dans la liste : true ou false - defaut = false.
 * ``marker`` *(optionnel)* : Afficher ou non un icône sur le résultat de la recherche d'adresse : true ou false (defaut = true).
 * ``static`` *(optionnel)* : En lien avec le paramètre doctypes. Active ou désactive la recherche associée à des documents requêtés systématiquement, indépendamment des couches affichées : true ou false (defaut = false).

index.html

@@ -430,11 +430,12 @@ <h5 class="modal-title" i18n="search.panel.title">
             </div>
             <div class="modal-body">
               <ul id="searchparameters">
-                <li class="mv-param-item" onclick="search.toggleParameter(this)">
+                <li
+                  class="mv-param-item"
+                  id="param_search_bbox"
+                  onclick="search.toggleParameter(this)">
                   <a href="#">
-                    <span
-                      id="param_search_bbox"
-                      class="state-icon far mv-unchecked"></span>
+                    <span class="state-icon far mv-unchecked"></span>
                     <div style="display: inline" i18n="search.modal.extent.text">
                       Rechercher dans l'emprise de la carte
                     </div>

js/search.js

@@ -346,22 +346,51 @@ var search = (function () {
     // OpenLS or IGN services
     if (_searchparams.localities) {
       if (["ign", "completion", "geoportail"].includes(_olsCompletionType)) {
-        displaySearchList(
-          `${_olsCompletionUrl}?text=${value}&type=StreetAddress,PositionOfInterest&ter=5`,
-          _olsCompletionType
-        );
+        const parameters = {
+          text: value,
+          type: "StreetAddress,PositionOfInterest",
+          ter: 5,
+        };
+        if (_searchparams.bbox) {
+          const extent = _map.getView().calculateExtent(_map.getSize());
+          const extent4326 = ol.proj.transformExtent(
+            extent,
+            _projection.getCode(),
+            _proj4326
+          );
+          parameters.bbox = extent4326.join(",");
+          const center = ol.extent.getCenter(extent4326);
+          parameters.lon = center[0];
+          parameters.lat = center[1];
+        }
+        const searchUrl = new URL(_olsCompletionUrl);
+        Object.entries(parameters).forEach(([key, val]) => {
+          if (val !== undefined && val !== null) {
+            searchUrl.searchParams.append(key, val);
+          }
+        });
+        displaySearchList(searchUrl.toString(), _olsCompletionType);
       } else if (["ban", "search"].includes(_olsCompletionType)) {
         var parameters = { q: value, limit: 5 };
         if (_searchparams.bbox) {
-          var center = _map.getView().getCenter();
-          var center = ol.proj.transform(center, _projection.getCode(), _proj4326);
+          var extent = _map.getView().calculateExtent(_map.getSize());
+          var extent4326 = ol.proj.transformExtent(
+            extent,
+            _projection.getCode(),
+            _proj4326
+          );
+          parameters.bbox = extent4326.join(",");
+          var center = ol.extent.getCenter(extent4326);
           parameters.lon = center[0];
           parameters.lat = center[1];
         }
         // create URL
         const searchUrl = new URL(_olsCompletionUrl);
-        searchUrl.searchParams.append("q", parameters.q);
-        searchUrl.searchParams.append("limit", parameters.limit);
+        Object.entries(parameters).forEach(([key, val]) => {
+          if (val !== undefined && val !== null) {
+            searchUrl.searchParams.append(key, val);
+          }
+        });
         const searchUrlString = searchUrl.toString();
         // display result
         displaySearchList(searchUrlString, _olsCompletionType);
@@ -1141,6 +1170,11 @@ var search = (function () {
     _projection = mviewer.getProjection();
     _overLayers = mviewer.getLayers();
     var sparams = configuration.searchparameters || false;
+    const asBool = (val, fallback) => {
+      if (val === true || val === "true") return true;
+      if (val === false || val === "false") return false;
+      return fallback;
+    };
     if (configuration.olscompletion) {
       _olsCompletionUrl = configuration.olscompletion.url;
       $("#adresse-attribution").text(configuration.olscompletion.attribution);
@@ -1233,13 +1267,19 @@ var search = (function () {
     if (!_olsCompletionUrl) {
       _searchparams.localities = false;
     }
-    if (sparams && sparams.bbox && sparams.localities && sparams.features) {
-      _searchparams.bbox = sparams.bbox === "true";
-      _searchparams.localities = sparams.localities === "true";
-      _searchparams.features = sparams.features === "true";
-      _searchparams.static = sparams.static === "true";
-      _searchparams.querymaponclick = sparams.querymaponclick === "true";
-      _searchparams.closeafterclick = sparams.closeafterclick === "true";
+    if (sparams) {
+      _searchparams.bbox = asBool(sparams.bbox, _searchparams.bbox);
+      _searchparams.localities = asBool(sparams.localities, _searchparams.localities);
+      _searchparams.features = asBool(sparams.features, _searchparams.features);
+      _searchparams.static = asBool(sparams.static, _searchparams.static);
+      _searchparams.querymaponclick = asBool(
+        sparams.querymaponclick,
+        _searchparams.querymaponclick
+      );
+      _searchparams.closeafterclick = asBool(
+        sparams.closeafterclick,
+        _searchparams.closeafterclick
+      );
       _searchparams.searchmaxzoomlevel = sparams.searchmaxzoomlevel
         ? sparams.searchmaxzoomlevel
         : 15;