alecrabbit / php-console-spinner

src/Spinner/Core/Widget/Builder/WidgetCompositeBuilder.php

<?php

declare(strict_types=1);

namespace AlecRabbit\Spinner\Core\Widget\Builder;

use AlecRabbit\Spinner\Contract\IFrame;
use AlecRabbit\Spinner\Core\Contract\IIntervalComparator;
use AlecRabbit\Spinner\Core\Widget\Builder\A\AWidgetBuilder;
use AlecRabbit\Spinner\Core\Widget\Contract\IWidgetComposite;
use AlecRabbit\Spinner\Core\Widget\Contract\IWidgetCompositeBuilder;
use AlecRabbit\Spinner\Core\Widget\Contract\IWidgetRevolver;
use AlecRabbit\Spinner\Core\Widget\WidgetComposite;
use AlecRabbit\Spinner\Exception\LogicException;

/**
 * @psalm-suppress PossiblyNullArgument
 */
final class WidgetCompositeBuilder extends AWidgetBuilder implements IWidgetCompositeBuilder
{
    private ?IIntervalComparator $intervalComparator = null;

    public function build(): IWidgetComposite
    {
        $this->validate();

        return new WidgetComposite(
            revolver: $this->revolver,

It seems like $this->revolver can also be of type null; however, parameter $revolver of AlecRabbit\Spinner\Core\Widget\WidgetComposite::__construct() does only seem to accept AlecRabbit\Spinner\Core\Widget\Contract\IWidgetRevolver, maybe add an additional type check?

            leadingSpacer: $this->leadingSpacer,

It seems like $this->leadingSpacer can also be of type null; however, parameter $leadingSpacer of AlecRabbit\Spinner\Core\Widget\WidgetComposite::__construct() does only seem to accept AlecRabbit\Spinner\Contract\IFrame, maybe add an additional type check?

            trailingSpacer: $this->trailingSpacer,

It seems like $this->trailingSpacer can also be of type null; however, parameter $trailingSpacer of AlecRabbit\Spinner\Core\Widget\WidgetComposite::__construct() does only seem to accept AlecRabbit\Spinner\Contract\IFrame, maybe add an additional type check?

            intervalComparator: $this->intervalComparator,

It seems like $this->intervalComparator can also be of type null; however, parameter $intervalComparator of AlecRabbit\Spinner\Core\Widget\WidgetComposite::__construct() does only seem to accept AlecRabbit\Spinner\Core\Contract\IIntervalComparator, maybe add an additional type check?

        );
    }

    protected function validate(): void
    {
        parent::validate();

        match (true) {
            $this->intervalComparator === null => throw new LogicException('Interval comparator is not set.'),
            default => null,
        };
    }

    public function withWidgetRevolver(IWidgetRevolver $revolver): IWidgetCompositeBuilder
    {
        $clone = clone $this;
        $clone->revolver = $revolver;
        return $clone;
    }

    public function withLeadingSpacer(IFrame $leadingSpacer): IWidgetCompositeBuilder
    {
        $clone = clone $this;
        $clone->leadingSpacer = $leadingSpacer;
        return $clone;
    }

    public function withTrailingSpacer(IFrame $trailingSpacer): IWidgetCompositeBuilder
    {
        $clone = clone $this;
        $clone->trailingSpacer = $trailingSpacer;
        return $clone;
    }

    public function withIntervalComparator(IIntervalComparator $intervalComparator): IWidgetCompositeBuilder
    {
        $clone = clone $this;
        $clone->intervalComparator = $intervalComparator;
        return $clone;
    }
}