Merge branch '3.4' into 4.4

* 3.4:
  Add missing dots at the end of exception messages

Author: fabpot

Command/AbstractConfigCommand.php

@@ -112,11 +112,11 @@ protected function findExtension($name)
     public function validateConfiguration(ExtensionInterface $extension, $configuration)
     {
         if (!$configuration) {
-            throw new \LogicException(sprintf('The extension with alias "%s" does not have its getConfiguration() method setup', $extension->getAlias()));
+            throw new \LogicException(sprintf('The extension with alias "%s" does not have its getConfiguration() method setup.', $extension->getAlias()));
         }
 
         if (!$configuration instanceof ConfigurationInterface) {
-            throw new \LogicException(sprintf('Configuration class "%s" should implement ConfigurationInterface in order to be dumpable', \get_class($configuration)));
+            throw new \LogicException(sprintf('Configuration class "%s" should implement ConfigurationInterface in order to be dumpable.', \get_class($configuration)));
         }
     }
 

Command/CacheClearCommand.php

@@ -85,7 +85,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $fs->remove($oldCacheDir);
 
         if (!is_writable($realCacheDir)) {
-            throw new RuntimeException(sprintf('Unable to write in the "%s" directory', $realCacheDir));
+            throw new RuntimeException(sprintf('Unable to write in the "%s" directory.', $realCacheDir));
         }
 
         $io->comment(sprintf('Clearing the cache for the <info>%s</info> environment with debug <info>%s</info>', $kernel->getEnvironment(), var_export($kernel->isDebug(), true)));

Command/ConfigDebugCommand.php

@@ -145,7 +145,7 @@ private function getConfigForPath(array $config, string $path, string $alias)
 
         foreach ($steps as $step) {
             if (!\array_key_exists($step, $config)) {
-                throw new LogicException(sprintf('Unable to find configuration for "%s.%s"', $alias, $path));
+                throw new LogicException(sprintf('Unable to find configuration for "%s.%s".', $alias, $path));
             }
 
             $config = $config[$step];

DependencyInjection/Compiler/ProfilerPass.php

@@ -39,7 +39,7 @@ public function process(ContainerBuilder $container)
 
             if (isset($attributes[0]['template'])) {
                 if (!isset($attributes[0]['id'])) {
-                    throw new InvalidArgumentException(sprintf('Data collector service "%s" must have an id attribute in order to specify a template', $id));
+                    throw new InvalidArgumentException(sprintf('Data collector service "%s" must have an id attribute in order to specify a template.', $id));
                 }
                 $template = [$attributes[0]['id'], $attributes[0]['template']];
             }

DependencyInjection/FrameworkExtension.php

@@ -1196,7 +1196,7 @@ private function registerTranslatorConfiguration(array $config, ContainerBuilder
             if ($container->fileExists($dir)) {
                 $dirs[] = $transPaths[] = $dir;
             } else {
-                throw new \UnexpectedValueException(sprintf('%s defined in translator.paths does not exist or is not a directory', $dir));
+                throw new \UnexpectedValueException(sprintf('%s defined in translator.paths does not exist or is not a directory.', $dir));
             }
         }
 

Templating/Helper/StopwatchHelper.php

@@ -47,6 +47,6 @@ public function __call($method, $arguments = [])
             return $this->stopwatch->{$method}(...$arguments);
         }
 
-        throw new \BadMethodCallException(sprintf('Method "%s" of Stopwatch does not exist', $method));
+        throw new \BadMethodCallException(sprintf('Method "%s" of Stopwatch does not exist.', $method));
     }
 }

Test/WebTestCase.php

@@ -53,7 +53,7 @@ protected static function createClient(array $options = [], array $server = [])
             if (class_exists(KernelBrowser::class)) {
                 throw new \LogicException('You cannot create the client used in functional tests if the "framework.test" config is not set to true.');
             }
-            throw new \LogicException('You cannot create the client used in functional tests if the BrowserKit component is not available. Try running "composer require symfony/browser-kit"');
+            throw new \LogicException('You cannot create the client used in functional tests if the BrowserKit component is not available. Try running "composer require symfony/browser-kit".');
         }
 
         $client->setServerParameters($server);