Fix double handleResponse() bug for nextCursorPage (intercom#323)

edudobay · GabrielAnca · web-flow · commit f12b66a44814 · 2020-12-01T09:51:34.000Z
Solves the same issue as intercom#318 but for the nextCursorPage method. intercom#318 Co-authored-by: Gabriel Anca Corral <GabrielAnca@users.noreply.github.com>
diff --git a/src/IntercomClient.php b/src/IntercomClient.php
@@ -296,8 +296,7 @@ public function nextSearchPage(string $path, array $query, $pages)
      */
     public function nextCursorPage(string $path, string $startingAfter)
     {
-        $response = $this->get($path . "?starting_after=" . $startingAfter);
-        return $this->handleResponse($response);
+        return $this->get($path . "?starting_after=" . $startingAfter);
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -296,8 +296,7 @@ public function nextSearchPage(string $path, array $query, $pages)`
`296`	`296`	`*/`
`297`	`297`	`public function nextCursorPage(string $path, string $startingAfter)`
`298`	`298`	`{`
`299`		`- $response = $this->get($path . "?starting_after=" . $startingAfter);`
`300`		`- return $this->handleResponse($response);`
	`299`	`+ return $this->get($path . "?starting_after=" . $startingAfter);`
`301`	`300`	`}`
`302`	`301`
`303`	`302`	`/**`