ByTIC\Migrations\Migrator\Traits\RunCommandsTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Test Coverage

Coverage

18.18%

Importance

Changes	2
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	migrate()	0	3	1
A	create()	0	3	1
A	runCommand()	0	11	1

<?php

namespace ByTIC\Migrations\Migrator\Traits;

use Phinx\Console\PhinxApplication;
use Symfony\Component\Console\Input\ArrayInput;

/**
 * Trait RunCommandsTrait
 * @package ByTIC\Migrations\Migrator\Traits
 */
trait RunCommandsTrait
{
    /**
     * @param array $arguments
     * @param $output
     * @return int
     */
    public function migrate($arguments = [], $output = null)
    {
        return $this->runCommand('migrate', $arguments, $output);
    }

    /**
     * @param array $arguments
     * @param $output
     * @return int
     */
    public function create($arguments = [], $output = null)
    {
        return $this->runCommand('create', $arguments, $output);
    }

    /**
     * @param $command
     * @param array $arguments
     * @param $output
     * @return int
     * @noinspection PhpDocMissingThrowsInspection
     */
    protected function runCommand($command, $arguments = [], $output = null)
    {
        $phinx = new PhinxApplication();
        $command = $phinx->find($command);

        $arguments['command'] = $command;
        $arguments['--configuration'] = $this->getCachedConfigPath();

It seems like getCachedConfigPath() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $input = new ArrayInput($arguments);

The assignment to $input is dead and can be removed.

        /** @noinspection PhpUnhandledExceptionInspection */
        return $command->run(new ArrayInput($arguments), $output);
    }
}