Facile\MongoDbBundle\Command\LoadFixturesCommand

Complexity

Total Complexity

9

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	8	1
C	execute()	0	33	7
A	loadFixture()	0	6	1

<?php

declare(strict_types=1);

namespace Facile\MongoDbBundle\Command;

use Facile\MongoDbBundle\Fixtures\MongoFixturesLoader;
use Facile\MongoDbBundle\Fixtures\MongoFixtureInterface;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * Class LoadFixturesCommand.
 */
class LoadFixturesCommand extends AbstractCommand
{
    /**
     * {@inheritdoc}
     */
    protected function configure()
    {
        parent::configure();
        $this
            ->setName('mongodb:fixtures:load')
            ->setDescription('Load fixtures and applies them');
        ;
    }

    /**
     * {@inheritdoc}
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $this->io->writeln('Loading index fixtures');

        $paths = array();
        foreach ($this->getApplication()->getKernel()->getBundles() as $bundle) {

It seems like you code against a specific sub-type and not the parent class Symfony\Component\Console\Application as the method getKernel() does only exist in the following sub-classes of Symfony\Component\Console\Application: Symfony\Bundle\FrameworkBundle\Console\Application. Maybe you want to instanceof check for one of these explicitly?

            $paths[] = $bundle->getPath().'/DataFixtures/Mongo';
        }

        $loader = new MongoFixturesLoader($this->getContainer());

        foreach ($paths as $path) {
            if (is_dir($path)) {
                $loader->loadFromDirectory($path);
            }
            if (is_file($path)) {
                $loader->loadFromFile($path);
            }
        }

        $fixtures = $loader->getLoadedClasses();
        if (!$fixtures) {

The expression $fixtures of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            throw new \InvalidArgumentException(
                sprintf('Could not find any class to load in: %s', "\n\n- ".implode("\n- ", $paths))
            );
        }

        foreach ($fixtures as $fixture){
            $this->loadFixture($fixture);
        }

        $this->io->writeln('Done.');
    }

    /**
     * @param MongoFixtureInterface $indexList
     */
    private function loadFixture(MongoFixtureInterface $indexList)
    {
        $indexList->loadData();
        $indexList->loadIndexes();
        $this->io->writeln('Loaded fixture: '. get_class($indexList));
    }
}