Optimize afterAddingMidMenuItem() - do not need to pay extra parameter hasMultipleSelectionSubContainer.

This commit is contained in:
Jessica Wang 2010-12-13 09:53:43 -08:00
parent dedfe57280
commit fbd2e32882
2 changed files with 4 additions and 4 deletions

View File

@ -1009,7 +1009,7 @@ function initVMWizard() {
if (item.passwordenabled == true) {
var extraMessage = "New password: " + item.password;
afterAddingMidMenuItem($midmenuItem1, true, extraMessage, true);
afterAddingMidMenuItem($midmenuItem1, true, extraMessage);
var afterActionInfo = "Instance " + getVmName(item.name, item.displayname) + " has been created successfully. New password is: " + item.password;
$midmenuItem1.data("afterActionInfo", afterActionInfo);
@ -1022,7 +1022,7 @@ function initVMWizard() {
}).dialog("open");
}
else {
afterAddingMidMenuItem($midmenuItem1, true, null, true);
afterAddingMidMenuItem($midmenuItem1, true, null);
}
} else if (result.jobstatus == 2) {
// Failed

View File

@ -667,7 +667,7 @@ function beforeAddingMidMenuItem() {
$("#midmenu_container").append($midmenuItem1.show());
return $midmenuItem1;
}
function afterAddingMidMenuItem($midmenuItem1, isSuccessful, extraMessage, hasMultipleSelectionSubContainer) {
function afterAddingMidMenuItem($midmenuItem1, isSuccessful, extraMessage) {
$midmenuItem1.find("#content").removeClass("inaction");
$midmenuItem1.find("#spinning_wheel").hide();
@ -676,7 +676,7 @@ function afterAddingMidMenuItem($midmenuItem1, isSuccessful, extraMessage, hasMu
$midmenuItem1.data("afterActionInfo", ("Adding succeeded."));
var $container = $("#midmenu_container");
if(hasMultipleSelectionSubContainer) {
if($("#midmenu_container").find("#multiple_selection_sub_container").length > 0) {
$container = $("#midmenu_container").find("#multiple_selection_sub_container");
$midmenuItem1.appendTo("#midmenu_container #multiple_selection_sub_container");
}