Liip\MonitorBundle\DependencyInjection\DoctrineMigrations\V2MigrationsLoader

Complexity

Total Complexity

26

Size/Duplication

Total Lines	219
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	11

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	process()	0	12	3
F	createMigrationConfigurationService()	0	77	11
A	createTemporaryConfiguration()	0	39	3
A	getConnection()	0	21	4
A	getPredefinedMigrations()	0	27	5

<?php

declare(strict_types=1);

namespace Liip\MonitorBundle\DependencyInjection\DoctrineMigrations;

use Doctrine\DBAL\Connection;
use Doctrine\DBAL\Driver\PDO\SQLite\Driver;
use Doctrine\DBAL\Driver\PDOSqlite\Driver as LegacyDriver;
use Doctrine\Migrations\Configuration\AbstractFileConfiguration;
use Doctrine\Migrations\Configuration\Configuration as DoctrineMigrationConfiguration;
use Liip\MonitorBundle\DoctrineMigrations\Configuration;
use Symfony\Component\DependencyInjection\ChildDefinition;
use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
use Symfony\Component\DependencyInjection\ContainerBuilder;
use Symfony\Component\DependencyInjection\Definition;
use Symfony\Component\DependencyInjection\Reference;

/**
 * Class V2MigrationsLoader
 */
final class V2MigrationsLoader extends AbstractDoctrineMigrationsLoader implements CompilerPassInterface
{
    /**
     * Connection object needed for correct migration loading.
     *
     * @var Connection
     */
    private $fakeConnection;

    /**
     * Tuple (migrationsConfiguration, tempConfiguration) for doctrine migrations check.
     *
     * @var array
     */
    private $migrationConfigurationsServices = [];

    /**
     * @inheritDoc
     */
    public function process(ContainerBuilder $container)
    {
        foreach ($this->migrationConfigurationsServices as $services) {
            [$configurationService, $configuration] = $services;

The variable $configurationService does not exist. Did you forget to declare it?

The variable $configuration does not exist. Did you forget to declare it?

            /** @var Definition $configurationService */
            /** @var DoctrineMigrationConfiguration $configuration */
            $versions = $this->getPredefinedMigrations($container, $configuration, $this->fakeConnection);
            if ($versions) {

The expression $versions of type string[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

                $configurationService->addMethodCall('registerMigrations', [$versions]);
            }
        }
    }

    /**
     * @inheritDoc
     */
    public function createMigrationConfigurationService(
        ContainerBuilder $container,
        string $connectionName,
        string $serviceId,
        string $filename = null
    ): void {
        if (!$container->has(Configuration::class)) {
            $container->register(Configuration::class)
                ->setAbstract(true)
                ->setPublic(true)
                ->setArguments([null])
                ->addMethodCall('setContainer', [new Reference('service_container')]);
        }

        $configuration = $this->createTemporaryConfiguration($container, $this->getConnection(), $filename);

        $serviceConfiguration = new ChildDefinition(Configuration::class);

        $this->migrationConfigurationsServices[] = [$serviceConfiguration, $configuration];

        $serviceConfiguration->replaceArgument(
            0,
            new Reference(sprintf('doctrine.dbal.%s_connection', $connectionName))
        );

        if ($configuration->getMigrationsNamespace()) {

The method getMigrationsNamespace() does not seem to exist on object<Doctrine\Migrations\Configuration\Configuration>.

            $serviceConfiguration->addMethodCall(
                'setMigrationsNamespace',
                [$configuration->getMigrationsNamespace()]

The method getMigrationsNamespace() does not seem to exist on object<Doctrine\Migrations\Configuration\Configuration>.

            );
        }

        if ($configuration->getMigrationsTableName()) {

The method getMigrationsTableName() does not seem to exist on object<Doctrine\Migrations\Configuration\Configuration>.

            $serviceConfiguration->addMethodCall(
                'setMigrationsTableName',
                [$configuration->getMigrationsTableName()]

The method getMigrationsTableName() does not seem to exist on object<Doctrine\Migrations\Configuration\Configuration>.

            );
        }

        if ($configuration->getMigrationsColumnName()) {

The method getMigrationsColumnName() does not seem to exist on object<Doctrine\Migrations\Configuration\Configuration>.

            $serviceConfiguration->addMethodCall(
                'setMigrationsColumnName',
                [$configuration->getMigrationsColumnName()]

The method getMigrationsColumnName() does not seem to exist on object<Doctrine\Migrations\Configuration\Configuration>.

            );
        }

        if ($configuration->getName()) {

The method getName() does not seem to exist on object<Doctrine\Migrations\Configuration\Configuration>.

            $serviceConfiguration->addMethodCall('setName', [$configuration->getName()]);

The method getName() does not seem to exist on object<Doctrine\Migrations\Configuration\Configuration>.

        }

        if ($configuration->getMigrationsDirectory()) {

The method getMigrationsDirectory() does not seem to exist on object<Doctrine\Migrations\Configuration\Configuration>.

            $directory = $configuration->getMigrationsDirectory();

The method getMigrationsDirectory() does not seem to exist on object<Doctrine\Migrations\Configuration\Configuration>.

            $pathPlaceholders = ['kernel.root_dir', 'kernel.cache_dir', 'kernel.logs_dir'];
            foreach ($pathPlaceholders as $parameter) {
                $kernelDir = realpath($container->getParameter($parameter));
                if (0 === strpos(realpath($directory), $kernelDir)) {
                    $directory = str_replace($kernelDir, "%{$parameter}%", $directory);
                    break;
                }
            }

            $serviceConfiguration->addMethodCall(
                'setMigrationsDirectory',
                [$directory]
            );
        }

        $serviceConfiguration->addMethodCall('configure', []);

        if ($configuration->areMigrationsOrganizedByYear()) {
            $serviceConfiguration->addMethodCall('setMigrationsAreOrganizedByYear', [true]);
        } elseif ($configuration->areMigrationsOrganizedByYearAndMonth()) {
            $serviceConfiguration->addMethodCall('setMigrationsAreOrganizedByYearAndMonth', [true]);
        }

        $container->setDefinition($serviceId, $serviceConfiguration);
    }

    /**
     * Creates in-memory migration configuration for setting up container service.
     *
     * @param ContainerBuilder $container  The container
     * @param Connection       $connection Fake connection
     * @param string           $filename   Migrations configuration file
     */
    private function createTemporaryConfiguration(
        ContainerBuilder $container,
        Connection $connection,
        string $filename = null
    ): DoctrineMigrationConfiguration {
        if (null === $filename) {
            // this is configured from migrations bundle
            return new DoctrineMigrationConfiguration($connection);

The call to Configuration::__construct() has too many arguments starting with $connection.

        }

        // -------
        // This part must be in sync with Doctrine\Migrations\Tools\Console\Helper\ConfigurationHelper::loadConfig
        $map = [
            'xml' => '\XmlConfiguration',
            'yaml' => '\YamlConfiguration',
            'yml' => '\YamlConfiguration',
            'php' => '\ArrayConfiguration',
            'json' => '\JsonConfiguration',
        ];
        // --------

        $filename = $container->getParameterBag()->resolveValue($filename);
        $info = pathinfo($filename);
        // check we can support this file type
        if (empty($map[$info['extension']])) {
            throw new \InvalidArgumentException('Given config file type is not supported');
        }

        $class = 'Doctrine\Migrations\Configuration';
        $class .= $map[$info['extension']];
        // -------

        /** @var AbstractFileConfiguration $configuration */
        $configuration = new $class($connection);
        $configuration->load($filename);
        $configuration->validate();

        return $configuration;
    }

    private function getConnection(): Connection
    {
        if ($this->fakeConnection === null) {
            if (!class_exists(Connection::class)) {
                throw new \InvalidArgumentException(
                    sprintf(
                        'Can not configure doctrine migration checks, because of absence of "%s" class',
                        Connection::class
                    )
                );
            }

            $driver = class_exists(Driver::class)
                ? new Driver()
                : new LegacyDriver();

The class Doctrine\DBAL\Driver\PDOSqlite\Driver has been deprecated with message: Use {@link PDO\SQLite\Driver} instead.

            $this->fakeConnection = new Connection([], $driver);
        }

        return $this->fakeConnection;
    }

    /**
     * Return key-value array with migration version as key and class as a value defined in config file.
     *
     * @param ContainerBuilder               $container  The container
     * @param DoctrineMigrationConfiguration $config     Current configuration
     * @param Connection                     $connection Fake connections
     *
     * @return string[]
     */
    private function getPredefinedMigrations(ContainerBuilder $container, DoctrineMigrationConfiguration $config, Connection $connection): array
    {
        $result = [];

        $diff = new Configuration($connection);

The call to Configuration::__construct() has too many arguments starting with $connection.

        if ($namespace = $config->getMigrationsNamespace()) {

The method getMigrationsNamespace() does not seem to exist on object<Doctrine\Migrations\Configuration\Configuration>.

$namespace is not used, you could remove the assignment.

            $diff->setMigrationsNamespace($config->getMigrationsNamespace());

The method getMigrationsNamespace() does not seem to exist on object<Doctrine\Migrations\Configuration\Configuration>.

The method setMigrationsNamespace() does not seem to exist on object<Liip\MonitorBundle\DoctrineMigrations\Configuration>.

        }

        if ($dir = $config->getMigrationsDirectory()) {

The method getMigrationsDirectory() does not seem to exist on object<Doctrine\Migrations\Configuration\Configuration>.

            $diff->setMigrationsDirectory($dir);

The method setMigrationsDirectory() does not seem to exist on object<Liip\MonitorBundle\DoctrineMigrations\Configuration>.

        }

        $diff->setContainer($container);
        $diff->configure();

        foreach ($config->getMigrations() as $version) {

The method getMigrations() does not seem to exist on object<Doctrine\Migrations\Configuration\Configuration>.

            $result[$version->getVersion()] = get_class($version->getMigration());
        }

        foreach ($diff->getAvailableVersions() as $version) {

The method getAvailableVersions() does not seem to exist on object<Liip\MonitorBundle\DoctrineMigrations\Configuration>.

            unset($result[$version]);
        }

        return $result;
    }
}