Liip\MonitorBundle\Check\DoctrineMigrationsCollection

Complexity

Total Complexity

4

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	getChecks()	0	14	3

<?php

namespace Liip\MonitorBundle\Check;

use Doctrine\Migrations\Configuration\Configuration;
use Laminas\Diagnostics\Check\CheckCollectionInterface;
use Laminas\Diagnostics\Check\CheckInterface;
use Laminas\Diagnostics\Check\DoctrineMigration as LaminasDoctrineMigration;
use Symfony\Component\DependencyInjection\ContainerInterface;

/**
 * Class DoctrineMigrationsCollection.
 */
class DoctrineMigrationsCollection implements CheckCollectionInterface
{
    /**
     * DI Container.
     *
     * @var ContainerInterface
     */
    private $container;

    /**
     * Available checks.
     *
     * @var CheckInterface[]
     */
    private $checks;

    /**
     * Migrations configuration service ids.
     *
     * @var string[]
     */
    private $migrations;

    /**
     * DoctrineMigrationsCollection constructor.
     *
     * @param ContainerInterface $container  DI container
     * @param Configuration[]    $migrations Migrations configuration service ids
     */
    public function __construct(
        ContainerInterface $container,
        array $migrations
    ) {
        $this->container = $container;
        $this->migrations = $migrations;

It seems like $migrations of type array<integer,object<Doctrine\Migrations\Configuration\Configuration>> is incompatible with the declared type array<integer,string> of property $migrations.

    }

    public function getChecks()
    {
        if (null === $this->checks) {
            $this->checks = [];
            foreach ($this->migrations as $key => $migration) {
                $check = new LaminasDoctrineMigration($this->container->get($migration));

$this->container->get($migration) is of type object|null, but the function expects a object<Doctrine\Migrations\Configuration\Configuration>.

                $check->setLabel(sprintf('Doctrine migrations "%s"', $key));

                $this->checks[$key] = $check;
            }
        }

        return $this->checks;
    }
}