Phase\TakeATicketBundle\Command\LoadSongsCommand

Complexity

Total Complexity

3

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	13	1
A	execute()	0	19	2

<?php
/**
 * Created by PhpStorm.
 * User: wechsler
 * Date: 11/02/2017
 * Time: 20:19
 */

namespace Phase\TakeATicketBundle\Command;

use Phase\TakeATicket\SongLoader;
use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

class LoadSongsCommand extends ContainerAwareCommand
{
    protected function configure()
    {
        $this
            // the name of the command (the part after "bin/console")
            ->setName('ticket:load-songs')
            // the short description shown while running "php bin/console list"
            ->setDescription('Load songlist')
            // the full command description shown when running the command with
            // the "--help" option
            ->setHelp("Load songlist from XLS file")
            ->addArgument('file', InputArgument::REQUIRED, 'Name of the CSV file.')
            ->addArgument('mapper', InputArgument::OPTIONAL, 'ShortName of mapper to use', 'rclrockband');
    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $rowMapperManager = $this->getContainer()->get('songloader.rowmappermanager');
        /** @var  SongLoader\RowMapperManager $rowMapperManager */
        $mapper = $input->getArgument('mapper');
        $mapperClass = $rowMapperManager->getRowMapperClassByShortName($mapper);

        if (!$mapperClass) {

The expression $mapperClass of type string|null is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

            throw new \InvalidArgumentException("'$mapper' is not a valid mapper name");
        }

        $loader = new SongLoader();
        $loader->setRowMapperClass($mapperClass);

        $file = $input->getArgument('file');
        $songsLoaded = $loader->run($file, $this->getContainer()->get('database_connection'));
        $output->writeln("");
        $output->writeln("Loaded $songsLoaded songs from $file");
    }
}