File::inc() - Code Metrics - Inspection of "New small features" - phpffcms/ffcms-core - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 337d70...6470c4 )

by Mihail

created 2016-01-23 18:09 UTC

File::inc() B

↳ Parent: File

Complexity

Conditions	5
Paths	5

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	14
rs	8.8571
cc	5
eloc	8
nc	5
nop	3

<?php

namespace Ffcms\Core\Helper\FileSystem;

use Ffcms\Core\Helper\Type\Arr;
use Ffcms\Core\Helper\Type\Str;

/**
 * Class File. Provide methods to work with files in current filesystem.
 * @package Ffcms\Core\Helper
 */
class File
{

    /**
     * Read file content from local storage
     * @param $path
     * @return bool|string
     */
    public static function read($path)
    {
        $path = Normalize::diskFullPath($path);

        if (!self::exist($path)) {
            return false;
        }

        return @file_get_contents($path);
    }

    /**
     * Check if $path is exist and readable in filesystem
     * @param string $path
     * @return bool
     */
    public static function exist($path)
    {
        $path = Normalize::diskFullPath($path);
        return (file_exists($path) && is_readable($path) && is_file($path));
    }

    /**
     * Alias for exist method
     * @param string $path
     * @return bool
     */
    public static function readable($path) {
        return self::exist($path);
    }

    /**
     * Check is file writable
     * @param string $path
     * @return bool
     */
    public static function writable($path)

    {
        $path = Normalize::diskFullPath($path);

        if (!self::exist($path)) {
            return false;
        }

        return is_writable($path);
    }

    /**
     * Check is file executable
     * @param string $path
     * @return bool
     */
    public static function executable($path)
    {
        $path = Normalize::diskFullPath($path);

        if (!self::exist($path)) {
            return false;
        }

        return is_executable($path);
    }

    /**
     * @param string $path
     * @param null|string $content
     * @param null|int $flags
     * @return int
     */
    public static function write($path, $content = null, $flags = null)
    {
        $path = Normalize::diskFullPath($path);

        $pathArray = explode(DIRECTORY_SEPARATOR, $path);
        array_pop($pathArray);
        $pathName = implode(DIRECTORY_SEPARATOR, $pathArray);

        if (Directory::exist($pathName)) {
            Directory::create($pathName);
        }
        return @file_put_contents($path, $content, $flags);
    }

    /**
     * Remove file
     * @param string $path
     * @return bool
     */
    public static function remove($path)
    {
        $path = Normalize::diskFullPath($path);

        if (!self::exist($path)) {
            return false;
        }
        return unlink($path);
    }


    /**
     * Alternative of functions include, require, include_once and etc in 1 function
     * @param string $path
     * @param bool|false $return
     * @param bool|false $once
     * @return bool|mixed
     */
    public static function inc($path, $return = false, $once = false)
    {
        $path = Normalize::diskFullPath($path);

        if (!self::exist($path)) {
            return false;
        }

        if ($return === true) {
            return $once === true ? require_once($path) : require $path;
        } else {
            ($once == true) ? require_once($path) : require $path;

        }
    }

    /**
     * Get file make time in unix timestamp
     * @param string $path
     * @return int
     */
    public static function mTime($path)
    {
        $path = Normalize::diskFullPath($path);
        if (!self::exist($path)) {
            return 0;
        }

        return filemtime($path);
    }

    /**
     * Recursive scan directory, based on $path and allowed extensions $ext or without it
     * @param string $path
     * @param array $ext
     * @param bool $returnRelative
     * @param $files
     * @return array
     */
    public static function listFiles($path, array $ext = null, $returnRelative = false, &$files = [])
    {
        $path = Normalize::diskFullPath($path);

        $dir = opendir($path . '/.');
        while ($item = readdir($dir)) {
            if (is_file($sub = $path . '/' . $item)) {
                $item_ext = Str::lastIn($item, '.');
                if ($ext === null || Arr::in($item_ext, $ext)) {
<?php

function getDate($date)
{
    if ($date !== null) {
        return new DateTime($date);
    }

    return false;
}
                    if ($returnRelative) {
                        $files[] = $item;
                    } else {
                        $files[] = $path . DIRECTORY_SEPARATOR . $item;
                    }
                }
            } else {
                if ($item !== '.' && $item !== '..') {
                    self::listFiles($sub, $ext, $returnRelative, $files);
                }
            }
        }

        return $files;
    }

    /**
     * Get file size in bytes
     * @param string $path
     * @return int
     */
    public static function size($path)
    {
        $path = Normalize::diskFullPath($path);

        if (!self::exist($path)) {
            return 0;
        }

        return filesize($path);
    }

}

1		<?php
2
3		namespace Ffcms\Core\Helper\FileSystem;
4
5		use Ffcms\Core\Helper\Type\Arr;
6		use Ffcms\Core\Helper\Type\Str;
7
8		/**
9		* Class File. Provide methods to work with files in current filesystem.
10		* @package Ffcms\Core\Helper
11		*/
12		class File
13		{
14
15		/**
16		* Read file content from local storage
17		* @param $path
18		* @return bool\|string
19		*/
20		public static function read($path)
21		{
22		$path = Normalize::diskFullPath($path);
23
24		if (!self::exist($path)) {
25		return false;
26		}
27
28		return @file_get_contents($path);
29		}
30
31		/**
32		* Check if $path is exist and readable in filesystem
33		* @param string $path
34		* @return bool
35		*/
36		public static function exist($path)
37		{
38		$path = Normalize::diskFullPath($path);
39		return (file_exists($path) && is_readable($path) && is_file($path));
40		}
41
42		/**
43		* Alias for exist method
44		* @param string $path
45		* @return bool
46		*/
47		public static function readable($path) {
48		return self::exist($path);
49		}
50
51		/**
52		* Check is file writable
53		* @param string $path
54		* @return bool
55		*/
56	View Code Duplication	public static function writable($path)
		0 ignored issues – show Duplication introduced 2015-12-14 09:59 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...
57		{
58		$path = Normalize::diskFullPath($path);
59
60		if (!self::exist($path)) {
61		return false;
62		}
63
64		return is_writable($path);
65		}
66
67		/**
68		* Check is file executable
69		* @param string $path
70		* @return bool
71		*/
72		public static function executable($path)
73		{
74		$path = Normalize::diskFullPath($path);
75
76		if (!self::exist($path)) {
77		return false;
78		}
79
80		return is_executable($path);
81		}
82
83		/**
84		* @param string $path
85		* @param null\|string $content
86		* @param null\|int $flags
87		* @return int
88		*/
89		public static function write($path, $content = null, $flags = null)
90		{
91		$path = Normalize::diskFullPath($path);
92
93		$pathArray = explode(DIRECTORY_SEPARATOR, $path);
94		array_pop($pathArray);
95		$pathName = implode(DIRECTORY_SEPARATOR, $pathArray);
96
97		if (Directory::exist($pathName)) {
98		Directory::create($pathName);
99		}
100		return @file_put_contents($path, $content, $flags);
101		}
102
103		/**
104		* Remove file
105		* @param string $path
106		* @return bool
107		*/
108		public static function remove($path)
109		{
110		$path = Normalize::diskFullPath($path);
111
112		if (!self::exist($path)) {
113		return false;
114		}
115		return unlink($path);
116		}
117
118
119		/**
120		* Alternative of functions include, require, include_once and etc in 1 function
121		* @param string $path
122		* @param bool\|false $return
123		* @param bool\|false $once
124		* @return bool\|mixed
125		*/
126		public static function inc($path, $return = false, $once = false)
127		{
128		$path = Normalize::diskFullPath($path);
129
130		if (!self::exist($path)) {
131		return false;
132		}
133
134		if ($return === true) {
135		return $once === true ? require_once($path) : require $path;
136		} else {
137		($once == true) ? require_once($path) : require $path;
		0 ignored issues – show Coding Style Best Practice introduced 2015-11-21 16:41 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing two booleans. Considering using the strict comparison `===` instead. When comparing two booleans, it is generally considered safer to use the strict comparison operator. Loading history...
138		}
139		}
140
141		/**
142		* Get file make time in unix timestamp
143		* @param string $path
144		* @return int
145		*/
146		public static function mTime($path)
147		{
148		$path = Normalize::diskFullPath($path);
149		if (!self::exist($path)) {
150		return 0;
151		}
152
153		return filemtime($path);
154		}
155
156		/**
157		* Recursive scan directory, based on $path and allowed extensions $ext or without it
158		* @param string $path
159		* @param array $ext
160		* @param bool $returnRelative
161		* @param $files
162		* @return array
163		*/
164		public static function listFiles($path, array $ext = null, $returnRelative = false, &$files = [])
165		{
166		$path = Normalize::diskFullPath($path);
167
168		$dir = opendir($path . '/.');
169		while ($item = readdir($dir)) {
170		if (is_file($sub = $path . '/' . $item)) {
171		$item_ext = Str::lastIn($item, '.');
172		if ($ext === null \|\| Arr::in($item_ext, $ext)) {
		0 ignored issues – show Security Bug introduced 2015-12-18 08:55 UTC by Report Bug Copy Issue Report It seems like `$item_ext` defined by `\Ffcms\Core\Helper\Type\Str::lastIn($item, '.')` on line `171` can also be of type `false`; however, `Ffcms\Core\Helper\Type\Arr::in()` does only seem to accept `string`, did you maybe forget to handle an error condition? This check looks for type mismatches where the missing type is `false`. This is usually indicative of an error condtion. Consider the follow example <?php function getDate($date) { if ($date !== null) { return new DateTime($date); } return false; } This function either returns a new `DateTime` object or false, if there was an error. This is a typical pattern in PHP programming to show that an error has occurred without raising an exception. The calling code should check for this returned `false` before passing on the value to another function or method that may not be able to handle a `false`. Loading history...
173		if ($returnRelative) {
174		$files[] = $item;
175		} else {
176		$files[] = $path . DIRECTORY_SEPARATOR . $item;
177		}
178		}
179		} else {
180		if ($item !== '.' && $item !== '..') {
181		self::listFiles($sub, $ext, $returnRelative, $files);
182		}
183		}
184		}
185
186		return $files;
187		}
188
189		/**
190		* Get file size in bytes
191		* @param string $path
192		* @return int
193		*/
194		public static function size($path)
195		{
196		$path = Normalize::diskFullPath($path);
197
198		if (!self::exist($path)) {
199		return 0;
200		}
201
202		return filesize($path);
203		}
204
205		}

phpffcms / ffcms-core

Push — master ( 337d70...6470c4 )

File::inc() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like