Basis\Provider\CoreProvider

Complexity

Total Complexity

1

Size/Duplication

Total Lines	46
Duplicated Lines	15.22 %

Coupling/Cohesion

Components	0
Dependencies	8

Test Coverage

Coverage

22.22%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	register()	7	34	1

<?php

namespace Basis\Provider;

use Basis\Application;
use Basis\Cache;
use Basis\Config;
use Basis\Converter;
use Basis\Dispatcher;
use Basis\Filesystem;
use Basis\Framework;
use Basis\Service;
use Basis\Http;
use GuzzleHttp\Client;
use League\Container\ServiceProvider\AbstractServiceProvider;

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

    public function register()
    {
        $this->getContainer()->share(Config::class, function () {

This code seems to be duplicated across your project.

            $app = $this->getContainer()->get(Application::class);
            $framework = $this->getContainer()->get(Framework::class);
            $fs = $this->getContainer()->get(Filesystem::class);
            $converter = $this->getContainer()->get(Converter::class);
            return new Config($app, $framework, $fs, $converter);
        });

        $this->getContainer()->share(Cache::class, function () {
            $fs = $this->getContainer()->get(Filesystem::class);
            $converter = $this->getContainer()->get(Converter::class);
            return new Cache($fs, $converter);
        });

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

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

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

$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.

        });

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