ElevenLabs\ApiServiceBundle\Command\ManifestCommand

Complexity

Total Complexity

6

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	execute()	0	17	5
A	configure()	0	5	1

<?php

declare(strict_types=1);

namespace ElevenLabs\ApiServiceBundle\Command;

use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * Class ManifestCommand.
 */
class ManifestCommand extends Command
{
    protected static $defaultName = 'api:manifest';

    /**
     *
     */
    protected function configure()
    {
        $this
            ->setDescription('Removes unnecessary files from the recorder')
            ->addArgument('dir')
        ;
    }

    /**
     * @param InputInterface  $input
     * @param OutputInterface $output
     *
     * @return int|void|null
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $dir = $input->getArgument('dir');
        $file = \realpath(sprintf('%s/manifest.json', $dir));

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

        if (!\is_file($file)) {
            return ;
        }

        $files = array_keys(json_decode(file_get_contents($file) ?? '[]', true));
        foreach (scandir($dir) as $file) {

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

            if ('.' === $file[0]) {
                continue;
            }

            if (!\in_array($file, $files)) {
                unlink($file);
            }
        }
    }
}