AntiMattr\MongoDB\Migrations\Tools\Console\Command\VersionCommand::execute()

Complexity

Conditions	10
Paths	11

Size

Total Lines	32
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	18
CRAP Score	10

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function execute(InputInterface $input, OutputInterface $output)
    {
        $configuration = $this->getMigrationConfiguration($input, $output);
        $migration = $this->createMigration($configuration);

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

        if (false === $input->getOption('add') && false === $input->getOption('delete')) {
            throw new \InvalidArgumentException('You must specify whether you want to --add or --delete the specified version.');
        }

        $version = $input->getArgument('version');
        $markMigrated = $input->getOption('add') ? true : false;

        if (!$configuration->hasVersion($version)) {

It seems like $version can also be of type string[]; however, parameter $version of AntiMattr\MongoDB\Migrations\Configuration\Configuration::hasVersion() does only seem to accept string, maybe add an additional type check?

            throw new UnknownVersionException($version);

It seems like $version can also be of type string[]; however, parameter $message of AntiMattr\MongoDB\Migrations\Exception\UnknownVersionException::__construct() does only seem to accept string, maybe add an additional type check?

        }

        $version = $configuration->getVersion($version);

It seems like $version can also be of type string[]; however, parameter $version of AntiMattr\MongoDB\Migrations\Configuration\Configuration::getVersion() does only seem to accept string, maybe add an additional type check?

        if ($markMigrated && $configuration->hasVersionMigrated($version)) {
            throw new \InvalidArgumentException(sprintf('The version "%s" already exists in the version collection.', $version));
        }

        if (!$markMigrated && !$configuration->hasVersionMigrated($version)) {
            throw new \InvalidArgumentException(sprintf('The version "%s" does not exists in the version collection.', $version));
        }

        if ($markMigrated) {
            $version->markMigrated();
        } else {
            $version->markNotMigrated();
        }

        return 0;
    }