From 732f12a278b882dd1a7fd92ca440307d95d98c9d Mon Sep 17 00:00:00 2001
From: Siva Bavanasi <kbsiva@juniper.net>
Date: Fri, 15 Apr 2016 14:26:23 +0530
Subject: [PATCH] Closes-bug: #1552959 - Allow network static routes to be
 tagged with communities

Change-Id: I05c1689c2f57cb613d96f434624e7af2247b0592
---
 .../routetable/ui/js/models/RtTableModel.js   | 29 ++++++-----
 .../routetable/ui/js/views/RtTableEditView.js | 49 ++++++++++---------
 2 files changed, 40 insertions(+), 38 deletions(-)

diff --git a/webroot/config/networking/routetable/ui/js/models/RtTableModel.js b/webroot/config/networking/routetable/ui/js/models/RtTableModel.js
index 18c38fa5b..77afdb1e4 100644
--- a/webroot/config/networking/routetable/ui/js/models/RtTableModel.js
+++ b/webroot/config/networking/routetable/ui/js/models/RtTableModel.js
@@ -87,25 +87,24 @@ define([
                 if(type === "route-table") {
                     obj["next_hop"] = nextHop;
                     obj["next_hop_type"] = nextHopType
-                } else {
-                    var commAttrs = routesCollection[i].community_attr();
-                    var arr = commAttrs.split('\n');
-                    var len = arr.length;
-                    commAttrArr = [];
-                    for (var j = 0; j < len; j++) {
-                        var tmpArr = arr[j].split(',');
-                        if (tmpArr.length > 0) {
-                            var arrLen = tmpArr.length;
-                            for (var k = 0; k < arrLen; k++) {
-                                if (tmpArr[k].length > 0) {
-                                    commAttrArr.push(tmpArr[k].trim());
-                                }
+                }
+                var commAttrs = routesCollection[i].community_attr();
+                var arr = commAttrs.split('\n');
+                var len = arr.length;
+                commAttrArr = [];
+                for (var j = 0; j < len; j++) {
+                    var tmpArr = arr[j].split(',');
+                    if (tmpArr.length > 0) {
+                        var arrLen = tmpArr.length;
+                        for (var k = 0; k < arrLen; k++) {
+                            if (tmpArr[k].length > 0) {
+                                commAttrArr.push(tmpArr[k].trim());
                             }
                         }
                     }
-                    obj['community_attributes'] = {};
-                    obj['community_attributes']['community_attribute'] = commAttrArr;
                 }
+                obj['community_attributes'] = {};
+                obj['community_attributes']['community_attribute'] = commAttrArr;
                 routesArr.push(obj);
             }
             return routesArr;
diff --git a/webroot/config/networking/routetable/ui/js/views/RtTableEditView.js b/webroot/config/networking/routetable/ui/js/views/RtTableEditView.js
index 08c7a6c36..462b650b8 100644
--- a/webroot/config/networking/routetable/ui/js/views/RtTableEditView.js
+++ b/webroot/config/networking/routetable/ui/js/views/RtTableEditView.js
@@ -119,7 +119,7 @@ define([
                                 name: 'Next Hop Type',
                                 viewConfig: {
                                     placeholder: 'Next Hop Type',
-                                    width: 200,
+                                    width: 150,
                                     templateId: cowc.TMPL_EDITABLE_GRID_DROPDOWN_VIEW,
                                     path: 'next_hop_type',
                                     dataBindValue: 'next_hop_type()',
@@ -133,39 +133,41 @@ define([
                                     }
                                 }
                             };
-   viewConfigNextHop = {
+    viewConfigNextHop = {
                                 elementId: 'next_hop',
                                 view: 'FormInputView',
                                 class: "",
                                 name: 'Next Hop',
                                 viewConfig: {
                                     placeholder: 'Enter IP Address',
-                                    width: 200,
+                                    width: 250,
                                     templateId: cowc.TMPL_EDITABLE_GRID_INPUT_VIEW,
                                     path: 'next_hop',
                                     dataBindValue: 'next_hop()'
                                 }
                         };
 
-   viewConfigCommunityAttr = {
-                                elementId: 'community_attr',
-                                view: 'FormMultiselectView',
-                                class: "",
-                                name: 'Communities',
-                                viewConfig: {
-                                    width: 400,
-                                    templateId: cowc.TMPL_EDITABLE_GRID_MULTISELECT_VIEW,
-                                    path: 'community_attr',
-                                    dataBindValue: 'community_attr()',
-                                    elementConfig: {
-                                        placeholder: 'Select or Enter Communities',
-                                        dataTextField: "text",
-                                        dataValueField: "id",
-                                        data : ctwc.DEFAULT_COMMUNITIES,
-                                        tags: true
-                                    }
-                                }
-                            };
+    viewConfigCommunityAttr = function(width) {
+        return {
+            elementId: 'community_attr',
+            view: 'FormMultiselectView',
+            class: "",
+            name: 'Communities',
+            viewConfig: {
+                width: width,
+                templateId: cowc.TMPL_EDITABLE_GRID_MULTISELECT_VIEW,
+                path: 'community_attr',
+                dataBindValue: 'community_attr()',
+                elementConfig: {
+                    placeholder: 'Select or Enter Communities',
+                    dataTextField: "text",
+                    dataValueField: "id",
+                    data : ctwc.DEFAULT_COMMUNITIES,
+                    tags: true
+                }
+            }
+        };
+    };
 
     function getRouteTableColViewConfigs() {
         var columns = [];
@@ -173,8 +175,9 @@ define([
         if(rtTableType === "route-table") {
             columns.push(viewConfigNextHopType);
             columns.push(viewConfigNextHop);
+            columns.push(viewConfigCommunityAttr(200));
         } else {
-            columns.push(viewConfigCommunityAttr);
+            columns.push(viewConfigCommunityAttr(400));
         }
         return columns;
     }