Prefix all sprintf() calls

derrabus · derrabus · commit a59a377b24ec · 2024-06-20T17:52:34.000+02:00
diff --git a/Envelope.php b/Envelope.php
@@ -46,7 +46,7 @@ public function setSender(Address $sender): void
     {
         // to ensure deliverability of bounce emails independent of UTF-8 capabilities of SMTP servers
         if (!preg_match('/^[^@\x80-\xFF]++@/', $sender->getAddress())) {
-            throw new InvalidArgumentException(sprintf('Invalid sender "%s": non-ASCII characters not supported in local-part of email.', $sender->getAddress()));
+            throw new InvalidArgumentException(\sprintf('Invalid sender "%s": non-ASCII characters not supported in local-part of email.', $sender->getAddress()));
         }
         $this->sender = $sender;
     }
@@ -72,7 +72,7 @@ public function setRecipients(array $recipients): void
         $this->recipients = [];
         foreach ($recipients as $recipient) {
             if (!$recipient instanceof Address) {
-                throw new InvalidArgumentException(sprintf('A recipient must be an instance of "%s" (got "%s").', Address::class, get_debug_type($recipient)));
+                throw new InvalidArgumentException(\sprintf('A recipient must be an instance of "%s" (got "%s").', Address::class, get_debug_type($recipient)));
             }
             $this->recipients[] = new Address($recipient->getAddress());
         }
diff --git a/Event/MessageEvent.php b/Event/MessageEvent.php
@@ -81,7 +81,7 @@ public function reject(): void
     public function addStamp(StampInterface $stamp): void
     {
         if (!$this->queued) {
-            throw new LogicException(sprintf('Cannot call "%s()" on a message that is not meant to be queued.', __METHOD__));
+            throw new LogicException(\sprintf('Cannot call "%s()" on a message that is not meant to be queued.', __METHOD__));
         }
 
         $this->stamps[] = $stamp;
@@ -93,7 +93,7 @@ public function addStamp(StampInterface $stamp): void
     public function getStamps(): array
     {
         if (!$this->queued) {
-            throw new LogicException(sprintf('Cannot call "%s()" on a message that is not meant to be queued.', __METHOD__));
+            throw new LogicException(\sprintf('Cannot call "%s()" on a message that is not meant to be queued.', __METHOD__));
         }
 
         return $this->stamps;
diff --git a/EventListener/MessageListener.php b/EventListener/MessageListener.php
@@ -54,7 +54,7 @@ public function __construct(
     public function addHeaderRule(string $headerName, int $rule): void
     {
         if ($rule < 1 || $rule > 3) {
-            throw new InvalidArgumentException(sprintf('The "%d" rule is not supported.', $rule));
+            throw new InvalidArgumentException(\sprintf('The "%d" rule is not supported.', $rule));
         }
 
         $this->headerRules[strtolower($headerName)] = $rule;
@@ -104,7 +104,7 @@ private function setHeaders(Message $message): void
 
                     $h = $headers->get($name);
                     if (!$h instanceof MailboxListHeader) {
-                        throw new RuntimeException(sprintf('Unable to set header "%s".', $name));
+                        throw new RuntimeException(\sprintf('Unable to set header "%s".', $name));
                     }
 
                     Headers::checkHeaderClass($header);
diff --git a/Exception/UnsupportedSchemeException.php b/Exception/UnsupportedSchemeException.php
@@ -86,14 +86,14 @@ public function __construct(Dsn $dsn, ?string $name = null, array $supported = [
         }
         $package = self::SCHEME_TO_PACKAGE_MAP[$provider] ?? null;
         if ($package && !class_exists($package['class'])) {
-            parent::__construct(sprintf('Unable to send emails via "%s" as the bridge is not installed. Try running "composer require %s".', $provider, $package['package']));
+            parent::__construct(\sprintf('Unable to send emails via "%s" as the bridge is not installed. Try running "composer require %s".', $provider, $package['package']));
 
             return;
         }
 
-        $message = sprintf('The "%s" scheme is not supported', $dsn->getScheme());
+        $message = \sprintf('The "%s" scheme is not supported', $dsn->getScheme());
         if ($name && $supported) {
-            $message .= sprintf('; supported schemes for mailer "%s" are: "%s"', $name, implode('", "', $supported));
+            $message .= \sprintf('; supported schemes for mailer "%s" are: "%s"', $name, implode('", "', $supported));
         }
 
         parent::__construct($message.'.');
diff --git a/Test/Constraint/EmailCount.php b/Test/Constraint/EmailCount.php
@@ -25,7 +25,7 @@ public function __construct(
 
     public function toString(): string
     {
-        return sprintf('%shas %s "%d" emails', $this->transport ? $this->transport.' ' : '', $this->queued ? 'queued' : 'sent', $this->expectedValue);
+        return \sprintf('%shas %s "%d" emails', $this->transport ? $this->transport.' ' : '', $this->queued ? 'queued' : 'sent', $this->expectedValue);
     }
 
     /**
@@ -41,7 +41,7 @@ protected function matches($events): bool
      */
     protected function failureDescription($events): string
     {
-        return sprintf('the Transport %s (%d %s)', $this->toString(), $this->countEmails($events), $this->queued ? 'queued' : 'sent');
+        return \sprintf('the Transport %s (%d %s)', $this->toString(), $this->countEmails($events), $this->queued ? 'queued' : 'sent');
     }
 
     private function countEmails(MessageEvents $events): int
diff --git a/Tests/Exception/UnsupportedSchemeExceptionTest.php b/Tests/Exception/UnsupportedSchemeExceptionTest.php
@@ -64,7 +64,7 @@ public function testMessageWhereSchemeIsPartOfSchemeToPackageMap(string $scheme,
         $dsn = new Dsn($scheme, 'localhost');
 
         $this->assertSame(
-            sprintf('Unable to send emails via "%s" as the bridge is not installed. Try running "composer require %s".', $scheme, $package),
+            \sprintf('Unable to send emails via "%s" as the bridge is not installed. Try running "composer require %s".', $scheme, $package),
             (new UnsupportedSchemeException($dsn))->getMessage()
         );
     }
diff --git a/Transport.php b/Transport.php
@@ -142,7 +142,7 @@ private function parseDsn(#[\SensitiveParameter] string $dsn, int $offset = 0):
             }
 
             if (preg_match('{(\w+)\(}A', $dsn, $matches, 0, $offset)) {
-                throw new InvalidArgumentException(sprintf('The "%s" keyword is not valid (valid ones are "%s"), ', $matches[1], implode('", "', array_keys($keywords))));
+                throw new InvalidArgumentException(\sprintf('The "%s" keyword is not valid (valid ones are "%s"), ', $matches[1], implode('", "', array_keys($keywords))));
             }
 
             if ($pos = strcspn($dsn, ' )', $offset)) {
diff --git a/Transport/AbstractApiTransport.php b/Transport/AbstractApiTransport.php
@@ -31,7 +31,7 @@ protected function doSendHttp(SentMessage $message): ResponseInterface
         try {
             $email = MessageConverter::toEmail($message->getOriginalMessage());
         } catch (\Exception $e) {
-            throw new RuntimeException(sprintf('Unable to send message with the "%s" transport: ', __CLASS__).$e->getMessage(), 0, $e);
+            throw new RuntimeException(\sprintf('Unable to send message with the "%s" transport: ', __CLASS__).$e->getMessage(), 0, $e);
         }
 
         return $this->doSendApi($message, $email, $message->getEnvelope());
diff --git a/Transport/AbstractHttpTransport.php b/Transport/AbstractHttpTransport.php
@@ -34,7 +34,7 @@ public function __construct(
     ) {
         if (null === $client) {
             if (!class_exists(HttpClient::class)) {
-                throw new \LogicException(sprintf('You cannot use "%s" as the HttpClient component is not installed. Try running "composer require symfony/http-client".', __CLASS__));
+                throw new \LogicException(\sprintf('You cannot use "%s" as the HttpClient component is not installed. Try running "composer require symfony/http-client".', __CLASS__));
             }
 
             $this->client = HttpClient::create();
diff --git a/Transport/AbstractTransport.php b/Transport/AbstractTransport.php
@@ -81,7 +81,7 @@ public function send(RawMessage $message, ?Envelope $envelope = null): ?SentMess
             $message = $event->getMessage();
 
             if ($message instanceof TemplatedEmail && !$message->isRendered()) {
-                throw new LogicException(sprintf('You must configure a "%s" when a "%s" instance has a text or HTML template set.', BodyRendererInterface::class, get_debug_type($message)));
+                throw new LogicException(\sprintf('You must configure a "%s" when a "%s" instance has a text or HTML template set.', BodyRendererInterface::class, get_debug_type($message)));
             }
 
             $sentMessage = new SentMessage($message, $envelope);
@@ -128,7 +128,7 @@ private function checkThrottling(): void
 
         $sleep = (1 / $this->rate) - (microtime(true) - $this->lastSent);
         if (0 < $sleep) {
-            $this->logger->debug(sprintf('Email transport "%s" sleeps for %.2f seconds', __CLASS__, $sleep));
+            $this->logger->debug(\sprintf('Email transport "%s" sleeps for %.2f seconds', __CLASS__, $sleep));
             usleep((int) ($sleep * 1000000));
         }
         $this->lastSent = microtime(true);
diff --git a/Transport/RoundRobinTransport.php b/Transport/RoundRobinTransport.php
@@ -38,7 +38,7 @@ public function __construct(
         private int $retryPeriod = 60,
     ) {
         if (!$transports) {
-            throw new TransportException(sprintf('"%s" must have at least one transport configured.', static::class));
+            throw new TransportException(\sprintf('"%s" must have at least one transport configured.', static::class));
         }
 
         $this->deadTransports = new \SplObjectStorage();
@@ -53,7 +53,7 @@ public function send(RawMessage $message, ?Envelope $envelope = null): ?SentMess
                 return $transport->send($message, $envelope);
             } catch (TransportExceptionInterface $e) {
                 $exception ??= new TransportException('All transports failed.');
-                $exception->appendDebug(sprintf("Transport \"%s\": %s\n", $transport, $e->getDebug()));
+                $exception->appendDebug(\sprintf("Transport \"%s\": %s\n", $transport, $e->getDebug()));
                 $this->deadTransports[$transport] = microtime(true);
             }
         }
diff --git a/Transport/SendmailTransport.php b/Transport/SendmailTransport.php
@@ -55,7 +55,7 @@ public function __construct(?string $command = null, ?EventDispatcherInterface $
 
         if (null !== $command) {
             if (!str_contains($command, ' -bs') && !str_contains($command, ' -t')) {
-                throw new \InvalidArgumentException(sprintf('Unsupported sendmail command flags "%s"; must be one of "-bs" or "-t" but can include additional flags.', $command));
+                throw new \InvalidArgumentException(\sprintf('Unsupported sendmail command flags "%s"; must be one of "-bs" or "-t" but can include additional flags.', $command));
             }
 
             $this->command = $command;
@@ -89,7 +89,7 @@ public function __toString(): string
 
     protected function doSend(SentMessage $message): void
     {
-        $this->getLogger()->debug(sprintf('Email transport "%s" starting', __CLASS__));
+        $this->getLogger()->debug(\sprintf('Email transport "%s" starting', __CLASS__));
 
         $command = $this->command;
 
@@ -119,6 +119,6 @@ protected function doSend(SentMessage $message): void
         $this->stream->flush();
         $this->stream->terminate();
 
-        $this->getLogger()->debug(sprintf('Email transport "%s" stopped', __CLASS__));
+        $this->getLogger()->debug(\sprintf('Email transport "%s" stopped', __CLASS__));
     }
 }
diff --git a/Transport/Smtp/Auth/CramMd5Authenticator.php b/Transport/Smtp/Auth/CramMd5Authenticator.php
@@ -34,7 +34,7 @@ public function authenticate(EsmtpTransport $client): void
         $challenge = $client->executeCommand("AUTH CRAM-MD5\r\n", [334]);
         $challenge = base64_decode(substr($challenge, 4));
         $message = base64_encode($client->getUsername().' '.$this->getResponse($client->getPassword(), $challenge));
-        $client->executeCommand(sprintf("%s\r\n", $message), [235]);
+        $client->executeCommand(\sprintf("%s\r\n", $message), [235]);
     }
 
     /**
diff --git a/Transport/Smtp/Auth/LoginAuthenticator.php b/Transport/Smtp/Auth/LoginAuthenticator.php
@@ -31,7 +31,7 @@ public function getAuthKeyword(): string
     public function authenticate(EsmtpTransport $client): void
     {
         $client->executeCommand("AUTH LOGIN\r\n", [334]);
-        $client->executeCommand(sprintf("%s\r\n", base64_encode($client->getUsername())), [334]);
-        $client->executeCommand(sprintf("%s\r\n", base64_encode($client->getPassword())), [235]);
+        $client->executeCommand(\sprintf("%s\r\n", base64_encode($client->getUsername())), [334]);
+        $client->executeCommand(\sprintf("%s\r\n", base64_encode($client->getPassword())), [235]);
     }
 }
diff --git a/Transport/Smtp/Auth/PlainAuthenticator.php b/Transport/Smtp/Auth/PlainAuthenticator.php
@@ -30,6 +30,6 @@ public function getAuthKeyword(): string
      */
     public function authenticate(EsmtpTransport $client): void
     {
-        $client->executeCommand(sprintf("AUTH PLAIN %s\r\n", base64_encode($client->getUsername().\chr(0).$client->getUsername().\chr(0).$client->getPassword())), [235]);
+        $client->executeCommand(\sprintf("AUTH PLAIN %s\r\n", base64_encode($client->getUsername().\chr(0).$client->getUsername().\chr(0).$client->getPassword())), [235]);
     }
 }
diff --git a/Transport/Smtp/EsmtpTransport.php b/Transport/Smtp/EsmtpTransport.php
@@ -142,10 +142,10 @@ final protected function getCapabilities(): array
     private function doEhloCommand(): string
     {
         try {
-            $response = $this->executeCommand(sprintf("EHLO %s\r\n", $this->getLocalDomain()), [250]);
+            $response = $this->executeCommand(\sprintf("EHLO %s\r\n", $this->getLocalDomain()), [250]);
         } catch (TransportExceptionInterface $e) {
             try {
-                return parent::executeCommand(sprintf("HELO %s\r\n", $this->getLocalDomain()), [250]);
+                return parent::executeCommand(\sprintf("HELO %s\r\n", $this->getLocalDomain()), [250]);
             } catch (TransportExceptionInterface $ex) {
                 if (!$ex->getCode()) {
                     throw $e;
@@ -169,7 +169,7 @@ private function doEhloCommand(): string
                 throw new TransportException('Unable to connect with STARTTLS.');
             }
 
-            $response = $this->executeCommand(sprintf("EHLO %s\r\n", $this->getLocalDomain()), [250]);
+            $response = $this->executeCommand(\sprintf("EHLO %s\r\n", $this->getLocalDomain()), [250]);
             $this->capabilities = $this->parseCapabilities($response);
         }
 
@@ -231,12 +231,12 @@ private function handleAuth(array $modes): void
         }
 
         if (!$authNames) {
-            throw new TransportException(sprintf('Failed to find an authenticator supported by the SMTP server, which currently supports: "%s".', implode('", "', $modes)), $code ?: 504);
+            throw new TransportException(\sprintf('Failed to find an authenticator supported by the SMTP server, which currently supports: "%s".', implode('", "', $modes)), $code ?: 504);
         }
 
-        $message = sprintf('Failed to authenticate on SMTP server with username "%s" using the following authenticators: "%s".', $this->username, implode('", "', $authNames));
+        $message = \sprintf('Failed to authenticate on SMTP server with username "%s" using the following authenticators: "%s".', $this->username, implode('", "', $authNames));
         foreach ($errors as $name => $error) {
-            $message .= sprintf(' Authenticator "%s" returned "%s".', $name, $error);
+            $message .= \sprintf(' Authenticator "%s" returned "%s".', $name, $error);
         }
 
         throw new TransportException($message, $code ?: 535);
diff --git a/Transport/Smtp/SmtpTransport.php b/Transport/Smtp/SmtpTransport.php
@@ -171,7 +171,7 @@ protected function parseMessageId(string $mtaResult): string
     public function __toString(): string
     {
         if ($this->stream instanceof SocketStream) {
-            $name = sprintf('smtp%s://%s', ($tls = $this->stream->isTLS()) ? 's' : '', $this->stream->getHost());
+            $name = \sprintf('smtp%s://%s', ($tls = $this->stream->isTLS()) ? 's' : '', $this->stream->getHost());
             $port = $this->stream->getPort();
             if (!(25 === $port || ($tls && 465 === $port))) {
                 $name .= ':'.$port;
@@ -227,7 +227,7 @@ protected function doSend(SentMessage $message): void
             } catch (\Exception $e) {
                 $this->stream->terminate();
                 $this->started = false;
-                $this->getLogger()->debug(sprintf('Email transport "%s" stopped', __CLASS__));
+                $this->getLogger()->debug(\sprintf('Email transport "%s" stopped', __CLASS__));
                 throw $e;
             }
             $mtaResult = $this->executeCommand("\r\n.\r\n", [250]);
@@ -246,17 +246,17 @@ protected function doSend(SentMessage $message): void
 
     private function doHeloCommand(): void
     {
-        $this->executeCommand(sprintf("HELO %s\r\n", $this->domain), [250]);
+        $this->executeCommand(\sprintf("HELO %s\r\n", $this->domain), [250]);
     }
 
     private function doMailFromCommand(string $address): void
     {
-        $this->executeCommand(sprintf("MAIL FROM:<%s>\r\n", $address), [250]);
+        $this->executeCommand(\sprintf("MAIL FROM:<%s>\r\n", $address), [250]);
     }
 
     private function doRcptToCommand(string $address): void
     {
-        $this->executeCommand(sprintf("RCPT TO:<%s>\r\n", $address), [250, 251, 252]);
+        $this->executeCommand(\sprintf("RCPT TO:<%s>\r\n", $address), [250, 251, 252]);
     }
 
     public function start(): void
@@ -265,15 +265,15 @@ public function start(): void
             return;
         }
 
-        $this->getLogger()->debug(sprintf('Email transport "%s" starting', __CLASS__));
+        $this->getLogger()->debug(\sprintf('Email transport "%s" starting', __CLASS__));
 
         $this->stream->initialize();
         $this->assertResponseCode($this->getFullResponse(), [220]);
         $this->doHeloCommand();
         $this->started = true;
         $this->lastMessageTime = 0;
 
-        $this->getLogger()->debug(sprintf('Email transport "%s" started', __CLASS__));
+        $this->getLogger()->debug(\sprintf('Email transport "%s" started', __CLASS__));
     }
 
     /**
@@ -289,15 +289,15 @@ public function stop(): void
             return;
         }
 
-        $this->getLogger()->debug(sprintf('Email transport "%s" stopping', __CLASS__));
+        $this->getLogger()->debug(\sprintf('Email transport "%s" stopping', __CLASS__));
 
         try {
             $this->executeCommand("QUIT\r\n", [221]);
         } catch (TransportExceptionInterface) {
         } finally {
             $this->stream->terminate();
             $this->started = false;
-            $this->getLogger()->debug(sprintf('Email transport "%s" stopped', __CLASS__));
+            $this->getLogger()->debug(\sprintf('Email transport "%s" stopped', __CLASS__));
         }
     }
 
@@ -327,10 +327,10 @@ private function assertResponseCode(string $response, array $codes): void
         $valid = \in_array($code, $codes);
 
         if (!$valid || !$response) {
-            $codeStr = $code ? sprintf('code "%s"', $code) : 'empty code';
-            $responseStr = $response ? sprintf(', with message "%s"', trim($response)) : '';
+            $codeStr = $code ? \sprintf('code "%s"', $code) : 'empty code';
+            $responseStr = $response ? \sprintf(', with message "%s"', trim($response)) : '';
 
-            throw new UnexpectedResponseException(sprintf('Expected response code "%s" but got ', implode('/', $codes)).$codeStr.$responseStr.'.', $code ?: 0);
+            throw new UnexpectedResponseException(\sprintf('Expected response code "%s" but got ', implode('/', $codes)).$codeStr.$responseStr.'.', $code ?: 0);
         }
     }
 
@@ -359,7 +359,7 @@ private function checkRestartThreshold(): void
 
         $this->stop();
         if (0 < $sleep = $this->restartThresholdSleep) {
-            $this->getLogger()->debug(sprintf('Email transport "%s" sleeps for %d seconds after stopping', __CLASS__, $sleep));
+            $this->getLogger()->debug(\sprintf('Email transport "%s" sleeps for %d seconds after stopping', __CLASS__, $sleep));
 
             sleep($sleep);
         }
diff --git a/Transport/Smtp/Stream/AbstractStream.php b/Transport/Smtp/Stream/AbstractStream.php
diff --git a/Transport/Smtp/Stream/SocketStream.php b/Transport/Smtp/Stream/SocketStream.php
diff --git a/Transport/Transports.php b/Transport/Transports.php

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ public function setSender(Address $sender): void`
`46`	`46`	`{`
`47`	`47`	`// to ensure deliverability of bounce emails independent of UTF-8 capabilities of SMTP servers`
`48`	`48`	`if (!preg_match('/^[^@\x80-\xFF]++@/', $sender->getAddress())) {`
`49`		`- throw new InvalidArgumentException(sprintf('Invalid sender "%s": non-ASCII characters not supported in local-part of email.', $sender->getAddress()));`
	`49`	`+ throw new InvalidArgumentException(\sprintf('Invalid sender "%s": non-ASCII characters not supported in local-part of email.', $sender->getAddress()));`
`50`	`50`	`}`
`51`	`51`	`$this->sender = $sender;`
`52`	`52`	`}`
`@@ -72,7 +72,7 @@ public function setRecipients(array $recipients): void`
`72`	`72`	`$this->recipients = [];`
`73`	`73`	`foreach ($recipients as $recipient) {`
`74`	`74`	`if (!$recipient instanceof Address) {`
`75`		`- throw new InvalidArgumentException(sprintf('A recipient must be an instance of "%s" (got "%s").', Address::class, get_debug_type($recipient)));`
	`75`	`+ throw new InvalidArgumentException(\sprintf('A recipient must be an instance of "%s" (got "%s").', Address::class, get_debug_type($recipient)));`
`76`	`76`	`}`
`77`	`77`	`$this->recipients[] = new Address($recipient->getAddress());`
`78`	`78`	`}`
Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,7 @@ public function reject(): void`
`81`	`81`	`public function addStamp(StampInterface $stamp): void`
`82`	`82`	`{`
`83`	`83`	`if (!$this->queued) {`
`84`		`- throw new LogicException(sprintf('Cannot call "%s()" on a message that is not meant to be queued.', __METHOD__));`
	`84`	`+ throw new LogicException(\sprintf('Cannot call "%s()" on a message that is not meant to be queued.', __METHOD__));`
`85`	`85`	`}`
`86`	`86`
`87`	`87`	`$this->stamps[] = $stamp;`
`@@ -93,7 +93,7 @@ public function addStamp(StampInterface $stamp): void`
`93`	`93`	`public function getStamps(): array`
`94`	`94`	`{`
`95`	`95`	`if (!$this->queued) {`
`96`		`- throw new LogicException(sprintf('Cannot call "%s()" on a message that is not meant to be queued.', __METHOD__));`
	`96`	`+ throw new LogicException(\sprintf('Cannot call "%s()" on a message that is not meant to be queued.', __METHOD__));`
`97`	`97`	`}`
`98`	`98`
`99`	`99`	`return $this->stamps;`
Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ public function __construct(`
`54`	`54`	`public function addHeaderRule(string $headerName, int $rule): void`
`55`	`55`	`{`
`56`	`56`	`if ($rule < 1 \|\| $rule > 3) {`
`57`		`- throw new InvalidArgumentException(sprintf('The "%d" rule is not supported.', $rule));`
	`57`	`+ throw new InvalidArgumentException(\sprintf('The "%d" rule is not supported.', $rule));`
`58`	`58`	`}`
`59`	`59`
`60`	`60`	`$this->headerRules[strtolower($headerName)] = $rule;`
`@@ -104,7 +104,7 @@ private function setHeaders(Message $message): void`
`104`	`104`
`105`	`105`	`$h = $headers->get($name);`
`106`	`106`	`if (!$h instanceof MailboxListHeader) {`
`107`		`- throw new RuntimeException(sprintf('Unable to set header "%s".', $name));`
	`107`	`+ throw new RuntimeException(\sprintf('Unable to set header "%s".', $name));`
`108`	`108`	`}`
`109`	`109`
`110`	`110`	`Headers::checkHeaderClass($header);`
Original file line number	Diff line number	Diff line change
`@@ -86,14 +86,14 @@ public function __construct(Dsn $dsn, ?string $name = null, array $supported = [`
`86`	`86`	`}`
`87`	`87`	`$package = self::SCHEME_TO_PACKAGE_MAP[$provider] ?? null;`
`88`	`88`	`if ($package && !class_exists($package['class'])) {`
`89`		`- parent::__construct(sprintf('Unable to send emails via "%s" as the bridge is not installed. Try running "composer require %s".', $provider, $package['package']));`
	`89`	`+ parent::__construct(\sprintf('Unable to send emails via "%s" as the bridge is not installed. Try running "composer require %s".', $provider, $package['package']));`
`90`	`90`
`91`	`91`	`return;`
`92`	`92`	`}`
`93`	`93`
`94`		`- $message = sprintf('The "%s" scheme is not supported', $dsn->getScheme());`
	`94`	`+ $message = \sprintf('The "%s" scheme is not supported', $dsn->getScheme());`
`95`	`95`	`if ($name && $supported) {`
`96`		`- $message .= sprintf('; supported schemes for mailer "%s" are: "%s"', $name, implode('", "', $supported));`
	`96`	`+ $message .= \sprintf('; supported schemes for mailer "%s" are: "%s"', $name, implode('", "', $supported));`
`97`	`97`	`}`
`98`	`98`
`99`	`99`	`parent::__construct($message.'.');`
Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ public function __construct(`
`25`	`25`
`26`	`26`	`public function toString(): string`
`27`	`27`	`{`
`28`		`- return sprintf('%shas %s "%d" emails', $this->transport ? $this->transport.' ' : '', $this->queued ? 'queued' : 'sent', $this->expectedValue);`
	`28`	`+ return \sprintf('%shas %s "%d" emails', $this->transport ? $this->transport.' ' : '', $this->queued ? 'queued' : 'sent', $this->expectedValue);`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`/**`
`@@ -41,7 +41,7 @@ protected function matches($events): bool`
`41`	`41`	`*/`
`42`	`42`	`protected function failureDescription($events): string`
`43`	`43`	`{`
`44`		`- return sprintf('the Transport %s (%d %s)', $this->toString(), $this->countEmails($events), $this->queued ? 'queued' : 'sent');`
	`44`	`+ return \sprintf('the Transport %s (%d %s)', $this->toString(), $this->countEmails($events), $this->queued ? 'queued' : 'sent');`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`private function countEmails(MessageEvents $events): int`
Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@ public function testMessageWhereSchemeIsPartOfSchemeToPackageMap(string $scheme,`
`64`	`64`	`$dsn = new Dsn($scheme, 'localhost');`
`65`	`65`
`66`	`66`	`$this->assertSame(`
`67`		`- sprintf('Unable to send emails via "%s" as the bridge is not installed. Try running "composer require %s".', $scheme, $package),`
	`67`	`+ \sprintf('Unable to send emails via "%s" as the bridge is not installed. Try running "composer require %s".', $scheme, $package),`
`68`	`68`	`(new UnsupportedSchemeException($dsn))->getMessage()`
`69`	`69`	`);`
`70`	`70`	`}`
Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@ private function parseDsn(#[\SensitiveParameter] string $dsn, int $offset = 0):`
`142`	`142`	`}`
`143`	`143`
`144`	`144`	`if (preg_match('{(\w+)\(}A', $dsn, $matches, 0, $offset)) {`
`145`		`- throw new InvalidArgumentException(sprintf('The "%s" keyword is not valid (valid ones are "%s"), ', $matches[1], implode('", "', array_keys($keywords))));`
	`145`	`+ throw new InvalidArgumentException(\sprintf('The "%s" keyword is not valid (valid ones are "%s"), ', $matches[1], implode('", "', array_keys($keywords))));`
`146`	`146`	`}`
`147`	`147`
`148`	`148`	`if ($pos = strcspn($dsn, ' )', $offset)) {`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ protected function doSendHttp(SentMessage $message): ResponseInterface`
`31`	`31`	`try {`
`32`	`32`	`$email = MessageConverter::toEmail($message->getOriginalMessage());`
`33`	`33`	`} catch (\Exception $e) {`
`34`		`- throw new RuntimeException(sprintf('Unable to send message with the "%s" transport: ', __CLASS__).$e->getMessage(), 0, $e);`
	`34`	`+ throw new RuntimeException(\sprintf('Unable to send message with the "%s" transport: ', __CLASS__).$e->getMessage(), 0, $e);`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`return $this->doSendApi($message, $email, $message->getEnvelope());`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ public function __construct(`
`34`	`34`	`) {`
`35`	`35`	`if (null === $client) {`
`36`	`36`	`if (!class_exists(HttpClient::class)) {`
`37`		`- throw new \LogicException(sprintf('You cannot use "%s" as the HttpClient component is not installed. Try running "composer require symfony/http-client".', __CLASS__));`
	`37`	`+ throw new \LogicException(\sprintf('You cannot use "%s" as the HttpClient component is not installed. Try running "composer require symfony/http-client".', __CLASS__));`
`38`	`38`	`}`
`39`	`39`
`40`	`40`	`$this->client = HttpClient::create();`
Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,7 @@ public function send(RawMessage $message, ?Envelope $envelope = null): ?SentMess`
`81`	`81`	`$message = $event->getMessage();`
`82`	`82`
`83`	`83`	`if ($message instanceof TemplatedEmail && !$message->isRendered()) {`
`84`		`- throw new LogicException(sprintf('You must configure a "%s" when a "%s" instance has a text or HTML template set.', BodyRendererInterface::class, get_debug_type($message)));`
	`84`	`+ throw new LogicException(\sprintf('You must configure a "%s" when a "%s" instance has a text or HTML template set.', BodyRendererInterface::class, get_debug_type($message)));`
`85`	`85`	`}`
`86`	`86`
`87`	`87`	`$sentMessage = new SentMessage($message, $envelope);`
`@@ -128,7 +128,7 @@ private function checkThrottling(): void`
`128`	`128`
`129`	`129`	`$sleep = (1 / $this->rate) - (microtime(true) - $this->lastSent);`
`130`	`130`	`if (0 < $sleep) {`
`131`		`- $this->logger->debug(sprintf('Email transport "%s" sleeps for %.2f seconds', __CLASS__, $sleep));`
	`131`	`+ $this->logger->debug(\sprintf('Email transport "%s" sleeps for %.2f seconds', __CLASS__, $sleep));`
`132`	`132`	`usleep((int) ($sleep * 1000000));`
`133`	`133`	`}`
`134`	`134`	`$this->lastSent = microtime(true);`