Spiral\Core\Exception\Resolver\ArgumentResolvingException

Complexity

Total Complexity

5

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	createWithParam()	0	3	1
A	__construct()	0	7	1
A	createStatic()	0	6	2
A	getParameter()	0	3	1

<?php

declare(strict_types=1);

namespace Spiral\Core\Exception\Resolver;

use Spiral\Core\Exception\Traits\ClosureRendererTrait;

final class ArgumentResolvingException extends ResolvingException
{
    use ClosureRendererTrait;

The trait Spiral\Core\Exception\Traits\ClosureRendererTrait requires the property $class which is not provided by Spiral\Core\Exception\Resolver\ArgumentResolvingException.

    private function __construct(
        private readonly \ReflectionFunctionAbstract $reflection,
        private readonly string $parameter,
        ?string $message = null,
    ) {
        $message ??= "Unable to resolve required argument `{$parameter}` when resolving `%s` %s.";
        parent::__construct($this->renderFunctionAndParameter($reflection, $message));
    }

    public static function createWithParam(\ReflectionFunctionAbstract $reflection, string $parameter): self
    {
        return new self($reflection, $parameter);
    }

    public function getParameter(): string
    {
        return $this->parameter;
    }

    protected static function createStatic(string $message, ?\Throwable $previous): static
    {
        $previous instanceof self or throw new \InvalidArgumentException(
            \sprintf('Previous exception must be an instance of %s', self::class),
        );
        return new self($previous->reflection, $previous->parameter);

Accessing reflection on the interface Throwable suggest that you code against a concrete implementation. How about adding an instanceof check?

Accessing parameter on the interface Throwable suggest that you code against a concrete implementation. How about adding an instanceof check?

    }
}