Columnis\Utils\Directory

Complexity

Total Complexity

8

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	recursiveSearchByExtension()	0	17	5
A	isSubpath()	0	6	3

<?php

/**
 * Utility class for some useful directory operations
 *
 * @author matias
 */
namespace Columnis\Utils;

use RecursiveDirectoryIterator;
use RecursiveIteratorIterator;

class Directory
{
    /**
     * Scans a dir recursively for files that match the given extension
     *
     * @param string $extension
     * @return array|Traversable collections of files
     * @throws \Exception
     */
    public static function recursiveSearchByExtension($path, $extension, Array $excludes = null)
    {
        $files = array();
        if (!is_dir($path)) {
            throw new \Exception('Path given is not an existant directory.');
        }

        $directory = new RecursiveDirectoryIterator($path, RecursiveDirectoryIterator::SKIP_DOTS);
        $iterator = new RecursiveIteratorIterator($directory, RecursiveIteratorIterator::LEAVES_ONLY);

        foreach ($iterator as $fileinfo) {
            if ($fileinfo->getExtension() == $extension && !Regex::matchesAny($fileinfo->getPathname(),$excludes)) {
                $files[] = realpath($fileinfo->getPathname());
            }
        }
        return $files;
    }

    /**
     * Checks if a path is inside another
     *
     * @param string $path
     * @param string $subpath
     * @return boolean
     */
    public static function isSubpath($path, $subpath)
    {
        $rpath = realpath($path);
        $rsubpath = realpath($subpath);
        return $rpath != false && $rsubpath != false && (strpos($rsubpath, $rpath) === 0);

It seems like you are loosely comparing $rpath of type string to the boolean false. If you are specifically checking for a non-empty string, consider using the more explicit !== '' instead.

It seems like you are loosely comparing $rsubpath of type string to the boolean false. If you are specifically checking for a non-empty string, consider using the more explicit !== '' instead.

    }
}