From 9b60aad3cd447d81d8aa440f64e00b6536abf23d Mon Sep 17 00:00:00 2001
From: Vishnu Vardhan V <vishnuvv@juniper.net>
Date: Tue, 9 Aug 2016 14:28:10 +0530
Subject: [PATCH] Closes-Bug:#1610713 Triming the operand1, operand2, variables
 in config alarms & pattern in userdefind counters.

Change-Id: I5863e70168d0814260619c6182c6e3597237dd0c
---
 webroot/config/alarm/common/ui/js/models/AlarmModel.js |  5 ++++-
 .../ui/js/models/userDefinedCountersModel.js           | 10 +++++++---
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/webroot/config/alarm/common/ui/js/models/AlarmModel.js b/webroot/config/alarm/common/ui/js/models/AlarmModel.js
index 14f088d4b..a3f1460c6 100644
--- a/webroot/config/alarm/common/ui/js/models/AlarmModel.js
+++ b/webroot/config/alarm/common/ui/js/models/AlarmModel.js
@@ -78,13 +78,16 @@ define([
                         var vars = andRuleObj.variables()();
                         if (vars != null && vars != "" && typeof vars == 'string') {
                             vars = vars.split(',')
+                            vars = $.map(vars, $.trim);
                         } else if (vars == "") {
                             vars = [];
                         }
                         var operation = andRuleObj.operation()();
                             operationArr = operation.split(cowc.DROPDOWN_VALUE_SEPARATOR),
                             operand2 = andRuleObj.operand2()(),
+                            operand1 = andRuleObj.operand1()(),
                             operand2Obj = {};
+                            operand2 = operand2.toString().trim();
                         if (operationArr[1] == 'uve_attribute') {
                             operand2Obj = {
                                 uve_attribute: operand2
@@ -95,7 +98,7 @@ define([
                             }
                         }
                         andRulePostObjArr.push({
-                            operand1: andRuleObj.operand1()(),
+                            operand1: operand1.toString().trim(),
                             operand2: operand2Obj,
                             operation: operationArr[0],
                             variables: vars
diff --git a/webroot/config/infra/globalconfig/ui/js/models/userDefinedCountersModel.js b/webroot/config/infra/globalconfig/ui/js/models/userDefinedCountersModel.js
index 4aba9d76a..d92ff31d5 100644
--- a/webroot/config/infra/globalconfig/ui/js/models/userDefinedCountersModel.js
+++ b/webroot/config/infra/globalconfig/ui/js/models/userDefinedCountersModel.js
@@ -133,6 +133,7 @@ define([
             if(self.model().isValid(true, "userDefinedCounterValidation")) {
               newUserDefinedCounterConfig =
                     $.extend({}, true, self.model().attributes);
+              newUserDefinedCounterConfig['pattern'] = newUserDefinedCounterConfig['pattern'].toString().trim();
               ctwu.deleteCGridData(newUserDefinedCounterConfig);
                 //Fetch the current global-system-config
               var deferredObj = $.Deferred();
@@ -147,16 +148,19 @@ define([
                        currentUserDefinedCounterList.push(newUserDefinedCounterConfig);
                    }
                    else{
+                       var found = false;
                        for (var i = 0; i < currentUserDefinedCounterList.length; i++) {
                            if (currentUserDefinedCounterList[i]['name'] ==
                                newUserDefinedCounterConfig['name']) {
+                               found = true;
                                currentUserDefinedCounterList[i]['pattern'] =
                                    newUserDefinedCounterConfig['pattern'];
-                           } else {
-                               currentUserDefinedCounterList.
-                                   push(newUserDefinedCounterConfig);
                            }
                        }
+                       if (!found) {
+                           currentUserDefinedCounterList.
+                               push(newUserDefinedCounterConfig);
+                       }
                    }
                    self.configureUserDefinedCounter(callbackObj,
                            currentGlobalSystemConfigData,