Schedules::__construct() - Code Metrics - humanmade/backupwordpress - Measure and Improve Code Quality continuously with Scrutinizer

Schedules::__construct() A
last analyzed 2016-11-04 16:23 UTC

↳ Parent: Schedules

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace HM\BackUpWordPress;

/**
 * A simple class for loading schedules
 */
class Schedules {

	/**
	 * An array of schedules
	 *
	 * @var mixed
	 * @access private
	 */
	private $schedules;

	/**
	 *
	 */
	protected static $instance;

	public static function get_instance() {

		if ( ! ( self::$instance instanceof Schedules ) ) {
			self::$instance = new Schedules();
		}

		return self::$instance;

	}

	/**
	 * Load the schedules from wp_options and store in $this->schedules
	 *
	 */
	private function __construct() {
		$this->refresh_schedules();
	}

	public function refresh_schedules() {

		$schedules = get_transient( 'hmbkp_schedules' );

		if ( ! $schedules ) {

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

			// Load all schedule options from the database.
			$schedules = $wpdb->get_col( "SELECT option_name from $wpdb->options WHERE option_name LIKE 'hmbkp\_schedule\_%'" );

			set_transient( 'hmbkp_schedules', $schedules, WEEK_IN_SECONDS );
		}

		// Instantiate each one as a Scheduled_Backup
		$this->schedules = array_map( array( $this, 'instantiate' ), array_filter( (array) $schedules ) );

	}

	/**
	 * Get an array of schedules
	 *
	 * @return Scheduled_Backup[]
	 */
	public function get_schedules() {
		return $this->schedules;
	}

	/**
	 * Get a schedule by ID
	 *
	 * @param $id
	 * @return Scheduled_Backup
	 */
	public function get_schedule( $id ) {

		foreach ( $this->schedules as $schedule ) {
			if ( $schedule->get_id() == $id ) {
				return $schedule;
			}
		}

		return null;
	}

	/**
	 * Instantiate the individual scheduled backup objects
	 *
	 * @access private
	 * @param string $id
	 * @return Scheduled_Backup
	 */
	private function instantiate( $id ) {
		return new Scheduled_Backup( str_replace( 'hmbkp_schedule_', '', $id ) );
	}
}


1			<?php
2
3			namespace HM\BackUpWordPress;
4
5			/**
6			* A simple class for loading schedules
7			*/
8			class Schedules {
9
10			/**
11			* An array of schedules
12			*
13			* @var mixed
14			* @access private
15			*/
16			private $schedules;
17
18			/**
19			*
20			*/
21			protected static $instance;
22
23			public static function get_instance() {
24
25			if ( ! ( self::$instance instanceof Schedules ) ) {
26			self::$instance = new Schedules();
27			}
28
29			return self::$instance;
30
31			}
32
33			/**
34			* Load the schedules from wp_options and store in $this->schedules
35			*
36			*/
37			private function __construct() {
38			$this->refresh_schedules();
39			}
40
41			public function refresh_schedules() {
42
43			$schedules = get_transient( 'hmbkp_schedules' );
44
45			if ( ! $schedules ) {
46
47			global $wpdb;
			0 ignored issues – show Compatibility Best Practice introduced 2015-11-21 01:52 UTC by Report Bug Copy Issue Report 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...
48
49			// Load all schedule options from the database.
50			$schedules = $wpdb->get_col( "SELECT option_name from $wpdb->options WHERE option_name LIKE 'hmbkp\_schedule\_%'" );
51
52			set_transient( 'hmbkp_schedules', $schedules, WEEK_IN_SECONDS );
53			}
54
55			// Instantiate each one as a Scheduled_Backup
56			$this->schedules = array_map( array( $this, 'instantiate' ), array_filter( (array) $schedules ) );
57
58			}
59
60			/**
61			* Get an array of schedules
62			*
63			* @return Scheduled_Backup[]
64			*/
65			public function get_schedules() {
66			return $this->schedules;
67			}
68
69			/**
70			* Get a schedule by ID
71			*
72			* @param $id
73			* @return Scheduled_Backup
74			*/
75			public function get_schedule( $id ) {
76
77			foreach ( $this->schedules as $schedule ) {
78			if ( $schedule->get_id() == $id ) {
79			return $schedule;
80			}
81			}
82
83			return null;
84			}
85
86			/**
87			* Instantiate the individual scheduled backup objects
88			*
89			* @access private
90			* @param string $id
91			* @return Scheduled_Backup
92			*/
93			private function instantiate( $id ) {
94			return new Scheduled_Backup( str_replace( 'hmbkp_schedule_', '', $id ) );
95			}
96			}
97

humanmade / backupwordpress

GitHub Access Token became invalid

Schedules::__construct() A last analyzed 2016-11-04 16:23 UTC

Complexity

Size

Duplication

Importance

1. Pass all data via parameters

2. Create a class that maintains your state

Duplication Side-by-Side

Filter issues like

Schedules::__construct() A
last analyzed 2016-11-04 16:23 UTC