fix autopagination for Service

lib/BaseStripeClient.php

@@ -139,6 +139,30 @@ public function request($method, $path, $params, $opts)
         return $obj;
     }
 
+    /**
+     * Sends a request to Stripe's API.
+     *
+     * @param string $method the HTTP method
+     * @param string $path the path of the request
+     * @param array $params the parameters of the request
+     * @param array|\Stripe\Util\RequestOptions $opts the special modifiers of the request
+     *
+     * @return \Stripe\Collection of ApiResources
+     */
+    public function requestCollection($method, $path, $params, $opts)
+    {
+        $obj = $this->request($method, $path, $params, $opts);
+        if (!($obj instanceof \Stripe\Collection)) {
+            $received_class = \get_class($obj);
+            $msg = "Expected to receive `Stripe\\Collection` object from Stripe API. Instead received `{$received_class}`.";
+
+            throw new \Stripe\Exception\UnexpectedValueException($msg);
+        }
+        $obj->setFilters($params);
+
+        return $obj;
+    }
+
     /**
      * @param \Stripe\Util\RequestOptions $opts
      *

lib/Service/AbstractService.php

@@ -59,6 +59,11 @@ protected function request($method, $path, $params, $opts)
         return $this->getClient()->request($method, $path, static::formatParams($params), $opts);
     }
 
+    protected function requestCollection($method, $path, $params, $opts)
+    {
+        return $this->getClient()->requestCollection($method, $path, static::formatParams($params), $opts);
+    }
+
     protected function buildPath($basePath, ...$ids)
     {
         foreach ($ids as $id) {

lib/Service/AccountService.php

@@ -17,7 +17,7 @@ class AccountService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/accounts', $params, $opts);
+        return $this->requestCollection('get', '/v1/accounts', $params, $opts);
     }
 
     /**
@@ -35,7 +35,7 @@ public function all($params = null, $opts = null)
      */
     public function allCapabilities($parentId, $params = null, $opts = null)
     {
-        return $this->request('get', $this->buildPath('/v1/accounts/%s/capabilities', $parentId), $params, $opts);
+        return $this->requestCollection('get', $this->buildPath('/v1/accounts/%s/capabilities', $parentId), $params, $opts);
     }
 
     /**
@@ -51,7 +51,7 @@ public function allCapabilities($parentId, $params = null, $opts = null)
      */
     public function allExternalAccounts($parentId, $params = null, $opts = null)
     {
-        return $this->request('get', $this->buildPath('/v1/accounts/%s/external_accounts', $parentId), $params, $opts);
+        return $this->requestCollection('get', $this->buildPath('/v1/accounts/%s/external_accounts', $parentId), $params, $opts);
     }
 
     /**
@@ -69,7 +69,7 @@ public function allExternalAccounts($parentId, $params = null, $opts = null)
      */
     public function allPersons($parentId, $params = null, $opts = null)
     {
-        return $this->request('get', $this->buildPath('/v1/accounts/%s/persons', $parentId), $params, $opts);
+        return $this->requestCollection('get', $this->buildPath('/v1/accounts/%s/persons', $parentId), $params, $opts);
     }
 
     /**

lib/Service/ApplePayDomainService.php

@@ -16,7 +16,7 @@ class ApplePayDomainService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/apple_pay/domains', $params, $opts);
+        return $this->requestCollection('get', '/v1/apple_pay/domains', $params, $opts);
     }
 
     /**

lib/Service/ApplicationFeeService.php

@@ -17,7 +17,7 @@ class ApplicationFeeService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/application_fees', $params, $opts);
+        return $this->requestCollection('get', '/v1/application_fees', $params, $opts);
     }
 
     /**
@@ -37,7 +37,7 @@ public function all($params = null, $opts = null)
      */
     public function allRefunds($parentId, $params = null, $opts = null)
     {
-        return $this->request('get', $this->buildPath('/v1/application_fees/%s/refunds', $parentId), $params, $opts);
+        return $this->requestCollection('get', $this->buildPath('/v1/application_fees/%s/refunds', $parentId), $params, $opts);
     }
 
     /**

lib/Service/BalanceTransactionService.php

@@ -21,7 +21,7 @@ class BalanceTransactionService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/balance_transactions', $params, $opts);
+        return $this->requestCollection('get', '/v1/balance_transactions', $params, $opts);
     }
 
     /**

lib/Service/ChargeService.php

@@ -17,7 +17,7 @@ class ChargeService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/charges', $params, $opts);
+        return $this->requestCollection('get', '/v1/charges', $params, $opts);
     }
 
     /**

lib/Service/Checkout/SessionService.php

@@ -16,7 +16,7 @@ class SessionService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/checkout/sessions', $params, $opts);
+        return $this->requestCollection('get', '/v1/checkout/sessions', $params, $opts);
     }
 
     /**
@@ -35,7 +35,7 @@ public function all($params = null, $opts = null)
      */
     public function allLineItems($parentId, $params = null, $opts = null)
     {
-        return $this->request('get', $this->buildPath('/v1/checkout/sessions/%s/line_items', $parentId), $params, $opts);
+        return $this->requestCollection('get', $this->buildPath('/v1/checkout/sessions/%s/line_items', $parentId), $params, $opts);
     }
 
     /**

lib/Service/CountrySpecService.php

@@ -16,7 +16,7 @@ class CountrySpecService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/country_specs', $params, $opts);
+        return $this->requestCollection('get', '/v1/country_specs', $params, $opts);
     }
 
     /**

lib/Service/CouponService.php

@@ -16,7 +16,7 @@ class CouponService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/coupons', $params, $opts);
+        return $this->requestCollection('get', '/v1/coupons', $params, $opts);
     }
 
     /**

lib/Service/CreditNoteService.php

@@ -16,7 +16,7 @@ class CreditNoteService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/credit_notes', $params, $opts);
+        return $this->requestCollection('get', '/v1/credit_notes', $params, $opts);
     }
 
     /**
@@ -34,7 +34,7 @@ public function all($params = null, $opts = null)
      */
     public function allLines($parentId, $params = null, $opts = null)
     {
-        return $this->request('get', $this->buildPath('/v1/credit_notes/%s/lines', $parentId), $params, $opts);
+        return $this->requestCollection('get', $this->buildPath('/v1/credit_notes/%s/lines', $parentId), $params, $opts);
     }
 
     /**

lib/Service/CustomerService.php

@@ -17,7 +17,7 @@ class CustomerService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/customers', $params, $opts);
+        return $this->requestCollection('get', '/v1/customers', $params, $opts);
     }
 
     /**
@@ -34,7 +34,7 @@ public function all($params = null, $opts = null)
      */
     public function allBalanceTransactions($parentId, $params = null, $opts = null)
     {
-        return $this->request('get', $this->buildPath('/v1/customers/%s/balance_transactions', $parentId), $params, $opts);
+        return $this->requestCollection('get', $this->buildPath('/v1/customers/%s/balance_transactions', $parentId), $params, $opts);
     }
 
     /**
@@ -50,7 +50,7 @@ public function allBalanceTransactions($parentId, $params = null, $opts = null)
      */
     public function allSources($parentId, $params = null, $opts = null)
     {
-        return $this->request('get', $this->buildPath('/v1/customers/%s/sources', $parentId), $params, $opts);
+        return $this->requestCollection('get', $this->buildPath('/v1/customers/%s/sources', $parentId), $params, $opts);
     }
 
     /**
@@ -66,7 +66,7 @@ public function allSources($parentId, $params = null, $opts = null)
      */
     public function allTaxIds($parentId, $params = null, $opts = null)
     {
-        return $this->request('get', $this->buildPath('/v1/customers/%s/tax_ids', $parentId), $params, $opts);
+        return $this->requestCollection('get', $this->buildPath('/v1/customers/%s/tax_ids', $parentId), $params, $opts);
     }
 
     /**

lib/Service/DisputeService.php

@@ -16,7 +16,7 @@ class DisputeService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/disputes', $params, $opts);
+        return $this->requestCollection('get', '/v1/disputes', $params, $opts);
     }
 
     /**

lib/Service/EventService.php

@@ -20,7 +20,7 @@ class EventService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/events', $params, $opts);
+        return $this->requestCollection('get', '/v1/events', $params, $opts);
     }
 
     /**

lib/Service/ExchangeRateService.php

@@ -17,7 +17,7 @@ class ExchangeRateService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/exchange_rates', $params, $opts);
+        return $this->requestCollection('get', '/v1/exchange_rates', $params, $opts);
     }
 
     /**

lib/Service/FileLinkService.php

@@ -16,7 +16,7 @@ class FileLinkService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/file_links', $params, $opts);
+        return $this->requestCollection('get', '/v1/file_links', $params, $opts);
     }
 
     /**

lib/Service/FileService.php

@@ -18,7 +18,7 @@ class FileService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/files', $params, $opts);
+        return $this->requestCollection('get', '/v1/files', $params, $opts);
     }
 
     /**

lib/Service/InvoiceItemService.php

@@ -17,7 +17,7 @@ class InvoiceItemService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/invoiceitems', $params, $opts);
+        return $this->requestCollection('get', '/v1/invoiceitems', $params, $opts);
     }
 
     /**

lib/Service/InvoiceService.php

@@ -18,7 +18,7 @@ class InvoiceService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/invoices', $params, $opts);
+        return $this->requestCollection('get', '/v1/invoices', $params, $opts);
     }
 
     /**
@@ -37,7 +37,7 @@ public function all($params = null, $opts = null)
      */
     public function allLines($parentId, $params = null, $opts = null)
     {
-        return $this->request('get', $this->buildPath('/v1/invoices/%s/lines', $parentId), $params, $opts);
+        return $this->requestCollection('get', $this->buildPath('/v1/invoices/%s/lines', $parentId), $params, $opts);
     }
 
     /**

lib/Service/Issuing/AuthorizationService.php

@@ -18,7 +18,7 @@ class AuthorizationService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/issuing/authorizations', $params, $opts);
+        return $this->requestCollection('get', '/v1/issuing/authorizations', $params, $opts);
     }
 
     /**

lib/Service/Issuing/CardService.php

@@ -18,7 +18,7 @@ class CardService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/issuing/cards', $params, $opts);
+        return $this->requestCollection('get', '/v1/issuing/cards', $params, $opts);
     }
 
     /**

lib/Service/Issuing/CardholderService.php

@@ -18,7 +18,7 @@ class CardholderService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/issuing/cardholders', $params, $opts);
+        return $this->requestCollection('get', '/v1/issuing/cardholders', $params, $opts);
     }
 
     /**

lib/Service/Issuing/DisputeService.php

@@ -18,7 +18,7 @@ class DisputeService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/issuing/disputes', $params, $opts);
+        return $this->requestCollection('get', '/v1/issuing/disputes', $params, $opts);
     }
 
     /**

lib/Service/Issuing/TransactionService.php

@@ -18,7 +18,7 @@ class TransactionService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/issuing/transactions', $params, $opts);
+        return $this->requestCollection('get', '/v1/issuing/transactions', $params, $opts);
     }
 
     /**

lib/Service/OrderReturnService.php

@@ -17,7 +17,7 @@ class OrderReturnService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/order_returns', $params, $opts);
+        return $this->requestCollection('get', '/v1/order_returns', $params, $opts);
     }
 
     /**

lib/Service/OrderService.php

@@ -17,7 +17,7 @@ class OrderService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/orders', $params, $opts);
+        return $this->requestCollection('get', '/v1/orders', $params, $opts);
     }
 
     /**

lib/Service/PaymentIntentService.php

@@ -16,7 +16,7 @@ class PaymentIntentService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/payment_intents', $params, $opts);
+        return $this->requestCollection('get', '/v1/payment_intents', $params, $opts);
     }
 
     /**

lib/Service/PaymentMethodService.php

@@ -16,7 +16,7 @@ class PaymentMethodService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/payment_methods', $params, $opts);
+        return $this->requestCollection('get', '/v1/payment_methods', $params, $opts);
     }
 
     /**

lib/Service/PayoutService.php

@@ -18,7 +18,7 @@ class PayoutService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/payouts', $params, $opts);
+        return $this->requestCollection('get', '/v1/payouts', $params, $opts);
     }
 
     /**

lib/Service/PlanService.php

@@ -16,7 +16,7 @@ class PlanService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/plans', $params, $opts);
+        return $this->requestCollection('get', '/v1/plans', $params, $opts);
     }
 
     /**

lib/Service/PriceService.php

@@ -16,7 +16,7 @@ class PriceService extends \Stripe\Service\AbstractService
      */
     public function all($params = null, $opts = null)
     {
-        return $this->request('get', '/v1/prices', $params, $opts);
+        return $this->requestCollection('get', '/v1/prices', $params, $opts);
     }
 
     /**