Roave\ApiCompare\Formatter\MarkdownPipedToSymfonyConsoleFormatter

Complexity

Total Complexity

3

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	write()	0	25	1
A	convertFilteredChangesToMarkdownBulletList()	0	7	1
A	__construct()	0	3	1

<?php
declare(strict_types=1);

namespace Roave\ApiCompare\Formatter;

use Roave\ApiCompare\Change;
use Roave\ApiCompare\Changes;
use Symfony\Component\Console\Output\ConsoleOutputInterface;
use Symfony\Component\Console\Output\OutputInterface;

final class MarkdownPipedToSymfonyConsoleFormatter implements OutputFormatter
{
    /**
     * @var ConsoleOutputInterface
     */
    private $output;

    public function __construct(OutputInterface $output)
    {
        $this->output = $output;

$output is of type Symfony\Component\Console\Output\OutputInterface, but the property $output was declared to be of type Symfony\Component\Console\Output\ConsoleOutputInterface. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    public function write(Changes $changes) : void
    {
        $arrayOfChanges = $changes->getIterator()->getArrayCopy();

        $this->output->writeln(
            "# Added\n"
            . implode('', $this->convertFilteredChangesToMarkdownBulletList(
                function (Change $change) : bool {
                    return $change->isAdded();
                },
                ...$arrayOfChanges
            ))
            . "\n# Changed\n"
            . implode('', $this->convertFilteredChangesToMarkdownBulletList(
                function (Change $change) : bool {
                    return $change->isChanged();
                },
                ...$arrayOfChanges
            ))
            . "\n# Removed\n"
            . implode('', $this->convertFilteredChangesToMarkdownBulletList(
                function (Change $change) : bool {
                    return $change->isRemoved();
                },
                ...$arrayOfChanges
            ))
        );
    }

    private function convertFilteredChangesToMarkdownBulletList(callable $filterFunction, Change ...$changes) : array
    {
        return array_map(
            function (Change $change) : string {
                return ' - ' . str_replace(['ADDED: ', 'CHANGED: ', 'REMOVED: '], '', trim((string)$change)) . "\n";
            },
            array_filter($changes, $filterFunction)
        );
    }
}