From ea1940f30c19e9764844848496de62cef48e2076 Mon Sep 17 00:00:00 2001 From: Jessica Wang Date: Thu, 20 Jan 2011 15:22:17 -0800 Subject: [PATCH] bug 7949: deleteSecurityGroup API has been changed to take in ID parameter instead of name parameter. Here is related UI change. --- ui/scripts/cloud.core.securitygroup.js | 29 ++------------------------ 1 file changed, 2 insertions(+), 27 deletions(-) diff --git a/ui/scripts/cloud.core.securitygroup.js b/ui/scripts/cloud.core.securitygroup.js index dcfe2e31118..44a6e23fef8 100644 --- a/ui/scripts/cloud.core.securitygroup.js +++ b/ui/scripts/cloud.core.securitygroup.js @@ -395,31 +395,6 @@ function doEditSecurityGroup2($actionLink, $detailsTab, $midmenuItem1, $readonly }); } -function doDeleteSecurityGroup($actionLink, $thisTab, $midmenuItem1) { - $("#dialog_info") - .text("Are you sure you want to delete this security group?") - .dialog('option', 'buttons', { - "Confirm": function() { - $(this).dialog("close"); - - var jsonObj = $midmenuItem1.data("jsonObj"); - - var array1 = []; - array1.push("&domainid="+jsonObj.domainid); - array1.push("&account="+jsonObj.account); - array1.push("&name="+jsonObj.name); - - var id = jsonObj.id; - var apiCommand = "command=deleteSecurityGroup" + array1.join(""); - doActionToTab(id, $actionLink, apiCommand, $midmenuItem1, $thisTab); - }, - "Cancel": function() { - $(this).dialog("close"); - - } - }).dialog("open"); -} - function securityGroupToMidmenu(jsonObj, $midmenuItem1) { $midmenuItem1.attr("id", getMidmenuId(jsonObj)); $midmenuItem1.data("jsonObj", jsonObj); @@ -659,8 +634,8 @@ var securityGroupActionMap = { dialogBeforeActionFn: doEditSecurityGroup }, "Delete Security Group": { - isAsyncJob: false, - dialogBeforeActionFn: doDeleteSecurityGroup, + api: "deleteSecurityGroup", + isAsyncJob: false, inProcessText: "Deleting Security Group....", afterActionSeccessFn: function(json, $midmenuItem1, id) { $midmenuItem1.slideUp("slow", function() {