Return-Path: X-Original-To: apmail-cloudstack-commits-archive@www.apache.org Delivered-To: apmail-cloudstack-commits-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 4F08AF176 for ; Fri, 29 Mar 2013 18:20:18 +0000 (UTC) Received: (qmail 2956 invoked by uid 500); 29 Mar 2013 18:20:17 -0000 Delivered-To: apmail-cloudstack-commits-archive@cloudstack.apache.org Received: (qmail 2917 invoked by uid 500); 29 Mar 2013 18:20:17 -0000 Mailing-List: contact commits-help@cloudstack.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cloudstack.apache.org Delivered-To: mailing list commits@cloudstack.apache.org Received: (qmail 2897 invoked by uid 99); 29 Mar 2013 18:20:17 -0000 Received: from tyr.zones.apache.org (HELO tyr.zones.apache.org) (140.211.11.114) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 29 Mar 2013 18:20:17 +0000 Received: by tyr.zones.apache.org (Postfix, from userid 65534) id 384888340B5; Fri, 29 Mar 2013 18:20:17 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: prachidamle@apache.org To: commits@cloudstack.apache.org Date: Fri, 29 Mar 2013 18:20:19 -0000 Message-Id: <44a577eb565d4d1ca4ce3fd912e87a95@git.apache.org> In-Reply-To: <249912a2528342cc981002e54fe8ed15@git.apache.org> References: <249912a2528342cc981002e54fe8ed15@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: [03/50] [abbrv] git commit: updated refs/heads/affinity_groups to 857ed16 CLOUDSTACK-452: cloudstack UI - Infrastructure menu - zone detail - guest network - add guest network - validation - Either IPv4 fields or IPv6 fields need to be filled when adding a guest network. Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/e806ff67 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/e806ff67 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/e806ff67 Branch: refs/heads/affinity_groups Commit: e806ff674397b90e7b8c99ed3cca64f0c2af1922 Parents: 26b892d Author: Jessica Wang Authored: Tue Mar 26 16:35:10 2013 -0700 Committer: Jessica Wang Committed: Tue Mar 26 16:35:42 2013 -0700 ---------------------------------------------------------------------- ui/scripts/system.js | 61 ++++++++++++--------------------------------- 1 files changed, 16 insertions(+), 45 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/e806ff67/ui/scripts/system.js ---------------------------------------------------------------------- diff --git a/ui/scripts/system.js b/ui/scripts/system.js index c0a5d14..830d622 100644 --- a/ui/scripts/system.js +++ b/ui/scripts/system.js @@ -1350,42 +1350,7 @@ var $form = $(this).closest("form"); var selectedNetworkOfferingId = $(this).val(); $(networkOfferingObjs).each(function(){ - if(this.id == selectedNetworkOfferingId) { - if(this.guestiptype == "Isolated") { //*** Isolated *** - if(this.specifyipranges == false) { - $form.find('.form-item[rel=startipv4]').hide(); - $form.find('.form-item[rel=endipv4]').hide(); - } - else { - $form.find('.form-item[rel=startipv4]').css('display', 'inline-block'); - $form.find('.form-item[rel=endipv4]').css('display', 'inline-block'); - } - - var includingSourceNat = false; - var serviceObjArray = this.service; - for(var k = 0; k < serviceObjArray.length; k++) { - if(serviceObjArray[k].name == "SourceNat") { - includingSourceNat = true; - break; - } - } - if(includingSourceNat == true) { //Isolated with SourceNat - cloudStack.dialog.createFormField.validation.required.remove($form.find('.form-item[rel=ip4gateway]')); //make ip4gateway optional - cloudStack.dialog.createFormField.validation.required.remove($form.find('.form-item[rel=ip4Netmask]')); //make ip4Netmask optional - } - else { //Isolated with no SourceNat - cloudStack.dialog.createFormField.validation.required.add($form.find('.form-item[rel=ip4gateway]')); //make ip4gateway required - cloudStack.dialog.createFormField.validation.required.add($form.find('.form-item[rel=ip4Netmask]')); //make ip4Netmask required - } - } - else { //*** Shared *** - $form.find('.form-item[rel=startipv4]').css('display', 'inline-block'); - $form.find('.form-item[rel=endipv4]').css('display', 'inline-block'); - - cloudStack.dialog.createFormField.validation.required.add($form.find('.form-item[rel=ip4gateway]')); //make ip4gateway required - cloudStack.dialog.createFormField.validation.required.add($form.find('.form-item[rel=ip4Netmask]')); //make ip4Netmask required - } - + if(this.id == selectedNetworkOfferingId) { if(this.specifyvlan == false) { $form.find('.form-item[rel=vlanId]').hide(); cloudStack.dialog.createFormField.validation.required.remove($form.find('.form-item[rel=vlanId]')); //make vlanId optional @@ -1411,13 +1376,11 @@ docID: 'helpGuestNetworkZoneNetmask' }, startipv4: { - label: 'IPv4 Start IP', - validation: { required: true }, + label: 'IPv4 Start IP', docID: 'helpGuestNetworkZoneStartIP' }, endipv4: { - label: 'IPv4 End IP', - validation: { required: true }, + label: 'IPv4 End IP', docID: 'helpGuestNetworkZoneEndIP' }, //IPv4 (end) @@ -1431,13 +1394,11 @@ label: 'IPv6 CIDR' }, startipv6: { - label: 'IPv6 Start IP', - validation: { required: true }, + label: 'IPv6 Start IP', docID: 'helpGuestNetworkZoneStartIP' }, endipv6: { - label: 'IPv6 End IP', - validation: { required: true }, + label: 'IPv6 End IP', docID: 'helpGuestNetworkZoneEndIP' }, //IPv6 (end) @@ -1449,7 +1410,17 @@ } }, - action: function(args) { //Add guest network in advanced zone + action: function(args) { //Add guest network in advanced zone + if ( + ((args.data.ip4gateway.length == 0) && (args.data.ip4Netmask.length == 0) && (args.data.startipv4.length == 0) && (args.data.endipv4.length == 0)) + && + ((args.data.ip6gateway.length == 0) && (args.data.ip6cidr.length == 0) && (args.data.startipv6.length == 0) && (args.data.endipv6.length == 0)) + ) + { + args.response.error("Either IPv4 fields or IPv6 fields need to be filled when adding a guest network"); + return; + } + var $form = args.$form; var array1 = [];