From 74163af2952abc2bacfd704f1e289128500b46a2 Mon Sep 17 00:00:00 2001 From: Jessica Wang Date: Wed, 7 Dec 2011 16:30:15 -0800 Subject: [PATCH] VM wizard - if create network fails, show error message and not proceed deploying VM. --- ui/scripts/instances.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/ui/scripts/instances.js b/ui/scripts/instances.js index a9356305e9c..21920556bf7 100644 --- a/ui/scripts/instances.js +++ b/ui/scripts/instances.js @@ -460,15 +460,15 @@ defaultNetwork = newNetwork.id; }, error: function(XMLHttpResponse) { - isCreateNetworkSuccessful = false; - args.response.error("Failed to create new network. Please try again."); //wait for Brian to implement + isCreateNetworkSuccessful = false; + var errorMsg = "Failed to create new network, unable to proceed to deploy VM. Error: " + parseXMLHttpResponse(XMLHttpResponse); + alert(errorMsg); + args.response.error(errorMsg); //args.response.error(errorMsg) here doesn't show errorMsg. Waiting for Brian to fix it. use alert(errorMsg) to show errorMsg for now. } }); - } - if(isCreateNetworkSuccessful == false) { - alert("Failed to create new network, unable to proceed to deploy VM."); + if(isCreateNetworkSuccessful == false) { return; } //create new network ends here