App\Console\StartImport::startImport()

Complexity

Conditions	12
Paths	10

Size

Total Lines	51
Code Lines	30

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    private function startImport(string $csv, array $configuration): int
    {
        Log::debug(sprintf('Now in %s', __METHOD__));
        $configObject = Configuration::fromFile($configuration);
        $manager      = new ImportRoutineManager;

        try {
            $manager->setConfiguration($configObject);
        } catch (ImportException $e) {
            $this->error($e->getMessage());

It seems like error() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            return 1;
        }
        $manager->setReader(FileReader::getReaderFromContent($csv));
        try {
            $manager->start();
        } catch (ImportException $e) {
            $this->error($e->getMessage());

            return 1;
        }

        $messages = $manager->getAllMessages();
        $warnings = $manager->getAllWarnings();
        $errors   = $manager->getAllErrors();

        if (count($errors) > 0) {
            foreach ($errors as $index => $error) {
                foreach ($error as $line) {
                    $this->error(sprintf('ERROR in line     #%d: %s', $index + 1, $line));
                }
            }
        }

        if (count($warnings) > 0) {
            foreach ($warnings as $index => $warning) {
                foreach ($warning as $line) {
                    $this->warn(sprintf('Warning from line #%d: %s', $index + 1, $line));

It seems like warn() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                }
            }
        }

        if (count($messages) > 0) {
            foreach ($messages as $index => $message) {
                foreach ($message as $line) {
                    $this->info(sprintf('Message from line #%d: %s', $index + 1, strip_tags($line)));

It seems like info() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                }
            }
        }

        return 0;
    }
}