PSFS\services\MigrationService::getConnectionManager()

Complexity

Conditions	1
Paths	1

Size

Total Lines	15
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function getConnectionManager(string $module, string $path): array {
        $modulePath = str_replace(CORE_DIR . DIRECTORY_SEPARATOR, '', $path . $module);
        $generatorConfig = $this->getConfigGenerator($modulePath);

        $manager = new MigrationManager();
        $manager->setGeneratorConfig($generatorConfig);
        $manager->setSchemas($this->getSchemas(
            $generatorConfig->getSection('paths')['schemaDir'],
            $generatorConfig->getSection('generator')['recursive'])
        );
        $connections = $generatorConfig->getBuildConnections();
        $manager->setConnections($connections);
        $manager->setMigrationTable($generatorConfig->getConfigProperty('migrations.tableName'));

It seems like $generatorConfig->getConfigProperty('migrations.tableName') can also be of type null; however, parameter $migrationTable of Propel\Generator\Manager\MigrationManager::setMigrationTable() does only seem to accept string, maybe add an additional type check?

        $manager->setWorkingDirectory($generatorConfig->getSection('paths')['migrationDir']);
        return [$manager, $generatorConfig];
    }