BackupFilesystem::locateCommand() - Code Metrics - Inspection of "Adding process locating logic." - bradcornford/Backup - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 0edaaa...4d67dd )

by Bradley

created 2016-01-13 15:21 UTC

BackupFilesystem::locateCommand() C

↳ Parent: BackupFilesystem

Complexity

Conditions	7
Paths	13

Size

Total Lines	26
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	26
rs	6.7273
cc	7
eloc	18
nc	13
nop	1

<?php namespace Cornford\Backup;

use Cornford\Backup\Contracts\BackupFilesystemInterface;

class BackupFilesystem implements BackupFilesystemInterface {

	const OS_UNKNOWN = 1;
	const OS_WIN = 2;
	const OS_LINUX = 3;
	const OS_OSX = 4;

	/**
	 * Check path exists.
	 *
	 * @param string $path
	 *
	 * @return boolean
	 */
	public function checkPathExists($path)
	{
		if (!is_dir($path)) {
			return false;
		}

		return true;
	}

	/**
	 * Check file exists.
	 *
	 * @param string $filepath
	 *
	 * @return boolean
	 */
	public function checkFileExists($filepath)
	{
		if (!$this->checkPathExists(dirname($filepath))) {
			return false;
		}

		if (!is_file($filepath)) {
			return false;
		}

		return true;
	}

	/**
	 * Check if file is empty.
	 *
	 * @param string $filepath
	 *
	 * @return boolean
	 */
	public function checkFileEmpty($filepath)
	{
		return filesize($filepath) == 0;
	}

	/**
	* Remove file.
	*
	* @param string $filepath
	*
	* @return void
	*/
	public function removeFile($filepath)
	{
		unlink($filepath);
	}

	/**
	* Write a compressed file.
	*
	* @param string $compressedFilepath
	* @param string $filepath
	*
	* @return void
	*/
	public function writeCompressedFile($compressedFilepath, $filepath)
	{
		$filePointer = fopen($compressedFilepath, 'w');
		fwrite($filePointer, gzencode(file_get_contents($filepath), 9));
		fclose($filePointer);
	}

	/**
	* Write an uncompressed file.
	*
	* @param string $uncompressedFilepath
	* @param string $filepath
	*
	* @return void
	*/
	public function writeUncompressedFile($uncompressedFilepath, $filepath)
	{
		$filePointer = fopen($uncompressedFilepath, 'w');
		fwrite($filePointer, gzdecode(file_get_contents($filepath, 9)));
		fclose($filePointer);
	}

	/**
	 * Get the operating system.
	 *
	 * @return integer
	 */
	public function getOperatingSystem()
	{
		switch (true) {

			case stristr(PHP_OS, 'DAR'):
				return self::OS_OSX;
			case stristr(PHP_OS, 'WIN'):
				return self::OS_WIN;
			case stristr(PHP_OS, 'LINUX'):
				return self::OS_LINUX;
			default:
				return self::OS_UNKNOWN;
		}
	}

	/**
	 * Locate command location.
	 *
	 * @param string $command
	 *
	 * @return string|false
	 */
	public function locateCommand($command)
	{
		switch ($this->getOperatingSystem()) {
			case self::OS_OSX:
			case self::OS_LINUX:
				exec(sprintf('which %s', $command), $result, $returnCode);
				if (isset($result[0])) {
					$result = substr($result[0], 0, strrpos($result[0], '/') + 1);
				}
				break;
			case self::OS_WIN:
				exec(sprintf('where %s', $command), $result, $returnCode);
				if (isset($result[0])) {
					$result = $result[0];
				}
				break;
			default:
				return false;
		}

		if (empty($result)) {
			return false;
		}

		return $result;
	}

	/**
	* Check a function exists.
	*
	* @param string $function
	*
	* @return boolean
	*/
	public function checkFunctionExists($function)
	{
		return function_exists($function);
	}

}

1			<?php namespace Cornford\Backup;
2
3			use Cornford\Backup\Contracts\BackupFilesystemInterface;
4
5			class BackupFilesystem implements BackupFilesystemInterface {
6
7			const OS_UNKNOWN = 1;
8			const OS_WIN = 2;
9			const OS_LINUX = 3;
10			const OS_OSX = 4;
11
12			/**
13			* Check path exists.
14			*
15			* @param string $path
16			*
17			* @return boolean
18			*/
19			public function checkPathExists($path)
20			{
21			if (!is_dir($path)) {
22			return false;
23			}
24
25			return true;
26			}
27
28			/**
29			* Check file exists.
30			*
31			* @param string $filepath
32			*
33			* @return boolean
34			*/
35			public function checkFileExists($filepath)
36			{
37			if (!$this->checkPathExists(dirname($filepath))) {
38			return false;
39			}
40
41			if (!is_file($filepath)) {
42			return false;
43			}
44
45			return true;
46			}
47
48			/**
49			* Check if file is empty.
50			*
51			* @param string $filepath
52			*
53			* @return boolean
54			*/
55			public function checkFileEmpty($filepath)
56			{
57			return filesize($filepath) == 0;
58			}
59
60			/**
61			* Remove file.
62			*
63			* @param string $filepath
64			*
65			* @return void
66			*/
67			public function removeFile($filepath)
68			{
69			unlink($filepath);
70			}
71
72			/**
73			* Write a compressed file.
74			*
75			* @param string $compressedFilepath
76			* @param string $filepath
77			*
78			* @return void
79			*/
80			public function writeCompressedFile($compressedFilepath, $filepath)
81			{
82			$filePointer = fopen($compressedFilepath, 'w');
83			fwrite($filePointer, gzencode(file_get_contents($filepath), 9));
84			fclose($filePointer);
85			}
86
87			/**
88			* Write an uncompressed file.
89			*
90			* @param string $uncompressedFilepath
91			* @param string $filepath
92			*
93			* @return void
94			*/
95			public function writeUncompressedFile($uncompressedFilepath, $filepath)
96			{
97			$filePointer = fopen($uncompressedFilepath, 'w');
98			fwrite($filePointer, gzdecode(file_get_contents($filepath, 9)));
99			fclose($filePointer);
100			}
101
102			/**
103			* Get the operating system.
104			*
105			* @return integer
106			*/
107			public function getOperatingSystem()
108			{
109			switch (true) {
			0 ignored issues – show Bug Best Practice introduced 2016-01-13 15:28 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing `stristr(PHP_OS, 'DAR')` of type `string` to the boolean `true`. If you are specifically checking for a non-empty string, consider using the more explicit `!== ''` instead. Loading history... Bug Best Practice introduced 2016-01-13 15:28 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing `stristr(PHP_OS, 'WIN')` of type `string` to the boolean `true`. If you are specifically checking for a non-empty string, consider using the more explicit `!== ''` instead. Loading history... Bug Best Practice introduced 2016-01-13 15:28 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing `stristr(PHP_OS, 'LINUX')` of type `string` to the boolean `true`. If you are specifically checking for a non-empty string, consider using the more explicit `!== ''` instead. Loading history...
110			case stristr(PHP_OS, 'DAR'):
111			return self::OS_OSX;
112			case stristr(PHP_OS, 'WIN'):
113			return self::OS_WIN;
114			case stristr(PHP_OS, 'LINUX'):
115			return self::OS_LINUX;
116			default:
117			return self::OS_UNKNOWN;
118			}
119			}
120
121			/**
122			* Locate command location.
123			*
124			* @param string $command
125			*
126			* @return string\|false
127			*/
128			public function locateCommand($command)
129			{
130			switch ($this->getOperatingSystem()) {
131			case self::OS_OSX:
132			case self::OS_LINUX:
133			exec(sprintf('which %s', $command), $result, $returnCode);
134			if (isset($result[0])) {
135			$result = substr($result[0], 0, strrpos($result[0], '/') + 1);
136			}
137			break;
138			case self::OS_WIN:
139			exec(sprintf('where %s', $command), $result, $returnCode);
140			if (isset($result[0])) {
141			$result = $result[0];
142			}
143			break;
144			default:
145			return false;
146			}
147
148			if (empty($result)) {
149			return false;
150			}
151
152			return $result;
153			}
154
155			/**
156			* Check a function exists.
157			*
158			* @param string $function
159			*
160			* @return boolean
161			*/
162			public function checkFunctionExists($function)
163			{
164			return function_exists($function);
165			}
166
167			}

bradcornford / Backup

Push — master ( 0edaaa...4d67dd )

BackupFilesystem::locateCommand() C

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like