Horat1us\Yii\BootstrapGroup

Complexity

Total Complexity

2

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	bootstrap()	0	8	2

<?php

declare(strict_types=1);

namespace Horat1us\Yii;

use yii\base;
use yii\di;

/**
 * Class ComposeBootstrap
 * @package Horat1us\Yii
 */
class BootstrapGroup extends base\BaseObject implements base\BootstrapInterface
{
    /**
     * All of this bootstraps will be used.
     * @var array|string|base\BootstrapInterface references
     */
    public $items = [];

    /**
     * @param base\Application $app
     * @throws base\InvalidConfigException
     */
    public function bootstrap($app): void
    {
        foreach ($this->items as $reference) {

The expression $this->items of type array|string|object<yii\base\BootstrapInterface> is not guaranteed to be traversable. How about adding an additional type check?

            /** @var base\BootstrapInterface $bootstrap */
            $bootstrap = di\Instance::ensure($reference, base\BootstrapInterface::class);
            $bootstrap->bootstrap($app);
        }
    }
}