LIN3S\CS\Application::doRun()

Complexity

Conditions	14
Paths	92

Size

Total Lines

65

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function doRun(InputInterface $input, OutputInterface $output)
    {
        $output->writeln(sprintf('<fg=white;options=bold;bg=red>%s</fg=white;options=bold;bg=red>', $this->name));
        $output->writeln('<info>Fetching files...</info>');
        $files = Git::committedFiles();

        $output->writeln('<info>Check composer</info>');
        Composer::check($files);

It seems like $files defined by \LIN3S\CS\Git\Git::committedFiles() on line 53 can also be of type null; however, LIN3S\CS\Checker\Composer::check() does only seem to accept array, maybe add an additional type check?

        if (in_array('phpcsfixer', $this->parameters['enabled'], true)) {
            $output->writeln('<info>Fixing PHP code style with PHP-CS-Fixer</info>');
            PhpCsFixer::check($files, $this->parameters);

It seems like $files defined by \LIN3S\CS\Git\Git::committedFiles() on line 53 can also be of type null; however, LIN3S\CS\Checker\PhpCsFixer::check() does only seem to accept array, maybe add an additional type check?

        }

        if (in_array('twigcs', $this->parameters['enabled'], true)) {
            $output->writeln('<info>Linting the Twig files with TwigCS</info>');
            $twigCsResult = TwigCs::check($files, $this->parameters);

It seems like $files defined by \LIN3S\CS\Git\Git::committedFiles() on line 53 can also be of type null; however, LIN3S\CS\Checker\TwigCs::check() does only seem to accept array, maybe add an additional type check?

            if (count($twigCsResult) > 0) {
                foreach ($twigCsResult as $error) {
                    $output->writeln($error->output());
                }
                throw new CheckFailException('TwigCS');
            }
        }

        if (in_array('phpmd', $this->parameters['enabled'], true)) {
            $output->writeln('<info>Checking code mess with PHPMD</info>');
            $phpmdResult = Phpmd::check($files, $this->parameters);

It seems like $files defined by \LIN3S\CS\Git\Git::committedFiles() on line 53 can also be of type null; however, LIN3S\CS\Checker\Phpmd::check() does only seem to accept array, maybe add an additional type check?

            if (count($phpmdResult) > 0) {
                foreach ($phpmdResult as $error) {
                    $output->writeln($error->output());
                }
                throw new CheckFailException('PHPMD');
            }
        }

        if (in_array('stylelint', $this->parameters['enabled'], true)) {
            $output->writeln('<info>Checking scss files with Stylelint</info>');
            $stylelintResult = Stylelint::check($files, $this->parameters);

It seems like $files defined by \LIN3S\CS\Git\Git::committedFiles() on line 53 can also be of type null; however, LIN3S\CS\Checker\Stylelint::check() does only seem to accept array, maybe add an additional type check?

            if (count($stylelintResult) > 0) {
                foreach ($stylelintResult as $error) {
                    $output->writeln($error->output());
                }
                throw new CheckFailException('Stylelint', 'Please, execute "npm update -g stylelint');
            }
        }

        if (in_array('eslint', $this->parameters['enabled'], true)) {
            $output->writeln('<info>Checking js files with ESLint</info>');
            $esLintResult = EsLint::check($files, $this->parameters);

It seems like $files defined by \LIN3S\CS\Git\Git::committedFiles() on line 53 can also be of type null; however, LIN3S\CS\Checker\EsLint::check() does only seem to accept array, maybe add an additional type check?

            if (count($esLintResult) > 0) {
                foreach ($esLintResult as $error) {
                    $output->writeln($error->output());
                }
                throw new CheckFailException(
                    'ESLint',
                    'Please, execute "npm update -g eslint eslint-plugin-class-property ' .
                    'eslint-plugin-react eslint-plugin-babel babel-eslint"'
                );
            }
        }

        Git::addFiles($files, $this->parameters['root_directory']);

It seems like $files defined by \LIN3S\CS\Git\Git::committedFiles() on line 53 can also be of type null; however, LIN3S\CS\Git\Git::addFiles() does only seem to accept array, maybe add an additional type check?

        $output->writeln('<info>Nice commit man!</info>');
    }