From cf1fada372622fe610c18d17fdaf1589acc50d03 Mon Sep 17 00:00:00 2001
From: Pranav Saxena <pranav.saxena@citrix.com>
Date: Wed, 15 May 2013 18:01:10 +0530
Subject: [PATCH] CLOUDSTACK-2491:NTier: Creation of ACL Rule for protocol AH
 (51) and ESP (50) Fails

---
 ui/scripts/vpc.js | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/ui/scripts/vpc.js b/ui/scripts/vpc.js
index 5d436f3536a..0c4a446485b 100644
--- a/ui/scripts/vpc.js
+++ b/ui/scripts/vpc.js
@@ -94,8 +94,8 @@
       },
 
       'protocolnumber': {label:'Protocol Number',isDisabled:true,isHidden:true,edit:true},
-      'startport': { edit: true, label: 'label.start.port' },
-      'endport': { edit: true, label: 'label.end.port' },
+      'startport': { edit: true, label: 'label.start.port' , isOptional:true },
+      'endport': { edit: true, label: 'label.end.port' , isOptional:true},
       'networkid': {
         label: 'Select Tier',
         select: function(args) {
@@ -165,7 +165,18 @@
         else
           delete args.data.protocolnumber;
 
+
        
+        if((args.data.protocol == 'tcp' || args.data.protocol == 'udp' || args.data.protocol == 'all') && (args.data.startport=="" || args.data.startport == undefined)){
+         cloudStack.dialog.notice({message:_l('Start Port or End Port value should not be blank')});
+          $(window).trigger('cloudStack.fullRefresh');
+        }
+        else if((args.data.protocol == 'tcp' || args.data.protocol == 'udp' || args.data.protocol == 'all')  && (args.data.endport=="" || args.data.endport == undefined)){
+         cloudStack.dialog.notice({message:_l('Start Port or End Port value should not be blank')});
+          $(window).trigger('cloudStack.fullRefresh');
+        }
+
+       else{       
         $.ajax({
           url: createURL('createNetworkACL'),
           data: $.extend(args.data, {
@@ -202,6 +213,7 @@
           }
         });
       }
+      }
     },
     actions: {
       destroy: {