Merge pull request #37 from johnmarkos/issue/37

replaceQueryParam() with three arguments adds newVal when oldVal missing

Author: derek-watson

Uri.js

@@ -284,7 +284,9 @@
           break;
         }
       }
-      this.deleteQueryParam(key, decode(oldVal)).addQueryParam(key, newVal, index);
+      if (index >= 0) {
+        this.deleteQueryParam(key, decode(oldVal)).addQueryParam(key, newVal, index);
+      }
     } else {
       for (i = 0; i < this.queryPairs.length; i++) {
         param = this.queryPairs[i];

test/uri/query.js

@@ -153,7 +153,7 @@ describe('Uri', function() {
 
       it('should be able to replace a param value with a specified value that does not exist', function() {
         q = new Uri('?page=4&page=2').replaceQueryParam('page', 3, 1)
-        expect(q.toString()).to.equal('?page=4&page=2&page=3')
+        expect(q.toString()).to.equal('?page=4&page=2')
       })
 
       it('should replace a param value with an empty value if not provided a value', function() {