hanneskod\readmetester\Output\AbstractOutputtingSubscriber

Complexity

Total Complexity

3

Size/Duplication

Total Lines	16
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

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

<?php

declare(strict_types=1);

namespace hanneskod\readmetester\Output;

use Symfony\Component\Console\Output\OutputInterface;

abstract class AbstractOutputtingSubscriber
{
    private ?OutputInterface $output;

    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;
    }
}