Filemanager\scan::scanAllFiles()

Complexity

Conditions	3
Paths	3

Size

Total Lines	18
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
  public static function scanAllFiles($dir, $exclude = '/^(.{1,2}|\.htaccess)$/s')
  {
    $di = new RecursiveDirectoryIterator($dir);
    $result = [];
    foreach (new RecursiveIteratorIterator($di) as $filename => $file) {
      $basename = basename($filename);

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

      if (preg_match($exclude, $basename)) {
        continue;
      }
      $b['path'] = realpath($filename);

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

      $b['data'] = \Filemanager\file::get(realpath($filename), true);
      $b['folder'] = dirname(realpath($filename));
      $b['filename'] = $basename;
      $b['size'] = $file->getSize();
      $result[] = $b;
    }

    return (array) $result;
  }