bug #20066 [FrameworkBundle] fix yaml:lint when yaml is not installed along side framework-bundle (fabpot)

This PR was merged into the 3.2-dev branch.

Discussion
----------

[FrameworkBundle] fix yaml:lint when yaml is not installed along side framework-bundle

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

YAML is not an explicit dependency of FrameworkBundle. If it is not installed, the console is broken as the yaml:lint commands tries to extends the one in the YAML component. This bug only exists in master as this refactoring happened in 3.2

Commits
-------

b1c5a68 [FrameworkBundle] fixed yaml:lint when yaml is not installed along side framwork-bundle

Author: fabpot

Command/YamlLintCommand.php

@@ -11,6 +11,9 @@
 
 namespace Symfony\Bundle\FrameworkBundle\Command;
 
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Yaml\Command\LintCommand as BaseLintCommand;
 
 /**
@@ -19,17 +22,39 @@
  * @author Grégoire Pineau <lyrixx@lyrixx.info>
  * @author Robin Chalas <robin.chalas@gmail.com>
  */
-class YamlLintCommand extends BaseLintCommand
+class YamlLintCommand extends Command
 {
+    private $command;
+
     /**
      * {@inheritdoc}
      */
     protected function configure()
     {
-        parent::configure();
+        $this->setName('lint:yaml');
+
+        if (!$this->isEnabled()) {
+            return;
+        }
+
+        $directoryIteratorProvider = function ($directory, $default) {
+            if (!is_dir($directory)) {
+                $directory = $this->getApplication()->getKernel()->locateResource($directory);
+            }
+
+            return $default($directory);
+        };
+
+        $isReadableProvider = function ($fileOrDirectory, $default) {
+            return 0 === strpos($fileOrDirectory, '@') || $default($fileOrDirectory);
+        };
 
-        $this->setHelp(
-            $this->getHelp().<<<EOF
+        $this->command = new BaseLintCommand(null, $directoryIteratorProvider, $isReadableProvider);
+
+        $this
+            ->setDescription($this->command->getDescription())
+            ->setDefinition($this->command->getDefinition())
+            ->setHelp($this->command->getHelp().<<<EOF
 
 Or find all files in a bundle:
 
@@ -39,17 +64,16 @@ protected function configure()
         );
     }
 
-    protected function getDirectoryIterator($directory)
+    /**
+     * {@inheritdoc}
+     */
+    public function isEnabled()
     {
-        if (!is_dir($directory)) {
-            $directory = $this->getApplication()->getKernel()->locateResource($directory);
-        }
-
-        return parent::getDirectoryIterator($directory);
+        return class_exists(BaseLintCommand::class) && parent::isEnabled();
     }
 
-    protected function isReadable($fileOrDirectory)
+    protected function execute(InputInterface $input, OutputInterface $output)
     {
-        return 0 === strpos($fileOrDirectory, '@') || parent::isReadable($fileOrDirectory);
+        return $this->command->execute($input, $output);
     }
 }