From 202e57a772017488293e0fcf183ae762439dbaaa Mon Sep 17 00:00:00 2001 From: Jessica Wang Date: Mon, 26 Aug 2013 15:59:18 -0700 Subject: [PATCH] CLOUDSTACK-4508: UI > Infrastructure > clusters > add cluster dialog > make NexusVSM fields required only when Override Traffic is checked AND vSwitch Type is "Cisco Nexus 1000v Distributed Virtual Switch". --- ui/scripts/system.js | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/ui/scripts/system.js b/ui/scripts/system.js index e6bf03a2246..07b0bb8fcd3 100644 --- a/ui/scripts/system.js +++ b/ui/scripts/system.js @@ -10292,8 +10292,13 @@ // // ** This is done by switching out optional fields for required fields; // need to check for *either* vsm[...]_req or vsm[...] - var $overrideGuestTraffic = $form.find('.form-item[rel=overrideguesttraffic] input[type=checkbox]'); + var $overridePublicTraffic = $form.find('.form-item[rel=overridepublictraffic] input[type=checkbox]'); + var $vSwitchPublicType = $form.find('.form-item[rel=vSwitchPublicType] select'); + + var $overrideGuestTraffic = $form.find('.form-item[rel=overrideguesttraffic] input[type=checkbox]'); + var $vSwitchGuestType = $form.find('.form-item[rel=vSwitchGuestType] select'); + var $vsmFields = $form.find('.form-item').filter(function() { var vsmFields = [ 'vsmipaddress', @@ -10314,8 +10319,8 @@ }); - if ($overridePublicTraffic.is(':checked') || - $overrideGuestTraffic.is(':checked')) { + if (($overridePublicTraffic.is(':checked') && $vSwitchPublicType.val() == 'nexusdvs') || + ($overrideGuestTraffic.is(':checked') && $vSwitchGuestType.val() == 'nexusdvs' )) { $vsmReqFields.css('display', 'inline-block'); $vsmFields.hide(); } else {