From 285b32b616824924fae5c8d5820cb27f34dc7a12 Mon Sep 17 00:00:00 2001
From: rldhont <rldhont@gmail.com>
Date: Fri, 7 Mar 2025 10:05:38 +0100
Subject: [PATCH 1/2] [Bugfix] LocatByLayer - Error after automatic ESlint

After https://github.com/3liz/lizmap-web-client/pull/5334 the locateByLayer contains an error
---
 assets/src/modules/LocateByLayer.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/assets/src/modules/LocateByLayer.js b/assets/src/modules/LocateByLayer.js
index d601ef8c49..15aad9a178 100644
--- a/assets/src/modules/LocateByLayer.js
+++ b/assets/src/modules/LocateByLayer.js
@@ -166,7 +166,7 @@ export default class LocateByLayer {
         if ( 'vectorjoins' in locate ) {
             var vectorjoins = locate.vectorjoins;
             for ( var k=0, vectorjoinsLen=vectorjoins.length; k<vectorjoinsLen; k++) {
-                var vectorjoin = vectorjoins[i];
+                var vectorjoin = vectorjoins[k];
                 fields.push( vectorjoin.targetFieldName );
             }
         }

From fdd8f5dd156470f96e16921e302251c8aa34ccd6 Mon Sep 17 00:00:00 2001
From: rldhont <rldhont@gmail.com>
Date: Fri, 7 Mar 2025 10:11:56 +0100
Subject: [PATCH 2/2] JS: LocateByLayer Linting

---
 assets/src/modules/LocateByLayer.js | 43 +++++++++++++++++++----------
 1 file changed, 28 insertions(+), 15 deletions(-)

diff --git a/assets/src/modules/LocateByLayer.js b/assets/src/modules/LocateByLayer.js
index 15aad9a178..92c7963b58 100644
--- a/assets/src/modules/LocateByLayer.js
+++ b/assets/src/modules/LocateByLayer.js
@@ -221,17 +221,22 @@ export default class LocateByLayer {
                 }
 
                 // add filter values list
-                $('#locate-layer-'+layerName).parent().before('<div class="locate-layer"><select id="locate-layer-'+layerName+'-'+locate.filterFieldName+'">'+fOptions+'</select></div><br/>');
+                $('#locate-layer-'+layerName).parent().before(
+                    `<div class="locate-layer">
+                    <select id="locate-layer-`+layerName+'-'+locate.filterFieldName+'">'+fOptions+`</select>
+                    </div><br/>`
+                );
                 // listen to filter select changes
-                document.getElementById('locate-layer-'+layerName+'-'+locate.filterFieldName).addEventListener("change", () => {
-                    var filterValue = $(this).children(':selected').val();
-                    this.updateLocateFeatureList( aName );
-                    if (filterValue == '-1')
-                        $('#locate-layer-'+layerName+'-'+locate.filterFieldName+' ~ span > input').val('');
-                    $('#locate-layer-'+layerName+' ~ span > input').val('');
-                    $('#locate-layer-'+layerName).val('-1');
-                    this.zoomToLocateFeature(aName);
-                });
+                document.getElementById('locate-layer-'+layerName+'-'+locate.filterFieldName)
+                    .addEventListener("change", () => {
+                        var filterValue = $(this).children(':selected').val();
+                        this.updateLocateFeatureList( aName );
+                        if (filterValue == '-1')
+                            $('#locate-layer-'+layerName+'-'+locate.filterFieldName+' ~ span > input').val('');
+                        $('#locate-layer-'+layerName+' ~ span > input').val('');
+                        $('#locate-layer-'+layerName).val('-1');
+                        this.zoomToLocateFeature(aName);
+                    });
                 // add combobox to the filter select
                 $('#locate-layer-'+layerName+'-'+locate.filterFieldName).combobox({
                     position: { my : "right top", at: "right bottom" },
@@ -248,8 +253,10 @@ export default class LocateByLayer {
                 });
 
                 // add place holder to the filter combobox input
-                $('#locate-layer-'+layerName+'-'+locate.filterFieldName+' ~ span > input').attr('placeholder', filterPlaceHolder).val('');
-                $('#locate-layer-'+layerName+'-'+locate.filterFieldName+' ~ span > input').autocomplete('close');
+                $('#locate-layer-'+layerName+'-'+locate.filterFieldName+' ~ span > input')
+                    .attr('placeholder', filterPlaceHolder).val('');
+                $('#locate-layer-'+layerName+'-'+locate.filterFieldName+' ~ span > input')
+                    .autocomplete('close');
             }
 
             // create combobox for the layer
@@ -285,7 +292,9 @@ export default class LocateByLayer {
                 var featElement = features[j];
                 locate.features[featElement.id.toString()] = featElement;
                 if ( !('filterFieldName' in locate) )
-                    options += '<option value="' + featElement.id + '">' + DOMPurify.sanitize(featElement.properties[locate.fieldName]) + '</option>';
+                    options += '<option value="' + featElement.id + '">'
+                        + DOMPurify.sanitize(featElement.properties[locate.fieldName])
+                        + '</option>';
             }
             document.getElementById('locate-layer-'+layerName).innerHTML = options;
             // listen to select changes
@@ -456,7 +465,9 @@ export default class LocateByLayer {
                     var jFeat = jLocate.features[jVal];
                     for (var featId in features) {
                         var feature = features[featId];
-                        if ( feature.properties[vectorjoin.targetFieldName] != jFeat.properties[vectorjoin.joinFieldName] )
+                        var targetProperty = feature.properties[vectorjoin.targetFieldName];
+                        var joinProperty = jFeat.properties[vectorjoin.joinFieldName];
+                        if ( targetProperty != joinProperty )
                             delete features[featId];
                     }
                 }
@@ -467,7 +478,9 @@ export default class LocateByLayer {
         var options = '<option value="-1" label="'+placeHolder+'"></option>';
         for (var featureElementId in features) {
             var featureElement = features[featureElementId];
-            options += '<option value="' + featureElement.id + '">' + DOMPurify.sanitize(featureElement.properties[locate.fieldName]) + '</option>';
+            options += '<option value="' + featureElement.id + '">'
+                + DOMPurify.sanitize(featureElement.properties[locate.fieldName])
+                + '</option>';
         }
         // add option list
         $('#locate-layer-'+ this._lizmap3.cleanName(aName)).html(options);