Laganica\Di\Resolver\AliasResolver

Complexity

Total Complexity

2

Size/Duplication

Total Lines	16
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

60%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	resolve()	0	10	2

<?php

namespace Laganica\Di\Resolver;

use InvalidArgumentException;
use Laganica\Di\Definition\AliasDefinition;

/**
 * Class AliasResolver
 *
 * @package Laganica\Di\Resolver
 */
class AliasResolver extends ReflectionResolver
{
    /**
     * @inheritDoc
     */
    public function resolve($definition)
    {
        if (!$definition instanceof AliasDefinition) {
            $definitionClass = AliasDefinition::class;

            throw new InvalidArgumentException("Argument $definition must be $definitionClass");
        }

        return $this->getContainer()->get($definition->getValue());

It seems like $definition->getValue() targeting Laganica\Di\Definition\Definition::getValue() can also be of type object<Closure>; however, Psr\Container\ContainerInterface::get() does only seem to accept string, maybe add an additional type check?

    }
}