GitHub Access Token became invalid

It seems like the GitHub access token used for retrieving details about this repository from GitHub became invalid. This might prevent certain types of inspections from being run (in particular, everything related to pull requests).
Please ask an admin of your repository to re-new the access token on this website.
Completed
Pull Request — master (#988)
by Paul
17:56
created

Path::get_home_path()   C

Complexity

Conditions 8
Paths 8

Size

Total Lines 33
Code Lines 21

Duplication

Lines 0
Ratio 0 %

Importance

Changes 8
Bugs 0 Features 1
Metric Value
c 8
b 0
f 1
dl 0
loc 33
rs 5.3846
cc 8
eloc 21
nc 8
nop 1
1
<?php
2
3
namespace HM\BackUpWordPress;
4
5
/**
6
 * Manages both the backup path and site root
7
 *
8
 * Handles calculating & protecting the directory that backups will be stored in
9
 * as well as the directory that is being backed up
10
 */
11
class Path {
12
13
	/**
14
	 * The path to the directory that backup files are stored in
15
	 *
16
	 * @var string $this->path
17
	 */
18
	private $path;
19
20
	/**
21
	 * The path to the directory that will be backed up
22
	 *
23
	 * @var string $this->root
24
	 */
25
	private $root;
26
27
	/**
28
	 * The path to the directory that backup files are stored in
29
	 *
30
	 * @var string $this->path
31
	 */
32
	private $custom_path;
33
34
	/**
35
	 * Contains the instantiated Path instance
36
	 *
37
	 * @var Path $this->instance
38
	 */
39
	private static $instance;
40
41
	/**
42
	 * Protected constructor to prevent creating a new instance of the
43
	 * *Singleton* via the `new` operator from outside of this class.
44
	 */
45
	protected function __construct() {}
46
47
	/**
48
	 * Private clone method to prevent cloning of the instance of the
49
	 * *Singleton* instance.
50
	 */
51
	private function __clone() {}
52
53
	/**
54
	 * Private unserialize method to prevent unserializing of the *Singleton*
55
	 * instance.
56
	 */
57
	private function __wakeup() {}
58
59
	/**
60
	 * Returns the *Singleton* instance of this class.
61
	 *
62
	 * @staticvar Path $instance The *Singleton* instances of this class.
63
	 *
64
	 * @return Path The *Singleton* instance.
65
	 */
66
	public static function get_instance() {
67
68
		if ( ! ( self::$instance instanceof Path ) ) {
69
			self::$instance = new Path();
70
		}
71
72
		return self::$instance;
73
	}
74
75
	/**
76
	 * Convenience method for quickly grabbing the path
77
	 */
78
	public static function get_path() {
0 ignored issues
show
Documentation introduced by
The return type could not be reliably inferred; please add a @return annotation.

Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a @return annotation as described here.

Loading history...
79
		return self::get_instance()->get_calculated_path();
80
	}
81
82
	/**
83
	 * Convenience method for quickly grabbing the root
84
	 */
85
	public static function get_root() {
0 ignored issues
show
Documentation introduced by
The return type could not be reliably inferred; please add a @return annotation.

Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a @return annotation as described here.

Loading history...
86
		return self::get_instance()->get_calculated_root();
87
	}
88
89
	/**
90
	 * Calculate the path to the site "home" directory.
91
	 *
92
	 * The home directory is the path equivalent to the home_url. That is,
93
	 * the path to the true root of the website. In situations where WordPress is
94
	 * installed in a subdirectory the home path is different to ABSPATH
95
	 *
96
	 * @param string $site_path The site_path to use when calculating the home path, defaults to ABSPATH
97
	 */
98
	public static function get_home_path( $site_path = ABSPATH ) {
1 ignored issue
show
Documentation introduced by
The return type could not be reliably inferred; please add a @return annotation.

Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a @return annotation as described here.

Loading history...
Coding Style introduced by
get_home_path uses the super-global variable $_SERVER which is generally not recommended.

Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable:

// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
Loading history...
99
100
		if ( defined( 'HMBKP_ROOT' ) && HMBKP_ROOT ) {
101
			return wp_normalize_path( HMBKP_ROOT );
102
		}
103
104
		$home_path = $site_path;
105
106
		if ( path_in_php_open_basedir( dirname( $site_path ) ) ) {
107
108
			$home    = set_url_scheme( get_option( 'home' ), 'http' );
109
			$siteurl = set_url_scheme( get_option( 'siteurl' ), 'http' );
110
			if ( ! empty( $home ) && 0 !== strcasecmp( $home, $siteurl ) ) {
111
				$wp_path_rel_to_home = str_ireplace( $home, '', $siteurl ); /* $siteurl - $home */
112
				$pos = strripos( wp_normalize_path( $_SERVER['SCRIPT_FILENAME'] ), trailingslashit( $wp_path_rel_to_home ) );
113
				$home_path = substr( $_SERVER['SCRIPT_FILENAME'], 0, $pos );
114
				$home_path = trailingslashit( $home_path );
115
			}
116
117
			if ( is_multisite() ) {
118
				require_once ABSPATH . '/wp-admin/includes/network.php';
119
				$hostname          = get_clean_basedomain();
0 ignored issues
show
Unused Code introduced by
$hostname is not used, you could remove the assignment.

This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.

$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.

Loading history...
120
				$slashed_home      = trailingslashit( get_option( 'home' ) );
121
				$base              = parse_url( $slashed_home, PHP_URL_PATH );
122
				$document_root_fix = wp_normalize_path( realpath( $_SERVER['DOCUMENT_ROOT'] ) );
123
				$abspath_fix       = wp_normalize_path( ABSPATH );
124
				$home_path         = 0 === strpos( $abspath_fix, $document_root_fix ) ? $document_root_fix . $base : $home_path;
125
			}
126
		}
127
128
		return wp_normalize_path( untrailingslashit( $home_path ) );
129
130
	}
131
132
	/**
133
	 * get the calculated path to the directory where backups will be stored
134
	 */
135
	private function get_calculated_path() {
0 ignored issues
show
Documentation introduced by
The return type could not be reliably inferred; please add a @return annotation.

Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a @return annotation as described here.

Loading history...
136
137
		// Calculate the path if needed
138
		if ( empty( $this->path ) || ! wp_is_writable( $this->path ) ) {
139
			$this->calculate_path();
140
		}
141
142
		// Ensure the backup directory is protected
143
		$this->protect_path();
144
145
		return wp_normalize_path( $this->path );
146
147
	}
148
149
	/**
150
	 * Set the path directly, overriding the default
151
	 *
152
	 * @param $path
153
	 */
154
	public function set_path( $path ) {
155
156
		$this->custom_path = $path;
157
158
		// Re-calculate the backup path
159
		$this->calculate_path();
160
161
	}
162
163
	/**
164
	 * get the calculated path to the directory that will be backed up
165
	 */
166
	private function get_calculated_root() {
0 ignored issues
show
Documentation introduced by
The return type could not be reliably inferred; please add a @return annotation.

Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a @return annotation as described here.

Loading history...
167
168
		$root = self::get_home_path();
169
170
		if ( defined( 'HMBKP_ROOT' ) && HMBKP_ROOT ) {
171
			$root = HMBKP_ROOT;
172
		}
173
174
		if ( $this->root ) {
175
			$root = $this->root;
176
		}
177
178
		return wp_normalize_path( $root );
179
180
	}
181
182
	/**
183
	 * Set the root path directly, overriding the default
184
	 *
185
	 * @param $root
186
	 */
187
	public function set_root( $root ) {
188
		$this->root = $root;
189
	}
190
191
	public function reset_path() {
192
		$this->set_path( false );
193
	}
194
195
	/**
196
	 * Get the path to the default backup location in wp-content
197
	 */
198
	public function get_default_path() {
199
		return trailingslashit( wp_normalize_path( WP_CONTENT_DIR ) ) . 'backupwordpress-' . substr( HMBKP_SECURE_KEY, 0, 10 ) . '-backups';
200
	}
201
202
	/**
203
	 * Get the path to the fallback backup location in uploads
204
	 */
205
	public function get_fallback_path() {
206
207
		$upload_dir = wp_upload_dir();
208
209
		return trailingslashit( wp_normalize_path( $upload_dir['basedir'] ) ) . 'backupwordpress-' . substr( HMBKP_SECURE_KEY, 0, 10 ) . '-backups';
210
211
	}
212
213
	/**
214
	 * Get the path to the custom backup location if it's been set
215
	 */
216
	public function get_custom_path() {
0 ignored issues
show
Documentation introduced by
The return type could not be reliably inferred; please add a @return annotation.

Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a @return annotation as described here.

Loading history...
217
218
		if ( $this->custom_path ) {
219
			return $this->custom_path;
220
		}
221
222
		if ( defined( 'HMBKP_PATH' ) && wp_is_writable( HMBKP_PATH ) ) {
223
			return HMBKP_PATH;
224
		}
225
226
		return '';
227
228
	}
229
230
	/**
231
	 * Builds an array containing existing backups folders.
232
	 *
233
	 * @return array
234
	 */
235
	public function get_existing_paths() {
236
237
		if ( false === $default = glob( WP_CONTENT_DIR . '/backupwordpress-*-backups', GLOB_ONLYDIR ) ) {
238
			$default = array();
239
		}
240
241
		$upload_dir = wp_upload_dir();
242
243
		if ( false === $fallback = glob( $upload_dir['basedir'] . '/backupwordpress-*-backups', GLOB_ONLYDIR ) ) {
244
			$fallback = array();
245
		}
246
247
		$paths = array_merge( $default, $fallback );
248
249
        $paths = array_map( 'wp_normalize_path', $paths );
250
251
		return $paths;
252
253
	}
254
255
	/**
256
	 * Returns the first existing path if there is one
257
	 *
258
	 * @return string Backup path if found empty string if not
259
	 */
260
	public function get_existing_path() {
261
262
		$paths = $this->get_existing_paths();
263
264
		if ( ! empty( $paths[0] ) ) {
265
			return $paths[0];
266
		}
267
268
		return '';
269
270
	}
271
272
	/**
273
	 * Calculate the backup path and create the directory if it doesn't exist.
274
	 *
275
	 * Tries all possible locations and uses the first one possible
276
	 *
277
	 * @return
278
	 */
279
	public function calculate_path() {
280
281
		$paths = array();
282
283
		// If we have a custom path then try to use it
284
		if ( $this->get_custom_path() ) {
285
			$paths[] = $this->get_custom_path();
286
		}
287
288
		// If there is already a backups directory then try to use that
289
		if ( $this->get_existing_path() ) {
290
			$paths[] = $this->get_existing_path();
291
		}
292
293
		// If not then default to a new directory in wp-content
294
		$paths[] = $this->get_default_path();
295
296
		// If that didn't work then fallback to a new directory in uploads
297
		$paths[] = $this->get_fallback_path();
298
299
		// Loop through possible paths, use the first one that exists/can be created and is writable
300
		foreach ( $paths as $path ) {
301
			if ( wp_mkdir_p( $path ) && wp_is_writable( $path ) ) { // Also handles fixing perms / directory already exists
302
				break;
303
			}
304
		}
305
306
		if ( file_exists( $path ) && wp_is_writable( $path ) ) {
307
			$this->path = $path;
0 ignored issues
show
Bug introduced by
The variable $path seems to be defined by a foreach iteration on line 300. Are you sure the iterator is never empty, otherwise this variable is not defined?

It seems like you are relying on a variable being defined by an iteration:

foreach ($a as $b) {
}

// $b is defined here only if $a has elements, for example if $a is array()
// then $b would not be defined here. To avoid that, we recommend to set a
// default value for $b.


// Better
$b = 0; // or whatever default makes sense in your context
foreach ($a as $b) {
}

// $b is now guaranteed to be defined here.
Loading history...
308
		}
309
310
	}
311
312
	/**
313
	 * Protect the directory that backups are stored in
314
	 *
315
	 * - Adds an index.html file in an attempt to disable directory browsing
316
	 * - Adds a .httaccess file to deny direct access if on Apache
317
	 *
318
	 * @param string $reset
319
	 */
320
	public function protect_path( $reset = 'no' ) {
321
322
		global $is_apache;
1 ignored issue
show
Compatibility Best Practice introduced by
Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

Instead of relying on global state, we recommend one of these alternatives:

1. Pass all data via parameters

function myFunction($a, $b) {
    // Do something
}

2. Create a class that maintains your state

class MyClass {
    private $a;
    private $b;

    public function __construct($a, $b) {
        $this->a = $a;
        $this->b = $b;
    }

    public function myFunction() {
        // Do something
    }
}
Loading history...
323
324
		// Protect against directory browsing by including an index.html file
325
		$index = $this->path . '/index.html';
326
327
		if ( 'reset' === $reset && file_exists( $index ) ) {
328
			@unlink( $index );
0 ignored issues
show
Security Best Practice introduced by
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...
329
		}
330
331
		if ( ! file_exists( $index ) && wp_is_writable( $this->path ) ) {
332
			file_put_contents( $index, '' );
333
		}
334
335
		$htaccess = $this->path . '/.htaccess';
336
337
		if ( ( 'reset' === $reset ) && file_exists( $htaccess ) ) {
338
			@unlink( $htaccess );
0 ignored issues
show
Security Best Practice introduced by
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...
339
		}
340
341
		// Protect the directory with a .htaccess file on Apache servers
342
		if ( $is_apache && function_exists( 'insert_with_markers' ) && ! file_exists( $htaccess ) && wp_is_writable( $this->path ) ) {
343
344
			$contents[] = '# ' . sprintf( __( 'This %s file ensures that other people cannot download your backup files.', 'backupwordpress' ), '.htaccess' );
0 ignored issues
show
Coding Style Comprehensibility introduced by
$contents was never initialized. Although not strictly required by PHP, it is generally a good practice to add $contents = array(); before regardless.

Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code.

Let’s take a look at an example:

foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}

As you can see in this example, the array $myArray is initialized the first time when the foreach loop is entered. You can also see that the value of the bar key is only written conditionally; thus, its value might result from a previous iteration.

This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization $myArray = array() either outside or inside the foreach loop.

Loading history...
345
			$contents[] = '';
346
			$contents[] = '<IfModule mod_rewrite.c>';
347
			$contents[] = 'RewriteEngine On';
348
			$contents[] = 'RewriteCond %{QUERY_STRING} !key=' . HMBKP_SECURE_KEY;
349
			$contents[] = 'RewriteRule (.*) - [F]';
350
			$contents[] = '</IfModule>';
351
			$contents[] = '';
352
353
			file_put_contents( $htaccess, '' );
354
355
			insert_with_markers( $htaccess, 'BackUpWordPress', $contents );
356
357
		}
358
359
	}
360
361
	/**
362
	 * If we have more than one path then move any existing backups to the current path and remove them
363
	 */
364
	public function merge_existing_paths() {
365
366
		$paths = $this->get_existing_paths();
367
368
		if ( ( $paths && $this->get_custom_path() ) || count( $paths ) > 1 ) {
0 ignored issues
show
Bug Best Practice introduced by
The expression $paths of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent.

Consider making the comparison explicit by using empty(..) or ! empty(...) instead.

Loading history...
369
			foreach ( $paths as $old_path ) {
370
				$this->move_old_backups( $old_path );
371
			}
372
		}
373
374
	}
375
376
	/**
377
	 * Move backup files from an existing directory and the new
378
	 * location
379
	 *
380
	 * @param string $path 	The path to move the backups from
0 ignored issues
show
Bug introduced by
There is no parameter named $path. Was it maybe removed?

This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function.

Consider the following example. The parameter $italy is not defined by the method finale(...).

/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}

The most likely cause is that the parameter was removed, but the annotation was not.

Loading history...
381
	 */
382
	public function move_old_backups( $from ) {
383
384
		if ( ! is_readable( $from ) ) {
385
			return;
386
		}
387
388
		if ( ! wp_is_writable( Path::get_path() ) ) {
389
			return;
390
		}
391
392
		// Move any existing backups
393
		if ( $handle = opendir( $from ) ) {
394
395
			// Loop through the backup directory
396
			while ( false !== ( $file = readdir( $handle ) ) ) {
397
398
				// Find all zips
399
				if ( 'zip' === pathinfo( $file, PATHINFO_EXTENSION ) ) {
400
401
					// Try to move them
402
					if ( ! @rename( trailingslashit( $from ) . $file, trailingslashit( Path::get_path() ) . $file ) ) {
403
404
405
						// If we can't move them then try to copy them
406
						copy( trailingslashit( $from ) . $file, trailingslashit( Path::get_path() ) . $file );
407
408
					}
409
410
				}
411
			}
412
413
			closedir( $handle );
414
415
		}
416
417
		// Delete the old directory if it's inside WP_CONTENT_DIR
418
		if ( false !== strpos( $from, WP_CONTENT_DIR ) && $from !== Path::get_path() ) {
419
			rmdirtree( $from );
420
		}
421
422
	}
423
424
	/**
425
	 * Clean any temporary / incomplete backups from the backups directory
426
	 */
427
	public function cleanup() {
428
429
		// Don't cleanup a custom path, who knows what other stuff is there
430
		if ( Path::get_path() === $this->get_custom_path() ) {
431
			return;
432
		}
433
434
		foreach ( new CleanUpIterator( new \DirectoryIterator( $this->path ) ) as $file ) {
435
436
			if ( $file->isDot() || ! $file->isReadable() || ! $file->isFile() ) {
437
				continue;
438
			}
439
440
			@unlink( $file->getPathname() );
0 ignored issues
show
Security Best Practice introduced by
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...
441
442
		}
443
444
	}
445
446
}
447
448
class CleanUpIterator extends \FilterIterator {
0 ignored issues
show
Coding Style Compatibility introduced by
PSR1 recommends that each class should be in its own file to aid autoloaders.

Having each class in a dedicated file usually plays nice with PSR autoloaders and is therefore a well established practice. If you use other autoloaders, you might not want to follow this rule.

Loading history...
449
450
	// Don't match index.html,files with zip extension or status logfiles.
451
	public function accept() {
452
		return ! preg_match( '/(index\.html|.*\.zip|.*-running)/', $this->current() );
453
	}
454
}
455