IQ2i\DataImporter\Bundle\Command\AbstractImportCommand

Complexity

Total Complexity

10

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Test Coverage

Coverage

87.8%

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	handleBatch()	0	3	1
A	getArchiver()	0	3	1
A	getSerializer()	0	3	1
A	handleBegin()	0	3	1
A	execute()	0	23	2
A	getProcessor()	0	5	2
A	handleEnd()	0	3	1
A	configure()	0	8	1

<?php

declare(strict_types=1);

/*
 * This file is part of the DataImporter package.
 *
 * (c) Loïc Sapone <loic@sapone.fr>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace IQ2i\DataImporter\Bundle\Command;

use IQ2i\DataImporter\Archiver\ArchiverInterface;
use IQ2i\DataImporter\Bundle\Exception\ItemHandlingException;
use IQ2i\DataImporter\Bundle\Processor\AsyncCliProcessor;
use IQ2i\DataImporter\Bundle\Processor\CliProcessor;
use IQ2i\DataImporter\DataImporter;
use IQ2i\DataImporter\Exchange\Message;
use IQ2i\DataImporter\Processor\ProcessorInterface;
use IQ2i\DataImporter\Reader\ReaderInterface;
use Symfony\Component\Console\Command\Command;

The type Symfony\Component\Console\Command\Command was not found. Maybe you did not declare it correctly or list all dependencies?

use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Style\SymfonyStyle;
use Symfony\Component\Serializer\Serializer;

abstract class AbstractImportCommand extends Command
{
    protected InputInterface $input;

    protected OutputInterface $output;

    abstract protected function handleItem(): callable;

    abstract protected function getReader(?string $filename = null): ReaderInterface;

    protected function configure(): void
    {
        $this
            ->addArgument('filename', InputArgument::OPTIONAL, 'File to import')
            ->addOption('async', null, InputOption::VALUE_NONE, 'Parallelize row process using symfony/messenger')
            ->addOption('step', null, InputOption::VALUE_NONE, 'Step through each record one-by-one')
            ->addOption('pause-on-error', null, InputOption::VALUE_NONE, 'Pause if an exception is thrown')
            ->addOption('batch-size', null, InputOption::VALUE_REQUIRED, 'Batch size', 100)
        ;
    }

    protected function execute(InputInterface $input, OutputInterface $output): int
    {
        $this->input = $input;
        $this->output = $output;

        $io = new SymfonyStyle($input, $output);
        $io->title('Start importing data');

        try {
            (new DataImporter(
                $this->getReader($input->getArgument('filename')),
                $this->getProcessor($input, $output),
                $this->getArchiver(),

Are you sure the usage of $this->getArchiver() targeting IQ2i\DataImporter\Bundle\Command\AbstractImportCommand::getArchiver() seems to always return null.

                $this->getSerializer()

Are you sure the usage of $this->getSerializer() targeting IQ2i\DataImporter\Bundle\Command\AbstractImportCommand::getSerializer() seems to always return null.

            ))->execute();
        } catch (ItemHandlingException $itemHandlingException) {
            $io->newLine(2);
            $io->error($itemHandlingException->getMessage());

            return $itemHandlingException->getCode();
        }

        return Command::SUCCESS;
    }

    protected function handleBegin(): callable
    {
        return static function (Message $message) {

The parameter $message is not used and could be removed.

        };
    }

    protected function handleBatch(): callable
    {
        return static function (Message $message) {

The parameter $message is not used and could be removed.

        };
    }

    protected function handleEnd(): callable
    {
        return static function (Message $message, array $errors) {

The parameter $message is not used and could be removed.

The parameter $errors is not used and could be removed.

        };
    }

    protected function getProcessor(InputInterface $input, OutputInterface $output): ProcessorInterface
    {
        return $input->getOption('async')
            ? new AsyncCliProcessor($input, $output, $this->handleBegin(), $this->handleItem(), $this->handleBatch(), $this->handleEnd(), $this->getSerializer())

Are you sure the usage of $this->getSerializer() targeting IQ2i\DataImporter\Bundle\Command\AbstractImportCommand::getSerializer() seems to always return null.

            : new CliProcessor($input, $output, $this->handleBegin(), $this->handleItem(), $this->handleBatch(), $this->handleEnd(), $this->getSerializer());

Are you sure the usage of $this->getSerializer() targeting IQ2i\DataImporter\Bundle\Command\AbstractImportCommand::getSerializer() seems to always return null.

    }

    protected function getArchiver(): ?ArchiverInterface
    {
        return null;
    }

    protected function getSerializer(): ?Serializer
    {
        return null;
    }
}