Merge pull request #19 from george-vrancianu/corweb-42

Disables "Execute" button in replica view after it is pressed CORWEB-42

src/actions/MigrationActions/MigrationActions.js

@@ -116,7 +116,7 @@ MigrationActions.deleteReplica.listen((replica, callback = null) => {
     .catch(MigrationActions.deleteReplica.failed);
 })
 
-MigrationActions.executeReplica.listen((replica, callback = null) => {
+MigrationActions.executeReplica.listen((replica, callback = null, errorCallback = null) => {
   if (replica.type == 'replica') {
     // check if endpoints exists
     let connections = Reflux.GlobalState.connectionStore.connections
@@ -158,8 +158,18 @@ MigrationActions.executeReplica.listen((replica, callback = null) => {
         if (callback) {
           callback(replica, response)
         }
-      }, MigrationActions.executeReplica.failed)
-      .catch(MigrationActions.executeReplica.failed);
+      }, (err) => {
+        MigrationActions.executeReplica.failed(err)
+        if (errorCallback) {
+          errorCallback(replica, err)
+        }
+      })
+      .catch((err) => {
+        MigrationActions.executeReplica.failed(err)
+        if (errorCallback) {
+          errorCallback(replica, err)
+        }
+      });
   } else {
     NotificationActions.notify("You cannot execute a migration.", "warning")
   }

src/components/ReplicaView/ReplicaView.js

@@ -44,6 +44,7 @@ class ReplicaView extends Reflux.Component {
 
     this.state = {
       title: 'Coriolis: View Replica',
+      isBeingExecuted: false,
       confirmationDialog: {
         visible: false,
         message: "Are you sure?",
@@ -64,8 +65,13 @@ class ReplicaView extends Reflux.Component {
   }
 
   executeReplica() {
+    this.setState({ isBeingExecuted: true })
     let item = this.state.replicas.filter(replica => replica.id == this.props.replicaId)[0]
-    MigrationActions.executeReplica(item)
+    MigrationActions.executeReplica(item, () => {
+      this.setState({ isBeingExecuted: false })
+    }, () => {
+      this.setState({ isBeingExecuted: false })
+    })
   }
 
   goBack() {
@@ -127,7 +133,7 @@ class ReplicaView extends Reflux.Component {
                 <div>
                   <button
                     className="gray"
-                    disabled={item.status === "RUNNING"}
+                    disabled={item.status === "RUNNING" || this.state.isBeingExecuted}
                     onClick={(e) => this.executeReplica(e)}
                   >
                     Execute Now