Utils::recurseDelete() - Code Metrics - Inspection of "Modification of getAllLanguages function" - YetiForceCompany/YetiForcePortal2 - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Pull Request — developer (#252)

by Arkadiusz

created 2021-05-05 09:45 UTC

Utils::recurseDelete() B

↳ Parent: Utils

Complexity

Conditions	7
Paths	15

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	28
rs	8.5386
c	0
b	0
f	0
cc	7
nc	15
nop	1

<?php
/**
 * The file contains: Utils class.
 *
 * @package App
 *
 * @copyright YetiForce Sp. z o.o.
 * @license   YetiForce Public License 3.0 (licenses/LicenseEN.txt or yetiforce.com)
 * @author    Mariusz Krzaczkowski <[email protected]>
 */

namespace App;

/**
 * Utils class.
 */
class Utils
{
	public static function recurseDelete(string $src)
	{
		$vendorDir = \dirname(\dirname(__FILE__));
		$rootDir = \dirname(\dirname($vendorDir)) . \DIRECTORY_SEPARATOR;
		if (!file_exists($rootDir . $src)) {
			return;
		}
		$dirs = [];
		if (is_dir($src)) {
			$dirs[] = $rootDir . $src;
		}
		@chmod($rootDir . $src, 0777);
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
		if (is_dir($src)) {
			foreach ($iterator = new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator($src, \RecursiveDirectoryIterator::SKIP_DOTS), \RecursiveIteratorIterator::SELF_FIRST) as $item) {
				if ($item->isDir()) {
					$dirs[] = $rootDir . $src . \DIRECTORY_SEPARATOR . $iterator->getSubPathName();
				} else {
					unlink($rootDir . $src . \DIRECTORY_SEPARATOR . $iterator->getSubPathName());
				}
			}
			arsort($dirs);
			foreach ($dirs as $dir) {
				rmdir($dir);
			}
		} else {
			unlink($rootDir . $src);
		}
	}

	/**
	 * Get absolute URL for Portal2.
	 *
	 * @param string $url
	 *
	 * @return string
	 */
	public static function absoluteUrl(string $url): string
	{
		return \App\Config::get('portalUrl') . $url;
	}

	/**
	 * Replacement for the ucfirst function for proper Multibyte String operation.
	 * Delete function will exist as mb_ucfirst.
	 *
	 * @param string $string
	 *
	 * @return string
	 */
	public static function mbUcfirst(string $string): string
	{
		return mb_strtoupper(mb_substr($string, 0, 1)) . mb_substr($string, 1);
	}
}


1			<?php
2			/**
3			* The file contains: Utils class.
4			*
5			* @package App
6			*
7			* @copyright YetiForce Sp. z o.o.
8			* @license YetiForce Public License 3.0 (licenses/LicenseEN.txt or yetiforce.com)
9			* @author Mariusz Krzaczkowski <[email protected]>
10			*/
11
12			namespace App;
13
14			/**
15			* Utils class.
16			*/
17			class Utils
18			{
19			public static function recurseDelete(string $src)
20			{
21			$vendorDir = \dirname(\dirname(__FILE__));
22			$rootDir = \dirname(\dirname($vendorDir)) . \DIRECTORY_SEPARATOR;
23			if (!file_exists($rootDir . $src)) {
24			return;
25			}
26			$dirs = [];
27			if (is_dir($src)) {
28			$dirs[] = $rootDir . $src;
29			}
30			@chmod($rootDir . $src, 0777);
			0 ignored issues – show Security Best Practice introduced 2021-05-04 12:33 UTC by Report Bug Copy Issue Report It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended. If you suppress an error, we recommend checking for the error condition explicitly: // For example instead of @mkdir($dir); // Better use if (@mkdir($dir) === false) { throw new \RuntimeException('The directory '.$dir.' could not be created.'); } Loading history...
31			if (is_dir($src)) {
32			foreach ($iterator = new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator($src, \RecursiveDirectoryIterator::SKIP_DOTS), \RecursiveIteratorIterator::SELF_FIRST) as $item) {
33			if ($item->isDir()) {
34			$dirs[] = $rootDir . $src . \DIRECTORY_SEPARATOR . $iterator->getSubPathName();
35			} else {
36			unlink($rootDir . $src . \DIRECTORY_SEPARATOR . $iterator->getSubPathName());
37			}
38			}
39			arsort($dirs);
40			foreach ($dirs as $dir) {
41			rmdir($dir);
42			}
43			} else {
44			unlink($rootDir . $src);
45			}
46			}
47
48			/**
49			* Get absolute URL for Portal2.
50			*
51			* @param string $url
52			*
53			* @return string
54			*/
55			public static function absoluteUrl(string $url): string
56			{
57			return \App\Config::get('portalUrl') . $url;
58			}
59
60			/**
61			* Replacement for the ucfirst function for proper Multibyte String operation.
62			* Delete function will exist as mb_ucfirst.
63			*
64			* @param string $string
65			*
66			* @return string
67			*/
68			public static function mbUcfirst(string $string): string
69			{
70			return mb_strtoupper(mb_substr($string, 0, 1)) . mb_substr($string, 1);
71			}
72			}
73

YetiForceCompany / YetiForcePortal2

Pull Request — developer (#252)

Utils::recurseDelete() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like