Improved removeNode action to pass through the original node, not onl…

…y the id
geosolutions-it · Apr 28, 2017 · c5fb1c2 · c5fb1c2
1 parent 9c9d929
commit c5fb1c2
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 5 deletions.
diff --git a/web/client/actions/layers.js b/web/client/actions/layers.js
@@ -88,11 +88,12 @@ function sortNode(node, order, sortLayers = null) {
     };
 }
 
-function removeNode(node, type) {
+function removeNode(node, type, properties) {
     return {
         type: REMOVE_NODE,
         node: node,
-        nodeType: type
+        nodeType: type,
+        properties
     };
 }
 

diff --git a/web/client/components/TOC/DefaultLayer.jsx b/web/client/components/TOC/DefaultLayer.jsx
@@ -82,7 +82,7 @@ var DefaultLayer = React.createClass({
         };
     },
     onConfirmDelete() {
-        this.props.removeNode(this.props.node.id, "layers");
+        this.props.removeNode(this.props.node.id, "layers", this.props.node);
         this.closeDeleteDialog();
     },
     getInitialState: function() {

diff --git a/web/client/components/maps/modals/ConfirmModal.jsx b/web/client/components/maps/modals/ConfirmModal.jsx
@@ -9,6 +9,7 @@
 const React = require('react');
 const {Button} = require('react-bootstrap');
 const Modal = require('../../misc/Modal');
+const Spinner = require('react-spinkit');
 
   /**
    * A Modal window to show a confirmation dialog
@@ -28,7 +29,8 @@ const ConfirmModal = React.createClass({
         body: React.PropTypes.oneOfType([React.PropTypes.string, React.PropTypes.element]),
         titleText: React.PropTypes.oneOfType([React.PropTypes.string, React.PropTypes.element]),
         confirmText: React.PropTypes.oneOfType([React.PropTypes.string, React.PropTypes.element]),
-        cancelText: React.PropTypes.oneOfType([React.PropTypes.string, React.PropTypes.element])
+        cancelText: React.PropTypes.oneOfType([React.PropTypes.string, React.PropTypes.element]),
+        running: React.PropTypes.bool
     },
     getDefaultProps() {
         return {
@@ -54,13 +56,14 @@ const ConfirmModal = React.createClass({
         const footer = (<span role="footer"><div style={{"float": "left"}}></div>
         <Button
             ref="confirmButton"
+            disabled={this.props.running}
             className={this.props.className}
             key="confirmButton"
             bsStyle="primary"
             bsSize={this.props.buttonSize}
             onClick={() => {
                 this.onConfirm();
-            }}>{this.props.confirmText}</Button>
+            }}>{this.props.running ? <Spinner spinnerName="circle" overrideSpinnerClassName="spinner" noFadeIn /> : null}{this.props.confirmText}</Button>
         {this.props.includeCloseButton ? <Button
             key="cancelButton"
             ref="cancelButton"