Yiisoft\Di\CompositeContainer::get()

Complexity

Conditions	13
Paths	15

Size

Total Lines	52
Code Lines	28

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	26
CRAP Score	13.0615

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function get($id)
    {
        /** @psalm-suppress TypeDoesNotContainType */
        if (!is_string($id)) {
            throw new InvalidArgumentException("Id must be a string, {$this->getVariableType($id)} given.");
        }

        if ($id === StateResetter::class) {
            $resetters = [];
            foreach ($this->containers as $container) {
                if ($container->has(StateResetter::class)) {
                    $resetters[] = $container->get(StateResetter::class);
                }
            }
            $stateResetter = new StateResetter($this);
            $stateResetter->setResetters($resetters);

            return $stateResetter;
        }

        if ($this->isTagAlias($id)) {
            $tags = [];
            foreach ($this->containers as $container) {
                if (!$container instanceof Container) {
                    continue;
                }
                if ($container->has($id)) {
                    $tags = array_merge($container->get($id), $tags);

It seems like $container->get($id) can also be of type object; however, parameter $arrays of array_merge() does only seem to accept array, maybe add an additional type check?

                }
            }

            return $tags;
        }

        foreach ($this->containers as $container) {
            if ($container->has($id)) {
                return $container->get($id);
            }
        }


        // Collect details from containers
        $exceptions = [];
        foreach ($this->containers as $container) {
            try {
                $container->get($id);
            } catch (Throwable $t) {
                $exceptions[] = $t;
            }
        }

        throw new CompositeNotFoundException($exceptions);
    }