Fix delete part of multipart. Wrong context methods

g3w-suite · Nov 22, 2023 · a24c227 · a24c227
1 parent 1203a9f
commit a24c227
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/workflows/steps/tasks/deletepartfrommultigeometriestask.js b/workflows/steps/tasks/deletepartfrommultigeometriestask.js
@@ -31,10 +31,13 @@ proto.run = function(inputs, context) {
     source,
     style: editingLayer.getStyle()
   });
+
   map.addLayer(tempLayer);
-  map.once('postrender', function(){
+  map.once('postrender', () => {
     let found = false;
-    this.forEachFeatureAtPixel(pixel, _feature => {
+    //need to call map.forEachFeatureAtPixel and not this.forEachFeatureAtPixel
+    //beacuse we use arrow function and it referred this to outside context
+    map.forEachFeatureAtPixel(pixel, _feature => {
       if (!found) {
         source.removeFeature(_feature);
         if (source.getFeatures().length) {
@@ -69,7 +72,9 @@ proto.run = function(inputs, context) {
         hitTolerance: 1
       }
     );
-    this.removeLayer(tempLayer);
+    //need to call map.forEachFeatureAtPixel and not this.forEachFeatureAtPixel
+    //beacuse we use arrow function and it referred this to outside context
+    map.removeLayer(tempLayer);
     tempLayer = null;
   });
   return d.promise()