From 791986268ae9cabebfe9f7b7c4d5761f5c8d4fc7 Mon Sep 17 00:00:00 2001
From: Jessica Wang <jessica.wang@citrix.com>
Date: Thu, 21 Jun 2012 13:48:11 -0700
Subject: [PATCH] (1) CS-15311: cloudstack 3.0 UI - Add Zone Wizard - add
 physical network step - no isolation method dropdown if network type is
 Basic.

(2) CS-15312 - cloudstack 3.0 UI - Add Zone Wizard - add physical network - remove "L3" option from isolation method.
---
 ui/scripts/ui-custom/zoneWizard.js | 53 +++++++++++++++++++-----------
 1 file changed, 34 insertions(+), 19 deletions(-)

diff --git a/ui/scripts/ui-custom/zoneWizard.js b/ui/scripts/ui-custom/zoneWizard.js
index 3a727149234..8f1a87297ab 100644
--- a/ui/scripts/ui-custom/zoneWizard.js
+++ b/ui/scripts/ui-custom/zoneWizard.js
@@ -560,31 +560,46 @@
             .attr({ title: 'Remove this physical network' })
             .append('<span>').addClass('icon').html('&nbsp;');
       var $icon = $('<div>').addClass('physical-network-icon');
-      var $nameField = $('<div>').addClass('field name').append(
+      
+			var $nameField = $('<div>').addClass('field name').append(
         $('<div>').addClass('name').append(
           $('<label>').html('Physical network name')
 				),
 				$('<div>').addClass('value').append(
 					$('<input>').attr({ type: 'text' }).addClass('required')
-        ),
-				$('<div>').append(
-          $('<span style=\"font-size:11px\;padding-right:5px;padding-left:50px">').html('Isolation method'),
-					$('<select>').append(						
-					  $('<option>').attr({
-							value: ''
-						}).html(''),					
-						$('<option>').attr({
-							value: 'VLAN'
-						}).html('VLAN'),						
-						$('<option>').attr({
-							value: 'L3'
-						}).html('L3'),						
-						$('<option>').attr({
-							value: 'GRE'
-						}).html('GRE')	
+        )				
+      );			
+			if($wizard.find('.select-network-model input:radio[name=network-model]:checked').val() == 'Advanced') {
+				$nameField.append(
+					$('<div>').addClass('name').append(
+						$('<label>').html('Physical network name')
+					),
+					$('<div>').addClass('value').append(
+						$('<input>').attr({ type: 'text' }).addClass('required')
+					),
+					$('<div>').append(
+						$('<span style=\"font-size:11px\;padding-right:5px;padding-left:50px">').html('Isolation method'),
+						$('<select>').append(						
+							$('<option>').attr({
+								value: ''
+							}).html(''),					
+							$('<option>').attr({
+								value: 'VLAN'
+							}).html('VLAN'),						
+							//User should not be given the option to create a physical network with "L3" isolation method. (CS-15312)
+							/*
+							$('<option>').attr({
+								value: 'L3'
+							}).html('L3'),		
+							*/						
+							$('<option>').attr({
+								value: 'GRE'
+							}).html('GRE')	
+						)
 					)
-        )
-      );
+				);
+			}
+			
       var $dropContainer = $('<div>').addClass('drop-container').append(
         $('<span>').addClass('empty-message').html(
           'Drag and drop traffic types you would like to add here.'