AppBundle\ConsolidateUsedFiles\Task::consolidate()

Complexity

Conditions	3
Paths	2

Size

Total Lines	26
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

<?php
    public function consolidate($userProvidedPathToConsolidate, StyleInterface $ioStyle = null)
    {
        if ($ioStyle === null) {
            $ioStyle = new NullStyle();
        }
        $ioStyle->progressStart(4);

        $pathToConsolidate = FileSystem::getRealPathToReadableAndWritableFile($userProvidedPathToConsolidate);
        if (!$pathToConsolidate) {

The expression $pathToConsolidate of type null|string is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

            $message = $userProvidedPathToConsolidate . ' has to be a file both readable and writable.';
            $ioStyle->error($message);
            throw new \InvalidArgumentException($message);
        }
        $ioStyle->progressAdvance();

        $usedFiles = FileSystem::readFileIntoArray($pathToConsolidate);
        $ioStyle->progressAdvance();

        $usedFiles = array_unique($usedFiles);

It seems like $usedFiles can also be of type boolean; however, parameter $array of array_unique() does only seem to accept array, maybe add an additional type check?

        sort($usedFiles);
        $ioStyle->progressAdvance();

        FileSystem::writeArrayToFile($usedFiles, $pathToConsolidate);
        $ioStyle->progressFinish();

        $ioStyle->success('Finished consolidating ' . $pathToConsolidate);
    }