minor #38394 [FrameworkBundle] Add check for installed yaml component (jschaedl)

nicolas-grekas · nicolas-grekas · commit 5c3ba3ab7851 · 2020-10-06T18:28:05.000+02:00
This PR was submitted for the master branch but it was squashed and merged into the 5.x branch instead. Discussion ---------- [FrameworkBundle] Add check for installed yaml component | Q | A | ------------- | --- | Branch? | master | Bug fix? | yes | New feature? | no  | Deprecations? | no  | Tickets | Fix #38301  | License | MIT | Doc PR | -   **Todo:** - [ ] add tests Commits ------- f174ad2 [FrameworkBundle] Add check for installed yaml component
diff --git a/src/Symfony/Bundle/FrameworkBundle/Command/ConfigDumpReferenceCommand.php b/src/Symfony/Bundle/FrameworkBundle/Command/ConfigDumpReferenceCommand.php
@@ -22,6 +22,7 @@
 use Symfony\Component\Console\Style\SymfonyStyle;
 use Symfony\Component\DependencyInjection\Extension\ConfigurationExtensionInterface;
 use Symfony\Component\DependencyInjection\Extension\ExtensionInterface;
+use Symfony\Component\Yaml\Yaml;
 
 /**
  * A console command for dumping available configuration reference.
@@ -112,6 +113,13 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $this->validateConfiguration($extension, $configuration);
 
         $format = $input->getOption('format');
+
+        if ('yaml' === $format && !class_exists(Yaml::class)) {
+            $errorIo->error('Setting the "format" option to "yaml" requires the Symfony Yaml component. Try running "composer install symfony/yaml" or use "--format=xml" instead.');
+
+            return 1;
+        }
+
         $path = $input->getArgument('path');
 
         if (null !== $path && 'yaml' !== $format) {
