Extract and add verbose output

Author: sebastianfeldmann

src/Git/ChangedFiles.php

@@ -0,0 +1,63 @@
+<?php
+
+/**
+ * This file is part of CaptainHook
+ *
+ * (c) Sebastian Feldmann <sf@sebastian-feldmann.info>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace CaptainHook\App\Git;
+
+use CaptainHook\App\Console\IO;
+use CaptainHook\App\Git\ChangedFiles\Detector\Factory;
+use SebastianFeldmann\Git\Repository;
+
+/**
+ * Class ChangedFiles
+ *
+ * @package CaptainHook
+ * @author  Sebastian Feldmann <sf@sebastian-feldmann.info>
+ * @link    https://github.com/captainhook-git/captainhook
+ * @since   Class available since Release 5.2.0
+ */
+abstract class ChangedFiles
+{
+    /**
+     * Returns the list of changed files using the necessary Detector
+     *
+     * @param  \CaptainHook\App\Console\IO       $io
+     * @param  \SebastianFeldmann\Git\Repository $repository
+     * @param  array<string>                     $filter
+     * @return array<string>
+     */
+    public static function getChangedFiles(IO $io, Repository $repository, array $filter): array
+    {
+        $factory  = new Factory();
+        $detector = $factory->getDetector($io, $repository);
+
+        $files = $detector->getChangedFiles($filter);
+        self::displayFilesFound($io, $files);
+        return $files;
+    }
+
+    /**
+     * Output the changed files in verbose mode
+     *
+     * @param  \CaptainHook\App\Console\IO $io
+     * @param  array<string>               $files
+     * @return void
+     */
+
+    private static function displayFilesFound(IO $io, array $files): void
+    {
+        if ($io->isVerbose()) {
+            $io->write('  <info>Changed files</info>', true, IO::VERBOSE);
+            foreach ($files as $file) {
+                $io->write('   - ' . $file, true, IO::VERBOSE);
+            }
+        }
+    }
+}

src/Hook/Condition/FileChanged.php

@@ -12,7 +12,7 @@
 namespace CaptainHook\App\Hook\Condition;
 
 use CaptainHook\App\Console\IO;
-use CaptainHook\App\Git\ChangedFiles\Detector\Factory;
+use CaptainHook\App\Git;
 use CaptainHook\App\Hook\Restriction;
 use CaptainHook\App\Hooks;
 use SebastianFeldmann\Git\Repository;
@@ -86,9 +86,6 @@ abstract public function isTrue(IO $io, Repository $repository): bool;
      */
     protected function getChangedFiles(IO $io, Repository $repository): array
     {
-        $factory  = new Factory();
-        $detector = $factory->getDetector($io, $repository);
-
-        return $detector->getChangedFiles($this->filter);
+        return Git\ChangedFiles::getChangedFiles($io, $repository, $this->filter);
     }
 }

src/Hook/Condition/FileChanged/OfType.php

@@ -76,11 +76,10 @@ public static function getRestriction(): Restriction
      */
     public function isTrue(IO $io, Repository $repository): bool
     {
-        $factory  = new Git\ChangedFiles\Detector\Factory();
-        $detector = $factory->getDetector($io, $repository);
-
-        $files = $detector->getChangedFiles(['A', 'C', 'M', 'R']);
-        $files = FileList::filterByType($files, ['of-type' => $this->suffix]);
+        $files = FileList::filterByType(
+            Git\ChangedFiles::getChangedFiles($io, $repository, ['A', 'C', 'M', 'R']),
+            ['of-type' => $this->suffix]
+        );
 
         if (count($files) > 0) {
             return true;