AntiMattr\Bundle\MongoDBMigrationsBundle\Command\MigrationsExecuteCommand

Complexity

Total Complexity

2

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Test Coverage

Coverage

45.45%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	7	1
A	execute()	0	10	1

<?php

/*
 * This file is part of the AntiMattr MongoDB Migrations Bundle, a library by Matthew Fitzgerald.
 *
 * (c) 2014 Matthew Fitzgerald
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace AntiMattr\Bundle\MongoDBMigrationsBundle\Command;

use AntiMattr\MongoDB\Migrations\Tools\Console\Command\ExecuteCommand;
use Symfony\Bundle\FrameworkBundle\Console\Application;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * @author Matthew Fitzgerald <matthewfitz@gmail.com>
 */
class MigrationsExecuteCommand extends ExecuteCommand
{
    protected function configure()
    {
        parent::configure();

        $this
            ->setName('mongodb:migrations:execute')
            ->addOption('dm', null, InputOption::VALUE_OPTIONAL, 'The document manager to use for this command.', 'default_document_manager')
        ;
    }

    public function execute(InputInterface $input, OutputInterface $output)
    {
        /** @var Application $application */
        $application = $this->getApplication();
        CommandHelper::setApplicationDocumentManager($application, $input->getOption('dm'));

It seems like $input->getOption('dm') can also be of type string[]; however, parameter $dmName of AntiMattr\Bundle\MongoDBMigrationsBundle\Command\CommandHelper::setApplicationDocumentManager() does only seem to accept string, maybe add an additional type check?

        $configuration = $this->getMigrationConfiguration($input, $output);
        CommandHelper::configureMigrations($application->getKernel()->getContainer(), $configuration);

        return parent::execute($input, $output);

Are you sure the usage of parent::execute($input, $output) targeting AntiMattr\MongoDB\Migrations\Tools\Console\Command\ExecuteCommand::execute() seems to always return null.

    }
}