Limoncello\Application\Data\FileMigrationRunner

Complexity

Total Complexity

6

Size/Duplication

Total Lines	85
Duplicated Lines	21.18 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getMigrationsPath()	0	4	1
A	setMigrationsPath()	0	6	1
A	migrate()	18	18	1
A	getMigrationClasses()	0	4	1
A	setMigrationClasses()	0	6	1

<?php namespace Limoncello\Application\Data;

/**
 * Copyright 2015-2017 info@neomerx.com
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

use Limoncello\Contracts\Commands\IoInterface;
use Limoncello\Contracts\FileSystem\FileSystemInterface;
use Psr\Container\ContainerInterface;

/**
 * @package Limoncello\Application
 */
class FileMigrationRunner extends BaseMigrationRunner
{
    /**
     * @var string
     */
    private $migrationsPath;

    /**
     * @var string[]
     */
    private $migrationClasses;

    /**
     * @param IoInterface $inOut
     * @param string      $migrationsPath
     */
    public function __construct(IoInterface $inOut, string $migrationsPath)
    {
        parent::__construct($inOut);

        $this->setMigrationsPath($migrationsPath);
    }

    /**
     * @inheritdoc
     */
    public function migrate(ContainerInterface $container): void

This method seems to be duplicated in your project.

    {
        // read & remember classes to migrate...
        assert($container->has(FileSystemInterface::class) === true);
        /** @var FileSystemInterface $fileSystem */
        $fileSystem = $container->get(FileSystemInterface::class);

        $path = $this->getMigrationsPath();
        assert($fileSystem->exists($path) === true);

        $this->getIO()->writeInfo("Migrations `$path` started." . PHP_EOL, IoInterface::VERBOSITY_VERBOSE);

The call to IoInterface::writeInfo() has too many arguments starting with \Limoncello\Contracts\Commands\IoInterface::VERBOSITY_VERBOSE.

        $migrationClasses = $fileSystem->requireFile($path);
        $this->setMigrationClasses($migrationClasses);

        // ... and run actual migration
        parent::migrate($container);
    }

    /**
     * @inheritdoc
     */
    protected function getMigrationClasses(): array
    {
        return $this->migrationClasses;
    }

    /**
     * @return string
     */
    protected function getMigrationsPath(): string
    {
        return $this->migrationsPath;
    }

    /**
     * @param string $migrationsPath
     *
     * @return self
     */
    protected function setMigrationsPath(string $migrationsPath): self
    {
        $this->migrationsPath = $migrationsPath;

        return $this;
    }

    /**
     * @param string[] $migrationClasses
     *
     * @return self
     */
    private function setMigrationClasses(array $migrationClasses): self
    {
        $this->migrationClasses = $migrationClasses;

        return $this;
    }
}