Basis\Provider\CoreProvider

Complexity

Total Complexity

1

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	7

Test Coverage

Coverage

94.44%

Importance

Changes	2
Bugs	2	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	register()	0	25	1

<?php

namespace Basis\Provider;

use Basis\Config;
use Basis\Converter;
use Basis\Dispatcher;
use Basis\Filesystem;
use Basis\Framework;
use Basis\Http;
use League\Container\ServiceProvider\AbstractServiceProvider;
use LinkORB\Component\Etcd\Client;

class CoreProvider extends AbstractServiceProvider
{
    protected $provides = [
        Config::class,
        Converter::class,
        Dispatcher::class,
        Framework::class,
        Http::class,
    ];

    public function register()
    {
        $this->getContainer()->share(Config::class, function () {
            $framework = $this->getContainer()->get(Framework::class);
            $fs = $this->getContainer()->get(Filesystem::class);
            $converter = $this->getContainer()->get(Converter::class);
            return new Config($framework, $fs, $converter);
        });

        $this->getContainer()->share(Converter::class, function () {
            return new Converter();
        });

        $this->getContainer()->share(Dispatcher::class, function () {
            return new Dispatcher($this->getContainer()->get(Client::class));
        });

        $this->getContainer()->share(Framework::class, function () {
            return new Framework($this->getContainer());

The call to Framework::__construct() has too many arguments starting with $this->getContainer().

        });

        $this->getContainer()->share(Http::class, function () {
            return new Http($this->getContainer());

$this->getContainer() of type object<League\Container\ContainerInterface> is not a sub-type of object<Basis\Application>. It seems like you assume a concrete implementation of the interface League\Container\ContainerInterface to be always present.

        });
    }
}