uuf6429\ElderBrother\Action\PhpCsFixer

Complexity

Total Complexity

10

Size/Duplication

Total Lines	130
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	9

Test Coverage

Coverage

94.12%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	getName()	0	4	1
A	isSupported()	0	12	2
A	getCsFixer()	0	8	1
A	resolveFixers()	0	14	1
B	execute()	0	23	4

<?php

namespace uuf6429\ElderBrother\Action;

use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Finder\SplFileInfo as SfyFileInfo;
use Symfony\Component\Process\Process;
use uuf6429\ElderBrother\Change\FileList;

class PhpCsFixer extends ActionAbstract
{
    const NONE_LEVEL = 0;
    const PSR0_LEVEL = 1;
    const PSR1_LEVEL = 3;
    const PSR2_LEVEL = 7;
    const SYMFONY_LEVEL = 15;
    const CONTRIB_LEVEL = 32;

    /**
     * @var FileList
     */
    protected $files;

    /**
     * @var int|null
     */
    protected $level;

    /**
     * @var string[]|null
     */
    protected $fixers;

    /**
     * @var bool
     */
    protected $addAutomatically;

    /**
     * Runs all the provided files through PHP-CS-Fixer, fixing any code style issues.
     *
     * @param FileList      $files            The files to check
     * @param int|null      $level            (Optional, defaults to NONE_LEVEL) Fixer level to use
     * @param string[]|null $fixers           (Optional, defaults to null) Set the fixers to use
     * @param bool          $addAutomatically (Optional, default is true) Whether to add modified files to commit or not
     */
    public function __construct(FileList $files, $level = self::NONE_LEVEL, $fixers = null, $addAutomatically = true)
    {
        $this->files = $files;
        $this->level = $level;
        $this->fixers = $fixers;
        $this->addAutomatically = $addAutomatically;
    }

    /**
     * {@inheritdoc}
     */
    public function getName()
    {
        return 'PHP Code Style Fixer (PhpCsFixer)';
    }

    /**
     * {@inheritdoc}
     */
    public function isSupported()
    {
        if (!class_exists('\Symfony\CS\Fixer')) {
            $this->logger->warning(
                'PHP-CS-Fixer could not be loaded: class \Symfony\CS\Fixer not found.'
            );

            return false;
        }

        return true;
    }

    /**
     * {@inheritdoc}
     */
    public function execute(InputInterface $input, OutputInterface $output)
    {
        $fixer = $this->getCsFixer();
        $fixers = $this->resolveFixers($fixer, $this->level, $this->fixers);

It seems like $this->fixers can also be of type null; however, uuf6429\ElderBrother\Action\PhpCsFixer::resolveFixers() does only seem to accept array<integer,string>, maybe add an additional type check?

        $cache = new \Symfony\CS\FileCacheManager(false, getcwd(), $fixers);

        $progress = $this->createProgressBar($input, $output, $this->files->count());
        $progress->start();

        /** @var SfyFileInfo $file */
        foreach ($this->files->getSourceIterator() as $file) {
            $progress->setMessage('Processing ' . $file->getRelativePathname() . '...');

            if ($fixer->fixFile($file, $fixers, false, false, $cache) && $this->addAutomatically) {
                $process = new Process('git add ' . escapeshellarg($file->getRelativePathname()));
                $process->mustRun();
            }

            $progress->advance();
        }

        $progress->finish();
    }

    /**
     * @return \Symfony\CS\Fixer
     */
    private function getCsFixer()
    {
        $fixer = new \Symfony\CS\Fixer();
        $fixer->registerBuiltInFixers();
        $fixer->registerBuiltInConfigs();

        return $fixer;
    }

    /**
     * @param \Symfony\CS\Fixer $fixer
     * @param int               $level
     * @param string[]          $fixers
     *
     * @return \Symfony\CS\FixerInterface[]
     */
    private function resolveFixers($fixer, $level, $fixers)
    {
        $resolver = new \Symfony\CS\ConfigurationResolver();
        $resolver
            ->setAllFixers($fixer->getFixers())
            ->setConfig(
                \Symfony\CS\Config\Config::create()
                    ->level($level)
                    ->fixers($fixers)
            )
            ->resolve();

        return $resolver->getFixers();
    }
}