Fixing bugs in the Pager

recurly · Apr 21, 2020 · 5b86c2f · 5b86c2f
1 parent 97e2656
commit 5b86c2f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/recurly/BaseClient.js b/lib/recurly/BaseClient.js
@@ -126,7 +126,7 @@ class BaseClient {
     const jsonContent = (resp.contentType && resp.contentType.includes(JSON_CONTENT_TYPE))
 
     if (resp.status < 200 || resp.status > 299) {
-      if (jsonContent) {
+      if (jsonContent && resp.body) {
         const errBody = resp.body && JSON.parse(resp.body).error
         // If we have a body, we determine the error based on
         // the contents of the body

diff --git a/lib/recurly/Pager.js b/lib/recurly/Pager.js
@@ -37,7 +37,7 @@ class Pager {
    * @return {Number} The count of resources
    */
   async count () {
-    const empty = await this.client._makeRequest('HEAD', this.path, null, { params: this._consumeParams() })
+    const empty = await this.client._makeRequest('HEAD', this.path, null, { params: this.params })
     return empty.getResponse().recordCount
   }
 
@@ -51,7 +51,7 @@ class Pager {
   async first () {
     this.params = this.params || {}
     this.params['limit'] = 1
-    let results = await this.client._makeRequest('GET', this.path, null, { params: this._consumeParams() })
+    let results = await this.client._makeRequest('GET', this.path, null, { params: this.params })
     return results.data && results.data[0]
   }