Fix CS (#510)

bocharsky-bw · web-flow · commit 0a8120dbc218 · 2024-08-21T12:50:40.000+02:00
diff --git a/Command/CheckMissingCommand.php b/Command/CheckMissingCommand.php
@@ -93,7 +93,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $io = new SymfonyStyle($input, $output);
 
         if ($newMessages > 0) {
-            $io->error(sprintf('%d new message(s) have been found, run bin/console translation:extract', $newMessages));
+            $io->error(\sprintf('%d new message(s) have been found, run bin/console translation:extract', $newMessages));
 
             return 1;
         }
@@ -102,7 +102,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
         if ($emptyTranslations > 0) {
             $io->error(
-                sprintf('%d messages have empty translations, please provide translations for them', $emptyTranslations)
+                \sprintf('%d messages have empty translations, please provide translations for them', $emptyTranslations)
             );
 
             return 1;
diff --git a/Command/DeleteEmptyCommand.php b/Command/DeleteEmptyCommand.php
@@ -99,7 +99,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         if ($input->isInteractive()) {
             /** @var QuestionHelper $helper */
             $helper = $this->getHelper('question');
-            $question = new ConfirmationQuestion(sprintf('You are about to remove %d translations. Do you wish to continue? (y/N) ', $messageCount), false);
+            $question = new ConfirmationQuestion(\sprintf('You are about to remove %d translations. Do you wish to continue? (y/N) ', $messageCount), false);
             if (!$helper->ask($input, $output, $question)) {
                 return 0;
             }
@@ -112,7 +112,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         foreach ($messages as $message) {
             $storage->delete($message->getLocale(), $message->getDomain(), $message->getKey());
             if ($output->getVerbosity() > OutputInterface::VERBOSITY_NORMAL) {
-                $output->writeln(sprintf(
+                $output->writeln(\sprintf(
                     'Deleted empty message "<info>%s</info>" from domain "<info>%s</info>" and locale "<info>%s</info>"',
                     $message->getKey(),
                     $message->getDomain(),
diff --git a/Command/DeleteObsoleteCommand.php b/Command/DeleteObsoleteCommand.php
@@ -100,7 +100,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         if ($input->isInteractive()) {
             /** @var QuestionHelper $helper */
             $helper = $this->getHelper('question');
-            $question = new ConfirmationQuestion(sprintf('You are about to remove %d translations. Do you wish to continue? (y/N) ', $messageCount), false);
+            $question = new ConfirmationQuestion(\sprintf('You are about to remove %d translations. Do you wish to continue? (y/N) ', $messageCount), false);
             if (!$helper->ask($input, $output, $question)) {
                 return 0;
             }
@@ -113,7 +113,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         foreach ($messages as $message) {
             $storage->delete($message->getLocale(), $message->getDomain(), $message->getKey());
             if ($output->getVerbosity() > OutputInterface::VERBOSITY_NORMAL) {
-                $output->writeln(sprintf(
+                $output->writeln(\sprintf(
                     'Deleted obsolete message "<info>%s</info>" from domain "<info>%s</info>" and locale "<info>%s</info>"',
                     $message->getKey(),
                     $message->getDomain(),
diff --git a/Command/ExtractCommand.php b/Command/ExtractCommand.php
@@ -130,7 +130,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             /** @var Error $error */
             foreach ($errors as $error) {
                 $io->error(
-                    sprintf("%s\nLine: %s\nMessage: %s", $error->getPath(), $error->getLine(), $error->getMessage())
+                    \sprintf("%s\nLine: %s\nMessage: %s", $error->getPath(), $error->getLine(), $error->getMessage())
                 );
             }
         }
diff --git a/Command/StorageTrait.php b/Command/StorageTrait.php
@@ -31,7 +31,7 @@ private function getStorage($configName): StorageService
         if (null === $storage = $this->storageManager->getStorage($configName)) {
             $availableStorages = $this->storageManager->getNames();
 
-            throw new \InvalidArgumentException(sprintf('Unknown storage "%s". Available storages are "%s".', $configName, implode('", "', $availableStorages)));
+            throw new \InvalidArgumentException(\sprintf('Unknown storage "%s". Available storages are "%s".', $configName, implode('", "', $availableStorages)));
         }
 
         return $storage;
diff --git a/Command/SyncCommand.php b/Command/SyncCommand.php
@@ -60,7 +60,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
                 break;
             default:
-                $output->writeln(sprintf('Direction must be either "up" or "down". Not "%s".', $input->getArgument('direction')));
+                $output->writeln(\sprintf('Direction must be either "up" or "down". Not "%s".', $input->getArgument('direction')));
 
                 return 0;
         }
diff --git a/Controller/SymfonyProfilerController.php b/Controller/SymfonyProfilerController.php
@@ -130,7 +130,7 @@ public function createAssetsAction(Request $request, string $token): Response
             $uploaded[] = $message;
         }
 
-        return new Response(sprintf('%s new assets created!', \count($uploaded)));
+        return new Response(\sprintf('%s new assets created!', \count($uploaded)));
     }
 
     private function getMessage(Request $request, string $token): SfProfilerMessage
@@ -142,7 +142,7 @@ private function getMessage(Request $request, string $token): SfProfilerMessage
         $collectorMessages = $this->getMessages($token);
 
         if (!isset($collectorMessages[$messageId])) {
-            throw new NotFoundHttpException(sprintf('No message with key "%s" was found.', $messageId));
+            throw new NotFoundHttpException(\sprintf('No message with key "%s" was found.', $messageId));
         }
         $message = SfProfilerMessage::create($collectorMessages[$messageId]);
 
@@ -152,7 +152,7 @@ private function getMessage(Request $request, string $token): SfProfilerMessage
 
             $message
                 ->setLocale($requestCollector->getLocale())
-                ->setTranslation(sprintf('[%s]', $message->getTranslation()))
+                ->setTranslation(\sprintf('[%s]', $message->getTranslation()))
             ;
         }
 
diff --git a/Controller/WebUIController.php b/Controller/WebUIController.php
@@ -179,7 +179,7 @@ public function createAction(Request $request, string $configName, string $local
         try {
             $storage->create($message);
         } catch (StorageException $e) {
-            throw new BadRequestHttpException(sprintf('Key "%s" does already exist for "%s" on domain "%s".', $message->getKey(), $locale, $domain), $e);
+            throw new BadRequestHttpException(\sprintf('Key "%s" does already exist for "%s" on domain "%s".', $message->getKey(), $locale, $domain), $e);
         } catch (\Exception $e) {
             return new Response($e->getMessage(), Response::HTTP_BAD_REQUEST);
         }
diff --git a/DependencyInjection/CompilerPass/StoragePass.php b/DependencyInjection/CompilerPass/StoragePass.php
@@ -51,7 +51,7 @@ public function process(ContainerBuilder $container): void
 
                         break;
                     default:
-                        throw new \LogicException(sprintf('The tag "php_translation.storage" must have a "type" of value "local" or "remote". Value "%s" was provided', $tag['type']));
+                        throw new \LogicException(\sprintf('The tag "php_translation.storage" must have a "type" of value "local" or "remote". Value "%s" was provided', $tag['type']));
                 }
             }
         }
diff --git a/DependencyInjection/Configuration.php b/DependencyInjection/Configuration.php
@@ -117,15 +117,15 @@ private function configsNode(ArrayNodeDefinition $root): void
 
                                             $bundles = $container->getParameter('kernel.bundles');
                                             if (!isset($bundles[$bundleName])) {
-                                                throw new \Exception(sprintf('The bundle "%s" does not exist. Available bundles: %s', $bundleName, array_keys($bundles)));
+                                                throw new \Exception(\sprintf('The bundle "%s" does not exist. Available bundles: %s', $bundleName, array_keys($bundles)));
                                             }
 
                                             $ref = new \ReflectionClass($bundles[$bundleName]);
                                             $value = false === $pos ? \dirname($ref->getFileName()) : \dirname($ref->getFileName()).substr($value, $pos);
                                         }
 
                                         if (!is_dir($value)) {
-                                            throw new \Exception(sprintf('The directory "%s" does not exist.', $value));
+                                            throw new \Exception(\sprintf('The directory "%s" does not exist.', $value));
                                         }
 
                                         return $value;
diff --git a/DependencyInjection/TranslationExtension.php b/DependencyInjection/TranslationExtension.php
@@ -182,7 +182,7 @@ private function enableEditInPlace(ContainerBuilder $container, array $config):
         $name = $config['edit_in_place']['config_name'];
 
         if ('default' !== $name && !isset($config['configs'][$name])) {
-            throw new InvalidArgumentException(sprintf('There is no config named "%s".', $name));
+            throw new InvalidArgumentException(\sprintf('There is no config named "%s".', $name));
         }
 
         $activatorRef = new Reference($config['edit_in_place']['activator']);
diff --git a/EventListener/EditInPlaceResponseListener.php b/EventListener/EditInPlaceResponseListener.php
@@ -104,7 +104,7 @@ public function onKernelResponse(ResponseEvent $event): void
         }
         $content = preg_replace($pattern, $replacement, $content);
 
-        $html = sprintf(
+        $html = \sprintf(
             self::HTML,
             $this->packages->getUrl('bundles/translation/css/content-tools.min.css'),
             $this->packages->getUrl('bundles/translation/js/content-tools.min.js'),
diff --git a/Service/CacheClearer.php b/Service/CacheClearer.php
@@ -60,7 +60,7 @@ public function __construct(string $kernelCacheDir, $translator, Filesystem $fil
      */
     public function clearAndWarmUp(?string $locale = null): void
     {
-        $translationDir = sprintf('%s/translations', $this->kernelCacheDir);
+        $translationDir = \sprintf('%s/translations', $this->kernelCacheDir);
 
         $finder = new Finder();
 
diff --git a/Service/ConfigurationManager.php b/Service/ConfigurationManager.php
@@ -50,7 +50,7 @@ public function getConfiguration($name = null): Configuration
             }
         }
 
-        throw new \InvalidArgumentException(sprintf('No configuration found for "%s"', $name));
+        throw new \InvalidArgumentException(\sprintf('No configuration found for "%s"', $name));
     }
 
     public function getFirstName(): ?string
diff --git a/Service/Importer.php b/Service/Importer.php
@@ -162,7 +162,7 @@ private function convertSourceLocationsToMessages(
             $trimLength = 1 + \strlen($this->config['project_root']);
 
             $meta = $this->getMetadata($catalogue, $key, $messageDomain);
-            $meta->addCategory('file-source', sprintf('%s:%s', substr($sourceLocation->getPath(), $trimLength), $sourceLocation->getLine()));
+            $meta->addCategory('file-source', \sprintf('%s:%s', substr($sourceLocation->getPath(), $trimLength), $sourceLocation->getLine()));
             if (isset($sourceLocation->getContext()['desc'])) {
                 $meta->addCategory('desc', $sourceLocation->getContext()['desc']);
             }
diff --git a/Service/StorageService.php b/Service/StorageService.php
@@ -87,7 +87,7 @@ public function sync(string $direction = self::DIRECTION_DOWN, array $importOpti
 
                 break;
             default:
-                throw new LogicException(sprintf('Direction must be either "up" or "down". Value "%s" was provided', $direction));
+                throw new LogicException(\sprintf('Direction must be either "up" or "down". Value "%s" was provided', $direction));
         }
     }
 
diff --git a/Translator/EditInPlaceTranslator.php b/Translator/EditInPlaceTranslator.php
@@ -76,7 +76,7 @@ public function getCatalogue($locale = null): MessageCatalogueInterface
     public function getCatalogues(): array
     {
         if (!method_exists($this->translator, 'getCatalogues')) {
-            throw new \Exception(sprintf('%s method is not available! Please, upgrade to Symfony 6 in order to to use it', __METHOD__));
+            throw new \Exception(\sprintf('%s method is not available! Please, upgrade to Symfony 6 in order to to use it', __METHOD__));
         }
 
         return $this->translator->getCatalogues();
@@ -103,7 +103,7 @@ public function trans($id, array $parameters = [], $domain = null, $locale = nul
         }
 
         // Render all data in the translation tag required to allow in-line translation
-        return sprintf('<x-trans data-key="%s|%s" data-value="%s" data-plain="%s" data-domain="%s" data-locale="%s">%s</x-trans>',
+        return \sprintf('<x-trans data-key="%s|%s" data-value="%s" data-plain="%s" data-domain="%s" data-locale="%s">%s</x-trans>',
             $domain,
             $id,
             htmlspecialchars($original),
diff --git a/Translator/FallbackTranslator.php b/Translator/FallbackTranslator.php
@@ -125,7 +125,7 @@ public function getCatalogue($locale = null): MessageCatalogueInterface
     public function getCatalogues(): array
     {
         if (!method_exists($this->symfonyTranslator, 'getCatalogues')) {
-            throw new \Exception(sprintf('%s method is not available! Please, upgrade to Symfony 6 in order to to use it', __METHOD__));
+            throw new \Exception(\sprintf('%s method is not available! Please, upgrade to Symfony 6 in order to to use it', __METHOD__));
         }
 
         return $this->symfonyTranslator->getCatalogues();
diff --git a/Twig/Node/Transchoice.php b/Twig/Node/Transchoice.php
@@ -25,7 +25,7 @@ public function __construct(ArrayExpression $arguments, $lineno)
     public function compile(Compiler $compiler): void
     {
         $compiler->raw(
-            sprintf(
+            \sprintf(
                 '$this->env->getExtension(\'%s\')->%s(',
                 'Translation\Bundle\Twig\TranslationExtension',
                 'transchoiceWithDefault'

Original file line number	Diff line number	Diff line change
`@@ -130,7 +130,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`130`	`130`	`/** @var Error $error */`
`131`	`131`	`foreach ($errors as $error) {`
`132`	`132`	`$io->error(`
`133`		`- sprintf("%s\nLine: %s\nMessage: %s", $error->getPath(), $error->getLine(), $error->getMessage())`
	`133`	`+ \sprintf("%s\nLine: %s\nMessage: %s", $error->getPath(), $error->getLine(), $error->getMessage())`
`134`	`134`	`);`
`135`	`135`	`}`
`136`	`136`	`}`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ private function getStorage($configName): StorageService`
`31`	`31`	`if (null === $storage = $this->storageManager->getStorage($configName)) {`
`32`	`32`	`$availableStorages = $this->storageManager->getNames();`
`33`	`33`
`34`		`- throw new \InvalidArgumentException(sprintf('Unknown storage "%s". Available storages are "%s".', $configName, implode('", "', $availableStorages)));`
	`34`	`+ throw new \InvalidArgumentException(\sprintf('Unknown storage "%s". Available storages are "%s".', $configName, implode('", "', $availableStorages)));`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`return $storage;`
Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`60`	`60`
`61`	`61`	`break;`
`62`	`62`	`default:`
`63`		`- $output->writeln(sprintf('Direction must be either "up" or "down". Not "%s".', $input->getArgument('direction')));`
	`63`	`+ $output->writeln(\sprintf('Direction must be either "up" or "down". Not "%s".', $input->getArgument('direction')));`
`64`	`64`
`65`	`65`	`return 0;`
`66`	`66`	`}`
Original file line number	Diff line number	Diff line change
`@@ -130,7 +130,7 @@ public function createAssetsAction(Request $request, string $token): Response`
`130`	`130`	`$uploaded[] = $message;`
`131`	`131`	`}`
`132`	`132`
`133`		`- return new Response(sprintf('%s new assets created!', \count($uploaded)));`
	`133`	`+ return new Response(\sprintf('%s new assets created!', \count($uploaded)));`
`134`	`134`	`}`
`135`	`135`
`136`	`136`	`private function getMessage(Request $request, string $token): SfProfilerMessage`
`@@ -142,7 +142,7 @@ private function getMessage(Request $request, string $token): SfProfilerMessage`
`142`	`142`	`$collectorMessages = $this->getMessages($token);`
`143`	`143`
`144`	`144`	`if (!isset($collectorMessages[$messageId])) {`
`145`		`- throw new NotFoundHttpException(sprintf('No message with key "%s" was found.', $messageId));`
	`145`	`+ throw new NotFoundHttpException(\sprintf('No message with key "%s" was found.', $messageId));`
`146`	`146`	`}`
`147`	`147`	`$message = SfProfilerMessage::create($collectorMessages[$messageId]);`
`148`	`148`
`@@ -152,7 +152,7 @@ private function getMessage(Request $request, string $token): SfProfilerMessage`
`152`	`152`
`153`	`153`	`$message`
`154`	`154`	`->setLocale($requestCollector->getLocale())`
`155`		`- ->setTranslation(sprintf('[%s]', $message->getTranslation()))`
	`155`	`+ ->setTranslation(\sprintf('[%s]', $message->getTranslation()))`
`156`	`156`	`;`
`157`	`157`	`}`
`158`	`158`
Original file line number	Diff line number	Diff line change
`@@ -179,7 +179,7 @@ public function createAction(Request $request, string $configName, string $local`
`179`	`179`	`try {`
`180`	`180`	`$storage->create($message);`
`181`	`181`	`} catch (StorageException $e) {`
`182`		`- throw new BadRequestHttpException(sprintf('Key "%s" does already exist for "%s" on domain "%s".', $message->getKey(), $locale, $domain), $e);`
	`182`	`+ throw new BadRequestHttpException(\sprintf('Key "%s" does already exist for "%s" on domain "%s".', $message->getKey(), $locale, $domain), $e);`
`183`	`183`	`} catch (\Exception $e) {`
`184`	`184`	`return new Response($e->getMessage(), Response::HTTP_BAD_REQUEST);`
`185`	`185`	`}`
Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ public function process(ContainerBuilder $container): void`
`51`	`51`
`52`	`52`	`break;`
`53`	`53`	`default:`
`54`		`- throw new \LogicException(sprintf('The tag "php_translation.storage" must have a "type" of value "local" or "remote". Value "%s" was provided', $tag['type']));`
	`54`	`+ throw new \LogicException(\sprintf('The tag "php_translation.storage" must have a "type" of value "local" or "remote". Value "%s" was provided', $tag['type']));`
`55`	`55`	`}`
`56`	`56`	`}`
`57`	`57`	`}`
Original file line number	Diff line number	Diff line change
`@@ -117,15 +117,15 @@ private function configsNode(ArrayNodeDefinition $root): void`
`117`	`117`
`118`	`118`	`$bundles = $container->getParameter('kernel.bundles');`
`119`	`119`	`if (!isset($bundles[$bundleName])) {`
`120`		`- throw new \Exception(sprintf('The bundle "%s" does not exist. Available bundles: %s', $bundleName, array_keys($bundles)));`
	`120`	`+ throw new \Exception(\sprintf('The bundle "%s" does not exist. Available bundles: %s', $bundleName, array_keys($bundles)));`
`121`	`121`	`}`
`122`	`122`
`123`	`123`	`$ref = new \ReflectionClass($bundles[$bundleName]);`
`124`	`124`	`$value = false === $pos ? \dirname($ref->getFileName()) : \dirname($ref->getFileName()).substr($value, $pos);`
`125`	`125`	`}`
`126`	`126`
`127`	`127`	`if (!is_dir($value)) {`
`128`		`- throw new \Exception(sprintf('The directory "%s" does not exist.', $value));`
	`128`	`+ throw new \Exception(\sprintf('The directory "%s" does not exist.', $value));`
`129`	`129`	`}`
`130`	`130`
`131`	`131`	`return $value;`