From 794ee6929d078fd919842f2c1f55156f133d7934 Mon Sep 17 00:00:00 2001 From: Jessica Wang Date: Tue, 7 Oct 2014 14:20:59 -0700 Subject: [PATCH] CLOUDSTACK-4987: UI > Instances > detailView > NICs tab > Add network to VM > Network dropdown => (1) For root-admin, populate networks of all accounts. (2) For regular-user/domain-admin, populate only networks belonging to this VM owner. --- ui/scripts/instances.js | 19 ++++++++++++++----- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/ui/scripts/instances.js b/ui/scripts/instances.js index 46df52d2842..a15fcd95c5f 100644 --- a/ui/scripts/instances.js +++ b/ui/scripts/instances.js @@ -1945,13 +1945,22 @@ networkid: { label: 'label.network', select: function(args) { + var data1 = { + zoneid: args.context.instances[0].zoneid + }; + if (isAdmin()) { + $.extend(data1, { + listAll: true + }); + } else { + $.extend(data1, { + account: args.context.instances[0].account, + domainid: args.context.instances[0].domainid + }); + } $.ajax({ url: createURL('listNetworks'), - data: { - zoneid: args.context.instances[0].zoneid, - account: args.context.instances[0].account, - domainid: args.context.instances[0].domainid - }, + data: data1, success: function(json) { args.response.success({ data: $.map(json.listnetworksresponse.network, function(network) {