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

Complexity

Conditions	4
Paths	9

Size

Total Lines	60
Code Lines	25

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	13.2581

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: 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, ['mp4']);

        $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', (string) $file, '.mov');

                if (!file_exists($outputFile)) {
                    $converter->convert((string) $file, $outputFile, $params, function ($stdOut, $stdErr) use ($output) {
                        $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;
    }