Soluble\MediaTools\Cli\Command\ConvertDirCommand::execute()

Complexity

Conditions	6
Paths	36

Size

Total Lines	96
Code Lines	46

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	40
CRAP Score	6.0038

Importance

Changes

0

<?php
    protected function execute(InputInterface $input, OutputInterface $output): int
    {
        // ########################
        // Step 1: Check directory
        // ########################

        $directory = $input->getOption('dir');
        Assert::stringNotEmpty($directory);
        Assert::directory($directory);

        // ########################
        // Step 2: Init preset
        // ########################

        Assert::stringNotEmpty($input->getOption('preset'));
        $preset = $this->getPreset($input->getOption('preset'));

        // ########################
        // Step 3: Output dir
        // ########################

        if ($input->getOption('output') !== null) {
            $outputDir = $input->getOption('output');
            Assert::directory($outputDir);
            Assert::writable($outputDir);
        } else {
            $outputDir = $directory;
        }
        Assert::stringNotEmpty($outputDir);

        // ########################
        // Step 4: Extensions
        // ########################

        if ($input->getOption('exts') !== null) {
            $tmp = $input->getOption('exts');
            Assert::stringNotEmpty($tmp);
            $exts = array_filter(
                array_map(
                    'trim',
                    explode(',', $tmp)

It seems like $tmp can also be of type string[]; however, parameter $string of explode() does only seem to accept string, maybe add an additional type check?

                )
            );
            Assert::minCount($exts, 1);
        } else {
            $exts = FileExtensions::BUILTIN_EXTENSIONS;
        }

        // ########################
        // Step 5: Scanning dir
        // ########################

        $output->writeln(sprintf('* Scanning %s for media files...', $directory));

It seems like $directory can also be of type string[]; however, parameter $args of sprintf() does only seem to accept string, maybe add an additional type check?

        // Get the videos in path
        $files = (new DirectoryScanner())->findFiles($directory, $exts);

        $output->writeln('* Reading metadata...');

        $progressBar = new ProgressBar($output, count($files));
        //$progressBar->start();

        $converter = $this->mediaTools->getConverter();

        /** @var \SplFileInfo $file */
        foreach ($files as $file) {
            try {
                $params = $preset->getParams($file->getPathname());

                $outputFile = sprintf(
                    '%s/%s%s',
                    $outputDir,
                    $file->getBasename($file->getExtension()),
                    $preset->getFileExtension()
                );

                if (!file_exists($outputFile)) {
                    $converter->convert((string) $file, $outputFile, $params, function ($stdOut, $stdErr) use ($output): void {
                        $output->write($stdErr);
                    });

                    $output->writeln(sprintf('<fg=green>- Converted:</> %s.', $file));
                } else {
                    $output->writeln(sprintf('<fg=yellow>- Skipped:</> %s : Output file already exists.', $file));
                }
            } catch (ProcessException $e) {
                $output->writeln(sprintf('<fg=red>- Skipped:</> %s : Not a valid media file.', $file));
            }

            //$progressBar->advance();
        }

        $progressBar->finish();
        $output->writeln('');

        return 0;
    }