Merge pull request #19 from catorda/master

Add support for more than one value per key

Author: joshghent

docs/QueryString.html

@@ -176,7 +176,7 @@ <h4 class="name" id="extract">
 
             <div class="container-source members">
                 <code><a href="index.js.html">index.js</a></code>,
-                <code><a href="index.js.html#line102">line 102</a></code>
+                <code><a href="index.js.html#line116">line 116</a></code>
             </div>
 
         </h4>

docs/index.html

@@ -55,7 +55,7 @@ <h1><a href="index.html" class="link">QueryString</a></h1>
 
 
     <section>
-        <article class="readme"><h1>Query-Stringifier <a href="https://travis-ci.org/joshghent/query-stringifier"><img src="https://travis-ci.org/joshghent/query-stringifier.svg?branch=master" alt="Build Status"></a> <a href="https://badge.fury.io/js/query-stringifier"><img src="https://badge.fury.io/js/query-stringifier.svg" alt="npm version"></a> <img src="https://david-dm.org/joshghent/query-stringifier.svg" alt="devDependencies"></h1><p>A small library for build query strings</p>
+        <article class="readme"><h1>Query-Stringifier <a href="https://travis-ci.org/joshghent/query-stringifier"><img src="https://travis-ci.org/joshghent/query-stringifier.svg?branch=master" alt="Build Status"></a> <a href="https://badge.fury.io/js/query-stringifier"><img src="https://badge.fury.io/js/query-stringifier.svg" alt="npm version"></a> <a href="https://greenkeeper.io/"><img src="https://badges.greenkeeper.io/joshghent/query-stringifier.svg" alt="Greenkeeper badge"></a> <img src="https://david-dm.org/joshghent/query-stringifier.svg" alt="devDependencies"></h1><p>A small library for build query strings</p>
 <h2>Install</h2><pre class="prettyprint source lang-bash"><code>$ npm install query-stringifier</code></pre><h2>Usage</h2><pre class="prettyprint source lang-js"><code>
 const qs = require('query-stringifier');
 

docs/index.js.html

@@ -132,10 +132,24 @@ <h1><a href="index.html" class="link">QueryString</a></h1>
                     obj[key][indexOfArray] = value;
                 } else {
                     obj[key] = [value];
-                }
+                }   
             }
         } else {
-            obj[key] = value;
+            // Check if the key is already in the object 
+            if (Object.keys(obj).indexOf(key) > -1) {
+                if (Array.isArray(obj[key])) {
+                    obj[key].push(value); 
+                } else {
+                    let newArray = []; 
+                    newArray.push(obj[key]); 
+                    newArray.push(value);
+                    obj[key] = newArray; 
+                }
+
+            } else {
+                obj[key] = value;
+            }
+            
         }
     });
 

index.js

@@ -84,10 +84,24 @@ QueryString.prototype.parse = function (queryStr) {
                     obj[key][indexOfArray] = value;
                 } else {
                     obj[key] = [value];
-                }
+                }   
             }
         } else {
-            obj[key] = value;
+            // Check if the key is already in the object 
+            if (Object.keys(obj).indexOf(key) > -1) {
+                if (Array.isArray(obj[key])) {
+                    obj[key].push(value); 
+                } else {
+                    let newArray = []; 
+                    newArray.push(obj[key]); 
+                    newArray.push(value);
+                    obj[key] = newArray; 
+                }
+
+            } else {
+                obj[key] = value;
+            }
+            
         }
     });
 

test.js

@@ -59,6 +59,12 @@ describe('query-stringifier', function () {
             expect(parsed).to.have.keys(['arr']);
             expect(parsed).to.deep.equal({ arr: ['2', '3', '1'] });
         });
+        it('converts query string with duplicate keys', function() {
+            var parsed = qs.parse('itemIds=1&itemIds=2'); 
+            expect(parsed).to.be.an('object'); 
+            expect(parsed).to.have.keys(['itemIds']); 
+            expect(parsed).to.deep.equal({ itemIds: ['1', '2'] })
+        })
     });
 
     describe('#extract', function () {