fixing addQuery() to work with null value - closes #171

README.md

@@ -242,6 +242,7 @@ URI.js is published under the [MIT license](http://www.opensource.org/licenses/m
 
 * fixing handling of String instances (not string primitives) - ([Issue #146](https://github.com/medialize/URI.js/issues/146))
 * fixing Firefox [`.watch()`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/watch) interfering with `.parseQuery()` - ([Issue #169](https://github.com/medialize/URI.js/issues/169))
+* fixing [`addQuery()`](http://medialize.github.io/URI.js/docs.html#search-add) to not throw error on null value - ([Issue #171](https://github.com/medialize/URI.js/issues/171))
 
 ### 1.14.0 (September 8th 2014) ###
 

src/URI.js

@@ -660,7 +660,7 @@
         value = [value];
       }
 
-      data[name] = data[name].concat(value);
+      data[name] = (data[name] || []).concat(value);
     } else {
       throw new TypeError('URI.addQuery() accepts an object, string as the name parameter');
     }

test/test.js

@@ -765,6 +765,10 @@
     u.addQuery('empty', '');
     equal(u.query(), 'foo=bar&empty=', 'add empty string');
 
+    u.query('?foo');
+    u.addQuery('foo', 'bar');
+    equal(u.query(), 'foo=bar', 'add to null value');
+
     u.query('');
     u.addQuery('some value', 'must be encoded because of = and ? and #');
     equal(u.query(), 'some+value=must+be+encoded+because+of+%3D+and+%3F+and+%23', 'encoding');