🐛 [BUG] Fix cannot click on objects after customizing map styles (refs …

…#3800)
GeotrekCE · Nov 3, 2023 · 1795689 · 1795689
1 parent 6cf5f1f
commit 1795689
Show file tree

Hide file tree

Showing 7 changed files with 19 additions and 7 deletions.
diff --git a/docs/changelog.rst b/docs/changelog.rst
@@ -8,6 +8,7 @@ CHANGELOG
 **Bug fixes**
 
 - Fix: filters choices can raise exception in lists and not updated until application restart (#3812)
+- Fix cannot click on objects after customizing map styles (#3800)
 
 
 2.101.3     (2023-10-26)

diff --git a/geotrek/feedback/static/feedback/main.js b/geotrek/feedback/static/feedback/main.js
@@ -10,9 +10,11 @@ $(window).on('entity:map', function (e, data) {
     var map = data.map;
 
     // Show report layer in application maps
+    var style = L.Util.extend({ clickable: false },
+        window.SETTINGS.map.styles[landLayer.id] || {});
     var layer = new L.ObjectsLayer(null, {
         modelname: modelname,
-        style: L.Util.extend(window.SETTINGS.map.styles[modelname] || {}, { clickable: false }),
+        style: style,
     });
 
     if (data.modelname != modelname) {

diff --git a/geotrek/infrastructure/static/infrastructure/main.js b/geotrek/infrastructure/static/infrastructure/main.js
@@ -10,9 +10,11 @@ $(window).on('entity:map', function (e, data) {
     var map = data.map;
 
     // Show infrastructure layer in application maps
+    var style = L.Util.extend({ clickable: false },
+        window.SETTINGS.map.styles[modelname] || {});
 	var layer = new L.ObjectsLayer(null, {
 		modelname: modelname,
-		style: L.Util.extend(window.SETTINGS.map.styles[modelname] || {}, { clickable:false }),
+		style: style,
 	});
 
     if (data.modelname != modelname){

diff --git a/geotrek/outdoor/static/outdoor/main.js b/geotrek/outdoor/static/outdoor/main.js
@@ -4,10 +4,12 @@ $(window).on('entity:map', function (e, data) {
     var loaded_site = false;
     var loaded_course = false;
     // Show outdoor layers in application maps
+    var style = L.Util.extend({ clickable: false },
+        window.SETTINGS.map.styles[modelname] || {});
     $.each(['site', 'course'], function (i, modelname) {
         var layer = new L.ObjectsLayer(null, {
             modelname: modelname,
-            style: L.Util.extend(window.SETTINGS.map.styles[modelname] || {}, { clickable: false }),
+            style: style,
         });
         if (data.modelname != modelname) {
             map.layerscontrol.addOverlay(layer, tr(modelname), tr('Outdoor'));

diff --git a/geotrek/sensitivity/static/sensitivity/main.js b/geotrek/sensitivity/static/sensitivity/main.js
@@ -10,9 +10,11 @@ $(window).on('entity:map', function (e, data) {
     var map = data.map;
 
     // Show sensitiveare layer in application maps
+    var style = L.Util.extend(window.SETTINGS.map.styles[modelname] || {}, {clickable: false})
+
     var layer = new L.ObjectsLayer(null, {
         modelname: modelname,
-        style: L.Util.extend(window.SETTINGS.map.styles[modelname] || {}, {clickable: false}),
+        style: style,
     });
 
     if (data.modelname != modelname) {

diff --git a/geotrek/signage/static/signage/main.js b/geotrek/signage/static/signage/main.js
@@ -8,11 +8,12 @@ $(window).on('entity:map', function (e, data) {
 	var url = window.SETTINGS.urls[layername];
     var loaded_infrastructure = false;
     var map = data.map;
-
+    var style = L.Util.extend({ clickable: false },
+        window.SETTINGS.map.styles[modelname] || {});
     // Show infrastructure layer in application maps
 	var layer = new L.ObjectsLayer(null, {
 		modelname: modelname,
-		style: L.Util.extend(window.SETTINGS.map.styles[modelname] || {}, { clickable:false }),
+		style: style,
 	});
 
     if (data.modelname != modelname){

diff --git a/geotrek/tourism/static/tourism/main.js b/geotrek/tourism/static/tourism/main.js
@@ -56,11 +56,13 @@ $(window).on('entity:map', function (e, data) {
     var map = data.map;
     var loaded_event = false;
     var loaded_touristic = false;
+    var style = L.Util.extend({ clickable: false },
+        window.SETTINGS.map.styles[modelname] || {});
     // Show tourism layer in application maps
     $.each(['touristiccontent', 'touristicevent'], function (i, modelname) {
         var layer = new L.ObjectsLayer(null, {
             modelname: modelname,
-            style: L.Util.extend(window.SETTINGS.map.styles[modelname] || {}, {clickable:false}),
+            style: style,
         });
         if (data.modelname != modelname){
             map.layerscontrol.addOverlay(layer, tr(modelname), tr('Tourism'));