Merge pull request #69 from recurly/convert-array-to-csv

Convert Array params to CSV strings

.bumpversion.cfg

@@ -1,5 +1,5 @@
 [bumpversion]
-current_version = 3.2.0
+current_version = 3.2.1
 parse = (?P<major>\d+)
 	\.(?P<minor>\d+)
 	\.(?P<patch>\d+)

doc/GETTING_STARTED.md

@@ -15,7 +15,7 @@ Or manually insert the dependency into the `dependencies` section of your `packa
 ```
 {
   // ...
-  "recurly" : "^3.2.0"
+  "recurly" : "^3.2.1"
   // ...
 }
 ```

lib/recurly/Pager.js

@@ -10,7 +10,7 @@ class Pager {
   constructor (client, path, params) {
     this.client = client
     this.path = path
-    this.params = params
+    this.params = this._mapArrayParams(params)
   }
 
   /**
@@ -123,6 +123,20 @@ class Pager {
     this._paramsConsumed = true
     return this.params
   }
+
+  // Converts array parameters to CSV strings to maintain consistency with
+  // how the server expects the request to be formatted while providing the
+  // developer with an array type to maintain developer happiness!
+  _mapArrayParams (params) {
+    return Object.keys(params).reduce((res, key) => {
+      if (Array.isArray(params[key])) {
+        res[key] = params[key].join(',')
+      } else {
+        res[key] = params[key]
+      }
+      return res
+    }, {})
+  }
 }
 
 module.exports = Pager

package.json

@@ -1,6 +1,6 @@
 {
   "name": "recurly",
-  "version": "3.2.0",
+  "version": "3.2.1",
   "description": "Recurly V3 node client",
   "main": "lib/recurly.js",
   "scripts": {