HM\BackUpWordPress\File_Backup_Engine

Complexity

Total Complexity

9

Size/Duplication

Total Lines	101
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	11
Bugs	1	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	set_excludes()	0	3	1
A	verify_backup()	0	15	4
A	__construct()	0	13	1
B	get_files()	0	29	3

<?php

namespace HM\BackUpWordPress;

use Symfony\Component\Finder\Finder as Finder;

/**
 * The File Backup Engine type
 *
 * All File Backup Engine implementations should extend this class
 */
abstract class File_Backup_Engine extends Backup_Engine {

	/**
	 * The array of excludes rules
	 *
	 * @var array
	 */
	protected $excludes;

	/**
	 * Set the default backup filename.
	 */
	public function __construct() {

		parent::__construct();

		$this->set_backup_filename( implode( '-', array(
			str_ireplace( array( 'http://', 'https://', 'www' ), '', home_url() ),
			'backup',
			current_time( 'Y-m-d-H-i-s' )
		) ) . '.zip' );

		$this->excludes = new Excludes;

It seems like new \HM\BackUpWordPress\Excludes() of type object<HM\BackUpWordPress\Excludes> is incompatible with the declared type array of property $excludes.

	}

	/**
	 * Set the excludes rules for the backup.
	 *
	 * @param array $excludes The exclude rules.

Should the type for parameter $excludes not be Excludes?

	 */
	public function set_excludes( Excludes $excludes ) {
		$this->excludes = $excludes;

It seems like $excludes of type object<HM\BackUpWordPress\Excludes> is incompatible with the declared type array of property $excludes.

	}

	/**
	 * Returns a Finder instance for the files that will be included in the
	 * backup.
	 *
	 * By default we ignore unreadable files and directories as well as, common
	 * version control folders / files, "Dot" files and anything matching the
	 * exclude rules.
	 *
	 * @uses Finder
	 * @return Finder The Finder iterator of all files to be included
	 */
	public function get_files() {

		$finder = new Finder();

		$finder->followLinks( true );

The call to Finder::followLinks() has too many arguments starting with true.

		$finder->ignoreDotFiles( false );
		$finder->ignoreVCS( true );
		$finder->ignoreUnreadableDirs( true );

		// Skip unreadable files too
		$finder->filter(
			function ( \SplFileInfo $file ) {
				if ( ! $file->isReadable() ) {
					return false;
				}
			}
		);

		// Finder expects exclude rules to be in a regex format
		$exclude_rules = $this->excludes->get_excludes_for_regex();

The method get_excludes_for_regex cannot be called on $this->excludes (of type array).

		// Skips folders/files that match default exclude patterns
		foreach ( $exclude_rules as $exclude ) {
			$finder->notPath( $exclude );
		}

		return $finder->in( Path::get_root() );

	}

	/**
	 * Verify that the file backup completed successfully.
	 *
	 * This should be called from backup method of any final file backup engine
	 * implementations.
	 *
	 * @return bool Whether the backup completed successfully.
	 */
	public function verify_backup() {

		// If there are errors delete the backup file.
		if ( $this->get_errors( __CLASS__ ) && file_exists( $this->get_backup_filepath() ) ) {
			unlink( $this->get_backup_filepath() );
		}

		// If the backup doesn't exist then we must have failed.
		if ( ! file_exists( $this->get_backup_filepath() ) ) {

This if statement, and the following return statement can be replaced with return file_exists($this->get_backup_filepath());.

			return false;
		}

		return true;

	}

}