refactor: use forEach instead of map in handleFeatureSelect (#259)

microsoft · May 14, 2020 · c1c590c · c1c590c
1 parent f953acd
commit c1c590c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/react/components/pages/editorPage/canvas.tsx b/src/react/components/pages/editorPage/canvas.tsx
@@ -811,12 +811,12 @@ export default class Canvas extends React.Component<ICanvasProps, ICanvasState>
         if (category === FeatureCategory.Checkbox ||
             (category === FeatureCategory.Label && this.state.currentAsset.regions
             .find((r) => r.id === regionId).category === FeatureCategory.Checkbox)) {
-                selectedRegions.map((region) => this.removeFromSelectedRegions(region.id));
+                selectedRegions.forEach((region) => this.removeFromSelectedRegions(region.id));
         } else if (category === FeatureCategory.Text ||
             (category === FeatureCategory.Label && this.state.currentAsset.regions
             .find((r) => r.id === regionId).category === FeatureCategory.Text)) {
                 selectedRegions.filter((region) => region.category === FeatureCategory.Checkbox)
-                    .map((region) => this.removeFromSelectedRegions(region.id));
+                    .forEach((region) => this.removeFromSelectedRegions(region.id));
         }
     }