Give_Cron

Complexity

Total Complexity

23

Size/Duplication

Total Lines	242
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	0

Test Coverage

Coverage

83.33%

Importance

Changes

0

14 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	2	1
A	get_instance()	0	8	2
A	setup()	0	9	3
A	__load_async_events()	0	20	3
A	__delete_async_events()	0	10	2
A	__add_schedules()	0	9	1
A	__schedule_events()	0	4	1
A	weekly_events()	0	5	2
A	daily_events()	0	5	2
A	get_cron_action()	0	13	2
A	add_event()	0	4	1
A	add_weekly_event()	0	3	1
A	add_daily_event()	0	3	1
A	add_async_event()	0	12	1

<?php
/**
 * Cron
 *
 * @package     Give
 * @subpackage  Classes/Give_Cron
 * @copyright   Copyright (c) 2016, WordImpress
 * @license     https://opensource.org/licenses/gpl-license GNU Public License
 * @since       1.3.2
 */

// Exit if accessed directly.
if ( ! defined( 'ABSPATH' ) ) {
	exit;
}


/**
 * Give_Cron Class
 *
 * This class handles scheduled events.
 *
 * @since 1.3.2
 */
class Give_Cron {

Since you have declared the constructor as private, maybe you should also declare the class as final.

	/**
	 * Instance.
	 *
	 * @since  1.8.13
	 * @access private
	 * @var
	 */
	private static $instance;

	/**
	 * Singleton pattern.
	 *
	 * @since  1.8.13
	 * @access private
	 */
	private function __construct() {
	}


	/**
	 * Get instance.
	 *
	 * @since  1.8.13
	 * @access public
	 * @return static
	 */
	public static function get_instance() {
		if ( null === static::$instance ) {
			self::$instance = new static();
			self::$instance->setup();
		}

		return self::$instance;
	}


	/**
	 * Setup
	 *
	 * @since 1.8.13
	 */
	private function setup() {
		add_filter( 'cron_schedules', array( self::$instance, '__add_schedules' ) );
		add_action( 'wp', array( self::$instance, '__schedule_events' ) );

		// Load async event only when cron is running.
		if( defined( 'DOING_CRON' ) && DOING_CRON ) {

Space after opening control structure is required

No space before opening parenthesis is prohibited

			add_action( 'init', array( self::$instance, '__load_async_events' ) );
		}
	}


	/**
	 * Load async events
	 *
	 * @since 1.8.13
	 */
	public function __load_async_events() {

Method name "Give_Cron::__load_async_events" is invalid; only PHP magic methods should be prefixed with a double underscore

		$async_events = get_option( 'give_async_events', array() );

		// Bailout.
		if ( empty( $async_events ) ) {
			return;
		}

		foreach ( $async_events as $index => $event ) {
			// Set cron name.
			$cron_name = "give_async_scheduled_events_{$index}";

			// Setup cron.
			wp_schedule_single_event( current_time( 'timestamp', 1 ), $cron_name, $event['params'] );

			// Add cron action.
			add_action( $cron_name, $event['callback'], 10, count( $event['params'] ) );
			add_action( $cron_name, array( $this, '__delete_async_events' ), 9999, 0 );
		}
	}

	/**
	 * Delete async cron info after run
	 *
	 * @since 1.8.13
	 */
	public function __delete_async_events() {

Method name "Give_Cron::__delete_async_events" is invalid; only PHP magic methods should be prefixed with a double underscore

		$async_events    = get_option( 'give_async_events', array() );
		$cron_name_parts = explode( '_', current_action() );
		$cron_id         = end( $cron_name_parts );

		if ( ! empty( $async_events[ $cron_id ] ) ) {
			unset( $async_events[ $cron_id ] );
			update_option( 'give_async_events', $async_events );
		}
	}

	/**
	 * Registers new cron schedules
	 *
	 * @since  1.3.2
	 * @access public
	 *
	 * @param  array $schedules An array of non-default cron schedules.
	 *
	 * @return array            An array of non-default cron schedules.
	 */
	public function __add_schedules( $schedules = array() ) {

Method name "Give_Cron::__add_schedules" is invalid; only PHP magic methods should be prefixed with a double underscore

		// Adds once weekly to the existing schedules.
		$schedules['weekly'] = array(
			'interval' => 604800,
			'display'  => __( 'Once Weekly', 'give' ),
		);

		return $schedules;
	}

	/**
	 * Schedules our events
	 *
	 * @since  1.3.2
	 * @access public
	 *
	 * @return void
	 */
	public function __schedule_events() {

Method name "Give_Cron::__schedule_events" is invalid; only PHP magic methods should be prefixed with a double underscore

		$this->weekly_events();
		$this->daily_events();
	}

	/**
	 * Schedule weekly events
	 *
	 * @since  1.3.2
	 * @access private
	 *
	 * @return void
	 */
	private function weekly_events() {
		if ( ! wp_next_scheduled( 'give_weekly_scheduled_events' ) ) {
			wp_schedule_event( current_time( 'timestamp' ), 'weekly', 'give_weekly_scheduled_events' );
		}
	}

	/**
	 * Schedule daily events
	 *
	 * @since  1.3.2
	 * @access private
	 *
	 * @return void
	 */
	private function daily_events() {
		if ( ! wp_next_scheduled( 'give_daily_scheduled_events' ) ) {
			wp_schedule_event( current_time( 'timestamp' ), 'daily', 'give_daily_scheduled_events' );
		}
	}

	/**
	 * get cron job action name
	 *
	 * @since  1.8.13
	 * @access public
	 *
	 * @param string $type
	 *
	 * @return string
	 */
	public static function get_cron_action( $type = 'weekly' ) {
		switch ( $type ) {
			case 'daily':
				$cron_action = 'give_daily_scheduled_events';
				break;

			default:
				$cron_action = 'give_weekly_scheduled_events';
				break;
		}

		return $cron_action;
	}

	/**
	 * Add action to cron action
	 *
	 * @since  1.8.13
	 * @access private
	 *
	 * @param        $action
	 * @param string $type
	 */
	private static function add_event( $action, $type = 'weekly' ) {
		$cron_event = self::get_cron_action( $type );
		add_action( $cron_event, $action );
	}

	/**
	 * Add weekly event
	 *
	 * @since  1.8.13
	 * @access public
	 *
	 * @param $action
	 */
	public static function add_weekly_event( $action ) {
		self::add_event( $action, 'weekly' );
	}

	/**
	 * Add daily event
	 *
	 * @since  1.8.13
	 * @access public
	 *
	 * @param $action
	 */
	public static function add_daily_event( $action ) {
		self::add_event( $action, 'daily' );
	}

	/**
	 * Add async event
	 * Note: it is good for small jobs if you have bigger task to do then either test it or manage with custom cron job.
	 *
	 * @since  1.8.13
	 * @access public
	 *
	 * @param string $action
	 * @param array  $args
	 */
	public static function add_async_event( $action, $args = array() ) {

		// Cache async events.
		$async_events             = get_option( 'give_async_events', array() );
		$async_events[ uniqid() ] = array(
			'callback' => $action,
			'params'   => $args,
		);

Functions must not contain multiple empty lines in a row; found 2 empty lines

		update_option( 'give_async_events', $async_events );
	}
}

// Initiate class.
Give_Cron::get_instance();