wapmorgan\UnifiedArchive\Commands\AddFileCommand::execute()

Complexity

Conditions	12
Paths	63

Size

Total Lines	52
Code Lines	40

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function execute(InputInterface $input, OutputInterface $output)
    {
        $source = $input->getArgument('source');
        $destination = $input->getArgument('destination');
        $path = $input->getOption('path');
        if (!in_array($path, ['full', 'relative', 'basename'], true)) {
            throw new \InvalidArgumentException('Path can not have this value');
        }

        if ($source === '-') {
            stream_set_blocking(STDIN, false);
            $data = stream_get_contents(STDIN);
            stream_set_blocking(STDIN, true);
            $data_size = strlen($data);
            if ($data_size === 0) {
                throw new \LogicException('Empty input!');
            }
            if (empty($destination)) {
                throw new \LogicException('Source and destination can not be empty');
            }
            $output->writeln('<info>Read ' . $data_size . ' from input</info>');
        } else {
            $data_size = filesize($source);

It seems like $source 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?

        }

        if (empty($destination)) {
            switch ($path) {
                case 'full':
                    $destination = ltrim(realpath($source), '/');

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

                    break;
                case 'relative':
                    $destination = ltrim($source, '/.');

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

                    break;
                case 'basename':
                    $destination = basename($source);

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

                    break;
            }
        }

        $archive = $this->getArchive($input, $output);
        if ($source === '-') {
            $added_files = $archive->addFileFromString($destination, $data) ? 1 : 0;

The variable $data does not seem to be defined for all execution paths leading up to this point.

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

        } else {
            $added_files = $archive->addFiles([$destination => $source]);
        }
        if ($added_files === 1) {
            $details = $archive->getFileData($destination);

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

            $output->writeln('Added <comment>' . $source . '</comment>('

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

                             . implode($this->formatSize($data_size)) . ') as '
                             . $destination . ' ('

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

                             . implode($this->formatSize($details->compressedSize))
                             . ')');
        }