CS fixes

nicolas-grekas · nicolas-grekas · commit 7d5fc82992d4 · 2025-07-10T10:14:14.000+02:00
diff --git a/RingCentralTransport.php b/RingCentralTransport.php
@@ -33,14 +33,14 @@ public function __construct(
         #[\SensitiveParameter] private readonly string $apiToken,
         private readonly string $from,
         ?HttpClientInterface $client = null,
-        ?EventDispatcherInterface $dispatcher = null
+        ?EventDispatcherInterface $dispatcher = null,
     ) {
         parent::__construct($client, $dispatcher);
     }
 
     public function __toString(): string
     {
-        return sprintf('ringcentral://%s%s', $this->getEndpoint(), null !== $this->from ? '?from='.$this->from : '');
+        return \sprintf('ringcentral://%s%s', $this->getEndpoint(), null !== $this->from ? '?from='.$this->from : '');
     }
 
     public function supports(MessageInterface $message): bool
@@ -60,10 +60,10 @@ protected function doSend(MessageInterface $message): SentMessage
         $options['to'][]['phoneNumber'] = $message->getPhone();
 
         if (!preg_match('/^\+[1-9]\d{1,14}$/', $options['from']['phoneNumber'] ?? '')) {
-            throw new InvalidArgumentException(sprintf('The "From" number "%s" is not a valid phone number. Phone number must be in E.164 format.', $options['from']['phoneNumber'] ?? ''));
+            throw new InvalidArgumentException(\sprintf('The "From" number "%s" is not a valid phone number. Phone number must be in E.164 format.', $options['from']['phoneNumber'] ?? ''));
         }
 
-        $endpoint = sprintf('https://%s/restapi/v1.0/account/~/extension/~/sms', $this->getEndpoint());
+        $endpoint = \sprintf('https://%s/restapi/v1.0/account/~/extension/~/sms', $this->getEndpoint());
         $response = $this->client->request('POST', $endpoint, [
             'auth_bearer' => $this->apiToken,
             'json' => array_filter($options),
@@ -77,7 +77,7 @@ protected function doSend(MessageInterface $message): SentMessage
 
         if (200 !== $statusCode) {
             $error = $response->toArray(false);
-            throw new TransportException(sprintf('Unable to send the SMS - "%s".', $error['message'] ?? $error['error_description'] ?? $error['description'] ?? 'unknown failure'), $response);
+            throw new TransportException(\sprintf('Unable to send the SMS - "%s".', $error['message'] ?? $error['error_description'] ?? $error['description'] ?? 'unknown failure'), $response);
         }
 
         $success = $response->toArray(false);
diff --git a/Tests/RingCentralTransportTest.php b/Tests/RingCentralTransportTest.php
@@ -49,7 +49,7 @@ public function testInvalidArgumentExceptionIsThrownIfFromIsInvalid(string $from
         $transport = $this->createTransport(null, $from);
 
         $this->expectException(InvalidArgumentException::class);
-        $this->expectExceptionMessage(sprintf('The "From" number "%s" is not a valid phone number.', $from));
+        $this->expectExceptionMessage(\sprintf('The "From" number "%s" is not a valid phone number.', $from));
 
         $transport->send(new SmsMessage('+33612345678', 'Hello!'));
     }

Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ public function testInvalidArgumentExceptionIsThrownIfFromIsInvalid(string $from`
`49`	`49`	`$transport = $this->createTransport(null, $from);`
`50`	`50`
`51`	`51`	`$this->expectException(InvalidArgumentException::class);`
`52`		`- $this->expectExceptionMessage(sprintf('The "From" number "%s" is not a valid phone number.', $from));`
	`52`	`+ $this->expectExceptionMessage(\sprintf('The "From" number "%s" is not a valid phone number.', $from));`
`53`	`53`
`54`	`54`	`$transport->send(new SmsMessage('+33612345678', 'Hello!'));`
`55`	`55`	`}`