PHP8.4 nullcheck fixes

src/Account/Config.php

@@ -19,9 +19,9 @@ class Config implements
     public function __construct(
         ?string $sms_callback_url = null,
         ?string $dr_callback_url = null,
-        int|string $max_outbound_request = null,
-        int|string $max_inbound_request = null,
-        int|string $max_calls_per_second = null
+        int|string|null $max_outbound_request = null,
+        int|string|null $max_inbound_request = null,
+        int|string|null $max_calls_per_second = null
     ) {
         if (!is_null($sms_callback_url)) {
             $this->data['sms_callback_url'] = $sms_callback_url;

src/Client.php

@@ -231,7 +231,13 @@ public function __construct(
         // Disable throwing E_USER_DEPRECATED notices by default, the user can turn it on during development
         if (array_key_exists('show_deprecations', $this->options) && ($this->options['show_deprecations'] == true)) {
             set_error_handler(
-                static fn (int $errno, string $errstr, string $errfile = null, int $errline = null, array $errorcontext = null) => true,
+                static fn (
+                    int $errno,
+                    string $errstr,
+                    ?string $errfile = null,
+                    ?int $errline = null,
+                    ?array $errorcontext = null
+                ) => true,
                 E_USER_DEPRECATED
             );
         }

src/Conversation/Client.php

@@ -30,8 +30,9 @@ public function getAPIResource(): APIResource
         return $this->api;
     }
 
-    public function listConversations(ListConversationFilter $conversationFilter = null): IterableAPICollection
-    {
+    public function listConversations(
+        ?ListConversationFilter $conversationFilter = null
+    ): IterableAPICollection {
         if (!$conversationFilter) {
             $conversationFilter = new ListConversationFilter();
         }

src/Numbers/Client.php

@@ -106,7 +106,7 @@ public function get(string $number): Number
      * @throws ClientException\Request
      * @throws ClientException\Server
      */
-    public function searchAvailable(string $country, FilterInterface $options = null): array
+    public function searchAvailable(string $country, ?FilterInterface $options = null): array
     {
         if (is_null($options)) {
             $options = new AvailableNumbers([
@@ -134,7 +134,7 @@ public function searchAvailable(string $country, FilterInterface $options = null
      * @throws ClientException\Request
      * @throws ClientException\Server
      */
-    public function searchOwned($number = null, FilterInterface $options = null): array
+    public function searchOwned($number = null, ?FilterInterface $options = null): array
     {
         if ($number !== null) {
             if ($options !== null) {

src/Subaccount/Client.php

@@ -86,7 +86,7 @@ public function updateSubaccount(string $apiKey, string $subaccountApiKey, Accou
         return $this->api->partiallyUpdate($apiKey . '/subaccounts/' . $subaccountApiKey, $account->toArray());
     }
 
-    public function getCreditTransfers(string $apiKey, FilterInterface $filter = null): mixed
+    public function getCreditTransfers(string $apiKey, ?FilterInterface $filter = null): mixed
     {
         if (!$filter) {
             $filter = new EmptyFilter();
@@ -101,7 +101,7 @@ public function getCreditTransfers(string $apiKey, FilterInterface $filter = nul
         return array_map(fn ($item) => $hydrator->hydrate($item), $transfers);
     }
 
-    public function getBalanceTransfers(string $apiKey, FilterInterface $filter = null): mixed
+    public function getBalanceTransfers(string $apiKey, ?FilterInterface $filter = null): mixed
     {
         if (!$filter) {
             $filter = new EmptyFilter();

src/Users/Client.php

@@ -29,7 +29,7 @@ public function getApiResource(): APIResource
         return $this->api;
     }
 
-    public function listUsers(FilterInterface $filter = null): IterableAPICollection
+    public function listUsers(?FilterInterface $filter = null): IterableAPICollection
     {
         if (is_null($filter)) {
             $filter = new EmptyFilter();

src/Verify/Client.php

@@ -178,7 +178,7 @@ public function trigger($verification): Verification
      * @throws ClientException\Request
      * @throws ClientException\Server
      */
-    public function check(Verification|array|string $verification, string $code, string $ip = null): Verification
+    public function check(Verification|array|string $verification, string $code, ?string $ip = null): Verification
     {
         if (is_array($verification)) {
             trigger_error(

src/Verify/RequestPSD2.php

@@ -39,7 +39,7 @@ public function __construct(
         protected string $number,
         protected string $payee,
         protected string $amount,
-        int $workflowId = null
+        ?int $workflowId = null
     ) {
         if ($workflowId) {
             $this->setWorkflowId($workflowId);

src/Verify2/Client.php

@@ -82,7 +82,7 @@ public static function isSilentAuthRequest(BaseVerifyRequest $request): bool
         return false;
     }
 
-    public function listCustomTemplates(TemplateFilter $filter = null): IterableAPICollection
+    public function listCustomTemplates(?TemplateFilter $filter = null): IterableAPICollection
     {
         $collection = $this->api->search($filter, '/templates');
         $collection->setNaiveCount(true);
@@ -158,7 +158,7 @@ public function deleteCustomTemplateFragment(string $templateId, string $fragmen
         return true;
     }
 
-    public function listTemplateFragments(string $templateId, TemplateFilter $filter = null): IterableAPICollection
+    public function listTemplateFragments(string $templateId, ?TemplateFilter $filter = null): IterableAPICollection
     {
         $api = clone $this->getAPIResource();
         $api->setCollectionName('template_fragments');

src/Voice/Client.php

@@ -182,7 +182,7 @@ public function playTTS(string $callId, Talk $action): array
         return $this->api->update($callId . '/talk', $payload);
     }
 
-    public function search(FilterInterface $filter = null): IterableAPICollection
+    public function search(?FilterInterface $filter = null): IterableAPICollection
     {
         $response = $this->api->search($filter);
         $response->setApiResource(clone $this->api);

test/ProactiveConnect/ClientTest.php

@@ -570,7 +570,7 @@ public function testWillDownloadItemCsv(): void
         fwrite($handle, (string) $response->getContents());
         fseek($handle, 0);
 
-        while (($row = fgetcsv($handle)) !== false) {
+        while (($row = fgetcsv($handle, 0, ',', '"', '\\')) !== false) {
             if (!$header) {
                 $header = $row;
             } else {