Add back removed logic copying feature properties for injected data p…

…rior to modification
elastic · Oct 25, 2019 · 07b71bc · 07b71bc
1 parent 467dfcc
commit 07b71bc
Showing 1 changed file with 15 additions and 5 deletions.
diff --git a/x-pack/legacy/plugins/maps/public/layers/vector_layer.js b/x-pack/legacy/plugins/maps/public/layers/vector_layer.js
@@ -137,11 +137,21 @@ export class VectorLayer extends AbstractLayer {
     if (!featureCollection) {
       return null;
     }
-    // Set default visible property on data
-    featureCollection.features.forEach(
-      feature => _.set(feature, `properties.${FEATURE_VISIBLE_PROPERTY_NAME}`, true)
-    );
-    return featureCollection;
+    const copiedPropsFeatures = featureCollection.features
+      .map(feature => ({
+        type: 'Feature',
+        geometry: feature.geometry,
+        properties: feature.properties
+          ? {
+            ...feature.properties,
+            [FEATURE_VISIBLE_PROPERTY_NAME]: true
+          }
+          : {}
+      }));
+    return {
+      type: 'FeatureCollection',
+      features: copiedPropsFeatures
+    };
   }
 
   getCustomIconAndTooltipContent() {