JK\Sam\Filter\Copy\CopyFilter

Complexity

Total Complexity

11

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
D	run()	0	33	9
A	checkRequirements()	0	3	1
A	getSupportedExtensions()	0	6	1

<?php

namespace JK\Sam\Filter\Copy;

use Exception;
use JK\Sam\Filter\Filter;
use SplFileInfo;
use Symfony\Component\Filesystem\Filesystem;

class CopyFilter extends Filter
{
    /**
     * @param SplFileInfo[] $sources
     * @param SplFileInfo[] $destinations
     * @return SplFileInfo[]
     * @throws Exception
     */
    public function run(array $sources, array $destinations)
    {
        $fileSystem = new Filesystem();

        // must have the same sources and destinations number
        if (count($sources) !== count($destinations) && count($destinations) !== 1) {
            throw new Exception('Sources and destinations count mismatch');
        }

        if (count($destinations) === 1 && !is_dir($destinations[0]) && count($sources) > 1) {
            throw new Exception('If only one destination is set for multiple source, it should be a directory');
        }
        $copiedFiles = [];

        // copy generated files to its destination
        foreach ($sources as $index => $source) {

            if (array_key_exists($index, $destinations)) {
                $destination = $destinations[$index];
            } else {
                $destination = $destinations[0];
            }

            if (is_dir($destination)) {
                $destination = $destination.'/'.$source->getFilename();
            }
            $this->addNotification('copying "'.$source.'" to "'.$destination.'"');
            $fileSystem->copy($source->getRealPath(), $destination);

It seems like $destination can also be of type object<SplFileInfo>; however, Symfony\Component\Filesystem\Filesystem::copy() does only seem to accept string, maybe add an additional type check?

            $copiedFiles[] = $destination;
        }

        return $copiedFiles;
    }

    /**
     * 
     */
    public function checkRequirements()
    {
    }

    /**
     * Return the file extensions supported by this filter.
     *
     * @return string[]
     */
    public function getSupportedExtensions()
    {
        return [
            '*'
        ];
    }
}