Fyuze\File\Cabinet

Complexity

Total Complexity

11

Size/Duplication

Total Lines	112
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	only()	0	11	3
A	in()	0	14	3
A	createFilter()	0	12	2
A	getIterator()	0	19	2
A	search()	0	5	1

<?php
namespace Fyuze\File;

use Fyuze\File\Iterators\ExtensionIterator;
use GlobIterator;
use AppendIterator;
use FilesystemIterator;
use RecursiveDirectoryIterator;
use IteratorAggregate;
use InvalidArgumentException;
use RuntimeException;

/**
 * Directory handler for fyuze
 *
 * @package Fyuze\File
 */
class Cabinet implements IteratorAggregate
{
    /**
     * @var
     */
    protected $search = [];

    /**
     * @var
     */
    protected $index;

    /**
     * @var
     */
    protected $flag;

    /**
     * @var array
     */
    public $directory = [];

    /**
     * @param $pattern
     * @return $this
     */
    public function search($pattern)
    {
        $this->search[] = $pattern;

        return $this;
    }

    /**
     * @param $flag
     * @return $this
     * @throws InvalidArgumentException
     */
    public function only($flag)
    {
        if ($flag !== 'files' && $flag !== 'folders') {
            throw new InvalidArgumentException(
                sprintf('The flag \'%s\' is not supported.', $flag)
            );
        }

        $this->flag = $flag;

        return $this;
    }

    /**
     * @param $directory
     * @return $this
     * @throws RuntimeException
     */
    public function in($directory)
    {
        if (file_exists($directory) === false || is_dir($directory) === false) {
            throw new RuntimeException(
                sprintf('The file %s either does not exist or is not a directory', $directory)
            );
        }

        $this->directory = new RecursiveDirectoryIterator(

It seems like new RecursiveDirectoryIterator($directory, FilesystemIterator::SKIP_DOTS) of type RecursiveDirectoryIterator is incompatible with the declared type array of property $directory.

            $directory,
            FilesystemIterator::SKIP_DOTS
        );

        return $this;
    }

    /**
     * @return AppendIterator
     */
    public function getIterator()
    {
        $iterator = new AppendIterator();

        $directoryIterator = $this->createFilter($this->directory);

        foreach ($this->search as $query) {

            $iterator->append(
                new ExtensionIterator(
                    $this->directory,

$this->directory of type array is incompatible with the type Iterator expected by parameter $iterator of Fyuze\File\Iterators\ExtensionIterator::__construct().

                    $query
                )
            );
        }

        $iterator->append($directoryIterator);

It seems like $directoryIterator can also be of type array; however, parameter $iterator of AppendIterator::append() does only seem to accept Iterator, maybe add an additional type check?

        return $iterator;
    }

    /**
     * @param $iterator
     * @return mixed
     */
    protected function createFilter($iterator)
    {
        if ($this->flag) {

            /** @var \FilterIterator $class */
            $class = sprintf('Fyuze\File\Iterators\%sIterator',
                ucfirst(str_replace('s', '', $this->flag)));

            return new $class($iterator);
        }

        return $iterator;
    }
}