Antidot\Container\MarshalDelegatorsConfig::__invoke()

Complexity

Conditions	3
Paths	3

Size

Total Lines	20
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	14
CRAP Score	3.0026

Importance

Changes	1
Bugs	0	Features	1

<?php
    public function __invoke(ContainerConfig $dependencies): ContainerConfig
    {
        foreach ($dependencies->get('delegators') as $service => $delegatorNames) {
            $factory = $this->delegateFactories($dependencies, $service);
            if (!is_callable($factory)) {
                continue;
            }
            $dependencies->set(ContainerDelegatorFactory::class, static function () use ($delegatorNames, $factory) {
                return new ContainerDelegatorFactory($delegatorNames, $factory);

It seems like $factory can also be of type null; however, parameter $factory of Antidot\Container\ContainerDelegatorFactory::__construct() does only seem to accept callable, maybe add an additional type check?

            });
            $dependencies->set(
                $service,
                static function (ContainerInterface $container) use ($service) {
                    $callable = $container->get(ContainerDelegatorFactory::class);
                    return $callable($container, $service);
                }
            );
        }

        return $dependencies;
    }