JoaoRobertoPB\PhpGitHookSniffer\Pipelines\CsFixerStage

Complexity

Total Complexity

4

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	28	4

<?php

namespace JoaoRobertoPB\PhpGitHookSniffer\Pipelines;

use JoaoRobertoPB\PhpGitHookSniffer\Contracts\PipelineInterface;

class CsFixerStage implements PipelineInterface
{
    /**
     * Runs php -l on PHP staged files.
     *
     *
     * @return void
     */
    public function __invoke($payload)
    {
        echo "[  ] ----- Fix Code Standards -----\n";

        foreach ($payload as $file) {
            $fileName = trim($file);
            $ext = pathinfo($fileName, PATHINFO_EXTENSION);

            $return = 0;

            if ('php' != $ext) {
                continue;
            }

            $csOutput = [];
            exec(__DIR__.'/../../../../../vendor/bin/php-cs-fixer --using-cache=no fix '.escapeshellarg($fileName), $csOutput, $return);
            exec("git add {$fileName}");
        }

        if ($return != 0) {

The variable $return seems to be defined by a foreach iteration on line 19. Are you sure the iterator is never empty, otherwise this variable is not defined?

            echo implode("\n", $csOutput), "\n";

The variable $csOutput does not seem to be defined for all execution paths leading up to this point.

            echo "[FAIL] ----- Fix Code Standards -----\n";
            exit(1);

Using exit here is not recommended.

        }

        echo "[OK] ----- Fix Code Standards -----\n";

        return $payload;
    }
}