File::delete() - Code Metrics - Inspection of "Upgrade project" - andrey-helldar/support - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — main (#22)

by Andrey

created 2020-12-29 14:32 UTC

File::delete() A

↳ Parent: File

Complexity

Conditions	4
Paths	4

Size

Total Lines	18
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	1	Features	0

Metric	Value
cc	4
eloc	9
c	1
b	1
f	0
nc	4
nop	1
dl	0
loc	18
rs	9.9666

<?php

namespace Helldar\Support\Helpers\Filesystem;

use DirectoryIterator;
use ErrorException;
use Helldar\Support\Facades\Helpers\Arr;
use Helldar\Support\Facades\Helpers\Filesystem\Directory;
.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php
use Helldar\Support\Facades\Helpers\Instance;
use SplFileInfo;

class File
{
    /**
     * Save content to file.
     *
     * @param  string  $path
     * @param  string  $content
     * @param  int  $mode
     */
    public function store(string $path, string $content, int $mode = 755): void
    {
        Directory::make(pathinfo($path, PATHINFO_DIRNAME), $mode);

        file_put_contents($path, $content);
    }

    /**
     * Checks if the file exists.
     *
     * @param  string  $path
     *
     * @return bool
     */
    public function exists(string $path): bool
    {
        return file_exists($path) && is_file($path);
    }

    /**
     * Deletes files in the specified paths.
     *
     * @param  string|string[]  $paths
     *
     * @return bool
     */
    public function delete($paths): bool
    {
        $paths = Arr::wrap($paths);

        $success = true;

        foreach ($paths as $path) {
            try {
                if (! @unlink($path)) {
                    $success = false;
                }
            }
            catch (ErrorException $e) {
                $success = false;
            }
        }

        return $success;
    }

    /**
     * Checks if an object or link is a file at the specified path.
     *
     * @param  \DirectoryIterator|\SplFileInfo|string  $value
     *
     * @return bool
     */
    public function isFile($value): bool
    {
        if (Instance::of($value, [SplFileInfo::class, DirectoryIterator::class])) {
            return $value->isFile();
        }

        return is_file($value);
    }
}


1			<?php
2
3			namespace Helldar\Support\Helpers\Filesystem;
4
5			use DirectoryIterator;
6			use ErrorException;
7			use Helldar\Support\Facades\Helpers\Arr;
8			use Helldar\Support\Facades\Helpers\Filesystem\Directory;
			0 ignored issues – show Bug introduced 2020-12-28 14:58 UTC by Report Bug Copy Issue Report This use statement conflicts with another class in this namespace, `Helldar\Support\Helpers\Filesystem\Directory`. Consider defining an alias. Let?s assume that you have a directory layout like this: . \|-- OtherDir \| \|-- Bar.php \| `-- Foo.php `-- SomeDir `-- Foo.php and let?s assume the following content of `Bar.php`: // Bar.php namespace OtherDir; use SomeDir\Foo; // This now conflicts the class OtherDir\Foo If both files `OtherDir/Foo.php` and `SomeDir/Foo.php` are loaded in the same runtime, you will see a PHP error such as the following: PHP Fatal error: Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php However, as `OtherDir/Foo.php` does not necessarily have to be loaded and the error is only triggered if it is loaded before `OtherDir/Bar.php`, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias: // Bar.php namespace OtherDir; use SomeDir\Foo as SomeDirFoo; // There is no conflict anymore. Loading history...
9			use Helldar\Support\Facades\Helpers\Instance;
10			use SplFileInfo;
11
12			class File
13			{
14			/**
15			* Save content to file.
16			*
17			* @param string $path
18			* @param string $content
19			* @param int $mode
20			*/
21			public function store(string $path, string $content, int $mode = 755): void
22			{
23			Directory::make(pathinfo($path, PATHINFO_DIRNAME), $mode);
24
25			file_put_contents($path, $content);
26			}
27
28			/**
29			* Checks if the file exists.
30			*
31			* @param string $path
32			*
33			* @return bool
34			*/
35			public function exists(string $path): bool
36			{
37			return file_exists($path) && is_file($path);
38			}
39
40			/**
41			* Deletes files in the specified paths.
42			*
43			* @param string\|string[] $paths
44			*
45			* @return bool
46			*/
47			public function delete($paths): bool
48			{
49			$paths = Arr::wrap($paths);
50
51			$success = true;
52
53			foreach ($paths as $path) {
54			try {
55			if (! @unlink($path)) {
56			$success = false;
57			}
58			}
59			catch (ErrorException $e) {
60			$success = false;
61			}
62			}
63
64			return $success;
65			}
66
67			/**
68			* Checks if an object or link is a file at the specified path.
69			*
70			* @param \DirectoryIterator\|\SplFileInfo\|string $value
71			*
72			* @return bool
73			*/
74			public function isFile($value): bool
75			{
76			if (Instance::of($value, [SplFileInfo::class, DirectoryIterator::class])) {
77			return $value->isFile();
78			}
79
80			return is_file($value);
81			}
82			}
83

andrey-helldar / support

Pull Request — main (#22)

File::delete() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like