wapmorgan\UnifiedArchive\Commands\CreateCommand

Complexity

Total Complexity

19

Size/Duplication

Total Lines	134
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	13	1
D	execute()	0	106	18

<?php

namespace wapmorgan\UnifiedArchive\Commands;

use http\Exception\InvalidArgumentException;
use Symfony\Component\Console\Helper\ProgressBar;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
use wapmorgan\UnifiedArchive\Drivers\BasicDriver;
use wapmorgan\UnifiedArchive\UnifiedArchive;

class CreateCommand extends BaseCommand
{
    protected static $defaultName = 'archive:create';

    protected function configure()
    {
        parent::configure();
        $this
            ->setDescription('Creates new archive')
            ->setHelp('Creates new archive.')
            ->addArgument('archive', InputArgument::REQUIRED, 'New archive filename. Type of archive will be recognized from extension')
            ->addArgument('file', InputArgument::IS_ARRAY | InputArgument::REQUIRED, 'Files to pack')
            ->addOption('password', NULL, InputOption::VALUE_REQUIRED, 'Password for new archive')
            ->addOption('compression', NULL, InputOption::VALUE_OPTIONAL, 'Compression level for new archive. Variants: none, weak, average, strong, maximum.', 'average')
            ->addOption('comment', NULL, InputOption::VALUE_OPTIONAL, 'Comment for new archive')
            ->addOption('path', NULL, InputOption::VALUE_OPTIONAL, 'Path resolving if destination is not passed. Variants: full, root, relative, basename', 'root')
            ->addOption('dry-run', NULL, InputOption::VALUE_NONE, 'Do not perform real archiving. Just print prepared data')
        ;
    }

    protected static $compressionLevels = [
        'none' => BasicDriver::COMPRESSION_NONE,
        'weak' => BasicDriver::COMPRESSION_WEAK,
        'average' => BasicDriver::COMPRESSION_AVERAGE,
        'strong' => BasicDriver::COMPRESSION_STRONG,
        'maximum' => BasicDriver::COMPRESSION_MAXIMUM,
    ];

    public function execute(InputInterface $input, OutputInterface $output)
    {
        $archive_file = $input->getArgument('archive');
        $files_to_pack = $input->getArgument('file');

        $password = $input->getOption('password');
        $compression = $input->getOption('compression');
        if (!isset(static::$compressionLevels[$compression])) {
            throw new \InvalidArgumentException('Compression "' . $compression . '" is not valid');
        }
        $compression = static::$compressionLevels[$compression];
        $path = $input->getOption('path');
        if (!in_array($path, ['full', 'root', 'relative', 'basename'], true)) {
            throw new \InvalidArgumentException('Path can not have this value');
        }
        $dry_run = $input->getOption('dry-run');
        $comment = $input->getOption('comment');

        if (file_exists($archive_file)) {

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

            if (is_dir($archive_file))

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

                throw new \InvalidArgumentException($archive_file . ' is a directory!');

Are you sure $archive_file of type null|string|string[] can be used in concatenation?

            else {
                throw new \InvalidArgumentException('File "' . $archive_file . '" already exists!');
            }
        }

        $files_list = [];

        foreach ($files_to_pack as $i => $file_to_pack) {
            $file_to_pack = realpath($file_to_pack);
            switch ($path) {
                case 'full':
                    $destination = ltrim($file_to_pack, '/');
                    $files_list[$destination] = $file_to_pack;
                    $output->writeln('<comment>' . $file_to_pack . ' => ' . $destination . '</comment>', OutputInterface::VERBOSITY_VERBOSE);
                    break;
                case 'root':
                    if (is_dir($file_to_pack)) {
                        $output->writeln('<comment>' . $file_to_pack . ' => root</comment>', OutputInterface::VERBOSITY_VERBOSE);
                        if (!isset($files_list[''])) {
                            $files_list[''] = $file_to_pack;
                        } elseif (is_string($files_list[''])) {
                            $files_list[''] = [$files_list[''], $file_to_pack];
                        } else {
                            $files_list[''][] = $file_to_pack;
                        }
                    } else {
                        $output->writeln('<comment>' . $file_to_pack . ' => ' . basename($file_to_pack) . '</comment>', OutputInterface::VERBOSITY_VERBOSE);
                        $files_list[basename($file_to_pack)] = $file_to_pack;
                    }
                    break;
                case 'relative':
                    $destination = ltrim($file_to_pack, '/.');
                    $files_list[$destination] = $file_to_pack;
                    $output->writeln('<comment>' . $file_to_pack . ' => ' . $destination . '</comment>', OutputInterface::VERBOSITY_VERBOSE);
                    break;
                case 'basename':
                    $destination = basename($file_to_pack);
                    $files_list[$destination] = $file_to_pack;
                    $output->writeln('<comment>' . $file_to_pack . ' => ' . $destination . '</comment>', OutputInterface::VERBOSITY_VERBOSE);
                    break;
            }
        }

        $information = UnifiedArchive::prepareForArchiving($files_list, $archive_file);

It seems like $archive_file can also be of type string[]; however, parameter $archiveName of wapmorgan\UnifiedArchive\UnifiedArchive::prepareForArchiving() does only seem to accept string, maybe add an additional type check?

        if ($dry_run) {
            $output->writeln('Format: <info>' . $information['type'] . '</info>');
            $output->writeln('Original size: <info>' . implode($this->formatSize($information['totalSize'])) . '</info>');
            $output->writeln('Files: <info>' . $information['numberOfFiles'] . '</info>');
            foreach ($information['files'] as $destination => $source) {
                // is folder
                if ($source === null) {
                    continue;
                }

                $output->writeln($source . ' => <comment>' . $destination . '</comment>');
            }
            return 0;
        }

        ProgressBar::setFormatDefinition('archiving', '  %current%/%max% [%bar%] %percent:3s%% %elapsed:6s%: %message%');

        $progressBar = new ProgressBar($output, $information['numberOfFiles']);
        $progressBar->setFormat('archiving');
        $progressBar->start();
        $archived_files = UnifiedArchive::archiveFiles($files_list, $archive_file, $compression, $password, function ($currentFile, $totalFiles, $fsFilename, $archiveFilename)

It seems like $archive_file can also be of type string[]; however, parameter $archiveName of wapmorgan\UnifiedArchive\UnifiedArchive::archiveFiles() does only seem to accept string, maybe add an additional type check?

The parameter $archiveFilename is not used and could be removed.

It seems like $password can also be of type string[]; however, parameter $password of wapmorgan\UnifiedArchive\UnifiedArchive::archiveFiles() does only seem to accept null|string, maybe add an additional type check?

            use ($progressBar) {
            $progressBar->advance();
            $progressBar->setMessage($fsFilename);
        });
        $progressBar->finish();
        $progressBar->clear();
        $output->writeln(null);

        if (!$archived_files) {
            throw new \RuntimeException('archiveFiles result is false');
        }

        $archive = $this->open($archive_file);
        if (!empty($comment)) {
            $archive->setComment($comment);

It seems like $comment can also be of type string[]; however, parameter $comment of wapmorgan\UnifiedArchive\UnifiedArchive::setComment() does only seem to accept null|string, maybe add an additional type check?

        }

        $output->writeln('Created <info>' . $archive_file . '</info> with <comment>' . $archived_files . '</comment> file(s) ('
                         . implode($this->formatSize($archive->getOriginalSize())) . ') of total size '
                         . implode($this->formatSize(filesize($archive_file))));

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

    }
}