FilesystemPath::isDir() - Code Metrics - Inspection of "Let's get sassy" - stakx-io/stakx - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#69)

by Vladimir

created 2018-06-26 06:21 UTC

FilesystemPath::isDir() A

↳ Parent: FilesystemPath

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	3.072

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
dl	0
loc	11
c	1
b	0
f	0
rs	9.9
ccs	4
cts	5
cp	0.8
cc	3
nc	3
nop	1
crap	3.072

<?php

/**
 * @copyright 2018 Vladimir Jimenez
 * @license   https://github.com/stakx-io/stakx/blob/master/LICENSE.md MIT
 */

namespace allejo\stakx\Filesystem;

use allejo\stakx\Filesystem\FilesystemLoader as fs;

/**
 * A cross-platform filesystem path wrapper.
 *
 * This class is a wrapper for handling file paths in stakx in a cross-platform way. Give it a Windows path and append
 * a Unix style path and it'll just work.
 */
final class FilesystemPath
{
    /** @var string */
    private $absolutePath;
    /** @var string */
    private $originalPath;
    /** @var bool */
    private $isWindows;

    /**
     * @param string $filePath
     * @param string $dirSep
     */
    public function __construct($filePath, $dirSep = DIRECTORY_SEPARATOR)
    {
        $this->originalPath = $filePath;
        $this->isWindows = ($dirSep === '\\');

        if ($this->isWindows)
        {
            $filePath = $this->unixifyPath($filePath);

        }

        $this->absolutePath = (fs::isAbsolutePath($filePath)) ? $filePath : fs::absolutePath($filePath);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    public function __toString()
    {
        return $this->getAbsolutePath();
    }

    /**
     * Append a path to a directory path.
     *
     * @param string $append The path to append
     */
    public function appendToPath($append)
    {
        if ($this->isFile(false))
        {
            throw new \InvalidArgumentException("Appending to a file's path is not possible");
        }

        $this->absolutePath = $this->buildPath($this->absolutePath, $this->unixifyPath($append));
    }

    /**
     * Generate a path based off this file path.
     *
     * This method will not modify the existing file path of this instance, use FilesystemPath::appendToPath() for that.
     *
     * @param string $append
     *
     * @return string
     */
    public function generatePath($append)
    {
        return $this->buildPath($this->absolutePath, $this->unixifyPath($append));
    }

    /**
     * Get the absolute path of the file path.
     *
     * @return string
     */
    public function getAbsolutePath()
    {
        if ($this->isWindows)
        {
            return str_replace('/', '\\', $this->absolutePath);
        }

        return $this->absolutePath;
    }

    /**
     * Check whether the given path is a directory.
     *
     * @param bool $checkExistence When set to true, it will check the filesystem for the existence of the directory.
     *                             When set to false, this function will guess based on the path ending in a directory
     *                             separator.
     *
     * @return bool
     */
    public function isDir($checkExistence = true)
    {
        $absPath = $this->absolutePath;

        if ($checkExistence)
        {
            return file_exists($absPath) && is_dir($absPath);
        }

        return substr($absPath, -1, 1) == '/';
    }

    /**
     * Check whether the given path is a file.
     *
     * @param bool $checkExistence When set to true, it will check the filesystem for the existence of the file. When
     *                             set to false, this function will guess based on the path ending in a directory
     *                             separator.
     *
     * @return bool
     */
    public function isFile($checkExistence = true)
    {
        $absPath = $this->absolutePath;

        if ($checkExistence)
        {
            return file_exists($absPath) && is_file($absPath);
        }

        return !$this->isDir($checkExistence);
    }

    /**
     * Build a path from multiple strings.
     *
     * This function will _always_ use the '/' as the directory separator, because internal that's all stakx will use.
     * The FilesystemPath::getAbsolutePath() function will worry about Windows paths when necessary.
     *
     * @return string
     */
    private function buildPath()

    {
        $paths = [];

        foreach (func_get_args() as $arg)
        {
            if ($arg !== '')
            {
                $paths[] = $arg;
            }
        }

        return preg_replace('#(?<!:)/+#', '/', join('/', $paths));
    }

    /**
     * Convert a Windows path into a blasphemous Unix path.
     *
     * @param string $filePath
     *
     * @return string
     */
    private function unixifyPath($filePath)
    {
        return str_replace('\\', '/', $filePath);
    }
}


1			<?php
2
3			/**
4			* @copyright 2018 Vladimir Jimenez
5			* @license https://github.com/stakx-io/stakx/blob/master/LICENSE.md MIT
6			*/
7
8			namespace allejo\stakx\Filesystem;
9
10			use allejo\stakx\Filesystem\FilesystemLoader as fs;
11
12			/**
13			* A cross-platform filesystem path wrapper.
14			*
15			* This class is a wrapper for handling file paths in stakx in a cross-platform way. Give it a Windows path and append
16			* a Unix style path and it'll just work.
17			*/
18			final class FilesystemPath
19			{
20			/** @var string */
21			private $absolutePath;
22			/** @var string */
23			private $originalPath;
24			/** @var bool */
25			private $isWindows;
26
27			/**
28			* @param string $filePath
29			* @param string $dirSep
30			*/
31	12		public function __construct($filePath, $dirSep = DIRECTORY_SEPARATOR)
32			{
33	12		$this->originalPath = $filePath;
34	12		$this->isWindows = ($dirSep === '\\');
35
36	12		if ($this->isWindows)
37			{
38	2		$filePath = $this->unixifyPath($filePath);
			0 ignored issues – show Coding Style introduced 2017-11-02 08:36 UTC by Report Bug Copy Issue Report Consider using a different name than the parameter `$filePath`. This often makes code more readable. Loading history...
39			}
40
41	12		$this->absolutePath = (fs::isAbsolutePath($filePath)) ? $filePath : fs::absolutePath($filePath);
			0 ignored issues – show Bug introduced 2017-12-20 02:49 UTC by Report Bug Copy Issue Report The method `isAbsolutePath()` does not exist on `allejo\stakx\Filesystem\FilesystemLoader`. Did you maybe mean `absolutePath()`? This check marks calls to methods that do not seem to exist on an object. This is most likely the result of a method being renamed without all references to it being renamed likewise. Loading history... Documentation introduced 2018-05-10 07:09 UTC by Report Bug Copy Issue Report `$filePath` is of type `string`, but the function expects a `object<string>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history... Documentation Bug introduced 2018-05-10 07:09 UTC by Report Bug Copy Issue Report It seems like `\allejo\stakx\Filesystem...absolutePath($filePath)` can also be of type `object<string>`. However, the property `$absolutePath` is declared as type `string`. Maybe add an additional type check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly. For example, imagine you have a variable `$accountId` that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to the `id` property of an instance of the `Account` class. This class holds a proper account, so the id value must no longer be false. Either this assignment is in error or a type check should be added for that assignment. class Id { public $id; public function __construct($id) { $this->id = $id; } } class Account { /** @var Id $id */ public $id; } $account_id = false; if (starsAreRight()) { $account_id = new Id(42); } $account = new Account(); if ($account instanceof Id) { $account->id = $account_id; } Loading history...
42	12		}
43
44	10		public function __toString()
45			{
46	10		return $this->getAbsolutePath();
47			}
48
49			/**
50			* Append a path to a directory path.
51			*
52			* @param string $append The path to append
53			*/
54	5		public function appendToPath($append)
55			{
56	5		if ($this->isFile(false))
57			{
58	1		throw new \InvalidArgumentException("Appending to a file's path is not possible");
59			}
60
61	4		$this->absolutePath = $this->buildPath($this->absolutePath, $this->unixifyPath($append));
62	4		}
63
64			/**
65			* Generate a path based off this file path.
66			*
67			* This method will not modify the existing file path of this instance, use FilesystemPath::appendToPath() for that.
68			*
69			* @param string $append
70			*
71			* @return string
72			*/
73			public function generatePath($append)
74			{
75			return $this->buildPath($this->absolutePath, $this->unixifyPath($append));
76			}
77
78			/**
79			* Get the absolute path of the file path.
80			*
81			* @return string
82			*/
83	10		public function getAbsolutePath()
84			{
85	10		if ($this->isWindows)
86			{
87	2		return str_replace('/', '\\', $this->absolutePath);
88			}
89
90	8		return $this->absolutePath;
91			}
92
93			/**
94			* Check whether the given path is a directory.
95			*
96			* @param bool $checkExistence When set to true, it will check the filesystem for the existence of the directory.
97			* When set to false, this function will guess based on the path ending in a directory
98			* separator.
99			*
100			* @return bool
101			*/
102	6		public function isDir($checkExistence = true)
103			{
104	6		$absPath = $this->absolutePath;
105
106	6		if ($checkExistence)
107			{
108			return file_exists($absPath) && is_dir($absPath);
109			}
110
111	6		return substr($absPath, -1, 1) == '/';
112			}
113
114			/**
115			* Check whether the given path is a file.
116			*
117			* @param bool $checkExistence When set to true, it will check the filesystem for the existence of the file. When
118			* set to false, this function will guess based on the path ending in a directory
119			* separator.
120			*
121			* @return bool
122			*/
123	5		public function isFile($checkExistence = true)
124			{
125	5		$absPath = $this->absolutePath;
126
127	5		if ($checkExistence)
128			{
129			return file_exists($absPath) && is_file($absPath);
130			}
131
132	5		return !$this->isDir($checkExistence);
133			}
134
135			/**
136			* Build a path from multiple strings.
137			*
138			* This function will _always_ use the '/' as the directory separator, because internal that's all stakx will use.
139			* The FilesystemPath::getAbsolutePath() function will worry about Windows paths when necessary.
140			*
141			* @return string
142			*/
143	4	View Code Duplication	private function buildPath()
			0 ignored issues – show Duplication introduced 2017-11-02 08:36 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
144			{
145	4		$paths = [];
146
147	4		foreach (func_get_args() as $arg)
148			{
149	4		if ($arg !== '')
150			{
151	4		$paths[] = $arg;
152			}
153			}
154
155	4		return preg_replace('#(?<!:)/+#', '/', join('/', $paths));
156			}
157
158			/**
159			* Convert a Windows path into a blasphemous Unix path.
160			*
161			* @param string $filePath
162			*
163			* @return string
164			*/
165	4		private function unixifyPath($filePath)
166			{
167	4		return str_replace('\\', '/', $filePath);
168			}
169			}
170

stakx-io / stakx

Pull Request — master (#69)

FilesystemPath::isDir() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like