Prozorov\Repositories\Resolvers\ContainerAwareResolver

Complexity

Total Complexity

5

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	fail()	0	6	1
A	resolve()	0	13	3
A	__construct()	0	3	1

<?php

declare(strict_types=1);

namespace Prozorov\Repositories\Resolvers;

use Prozorov\Repositories\Contracts\{RepositoryInterface, ResolverInterface};
use Prozorov\Repositories\Exceptions\CouldNotResolve;
use Psr\Container\{ContainerInterface, ContainerExceptionInterface};
use InvalidArgumentException;

class ContainerAwareResolver implements ResolverInterface
{
    /**
     * @var ContainerInterface $container
     */
    protected $container;

    public function __construct(ContainerInterface $container)
    {
        $this->container = $container;
    }

    /**
     * @inheritDoc
     */
    public function resolve(string $className): RepositoryInterface
    {
        try {
            $resolved = $this->container->get($className);

            if (! ($resolved instanceof RepositoryInterface)) {
                $class = get_class($resolved);

                $message = $class . ' does not implements RepositoryInterface. Refer to documentation';

The assignment to $message is dead and can be removed.

                $this->fail('Container must return ');

In this branch, the function will implicitly return null which is incompatible with the type-hinted return Prozorov\Repositories\Contracts\RepositoryInterface. Consider adding a return statement or allowing null as return value.

            }
        } catch (ContainerExceptionInterface $exception) {
            $this->fail($exception->getMessage());

The method getMessage() does not exist on Psr\Container\ContainerExceptionInterface. It seems like you code against a sub-type of Psr\Container\ContainerExceptionInterface such as Illuminate\Contracts\Container\BindingResolutionException.

        }
    }

    /**
     * fail.
     *
     * @access	protected
     * @param	string	$className	
     * @return	void
     */
    protected function fail(string $className, string $message = ''): void
    {
        $exception = new CouldNotResolve($message);
        $exception->setRepositoryCode($className);

        throw $exception;
    }
}