wapmorgan / UnifiedArchive

src/Commands/ExtractFilesCommand.php

<?php

namespace wapmorgan\UnifiedArchive\Commands;

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

class ExtractFilesCommand extends BaseArchiveCommand
{
    protected static $defaultName = 'files:extract';

    protected function configure()
    {
        parent::configure();
        $this
            ->setDescription('Extracts archive contents by selector')
            ->setHelp('Extracts all archive contents by selector.')
            ->addArgument('outputDir', InputArgument::REQUIRED, 'Folder to extract contents')
            ->addArgument('entrySelector', InputArgument::OPTIONAL, 'Prefix of entry selector. Default is null, means root of archive')
        ;
    }

    public function execute(InputInterface $input, OutputInterface $output)
    {
        $output_dir = $input->getArgument('outputDir');
        if (!is_dir($output_dir)) {
            mkdir($output_dir, 0777, true);
        } else if (!is_writable($output_dir)) {
            chmod($output_dir, 0777);
        }
        $archive = $this->getArchive($input, $output);
        $entry_selector = $input->getArgument('entrySelector');

        $archive->extract($output_dir, $entry_selector, true);
        $output->writeln('<info>Extracted:</info> ' . implode(', ', $entry_selector) . ' (' . count($entry_selector) . ') file(s)');

It seems like $entry_selector can also be of type null and string; however, parameter $pieces of implode() does only seem to accept array, maybe add an additional type check?

It seems like $entry_selector can also be of type null and string; however, parameter $value of count() does only seem to accept Countable|array, maybe add an additional type check?

        return 0;
    }
}