hanneskod\readmetester\Output\AbstractOutputtingSubscriber

Complexity

Total Complexity

5

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setOutput()	0	3	1
A	getOutput()	0	7	2
A	onErrorEvent()	0	6	2

<?php

declare(strict_types=1);

namespace hanneskod\readmetester\Output;

use hanneskod\readmetester\Event;
use Symfony\Component\Console\Output\OutputInterface;

abstract class AbstractOutputtingSubscriber
{
    private ?OutputInterface $output;

    public function onErrorEvent(Event\ErrorEvent $event): void
    {
        $this->getOutput()->writeln("<error>{$event->getMessage()}</error>");

        if ($this->getOutput()->isVerbose()) {
            $this->getOutput()->writeln("<error>{$event->getException()}</error>");
        }
    }

    public function getOutput(): OutputInterface
    {
        if (!isset($this->output)) {
            throw new \LogicException('Unable to get output, did you call setOutput()?');
        }

        return $this->output;

The expression return $this->output could return the type null which is incompatible with the type-hinted return Symfony\Component\Console\Output\OutputInterface. Consider adding an additional type-check to rule them out.

    }

    public function setOutput(OutputInterface $output): void
    {
        $this->output = $output;
    }
}