qs 0.4.1 → 0.4.2

package/History.md

@@ -1,4 +1,9 @@
 
+0.4.2 / 2012-02-08 
+==================
+
+  * Fixed: ensure objects are created when appropriate not arrays [aheckmann]
+
 0.4.1 / 2012-01-26 
 ==================
 

package/lib/querystring.js

@@ -9,7 +9,7 @@
  * Library version.
  */
 
-exports.version = '0.4.1';
+exports.version = '0.4.2';
 
 /**
  * Object#toString() ref for stringify().
@@ -21,7 +21,7 @@ var toString = Object.prototype.toString;
  * Cache non-integer test regexp.
  */
 
-var notint = /^[^0-9]+$/;
+var isint = /^[0-9]+$/;
 
 function promote(parent, key) {
   if (parent[key].length == 0) return parent[key] = {};
@@ -58,11 +58,11 @@ function parse(parts, parent, key, val) {
       // prop
     } else if (~part.indexOf(']')) {
       part = part.substr(0, part.length - 1);
-      if(notint.test(part) && Array.isArray(obj)) obj = promote(parent, key);
+      if (!isint.test(part) && Array.isArray(obj)) obj = promote(parent, key);
       parse(parts, obj, part, val);
       // key
     } else {
-      if(notint.test(part) && Array.isArray(obj)) obj = promote(parent, key);
+      if (!isint.test(part) && Array.isArray(obj)) obj = promote(parent, key);
       parse(parts, obj, part, val);
     }
   }
@@ -80,7 +80,7 @@ function merge(parent, key, val){
     parse(parts, parent, 'base', val);
     // optimize
   } else {
-    if (notint.test(key) && Array.isArray(parent.base)) {
+    if (!isint.test(key) && Array.isArray(parent.base)) {
       var t = {};
       for (var k in parent.base) t[k] = parent.base[k];
       parent.base = t;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "qs",
   "description": "querystring parser",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "repository": {
     "type" : "git",
     "url" : "git://github.com/visionmedia/node-querystring.git"

package/test/parse.js

@@ -91,8 +91,12 @@ module.exports = {
 
     qs.parse('user[name][first]=tj&user[name][first]=TJ')
       .should.eql({ user: { name: { first: ['tj', 'TJ'] }}});
+
+    var o = qs.parse('existing[fcbaebfecc][name][last]=tj')
+    o.should.eql({ existing: { 'fcbaebfecc': { name: { last: 'tj' }}}})
+    Array.isArray(o.existing).should.be.false;
   },
-  
+
   'test right-hand brackets': function(){
     qs.parse('pets=["tobi"]')
       .should.eql({ pets: '["tobi"]' });