rafaelweingartner closed pull request #2464: [4.11] CLOUDSTACK-10299: UI: fix error in network listing in project mode URL: https://github.com/apache/cloudstack/pull/2464
This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/ui/scripts/network.js b/ui/scripts/network.js index 7b9f75e7fda..bcb748d49de 100644 --- a/ui/scripts/network.js +++ b/ui/scripts/network.js @@ -86,12 +86,6 @@ }); } else { //non-portable IP which has only one NIC - /* - var nic = $.grep(instance.nic, function(nic) { - return nic.networkid == network.id; - })[0]; - */ - // Get NIC IPs $.ajax({ url: createURL('listNics'), @@ -170,12 +164,6 @@ }); } else { //non-portable IP which has only one NIC - /* - var nic = $.grep(instance.nic, function(nic) { - return nic.networkid == network.id; - })[0]; - */ - // Get NIC IPs $.ajax({ url: createURL('listNics'), @@ -412,8 +400,6 @@ $.ajax({ url: createURL('listRemoteAccessVpns'), data: { - account: g_account, - domainid: g_domainid, listAll: true }, async: false, @@ -1995,8 +1981,6 @@ $.ajax({ url: createURL('listZones'), data: dataObj, - // id: args.context.networks[0].zoneid - // }, async: false, success: function(json) { zoneObj = json.listzonesresponse.zone[0]; @@ -3524,7 +3508,7 @@ var stickyData = $.extend(true, {}, args.data.sticky); var certificateData = $.extend(true, {}, args.data.sslcertificate); - + //***** create new LB rule > Add VMs ***** $.ajax({ url: createURL('createLoadBalancerRule'), @@ -3537,70 +3521,27 @@ var lbID = data.createloadbalancerruleresponse.id; var inputData = { - id: data.createloadbalancerruleresponse.id - }; - + id: data.createloadbalancerruleresponse.id + }; + var selectedVMs = args.itemData; if (selectedVMs != null) { var vmidipmapIndex = 0; - for (var vmIndex = 0; vmIndex < selectedVMs.length; vmIndex++) { + for (var vmIndex = 0; vmIndex < selectedVMs.length; vmIndex++) { var selectedIPs = selectedVMs[vmIndex]._subselect; for (var ipIndex = 0; ipIndex < selectedIPs.length; ipIndex++) { inputData['vmidipmap[' + vmidipmapIndex + '].vmid'] = selectedVMs[vmIndex].id; - + if (args.context.ipAddresses[0].isportable) { - inputData['vmidipmap[' + vmidipmapIndex + '].vmip'] = selectedIPs[ipIndex].split(',')[1]; + inputData['vmidipmap[' + vmidipmapIndex + '].vmip'] = selectedIPs[ipIndex].split(',')[1]; } else { inputData['vmidipmap[' + vmidipmapIndex + '].vmip'] = selectedIPs[ipIndex]; } - + vmidipmapIndex++; - } + } } - } - - /*$.ajax({ - url: createURL('assignCertToLoadBalancer'), - data: {certid: certificateData.certificate, lbruleid: lbID}, - success: function(data) { - var jobID = data.assigncerttoloadbalancerresponse.jobid; - var lbProtocolCreated = false; - - args.response.success({ - _custom: { - jobId: jobID - }, - notification: { - label: 'label.add.certificate', - poll: function(args) { - var complete = args.complete; - var error = args.error; - - pollAsyncJobResult({ - _custom: { - jobId: jobID - }, - complete: function(args) { - if (lbProtocolCreated) return; - - lbProtocolCreated = true; - - if (certificateData && certificateData.certificate) { - cloudStack.lbCertificatePolicy.actions.add(lbID, certificateData, complete, error); - } else { - complete(); - } - }, - error: error - }); - } - } - }); - }, - error: function(data) { - args.response.error(parseXMLHttpResponse(data)); - } - });*/ + } $.ajax({ url: createURL('assignToLoadBalancerRule'), @@ -5974,11 +5915,6 @@ docID: 'helpVPNGatewayIKEDH', select: function(args) { var items = []; - // StrongSwan now requires a DH group to be specified... - //items.push({ - // id: '', - // description: _l('label.none') - //}); items.push({ id: 'modp1536', description: 'Group 5(modp1536)' ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services