CS fixes

Author: nicolas-grekas

Command/ServerLogCommand.php

@@ -106,7 +106,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         }
 
         if (!$socket = stream_socket_server($host, $errno, $errstr)) {
-            throw new RuntimeException(sprintf('Server start failed on "%s": ', $host).$errstr.' '.$errno);
+            throw new RuntimeException(\sprintf('Server start failed on "%s": ', $host).$errstr.' '.$errno);
         }
 
         foreach ($this->getLogs($socket) as $clientId => $message) {
@@ -155,7 +155,7 @@ private function displayLog(OutputInterface $output, int $clientId, array $recor
         if (isset($record['log_id'])) {
             $clientId = unpack('H*', $record['log_id'])[1];
         }
-        $logBlock = sprintf('<bg=%s> </>', self::BG_COLOR[$clientId % 8]);
+        $logBlock = \sprintf('<bg=%s> </>', self::BG_COLOR[$clientId % 8]);
         $output->write($logBlock);
 
         if (Logger::API >= 3) {

Formatter/ConsoleFormatter.php

@@ -123,8 +123,8 @@ private function doFormat(array|LogRecord $record): mixed
             '%datetime%' => $record['datetime'] instanceof \DateTimeInterface
                 ? $record['datetime']->format($this->options['date_format'])
                 : $record['datetime'],
-            '%start_tag%' => sprintf('<%s>', self::LEVEL_COLOR_MAP[$record['level']]),
-            '%level_name%' => sprintf($this->options['level_name_format'], $record['level_name']),
+            '%start_tag%' => \sprintf('<%s>', self::LEVEL_COLOR_MAP[$record['level']]),
+            '%level_name%' => \sprintf($this->options['level_name_format'], $record['level_name']),
             '%end_tag%' => '</>',
             '%channel%' => $record['channel'],
             '%message%' => $this->replacePlaceHolder($record)['message'],
@@ -177,7 +177,7 @@ private function replacePlaceHolder(array $record): array
             // Remove quotes added by the dumper around string.
             $v = trim($this->dumpData($v, false), '"');
             $v = OutputFormatter::escape($v);
-            $replacements['{'.$k.'}'] = sprintf('<comment>%s</>', $v);
+            $replacements['{'.$k.'}'] = \sprintf('<comment>%s</>', $v);
         }
 
         $record['message'] = strtr($message, $replacements);

Handler/ElasticsearchLogstashHandler.php

@@ -64,7 +64,7 @@ class ElasticsearchLogstashHandler extends AbstractHandler
     public function __construct(string $endpoint = 'http://127.0.0.1:9200', string $index = 'monolog', ?HttpClientInterface $client = null, string|int|Level $level = Logger::DEBUG, bool $bubble = true, string $elasticsearchVersion = '1.0.0')
     {
         if (!interface_exists(HttpClientInterface::class)) {
-            throw new \LogicException(sprintf('The "%s" handler needs an HTTP client. Try running "composer require symfony/http-client".', __CLASS__));
+            throw new \LogicException(\sprintf('The "%s" handler needs an HTTP client. Try running "composer require symfony/http-client".', __CLASS__));
         }
 
         parent::__construct($level, $bubble);
@@ -182,7 +182,7 @@ private function wait(bool $blocking): void
                 }
             } catch (ExceptionInterface $e) {
                 $this->responses->detach($response);
-                error_log(sprintf("Could not push logs to Elasticsearch:\n%s", (string) $e));
+                error_log(\sprintf("Could not push logs to Elasticsearch:\n%s", (string) $e));
             }
         }
     }

Handler/FingersCrossed/NotFoundActivationStrategy.php

@@ -30,7 +30,7 @@ final class NotFoundActivationStrategy implements ActivationStrategyInterface
     public function __construct(
         private RequestStack $requestStack,
         array $excludedUrls,
-        private ActivationStrategyInterface $inner
+        private ActivationStrategyInterface $inner,
     ) {
         $this->exclude = '{('.implode('|', $excludedUrls).')}i';
     }

Handler/MailerHandler.php

@@ -108,7 +108,7 @@ protected function buildMessage(string $content, array $records): Email
         } elseif (\is_callable($this->messageTemplate)) {
             $message = ($this->messageTemplate)($content, $records);
             if (!$message instanceof Email) {
-                throw new \InvalidArgumentException(sprintf('Could not resolve message from a callable. Instance of "%s" is expected.', Email::class));
+                throw new \InvalidArgumentException(\sprintf('Could not resolve message from a callable. Instance of "%s" is expected.', Email::class));
             }
         } else {
             throw new \InvalidArgumentException('Could not resolve message as instance of Email or a callable returning it.');

Tests/Formatter/ConsoleFormatterTest.php

@@ -35,7 +35,7 @@ public static function providerFormatTests(): array
         $tests = [
             'record with DateTime object in datetime field' => [
                 'record' => RecordFactory::create(datetime: $currentDateTime),
-                'expectedMessage' => sprintf(
+                'expectedMessage' => \sprintf(
                     "%s <fg=cyan>WARNING  </> <comment>[test]</> test\n",
                     $currentDateTime->format(ConsoleFormatter::SIMPLE_DATE)
                 ),

Tests/Handler/ConsoleHandlerTest.php

@@ -63,7 +63,7 @@ public function testVerbosityMapping($verbosity, $level, $isHandling, array $map
 
         // check that the handler actually outputs the record if it handles it
         $levelName = Logger::getLevelName($level);
-        $levelName = sprintf('%-9s', $levelName);
+        $levelName = \sprintf('%-9s', $levelName);
 
         $realOutput = $this->getMockBuilder(Output::class)->onlyMethods(['doWrite'])->getMock();
         $realOutput->setVerbosity($verbosity);

Tests/Handler/ServerLogHandlerTest.php

@@ -58,7 +58,7 @@ public function testWritingAndFormatting()
         $infoRecord = RecordFactory::create(Logger::INFO, 'My info message', 'app', datetime: new \DateTimeImmutable('2013-05-29 16:21:54'));
 
         $socket = stream_socket_server($host, $errno, $errstr);
-        $this->assertIsResource($socket, sprintf('Server start failed on "%s": %s %s.', $host, $errstr, $errno));
+        $this->assertIsResource($socket, \sprintf('Server start failed on "%s": %s %s.', $host, $errstr, $errno));
 
         $this->assertTrue($handler->handle($infoRecord), 'The handler finished handling the log as bubble is false.');