Druc\Flysystem\IncrementalNaming\IncrementsPath

Complexity

Total Complexity

6

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getIncrementedPath()	0	12	2
A	fileExists()	0	7	1
A	incrementPath()	0	8	2
A	pathHasExtension()	0	4	1

<?php

namespace Druc\Flysystem\IncrementalNaming;

trait IncrementsPath
{
    /**
     * @param $filePath
     * @return string
     */
    private function getIncrementedPath($filePath)
    {
        $originalFilePath = $filePath;
        $increment = 1;

        while ($this->fileExists($filePath)) {
            $filePath = $this->incrementPath($originalFilePath, $increment);
            $increment++;
        }

        return $filePath;
    }

    /**
     * @param $filePath
     * @return bool
     */
    private function fileExists($filePath)
    {
        $directory = pathinfo($filePath, PATHINFO_DIRNAME);
        $existingPaths = array_column($this->filesystem->listContents($directory), 'path');

The property filesystem does not exist. Did you maybe forget to declare it?

        
        return in_array($filePath, $existingPaths);
    }

    /**
     * @param $originalFilePath
     * @param $increment
     * @return null|string|string[]
     */
    private function incrementPath($originalFilePath, $increment)
    {
        if ($this->pathHasExtension($originalFilePath)) {
            return preg_replace('#^(.+)\.([\w]+)$#i', '$1_' . $increment . '.$2', $originalFilePath);
        }

        return $originalFilePath . '_' . $increment;
    }

    /**
     * @param $filePath
     * @return bool|int
     */
    private function pathHasExtension($filePath)
    {
        return (bool)strpos($filePath, '.');
    }
}