humbug / box

src/Phar/Differ/FilenameDiffer.php

<?php

declare(strict_types=1);

/*
 * This file is part of the box project.
 *
 * (c) Kevin Herrera <kevin@herrera.io>
 *     Théo Fidry <theo.fidry@gmail.com>
 *
 * This source file is subject to the MIT license that is bundled
 * with this source code in the file LICENSE.
 */

namespace KevinGH\Box\Phar\Differ;

use Fidry\Console\IO;

The type Fidry\Console\IO was not found. Maybe you did not declare it correctly or list all dependencies?

use KevinGH\Box\Console\PharInfoRenderer;
use KevinGH\Box\Phar\PharInfo;
use SplFileInfo;
use Symfony\Component\Console\Output\BufferedOutput;
use Symfony\Component\Finder\Finder;
use function array_diff;
use function array_map;
use function array_sum;
use function count;
use function explode;
use function iterator_to_array;
use function sprintf;
use function str_replace;

final class FilenameDiffer implements Differ
{
    public function diff(
        PharInfo $pharInfoA,
        PharInfo $pharInfoB,
        IO $io,
    ): void {
        $pharAFiles = self::collectFiles($pharInfoA);
        $pharBFiles = self::collectFiles($pharInfoB);

        $diffResult = [
            array_diff($pharAFiles, $pharBFiles),
            array_diff($pharBFiles, $pharAFiles),
        ];
        $diffCount = array_sum(array_map('count', $diffResult));

        if (0 === $diffCount) {
            $io->writeln(Differ::NO_DIFF_MESSAGE);

            return;
        }

        self::printDiff(
            $pharInfoA,
            $pharInfoB,
            $diffResult[0],
            $diffResult[1],
            $io,
        );
    }

    /**
     * @param list<non-empty-string> $filesInANotInB
     * @param list<non-empty-string> $filesInBNotInA

The type KevinGH\Box\Phar\Differ\list was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    private static function printDiff(
        PharInfo $pharInfoA,
        PharInfo $pharInfoB,
        array $filesInANotInB,
        array $filesInBNotInA,
        IO $io,
    ): void {
        $io->writeln(sprintf(
            '--- Files present in "%s" but not in "%s"',
            $pharInfoA->getFileName(),
            $pharInfoB->getFileName(),
        ));
        $io->writeln(sprintf(
            '+++ Files present in "%s" but not in "%s"',
            $pharInfoB->getFileName(),
            $pharInfoA->getFileName(),
        ));

        $io->newLine();

        self::renderPaths('-', $pharInfoA, $filesInANotInB, $io);
        $io->newLine();
        self::renderPaths('+', $pharInfoB, $filesInBNotInA, $io);

        $io->newLine(2);

        $io->error(
            sprintf(
                '%d file(s) difference',
                count($filesInANotInB) + count($filesInBNotInA),
            ),
        );
    }

    /**
     * @param list<non-empty-string> $paths
     */
    private static function renderPaths(string $symbol, PharInfo $pharInfo, array $paths, IO $io): void

The parameter $paths is not used and could be removed.

    {
        $bufferedOutput = new BufferedOutput(
            $io->getVerbosity(),
            $io->isDecorated(),
            $io->getOutput()->getFormatter(),
        );

        PharInfoRenderer::renderContent(
            $bufferedOutput,
            $pharInfo,
            false,
            false,
        );

        $lines = array_map(
            static fn (string $line) => '' === $line ? '' : $symbol.' '.$line,
            explode(
                PHP_EOL,
                $bufferedOutput->fetch(),
            ),
        );

        $io->write($lines);
    }

    /**
     * @return string[]
     */
    private static function collectFiles(PharInfo $pharInfo): array
    {
        $basePath = $pharInfo->getTmp().DIRECTORY_SEPARATOR;

        return array_map(
            static fn (SplFileInfo $fileInfo): string => str_replace($basePath, '', $fileInfo->getRealPath()),
            iterator_to_array(
                Finder::create()
                    ->files()
                    ->in($basePath)
                    ->ignoreDotFiles(false),
                false,
            ),
        );
    }
}