Backup_Utilities::is_safe_mode_on() - Code Metrics - Inspection of "Use Process for all our system calls" - humanmade/backupwordpress - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#1001)

by Paul

created 2016-02-29 17:56 UTC

Backup_Utilities::is_safe_mode_on() A

↳ Parent: Backup_Utilities

Complexity

Conditions	3
Paths	2

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	11
rs	9.4285
cc	3
eloc	5
nc	2
nop	1

<?php

namespace HM\BackUpWordPress;

use Symfony\Component\Process\Process as Process;

/**
 * A set of Backup Utility functions
 */
class Backup_Utilities {

	/**
	 * Checks whether Safe Mode is currently on
	 *
	 * @param  string  $ini_get_callback By default we use `ini_get` to check for
	 *                                   the Safe Mode setting but this can be
	 *                                   overridden for testing purposes.
	 *
	 * @return boolean                   Whether Safe Mode is on or off.
	 */
	public static function is_safe_mode_on( $ini_get_callback = 'ini_get' ) {

		$safe_mode = @call_user_func( $ini_get_callback, 'safe_mode' );

		if ( $safe_mode && strtolower( $safe_mode ) !== 'off' ) {

			return true;
		}

		return false;

	}

	/**
	 * Attempt to work out path to a cli executable.
	 *
	 * @param  array $paths An array of paths to check against.
	 *
	 * @return string|false        The path to the executable.
	 */
	public static function get_executable_path( $paths ) {

		if ( ! function_exists( 'proc_open' ) ) {
			return false;
		}

		$paths = array_map( 'wp_normalize_path', $paths );

		foreach ( $paths as $path ) {

			/**
			 * Attempt to call `--version` on each path, the one which works
			 * must be the correct path.
			 */
			 $process = new Process( $path . ' --version' );

			try {
				$process->run();

			} catch ( \Exception $e ) {
				return false;
			}

			// If the command executed successfully then this must be the correct path
			if ( $process->isSuccessful() ) {
				return $path;
			}

		}

		return false;

	}
}


1			<?php
2
3			namespace HM\BackUpWordPress;
4
5			use Symfony\Component\Process\Process as Process;
6
7			/**
8			* A set of Backup Utility functions
9			*/
10			class Backup_Utilities {
11
12			/**
13			* Checks whether Safe Mode is currently on
14			*
15			* @param string $ini_get_callback By default we use `ini_get` to check for
16			* the Safe Mode setting but this can be
17			* overridden for testing purposes.
18			*
19			* @return boolean Whether Safe Mode is on or off.
20			*/
21			public static function is_safe_mode_on( $ini_get_callback = 'ini_get' ) {
22
23			$safe_mode = @call_user_func( $ini_get_callback, 'safe_mode' );
24
25			if ( $safe_mode && strtolower( $safe_mode ) !== 'off' ) {
			1 ignored issue – show Unused Code introduced 2015-11-21 01:52 UTC by Report Bug Copy Issue Report This `if` statement, and the following `return` statement can be replaced with `return $safe_mode && str...($safe_mode) !== 'off';`. Loading history...
26			return true;
27			}
28
29			return false;
30
31			}
32
33			/**
34			* Attempt to work out path to a cli executable.
35			*
36			* @param array $paths An array of paths to check against.
37			*
38			* @return string\|false The path to the executable.
39			*/
40			public static function get_executable_path( $paths ) {
41
42			if ( ! function_exists( 'proc_open' ) ) {
43			return false;
44			}
45
46			$paths = array_map( 'wp_normalize_path', $paths );
47
48			foreach ( $paths as $path ) {
49
50			/**
51			* Attempt to call `--version` on each path, the one which works
52			* must be the correct path.
53			*/
54			$process = new Process( $path . ' --version' );
55
56			try {
57			$process->run();
58
59			} catch ( \Exception $e ) {
60			return false;
61			}
62
63			// If the command executed successfully then this must be the correct path
64			if ( $process->isSuccessful() ) {
65			return $path;
66			}
67
68			}
69
70			return false;
71
72			}
73			}
74

humanmade / backupwordpress

GitHub Access Token became invalid

Pull Request — master (#1001)

Backup_Utilities::is_safe_mode_on() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like