AyeCode / invoicing

includes/libraries/action-scheduler/action-scheduler.php

Indentation +19 added lines, -19 removed lines

@@ -27,27 +27,27 @@
 
 if ( ! function_exists( 'action_scheduler_register_3_dot_1_dot_4' ) ) {
 
-	if ( ! class_exists( 'ActionScheduler_Versions' ) ) {
-		require_once( 'classes/ActionScheduler_Versions.php' );
-		add_action( 'plugins_loaded', array( 'ActionScheduler_Versions', 'initialize_latest_version' ), 1, 0 );
-	}
+    if ( ! class_exists( 'ActionScheduler_Versions' ) ) {
+        require_once( 'classes/ActionScheduler_Versions.php' );
+        add_action( 'plugins_loaded', array( 'ActionScheduler_Versions', 'initialize_latest_version' ), 1, 0 );
+    }
 
-	add_action( 'plugins_loaded', 'action_scheduler_register_3_dot_1_dot_4', 0, 0 );
+    add_action( 'plugins_loaded', 'action_scheduler_register_3_dot_1_dot_4', 0, 0 );
 
-	function action_scheduler_register_3_dot_1_dot_4() {
-		$versions = ActionScheduler_Versions::instance();
-		$versions->register( '3.1.4', 'action_scheduler_initialize_3_dot_1_dot_4' );
-	}
+    function action_scheduler_register_3_dot_1_dot_4() {
+        $versions = ActionScheduler_Versions::instance();
+        $versions->register( '3.1.4', 'action_scheduler_initialize_3_dot_1_dot_4' );
+    }
 
-	function action_scheduler_initialize_3_dot_1_dot_4() {
-		require_once( 'classes/abstracts/ActionScheduler.php' );
-		ActionScheduler::init( __FILE__ );
-	}
+    function action_scheduler_initialize_3_dot_1_dot_4() {
+        require_once( 'classes/abstracts/ActionScheduler.php' );
+        ActionScheduler::init( __FILE__ );
+    }
 
-	// Support usage in themes - load this version if no plugin has loaded a version yet.
-	if ( did_action( 'plugins_loaded' ) && ! class_exists( 'ActionScheduler' ) ) {
-		action_scheduler_initialize_3_dot_1_dot_4();
-		do_action( 'action_scheduler_pre_theme_init' );
-		ActionScheduler_Versions::initialize_latest_version();
-	}
+    // Support usage in themes - load this version if no plugin has loaded a version yet.
+    if ( did_action( 'plugins_loaded' ) && ! class_exists( 'ActionScheduler' ) ) {
+        action_scheduler_initialize_3_dot_1_dot_4();
+        do_action( 'action_scheduler_pre_theme_init' );
+        ActionScheduler_Versions::initialize_latest_version();
+    }
 }

Spacing +10 added lines, -10 removed lines

@@ -25,29 +25,29 @@
  *
  */
 
-if ( ! function_exists( 'action_scheduler_register_3_dot_1_dot_4' ) ) {
+if (!function_exists('action_scheduler_register_3_dot_1_dot_4')) {
 
-	if ( ! class_exists( 'ActionScheduler_Versions' ) ) {
-		require_once( 'classes/ActionScheduler_Versions.php' );
-		add_action( 'plugins_loaded', array( 'ActionScheduler_Versions', 'initialize_latest_version' ), 1, 0 );
+	if (!class_exists('ActionScheduler_Versions')) {
+		require_once('classes/ActionScheduler_Versions.php');
+		add_action('plugins_loaded', array('ActionScheduler_Versions', 'initialize_latest_version'), 1, 0);
 	}
 
-	add_action( 'plugins_loaded', 'action_scheduler_register_3_dot_1_dot_4', 0, 0 );
+	add_action('plugins_loaded', 'action_scheduler_register_3_dot_1_dot_4', 0, 0);
 
 	function action_scheduler_register_3_dot_1_dot_4() {
 		$versions = ActionScheduler_Versions::instance();
-		$versions->register( '3.1.4', 'action_scheduler_initialize_3_dot_1_dot_4' );
+		$versions->register('3.1.4', 'action_scheduler_initialize_3_dot_1_dot_4');
 	}
 
 	function action_scheduler_initialize_3_dot_1_dot_4() {
-		require_once( 'classes/abstracts/ActionScheduler.php' );
-		ActionScheduler::init( __FILE__ );
+		require_once('classes/abstracts/ActionScheduler.php');
+		ActionScheduler::init(__FILE__);
 	}
 
 	// Support usage in themes - load this version if no plugin has loaded a version yet.
-	if ( did_action( 'plugins_loaded' ) && ! class_exists( 'ActionScheduler' ) ) {
+	if (did_action('plugins_loaded') && !class_exists('ActionScheduler')) {
 		action_scheduler_initialize_3_dot_1_dot_4();
-		do_action( 'action_scheduler_pre_theme_init' );
+		do_action('action_scheduler_pre_theme_init');
 		ActionScheduler_Versions::initialize_latest_version();
 	}
 }

includes/libraries/action-scheduler/classes/ActionScheduler_FatalErrorMonitor.php

Indentation +42 added lines, -42 removed lines

@@ -4,52 +4,52 @@
  * Class ActionScheduler_FatalErrorMonitor
  */
 class ActionScheduler_FatalErrorMonitor {
-	/** @var ActionScheduler_ActionClaim */
-	private $claim = NULL;
-	/** @var ActionScheduler_Store */
-	private $store = NULL;
-	private $action_id = 0;
+    /** @var ActionScheduler_ActionClaim */
+    private $claim = NULL;
+    /** @var ActionScheduler_Store */
+    private $store = NULL;
+    private $action_id = 0;
 
-	public function __construct( ActionScheduler_Store $store ) {
-		$this->store = $store;
-	}
+    public function __construct( ActionScheduler_Store $store ) {
+        $this->store = $store;
+    }
 
-	public function attach( ActionScheduler_ActionClaim $claim ) {
-		$this->claim = $claim;
-		add_action( 'shutdown', array( $this, 'handle_unexpected_shutdown' ) );
-		add_action( 'action_scheduler_before_execute', array( $this, 'track_current_action' ), 0, 1 );
-		add_action( 'action_scheduler_after_execute',  array( $this, 'untrack_action' ), 0, 0 );
-		add_action( 'action_scheduler_execution_ignored',  array( $this, 'untrack_action' ), 0, 0 );
-		add_action( 'action_scheduler_failed_execution',  array( $this, 'untrack_action' ), 0, 0 );
-	}
+    public function attach( ActionScheduler_ActionClaim $claim ) {
+        $this->claim = $claim;
+        add_action( 'shutdown', array( $this, 'handle_unexpected_shutdown' ) );
+        add_action( 'action_scheduler_before_execute', array( $this, 'track_current_action' ), 0, 1 );
+        add_action( 'action_scheduler_after_execute',  array( $this, 'untrack_action' ), 0, 0 );
+        add_action( 'action_scheduler_execution_ignored',  array( $this, 'untrack_action' ), 0, 0 );
+        add_action( 'action_scheduler_failed_execution',  array( $this, 'untrack_action' ), 0, 0 );
+    }
 
-	public function detach() {
-		$this->claim = NULL;
-		$this->untrack_action();
-		remove_action( 'shutdown', array( $this, 'handle_unexpected_shutdown' ) );
-		remove_action( 'action_scheduler_before_execute', array( $this, 'track_current_action' ), 0 );
-		remove_action( 'action_scheduler_after_execute',  array( $this, 'untrack_action' ), 0 );
-		remove_action( 'action_scheduler_execution_ignored',  array( $this, 'untrack_action' ), 0 );
-		remove_action( 'action_scheduler_failed_execution',  array( $this, 'untrack_action' ), 0 );
-	}
+    public function detach() {
+        $this->claim = NULL;
+        $this->untrack_action();
+        remove_action( 'shutdown', array( $this, 'handle_unexpected_shutdown' ) );
+        remove_action( 'action_scheduler_before_execute', array( $this, 'track_current_action' ), 0 );
+        remove_action( 'action_scheduler_after_execute',  array( $this, 'untrack_action' ), 0 );
+        remove_action( 'action_scheduler_execution_ignored',  array( $this, 'untrack_action' ), 0 );
+        remove_action( 'action_scheduler_failed_execution',  array( $this, 'untrack_action' ), 0 );
+    }
 
-	public function track_current_action( $action_id ) {
-		$this->action_id = $action_id;
-	}
+    public function track_current_action( $action_id ) {
+        $this->action_id = $action_id;
+    }
 
-	public function untrack_action() {
-		$this->action_id = 0;
-	}
+    public function untrack_action() {
+        $this->action_id = 0;
+    }
 
-	public function handle_unexpected_shutdown() {
-		if ( $error = error_get_last() ) {
-			if ( in_array( $error['type'], array( E_ERROR, E_PARSE, E_COMPILE_ERROR, E_USER_ERROR, E_RECOVERABLE_ERROR ) ) ) {
-				if ( !empty($this->action_id) ) {
-					$this->store->mark_failure( $this->action_id );
-					do_action( 'action_scheduler_unexpected_shutdown', $this->action_id, $error );
-				}
-			}
-			$this->store->release_claim( $this->claim );
-		}
-	}
+    public function handle_unexpected_shutdown() {
+        if ( $error = error_get_last() ) {
+            if ( in_array( $error['type'], array( E_ERROR, E_PARSE, E_COMPILE_ERROR, E_USER_ERROR, E_RECOVERABLE_ERROR ) ) ) {
+                if ( !empty($this->action_id) ) {
+                    $this->store->mark_failure( $this->action_id );
+                    do_action( 'action_scheduler_unexpected_shutdown', $this->action_id, $error );
+                }
+            }
+            $this->store->release_claim( $this->claim );
+        }
+    }
 }

Spacing +19 added lines, -19 removed lines

@@ -10,30 +10,30 @@ 
 	private $store = NULL;
 	private $action_id = 0;
 
-	public function __construct( ActionScheduler_Store $store ) {
+	public function __construct(ActionScheduler_Store $store) {
 		$this->store = $store;
 	}
 
-	public function attach( ActionScheduler_ActionClaim $claim ) {
+	public function attach(ActionScheduler_ActionClaim $claim) {
 		$this->claim = $claim;
-		add_action( 'shutdown', array( $this, 'handle_unexpected_shutdown' ) );
-		add_action( 'action_scheduler_before_execute', array( $this, 'track_current_action' ), 0, 1 );
-		add_action( 'action_scheduler_after_execute',  array( $this, 'untrack_action' ), 0, 0 );
-		add_action( 'action_scheduler_execution_ignored',  array( $this, 'untrack_action' ), 0, 0 );
-		add_action( 'action_scheduler_failed_execution',  array( $this, 'untrack_action' ), 0, 0 );
+		add_action('shutdown', array($this, 'handle_unexpected_shutdown'));
+		add_action('action_scheduler_before_execute', array($this, 'track_current_action'), 0, 1);
+		add_action('action_scheduler_after_execute', array($this, 'untrack_action'), 0, 0);
+		add_action('action_scheduler_execution_ignored', array($this, 'untrack_action'), 0, 0);
+		add_action('action_scheduler_failed_execution', array($this, 'untrack_action'), 0, 0);
 	}
 
 	public function detach() {
 		$this->claim = NULL;
 		$this->untrack_action();
-		remove_action( 'shutdown', array( $this, 'handle_unexpected_shutdown' ) );
-		remove_action( 'action_scheduler_before_execute', array( $this, 'track_current_action' ), 0 );
-		remove_action( 'action_scheduler_after_execute',  array( $this, 'untrack_action' ), 0 );
-		remove_action( 'action_scheduler_execution_ignored',  array( $this, 'untrack_action' ), 0 );
-		remove_action( 'action_scheduler_failed_execution',  array( $this, 'untrack_action' ), 0 );
+		remove_action('shutdown', array($this, 'handle_unexpected_shutdown'));
+		remove_action('action_scheduler_before_execute', array($this, 'track_current_action'), 0);
+		remove_action('action_scheduler_after_execute', array($this, 'untrack_action'), 0);
+		remove_action('action_scheduler_execution_ignored', array($this, 'untrack_action'), 0);
+		remove_action('action_scheduler_failed_execution', array($this, 'untrack_action'), 0);
 	}
 
-	public function track_current_action( $action_id ) {
+	public function track_current_action($action_id) {
 		$this->action_id = $action_id;
 	}
 
@@ -42,14 +42,14 @@ 
 	}
 
 	public function handle_unexpected_shutdown() {
-		if ( $error = error_get_last() ) {
-			if ( in_array( $error['type'], array( E_ERROR, E_PARSE, E_COMPILE_ERROR, E_USER_ERROR, E_RECOVERABLE_ERROR ) ) ) {
-				if ( !empty($this->action_id) ) {
-					$this->store->mark_failure( $this->action_id );
-					do_action( 'action_scheduler_unexpected_shutdown', $this->action_id, $error );
+		if ($error = error_get_last()) {
+			if (in_array($error['type'], array(E_ERROR, E_PARSE, E_COMPILE_ERROR, E_USER_ERROR, E_RECOVERABLE_ERROR))) {
+				if (!empty($this->action_id)) {
+					$this->store->mark_failure($this->action_id);
+					do_action('action_scheduler_unexpected_shutdown', $this->action_id, $error);
 				}
 			}
-			$this->store->release_claim( $this->claim );
+			$this->store->release_claim($this->claim);
 		}
 	}
 }

includes/libraries/action-scheduler/classes/ActionScheduler_wcSystemStatus.php

Indentation +120 added lines, -120 removed lines

@@ -5,95 +5,95 @@ 
  */
 class ActionScheduler_wcSystemStatus {
 
-	/**
-	 * The active data stores
-	 *
-	 * @var ActionScheduler_Store
-	 */
-	protected $store;
-
-	function __construct( $store ) {
-		$this->store = $store;
-	}
-
-	/**
-	 * Display action data, including number of actions grouped by status and the oldest & newest action in each status.
-	 *
-	 * Helpful to identify issues, like a clogged queue.
-	 */
-	public function render() {
-		$action_counts     = $this->store->action_counts();
-		$status_labels     = $this->store->get_status_labels();
-		$oldest_and_newest = $this->get_oldest_and_newest( array_keys( $status_labels ) );
-
-		$this->get_template( $status_labels, $action_counts, $oldest_and_newest );
-	}
-
-	/**
-	 * Get oldest and newest scheduled dates for a given set of statuses.
-	 *
-	 * @param array $status_keys Set of statuses to find oldest & newest action for.
-	 * @return array
-	 */
-	protected function get_oldest_and_newest( $status_keys ) {
-
-		$oldest_and_newest = array();
-
-		foreach ( $status_keys as $status ) {
-			$oldest_and_newest[ $status ] = array(
-				'oldest' => '–',
-				'newest' => '–',
-			);
-
-			if ( 'in-progress' === $status ) {
-				continue;
-			}
-
-			$oldest_and_newest[ $status ]['oldest'] = $this->get_action_status_date( $status, 'oldest' );
-			$oldest_and_newest[ $status ]['newest'] = $this->get_action_status_date( $status, 'newest' );
-		}
-
-		return $oldest_and_newest;
-	}
-
-	/**
-	 * Get oldest or newest scheduled date for a given status.
-	 *
-	 * @param string $status Action status label/name string.
-	 * @param string $date_type Oldest or Newest.
-	 * @return DateTime
-	 */
-	protected function get_action_status_date( $status, $date_type = 'oldest' ) {
-
-		$order = 'oldest' === $date_type ? 'ASC' : 'DESC';
-
-		$action = $this->store->query_actions( array(
-			'claimed'  => false,
-			'status'   => $status,
-			'per_page' => 1,
-			'order'    => $order,
-		) );
-
-		if ( ! empty( $action ) ) {
-			$date_object = $this->store->get_date( $action[0] );
-			$action_date = $date_object->format( 'Y-m-d H:i:s O' );
-		} else {
-			$action_date = '–';
-		}
-
-		return $action_date;
-	}
-
-	/**
-	 * Get oldest or newest scheduled date for a given status.
-	 *
-	 * @param array $status_labels Set of statuses to find oldest & newest action for.
-	 * @param array $action_counts Number of actions grouped by status.
-	 * @param array $oldest_and_newest Date of the oldest and newest action with each status.
-	 */
-	protected function get_template( $status_labels, $action_counts, $oldest_and_newest ) {
-		$as_version = ActionScheduler_Versions::instance()->latest_version();
-		?>
+    /**
+     * The active data stores
+     *
+     * @var ActionScheduler_Store
+     */
+    protected $store;
+
+    function __construct( $store ) {
+        $this->store = $store;
+    }
+
+    /**
+     * Display action data, including number of actions grouped by status and the oldest & newest action in each status.
+     *
+     * Helpful to identify issues, like a clogged queue.
+     */
+    public function render() {
+        $action_counts     = $this->store->action_counts();
+        $status_labels     = $this->store->get_status_labels();
+        $oldest_and_newest = $this->get_oldest_and_newest( array_keys( $status_labels ) );
+
+        $this->get_template( $status_labels, $action_counts, $oldest_and_newest );
+    }
+
+    /**
+     * Get oldest and newest scheduled dates for a given set of statuses.
+     *
+     * @param array $status_keys Set of statuses to find oldest & newest action for.
+     * @return array
+     */
+    protected function get_oldest_and_newest( $status_keys ) {
+
+        $oldest_and_newest = array();
+
+        foreach ( $status_keys as $status ) {
+            $oldest_and_newest[ $status ] = array(
+                'oldest' => '–',
+                'newest' => '–',
+            );
+
+            if ( 'in-progress' === $status ) {
+                continue;
+            }
+
+            $oldest_and_newest[ $status ]['oldest'] = $this->get_action_status_date( $status, 'oldest' );
+            $oldest_and_newest[ $status ]['newest'] = $this->get_action_status_date( $status, 'newest' );
+        }
+
+        return $oldest_and_newest;
+    }
+
+    /**
+     * Get oldest or newest scheduled date for a given status.
+     *
+     * @param string $status Action status label/name string.
+     * @param string $date_type Oldest or Newest.
+     * @return DateTime
+     */
+    protected function get_action_status_date( $status, $date_type = 'oldest' ) {
+
+        $order = 'oldest' === $date_type ? 'ASC' : 'DESC';
+
+        $action = $this->store->query_actions( array(
+            'claimed'  => false,
+            'status'   => $status,
+            'per_page' => 1,
+            'order'    => $order,
+        ) );
+
+        if ( ! empty( $action ) ) {
+            $date_object = $this->store->get_date( $action[0] );
+            $action_date = $date_object->format( 'Y-m-d H:i:s O' );
+        } else {
+            $action_date = '–';
+        }
+
+        return $action_date;
+    }
+
+    /**
+     * Get oldest or newest scheduled date for a given status.
+     *
+     * @param array $status_labels Set of statuses to find oldest & newest action for.
+     * @param array $action_counts Number of actions grouped by status.
+     * @param array $oldest_and_newest Date of the oldest and newest action with each status.
+     */
+    protected function get_template( $status_labels, $action_counts, $oldest_and_newest ) {
+        $as_version = ActionScheduler_Versions::instance()->latest_version();
+        ?>
 
 		<table class="wc_status_table widefat" cellspacing="0">
 			<thead>
@@ -114,39 +114,39 @@ 
 			</thead>
 			<tbody>
 				<?php
-				foreach ( $action_counts as $status => $count ) {
-					// WC uses the 3rd column for export, so we need to display more data in that (hidden when viewed as part of the table) and add an empty 2nd column.
-					printf(
-						'<tr><td>%1$s</td><td>&nbsp;</td><td>%2$s<span style="display: none;">, Oldest: %3$s, Newest: %4$s</span></td><td>%3$s</td><td>%4$s</td></tr>',
-						esc_html( $status_labels[ $status ] ),
-						number_format_i18n( $count ),
-						$oldest_and_newest[ $status ]['oldest'],
-						$oldest_and_newest[ $status ]['newest']
-					);
-				}
-				?>
+                foreach ( $action_counts as $status => $count ) {
+                    // WC uses the 3rd column for export, so we need to display more data in that (hidden when viewed as part of the table) and add an empty 2nd column.
+                    printf(
+                        '<tr><td>%1$s</td><td>&nbsp;</td><td>%2$s<span style="display: none;">, Oldest: %3$s, Newest: %4$s</span></td><td>%3$s</td><td>%4$s</td></tr>',
+                        esc_html( $status_labels[ $status ] ),
+                        number_format_i18n( $count ),
+                        $oldest_and_newest[ $status ]['oldest'],
+                        $oldest_and_newest[ $status ]['newest']
+                    );
+                }
+                ?>
 			</tbody>
 		</table>
 
 		<?php
-	}
-
-	/**
-	 * is triggered when invoking inaccessible methods in an object context.
-	 *
-	 * @param string $name
-	 * @param array  $arguments
-	 *
-	 * @return mixed
-	 * @link https://php.net/manual/en/language.oop5.overloading.php#language.oop5.overloading.methods
-	 */
-	public function __call( $name, $arguments ) {
-		switch ( $name ) {
-			case 'print':
-				_deprecated_function( __CLASS__ . '::print()', '2.2.4', __CLASS__ . '::render()' );
-				return call_user_func_array( array( $this, 'render' ), $arguments );
-		}
-
-		return null;
-	}
+    }
+
+    /**
+     * is triggered when invoking inaccessible methods in an object context.
+     *
+     * @param string $name
+     * @param array  $arguments
+     *
+     * @return mixed
+     * @link https://php.net/manual/en/language.oop5.overloading.php#language.oop5.overloading.methods
+     */
+    public function __call( $name, $arguments ) {
+        switch ( $name ) {
+            case 'print':
+                _deprecated_function( __CLASS__ . '::print()', '2.2.4', __CLASS__ . '::render()' );
+                return call_user_func_array( array( $this, 'render' ), $arguments );
+        }
+
+        return null;
+    }
 }

Spacing +32 added lines, -32 removed lines

@@ -12,7 +12,7 @@ 
 	 */
 	protected $store;
 
-	function __construct( $store ) {
+	function __construct($store) {
 		$this->store = $store;
 	}
 
@@ -24,9 +24,9 @@ 
 	public function render() {
 		$action_counts     = $this->store->action_counts();
 		$status_labels     = $this->store->get_status_labels();
-		$oldest_and_newest = $this->get_oldest_and_newest( array_keys( $status_labels ) );
+		$oldest_and_newest = $this->get_oldest_and_newest(array_keys($status_labels));
 
-		$this->get_template( $status_labels, $action_counts, $oldest_and_newest );
+		$this->get_template($status_labels, $action_counts, $oldest_and_newest);
 	}
 
 	/**
@@ -35,22 +35,22 @@ 
 	 * @param array $status_keys Set of statuses to find oldest & newest action for.
 	 * @return array
 	 */
-	protected function get_oldest_and_newest( $status_keys ) {
+	protected function get_oldest_and_newest($status_keys) {
 
 		$oldest_and_newest = array();
 
-		foreach ( $status_keys as $status ) {
-			$oldest_and_newest[ $status ] = array(
+		foreach ($status_keys as $status) {
+			$oldest_and_newest[$status] = array(
 				'oldest' => '–',
 				'newest' => '–',
 			);
 
-			if ( 'in-progress' === $status ) {
+			if ('in-progress' === $status) {
 				continue;
 			}
 
-			$oldest_and_newest[ $status ]['oldest'] = $this->get_action_status_date( $status, 'oldest' );
-			$oldest_and_newest[ $status ]['newest'] = $this->get_action_status_date( $status, 'newest' );
+			$oldest_and_newest[$status]['oldest'] = $this->get_action_status_date($status, 'oldest');
+			$oldest_and_newest[$status]['newest'] = $this->get_action_status_date($status, 'newest');
 		}
 
 		return $oldest_and_newest;
@@ -63,20 +63,20 @@ 
 	 * @param string $date_type Oldest or Newest.
 	 * @return DateTime
 	 */
-	protected function get_action_status_date( $status, $date_type = 'oldest' ) {
+	protected function get_action_status_date($status, $date_type = 'oldest') {
 
 		$order = 'oldest' === $date_type ? 'ASC' : 'DESC';
 
-		$action = $this->store->query_actions( array(
+		$action = $this->store->query_actions(array(
 			'claimed'  => false,
 			'status'   => $status,
 			'per_page' => 1,
 			'order'    => $order,
-		) );
+		));
 
-		if ( ! empty( $action ) ) {
-			$date_object = $this->store->get_date( $action[0] );
-			$action_date = $date_object->format( 'Y-m-d H:i:s O' );
+		if (!empty($action)) {
+			$date_object = $this->store->get_date($action[0]);
+			$action_date = $date_object->format('Y-m-d H:i:s O');
 		} else {
 			$action_date = '–';
 		}
@@ -91,37 +91,37 @@ 
 	 * @param array $action_counts Number of actions grouped by status.
 	 * @param array $oldest_and_newest Date of the oldest and newest action with each status.
 	 */
-	protected function get_template( $status_labels, $action_counts, $oldest_and_newest ) {
+	protected function get_template($status_labels, $action_counts, $oldest_and_newest) {
 		$as_version = ActionScheduler_Versions::instance()->latest_version();
 		?>
 
 		<table class="wc_status_table widefat" cellspacing="0">
 			<thead>
 				<tr>
-					<th colspan="5" data-export-label="Action Scheduler"><h2><?php esc_html_e( 'Action Scheduler', 'action-scheduler' ); ?><?php echo wc_help_tip( esc_html__( 'This section shows scheduled action counts.', 'action-scheduler' ) ); ?></h2></th>
+					<th colspan="5" data-export-label="Action Scheduler"><h2><?php esc_html_e('Action Scheduler', 'action-scheduler'); ?><?php echo wc_help_tip(esc_html__('This section shows scheduled action counts.', 'action-scheduler')); ?></h2></th>
 				</tr>
 				<tr>
-					<td colspan="2" data-export-label="Version"><?php esc_html_e( 'Version:', 'action-scheduler' ); ?></td>
-					<td colspan="3"><?php echo esc_html( $as_version ); ?></td>
+					<td colspan="2" data-export-label="Version"><?php esc_html_e('Version:', 'action-scheduler'); ?></td>
+					<td colspan="3"><?php echo esc_html($as_version); ?></td>
 				</tr>
 				<tr>
-					<td><strong><?php esc_html_e( 'Action Status', 'action-scheduler' ); ?></strong></td>
+					<td><strong><?php esc_html_e('Action Status', 'action-scheduler'); ?></strong></td>
 					<td class="help">&nbsp;</td>
-					<td><strong><?php esc_html_e( 'Count', 'action-scheduler' ); ?></strong></td>
-					<td><strong><?php esc_html_e( 'Oldest Scheduled Date', 'action-scheduler' ); ?></strong></td>
-					<td><strong><?php esc_html_e( 'Newest Scheduled Date', 'action-scheduler' ); ?></strong></td>
+					<td><strong><?php esc_html_e('Count', 'action-scheduler'); ?></strong></td>
+					<td><strong><?php esc_html_e('Oldest Scheduled Date', 'action-scheduler'); ?></strong></td>
+					<td><strong><?php esc_html_e('Newest Scheduled Date', 'action-scheduler'); ?></strong></td>
 				</tr>
 			</thead>
 			<tbody>
 				<?php
-				foreach ( $action_counts as $status => $count ) {
+				foreach ($action_counts as $status => $count) {
 					// WC uses the 3rd column for export, so we need to display more data in that (hidden when viewed as part of the table) and add an empty 2nd column.
 					printf(
 						'<tr><td>%1$s</td><td>&nbsp;</td><td>%2$s<span style="display: none;">, Oldest: %3$s, Newest: %4$s</span></td><td>%3$s</td><td>%4$s</td></tr>',
-						esc_html( $status_labels[ $status ] ),
-						number_format_i18n( $count ),
-						$oldest_and_newest[ $status ]['oldest'],
-						$oldest_and_newest[ $status ]['newest']
+						esc_html($status_labels[$status]),
+						number_format_i18n($count),
+						$oldest_and_newest[$status]['oldest'],
+						$oldest_and_newest[$status]['newest']
 					);
 				}
 				?>
@@ -140,11 +140,11 @@ 
 	 * @return mixed
 	 * @link https://php.net/manual/en/language.oop5.overloading.php#language.oop5.overloading.methods
 	 */
-	public function __call( $name, $arguments ) {
-		switch ( $name ) {
+	public function __call($name, $arguments) {
+		switch ($name) {
 			case 'print':
-				_deprecated_function( __CLASS__ . '::print()', '2.2.4', __CLASS__ . '::render()' );
-				return call_user_func_array( array( $this, 'render' ), $arguments );
+				_deprecated_function(__CLASS__ . '::print()', '2.2.4', __CLASS__ . '::render()');
+				return call_user_func_array(array($this, 'render'), $arguments);
 		}
 
 		return null;

includes/libraries/action-scheduler/classes/data-stores/ActionScheduler_wpPostStore_PostStatusRegistrar.php

Indentation +44 added lines, -44 removed lines

@@ -5,54 +5,54 @@
  * @codeCoverageIgnore
  */
 class ActionScheduler_wpPostStore_PostStatusRegistrar {
-	public function register() {
-		register_post_status( ActionScheduler_Store::STATUS_RUNNING, array_merge( $this->post_status_args(), $this->post_status_running_labels() ) );
-		register_post_status( ActionScheduler_Store::STATUS_FAILED, array_merge( $this->post_status_args(), $this->post_status_failed_labels() ) );
-	}
+    public function register() {
+        register_post_status( ActionScheduler_Store::STATUS_RUNNING, array_merge( $this->post_status_args(), $this->post_status_running_labels() ) );
+        register_post_status( ActionScheduler_Store::STATUS_FAILED, array_merge( $this->post_status_args(), $this->post_status_failed_labels() ) );
+    }
 
-	/**
-	 * Build the args array for the post type definition
-	 *
-	 * @return array
-	 */
-	protected function post_status_args() {
-		$args = array(
-			'public'                    => false,
-			'exclude_from_search'       => false,
-			'show_in_admin_all_list'    => true,
-			'show_in_admin_status_list' => true,
-		);
+    /**
+     * Build the args array for the post type definition
+     *
+     * @return array
+     */
+    protected function post_status_args() {
+        $args = array(
+            'public'                    => false,
+            'exclude_from_search'       => false,
+            'show_in_admin_all_list'    => true,
+            'show_in_admin_status_list' => true,
+        );
 
-		return apply_filters( 'action_scheduler_post_status_args', $args );
-	}
+        return apply_filters( 'action_scheduler_post_status_args', $args );
+    }
 
-	/**
-	 * Build the args array for the post type definition
-	 *
-	 * @return array
-	 */
-	protected function post_status_failed_labels() {
-		$labels = array(
-			'label'       => _x( 'Failed', 'post', 'action-scheduler' ),
-			/* translators: %s: count */
-			'label_count' => _n_noop( 'Failed <span class="count">(%s)</span>', 'Failed <span class="count">(%s)</span>', 'action-scheduler' ),
-		);
+    /**
+     * Build the args array for the post type definition
+     *
+     * @return array
+     */
+    protected function post_status_failed_labels() {
+        $labels = array(
+            'label'       => _x( 'Failed', 'post', 'action-scheduler' ),
+            /* translators: %s: count */
+            'label_count' => _n_noop( 'Failed <span class="count">(%s)</span>', 'Failed <span class="count">(%s)</span>', 'action-scheduler' ),
+        );
 
-		return apply_filters( 'action_scheduler_post_status_failed_labels', $labels );
-	}
+        return apply_filters( 'action_scheduler_post_status_failed_labels', $labels );
+    }
 
-	/**
-	 * Build the args array for the post type definition
-	 *
-	 * @return array
-	 */
-	protected function post_status_running_labels() {
-		$labels = array(
-			'label'       => _x( 'In-Progress', 'post', 'action-scheduler' ),
-			/* translators: %s: count */
-			'label_count' => _n_noop( 'In-Progress <span class="count">(%s)</span>', 'In-Progress <span class="count">(%s)</span>', 'action-scheduler' ),
-		);
+    /**
+     * Build the args array for the post type definition
+     *
+     * @return array
+     */
+    protected function post_status_running_labels() {
+        $labels = array(
+            'label'       => _x( 'In-Progress', 'post', 'action-scheduler' ),
+            /* translators: %s: count */
+            'label_count' => _n_noop( 'In-Progress <span class="count">(%s)</span>', 'In-Progress <span class="count">(%s)</span>', 'action-scheduler' ),
+        );
 
-		return apply_filters( 'action_scheduler_post_status_running_labels', $labels );
-	}
+        return apply_filters( 'action_scheduler_post_status_running_labels', $labels );
+    }
 }

Spacing +9 added lines, -9 removed lines

@@ -6,8 +6,8 @@ 
  */
 class ActionScheduler_wpPostStore_PostStatusRegistrar {
 	public function register() {
-		register_post_status( ActionScheduler_Store::STATUS_RUNNING, array_merge( $this->post_status_args(), $this->post_status_running_labels() ) );
-		register_post_status( ActionScheduler_Store::STATUS_FAILED, array_merge( $this->post_status_args(), $this->post_status_failed_labels() ) );
+		register_post_status(ActionScheduler_Store::STATUS_RUNNING, array_merge($this->post_status_args(), $this->post_status_running_labels()));
+		register_post_status(ActionScheduler_Store::STATUS_FAILED, array_merge($this->post_status_args(), $this->post_status_failed_labels()));
 	}
 
 	/**
@@ -23,7 +23,7 @@ 
 			'show_in_admin_status_list' => true,
 		);
 
-		return apply_filters( 'action_scheduler_post_status_args', $args );
+		return apply_filters('action_scheduler_post_status_args', $args);
 	}
 
 	/**
@@ -33,12 +33,12 @@ 
 	 */
 	protected function post_status_failed_labels() {
 		$labels = array(
-			'label'       => _x( 'Failed', 'post', 'action-scheduler' ),
+			'label'       => _x('Failed', 'post', 'action-scheduler'),
 			/* translators: %s: count */
-			'label_count' => _n_noop( 'Failed <span class="count">(%s)</span>', 'Failed <span class="count">(%s)</span>', 'action-scheduler' ),
+			'label_count' => _n_noop('Failed <span class="count">(%s)</span>', 'Failed <span class="count">(%s)</span>', 'action-scheduler'),
 		);
 
-		return apply_filters( 'action_scheduler_post_status_failed_labels', $labels );
+		return apply_filters('action_scheduler_post_status_failed_labels', $labels);
 	}
 
 	/**
@@ -48,11 +48,11 @@ 
 	 */
 	protected function post_status_running_labels() {
 		$labels = array(
-			'label'       => _x( 'In-Progress', 'post', 'action-scheduler' ),
+			'label'       => _x('In-Progress', 'post', 'action-scheduler'),
 			/* translators: %s: count */
-			'label_count' => _n_noop( 'In-Progress <span class="count">(%s)</span>', 'In-Progress <span class="count">(%s)</span>', 'action-scheduler' ),
+			'label_count' => _n_noop('In-Progress <span class="count">(%s)</span>', 'In-Progress <span class="count">(%s)</span>', 'action-scheduler'),
 		);
 
-		return apply_filters( 'action_scheduler_post_status_running_labels', $labels );
+		return apply_filters('action_scheduler_post_status_running_labels', $labels);
 	}
 }

includes/libraries/action-scheduler/classes/data-stores/ActionScheduler_HybridStore.php

Indentation +360 added lines, -360 removed lines

@@ -13,368 +13,368 @@
  * @since 3.0.0
  */
 class ActionScheduler_HybridStore extends Store {
-	const DEMARKATION_OPTION = 'action_scheduler_hybrid_store_demarkation';
-
-	private $primary_store;
-	private $secondary_store;
-	private $migration_runner;
-
-	/**
-	 * @var int The dividing line between IDs of actions created
-	 *          by the primary and secondary stores.
-	 *
-	 * Methods that accept an action ID will compare the ID against
-	 * this to determine which store will contain that ID. In almost
-	 * all cases, the ID should come from the primary store, but if
-	 * client code is bypassing the API functions and fetching IDs
-	 * from elsewhere, then there is a chance that an unmigrated ID
-	 * might be requested.
-	 */
-	private $demarkation_id = 0;
-
-	/**
-	 * ActionScheduler_HybridStore constructor.
-	 *
-	 * @param Config $config Migration config object.
-	 */
-	public function __construct( Config $config = null ) {
-		$this->demarkation_id = (int) get_option( self::DEMARKATION_OPTION, 0 );
-		if ( empty( $config ) ) {
-			$config = Controller::instance()->get_migration_config_object();
-		}
-		$this->primary_store    = $config->get_destination_store();
-		$this->secondary_store  = $config->get_source_store();
-		$this->migration_runner = new Runner( $config );
-	}
-
-	/**
-	 * Initialize the table data store tables.
-	 *
-	 * @codeCoverageIgnore
-	 */
-	public function init() {
-		add_action( 'action_scheduler/created_table', [ $this, 'set_autoincrement' ], 10, 2 );
-		$this->primary_store->init();
-		$this->secondary_store->init();
-		remove_action( 'action_scheduler/created_table', [ $this, 'set_autoincrement' ], 10 );
-	}
-
-	/**
-	 * When the actions table is created, set its autoincrement
-	 * value to be one higher than the posts table to ensure that
-	 * there are no ID collisions.
-	 *
-	 * @param string $table_name
-	 * @param string $table_suffix
-	 *
-	 * @return void
-	 * @codeCoverageIgnore
-	 */
-	public function set_autoincrement( $table_name, $table_suffix ) {
-		if ( ActionScheduler_StoreSchema::ACTIONS_TABLE === $table_suffix ) {
-			if ( empty( $this->demarkation_id ) ) {
-				$this->demarkation_id = $this->set_demarkation_id();
-			}
-			/** @var \wpdb $wpdb */
-			global $wpdb;
-			$wpdb->insert(
-				$wpdb->{ActionScheduler_StoreSchema::ACTIONS_TABLE},
-				[
-					'action_id' => $this->demarkation_id,
-					'hook'      => '',
-					'status'    => '',
-				]
-			);
-			$wpdb->delete(
-				$wpdb->{ActionScheduler_StoreSchema::ACTIONS_TABLE},
-				[ 'action_id' => $this->demarkation_id ]
-			);
-		}
-	}
-
-	/**
-	 * Store the demarkation id in WP options.
-	 *
-	 * @param int $id The ID to set as the demarkation point between the two stores
-	 *                Leave null to use the next ID from the WP posts table.
-	 *
-	 * @return int The new ID.
-	 *
-	 * @codeCoverageIgnore
-	 */
-	private function set_demarkation_id( $id = null ) {
-		if ( empty( $id ) ) {
-			/** @var \wpdb $wpdb */
-			global $wpdb;
-			$id = (int) $wpdb->get_var( "SELECT MAX(ID) FROM $wpdb->posts" );
-			$id ++;
-		}
-		update_option( self::DEMARKATION_OPTION, $id );
-
-		return $id;
-	}
-
-	/**
-	 * Find the first matching action from the secondary store.
-	 * If it exists, migrate it to the primary store immediately.
-	 * After it migrates, the secondary store will logically contain
-	 * the next matching action, so return the result thence.
-	 *
-	 * @param string $hook
-	 * @param array  $params
-	 *
-	 * @return string
-	 */
-	public function find_action( $hook, $params = [] ) {
-		$found_unmigrated_action = $this->secondary_store->find_action( $hook, $params );
-		if ( ! empty( $found_unmigrated_action ) ) {
-			$this->migrate( [ $found_unmigrated_action ] );
-		}
-
-		return $this->primary_store->find_action( $hook, $params );
-	}
-
-	/**
-	 * Find actions matching the query in the secondary source first.
-	 * If any are found, migrate them immediately. Then the secondary
-	 * store will contain the canonical results.
-	 *
-	 * @param array $query
-	 * @param string $query_type Whether to select or count the results. Default, select.
-	 *
-	 * @return int[]
-	 */
-	public function query_actions( $query = [], $query_type = 'select' ) {
-		$found_unmigrated_actions = $this->secondary_store->query_actions( $query, 'select' );
-		if ( ! empty( $found_unmigrated_actions ) ) {
-			$this->migrate( $found_unmigrated_actions );
-		}
-
-		return $this->primary_store->query_actions( $query, $query_type );
-	}
-
-	/**
-	 * Get a count of all actions in the store, grouped by status
-	 *
-	 * @return array Set of 'status' => int $count pairs for statuses with 1 or more actions of that status.
-	 */
-	public function action_counts() {
-		$unmigrated_actions_count = $this->secondary_store->action_counts();
-		$migrated_actions_count   = $this->primary_store->action_counts();
-		$actions_count_by_status  = array();
-
-		foreach ( $this->get_status_labels() as $status_key => $status_label ) {
-
-			$count = 0;
-
-			if ( isset( $unmigrated_actions_count[ $status_key ] ) ) {
-				$count += $unmigrated_actions_count[ $status_key ];
-			}
-
-			if ( isset( $migrated_actions_count[ $status_key ] ) ) {
-				$count += $migrated_actions_count[ $status_key ];
-			}
-
-			$actions_count_by_status[ $status_key ] = $count;
-		}
-
-		$actions_count_by_status = array_filter( $actions_count_by_status );
-
-		return $actions_count_by_status;
-	}
-
-	/**
-	 * If any actions would have been claimed by the secondary store,
-	 * migrate them immediately, then ask the primary store for the
-	 * canonical claim.
-	 *
-	 * @param int           $max_actions
-	 * @param DateTime|null $before_date
-	 *
-	 * @return ActionScheduler_ActionClaim
-	 */
-	public function stake_claim( $max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '' ) {
-		$claim = $this->secondary_store->stake_claim( $max_actions, $before_date, $hooks, $group );
-
-		$claimed_actions = $claim->get_actions();
-		if ( ! empty( $claimed_actions ) ) {
-			$this->migrate( $claimed_actions );
-		}
-
-		$this->secondary_store->release_claim( $claim );
-
-		return $this->primary_store->stake_claim( $max_actions, $before_date, $hooks, $group );
-	}
-
-	/**
-	 * Migrate a list of actions to the table data store.
-	 *
-	 * @param array $action_ids List of action IDs.
-	 */
-	private function migrate( $action_ids ) {
-		$this->migration_runner->migrate_actions( $action_ids );
-	}
-
-	/**
-	 * Save an action to the primary store.
-	 *
-	 * @param ActionScheduler_Action $action Action object to be saved.
-	 * @param DateTime               $date Optional. Schedule date. Default null.
-	 */
-	public function save_action( ActionScheduler_Action $action, DateTime $date = null ) {
-		return $this->primary_store->save_action( $action, $date );
-	}
-
-	/**
-	 * Retrieve an existing action whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function fetch_action( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			return $this->secondary_store->fetch_action( $action_id );
-		} else {
-			return $this->primary_store->fetch_action( $action_id );
-		}
-	}
-
-	/**
-	 * Cancel an existing action whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function cancel_action( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			$this->secondary_store->cancel_action( $action_id );
-		} else {
-			$this->primary_store->cancel_action( $action_id );
-		}
-	}
-
-	/**
-	 * Delete an existing action whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function delete_action( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			$this->secondary_store->delete_action( $action_id );
-		} else {
-			$this->primary_store->delete_action( $action_id );
-		}
-	}
-
-	/**
-	 * Get the schedule date an existing action whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function get_date( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			return $this->secondary_store->get_date( $action_id );
-		} else {
-			return $this->primary_store->get_date( $action_id );
-		}
-	}
-
-	/**
-	 * Mark an existing action as failed whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function mark_failure( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			$this->secondary_store->mark_failure( $action_id );
-		} else {
-			$this->primary_store->mark_failure( $action_id );
-		}
-	}
-
-	/**
-	 * Log the execution of an existing action whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function log_execution( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			$this->secondary_store->log_execution( $action_id );
-		} else {
-			$this->primary_store->log_execution( $action_id );
-		}
-	}
-
-	/**
-	 * Mark an existing action complete whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function mark_complete( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			$this->secondary_store->mark_complete( $action_id );
-		} else {
-			$this->primary_store->mark_complete( $action_id );
-		}
-	}
-
-	/**
-	 * Get an existing action status whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function get_status( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			return $this->secondary_store->get_status( $action_id );
-		} else {
-			return $this->primary_store->get_status( $action_id );
-		}
-	}
-
-
-	/* * * * * * * * * * * * * * * * * * * * * * * * * * *
+    const DEMARKATION_OPTION = 'action_scheduler_hybrid_store_demarkation';
+
+    private $primary_store;
+    private $secondary_store;
+    private $migration_runner;
+
+    /**
+     * @var int The dividing line between IDs of actions created
+     *          by the primary and secondary stores.
+     *
+     * Methods that accept an action ID will compare the ID against
+     * this to determine which store will contain that ID. In almost
+     * all cases, the ID should come from the primary store, but if
+     * client code is bypassing the API functions and fetching IDs
+     * from elsewhere, then there is a chance that an unmigrated ID
+     * might be requested.
+     */
+    private $demarkation_id = 0;
+
+    /**
+     * ActionScheduler_HybridStore constructor.
+     *
+     * @param Config $config Migration config object.
+     */
+    public function __construct( Config $config = null ) {
+        $this->demarkation_id = (int) get_option( self::DEMARKATION_OPTION, 0 );
+        if ( empty( $config ) ) {
+            $config = Controller::instance()->get_migration_config_object();
+        }
+        $this->primary_store    = $config->get_destination_store();
+        $this->secondary_store  = $config->get_source_store();
+        $this->migration_runner = new Runner( $config );
+    }
+
+    /**
+     * Initialize the table data store tables.
+     *
+     * @codeCoverageIgnore
+     */
+    public function init() {
+        add_action( 'action_scheduler/created_table', [ $this, 'set_autoincrement' ], 10, 2 );
+        $this->primary_store->init();
+        $this->secondary_store->init();
+        remove_action( 'action_scheduler/created_table', [ $this, 'set_autoincrement' ], 10 );
+    }
+
+    /**
+     * When the actions table is created, set its autoincrement
+     * value to be one higher than the posts table to ensure that
+     * there are no ID collisions.
+     *
+     * @param string $table_name
+     * @param string $table_suffix
+     *
+     * @return void
+     * @codeCoverageIgnore
+     */
+    public function set_autoincrement( $table_name, $table_suffix ) {
+        if ( ActionScheduler_StoreSchema::ACTIONS_TABLE === $table_suffix ) {
+            if ( empty( $this->demarkation_id ) ) {
+                $this->demarkation_id = $this->set_demarkation_id();
+            }
+            /** @var \wpdb $wpdb */
+            global $wpdb;
+            $wpdb->insert(
+                $wpdb->{ActionScheduler_StoreSchema::ACTIONS_TABLE},
+                [
+                    'action_id' => $this->demarkation_id,
+                    'hook'      => '',
+                    'status'    => '',
+                ]
+            );
+            $wpdb->delete(
+                $wpdb->{ActionScheduler_StoreSchema::ACTIONS_TABLE},
+                [ 'action_id' => $this->demarkation_id ]
+            );
+        }
+    }
+
+    /**
+     * Store the demarkation id in WP options.
+     *
+     * @param int $id The ID to set as the demarkation point between the two stores
+     *                Leave null to use the next ID from the WP posts table.
+     *
+     * @return int The new ID.
+     *
+     * @codeCoverageIgnore
+     */
+    private function set_demarkation_id( $id = null ) {
+        if ( empty( $id ) ) {
+            /** @var \wpdb $wpdb */
+            global $wpdb;
+            $id = (int) $wpdb->get_var( "SELECT MAX(ID) FROM $wpdb->posts" );
+            $id ++;
+        }
+        update_option( self::DEMARKATION_OPTION, $id );
+
+        return $id;
+    }
+
+    /**
+     * Find the first matching action from the secondary store.
+     * If it exists, migrate it to the primary store immediately.
+     * After it migrates, the secondary store will logically contain
+     * the next matching action, so return the result thence.
+     *
+     * @param string $hook
+     * @param array  $params
+     *
+     * @return string
+     */
+    public function find_action( $hook, $params = [] ) {
+        $found_unmigrated_action = $this->secondary_store->find_action( $hook, $params );
+        if ( ! empty( $found_unmigrated_action ) ) {
+            $this->migrate( [ $found_unmigrated_action ] );
+        }
+
+        return $this->primary_store->find_action( $hook, $params );
+    }
+
+    /**
+     * Find actions matching the query in the secondary source first.
+     * If any are found, migrate them immediately. Then the secondary
+     * store will contain the canonical results.
+     *
+     * @param array $query
+     * @param string $query_type Whether to select or count the results. Default, select.
+     *
+     * @return int[]
+     */
+    public function query_actions( $query = [], $query_type = 'select' ) {
+        $found_unmigrated_actions = $this->secondary_store->query_actions( $query, 'select' );
+        if ( ! empty( $found_unmigrated_actions ) ) {
+            $this->migrate( $found_unmigrated_actions );
+        }
+
+        return $this->primary_store->query_actions( $query, $query_type );
+    }
+
+    /**
+     * Get a count of all actions in the store, grouped by status
+     *
+     * @return array Set of 'status' => int $count pairs for statuses with 1 or more actions of that status.
+     */
+    public function action_counts() {
+        $unmigrated_actions_count = $this->secondary_store->action_counts();
+        $migrated_actions_count   = $this->primary_store->action_counts();
+        $actions_count_by_status  = array();
+
+        foreach ( $this->get_status_labels() as $status_key => $status_label ) {
+
+            $count = 0;
+
+            if ( isset( $unmigrated_actions_count[ $status_key ] ) ) {
+                $count += $unmigrated_actions_count[ $status_key ];
+            }
+
+            if ( isset( $migrated_actions_count[ $status_key ] ) ) {
+                $count += $migrated_actions_count[ $status_key ];
+            }
+
+            $actions_count_by_status[ $status_key ] = $count;
+        }
+
+        $actions_count_by_status = array_filter( $actions_count_by_status );
+
+        return $actions_count_by_status;
+    }
+
+    /**
+     * If any actions would have been claimed by the secondary store,
+     * migrate them immediately, then ask the primary store for the
+     * canonical claim.
+     *
+     * @param int           $max_actions
+     * @param DateTime|null $before_date
+     *
+     * @return ActionScheduler_ActionClaim
+     */
+    public function stake_claim( $max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '' ) {
+        $claim = $this->secondary_store->stake_claim( $max_actions, $before_date, $hooks, $group );
+
+        $claimed_actions = $claim->get_actions();
+        if ( ! empty( $claimed_actions ) ) {
+            $this->migrate( $claimed_actions );
+        }
+
+        $this->secondary_store->release_claim( $claim );
+
+        return $this->primary_store->stake_claim( $max_actions, $before_date, $hooks, $group );
+    }
+
+    /**
+     * Migrate a list of actions to the table data store.
+     *
+     * @param array $action_ids List of action IDs.
+     */
+    private function migrate( $action_ids ) {
+        $this->migration_runner->migrate_actions( $action_ids );
+    }
+
+    /**
+     * Save an action to the primary store.
+     *
+     * @param ActionScheduler_Action $action Action object to be saved.
+     * @param DateTime               $date Optional. Schedule date. Default null.
+     */
+    public function save_action( ActionScheduler_Action $action, DateTime $date = null ) {
+        return $this->primary_store->save_action( $action, $date );
+    }
+
+    /**
+     * Retrieve an existing action whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function fetch_action( $action_id ) {
+        if ( $action_id < $this->demarkation_id ) {
+            return $this->secondary_store->fetch_action( $action_id );
+        } else {
+            return $this->primary_store->fetch_action( $action_id );
+        }
+    }
+
+    /**
+     * Cancel an existing action whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function cancel_action( $action_id ) {
+        if ( $action_id < $this->demarkation_id ) {
+            $this->secondary_store->cancel_action( $action_id );
+        } else {
+            $this->primary_store->cancel_action( $action_id );
+        }
+    }
+
+    /**
+     * Delete an existing action whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function delete_action( $action_id ) {
+        if ( $action_id < $this->demarkation_id ) {
+            $this->secondary_store->delete_action( $action_id );
+        } else {
+            $this->primary_store->delete_action( $action_id );
+        }
+    }
+
+    /**
+     * Get the schedule date an existing action whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function get_date( $action_id ) {
+        if ( $action_id < $this->demarkation_id ) {
+            return $this->secondary_store->get_date( $action_id );
+        } else {
+            return $this->primary_store->get_date( $action_id );
+        }
+    }
+
+    /**
+     * Mark an existing action as failed whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function mark_failure( $action_id ) {
+        if ( $action_id < $this->demarkation_id ) {
+            $this->secondary_store->mark_failure( $action_id );
+        } else {
+            $this->primary_store->mark_failure( $action_id );
+        }
+    }
+
+    /**
+     * Log the execution of an existing action whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function log_execution( $action_id ) {
+        if ( $action_id < $this->demarkation_id ) {
+            $this->secondary_store->log_execution( $action_id );
+        } else {
+            $this->primary_store->log_execution( $action_id );
+        }
+    }
+
+    /**
+     * Mark an existing action complete whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function mark_complete( $action_id ) {
+        if ( $action_id < $this->demarkation_id ) {
+            $this->secondary_store->mark_complete( $action_id );
+        } else {
+            $this->primary_store->mark_complete( $action_id );
+        }
+    }
+
+    /**
+     * Get an existing action status whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function get_status( $action_id ) {
+        if ( $action_id < $this->demarkation_id ) {
+            return $this->secondary_store->get_status( $action_id );
+        } else {
+            return $this->primary_store->get_status( $action_id );
+        }
+    }
+
+
+    /* * * * * * * * * * * * * * * * * * * * * * * * * * *
 	 * All claim-related functions should operate solely
 	 * on the primary store.
 	 * * * * * * * * * * * * * * * * * * * * * * * * * * */
 
-	/**
-	 * Get the claim count from the table data store.
-	 */
-	public function get_claim_count() {
-		return $this->primary_store->get_claim_count();
-	}
-
-	/**
-	 * Retrieve the claim ID for an action from the table data store.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function get_claim_id( $action_id ) {
-		return $this->primary_store->get_claim_id( $action_id );
-	}
-
-	/**
-	 * Release a claim in the table data store.
-	 *
-	 * @param ActionScheduler_ActionClaim $claim Claim object.
-	 */
-	public function release_claim( ActionScheduler_ActionClaim $claim ) {
-		$this->primary_store->release_claim( $claim );
-	}
-
-	/**
-	 * Release claims on an action in the table data store.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function unclaim_action( $action_id ) {
-		$this->primary_store->unclaim_action( $action_id );
-	}
-
-	/**
-	 * Retrieve a list of action IDs by claim.
-	 *
-	 * @param int $claim_id Claim ID.
-	 */
-	public function find_actions_by_claim_id( $claim_id ) {
-		return $this->primary_store->find_actions_by_claim_id( $claim_id );
-	}
+    /**
+     * Get the claim count from the table data store.
+     */
+    public function get_claim_count() {
+        return $this->primary_store->get_claim_count();
+    }
+
+    /**
+     * Retrieve the claim ID for an action from the table data store.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function get_claim_id( $action_id ) {
+        return $this->primary_store->get_claim_id( $action_id );
+    }
+
+    /**
+     * Release a claim in the table data store.
+     *
+     * @param ActionScheduler_ActionClaim $claim Claim object.
+     */
+    public function release_claim( ActionScheduler_ActionClaim $claim ) {
+        $this->primary_store->release_claim( $claim );
+    }
+
+    /**
+     * Release claims on an action in the table data store.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function unclaim_action( $action_id ) {
+        $this->primary_store->unclaim_action( $action_id );
+    }
+
+    /**
+     * Retrieve a list of action IDs by claim.
+     *
+     * @param int $claim_id Claim ID.
+     */
+    public function find_actions_by_claim_id( $claim_id ) {
+        return $this->primary_store->find_actions_by_claim_id( $claim_id );
+    }
 }

Spacing +82 added lines, -82 removed lines

@@ -37,14 +37,14 @@ 
 	 *
 	 * @param Config $config Migration config object.
 	 */
-	public function __construct( Config $config = null ) {
-		$this->demarkation_id = (int) get_option( self::DEMARKATION_OPTION, 0 );
-		if ( empty( $config ) ) {
+	public function __construct(Config $config = null) {
+		$this->demarkation_id = (int) get_option(self::DEMARKATION_OPTION, 0);
+		if (empty($config)) {
 			$config = Controller::instance()->get_migration_config_object();
 		}
 		$this->primary_store    = $config->get_destination_store();
 		$this->secondary_store  = $config->get_source_store();
-		$this->migration_runner = new Runner( $config );
+		$this->migration_runner = new Runner($config);
 	}
 
 	/**
@@ -53,10 +53,10 @@ 
 	 * @codeCoverageIgnore
 	 */
 	public function init() {
-		add_action( 'action_scheduler/created_table', [ $this, 'set_autoincrement' ], 10, 2 );
+		add_action('action_scheduler/created_table', [$this, 'set_autoincrement'], 10, 2);
 		$this->primary_store->init();
 		$this->secondary_store->init();
-		remove_action( 'action_scheduler/created_table', [ $this, 'set_autoincrement' ], 10 );
+		remove_action('action_scheduler/created_table', [$this, 'set_autoincrement'], 10);
 	}
 
 	/**
@@ -70,9 +70,9 @@ 
 	 * @return void
 	 * @codeCoverageIgnore
 	 */
-	public function set_autoincrement( $table_name, $table_suffix ) {
-		if ( ActionScheduler_StoreSchema::ACTIONS_TABLE === $table_suffix ) {
-			if ( empty( $this->demarkation_id ) ) {
+	public function set_autoincrement($table_name, $table_suffix) {
+		if (ActionScheduler_StoreSchema::ACTIONS_TABLE === $table_suffix) {
+			if (empty($this->demarkation_id)) {
 				$this->demarkation_id = $this->set_demarkation_id();
 			}
 			/** @var \wpdb $wpdb */
@@ -87,7 +87,7 @@ 
 			);
 			$wpdb->delete(
 				$wpdb->{ActionScheduler_StoreSchema::ACTIONS_TABLE},
-				[ 'action_id' => $this->demarkation_id ]
+				['action_id' => $this->demarkation_id]
 			);
 		}
 	}
@@ -102,14 +102,14 @@ 
 	 *
 	 * @codeCoverageIgnore
 	 */
-	private function set_demarkation_id( $id = null ) {
-		if ( empty( $id ) ) {
+	private function set_demarkation_id($id = null) {
+		if (empty($id)) {
 			/** @var \wpdb $wpdb */
 			global $wpdb;
-			$id = (int) $wpdb->get_var( "SELECT MAX(ID) FROM $wpdb->posts" );
-			$id ++;
+			$id = (int) $wpdb->get_var("SELECT MAX(ID) FROM $wpdb->posts");
+			$id++;
 		}
-		update_option( self::DEMARKATION_OPTION, $id );
+		update_option(self::DEMARKATION_OPTION, $id);
 
 		return $id;
 	}
@@ -125,13 +125,13 @@ 
 	 *
 	 * @return string
 	 */
-	public function find_action( $hook, $params = [] ) {
-		$found_unmigrated_action = $this->secondary_store->find_action( $hook, $params );
-		if ( ! empty( $found_unmigrated_action ) ) {
-			$this->migrate( [ $found_unmigrated_action ] );
+	public function find_action($hook, $params = []) {
+		$found_unmigrated_action = $this->secondary_store->find_action($hook, $params);
+		if (!empty($found_unmigrated_action)) {
+			$this->migrate([$found_unmigrated_action]);
 		}
 
-		return $this->primary_store->find_action( $hook, $params );
+		return $this->primary_store->find_action($hook, $params);
 	}
 
 	/**
@@ -144,13 +144,13 @@ 
 	 *
 	 * @return int[]
 	 */
-	public function query_actions( $query = [], $query_type = 'select' ) {
-		$found_unmigrated_actions = $this->secondary_store->query_actions( $query, 'select' );
-		if ( ! empty( $found_unmigrated_actions ) ) {
-			$this->migrate( $found_unmigrated_actions );
+	public function query_actions($query = [], $query_type = 'select') {
+		$found_unmigrated_actions = $this->secondary_store->query_actions($query, 'select');
+		if (!empty($found_unmigrated_actions)) {
+			$this->migrate($found_unmigrated_actions);
 		}
 
-		return $this->primary_store->query_actions( $query, $query_type );
+		return $this->primary_store->query_actions($query, $query_type);
 	}
 
 	/**
@@ -163,22 +163,22 @@ 
 		$migrated_actions_count   = $this->primary_store->action_counts();
 		$actions_count_by_status  = array();
 
-		foreach ( $this->get_status_labels() as $status_key => $status_label ) {
+		foreach ($this->get_status_labels() as $status_key => $status_label) {
 
 			$count = 0;
 
-			if ( isset( $unmigrated_actions_count[ $status_key ] ) ) {
-				$count += $unmigrated_actions_count[ $status_key ];
+			if (isset($unmigrated_actions_count[$status_key])) {
+				$count += $unmigrated_actions_count[$status_key];
 			}
 
-			if ( isset( $migrated_actions_count[ $status_key ] ) ) {
-				$count += $migrated_actions_count[ $status_key ];
+			if (isset($migrated_actions_count[$status_key])) {
+				$count += $migrated_actions_count[$status_key];
 			}
 
-			$actions_count_by_status[ $status_key ] = $count;
+			$actions_count_by_status[$status_key] = $count;
 		}
 
-		$actions_count_by_status = array_filter( $actions_count_by_status );
+		$actions_count_by_status = array_filter($actions_count_by_status);
 
 		return $actions_count_by_status;
 	}
@@ -193,17 +193,17 @@ 
 	 *
 	 * @return ActionScheduler_ActionClaim
 	 */
-	public function stake_claim( $max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '' ) {
-		$claim = $this->secondary_store->stake_claim( $max_actions, $before_date, $hooks, $group );
+	public function stake_claim($max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '') {
+		$claim = $this->secondary_store->stake_claim($max_actions, $before_date, $hooks, $group);
 
 		$claimed_actions = $claim->get_actions();
-		if ( ! empty( $claimed_actions ) ) {
-			$this->migrate( $claimed_actions );
+		if (!empty($claimed_actions)) {
+			$this->migrate($claimed_actions);
 		}
 
-		$this->secondary_store->release_claim( $claim );
+		$this->secondary_store->release_claim($claim);
 
-		return $this->primary_store->stake_claim( $max_actions, $before_date, $hooks, $group );
+		return $this->primary_store->stake_claim($max_actions, $before_date, $hooks, $group);
 	}
 
 	/**
@@ -211,8 +211,8 @@ 
 	 *
 	 * @param array $action_ids List of action IDs.
 	 */
-	private function migrate( $action_ids ) {
-		$this->migration_runner->migrate_actions( $action_ids );
+	private function migrate($action_ids) {
+		$this->migration_runner->migrate_actions($action_ids);
 	}
 
 	/**
@@ -221,8 +221,8 @@ 
 	 * @param ActionScheduler_Action $action Action object to be saved.
 	 * @param DateTime               $date Optional. Schedule date. Default null.
 	 */
-	public function save_action( ActionScheduler_Action $action, DateTime $date = null ) {
-		return $this->primary_store->save_action( $action, $date );
+	public function save_action(ActionScheduler_Action $action, DateTime $date = null) {
+		return $this->primary_store->save_action($action, $date);
 	}
 
 	/**
@@ -230,11 +230,11 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function fetch_action( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			return $this->secondary_store->fetch_action( $action_id );
+	public function fetch_action($action_id) {
+		if ($action_id < $this->demarkation_id) {
+			return $this->secondary_store->fetch_action($action_id);
 		} else {
-			return $this->primary_store->fetch_action( $action_id );
+			return $this->primary_store->fetch_action($action_id);
 		}
 	}
 
@@ -243,11 +243,11 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function cancel_action( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			$this->secondary_store->cancel_action( $action_id );
+	public function cancel_action($action_id) {
+		if ($action_id < $this->demarkation_id) {
+			$this->secondary_store->cancel_action($action_id);
 		} else {
-			$this->primary_store->cancel_action( $action_id );
+			$this->primary_store->cancel_action($action_id);
 		}
 	}
 
@@ -256,11 +256,11 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function delete_action( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			$this->secondary_store->delete_action( $action_id );
+	public function delete_action($action_id) {
+		if ($action_id < $this->demarkation_id) {
+			$this->secondary_store->delete_action($action_id);
 		} else {
-			$this->primary_store->delete_action( $action_id );
+			$this->primary_store->delete_action($action_id);
 		}
 	}
 
@@ -269,11 +269,11 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function get_date( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			return $this->secondary_store->get_date( $action_id );
+	public function get_date($action_id) {
+		if ($action_id < $this->demarkation_id) {
+			return $this->secondary_store->get_date($action_id);
 		} else {
-			return $this->primary_store->get_date( $action_id );
+			return $this->primary_store->get_date($action_id);
 		}
 	}
 
@@ -282,11 +282,11 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function mark_failure( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			$this->secondary_store->mark_failure( $action_id );
+	public function mark_failure($action_id) {
+		if ($action_id < $this->demarkation_id) {
+			$this->secondary_store->mark_failure($action_id);
 		} else {
-			$this->primary_store->mark_failure( $action_id );
+			$this->primary_store->mark_failure($action_id);
 		}
 	}
 
@@ -295,11 +295,11 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function log_execution( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			$this->secondary_store->log_execution( $action_id );
+	public function log_execution($action_id) {
+		if ($action_id < $this->demarkation_id) {
+			$this->secondary_store->log_execution($action_id);
 		} else {
-			$this->primary_store->log_execution( $action_id );
+			$this->primary_store->log_execution($action_id);
 		}
 	}
 
@@ -308,11 +308,11 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function mark_complete( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			$this->secondary_store->mark_complete( $action_id );
+	public function mark_complete($action_id) {
+		if ($action_id < $this->demarkation_id) {
+			$this->secondary_store->mark_complete($action_id);
 		} else {
-			$this->primary_store->mark_complete( $action_id );
+			$this->primary_store->mark_complete($action_id);
 		}
 	}
 
@@ -321,11 +321,11 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function get_status( $action_id ) {
-		if ( $action_id < $this->demarkation_id ) {
-			return $this->secondary_store->get_status( $action_id );
+	public function get_status($action_id) {
+		if ($action_id < $this->demarkation_id) {
+			return $this->secondary_store->get_status($action_id);
 		} else {
-			return $this->primary_store->get_status( $action_id );
+			return $this->primary_store->get_status($action_id);
 		}
 	}
 
@@ -347,8 +347,8 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function get_claim_id( $action_id ) {
-		return $this->primary_store->get_claim_id( $action_id );
+	public function get_claim_id($action_id) {
+		return $this->primary_store->get_claim_id($action_id);
 	}
 
 	/**
@@ -356,8 +356,8 @@ 
 	 *
 	 * @param ActionScheduler_ActionClaim $claim Claim object.
 	 */
-	public function release_claim( ActionScheduler_ActionClaim $claim ) {
-		$this->primary_store->release_claim( $claim );
+	public function release_claim(ActionScheduler_ActionClaim $claim) {
+		$this->primary_store->release_claim($claim);
 	}
 
 	/**
@@ -365,8 +365,8 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function unclaim_action( $action_id ) {
-		$this->primary_store->unclaim_action( $action_id );
+	public function unclaim_action($action_id) {
+		$this->primary_store->unclaim_action($action_id);
 	}
 
 	/**
@@ -374,7 +374,7 @@ 
 	 *
 	 * @param int $claim_id Claim ID.
 	 */
-	public function find_actions_by_claim_id( $claim_id ) {
-		return $this->primary_store->find_actions_by_claim_id( $claim_id );
+	public function find_actions_by_claim_id($claim_id) {
+		return $this->primary_store->find_actions_by_claim_id($claim_id);
 	}
 }

includes/libraries/action-scheduler/classes/data-stores/ActionScheduler_DBLogger.php

Indentation +134 added lines, -134 removed lines

@@ -9,138 +9,138 @@
  */
 class ActionScheduler_DBLogger extends ActionScheduler_Logger {
 
-	/**
-	 * Add a record to an action log.
-	 *
-	 * @param int      $action_id Action ID.
-	 * @param string   $message Message to be saved in the log entry.
-	 * @param DateTime $date Timestamp of the log entry.
-	 *
-	 * @return int     The log entry ID.
-	 */
-	public function log( $action_id, $message, DateTime $date = null ) {
-		if ( empty( $date ) ) {
-			$date = as_get_datetime_object();
-		} else {
-			$date = clone $date;
-		}
-
-		$date_gmt = $date->format( 'Y-m-d H:i:s' );
-		ActionScheduler_TimezoneHelper::set_local_timezone( $date );
-		$date_local = $date->format( 'Y-m-d H:i:s' );
-
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$wpdb->insert( $wpdb->actionscheduler_logs, [
-			'action_id'      => $action_id,
-			'message'        => $message,
-			'log_date_gmt'   => $date_gmt,
-			'log_date_local' => $date_local,
-		], [ '%d', '%s', '%s', '%s' ] );
-
-		return $wpdb->insert_id;
-	}
-
-	/**
-	 * Retrieve an action log entry.
-	 *
-	 * @param int $entry_id Log entry ID.
-	 *
-	 * @return ActionScheduler_LogEntry
-	 */
-	public function get_entry( $entry_id ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$entry = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_logs} WHERE log_id=%d", $entry_id ) );
-
-		return $this->create_entry_from_db_record( $entry );
-	}
-
-	/**
-	 * Create an action log entry from a database record.
-	 *
-	 * @param object $record Log entry database record object.
-	 *
-	 * @return ActionScheduler_LogEntry
-	 */
-	private function create_entry_from_db_record( $record ) {
-		if ( empty( $record ) ) {
-			return new ActionScheduler_NullLogEntry();
-		}
-
-		$date = as_get_datetime_object( $record->log_date_gmt );
-
-		return new ActionScheduler_LogEntry( $record->action_id, $record->message, $date );
-	}
-
-	/**
-	 * Retrieve the an action's log entries from the database.
-	 *
-	 * @param int $action_id Action ID.
-	 *
-	 * @return ActionScheduler_LogEntry[]
-	 */
-	public function get_logs( $action_id ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-
-		$records = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_logs} WHERE action_id=%d", $action_id ) );
-
-		return array_map( [ $this, 'create_entry_from_db_record' ], $records );
-	}
-
-	/**
-	 * Initialize the data store.
-	 *
-	 * @codeCoverageIgnore
-	 */
-	public function init() {
-
-		$table_maker = new ActionScheduler_LoggerSchema();
-		$table_maker->register_tables();
-
-		parent::init();
-
-		add_action( 'action_scheduler_deleted_action', [ $this, 'clear_deleted_action_logs' ], 10, 1 );
-	}
-
-	/**
-	 * Delete the action logs for an action.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function clear_deleted_action_logs( $action_id ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$wpdb->delete( $wpdb->actionscheduler_logs, [ 'action_id' => $action_id, ], [ '%d' ] );
-	}
-
-	/**
-	 * Bulk add cancel action log entries.
-	 *
-	 * @param array $action_ids List of action ID.
-	 */
-	public function bulk_log_cancel_actions( $action_ids ) {
-		if ( empty( $action_ids ) ) {
-			return;
-		}
-
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$date     = as_get_datetime_object();
-		$date_gmt = $date->format( 'Y-m-d H:i:s' );
-		ActionScheduler_TimezoneHelper::set_local_timezone( $date );
-		$date_local = $date->format( 'Y-m-d H:i:s' );
-		$message    = __( 'action canceled', 'action-scheduler' );
-		$format     = '(%d, ' . $wpdb->prepare( '%s, %s, %s', $message, $date_gmt, $date_local ) . ')';
-		$sql_query  = "INSERT {$wpdb->actionscheduler_logs} (action_id, message, log_date_gmt, log_date_local) VALUES ";
-		$value_rows = [];
-
-		foreach ( $action_ids as $action_id ) {
-			$value_rows[] = $wpdb->prepare( $format, $action_id );
-		}
-		$sql_query .= implode( ',', $value_rows );
-
-		$wpdb->query( $sql_query );
-	}
+    /**
+     * Add a record to an action log.
+     *
+     * @param int      $action_id Action ID.
+     * @param string   $message Message to be saved in the log entry.
+     * @param DateTime $date Timestamp of the log entry.
+     *
+     * @return int     The log entry ID.
+     */
+    public function log( $action_id, $message, DateTime $date = null ) {
+        if ( empty( $date ) ) {
+            $date = as_get_datetime_object();
+        } else {
+            $date = clone $date;
+        }
+
+        $date_gmt = $date->format( 'Y-m-d H:i:s' );
+        ActionScheduler_TimezoneHelper::set_local_timezone( $date );
+        $date_local = $date->format( 'Y-m-d H:i:s' );
+
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $wpdb->insert( $wpdb->actionscheduler_logs, [
+            'action_id'      => $action_id,
+            'message'        => $message,
+            'log_date_gmt'   => $date_gmt,
+            'log_date_local' => $date_local,
+        ], [ '%d', '%s', '%s', '%s' ] );
+
+        return $wpdb->insert_id;
+    }
+
+    /**
+     * Retrieve an action log entry.
+     *
+     * @param int $entry_id Log entry ID.
+     *
+     * @return ActionScheduler_LogEntry
+     */
+    public function get_entry( $entry_id ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $entry = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_logs} WHERE log_id=%d", $entry_id ) );
+
+        return $this->create_entry_from_db_record( $entry );
+    }
+
+    /**
+     * Create an action log entry from a database record.
+     *
+     * @param object $record Log entry database record object.
+     *
+     * @return ActionScheduler_LogEntry
+     */
+    private function create_entry_from_db_record( $record ) {
+        if ( empty( $record ) ) {
+            return new ActionScheduler_NullLogEntry();
+        }
+
+        $date = as_get_datetime_object( $record->log_date_gmt );
+
+        return new ActionScheduler_LogEntry( $record->action_id, $record->message, $date );
+    }
+
+    /**
+     * Retrieve the an action's log entries from the database.
+     *
+     * @param int $action_id Action ID.
+     *
+     * @return ActionScheduler_LogEntry[]
+     */
+    public function get_logs( $action_id ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+
+        $records = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_logs} WHERE action_id=%d", $action_id ) );
+
+        return array_map( [ $this, 'create_entry_from_db_record' ], $records );
+    }
+
+    /**
+     * Initialize the data store.
+     *
+     * @codeCoverageIgnore
+     */
+    public function init() {
+
+        $table_maker = new ActionScheduler_LoggerSchema();
+        $table_maker->register_tables();
+
+        parent::init();
+
+        add_action( 'action_scheduler_deleted_action', [ $this, 'clear_deleted_action_logs' ], 10, 1 );
+    }
+
+    /**
+     * Delete the action logs for an action.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function clear_deleted_action_logs( $action_id ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $wpdb->delete( $wpdb->actionscheduler_logs, [ 'action_id' => $action_id, ], [ '%d' ] );
+    }
+
+    /**
+     * Bulk add cancel action log entries.
+     *
+     * @param array $action_ids List of action ID.
+     */
+    public function bulk_log_cancel_actions( $action_ids ) {
+        if ( empty( $action_ids ) ) {
+            return;
+        }
+
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $date     = as_get_datetime_object();
+        $date_gmt = $date->format( 'Y-m-d H:i:s' );
+        ActionScheduler_TimezoneHelper::set_local_timezone( $date );
+        $date_local = $date->format( 'Y-m-d H:i:s' );
+        $message    = __( 'action canceled', 'action-scheduler' );
+        $format     = '(%d, ' . $wpdb->prepare( '%s, %s, %s', $message, $date_gmt, $date_local ) . ')';
+        $sql_query  = "INSERT {$wpdb->actionscheduler_logs} (action_id, message, log_date_gmt, log_date_local) VALUES ";
+        $value_rows = [];
+
+        foreach ( $action_ids as $action_id ) {
+            $value_rows[] = $wpdb->prepare( $format, $action_id );
+        }
+        $sql_query .= implode( ',', $value_rows );
+
+        $wpdb->query( $sql_query );
+    }
 }

Spacing +31 added lines, -31 removed lines

@@ -18,25 +18,25 @@ 
 	 *
 	 * @return int     The log entry ID.
 	 */
-	public function log( $action_id, $message, DateTime $date = null ) {
-		if ( empty( $date ) ) {
+	public function log($action_id, $message, DateTime $date = null) {
+		if (empty($date)) {
 			$date = as_get_datetime_object();
 		} else {
 			$date = clone $date;
 		}
 
-		$date_gmt = $date->format( 'Y-m-d H:i:s' );
-		ActionScheduler_TimezoneHelper::set_local_timezone( $date );
-		$date_local = $date->format( 'Y-m-d H:i:s' );
+		$date_gmt = $date->format('Y-m-d H:i:s');
+		ActionScheduler_TimezoneHelper::set_local_timezone($date);
+		$date_local = $date->format('Y-m-d H:i:s');
 
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$wpdb->insert( $wpdb->actionscheduler_logs, [
+		$wpdb->insert($wpdb->actionscheduler_logs, [
 			'action_id'      => $action_id,
 			'message'        => $message,
 			'log_date_gmt'   => $date_gmt,
 			'log_date_local' => $date_local,
-		], [ '%d', '%s', '%s', '%s' ] );
+		], ['%d', '%s', '%s', '%s']);
 
 		return $wpdb->insert_id;
 	}
@@ -48,12 +48,12 @@ 
 	 *
 	 * @return ActionScheduler_LogEntry
 	 */
-	public function get_entry( $entry_id ) {
+	public function get_entry($entry_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$entry = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_logs} WHERE log_id=%d", $entry_id ) );
+		$entry = $wpdb->get_row($wpdb->prepare("SELECT * FROM {$wpdb->actionscheduler_logs} WHERE log_id=%d", $entry_id));
 
-		return $this->create_entry_from_db_record( $entry );
+		return $this->create_entry_from_db_record($entry);
 	}
 
 	/**
@@ -63,14 +63,14 @@ 
 	 *
 	 * @return ActionScheduler_LogEntry
 	 */
-	private function create_entry_from_db_record( $record ) {
-		if ( empty( $record ) ) {
+	private function create_entry_from_db_record($record) {
+		if (empty($record)) {
 			return new ActionScheduler_NullLogEntry();
 		}
 
-		$date = as_get_datetime_object( $record->log_date_gmt );
+		$date = as_get_datetime_object($record->log_date_gmt);
 
-		return new ActionScheduler_LogEntry( $record->action_id, $record->message, $date );
+		return new ActionScheduler_LogEntry($record->action_id, $record->message, $date);
 	}
 
 	/**
@@ -80,13 +80,13 @@ 
 	 *
 	 * @return ActionScheduler_LogEntry[]
 	 */
-	public function get_logs( $action_id ) {
+	public function get_logs($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 
-		$records = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_logs} WHERE action_id=%d", $action_id ) );
+		$records = $wpdb->get_results($wpdb->prepare("SELECT * FROM {$wpdb->actionscheduler_logs} WHERE action_id=%d", $action_id));
 
-		return array_map( [ $this, 'create_entry_from_db_record' ], $records );
+		return array_map([$this, 'create_entry_from_db_record'], $records);
 	}
 
 	/**
@@ -101,7 +101,7 @@ 
 
 		parent::init();
 
-		add_action( 'action_scheduler_deleted_action', [ $this, 'clear_deleted_action_logs' ], 10, 1 );
+		add_action('action_scheduler_deleted_action', [$this, 'clear_deleted_action_logs'], 10, 1);
 	}
 
 	/**
@@ -109,10 +109,10 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function clear_deleted_action_logs( $action_id ) {
+	public function clear_deleted_action_logs($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$wpdb->delete( $wpdb->actionscheduler_logs, [ 'action_id' => $action_id, ], [ '%d' ] );
+		$wpdb->delete($wpdb->actionscheduler_logs, ['action_id' => $action_id, ], ['%d']);
 	}
 
 	/**
@@ -120,27 +120,27 @@ 
 	 *
 	 * @param array $action_ids List of action ID.
 	 */
-	public function bulk_log_cancel_actions( $action_ids ) {
-		if ( empty( $action_ids ) ) {
+	public function bulk_log_cancel_actions($action_ids) {
+		if (empty($action_ids)) {
 			return;
 		}
 
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 		$date     = as_get_datetime_object();
-		$date_gmt = $date->format( 'Y-m-d H:i:s' );
-		ActionScheduler_TimezoneHelper::set_local_timezone( $date );
-		$date_local = $date->format( 'Y-m-d H:i:s' );
-		$message    = __( 'action canceled', 'action-scheduler' );
-		$format     = '(%d, ' . $wpdb->prepare( '%s, %s, %s', $message, $date_gmt, $date_local ) . ')';
+		$date_gmt = $date->format('Y-m-d H:i:s');
+		ActionScheduler_TimezoneHelper::set_local_timezone($date);
+		$date_local = $date->format('Y-m-d H:i:s');
+		$message    = __('action canceled', 'action-scheduler');
+		$format     = '(%d, ' . $wpdb->prepare('%s, %s, %s', $message, $date_gmt, $date_local) . ')';
 		$sql_query  = "INSERT {$wpdb->actionscheduler_logs} (action_id, message, log_date_gmt, log_date_local) VALUES ";
 		$value_rows = [];
 
-		foreach ( $action_ids as $action_id ) {
-			$value_rows[] = $wpdb->prepare( $format, $action_id );
+		foreach ($action_ids as $action_id) {
+			$value_rows[] = $wpdb->prepare($format, $action_id);
 		}
-		$sql_query .= implode( ',', $value_rows );
+		$sql_query .= implode(',', $value_rows);
 
-		$wpdb->query( $sql_query );
+		$wpdb->query($sql_query);
 	}
 }

includes/libraries/action-scheduler/classes/data-stores/ActionScheduler_wpPostStore_TaxonomyRegistrar.php

Indentation +16 added lines, -16 removed lines

@@ -5,22 +5,22 @@
  * @codeCoverageIgnore
  */
 class ActionScheduler_wpPostStore_TaxonomyRegistrar {
-	public function register() {
-		register_taxonomy( ActionScheduler_wpPostStore::GROUP_TAXONOMY, ActionScheduler_wpPostStore::POST_TYPE, $this->taxonomy_args() );
-	}
+    public function register() {
+        register_taxonomy( ActionScheduler_wpPostStore::GROUP_TAXONOMY, ActionScheduler_wpPostStore::POST_TYPE, $this->taxonomy_args() );
+    }
 
-	protected function taxonomy_args() {
-		$args = array(
-			'label' => __( 'Action Group', 'action-scheduler' ),
-			'public' => false,
-			'hierarchical' => false,
-			'show_admin_column' => true,
-			'query_var' => false,
-			'rewrite' => false,
-		);
+    protected function taxonomy_args() {
+        $args = array(
+            'label' => __( 'Action Group', 'action-scheduler' ),
+            'public' => false,
+            'hierarchical' => false,
+            'show_admin_column' => true,
+            'query_var' => false,
+            'rewrite' => false,
+        );
 
-		$args = apply_filters( 'action_scheduler_taxonomy_args', $args );
-		return $args;
-	}
+        $args = apply_filters( 'action_scheduler_taxonomy_args', $args );
+        return $args;
+    }
 }
- 
\ No newline at end of file
+    
\ No newline at end of file

Spacing +3 added lines, -3 removed lines

@@ -6,12 +6,12 @@ 
  */
 class ActionScheduler_wpPostStore_TaxonomyRegistrar {
 	public function register() {
-		register_taxonomy( ActionScheduler_wpPostStore::GROUP_TAXONOMY, ActionScheduler_wpPostStore::POST_TYPE, $this->taxonomy_args() );
+		register_taxonomy(ActionScheduler_wpPostStore::GROUP_TAXONOMY, ActionScheduler_wpPostStore::POST_TYPE, $this->taxonomy_args());
 	}
 
 	protected function taxonomy_args() {
 		$args = array(
-			'label' => __( 'Action Group', 'action-scheduler' ),
+			'label' => __('Action Group', 'action-scheduler'),
 			'public' => false,
 			'hierarchical' => false,
 			'show_admin_column' => true,
@@ -19,7 +19,7 @@ 
 			'rewrite' => false,
 		);
 
-		$args = apply_filters( 'action_scheduler_taxonomy_args', $args );
+		$args = apply_filters('action_scheduler_taxonomy_args', $args);
 		return $args;
 	}
 }

includes/libraries/action-scheduler/classes/data-stores/ActionScheduler_wpPostStore_PostTypeRegistrar.php

Indentation +40 added lines, -40 removed lines

@@ -5,46 +5,46 @@
  * @codeCoverageIgnore
  */
 class ActionScheduler_wpPostStore_PostTypeRegistrar {
-	public function register() {
-		register_post_type( ActionScheduler_wpPostStore::POST_TYPE, $this->post_type_args() );
-	}
+    public function register() {
+        register_post_type( ActionScheduler_wpPostStore::POST_TYPE, $this->post_type_args() );
+    }
 
-	/**
-	 * Build the args array for the post type definition
-	 *
-	 * @return array
-	 */
-	protected function post_type_args() {
-		$args = array(
-			'label' => __( 'Scheduled Actions', 'action-scheduler' ),
-			'description' => __( 'Scheduled actions are hooks triggered on a cetain date and time.', 'action-scheduler' ),
-			'public' => false,
-			'map_meta_cap' => true,
-			'hierarchical' => false,
-			'supports' => array('title', 'editor','comments'),
-			'rewrite' => false,
-			'query_var' => false,
-			'can_export' => true,
-			'ep_mask' => EP_NONE,
-			'labels' => array(
-				'name' => __( 'Scheduled Actions', 'action-scheduler' ),
-				'singular_name' => __( 'Scheduled Action', 'action-scheduler' ),
-				'menu_name' => _x( 'Scheduled Actions', 'Admin menu name', 'action-scheduler' ),
-				'add_new' => __( 'Add', 'action-scheduler' ),
-				'add_new_item' => __( 'Add New Scheduled Action', 'action-scheduler' ),
-				'edit' => __( 'Edit', 'action-scheduler' ),
-				'edit_item' => __( 'Edit Scheduled Action', 'action-scheduler' ),
-				'new_item' => __( 'New Scheduled Action', 'action-scheduler' ),
-				'view' => __( 'View Action', 'action-scheduler' ),
-				'view_item' => __( 'View Action', 'action-scheduler' ),
-				'search_items' => __( 'Search Scheduled Actions', 'action-scheduler' ),
-				'not_found' => __( 'No actions found', 'action-scheduler' ),
-				'not_found_in_trash' => __( 'No actions found in trash', 'action-scheduler' ),
-			),
-		);
+    /**
+     * Build the args array for the post type definition
+     *
+     * @return array
+     */
+    protected function post_type_args() {
+        $args = array(
+            'label' => __( 'Scheduled Actions', 'action-scheduler' ),
+            'description' => __( 'Scheduled actions are hooks triggered on a cetain date and time.', 'action-scheduler' ),
+            'public' => false,
+            'map_meta_cap' => true,
+            'hierarchical' => false,
+            'supports' => array('title', 'editor','comments'),
+            'rewrite' => false,
+            'query_var' => false,
+            'can_export' => true,
+            'ep_mask' => EP_NONE,
+            'labels' => array(
+                'name' => __( 'Scheduled Actions', 'action-scheduler' ),
+                'singular_name' => __( 'Scheduled Action', 'action-scheduler' ),
+                'menu_name' => _x( 'Scheduled Actions', 'Admin menu name', 'action-scheduler' ),
+                'add_new' => __( 'Add', 'action-scheduler' ),
+                'add_new_item' => __( 'Add New Scheduled Action', 'action-scheduler' ),
+                'edit' => __( 'Edit', 'action-scheduler' ),
+                'edit_item' => __( 'Edit Scheduled Action', 'action-scheduler' ),
+                'new_item' => __( 'New Scheduled Action', 'action-scheduler' ),
+                'view' => __( 'View Action', 'action-scheduler' ),
+                'view_item' => __( 'View Action', 'action-scheduler' ),
+                'search_items' => __( 'Search Scheduled Actions', 'action-scheduler' ),
+                'not_found' => __( 'No actions found', 'action-scheduler' ),
+                'not_found_in_trash' => __( 'No actions found in trash', 'action-scheduler' ),
+            ),
+        );
 
-		$args = apply_filters('action_scheduler_post_type_args', $args);
-		return $args;
-	}
+        $args = apply_filters('action_scheduler_post_type_args', $args);
+        return $args;
+    }
 }
- 
\ No newline at end of file
+    
\ No newline at end of file

Spacing +17 added lines, -17 removed lines

@@ -6,7 +6,7 @@ 
  */
 class ActionScheduler_wpPostStore_PostTypeRegistrar {
 	public function register() {
-		register_post_type( ActionScheduler_wpPostStore::POST_TYPE, $this->post_type_args() );
+		register_post_type(ActionScheduler_wpPostStore::POST_TYPE, $this->post_type_args());
 	}
 
 	/**
@@ -16,30 +16,30 @@ 
 	 */
 	protected function post_type_args() {
 		$args = array(
-			'label' => __( 'Scheduled Actions', 'action-scheduler' ),
-			'description' => __( 'Scheduled actions are hooks triggered on a cetain date and time.', 'action-scheduler' ),
+			'label' => __('Scheduled Actions', 'action-scheduler'),
+			'description' => __('Scheduled actions are hooks triggered on a cetain date and time.', 'action-scheduler'),
 			'public' => false,
 			'map_meta_cap' => true,
 			'hierarchical' => false,
-			'supports' => array('title', 'editor','comments'),
+			'supports' => array('title', 'editor', 'comments'),
 			'rewrite' => false,
 			'query_var' => false,
 			'can_export' => true,
 			'ep_mask' => EP_NONE,
 			'labels' => array(
-				'name' => __( 'Scheduled Actions', 'action-scheduler' ),
-				'singular_name' => __( 'Scheduled Action', 'action-scheduler' ),
-				'menu_name' => _x( 'Scheduled Actions', 'Admin menu name', 'action-scheduler' ),
-				'add_new' => __( 'Add', 'action-scheduler' ),
-				'add_new_item' => __( 'Add New Scheduled Action', 'action-scheduler' ),
-				'edit' => __( 'Edit', 'action-scheduler' ),
-				'edit_item' => __( 'Edit Scheduled Action', 'action-scheduler' ),
-				'new_item' => __( 'New Scheduled Action', 'action-scheduler' ),
-				'view' => __( 'View Action', 'action-scheduler' ),
-				'view_item' => __( 'View Action', 'action-scheduler' ),
-				'search_items' => __( 'Search Scheduled Actions', 'action-scheduler' ),
-				'not_found' => __( 'No actions found', 'action-scheduler' ),
-				'not_found_in_trash' => __( 'No actions found in trash', 'action-scheduler' ),
+				'name' => __('Scheduled Actions', 'action-scheduler'),
+				'singular_name' => __('Scheduled Action', 'action-scheduler'),
+				'menu_name' => _x('Scheduled Actions', 'Admin menu name', 'action-scheduler'),
+				'add_new' => __('Add', 'action-scheduler'),
+				'add_new_item' => __('Add New Scheduled Action', 'action-scheduler'),
+				'edit' => __('Edit', 'action-scheduler'),
+				'edit_item' => __('Edit Scheduled Action', 'action-scheduler'),
+				'new_item' => __('New Scheduled Action', 'action-scheduler'),
+				'view' => __('View Action', 'action-scheduler'),
+				'view_item' => __('View Action', 'action-scheduler'),
+				'search_items' => __('Search Scheduled Actions', 'action-scheduler'),
+				'not_found' => __('No actions found', 'action-scheduler'),
+				'not_found_in_trash' => __('No actions found in trash', 'action-scheduler'),
 			),
 		);
 

includes/libraries/action-scheduler/classes/data-stores/ActionScheduler_DBStore.php

Indentation +833 added lines, -833 removed lines

@@ -9,837 +9,837 @@
  */
 class ActionScheduler_DBStore extends ActionScheduler_Store {
 
-	/** @var int */
-	protected static $max_args_length = 8000;
-
-	/** @var int */
-	protected static $max_index_length = 191;
-
-	/**
-	 * Initialize the data store
-	 *
-	 * @codeCoverageIgnore
-	 */
-	public function init() {
-		$table_maker = new ActionScheduler_StoreSchema();
-		$table_maker->register_tables();
-	}
-
-	/**
-	 * Save an action.
-	 *
-	 * @param ActionScheduler_Action $action Action object.
-	 * @param DateTime               $date Optional schedule date. Default null.
-	 *
-	 * @return int Action ID.
-	 */
-	public function save_action( ActionScheduler_Action $action, \DateTime $date = null ) {
-		try {
-
-			$this->validate_action( $action );
-
-			/** @var \wpdb $wpdb */
-			global $wpdb;
-			$data = [
-				'hook'                 => $action->get_hook(),
-				'status'               => ( $action->is_finished() ? self::STATUS_COMPLETE : self::STATUS_PENDING ),
-				'scheduled_date_gmt'   => $this->get_scheduled_date_string( $action, $date ),
-				'scheduled_date_local' => $this->get_scheduled_date_string_local( $action, $date ),
-				'schedule'             => serialize( $action->get_schedule() ),
-				'group_id'             => $this->get_group_id( $action->get_group() ),
-			];
-			$args = wp_json_encode( $action->get_args() );
-			if ( strlen( $args ) <= static::$max_index_length ) {
-				$data['args'] = $args;
-			} else {
-				$data['args']          = $this->hash_args( $args );
-				$data['extended_args'] = $args;
-			}
-
-			$table_name = ! empty( $wpdb->actionscheduler_actions ) ? $wpdb->actionscheduler_actions : $wpdb->prefix . 'actionscheduler_actions';
-			$wpdb->insert( $table_name, $data );
-			$action_id = $wpdb->insert_id;
-
-			if ( is_wp_error( $action_id ) ) {
-				throw new RuntimeException( $action_id->get_error_message() );
-			}
-			elseif ( empty( $action_id ) ) {
-				throw new RuntimeException( $wpdb->last_error ? $wpdb->last_error : __( 'Database error.', 'action-scheduler' ) );
-			}
-
-			do_action( 'action_scheduler_stored_action', $action_id );
-
-			return $action_id;
-		} catch ( \Exception $e ) {
-			/* translators: %s: error message */
-			throw new \RuntimeException( sprintf( __( 'Error saving action: %s', 'action-scheduler' ), $e->getMessage() ), 0 );
-		}
-	}
-
-	/**
-	 * Generate a hash from json_encoded $args using MD5 as this isn't for security.
-	 *
-	 * @param string $args JSON encoded action args.
-	 * @return string
-	 */
-	protected function hash_args( $args ) {
-		return md5( $args );
-	}
-
-	/**
-	 * Get action args query param value from action args.
-	 *
-	 * @param array $args Action args.
-	 * @return string
-	 */
-	protected function get_args_for_query( $args ) {
-		$encoded = wp_json_encode( $args );
-		if ( strlen( $encoded ) <= static::$max_index_length ) {
-			return $encoded;
-		}
-		return $this->hash_args( $encoded );
-	}
-	/**
-	 * Get a group's ID based on its name/slug.
-	 *
-	 * @param string $slug The string name of a group.
-	 * @param bool $create_if_not_exists Whether to create the group if it does not already exist. Default, true - create the group.
-	 *
-	 * @return int The group's ID, if it exists or is created, or 0 if it does not exist and is not created.
-	 */
-	protected function get_group_id( $slug, $create_if_not_exists = true ) {
-		if ( empty( $slug ) ) {
-			return 0;
-		}
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$group_id = (int) $wpdb->get_var( $wpdb->prepare( "SELECT group_id FROM {$wpdb->actionscheduler_groups} WHERE slug=%s", $slug ) );
-		if ( empty( $group_id ) && $create_if_not_exists ) {
-			$group_id = $this->create_group( $slug );
-		}
-
-		return $group_id;
-	}
-
-	/**
-	 * Create an action group.
-	 *
-	 * @param string $slug Group slug.
-	 *
-	 * @return int Group ID.
-	 */
-	protected function create_group( $slug ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$wpdb->insert( $wpdb->actionscheduler_groups, [ 'slug' => $slug ] );
-
-		return (int) $wpdb->insert_id;
-	}
-
-	/**
-	 * Retrieve an action.
-	 *
-	 * @param int $action_id Action ID.
-	 *
-	 * @return ActionScheduler_Action
-	 */
-	public function fetch_action( $action_id ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$data = $wpdb->get_row( $wpdb->prepare(
-			"SELECT a.*, g.slug AS `group` FROM {$wpdb->actionscheduler_actions} a LEFT JOIN {$wpdb->actionscheduler_groups} g ON a.group_id=g.group_id WHERE a.action_id=%d",
-			$action_id
-		) );
-
-		if ( empty( $data ) ) {
-			return $this->get_null_action();
-		}
-
-		if ( ! empty( $data->extended_args ) ) {
-			$data->args = $data->extended_args;
-			unset( $data->extended_args );
-		}
-
-		try {
-			$action = $this->make_action_from_db_record( $data );
-		} catch ( ActionScheduler_InvalidActionException $exception ) {
-			do_action( 'action_scheduler_failed_fetch_action', $action_id, $exception );
-			return $this->get_null_action();
-		}
-
-		return $action;
-	}
-
-	/**
-	 * Create a null action.
-	 *
-	 * @return ActionScheduler_NullAction
-	 */
-	protected function get_null_action() {
-		return new ActionScheduler_NullAction();
-	}
-
-	/**
-	 * Create an action from a database record.
-	 *
-	 * @param object $data Action database record.
-	 *
-	 * @return ActionScheduler_Action|ActionScheduler_CanceledAction|ActionScheduler_FinishedAction
-	 */
-	protected function make_action_from_db_record( $data ) {
-
-		$hook     = $data->hook;
-		$args     = json_decode( $data->args, true );
-		$schedule = unserialize( $data->schedule );
-
-		$this->validate_args( $args, $data->action_id );
-		$this->validate_schedule( $schedule, $data->action_id );
-
-		if ( empty( $schedule ) ) {
-			$schedule = new ActionScheduler_NullSchedule();
-		}
-		$group = $data->group ? $data->group : '';
-
-		return ActionScheduler::factory()->get_stored_action( $data->status, $data->hook, $args, $schedule, $group );
-	}
-
-	/**
-	 * Find an action.
-	 *
-	 * @param string $hook Action hook.
-	 * @param array  $params Parameters of the action to find.
-	 *
-	 * @return string|null ID of the next action matching the criteria or NULL if not found.
-	 */
-	public function find_action( $hook, $params = [] ) {
-		$params = wp_parse_args( $params, [
-			'args'   => null,
-			'status' => self::STATUS_PENDING,
-			'group'  => '',
-		] );
-
-		/** @var wpdb $wpdb */
-		global $wpdb;
-		$query = "SELECT a.action_id FROM {$wpdb->actionscheduler_actions} a";
-		$args  = [];
-		if ( ! empty( $params[ 'group' ] ) ) {
-			$query  .= " INNER JOIN {$wpdb->actionscheduler_groups} g ON g.group_id=a.group_id AND g.slug=%s";
-			$args[] = $params[ 'group' ];
-		}
-		$query  .= " WHERE a.hook=%s";
-		$args[] = $hook;
-		if ( ! is_null( $params[ 'args' ] ) ) {
-			$query  .= " AND a.args=%s";
-			$args[] = $this->get_args_for_query( $params[ 'args' ] );
-		}
-
-		$order = 'ASC';
-		if ( ! empty( $params[ 'status' ] ) ) {
-			$query  .= " AND a.status=%s";
-			$args[] = $params[ 'status' ];
-
-			if ( self::STATUS_PENDING == $params[ 'status' ] ) {
-				$order = 'ASC'; // Find the next action that matches.
-			} else {
-				$order = 'DESC'; // Find the most recent action that matches.
-			}
-		}
-
-		$query .= " ORDER BY scheduled_date_gmt $order LIMIT 1";
-
-		$query = $wpdb->prepare( $query, $args );
-
-		$id = $wpdb->get_var( $query );
-
-		return $id;
-	}
-
-	/**
-	 * Returns the SQL statement to query (or count) actions.
-	 *
-	 * @param array  $query Filtering options.
-	 * @param string $select_or_count  Whether the SQL should select and return the IDs or just the row count.
-	 *
-	 * @return string SQL statement already properly escaped.
-	 */
-	protected function get_query_actions_sql( array $query, $select_or_count = 'select' ) {
-
-		if ( ! in_array( $select_or_count, array( 'select', 'count' ) ) ) {
-			throw new InvalidArgumentException( __( 'Invalid value for select or count parameter. Cannot query actions.', 'action-scheduler' ) );
-		}
-
-		$query = wp_parse_args( $query, [
-			'hook'             => '',
-			'args'             => null,
-			'date'             => null,
-			'date_compare'     => '<=',
-			'modified'         => null,
-			'modified_compare' => '<=',
-			'group'            => '',
-			'status'           => '',
-			'claimed'          => null,
-			'per_page'         => 5,
-			'offset'           => 0,
-			'orderby'          => 'date',
-			'order'            => 'ASC',
-		] );
-
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$sql  = ( 'count' === $select_or_count ) ? 'SELECT count(a.action_id)' : 'SELECT a.action_id';
-		$sql .= " FROM {$wpdb->actionscheduler_actions} a";
-		$sql_params = [];
-
-		if ( ! empty( $query[ 'group' ] ) || 'group' === $query[ 'orderby' ] ) {
-			$sql .= " LEFT JOIN {$wpdb->actionscheduler_groups} g ON g.group_id=a.group_id";
-		}
-
-		$sql .= " WHERE 1=1";
-
-		if ( ! empty( $query[ 'group' ] ) ) {
-			$sql          .= " AND g.slug=%s";
-			$sql_params[] = $query[ 'group' ];
-		}
-
-		if ( $query[ 'hook' ] ) {
-			$sql          .= " AND a.hook=%s";
-			$sql_params[] = $query[ 'hook' ];
-		}
-		if ( ! is_null( $query[ 'args' ] ) ) {
-			$sql          .= " AND a.args=%s";
-			$sql_params[] = $this->get_args_for_query( $query[ 'args' ] );
-		}
-
-		if ( $query[ 'status' ] ) {
-			$sql          .= " AND a.status=%s";
-			$sql_params[] = $query[ 'status' ];
-		}
-
-		if ( $query[ 'date' ] instanceof \DateTime ) {
-			$date = clone $query[ 'date' ];
-			$date->setTimezone( new \DateTimeZone( 'UTC' ) );
-			$date_string  = $date->format( 'Y-m-d H:i:s' );
-			$comparator   = $this->validate_sql_comparator( $query[ 'date_compare' ] );
-			$sql          .= " AND a.scheduled_date_gmt $comparator %s";
-			$sql_params[] = $date_string;
-		}
-
-		if ( $query[ 'modified' ] instanceof \DateTime ) {
-			$modified = clone $query[ 'modified' ];
-			$modified->setTimezone( new \DateTimeZone( 'UTC' ) );
-			$date_string  = $modified->format( 'Y-m-d H:i:s' );
-			$comparator   = $this->validate_sql_comparator( $query[ 'modified_compare' ] );
-			$sql          .= " AND a.last_attempt_gmt $comparator %s";
-			$sql_params[] = $date_string;
-		}
-
-		if ( $query[ 'claimed' ] === true ) {
-			$sql .= " AND a.claim_id != 0";
-		} elseif ( $query[ 'claimed' ] === false ) {
-			$sql .= " AND a.claim_id = 0";
-		} elseif ( ! is_null( $query[ 'claimed' ] ) ) {
-			$sql          .= " AND a.claim_id = %d";
-			$sql_params[] = $query[ 'claimed' ];
-		}
-
-		if ( ! empty( $query['search'] ) ) {
-			$sql .= " AND (a.hook LIKE %s OR (a.extended_args IS NULL AND a.args LIKE %s) OR a.extended_args LIKE %s";
-			for( $i = 0; $i < 3; $i++ ) {
-				$sql_params[] = sprintf( '%%%s%%', $query['search'] );
-			}
-
-			$search_claim_id = (int) $query['search'];
-			if ( $search_claim_id ) {
-				$sql .= ' OR a.claim_id = %d';
-				$sql_params[] = $search_claim_id;
-			}
-
-			$sql .= ')';
-		}
-
-		if ( 'select' === $select_or_count ) {
-			switch ( $query['orderby'] ) {
-				case 'hook':
-					$orderby = 'a.hook';
-					break;
-				case 'group':
-					$orderby = 'g.slug';
-					break;
-				case 'modified':
-					$orderby = 'a.last_attempt_gmt';
-					break;
-				case 'date':
-				default:
-					$orderby = 'a.scheduled_date_gmt';
-					break;
-			}
-			if ( strtoupper( $query[ 'order' ] ) == 'ASC' ) {
-				$order = 'ASC';
-			} else {
-				$order = 'DESC';
-			}
-			$sql .= " ORDER BY $orderby $order";
-			if ( $query[ 'per_page' ] > 0 ) {
-				$sql          .= " LIMIT %d, %d";
-				$sql_params[] = $query[ 'offset' ];
-				$sql_params[] = $query[ 'per_page' ];
-			}
-		}
-
-		if ( ! empty( $sql_params ) ) {
-			$sql = $wpdb->prepare( $sql, $sql_params );
-		}
-
-		return $sql;
-	}
-
-	/**
-	 * Query for action count of list of action IDs.
-	 *
-	 * @param array  $query Query parameters.
-	 * @param string $query_type Whether to select or count the results. Default, select.
-	 *
-	 * @return null|string|array The IDs of actions matching the query
-	 */
-	public function query_actions( $query = [], $query_type = 'select' ) {
-		/** @var wpdb $wpdb */
-		global $wpdb;
-
-		$sql = $this->get_query_actions_sql( $query, $query_type );
-
-		return ( 'count' === $query_type ) ? $wpdb->get_var( $sql ) : $wpdb->get_col( $sql );
-	}
-
-	/**
-	 * Get a count of all actions in the store, grouped by status.
-	 *
-	 * @return array Set of 'status' => int $count pairs for statuses with 1 or more actions of that status.
-	 */
-	public function action_counts() {
-		global $wpdb;
-
-		$sql  = "SELECT a.status, count(a.status) as 'count'";
-		$sql .= " FROM {$wpdb->actionscheduler_actions} a";
-		$sql .= " GROUP BY a.status";
-
-		$actions_count_by_status = array();
-		$action_stati_and_labels = $this->get_status_labels();
-
-		foreach ( $wpdb->get_results( $sql ) as $action_data ) {
-			// Ignore any actions with invalid status
-			if ( array_key_exists( $action_data->status, $action_stati_and_labels ) ) {
-				$actions_count_by_status[ $action_data->status ] = $action_data->count;
-			}
-		}
-
-		return $actions_count_by_status;
-	}
-
-	/**
-	 * Cancel an action.
-	 *
-	 * @param int $action_id Action ID.
-	 *
-	 * @return void
-	 */
-	public function cancel_action( $action_id ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-
-		$updated = $wpdb->update(
-			$wpdb->actionscheduler_actions,
-			[ 'status' => self::STATUS_CANCELED ],
-			[ 'action_id' => $action_id ],
-			[ '%s' ],
-			[ '%d' ]
-		);
-		if ( empty( $updated ) ) {
-			/* translators: %s: action ID */
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
-		}
-		do_action( 'action_scheduler_canceled_action', $action_id );
-	}
-
-	/**
-	 * Cancel pending actions by hook.
-	 *
-	 * @since 3.0.0
-	 *
-	 * @param string $hook Hook name.
-	 *
-	 * @return void
-	 */
-	public function cancel_actions_by_hook( $hook ) {
-		$this->bulk_cancel_actions( [ 'hook' => $hook ] );
-	}
-
-	/**
-	 * Cancel pending actions by group.
-	 *
-	 * @param string $group Group slug.
-	 *
-	 * @return void
-	 */
-	public function cancel_actions_by_group( $group ) {
-		$this->bulk_cancel_actions( [ 'group' => $group ] );
-	}
-
-	/**
-	 * Bulk cancel actions.
-	 *
-	 * @since 3.0.0
-	 *
-	 * @param array $query_args Query parameters.
-	 */
-	protected function bulk_cancel_actions( $query_args ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-
-		if ( ! is_array( $query_args ) ) {
-			return;
-		}
-
-		// Don't cancel actions that are already canceled.
-		if ( isset( $query_args['status'] ) && $query_args['status'] == self::STATUS_CANCELED ) {
-			return;
-		}
-
-		$action_ids = true;
-		$query_args = wp_parse_args(
-			$query_args,
-			[
-				'per_page' => 1000,
-				'status' => self::STATUS_PENDING,
-			]
-		);
-
-		while ( $action_ids ) {
-			$action_ids = $this->query_actions( $query_args );
-			if ( empty( $action_ids ) ) {
-				break;
-			}
-
-			$format     = array_fill( 0, count( $action_ids ), '%d' );
-			$query_in   = '(' . implode( ',', $format ) . ')';
-			$parameters = $action_ids;
-			array_unshift( $parameters, self::STATUS_CANCELED );
-
-			$wpdb->query(
-				$wpdb->prepare( // wpcs: PreparedSQLPlaceholders replacement count ok.
-					"UPDATE {$wpdb->actionscheduler_actions} SET status = %s WHERE action_id IN {$query_in}",
-					$parameters
-				)
-			);
-
-			do_action( 'action_scheduler_bulk_cancel_actions', $action_ids );
-		}
-	}
-
-	/**
-	 * Delete an action.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function delete_action( $action_id ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$deleted = $wpdb->delete( $wpdb->actionscheduler_actions, [ 'action_id' => $action_id ], [ '%d' ] );
-		if ( empty( $deleted ) ) {
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
-		}
-		do_action( 'action_scheduler_deleted_action', $action_id );
-	}
-
-	/**
-	 * Get the schedule date for an action.
-	 *
-	 * @param string $action_id Action ID.
-	 *
-	 * @throws \InvalidArgumentException
-	 * @return \DateTime The local date the action is scheduled to run, or the date that it ran.
-	 */
-	public function get_date( $action_id ) {
-		$date = $this->get_date_gmt( $action_id );
-		ActionScheduler_TimezoneHelper::set_local_timezone( $date );
-		return $date;
-	}
-
-	/**
-	 * Get the GMT schedule date for an action.
-	 *
-	 * @param int $action_id Action ID.
-	 *
-	 * @throws \InvalidArgumentException
-	 * @return \DateTime The GMT date the action is scheduled to run, or the date that it ran.
-	 */
-	protected function get_date_gmt( $action_id ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$record = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d", $action_id ) );
-		if ( empty( $record ) ) {
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
-		}
-		if ( $record->status == self::STATUS_PENDING ) {
-			return as_get_datetime_object( $record->scheduled_date_gmt );
-		} else {
-			return as_get_datetime_object( $record->last_attempt_gmt );
-		}
-	}
-
-	/**
-	 * Stake a claim on actions.
-	 *
-	 * @param int       $max_actions Maximum number of action to include in claim.
-	 * @param \DateTime $before_date Jobs must be schedule before this date. Defaults to now.
-	 *
-	 * @return ActionScheduler_ActionClaim
-	 */
-	public function stake_claim( $max_actions = 10, \DateTime $before_date = null, $hooks = array(), $group = '' ) {
-		$claim_id = $this->generate_claim_id();
-		$this->claim_actions( $claim_id, $max_actions, $before_date, $hooks, $group );
-		$action_ids = $this->find_actions_by_claim_id( $claim_id );
-
-		return new ActionScheduler_ActionClaim( $claim_id, $action_ids );
-	}
-
-	/**
-	 * Generate a new action claim.
-	 *
-	 * @return int Claim ID.
-	 */
-	protected function generate_claim_id() {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$now = as_get_datetime_object();
-		$wpdb->insert( $wpdb->actionscheduler_claims, [ 'date_created_gmt' => $now->format( 'Y-m-d H:i:s' ) ] );
-
-		return $wpdb->insert_id;
-	}
-
-	/**
-	 * Mark actions claimed.
-	 *
-	 * @param string    $claim_id Claim Id.
-	 * @param int       $limit Number of action to include in claim.
-	 * @param \DateTime $before_date Should use UTC timezone.
-	 *
-	 * @return int The number of actions that were claimed.
-	 * @throws \RuntimeException
-	 */
-	protected function claim_actions( $claim_id, $limit, \DateTime $before_date = null, $hooks = array(), $group = '' ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-
-		$now  = as_get_datetime_object();
-		$date = is_null( $before_date ) ? $now : clone $before_date;
-
-		// can't use $wpdb->update() because of the <= condition
-		$update = "UPDATE {$wpdb->actionscheduler_actions} SET claim_id=%d, last_attempt_gmt=%s, last_attempt_local=%s";
-		$params = array(
-			$claim_id,
-			$now->format( 'Y-m-d H:i:s' ),
-			current_time( 'mysql' ),
-		);
-
-		$where    = "WHERE claim_id = 0 AND scheduled_date_gmt <= %s AND status=%s";
-		$params[] = $date->format( 'Y-m-d H:i:s' );
-		$params[] = self::STATUS_PENDING;
-
-		if ( ! empty( $hooks ) ) {
-			$placeholders = array_fill( 0, count( $hooks ), '%s' );
-			$where       .= ' AND hook IN (' . join( ', ', $placeholders ) . ')';
-			$params       = array_merge( $params, array_values( $hooks ) );
-		}
-
-		if ( ! empty( $group ) ) {
-
-			$group_id = $this->get_group_id( $group, false );
-
-			// throw exception if no matching group found, this matches ActionScheduler_wpPostStore's behaviour
-			if ( empty( $group_id ) ) {
-				/* translators: %s: group name */
-				throw new InvalidArgumentException( sprintf( __( 'The group "%s" does not exist.', 'action-scheduler' ), $group ) );
-			}
-
-			$where   .= ' AND group_id = %d';
-			$params[] = $group_id;
-		}
-
-		$order    = "ORDER BY attempts ASC, scheduled_date_gmt ASC, action_id ASC LIMIT %d";
-		$params[] = $limit;
-
-		$sql = $wpdb->prepare( "{$update} {$where} {$order}", $params );
-
-		$rows_affected = $wpdb->query( $sql );
-		if ( $rows_affected === false ) {
-			throw new \RuntimeException( __( 'Unable to claim actions. Database error.', 'action-scheduler' ) );
-		}
-
-		return (int) $rows_affected;
-	}
-
-	/**
-	 * Get the number of active claims.
-	 *
-	 * @return int
-	 */
-	public function get_claim_count() {
-		global $wpdb;
-
-		$sql = "SELECT COUNT(DISTINCT claim_id) FROM {$wpdb->actionscheduler_actions} WHERE claim_id != 0 AND status IN ( %s, %s)";
-		$sql = $wpdb->prepare( $sql, [ self::STATUS_PENDING, self::STATUS_RUNNING ] );
-
-		return (int) $wpdb->get_var( $sql );
-	}
-
-	/**
-	 * Return an action's claim ID, as stored in the claim_id column.
-	 *
-	 * @param string $action_id Action ID.
-	 * @return mixed
-	 */
-	public function get_claim_id( $action_id ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-
-		$sql = "SELECT claim_id FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d";
-		$sql = $wpdb->prepare( $sql, $action_id );
-
-		return (int) $wpdb->get_var( $sql );
-	}
-
-	/**
-	 * Retrieve the action IDs of action in a claim.
-	 *
-	 * @param string $claim_id Claim ID.
-	 *
-	 * @return int[]
-	 */
-	public function find_actions_by_claim_id( $claim_id ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-
-		$sql = "SELECT action_id FROM {$wpdb->actionscheduler_actions} WHERE claim_id=%d";
-		$sql = $wpdb->prepare( $sql, $claim_id );
-
-		$action_ids = $wpdb->get_col( $sql );
-
-		return array_map( 'intval', $action_ids );
-	}
-
-	/**
-	 * Release actions from a claim and delete the claim.
-	 *
-	 * @param ActionScheduler_ActionClaim $claim Claim object.
-	 */
-	public function release_claim( ActionScheduler_ActionClaim $claim ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$wpdb->update( $wpdb->actionscheduler_actions, [ 'claim_id' => 0 ], [ 'claim_id' => $claim->get_id() ], [ '%d' ], [ '%d' ] );
-		$wpdb->delete( $wpdb->actionscheduler_claims, [ 'claim_id' => $claim->get_id() ], [ '%d' ] );
-	}
-
-	/**
-	 * Remove the claim from an action.
-	 *
-	 * @param int $action_id Action ID.
-	 *
-	 * @return void
-	 */
-	public function unclaim_action( $action_id ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$wpdb->update(
-			$wpdb->actionscheduler_actions,
-			[ 'claim_id' => 0 ],
-			[ 'action_id' => $action_id ],
-			[ '%s' ],
-			[ '%d' ]
-		);
-	}
-
-	/**
-	 * Mark an action as failed.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function mark_failure( $action_id ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$updated = $wpdb->update(
-			$wpdb->actionscheduler_actions,
-			[ 'status' => self::STATUS_FAILED ],
-			[ 'action_id' => $action_id ],
-			[ '%s' ],
-			[ '%d' ]
-		);
-		if ( empty( $updated ) ) {
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
-		}
-	}
-
-	/**
-	 * Add execution message to action log.
-	 *
-	 * @param int $action_id Action ID.
-	 *
-	 * @return void
-	 */
-	public function log_execution( $action_id ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-
-		$sql = "UPDATE {$wpdb->actionscheduler_actions} SET attempts = attempts+1, status=%s, last_attempt_gmt = %s, last_attempt_local = %s WHERE action_id = %d";
-		$sql = $wpdb->prepare( $sql, self::STATUS_RUNNING, current_time( 'mysql', true ), current_time( 'mysql' ), $action_id );
-		$wpdb->query( $sql );
-	}
-
-	/**
-	 * Mark an action as complete.
-	 *
-	 * @param int $action_id Action ID.
-	 *
-	 * @return void
-	 */
-	public function mark_complete( $action_id ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$updated = $wpdb->update(
-			$wpdb->actionscheduler_actions,
-			[
-				'status'             => self::STATUS_COMPLETE,
-				'last_attempt_gmt'   => current_time( 'mysql', true ),
-				'last_attempt_local' => current_time( 'mysql' ),
-			],
-			[ 'action_id' => $action_id ],
-			[ '%s' ],
-			[ '%d' ]
-		);
-		if ( empty( $updated ) ) {
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
-		}
-	}
-
-	/**
-	 * Get an action's status.
-	 *
-	 * @param int $action_id Action ID.
-	 *
-	 * @return string
-	 */
-	public function get_status( $action_id ) {
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$sql    = "SELECT status FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d";
-		$sql    = $wpdb->prepare( $sql, $action_id );
-		$status = $wpdb->get_var( $sql );
-
-		if ( $status === null ) {
-			throw new \InvalidArgumentException( __( 'Invalid action ID. No status found.', 'action-scheduler' ) );
-		} elseif ( empty( $status ) ) {
-			throw new \RuntimeException( __( 'Unknown status found for action.', 'action-scheduler' ) );
-		} else {
-			return $status;
-		}
-	}
+    /** @var int */
+    protected static $max_args_length = 8000;
+
+    /** @var int */
+    protected static $max_index_length = 191;
+
+    /**
+     * Initialize the data store
+     *
+     * @codeCoverageIgnore
+     */
+    public function init() {
+        $table_maker = new ActionScheduler_StoreSchema();
+        $table_maker->register_tables();
+    }
+
+    /**
+     * Save an action.
+     *
+     * @param ActionScheduler_Action $action Action object.
+     * @param DateTime               $date Optional schedule date. Default null.
+     *
+     * @return int Action ID.
+     */
+    public function save_action( ActionScheduler_Action $action, \DateTime $date = null ) {
+        try {
+
+            $this->validate_action( $action );
+
+            /** @var \wpdb $wpdb */
+            global $wpdb;
+            $data = [
+                'hook'                 => $action->get_hook(),
+                'status'               => ( $action->is_finished() ? self::STATUS_COMPLETE : self::STATUS_PENDING ),
+                'scheduled_date_gmt'   => $this->get_scheduled_date_string( $action, $date ),
+                'scheduled_date_local' => $this->get_scheduled_date_string_local( $action, $date ),
+                'schedule'             => serialize( $action->get_schedule() ),
+                'group_id'             => $this->get_group_id( $action->get_group() ),
+            ];
+            $args = wp_json_encode( $action->get_args() );
+            if ( strlen( $args ) <= static::$max_index_length ) {
+                $data['args'] = $args;
+            } else {
+                $data['args']          = $this->hash_args( $args );
+                $data['extended_args'] = $args;
+            }
+
+            $table_name = ! empty( $wpdb->actionscheduler_actions ) ? $wpdb->actionscheduler_actions : $wpdb->prefix . 'actionscheduler_actions';
+            $wpdb->insert( $table_name, $data );
+            $action_id = $wpdb->insert_id;
+
+            if ( is_wp_error( $action_id ) ) {
+                throw new RuntimeException( $action_id->get_error_message() );
+            }
+            elseif ( empty( $action_id ) ) {
+                throw new RuntimeException( $wpdb->last_error ? $wpdb->last_error : __( 'Database error.', 'action-scheduler' ) );
+            }
+
+            do_action( 'action_scheduler_stored_action', $action_id );
+
+            return $action_id;
+        } catch ( \Exception $e ) {
+            /* translators: %s: error message */
+            throw new \RuntimeException( sprintf( __( 'Error saving action: %s', 'action-scheduler' ), $e->getMessage() ), 0 );
+        }
+    }
+
+    /**
+     * Generate a hash from json_encoded $args using MD5 as this isn't for security.
+     *
+     * @param string $args JSON encoded action args.
+     * @return string
+     */
+    protected function hash_args( $args ) {
+        return md5( $args );
+    }
+
+    /**
+     * Get action args query param value from action args.
+     *
+     * @param array $args Action args.
+     * @return string
+     */
+    protected function get_args_for_query( $args ) {
+        $encoded = wp_json_encode( $args );
+        if ( strlen( $encoded ) <= static::$max_index_length ) {
+            return $encoded;
+        }
+        return $this->hash_args( $encoded );
+    }
+    /**
+     * Get a group's ID based on its name/slug.
+     *
+     * @param string $slug The string name of a group.
+     * @param bool $create_if_not_exists Whether to create the group if it does not already exist. Default, true - create the group.
+     *
+     * @return int The group's ID, if it exists or is created, or 0 if it does not exist and is not created.
+     */
+    protected function get_group_id( $slug, $create_if_not_exists = true ) {
+        if ( empty( $slug ) ) {
+            return 0;
+        }
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $group_id = (int) $wpdb->get_var( $wpdb->prepare( "SELECT group_id FROM {$wpdb->actionscheduler_groups} WHERE slug=%s", $slug ) );
+        if ( empty( $group_id ) && $create_if_not_exists ) {
+            $group_id = $this->create_group( $slug );
+        }
+
+        return $group_id;
+    }
+
+    /**
+     * Create an action group.
+     *
+     * @param string $slug Group slug.
+     *
+     * @return int Group ID.
+     */
+    protected function create_group( $slug ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $wpdb->insert( $wpdb->actionscheduler_groups, [ 'slug' => $slug ] );
+
+        return (int) $wpdb->insert_id;
+    }
+
+    /**
+     * Retrieve an action.
+     *
+     * @param int $action_id Action ID.
+     *
+     * @return ActionScheduler_Action
+     */
+    public function fetch_action( $action_id ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $data = $wpdb->get_row( $wpdb->prepare(
+            "SELECT a.*, g.slug AS `group` FROM {$wpdb->actionscheduler_actions} a LEFT JOIN {$wpdb->actionscheduler_groups} g ON a.group_id=g.group_id WHERE a.action_id=%d",
+            $action_id
+        ) );
+
+        if ( empty( $data ) ) {
+            return $this->get_null_action();
+        }
+
+        if ( ! empty( $data->extended_args ) ) {
+            $data->args = $data->extended_args;
+            unset( $data->extended_args );
+        }
+
+        try {
+            $action = $this->make_action_from_db_record( $data );
+        } catch ( ActionScheduler_InvalidActionException $exception ) {
+            do_action( 'action_scheduler_failed_fetch_action', $action_id, $exception );
+            return $this->get_null_action();
+        }
+
+        return $action;
+    }
+
+    /**
+     * Create a null action.
+     *
+     * @return ActionScheduler_NullAction
+     */
+    protected function get_null_action() {
+        return new ActionScheduler_NullAction();
+    }
+
+    /**
+     * Create an action from a database record.
+     *
+     * @param object $data Action database record.
+     *
+     * @return ActionScheduler_Action|ActionScheduler_CanceledAction|ActionScheduler_FinishedAction
+     */
+    protected function make_action_from_db_record( $data ) {
+
+        $hook     = $data->hook;
+        $args     = json_decode( $data->args, true );
+        $schedule = unserialize( $data->schedule );
+
+        $this->validate_args( $args, $data->action_id );
+        $this->validate_schedule( $schedule, $data->action_id );
+
+        if ( empty( $schedule ) ) {
+            $schedule = new ActionScheduler_NullSchedule();
+        }
+        $group = $data->group ? $data->group : '';
+
+        return ActionScheduler::factory()->get_stored_action( $data->status, $data->hook, $args, $schedule, $group );
+    }
+
+    /**
+     * Find an action.
+     *
+     * @param string $hook Action hook.
+     * @param array  $params Parameters of the action to find.
+     *
+     * @return string|null ID of the next action matching the criteria or NULL if not found.
+     */
+    public function find_action( $hook, $params = [] ) {
+        $params = wp_parse_args( $params, [
+            'args'   => null,
+            'status' => self::STATUS_PENDING,
+            'group'  => '',
+        ] );
+
+        /** @var wpdb $wpdb */
+        global $wpdb;
+        $query = "SELECT a.action_id FROM {$wpdb->actionscheduler_actions} a";
+        $args  = [];
+        if ( ! empty( $params[ 'group' ] ) ) {
+            $query  .= " INNER JOIN {$wpdb->actionscheduler_groups} g ON g.group_id=a.group_id AND g.slug=%s";
+            $args[] = $params[ 'group' ];
+        }
+        $query  .= " WHERE a.hook=%s";
+        $args[] = $hook;
+        if ( ! is_null( $params[ 'args' ] ) ) {
+            $query  .= " AND a.args=%s";
+            $args[] = $this->get_args_for_query( $params[ 'args' ] );
+        }
+
+        $order = 'ASC';
+        if ( ! empty( $params[ 'status' ] ) ) {
+            $query  .= " AND a.status=%s";
+            $args[] = $params[ 'status' ];
+
+            if ( self::STATUS_PENDING == $params[ 'status' ] ) {
+                $order = 'ASC'; // Find the next action that matches.
+            } else {
+                $order = 'DESC'; // Find the most recent action that matches.
+            }
+        }
+
+        $query .= " ORDER BY scheduled_date_gmt $order LIMIT 1";
+
+        $query = $wpdb->prepare( $query, $args );
+
+        $id = $wpdb->get_var( $query );
+
+        return $id;
+    }
+
+    /**
+     * Returns the SQL statement to query (or count) actions.
+     *
+     * @param array  $query Filtering options.
+     * @param string $select_or_count  Whether the SQL should select and return the IDs or just the row count.
+     *
+     * @return string SQL statement already properly escaped.
+     */
+    protected function get_query_actions_sql( array $query, $select_or_count = 'select' ) {
+
+        if ( ! in_array( $select_or_count, array( 'select', 'count' ) ) ) {
+            throw new InvalidArgumentException( __( 'Invalid value for select or count parameter. Cannot query actions.', 'action-scheduler' ) );
+        }
+
+        $query = wp_parse_args( $query, [
+            'hook'             => '',
+            'args'             => null,
+            'date'             => null,
+            'date_compare'     => '<=',
+            'modified'         => null,
+            'modified_compare' => '<=',
+            'group'            => '',
+            'status'           => '',
+            'claimed'          => null,
+            'per_page'         => 5,
+            'offset'           => 0,
+            'orderby'          => 'date',
+            'order'            => 'ASC',
+        ] );
+
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $sql  = ( 'count' === $select_or_count ) ? 'SELECT count(a.action_id)' : 'SELECT a.action_id';
+        $sql .= " FROM {$wpdb->actionscheduler_actions} a";
+        $sql_params = [];
+
+        if ( ! empty( $query[ 'group' ] ) || 'group' === $query[ 'orderby' ] ) {
+            $sql .= " LEFT JOIN {$wpdb->actionscheduler_groups} g ON g.group_id=a.group_id";
+        }
+
+        $sql .= " WHERE 1=1";
+
+        if ( ! empty( $query[ 'group' ] ) ) {
+            $sql          .= " AND g.slug=%s";
+            $sql_params[] = $query[ 'group' ];
+        }
+
+        if ( $query[ 'hook' ] ) {
+            $sql          .= " AND a.hook=%s";
+            $sql_params[] = $query[ 'hook' ];
+        }
+        if ( ! is_null( $query[ 'args' ] ) ) {
+            $sql          .= " AND a.args=%s";
+            $sql_params[] = $this->get_args_for_query( $query[ 'args' ] );
+        }
+
+        if ( $query[ 'status' ] ) {
+            $sql          .= " AND a.status=%s";
+            $sql_params[] = $query[ 'status' ];
+        }
+
+        if ( $query[ 'date' ] instanceof \DateTime ) {
+            $date = clone $query[ 'date' ];
+            $date->setTimezone( new \DateTimeZone( 'UTC' ) );
+            $date_string  = $date->format( 'Y-m-d H:i:s' );
+            $comparator   = $this->validate_sql_comparator( $query[ 'date_compare' ] );
+            $sql          .= " AND a.scheduled_date_gmt $comparator %s";
+            $sql_params[] = $date_string;
+        }
+
+        if ( $query[ 'modified' ] instanceof \DateTime ) {
+            $modified = clone $query[ 'modified' ];
+            $modified->setTimezone( new \DateTimeZone( 'UTC' ) );
+            $date_string  = $modified->format( 'Y-m-d H:i:s' );
+            $comparator   = $this->validate_sql_comparator( $query[ 'modified_compare' ] );
+            $sql          .= " AND a.last_attempt_gmt $comparator %s";
+            $sql_params[] = $date_string;
+        }
+
+        if ( $query[ 'claimed' ] === true ) {
+            $sql .= " AND a.claim_id != 0";
+        } elseif ( $query[ 'claimed' ] === false ) {
+            $sql .= " AND a.claim_id = 0";
+        } elseif ( ! is_null( $query[ 'claimed' ] ) ) {
+            $sql          .= " AND a.claim_id = %d";
+            $sql_params[] = $query[ 'claimed' ];
+        }
+
+        if ( ! empty( $query['search'] ) ) {
+            $sql .= " AND (a.hook LIKE %s OR (a.extended_args IS NULL AND a.args LIKE %s) OR a.extended_args LIKE %s";
+            for( $i = 0; $i < 3; $i++ ) {
+                $sql_params[] = sprintf( '%%%s%%', $query['search'] );
+            }
+
+            $search_claim_id = (int) $query['search'];
+            if ( $search_claim_id ) {
+                $sql .= ' OR a.claim_id = %d';
+                $sql_params[] = $search_claim_id;
+            }
+
+            $sql .= ')';
+        }
+
+        if ( 'select' === $select_or_count ) {
+            switch ( $query['orderby'] ) {
+                case 'hook':
+                    $orderby = 'a.hook';
+                    break;
+                case 'group':
+                    $orderby = 'g.slug';
+                    break;
+                case 'modified':
+                    $orderby = 'a.last_attempt_gmt';
+                    break;
+                case 'date':
+                default:
+                    $orderby = 'a.scheduled_date_gmt';
+                    break;
+            }
+            if ( strtoupper( $query[ 'order' ] ) == 'ASC' ) {
+                $order = 'ASC';
+            } else {
+                $order = 'DESC';
+            }
+            $sql .= " ORDER BY $orderby $order";
+            if ( $query[ 'per_page' ] > 0 ) {
+                $sql          .= " LIMIT %d, %d";
+                $sql_params[] = $query[ 'offset' ];
+                $sql_params[] = $query[ 'per_page' ];
+            }
+        }
+
+        if ( ! empty( $sql_params ) ) {
+            $sql = $wpdb->prepare( $sql, $sql_params );
+        }
+
+        return $sql;
+    }
+
+    /**
+     * Query for action count of list of action IDs.
+     *
+     * @param array  $query Query parameters.
+     * @param string $query_type Whether to select or count the results. Default, select.
+     *
+     * @return null|string|array The IDs of actions matching the query
+     */
+    public function query_actions( $query = [], $query_type = 'select' ) {
+        /** @var wpdb $wpdb */
+        global $wpdb;
+
+        $sql = $this->get_query_actions_sql( $query, $query_type );
+
+        return ( 'count' === $query_type ) ? $wpdb->get_var( $sql ) : $wpdb->get_col( $sql );
+    }
+
+    /**
+     * Get a count of all actions in the store, grouped by status.
+     *
+     * @return array Set of 'status' => int $count pairs for statuses with 1 or more actions of that status.
+     */
+    public function action_counts() {
+        global $wpdb;
+
+        $sql  = "SELECT a.status, count(a.status) as 'count'";
+        $sql .= " FROM {$wpdb->actionscheduler_actions} a";
+        $sql .= " GROUP BY a.status";
+
+        $actions_count_by_status = array();
+        $action_stati_and_labels = $this->get_status_labels();
+
+        foreach ( $wpdb->get_results( $sql ) as $action_data ) {
+            // Ignore any actions with invalid status
+            if ( array_key_exists( $action_data->status, $action_stati_and_labels ) ) {
+                $actions_count_by_status[ $action_data->status ] = $action_data->count;
+            }
+        }
+
+        return $actions_count_by_status;
+    }
+
+    /**
+     * Cancel an action.
+     *
+     * @param int $action_id Action ID.
+     *
+     * @return void
+     */
+    public function cancel_action( $action_id ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+
+        $updated = $wpdb->update(
+            $wpdb->actionscheduler_actions,
+            [ 'status' => self::STATUS_CANCELED ],
+            [ 'action_id' => $action_id ],
+            [ '%s' ],
+            [ '%d' ]
+        );
+        if ( empty( $updated ) ) {
+            /* translators: %s: action ID */
+            throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+        }
+        do_action( 'action_scheduler_canceled_action', $action_id );
+    }
+
+    /**
+     * Cancel pending actions by hook.
+     *
+     * @since 3.0.0
+     *
+     * @param string $hook Hook name.
+     *
+     * @return void
+     */
+    public function cancel_actions_by_hook( $hook ) {
+        $this->bulk_cancel_actions( [ 'hook' => $hook ] );
+    }
+
+    /**
+     * Cancel pending actions by group.
+     *
+     * @param string $group Group slug.
+     *
+     * @return void
+     */
+    public function cancel_actions_by_group( $group ) {
+        $this->bulk_cancel_actions( [ 'group' => $group ] );
+    }
+
+    /**
+     * Bulk cancel actions.
+     *
+     * @since 3.0.0
+     *
+     * @param array $query_args Query parameters.
+     */
+    protected function bulk_cancel_actions( $query_args ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+
+        if ( ! is_array( $query_args ) ) {
+            return;
+        }
+
+        // Don't cancel actions that are already canceled.
+        if ( isset( $query_args['status'] ) && $query_args['status'] == self::STATUS_CANCELED ) {
+            return;
+        }
+
+        $action_ids = true;
+        $query_args = wp_parse_args(
+            $query_args,
+            [
+                'per_page' => 1000,
+                'status' => self::STATUS_PENDING,
+            ]
+        );
+
+        while ( $action_ids ) {
+            $action_ids = $this->query_actions( $query_args );
+            if ( empty( $action_ids ) ) {
+                break;
+            }
+
+            $format     = array_fill( 0, count( $action_ids ), '%d' );
+            $query_in   = '(' . implode( ',', $format ) . ')';
+            $parameters = $action_ids;
+            array_unshift( $parameters, self::STATUS_CANCELED );
+
+            $wpdb->query(
+                $wpdb->prepare( // wpcs: PreparedSQLPlaceholders replacement count ok.
+                    "UPDATE {$wpdb->actionscheduler_actions} SET status = %s WHERE action_id IN {$query_in}",
+                    $parameters
+                )
+            );
+
+            do_action( 'action_scheduler_bulk_cancel_actions', $action_ids );
+        }
+    }
+
+    /**
+     * Delete an action.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function delete_action( $action_id ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $deleted = $wpdb->delete( $wpdb->actionscheduler_actions, [ 'action_id' => $action_id ], [ '%d' ] );
+        if ( empty( $deleted ) ) {
+            throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+        }
+        do_action( 'action_scheduler_deleted_action', $action_id );
+    }
+
+    /**
+     * Get the schedule date for an action.
+     *
+     * @param string $action_id Action ID.
+     *
+     * @throws \InvalidArgumentException
+     * @return \DateTime The local date the action is scheduled to run, or the date that it ran.
+     */
+    public function get_date( $action_id ) {
+        $date = $this->get_date_gmt( $action_id );
+        ActionScheduler_TimezoneHelper::set_local_timezone( $date );
+        return $date;
+    }
+
+    /**
+     * Get the GMT schedule date for an action.
+     *
+     * @param int $action_id Action ID.
+     *
+     * @throws \InvalidArgumentException
+     * @return \DateTime The GMT date the action is scheduled to run, or the date that it ran.
+     */
+    protected function get_date_gmt( $action_id ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $record = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d", $action_id ) );
+        if ( empty( $record ) ) {
+            throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+        }
+        if ( $record->status == self::STATUS_PENDING ) {
+            return as_get_datetime_object( $record->scheduled_date_gmt );
+        } else {
+            return as_get_datetime_object( $record->last_attempt_gmt );
+        }
+    }
+
+    /**
+     * Stake a claim on actions.
+     *
+     * @param int       $max_actions Maximum number of action to include in claim.
+     * @param \DateTime $before_date Jobs must be schedule before this date. Defaults to now.
+     *
+     * @return ActionScheduler_ActionClaim
+     */
+    public function stake_claim( $max_actions = 10, \DateTime $before_date = null, $hooks = array(), $group = '' ) {
+        $claim_id = $this->generate_claim_id();
+        $this->claim_actions( $claim_id, $max_actions, $before_date, $hooks, $group );
+        $action_ids = $this->find_actions_by_claim_id( $claim_id );
+
+        return new ActionScheduler_ActionClaim( $claim_id, $action_ids );
+    }
+
+    /**
+     * Generate a new action claim.
+     *
+     * @return int Claim ID.
+     */
+    protected function generate_claim_id() {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $now = as_get_datetime_object();
+        $wpdb->insert( $wpdb->actionscheduler_claims, [ 'date_created_gmt' => $now->format( 'Y-m-d H:i:s' ) ] );
+
+        return $wpdb->insert_id;
+    }
+
+    /**
+     * Mark actions claimed.
+     *
+     * @param string    $claim_id Claim Id.
+     * @param int       $limit Number of action to include in claim.
+     * @param \DateTime $before_date Should use UTC timezone.
+     *
+     * @return int The number of actions that were claimed.
+     * @throws \RuntimeException
+     */
+    protected function claim_actions( $claim_id, $limit, \DateTime $before_date = null, $hooks = array(), $group = '' ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+
+        $now  = as_get_datetime_object();
+        $date = is_null( $before_date ) ? $now : clone $before_date;
+
+        // can't use $wpdb->update() because of the <= condition
+        $update = "UPDATE {$wpdb->actionscheduler_actions} SET claim_id=%d, last_attempt_gmt=%s, last_attempt_local=%s";
+        $params = array(
+            $claim_id,
+            $now->format( 'Y-m-d H:i:s' ),
+            current_time( 'mysql' ),
+        );
+
+        $where    = "WHERE claim_id = 0 AND scheduled_date_gmt <= %s AND status=%s";
+        $params[] = $date->format( 'Y-m-d H:i:s' );
+        $params[] = self::STATUS_PENDING;
+
+        if ( ! empty( $hooks ) ) {
+            $placeholders = array_fill( 0, count( $hooks ), '%s' );
+            $where       .= ' AND hook IN (' . join( ', ', $placeholders ) . ')';
+            $params       = array_merge( $params, array_values( $hooks ) );
+        }
+
+        if ( ! empty( $group ) ) {
+
+            $group_id = $this->get_group_id( $group, false );
+
+            // throw exception if no matching group found, this matches ActionScheduler_wpPostStore's behaviour
+            if ( empty( $group_id ) ) {
+                /* translators: %s: group name */
+                throw new InvalidArgumentException( sprintf( __( 'The group "%s" does not exist.', 'action-scheduler' ), $group ) );
+            }
+
+            $where   .= ' AND group_id = %d';
+            $params[] = $group_id;
+        }
+
+        $order    = "ORDER BY attempts ASC, scheduled_date_gmt ASC, action_id ASC LIMIT %d";
+        $params[] = $limit;
+
+        $sql = $wpdb->prepare( "{$update} {$where} {$order}", $params );
+
+        $rows_affected = $wpdb->query( $sql );
+        if ( $rows_affected === false ) {
+            throw new \RuntimeException( __( 'Unable to claim actions. Database error.', 'action-scheduler' ) );
+        }
+
+        return (int) $rows_affected;
+    }
+
+    /**
+     * Get the number of active claims.
+     *
+     * @return int
+     */
+    public function get_claim_count() {
+        global $wpdb;
+
+        $sql = "SELECT COUNT(DISTINCT claim_id) FROM {$wpdb->actionscheduler_actions} WHERE claim_id != 0 AND status IN ( %s, %s)";
+        $sql = $wpdb->prepare( $sql, [ self::STATUS_PENDING, self::STATUS_RUNNING ] );
+
+        return (int) $wpdb->get_var( $sql );
+    }
+
+    /**
+     * Return an action's claim ID, as stored in the claim_id column.
+     *
+     * @param string $action_id Action ID.
+     * @return mixed
+     */
+    public function get_claim_id( $action_id ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+
+        $sql = "SELECT claim_id FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d";
+        $sql = $wpdb->prepare( $sql, $action_id );
+
+        return (int) $wpdb->get_var( $sql );
+    }
+
+    /**
+     * Retrieve the action IDs of action in a claim.
+     *
+     * @param string $claim_id Claim ID.
+     *
+     * @return int[]
+     */
+    public function find_actions_by_claim_id( $claim_id ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+
+        $sql = "SELECT action_id FROM {$wpdb->actionscheduler_actions} WHERE claim_id=%d";
+        $sql = $wpdb->prepare( $sql, $claim_id );
+
+        $action_ids = $wpdb->get_col( $sql );
+
+        return array_map( 'intval', $action_ids );
+    }
+
+    /**
+     * Release actions from a claim and delete the claim.
+     *
+     * @param ActionScheduler_ActionClaim $claim Claim object.
+     */
+    public function release_claim( ActionScheduler_ActionClaim $claim ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $wpdb->update( $wpdb->actionscheduler_actions, [ 'claim_id' => 0 ], [ 'claim_id' => $claim->get_id() ], [ '%d' ], [ '%d' ] );
+        $wpdb->delete( $wpdb->actionscheduler_claims, [ 'claim_id' => $claim->get_id() ], [ '%d' ] );
+    }
+
+    /**
+     * Remove the claim from an action.
+     *
+     * @param int $action_id Action ID.
+     *
+     * @return void
+     */
+    public function unclaim_action( $action_id ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $wpdb->update(
+            $wpdb->actionscheduler_actions,
+            [ 'claim_id' => 0 ],
+            [ 'action_id' => $action_id ],
+            [ '%s' ],
+            [ '%d' ]
+        );
+    }
+
+    /**
+     * Mark an action as failed.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function mark_failure( $action_id ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $updated = $wpdb->update(
+            $wpdb->actionscheduler_actions,
+            [ 'status' => self::STATUS_FAILED ],
+            [ 'action_id' => $action_id ],
+            [ '%s' ],
+            [ '%d' ]
+        );
+        if ( empty( $updated ) ) {
+            throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+        }
+    }
+
+    /**
+     * Add execution message to action log.
+     *
+     * @param int $action_id Action ID.
+     *
+     * @return void
+     */
+    public function log_execution( $action_id ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+
+        $sql = "UPDATE {$wpdb->actionscheduler_actions} SET attempts = attempts+1, status=%s, last_attempt_gmt = %s, last_attempt_local = %s WHERE action_id = %d";
+        $sql = $wpdb->prepare( $sql, self::STATUS_RUNNING, current_time( 'mysql', true ), current_time( 'mysql' ), $action_id );
+        $wpdb->query( $sql );
+    }
+
+    /**
+     * Mark an action as complete.
+     *
+     * @param int $action_id Action ID.
+     *
+     * @return void
+     */
+    public function mark_complete( $action_id ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $updated = $wpdb->update(
+            $wpdb->actionscheduler_actions,
+            [
+                'status'             => self::STATUS_COMPLETE,
+                'last_attempt_gmt'   => current_time( 'mysql', true ),
+                'last_attempt_local' => current_time( 'mysql' ),
+            ],
+            [ 'action_id' => $action_id ],
+            [ '%s' ],
+            [ '%d' ]
+        );
+        if ( empty( $updated ) ) {
+            throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+        }
+    }
+
+    /**
+     * Get an action's status.
+     *
+     * @param int $action_id Action ID.
+     *
+     * @return string
+     */
+    public function get_status( $action_id ) {
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $sql    = "SELECT status FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d";
+        $sql    = $wpdb->prepare( $sql, $action_id );
+        $status = $wpdb->get_var( $sql );
+
+        if ( $status === null ) {
+            throw new \InvalidArgumentException( __( 'Invalid action ID. No status found.', 'action-scheduler' ) );
+        } elseif ( empty( $status ) ) {
+            throw new \RuntimeException( __( 'Unknown status found for action.', 'action-scheduler' ) );
+        } else {
+            return $status;
+        }
+    }
 }

Spacing +219 added lines, -219 removed lines

@@ -33,46 +33,46 @@ 
 	 *
 	 * @return int Action ID.
 	 */
-	public function save_action( ActionScheduler_Action $action, \DateTime $date = null ) {
+	public function save_action(ActionScheduler_Action $action, \DateTime $date = null) {
 		try {
 
-			$this->validate_action( $action );
+			$this->validate_action($action);
 
 			/** @var \wpdb $wpdb */
 			global $wpdb;
 			$data = [
 				'hook'                 => $action->get_hook(),
-				'status'               => ( $action->is_finished() ? self::STATUS_COMPLETE : self::STATUS_PENDING ),
-				'scheduled_date_gmt'   => $this->get_scheduled_date_string( $action, $date ),
-				'scheduled_date_local' => $this->get_scheduled_date_string_local( $action, $date ),
-				'schedule'             => serialize( $action->get_schedule() ),
-				'group_id'             => $this->get_group_id( $action->get_group() ),
+				'status'               => ($action->is_finished() ? self::STATUS_COMPLETE : self::STATUS_PENDING),
+				'scheduled_date_gmt'   => $this->get_scheduled_date_string($action, $date),
+				'scheduled_date_local' => $this->get_scheduled_date_string_local($action, $date),
+				'schedule'             => serialize($action->get_schedule()),
+				'group_id'             => $this->get_group_id($action->get_group()),
 			];
-			$args = wp_json_encode( $action->get_args() );
-			if ( strlen( $args ) <= static::$max_index_length ) {
+			$args = wp_json_encode($action->get_args());
+			if (strlen($args) <= static::$max_index_length) {
 				$data['args'] = $args;
 			} else {
-				$data['args']          = $this->hash_args( $args );
+				$data['args']          = $this->hash_args($args);
 				$data['extended_args'] = $args;
 			}
 
-			$table_name = ! empty( $wpdb->actionscheduler_actions ) ? $wpdb->actionscheduler_actions : $wpdb->prefix . 'actionscheduler_actions';
-			$wpdb->insert( $table_name, $data );
+			$table_name = !empty($wpdb->actionscheduler_actions) ? $wpdb->actionscheduler_actions : $wpdb->prefix . 'actionscheduler_actions';
+			$wpdb->insert($table_name, $data);
 			$action_id = $wpdb->insert_id;
 
-			if ( is_wp_error( $action_id ) ) {
-				throw new RuntimeException( $action_id->get_error_message() );
+			if (is_wp_error($action_id)) {
+				throw new RuntimeException($action_id->get_error_message());
 			}
-			elseif ( empty( $action_id ) ) {
-				throw new RuntimeException( $wpdb->last_error ? $wpdb->last_error : __( 'Database error.', 'action-scheduler' ) );
+			elseif (empty($action_id)) {
+				throw new RuntimeException($wpdb->last_error ? $wpdb->last_error : __('Database error.', 'action-scheduler'));
 			}
 
-			do_action( 'action_scheduler_stored_action', $action_id );
+			do_action('action_scheduler_stored_action', $action_id);
 
 			return $action_id;
-		} catch ( \Exception $e ) {
+		} catch (\Exception $e) {
 			/* translators: %s: error message */
-			throw new \RuntimeException( sprintf( __( 'Error saving action: %s', 'action-scheduler' ), $e->getMessage() ), 0 );
+			throw new \RuntimeException(sprintf(__('Error saving action: %s', 'action-scheduler'), $e->getMessage()), 0);
 		}
 	}
 
@@ -82,8 +82,8 @@ 
 	 * @param string $args JSON encoded action args.
 	 * @return string
 	 */
-	protected function hash_args( $args ) {
-		return md5( $args );
+	protected function hash_args($args) {
+		return md5($args);
 	}
 
 	/**
@@ -92,12 +92,12 @@ 
 	 * @param array $args Action args.
 	 * @return string
 	 */
-	protected function get_args_for_query( $args ) {
-		$encoded = wp_json_encode( $args );
-		if ( strlen( $encoded ) <= static::$max_index_length ) {
+	protected function get_args_for_query($args) {
+		$encoded = wp_json_encode($args);
+		if (strlen($encoded) <= static::$max_index_length) {
 			return $encoded;
 		}
-		return $this->hash_args( $encoded );
+		return $this->hash_args($encoded);
 	}
 	/**
 	 * Get a group's ID based on its name/slug.
@@ -107,15 +107,15 @@ 
 	 *
 	 * @return int The group's ID, if it exists or is created, or 0 if it does not exist and is not created.
 	 */
-	protected function get_group_id( $slug, $create_if_not_exists = true ) {
-		if ( empty( $slug ) ) {
+	protected function get_group_id($slug, $create_if_not_exists = true) {
+		if (empty($slug)) {
 			return 0;
 		}
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$group_id = (int) $wpdb->get_var( $wpdb->prepare( "SELECT group_id FROM {$wpdb->actionscheduler_groups} WHERE slug=%s", $slug ) );
-		if ( empty( $group_id ) && $create_if_not_exists ) {
-			$group_id = $this->create_group( $slug );
+		$group_id = (int) $wpdb->get_var($wpdb->prepare("SELECT group_id FROM {$wpdb->actionscheduler_groups} WHERE slug=%s", $slug));
+		if (empty($group_id) && $create_if_not_exists) {
+			$group_id = $this->create_group($slug);
 		}
 
 		return $group_id;
@@ -128,10 +128,10 @@ 
 	 *
 	 * @return int Group ID.
 	 */
-	protected function create_group( $slug ) {
+	protected function create_group($slug) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$wpdb->insert( $wpdb->actionscheduler_groups, [ 'slug' => $slug ] );
+		$wpdb->insert($wpdb->actionscheduler_groups, ['slug' => $slug]);
 
 		return (int) $wpdb->insert_id;
 	}
@@ -143,27 +143,27 @@ 
 	 *
 	 * @return ActionScheduler_Action
 	 */
-	public function fetch_action( $action_id ) {
+	public function fetch_action($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$data = $wpdb->get_row( $wpdb->prepare(
+		$data = $wpdb->get_row($wpdb->prepare(
 			"SELECT a.*, g.slug AS `group` FROM {$wpdb->actionscheduler_actions} a LEFT JOIN {$wpdb->actionscheduler_groups} g ON a.group_id=g.group_id WHERE a.action_id=%d",
 			$action_id
-		) );
+		));
 
-		if ( empty( $data ) ) {
+		if (empty($data)) {
 			return $this->get_null_action();
 		}
 
-		if ( ! empty( $data->extended_args ) ) {
+		if (!empty($data->extended_args)) {
 			$data->args = $data->extended_args;
-			unset( $data->extended_args );
+			unset($data->extended_args);
 		}
 
 		try {
-			$action = $this->make_action_from_db_record( $data );
-		} catch ( ActionScheduler_InvalidActionException $exception ) {
-			do_action( 'action_scheduler_failed_fetch_action', $action_id, $exception );
+			$action = $this->make_action_from_db_record($data);
+		} catch (ActionScheduler_InvalidActionException $exception) {
+			do_action('action_scheduler_failed_fetch_action', $action_id, $exception);
 			return $this->get_null_action();
 		}
 
@@ -186,21 +186,21 @@ 
 	 *
 	 * @return ActionScheduler_Action|ActionScheduler_CanceledAction|ActionScheduler_FinishedAction
 	 */
-	protected function make_action_from_db_record( $data ) {
+	protected function make_action_from_db_record($data) {
 
 		$hook     = $data->hook;
-		$args     = json_decode( $data->args, true );
-		$schedule = unserialize( $data->schedule );
+		$args     = json_decode($data->args, true);
+		$schedule = unserialize($data->schedule);
 
-		$this->validate_args( $args, $data->action_id );
-		$this->validate_schedule( $schedule, $data->action_id );
+		$this->validate_args($args, $data->action_id);
+		$this->validate_schedule($schedule, $data->action_id);
 
-		if ( empty( $schedule ) ) {
+		if (empty($schedule)) {
 			$schedule = new ActionScheduler_NullSchedule();
 		}
 		$group = $data->group ? $data->group : '';
 
-		return ActionScheduler::factory()->get_stored_action( $data->status, $data->hook, $args, $schedule, $group );
+		return ActionScheduler::factory()->get_stored_action($data->status, $data->hook, $args, $schedule, $group);
 	}
 
 	/**
@@ -211,34 +211,34 @@ 
 	 *
 	 * @return string|null ID of the next action matching the criteria or NULL if not found.
 	 */
-	public function find_action( $hook, $params = [] ) {
-		$params = wp_parse_args( $params, [
+	public function find_action($hook, $params = []) {
+		$params = wp_parse_args($params, [
 			'args'   => null,
 			'status' => self::STATUS_PENDING,
 			'group'  => '',
-		] );
+		]);
 
 		/** @var wpdb $wpdb */
 		global $wpdb;
 		$query = "SELECT a.action_id FROM {$wpdb->actionscheduler_actions} a";
 		$args  = [];
-		if ( ! empty( $params[ 'group' ] ) ) {
-			$query  .= " INNER JOIN {$wpdb->actionscheduler_groups} g ON g.group_id=a.group_id AND g.slug=%s";
-			$args[] = $params[ 'group' ];
+		if (!empty($params['group'])) {
+			$query .= " INNER JOIN {$wpdb->actionscheduler_groups} g ON g.group_id=a.group_id AND g.slug=%s";
+			$args[] = $params['group'];
 		}
 		$query  .= " WHERE a.hook=%s";
 		$args[] = $hook;
-		if ( ! is_null( $params[ 'args' ] ) ) {
-			$query  .= " AND a.args=%s";
-			$args[] = $this->get_args_for_query( $params[ 'args' ] );
+		if (!is_null($params['args'])) {
+			$query .= " AND a.args=%s";
+			$args[] = $this->get_args_for_query($params['args']);
 		}
 
 		$order = 'ASC';
-		if ( ! empty( $params[ 'status' ] ) ) {
-			$query  .= " AND a.status=%s";
-			$args[] = $params[ 'status' ];
+		if (!empty($params['status'])) {
+			$query .= " AND a.status=%s";
+			$args[] = $params['status'];
 
-			if ( self::STATUS_PENDING == $params[ 'status' ] ) {
+			if (self::STATUS_PENDING == $params['status']) {
 				$order = 'ASC'; // Find the next action that matches.
 			} else {
 				$order = 'DESC'; // Find the most recent action that matches.
@@ -247,9 +247,9 @@ 
 
 		$query .= " ORDER BY scheduled_date_gmt $order LIMIT 1";
 
-		$query = $wpdb->prepare( $query, $args );
+		$query = $wpdb->prepare($query, $args);
 
-		$id = $wpdb->get_var( $query );
+		$id = $wpdb->get_var($query);
 
 		return $id;
 	}
@@ -262,13 +262,13 @@ 
 	 *
 	 * @return string SQL statement already properly escaped.
 	 */
-	protected function get_query_actions_sql( array $query, $select_or_count = 'select' ) {
+	protected function get_query_actions_sql(array $query, $select_or_count = 'select') {
 
-		if ( ! in_array( $select_or_count, array( 'select', 'count' ) ) ) {
-			throw new InvalidArgumentException( __( 'Invalid value for select or count parameter. Cannot query actions.', 'action-scheduler' ) );
+		if (!in_array($select_or_count, array('select', 'count'))) {
+			throw new InvalidArgumentException(__('Invalid value for select or count parameter. Cannot query actions.', 'action-scheduler'));
 		}
 
-		$query = wp_parse_args( $query, [
+		$query = wp_parse_args($query, [
 			'hook'             => '',
 			'args'             => null,
 			'date'             => null,
@@ -282,74 +282,74 @@ 
 			'offset'           => 0,
 			'orderby'          => 'date',
 			'order'            => 'ASC',
-		] );
+		]);
 
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$sql  = ( 'count' === $select_or_count ) ? 'SELECT count(a.action_id)' : 'SELECT a.action_id';
+		$sql  = ('count' === $select_or_count) ? 'SELECT count(a.action_id)' : 'SELECT a.action_id';
 		$sql .= " FROM {$wpdb->actionscheduler_actions} a";
 		$sql_params = [];
 
-		if ( ! empty( $query[ 'group' ] ) || 'group' === $query[ 'orderby' ] ) {
+		if (!empty($query['group']) || 'group' === $query['orderby']) {
 			$sql .= " LEFT JOIN {$wpdb->actionscheduler_groups} g ON g.group_id=a.group_id";
 		}
 
 		$sql .= " WHERE 1=1";
 
-		if ( ! empty( $query[ 'group' ] ) ) {
-			$sql          .= " AND g.slug=%s";
-			$sql_params[] = $query[ 'group' ];
+		if (!empty($query['group'])) {
+			$sql .= " AND g.slug=%s";
+			$sql_params[] = $query['group'];
 		}
 
-		if ( $query[ 'hook' ] ) {
-			$sql          .= " AND a.hook=%s";
-			$sql_params[] = $query[ 'hook' ];
+		if ($query['hook']) {
+			$sql .= " AND a.hook=%s";
+			$sql_params[] = $query['hook'];
 		}
-		if ( ! is_null( $query[ 'args' ] ) ) {
-			$sql          .= " AND a.args=%s";
-			$sql_params[] = $this->get_args_for_query( $query[ 'args' ] );
+		if (!is_null($query['args'])) {
+			$sql .= " AND a.args=%s";
+			$sql_params[] = $this->get_args_for_query($query['args']);
 		}
 
-		if ( $query[ 'status' ] ) {
-			$sql          .= " AND a.status=%s";
-			$sql_params[] = $query[ 'status' ];
+		if ($query['status']) {
+			$sql .= " AND a.status=%s";
+			$sql_params[] = $query['status'];
 		}
 
-		if ( $query[ 'date' ] instanceof \DateTime ) {
-			$date = clone $query[ 'date' ];
-			$date->setTimezone( new \DateTimeZone( 'UTC' ) );
-			$date_string  = $date->format( 'Y-m-d H:i:s' );
-			$comparator   = $this->validate_sql_comparator( $query[ 'date_compare' ] );
-			$sql          .= " AND a.scheduled_date_gmt $comparator %s";
+		if ($query['date'] instanceof \DateTime) {
+			$date = clone $query['date'];
+			$date->setTimezone(new \DateTimeZone('UTC'));
+			$date_string  = $date->format('Y-m-d H:i:s');
+			$comparator   = $this->validate_sql_comparator($query['date_compare']);
+			$sql .= " AND a.scheduled_date_gmt $comparator %s";
 			$sql_params[] = $date_string;
 		}
 
-		if ( $query[ 'modified' ] instanceof \DateTime ) {
-			$modified = clone $query[ 'modified' ];
-			$modified->setTimezone( new \DateTimeZone( 'UTC' ) );
-			$date_string  = $modified->format( 'Y-m-d H:i:s' );
-			$comparator   = $this->validate_sql_comparator( $query[ 'modified_compare' ] );
-			$sql          .= " AND a.last_attempt_gmt $comparator %s";
+		if ($query['modified'] instanceof \DateTime) {
+			$modified = clone $query['modified'];
+			$modified->setTimezone(new \DateTimeZone('UTC'));
+			$date_string  = $modified->format('Y-m-d H:i:s');
+			$comparator   = $this->validate_sql_comparator($query['modified_compare']);
+			$sql .= " AND a.last_attempt_gmt $comparator %s";
 			$sql_params[] = $date_string;
 		}
 
-		if ( $query[ 'claimed' ] === true ) {
+		if ($query['claimed'] === true) {
 			$sql .= " AND a.claim_id != 0";
-		} elseif ( $query[ 'claimed' ] === false ) {
+		} elseif ($query['claimed'] === false) {
 			$sql .= " AND a.claim_id = 0";
-		} elseif ( ! is_null( $query[ 'claimed' ] ) ) {
-			$sql          .= " AND a.claim_id = %d";
-			$sql_params[] = $query[ 'claimed' ];
+		} elseif (!is_null($query['claimed'])) {
+			$sql .= " AND a.claim_id = %d";
+			$sql_params[] = $query['claimed'];
 		}
 
-		if ( ! empty( $query['search'] ) ) {
+		if (!empty($query['search'])) {
 			$sql .= " AND (a.hook LIKE %s OR (a.extended_args IS NULL AND a.args LIKE %s) OR a.extended_args LIKE %s";
-			for( $i = 0; $i < 3; $i++ ) {
-				$sql_params[] = sprintf( '%%%s%%', $query['search'] );
+			for ($i = 0; $i < 3; $i++) {
+				$sql_params[] = sprintf('%%%s%%', $query['search']);
 			}
 
 			$search_claim_id = (int) $query['search'];
-			if ( $search_claim_id ) {
+			if ($search_claim_id) {
 				$sql .= ' OR a.claim_id = %d';
 				$sql_params[] = $search_claim_id;
 			}
@@ -357,8 +357,8 @@ 
 			$sql .= ')';
 		}
 
-		if ( 'select' === $select_or_count ) {
-			switch ( $query['orderby'] ) {
+		if ('select' === $select_or_count) {
+			switch ($query['orderby']) {
 				case 'hook':
 					$orderby = 'a.hook';
 					break;
@@ -373,21 +373,21 @@ 
 					$orderby = 'a.scheduled_date_gmt';
 					break;
 			}
-			if ( strtoupper( $query[ 'order' ] ) == 'ASC' ) {
+			if (strtoupper($query['order']) == 'ASC') {
 				$order = 'ASC';
 			} else {
 				$order = 'DESC';
 			}
 			$sql .= " ORDER BY $orderby $order";
-			if ( $query[ 'per_page' ] > 0 ) {
-				$sql          .= " LIMIT %d, %d";
-				$sql_params[] = $query[ 'offset' ];
-				$sql_params[] = $query[ 'per_page' ];
+			if ($query['per_page'] > 0) {
+				$sql .= " LIMIT %d, %d";
+				$sql_params[] = $query['offset'];
+				$sql_params[] = $query['per_page'];
 			}
 		}
 
-		if ( ! empty( $sql_params ) ) {
-			$sql = $wpdb->prepare( $sql, $sql_params );
+		if (!empty($sql_params)) {
+			$sql = $wpdb->prepare($sql, $sql_params);
 		}
 
 		return $sql;
@@ -401,13 +401,13 @@ 
 	 *
 	 * @return null|string|array The IDs of actions matching the query
 	 */
-	public function query_actions( $query = [], $query_type = 'select' ) {
+	public function query_actions($query = [], $query_type = 'select') {
 		/** @var wpdb $wpdb */
 		global $wpdb;
 
-		$sql = $this->get_query_actions_sql( $query, $query_type );
+		$sql = $this->get_query_actions_sql($query, $query_type);
 
-		return ( 'count' === $query_type ) ? $wpdb->get_var( $sql ) : $wpdb->get_col( $sql );
+		return ('count' === $query_type) ? $wpdb->get_var($sql) : $wpdb->get_col($sql);
 	}
 
 	/**
@@ -425,10 +425,10 @@ 
 		$actions_count_by_status = array();
 		$action_stati_and_labels = $this->get_status_labels();
 
-		foreach ( $wpdb->get_results( $sql ) as $action_data ) {
+		foreach ($wpdb->get_results($sql) as $action_data) {
 			// Ignore any actions with invalid status
-			if ( array_key_exists( $action_data->status, $action_stati_and_labels ) ) {
-				$actions_count_by_status[ $action_data->status ] = $action_data->count;
+			if (array_key_exists($action_data->status, $action_stati_and_labels)) {
+				$actions_count_by_status[$action_data->status] = $action_data->count;
 			}
 		}
 
@@ -442,22 +442,22 @@ 
 	 *
 	 * @return void
 	 */
-	public function cancel_action( $action_id ) {
+	public function cancel_action($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 
 		$updated = $wpdb->update(
 			$wpdb->actionscheduler_actions,
-			[ 'status' => self::STATUS_CANCELED ],
-			[ 'action_id' => $action_id ],
-			[ '%s' ],
-			[ '%d' ]
+			['status' => self::STATUS_CANCELED],
+			['action_id' => $action_id],
+			['%s'],
+			['%d']
 		);
-		if ( empty( $updated ) ) {
+		if (empty($updated)) {
 			/* translators: %s: action ID */
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+			throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'action-scheduler'), $action_id));
 		}
-		do_action( 'action_scheduler_canceled_action', $action_id );
+		do_action('action_scheduler_canceled_action', $action_id);
 	}
 
 	/**
@@ -469,8 +469,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function cancel_actions_by_hook( $hook ) {
-		$this->bulk_cancel_actions( [ 'hook' => $hook ] );
+	public function cancel_actions_by_hook($hook) {
+		$this->bulk_cancel_actions(['hook' => $hook]);
 	}
 
 	/**
@@ -480,8 +480,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function cancel_actions_by_group( $group ) {
-		$this->bulk_cancel_actions( [ 'group' => $group ] );
+	public function cancel_actions_by_group($group) {
+		$this->bulk_cancel_actions(['group' => $group]);
 	}
 
 	/**
@@ -491,16 +491,16 @@ 
 	 *
 	 * @param array $query_args Query parameters.
 	 */
-	protected function bulk_cancel_actions( $query_args ) {
+	protected function bulk_cancel_actions($query_args) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 
-		if ( ! is_array( $query_args ) ) {
+		if (!is_array($query_args)) {
 			return;
 		}
 
 		// Don't cancel actions that are already canceled.
-		if ( isset( $query_args['status'] ) && $query_args['status'] == self::STATUS_CANCELED ) {
+		if (isset($query_args['status']) && $query_args['status'] == self::STATUS_CANCELED) {
 			return;
 		}
 
@@ -513,16 +513,16 @@ 
 			]
 		);
 
-		while ( $action_ids ) {
-			$action_ids = $this->query_actions( $query_args );
-			if ( empty( $action_ids ) ) {
+		while ($action_ids) {
+			$action_ids = $this->query_actions($query_args);
+			if (empty($action_ids)) {
 				break;
 			}
 
-			$format     = array_fill( 0, count( $action_ids ), '%d' );
-			$query_in   = '(' . implode( ',', $format ) . ')';
+			$format     = array_fill(0, count($action_ids), '%d');
+			$query_in   = '(' . implode(',', $format) . ')';
 			$parameters = $action_ids;
-			array_unshift( $parameters, self::STATUS_CANCELED );
+			array_unshift($parameters, self::STATUS_CANCELED);
 
 			$wpdb->query(
 				$wpdb->prepare( // wpcs: PreparedSQLPlaceholders replacement count ok.
@@ -531,7 +531,7 @@ 
 				)
 			);
 
-			do_action( 'action_scheduler_bulk_cancel_actions', $action_ids );
+			do_action('action_scheduler_bulk_cancel_actions', $action_ids);
 		}
 	}
 
@@ -540,14 +540,14 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function delete_action( $action_id ) {
+	public function delete_action($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$deleted = $wpdb->delete( $wpdb->actionscheduler_actions, [ 'action_id' => $action_id ], [ '%d' ] );
-		if ( empty( $deleted ) ) {
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+		$deleted = $wpdb->delete($wpdb->actionscheduler_actions, ['action_id' => $action_id], ['%d']);
+		if (empty($deleted)) {
+			throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'action-scheduler'), $action_id));
 		}
-		do_action( 'action_scheduler_deleted_action', $action_id );
+		do_action('action_scheduler_deleted_action', $action_id);
 	}
 
 	/**
@@ -558,9 +558,9 @@ 
 	 * @throws \InvalidArgumentException
 	 * @return \DateTime The local date the action is scheduled to run, or the date that it ran.
 	 */
-	public function get_date( $action_id ) {
-		$date = $this->get_date_gmt( $action_id );
-		ActionScheduler_TimezoneHelper::set_local_timezone( $date );
+	public function get_date($action_id) {
+		$date = $this->get_date_gmt($action_id);
+		ActionScheduler_TimezoneHelper::set_local_timezone($date);
 		return $date;
 	}
 
@@ -572,17 +572,17 @@ 
 	 * @throws \InvalidArgumentException
 	 * @return \DateTime The GMT date the action is scheduled to run, or the date that it ran.
 	 */
-	protected function get_date_gmt( $action_id ) {
+	protected function get_date_gmt($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$record = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d", $action_id ) );
-		if ( empty( $record ) ) {
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+		$record = $wpdb->get_row($wpdb->prepare("SELECT * FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d", $action_id));
+		if (empty($record)) {
+			throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'action-scheduler'), $action_id));
 		}
-		if ( $record->status == self::STATUS_PENDING ) {
-			return as_get_datetime_object( $record->scheduled_date_gmt );
+		if ($record->status == self::STATUS_PENDING) {
+			return as_get_datetime_object($record->scheduled_date_gmt);
 		} else {
-			return as_get_datetime_object( $record->last_attempt_gmt );
+			return as_get_datetime_object($record->last_attempt_gmt);
 		}
 	}
 
@@ -594,12 +594,12 @@ 
 	 *
 	 * @return ActionScheduler_ActionClaim
 	 */
-	public function stake_claim( $max_actions = 10, \DateTime $before_date = null, $hooks = array(), $group = '' ) {
+	public function stake_claim($max_actions = 10, \DateTime $before_date = null, $hooks = array(), $group = '') {
 		$claim_id = $this->generate_claim_id();
-		$this->claim_actions( $claim_id, $max_actions, $before_date, $hooks, $group );
-		$action_ids = $this->find_actions_by_claim_id( $claim_id );
+		$this->claim_actions($claim_id, $max_actions, $before_date, $hooks, $group);
+		$action_ids = $this->find_actions_by_claim_id($claim_id);
 
-		return new ActionScheduler_ActionClaim( $claim_id, $action_ids );
+		return new ActionScheduler_ActionClaim($claim_id, $action_ids);
 	}
 
 	/**
@@ -611,7 +611,7 @@ 
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 		$now = as_get_datetime_object();
-		$wpdb->insert( $wpdb->actionscheduler_claims, [ 'date_created_gmt' => $now->format( 'Y-m-d H:i:s' ) ] );
+		$wpdb->insert($wpdb->actionscheduler_claims, ['date_created_gmt' => $now->format('Y-m-d H:i:s')]);
 
 		return $wpdb->insert_id;
 	}
@@ -626,39 +626,39 @@ 
 	 * @return int The number of actions that were claimed.
 	 * @throws \RuntimeException
 	 */
-	protected function claim_actions( $claim_id, $limit, \DateTime $before_date = null, $hooks = array(), $group = '' ) {
+	protected function claim_actions($claim_id, $limit, \DateTime $before_date = null, $hooks = array(), $group = '') {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 
 		$now  = as_get_datetime_object();
-		$date = is_null( $before_date ) ? $now : clone $before_date;
+		$date = is_null($before_date) ? $now : clone $before_date;
 
 		// can't use $wpdb->update() because of the <= condition
 		$update = "UPDATE {$wpdb->actionscheduler_actions} SET claim_id=%d, last_attempt_gmt=%s, last_attempt_local=%s";
 		$params = array(
 			$claim_id,
-			$now->format( 'Y-m-d H:i:s' ),
-			current_time( 'mysql' ),
+			$now->format('Y-m-d H:i:s'),
+			current_time('mysql'),
 		);
 
 		$where    = "WHERE claim_id = 0 AND scheduled_date_gmt <= %s AND status=%s";
-		$params[] = $date->format( 'Y-m-d H:i:s' );
+		$params[] = $date->format('Y-m-d H:i:s');
 		$params[] = self::STATUS_PENDING;
 
-		if ( ! empty( $hooks ) ) {
-			$placeholders = array_fill( 0, count( $hooks ), '%s' );
-			$where       .= ' AND hook IN (' . join( ', ', $placeholders ) . ')';
-			$params       = array_merge( $params, array_values( $hooks ) );
+		if (!empty($hooks)) {
+			$placeholders = array_fill(0, count($hooks), '%s');
+			$where       .= ' AND hook IN (' . join(', ', $placeholders) . ')';
+			$params       = array_merge($params, array_values($hooks));
 		}
 
-		if ( ! empty( $group ) ) {
+		if (!empty($group)) {
 
-			$group_id = $this->get_group_id( $group, false );
+			$group_id = $this->get_group_id($group, false);
 
 			// throw exception if no matching group found, this matches ActionScheduler_wpPostStore's behaviour
-			if ( empty( $group_id ) ) {
+			if (empty($group_id)) {
 				/* translators: %s: group name */
-				throw new InvalidArgumentException( sprintf( __( 'The group "%s" does not exist.', 'action-scheduler' ), $group ) );
+				throw new InvalidArgumentException(sprintf(__('The group "%s" does not exist.', 'action-scheduler'), $group));
 			}
 
 			$where   .= ' AND group_id = %d';
@@ -668,11 +668,11 @@ 
 		$order    = "ORDER BY attempts ASC, scheduled_date_gmt ASC, action_id ASC LIMIT %d";
 		$params[] = $limit;
 
-		$sql = $wpdb->prepare( "{$update} {$where} {$order}", $params );
+		$sql = $wpdb->prepare("{$update} {$where} {$order}", $params);
 
-		$rows_affected = $wpdb->query( $sql );
-		if ( $rows_affected === false ) {
-			throw new \RuntimeException( __( 'Unable to claim actions. Database error.', 'action-scheduler' ) );
+		$rows_affected = $wpdb->query($sql);
+		if ($rows_affected === false) {
+			throw new \RuntimeException(__('Unable to claim actions. Database error.', 'action-scheduler'));
 		}
 
 		return (int) $rows_affected;
@@ -687,9 +687,9 @@ 
 		global $wpdb;
 
 		$sql = "SELECT COUNT(DISTINCT claim_id) FROM {$wpdb->actionscheduler_actions} WHERE claim_id != 0 AND status IN ( %s, %s)";
-		$sql = $wpdb->prepare( $sql, [ self::STATUS_PENDING, self::STATUS_RUNNING ] );
+		$sql = $wpdb->prepare($sql, [self::STATUS_PENDING, self::STATUS_RUNNING]);
 
-		return (int) $wpdb->get_var( $sql );
+		return (int) $wpdb->get_var($sql);
 	}
 
 	/**
@@ -698,14 +698,14 @@ 
 	 * @param string $action_id Action ID.
 	 * @return mixed
 	 */
-	public function get_claim_id( $action_id ) {
+	public function get_claim_id($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 
 		$sql = "SELECT claim_id FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d";
-		$sql = $wpdb->prepare( $sql, $action_id );
+		$sql = $wpdb->prepare($sql, $action_id);
 
-		return (int) $wpdb->get_var( $sql );
+		return (int) $wpdb->get_var($sql);
 	}
 
 	/**
@@ -715,16 +715,16 @@ 
 	 *
 	 * @return int[]
 	 */
-	public function find_actions_by_claim_id( $claim_id ) {
+	public function find_actions_by_claim_id($claim_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 
 		$sql = "SELECT action_id FROM {$wpdb->actionscheduler_actions} WHERE claim_id=%d";
-		$sql = $wpdb->prepare( $sql, $claim_id );
+		$sql = $wpdb->prepare($sql, $claim_id);
 
-		$action_ids = $wpdb->get_col( $sql );
+		$action_ids = $wpdb->get_col($sql);
 
-		return array_map( 'intval', $action_ids );
+		return array_map('intval', $action_ids);
 	}
 
 	/**
@@ -732,11 +732,11 @@ 
 	 *
 	 * @param ActionScheduler_ActionClaim $claim Claim object.
 	 */
-	public function release_claim( ActionScheduler_ActionClaim $claim ) {
+	public function release_claim(ActionScheduler_ActionClaim $claim) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$wpdb->update( $wpdb->actionscheduler_actions, [ 'claim_id' => 0 ], [ 'claim_id' => $claim->get_id() ], [ '%d' ], [ '%d' ] );
-		$wpdb->delete( $wpdb->actionscheduler_claims, [ 'claim_id' => $claim->get_id() ], [ '%d' ] );
+		$wpdb->update($wpdb->actionscheduler_actions, ['claim_id' => 0], ['claim_id' => $claim->get_id()], ['%d'], ['%d']);
+		$wpdb->delete($wpdb->actionscheduler_claims, ['claim_id' => $claim->get_id()], ['%d']);
 	}
 
 	/**
@@ -746,15 +746,15 @@ 
 	 *
 	 * @return void
 	 */
-	public function unclaim_action( $action_id ) {
+	public function unclaim_action($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 		$wpdb->update(
 			$wpdb->actionscheduler_actions,
-			[ 'claim_id' => 0 ],
-			[ 'action_id' => $action_id ],
-			[ '%s' ],
-			[ '%d' ]
+			['claim_id' => 0],
+			['action_id' => $action_id],
+			['%s'],
+			['%d']
 		);
 	}
 
@@ -763,18 +763,18 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function mark_failure( $action_id ) {
+	public function mark_failure($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 		$updated = $wpdb->update(
 			$wpdb->actionscheduler_actions,
-			[ 'status' => self::STATUS_FAILED ],
-			[ 'action_id' => $action_id ],
-			[ '%s' ],
-			[ '%d' ]
+			['status' => self::STATUS_FAILED],
+			['action_id' => $action_id],
+			['%s'],
+			['%d']
 		);
-		if ( empty( $updated ) ) {
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+		if (empty($updated)) {
+			throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'action-scheduler'), $action_id));
 		}
 	}
 
@@ -785,13 +785,13 @@ 
 	 *
 	 * @return void
 	 */
-	public function log_execution( $action_id ) {
+	public function log_execution($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 
 		$sql = "UPDATE {$wpdb->actionscheduler_actions} SET attempts = attempts+1, status=%s, last_attempt_gmt = %s, last_attempt_local = %s WHERE action_id = %d";
-		$sql = $wpdb->prepare( $sql, self::STATUS_RUNNING, current_time( 'mysql', true ), current_time( 'mysql' ), $action_id );
-		$wpdb->query( $sql );
+		$sql = $wpdb->prepare($sql, self::STATUS_RUNNING, current_time('mysql', true), current_time('mysql'), $action_id);
+		$wpdb->query($sql);
 	}
 
 	/**
@@ -801,22 +801,22 @@ 
 	 *
 	 * @return void
 	 */
-	public function mark_complete( $action_id ) {
+	public function mark_complete($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 		$updated = $wpdb->update(
 			$wpdb->actionscheduler_actions,
 			[
 				'status'             => self::STATUS_COMPLETE,
-				'last_attempt_gmt'   => current_time( 'mysql', true ),
-				'last_attempt_local' => current_time( 'mysql' ),
+				'last_attempt_gmt'   => current_time('mysql', true),
+				'last_attempt_local' => current_time('mysql'),
 			],
-			[ 'action_id' => $action_id ],
-			[ '%s' ],
-			[ '%d' ]
+			['action_id' => $action_id],
+			['%s'],
+			['%d']
 		);
-		if ( empty( $updated ) ) {
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+		if (empty($updated)) {
+			throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'action-scheduler'), $action_id));
 		}
 	}
 
@@ -827,17 +827,17 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_status( $action_id ) {
+	public function get_status($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 		$sql    = "SELECT status FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d";
-		$sql    = $wpdb->prepare( $sql, $action_id );
-		$status = $wpdb->get_var( $sql );
+		$sql    = $wpdb->prepare($sql, $action_id);
+		$status = $wpdb->get_var($sql);
 
-		if ( $status === null ) {
-			throw new \InvalidArgumentException( __( 'Invalid action ID. No status found.', 'action-scheduler' ) );
-		} elseif ( empty( $status ) ) {
-			throw new \RuntimeException( __( 'Unknown status found for action.', 'action-scheduler' ) );
+		if ($status === null) {
+			throw new \InvalidArgumentException(__('Invalid action ID. No status found.', 'action-scheduler'));
+		} elseif (empty($status)) {
+			throw new \RuntimeException(__('Unknown status found for action.', 'action-scheduler'));
 		} else {
 			return $status;
 		}

Braces +1 added lines, -2 removed lines

@@ -62,8 +62,7 @@
 
 			if ( is_wp_error( $action_id ) ) {
 				throw new RuntimeException( $action_id->get_error_message() );
-			}
-			elseif ( empty( $action_id ) ) {
+			} elseif ( empty( $action_id ) ) {
 				throw new RuntimeException( $wpdb->last_error ? $wpdb->last_error : __( 'Database error.', 'action-scheduler' ) );
 			}