Basis\Provider\CoreProvider::register()

Complexity

Conditions	1
Paths	1

Size

Total Lines

44

Duplication

Lines	7
Ratio	15.91 %

Code Coverage

Tests	9
CRAP Score	1.41

Importance

Changes

0

<?php
    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(Lock::class, function () {
            $redis = $this->getContainer()->get(PredisClient::class);
            return new Lock($redis);
        });

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

        $this->getContainer()->share(Dispatcher::class, function () {
            $context = $this->getContainer()->get(Context::class);
            $client = $this->getContainer()->get(GuzzleHttpClient::class);
            $service = $this->getContainer()->get(Service::class);
            return new Dispatcher($client, $context, $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));
        });

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

The call to Context::__construct() has too many arguments starting with $this->getContainer()->get(\Basis\Application::class).

        });
    }
}