qs 0.5.0 → 0.5.1

package/History.md

@@ -1,4 +1,9 @@
 
+0.5.1 / 2012-09-18 
+==================
+
+  * fix encoded `=`. Closes #43
+
 0.5.0 / 2012-05-04 
 ==================
 

package/component.json

@@ -0,0 +1,6 @@
+{
+  "name": "querystring",
+  "description": "Querystring parser / stringifier with nesting support",
+  "keywords": ["querystring", "query", "parser"],
+  "main": "lib/querystring.js"
+}

package/lib/querystring.js

@@ -99,12 +99,6 @@ function parseString(str){
   return String(str)
     .split('&')
     .reduce(function(ret, pair){
-      try{
-        pair = decodeURIComponent(pair.replace(/\+/g, ' '));
-      } catch(e) {
-        // ignore
-      }
-
       var eql = pair.indexOf('=')
         , brace = lastBraceInKey(pair)
         , key = pair.substr(0, brace || eql)
@@ -114,7 +108,7 @@ function parseString(str){
       // ?foo
       if ('' == key) key = pair, val = '';
 
-      return merge(ret, key, val);
+      return merge(ret, decode(key), decode(val));
     }, { base: {} }).base;
 }
 
@@ -250,3 +244,19 @@ function lastBraceInKey(str) {
     if ('=' == c && !brace) return i;
   }
 }
+
+/**
+ * Decode `str`.
+ *
+ * @param {String} str
+ * @return {String}
+ * @api private
+ */
+
+function decode(str) {
+  try {
+    return decodeURIComponent(str.replace(/\+/g, ' '));
+  } catch (err) {
+    return str;
+  }
+}

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "qs",
   "description": "querystring parser",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "keywords": ["query string", "parser", "component"],
   "repository": {
     "type" : "git",

package/test/parse.js

@@ -28,9 +28,6 @@ describe('qs.parse()', function(){
     expect(qs.parse('foo=bar'))
       .to.eql({ foo: 'bar' });
 
-    expect(qs.parse('foo%3Dbar=baz'))
-      .to.eql({ foo: 'bar=baz' });
-
     expect(qs.parse(' foo = bar = baz '))
       .to.eql({ ' foo ': ' bar = baz ' });
 
@@ -52,6 +49,11 @@ describe('qs.parse()', function(){
       });
   })
 
+  it('should support encoded = signs', function(){
+    expect(qs.parse('he%3Dllo=th%3Dere'))
+      .to.eql({ 'he=llo': 'th=ere' });
+  })
+
   it('should support nesting', function(){
     expect(qs.parse('ops[>=]=25'))
       .to.eql({ ops: { '>=': '25' }});