Author:   Lars Michelsen <[email protected]>
Date:     Fri Mar 30 09:33:17 2012 +0200
Committer:   Lars Michelsen <[email protected]>
Commit-Date: Fri Mar 30 09:33:17 2012 +0200

Fixed makeuri function not to create duplicate params anymore

---

 share/frontend/nagvis-js/js/nagvis.js |   16 +++++++++++-----
 1 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/share/frontend/nagvis-js/js/nagvis.js 
b/share/frontend/nagvis-js/js/nagvis.js
index 7fed049..bdf5c46 100644
--- a/share/frontend/nagvis-js/js/nagvis.js
+++ b/share/frontend/nagvis-js/js/nagvis.js
@@ -419,21 +419,27 @@ function makeuri(addvars) {
     tmp = tmp[1].split('#');
     tmp = tmp[0].split('&');
     var len = tmp.length;
-    var params = [];
+    var params = {};
     var pair = null;
 
     // Skip unwanted parmas
     for(var i = 0; i < tmp.length; i++) {
         pair = tmp[i].split('=');
-        params.push(tmp[i]);
+        params[pair[0]] = pair[1];
     }
 
-    // Add new params
+    // Add new params to the existing params. Overwrite duplicates
     for (var key in addvars) {
-        params.push(key + '=' + addvars[key]);
+        params[key] = addvars[key];
     }
 
-    return base + '?' + params.join('&')
+    // Build list of key/value pairs
+    var aparams = [];
+    for (var key in params) {
+        aparams.push(key + '=' + params[key]);
+    }
+
+    return base + '?' + aparams.join('&')
 }
 
 /**


------------------------------------------------------------------------------
Better than sec? Nothing is better than sec when it comes to
monitoring Big Data applications. Try Boundary one-second 
resolution app monitoring today. Free.
http://p.sf.net/sfu/Boundary-dev2dev
_______________________________________________
Nagvis-checkins mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/nagvis-checkins

Reply via email to