implemented remove layer

uninstalled lodash

Author: eran-pinhas

package.json

@@ -7,7 +7,6 @@
  "bluebird": "^3.5.2",
  "immutable": "^3.8.2",
  "leaflet": "^1.3.3",
- "lodash": "^4.17.10",
  "moment": "^2.22.2",
  "prop-types": "^15.6.2",
  "react": "^16.4.2",

src/action-creators/map.js

@@ -1,20 +1,20 @@
 import { ADD_LAYER, REMOVE_LAYER } from '../action-types/map';
 
-export const createAddLayerAction = (name, data) => {
+export const createAddLayerAction = (id, data) => {
  return {
  type: ADD_LAYER,
  payload: {
- name,
+ id,
  data,
  },
  };
 };
 
-export const createRemoveLayerAction = (name) => {
+export const createRemoveLayerAction = (id) => {
  return {
  type: REMOVE_LAYER,
  payload: {
- name,
+ id,
  },
  };
 };

src/components/Layers/Layers.js

@@ -72,7 +72,9 @@ export default class Layers extends Component {
  // return new TextDecoder("utf-8").decode(data)
  // })
  .then(({data})=> data)
- .then(geojson => this.props.addLayer(layer.name, geojson));
+ .then(geojson => this.props.addLayer(layer.id, geojson));
+ } else {
+ this.props.removeLayer(layer.id)
  }
 
  const layersList = this.state.layersList.map(

src/containers/LayersContainer/LayersContainer.js

@@ -1,14 +1,21 @@
-import { connect } from 'react-redux';
+import { connect } from "react-redux";
 
-import LayerInput from '../../components/Layers';
-import { createAddLayerAction } from '../../action-creators/map';
+import LayerInput from "../../components/Layers";
+import {
+ createAddLayerAction,
+ createRemoveLayerAction
+} from "../../action-creators/map";
 
-function mapDispatchToProps (dispatch) {
+function mapDispatchToProps(dispatch) {
  return {
- addLayer: (name, data) => dispatch(createAddLayerAction(name, data)),
+ addLayer: (id, data) => dispatch(createAddLayerAction(id, data)),
+ removeLayer: id => dispatch(createRemoveLayerAction(id))
  };
 }
 
-const LayerInputContainer = connect(null, mapDispatchToProps)(LayerInput);
+const LayerInputContainer = connect(
+ null,
+ mapDispatchToProps
+)(LayerInput);
 
-export default LayerInputContainer;
+export default LayerInputContainer;

src/store/reducers/layers.js

@@ -1,4 +1,3 @@
-import _ from 'lodash';
 import Immutable from 'immutable';
 
 import { ADD_LAYER, REMOVE_LAYER } from '../../action-types/map';
@@ -12,7 +11,7 @@ export default function (state = initialState, action) {
  newState = state.push(action.payload);
  break;
  case REMOVE_LAYER:
- newState = _.pullAllBy(state, action.payload, 'name');
+ newState = state.filter(layer => action.payload.id !== layer.id )
  break;
  default:
  newState = state;