AbterPhp\Framework\Bootstrappers\Http\Middleware\EnvironmentWarningBootstrapper

Complexity

Total Complexity

2

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getBindings()	0	4	1
A	registerBindings()	0	9	1

<?php

declare(strict_types=1);

namespace AbterPhp\Framework\Bootstrappers\Http\Middleware;

use AbterPhp\Framework\Constant\Env;
use AbterPhp\Framework\Http\Middleware\EnvironmentWarning;
use AbterPhp\Framework\I18n\ITranslator;
use Opulence\Environments\Environment;
use Opulence\Ioc\Bootstrappers\Bootstrapper;
use Opulence\Ioc\Bootstrappers\ILazyBootstrapper;
use Opulence\Ioc\IContainer;

class EnvironmentWarningBootstrapper extends Bootstrapper implements ILazyBootstrapper
{
    /**
     * @return array
     */
    public function getBindings(): array
    {
        return [
            EnvironmentWarning::class,
        ];
    }

    /**
     * @param IContainer $container
     *
     * @throws \Opulence\Ioc\IocException
     */
    public function registerBindings(IContainer $container)
    {
        /** @var ITranslator $translator */
        $translator  = $container->resolve(ITranslator::class);
        $environment = Environment::getVar(Env::ENV_NAME);

        $environmentWarning = new EnvironmentWarning($translator, $environment);

It seems like $environment can also be of type null; however, parameter $environment of AbterPhp\Framework\Http\Middleware\EnvironmentWarning::__construct() does only seem to accept string, maybe add an additional type check?

        $container->bindInstance(EnvironmentWarning::class, $environmentWarning);
    }
}