Reset measure state on reset controls action. Fixes#1464

geosolutions-it · Feb 16, 2017 · b09ac28 · b09ac28
1 parent 5399d00
commit b09ac28
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 5 deletions.
diff --git a/web/client/actions/measurement.js b/web/client/actions/measurement.js
@@ -27,7 +27,6 @@ function changeMeasurement(measurement) {
 function changeMeasurementState(measureState) {
     return {
         type: CHANGE_MEASUREMENT_STATE,
-        pointMeasureEnabled: measureState.pointMeasureEnabled,
         lineMeasureEnabled: measureState.lineMeasureEnabled,
         areaMeasureEnabled: measureState.areaMeasureEnabled,
         bearingMeasureEnabled: measureState.bearingMeasureEnabled,

diff --git a/web/client/reducers/__tests__/controls-test.js b/web/client/reducers/__tests__/controls-test.js
@@ -8,7 +8,7 @@
 const expect = require('expect');
 
 const controls = require('../controls');
-const {TOGGLE_CONTROL, SET_CONTROL_PROPERTY} = require('../../actions/controls');
+const {TOGGLE_CONTROL, SET_CONTROL_PROPERTY, RESET_CONTROLS} = require('../../actions/controls');
 
 describe('Test the constrols reducer', () => {
     it('toggles a control the first time', () => {
@@ -57,4 +57,22 @@ describe('Test the constrols reducer', () => {
         expect(state.mycontrol).toExist();
         expect(state.mycontrol.prop).toBe('val');
     });
+
+
+    it('reset the controls', () => {
+        const state = controls(
+            {
+                c1: { enabled: true},
+                c2: { enabled: false},
+                c3: { idonthaveenabledfield: "whatever"}
+            }, {
+            type: RESET_CONTROLS
+        });
+        expect(state.c1).toExist();
+        expect(state.c2).toExist();
+        expect(state.c3).toExist();
+        expect(state.c1.enabled).toBe(false);
+        expect(state.c2.enabled).toBe(false);
+        expect(state.c3.enabled).toNotExist();
+    });
 });
diff --git a/web/client/reducers/measurement.js b/web/client/reducers/measurement.js
@@ -11,7 +11,8 @@ const {
     CHANGE_MEASUREMENT_STATE
 } = require('../actions/measurement');
 
-const {TOGGLE_CONTROL} = require('../actions/controls');
+const {TOGGLE_CONTROL, RESET_CONTROLS} = require('../actions/controls');
+
 const assign = require('object-assign');
 
 function measurement(state = {
@@ -22,15 +23,13 @@ function measurement(state = {
     switch (action.type) {
         case CHANGE_MEASUREMENT_TOOL:
             return assign({}, state, {
-                pointMeasureEnabled: ((action.geomType !== state.geomType) && (action.geomType === 'Point')),
                 lineMeasureEnabled: ((action.geomType !== state.geomType) && (action.geomType === 'LineString')),
                 areaMeasureEnabled: ((action.geomType !== state.geomType) && (action.geomType === 'Polygon')),
                 bearingMeasureEnabled: ((action.geomType !== state.geomType) && (action.geomType === 'Bearing')),
                 geomType: (action.geomType === state.geomType) ? null : action.geomType
             });
         case CHANGE_MEASUREMENT_STATE:
             return assign({}, state, {
-                pointMeasureEnabled: action.pointMeasureEnabled,
                 lineMeasureEnabled: action.lineMeasureEnabled,
                 areaMeasureEnabled: action.areaMeasureEnabled,
                 bearingMeasureEnabled: action.bearingMeasureEnabled,
@@ -53,6 +52,12 @@ function measurement(state = {
                 };
             }
         }
+        case RESET_CONTROLS:
+            return {
+                lineMeasureEnabled: false,
+                areaMeasureEnabled: false,
+                bearingMeasureEnabled: false
+            };
         default:
             return state;
     }