feat(query): add support for express array parsing

Express.js and Hapi.js use the "qs" query string parser module. This
module does a lot of magic including removing trailing `[]` from end
of array key names.

Related: https://github.com/hapijs/qs

Author: Hans Kristian Flaatten

index.js

@@ -11,7 +11,7 @@ module.exports = function MongoQS(opts) {
   this.custom = opts.custom || {};
 
   this.keyRegex = opts.keyRegex || /^[a-zæøå0-9-_.]+$/i;
-  this.arrRegex = opts.arrRegex || /^[a-zæøå0-9-_.]+\[\]$/i;
+  this.arrRegex = opts.arrRegex || /^[a-zæøå0-9-_.]+(\[\])?$/i;
 
   for (var param in this.custom) {
     switch (param) {
@@ -129,8 +129,9 @@ module.exports.prototype.parse = function(query) {
 
     // array key
     if (val instanceof Array && this.arrRegex.test(key)) {
-      if (this.ops.indexOf('$in') >= 0) {
-        key = key.substr(0, key.length-2);
+      if (this.ops.indexOf('$in') >= 0 && val.length > 0) {
+        // remove [] at end of key name (unless it has already been removed)
+        key = key.replace(/\[\]$/, '');
 
         // $in query
         if (val[0][0] !== '!') {

package.json

@@ -43,6 +43,7 @@
   "devDependencies": {
     "jshint": "^2.8.0",
     "mocha": "^2.3.4",
+    "qs": "^5.2.0",
     "semantic-release": "^4.3.5"
   }
 }

test.js

@@ -299,6 +299,17 @@ describe('parse()', function() {
         });
       });
 
+      it('returns in array query with "qs" parser module (GH-06)', function() {
+        var string = 'foo[]=10&foo[]=10.011&foo[]=bar';
+        var params = require('qs').parse(string);
+
+        assert.deepEqual(qs.parse(params), {
+          foo: {
+            $in: [10, 10.011, 'bar']
+          }
+        });
+      });
+
       it('returns in array without any not in array query', function() {
         var string = 'foo[]=10&foo[]=!10.011&foo[]=!bar&foo[]=baz';
         var params = require('querystring').parse(string);
@@ -321,6 +332,18 @@ describe('parse()', function() {
         });
       });
 
+      it('returns not in array query with "gs" parser module (GH-06)', function() {
+        var string = 'foo[]=!10&foo[]=!10.011&foo[]=!bar';
+        var params = require('qs').parse(string);
+
+        assert.deepEqual(qs.parse(params), {
+          foo: {
+            $nin: [10, 10.011, 'bar']
+          }
+        });
+      });
+
+
       it('returns not in array without any in array query', function() {
         var string = 'foo[]=!10&foo[]=10.011&foo[]=bar&foo[]=!baz';
         var params = require('querystring').parse(string);