Funivan\Cs\Fs\FileFinder\StandardFileFinder

Complexity

Total Complexity

9

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	5

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
C	findFiles()	0	61	9

<?php

  namespace Funivan\Cs\Fs\FileFinder;

  use Funivan\Cs\Fs\File;
  use Funivan\Cs\Fs\FilesCollection;
  use Symfony\Component\Finder\Finder;
  use Symfony\Component\Finder\SplFileInfo;
  use Symfony\Component\Process\Process;

  /**
   *
   */
  class StandardFileFinder implements FileFinderInterface {

    /**
     * @param FinderParameters $finderParameters
     * @return FilesCollection
     * @internal param FinderParameters $params
     */
    public function findFiles(FinderParameters $finderParameters) {
      $filesCollection = new FilesCollection();

      if ($finderParameters->get('dir')) {

The expression $finderParameters->get('dir') of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

        $dir = $finderParameters->get('dir');

        $files = (new Finder())->in($dir)->files();

        /** @var SplFileInfo $file */
        foreach ($files as $file) {
          $filesCollection->add(new File($file->getRealPath(), File::STATUS_UNKNOWN));
        }

        return $filesCollection;
      }

      $baseDir = $finderParameters->get('baseDir');
      if (empty($baseDir)) {
        throw new \RuntimeException('Can not find files. Invalid baseDir finder parameter.');
      }
      $baseDir = rtrim($baseDir, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR;
      if (!empty($commitHash)) {

The variable $commitHash seems to never exist, and therefore empty should always return true. Did you maybe rename this variable?

        $process = new Process('git diff-tree --no-commit-id --name-status  -r ' . $commitHash);
      } else {
        $process = new Process('git diff --name-status ; git diff --cached --name-status');
      }


      $process->run();
      if (!$process->isSuccessful()) {
        return $filesCollection;
      }


      $filesList = array_filter(explode("\n", $process->getOutput()));

      $statusMap = [
        'A' => File::STATUS_ADDED,
        'C' => File::STATUS_COPIED,
        'M' => File::STATUS_MODIFIED,
        'R' => File::STATUS_RENAMED,
        'D' => File::STATUS_DELETED,
      ];

      foreach ($filesList as $fileInfo) {

        preg_match('!^([^\s]+)\s+(.+)$!', $fileInfo, $matchedFileInfo);
        if (empty($matchedFileInfo[2])) {
          continue;
        }
        $gitStatus = trim($matchedFileInfo[1]);

        $status = !empty($statusMap[$gitStatus]) ? $statusMap[$gitStatus] : File::STATUS_UNKNOWN;

        $fullPath = $baseDir . ltrim($matchedFileInfo[2], DIRECTORY_SEPARATOR);
        $filesCollection->add(new File($fullPath, $status));
      }

      return $filesCollection;

    }

  }