AyeCode / invoicing

includes/libraries/action-scheduler/classes/migration/ActionMigrator.php

Indentation +93 added lines, -93 removed lines

@@ -13,97 +13,97 @@
  * @codeCoverageIgnore
  */
 class ActionMigrator {
-	/** var ActionScheduler_Store */
-	private $source;
-
-	/** var ActionScheduler_Store */
-	private $destination;
-
-	/** var LogMigrator */
-	private $log_migrator;
-
-	/**
-	 * ActionMigrator constructor.
-	 *
-	 * @param ActionScheduler_Store $source_store Source store object.
-	 * @param ActionScheduler_Store $destination_store Destination store object.
-	 * @param LogMigrator           $log_migrator Log migrator object.
-	 */
-	public function __construct( \ActionScheduler_Store $source_store, \ActionScheduler_Store $destination_store, LogMigrator $log_migrator ) {
-		$this->source       = $source_store;
-		$this->destination  = $destination_store;
-		$this->log_migrator = $log_migrator;
-	}
-
-	/**
-	 * Migrate an action.
-	 *
-	 * @param int $source_action_id Action ID.
-	 *
-	 * @return int 0|new action ID
-	 */
-	public function migrate( $source_action_id ) {
-		try {
-			$action = $this->source->fetch_action( $source_action_id );
-			$status = $this->source->get_status( $source_action_id );
-		} catch ( \Exception $e ) {
-			$action = null;
-			$status = '';
-		}
-
-		if ( is_null( $action ) || empty( $status ) || ! $action->get_schedule()->get_date() ) {
-			// null action or empty status means the fetch operation failed or the action didn't exist
-			// null schedule means it's missing vital data
-			// delete it and move on
-			try {
-				$this->source->delete_action( $source_action_id );
-			} catch ( \Exception $e ) {
-				// nothing to do, it didn't exist in the first place
-			}
-			do_action( 'action_scheduler/no_action_to_migrate', $source_action_id, $this->source, $this->destination );
-
-			return 0;
-		}
-
-		try {
-
-			// Make sure the last attempt date is set correctly for completed and failed actions
-			$last_attempt_date = ( $status !== \ActionScheduler_Store::STATUS_PENDING ) ? $this->source->get_date( $source_action_id ) : null;
-
-			$destination_action_id = $this->destination->save_action( $action, null, $last_attempt_date );
-		} catch ( \Exception $e ) {
-			do_action( 'action_scheduler/migrate_action_failed', $source_action_id, $this->source, $this->destination );
-
-			return 0; // could not save the action in the new store
-		}
-
-		try {
-			switch ( $status ) {
-				case \ActionScheduler_Store::STATUS_FAILED :
-					$this->destination->mark_failure( $destination_action_id );
-					break;
-				case \ActionScheduler_Store::STATUS_CANCELED :
-					$this->destination->cancel_action( $destination_action_id );
-					break;
-			}
-
-			$this->log_migrator->migrate( $source_action_id, $destination_action_id );
-			$this->source->delete_action( $source_action_id );
-
-			$test_action = $this->source->fetch_action( $source_action_id );
-			if ( ! is_a( $test_action, 'ActionScheduler_NullAction' ) ) {
-				throw new \RuntimeException( sprintf( __( 'Unable to remove source migrated action %s', 'action-scheduler' ), $source_action_id ) );
-			}
-			do_action( 'action_scheduler/migrated_action', $source_action_id, $destination_action_id, $this->source, $this->destination );
-
-			return $destination_action_id;
-		} catch ( \Exception $e ) {
-			// could not delete from the old store
-			$this->source->mark_migrated( $source_action_id );
-			do_action( 'action_scheduler/migrate_action_incomplete', $source_action_id, $destination_action_id, $this->source, $this->destination );
-			do_action( 'action_scheduler/migrated_action', $source_action_id, $destination_action_id, $this->source, $this->destination );
-
-			return $destination_action_id;
-		}
-	}
+    /** var ActionScheduler_Store */
+    private $source;
+
+    /** var ActionScheduler_Store */
+    private $destination;
+
+    /** var LogMigrator */
+    private $log_migrator;
+
+    /**
+     * ActionMigrator constructor.
+     *
+     * @param ActionScheduler_Store $source_store Source store object.
+     * @param ActionScheduler_Store $destination_store Destination store object.
+     * @param LogMigrator           $log_migrator Log migrator object.
+     */
+    public function __construct( \ActionScheduler_Store $source_store, \ActionScheduler_Store $destination_store, LogMigrator $log_migrator ) {
+        $this->source       = $source_store;
+        $this->destination  = $destination_store;
+        $this->log_migrator = $log_migrator;
+    }
+
+    /**
+     * Migrate an action.
+     *
+     * @param int $source_action_id Action ID.
+     *
+     * @return int 0|new action ID
+     */
+    public function migrate( $source_action_id ) {
+        try {
+            $action = $this->source->fetch_action( $source_action_id );
+            $status = $this->source->get_status( $source_action_id );
+        } catch ( \Exception $e ) {
+            $action = null;
+            $status = '';
+        }
+
+        if ( is_null( $action ) || empty( $status ) || ! $action->get_schedule()->get_date() ) {
+            // null action or empty status means the fetch operation failed or the action didn't exist
+            // null schedule means it's missing vital data
+            // delete it and move on
+            try {
+                $this->source->delete_action( $source_action_id );
+            } catch ( \Exception $e ) {
+                // nothing to do, it didn't exist in the first place
+            }
+            do_action( 'action_scheduler/no_action_to_migrate', $source_action_id, $this->source, $this->destination );
+
+            return 0;
+        }
+
+        try {
+
+            // Make sure the last attempt date is set correctly for completed and failed actions
+            $last_attempt_date = ( $status !== \ActionScheduler_Store::STATUS_PENDING ) ? $this->source->get_date( $source_action_id ) : null;
+
+            $destination_action_id = $this->destination->save_action( $action, null, $last_attempt_date );
+        } catch ( \Exception $e ) {
+            do_action( 'action_scheduler/migrate_action_failed', $source_action_id, $this->source, $this->destination );
+
+            return 0; // could not save the action in the new store
+        }
+
+        try {
+            switch ( $status ) {
+                case \ActionScheduler_Store::STATUS_FAILED :
+                    $this->destination->mark_failure( $destination_action_id );
+                    break;
+                case \ActionScheduler_Store::STATUS_CANCELED :
+                    $this->destination->cancel_action( $destination_action_id );
+                    break;
+            }
+
+            $this->log_migrator->migrate( $source_action_id, $destination_action_id );
+            $this->source->delete_action( $source_action_id );
+
+            $test_action = $this->source->fetch_action( $source_action_id );
+            if ( ! is_a( $test_action, 'ActionScheduler_NullAction' ) ) {
+                throw new \RuntimeException( sprintf( __( 'Unable to remove source migrated action %s', 'action-scheduler' ), $source_action_id ) );
+            }
+            do_action( 'action_scheduler/migrated_action', $source_action_id, $destination_action_id, $this->source, $this->destination );
+
+            return $destination_action_id;
+        } catch ( \Exception $e ) {
+            // could not delete from the old store
+            $this->source->mark_migrated( $source_action_id );
+            do_action( 'action_scheduler/migrate_action_incomplete', $source_action_id, $destination_action_id, $this->source, $this->destination );
+            do_action( 'action_scheduler/migrated_action', $source_action_id, $destination_action_id, $this->source, $this->destination );
+
+            return $destination_action_id;
+        }
+    }
 }

Spacing +26 added lines, -26 removed lines

@@ -29,7 +29,7 @@ 
 	 * @param ActionScheduler_Store $destination_store Destination store object.
 	 * @param LogMigrator           $log_migrator Log migrator object.
 	 */
-	public function __construct( \ActionScheduler_Store $source_store, \ActionScheduler_Store $destination_store, LogMigrator $log_migrator ) {
+	public function __construct(\ActionScheduler_Store $source_store, \ActionScheduler_Store $destination_store, LogMigrator $log_migrator) {
 		$this->source       = $source_store;
 		$this->destination  = $destination_store;
 		$this->log_migrator = $log_migrator;
@@ -42,25 +42,25 @@ 
 	 *
 	 * @return int 0|new action ID
 	 */
-	public function migrate( $source_action_id ) {
+	public function migrate($source_action_id) {
 		try {
-			$action = $this->source->fetch_action( $source_action_id );
-			$status = $this->source->get_status( $source_action_id );
-		} catch ( \Exception $e ) {
+			$action = $this->source->fetch_action($source_action_id);
+			$status = $this->source->get_status($source_action_id);
+		} catch (\Exception $e) {
 			$action = null;
 			$status = '';
 		}
 
-		if ( is_null( $action ) || empty( $status ) || ! $action->get_schedule()->get_date() ) {
+		if (is_null($action) || empty($status) || !$action->get_schedule()->get_date()) {
 			// null action or empty status means the fetch operation failed or the action didn't exist
 			// null schedule means it's missing vital data
 			// delete it and move on
 			try {
-				$this->source->delete_action( $source_action_id );
-			} catch ( \Exception $e ) {
+				$this->source->delete_action($source_action_id);
+			} catch (\Exception $e) {
 				// nothing to do, it didn't exist in the first place
 			}
-			do_action( 'action_scheduler/no_action_to_migrate', $source_action_id, $this->source, $this->destination );
+			do_action('action_scheduler/no_action_to_migrate', $source_action_id, $this->source, $this->destination);
 
 			return 0;
 		}
@@ -68,40 +68,40 @@ 
 		try {
 
 			// Make sure the last attempt date is set correctly for completed and failed actions
-			$last_attempt_date = ( $status !== \ActionScheduler_Store::STATUS_PENDING ) ? $this->source->get_date( $source_action_id ) : null;
+			$last_attempt_date = ($status !== \ActionScheduler_Store::STATUS_PENDING) ? $this->source->get_date($source_action_id) : null;
 
-			$destination_action_id = $this->destination->save_action( $action, null, $last_attempt_date );
-		} catch ( \Exception $e ) {
-			do_action( 'action_scheduler/migrate_action_failed', $source_action_id, $this->source, $this->destination );
+			$destination_action_id = $this->destination->save_action($action, null, $last_attempt_date);
+		} catch (\Exception $e) {
+			do_action('action_scheduler/migrate_action_failed', $source_action_id, $this->source, $this->destination);
 
 			return 0; // could not save the action in the new store
 		}
 
 		try {
-			switch ( $status ) {
+			switch ($status) {
 				case \ActionScheduler_Store::STATUS_FAILED :
-					$this->destination->mark_failure( $destination_action_id );
+					$this->destination->mark_failure($destination_action_id);
 					break;
 				case \ActionScheduler_Store::STATUS_CANCELED :
-					$this->destination->cancel_action( $destination_action_id );
+					$this->destination->cancel_action($destination_action_id);
 					break;
 			}
 
-			$this->log_migrator->migrate( $source_action_id, $destination_action_id );
-			$this->source->delete_action( $source_action_id );
+			$this->log_migrator->migrate($source_action_id, $destination_action_id);
+			$this->source->delete_action($source_action_id);
 
-			$test_action = $this->source->fetch_action( $source_action_id );
-			if ( ! is_a( $test_action, 'ActionScheduler_NullAction' ) ) {
-				throw new \RuntimeException( sprintf( __( 'Unable to remove source migrated action %s', 'action-scheduler' ), $source_action_id ) );
+			$test_action = $this->source->fetch_action($source_action_id);
+			if (!is_a($test_action, 'ActionScheduler_NullAction')) {
+				throw new \RuntimeException(sprintf(__('Unable to remove source migrated action %s', 'action-scheduler'), $source_action_id));
 			}
-			do_action( 'action_scheduler/migrated_action', $source_action_id, $destination_action_id, $this->source, $this->destination );
+			do_action('action_scheduler/migrated_action', $source_action_id, $destination_action_id, $this->source, $this->destination);
 
 			return $destination_action_id;
-		} catch ( \Exception $e ) {
+		} catch (\Exception $e) {
 			// could not delete from the old store
-			$this->source->mark_migrated( $source_action_id );
-			do_action( 'action_scheduler/migrate_action_incomplete', $source_action_id, $destination_action_id, $this->source, $this->destination );
-			do_action( 'action_scheduler/migrated_action', $source_action_id, $destination_action_id, $this->source, $this->destination );
+			$this->source->mark_migrated($source_action_id);
+			do_action('action_scheduler/migrate_action_incomplete', $source_action_id, $destination_action_id, $this->source, $this->destination);
+			do_action('action_scheduler/migrated_action', $source_action_id, $destination_action_id, $this->source, $this->destination);
 
 			return $destination_action_id;
 		}

includes/libraries/action-scheduler/classes/schedules/ActionScheduler_IntervalSchedule.php

Indentation +64 added lines, -64 removed lines

@@ -5,77 +5,77 @@
  */
 class ActionScheduler_IntervalSchedule extends ActionScheduler_Abstract_RecurringSchedule implements ActionScheduler_Schedule {
 
-	/**
-	 * Deprecated property @see $this->__wakeup() for details.
-	 **/
-	private $start_timestamp = NULL;
+    /**
+     * Deprecated property @see $this->__wakeup() for details.
+     **/
+    private $start_timestamp = NULL;
 
-	/**
-	 * Deprecated property @see $this->__wakeup() for details.
-	 **/
-	private $interval_in_seconds = NULL;
+    /**
+     * Deprecated property @see $this->__wakeup() for details.
+     **/
+    private $interval_in_seconds = NULL;
 
-	/**
-	 * Calculate when this schedule should start after a given date & time using
-	 * the number of seconds between recurrences.
-	 *
-	 * @param DateTime $after
-	 * @return DateTime
-	 */
-	protected function calculate_next( DateTime $after ) {
-		$after->modify( '+' . (int) $this->get_recurrence() . ' seconds' );
-		return $after;
-	}
+    /**
+     * Calculate when this schedule should start after a given date & time using
+     * the number of seconds between recurrences.
+     *
+     * @param DateTime $after
+     * @return DateTime
+     */
+    protected function calculate_next( DateTime $after ) {
+        $after->modify( '+' . (int) $this->get_recurrence() . ' seconds' );
+        return $after;
+    }
 
-	/**
-	 * @return int
-	 */
-	public function interval_in_seconds() {
-		_deprecated_function( __METHOD__, '3.0.0', '(int)ActionScheduler_Abstract_RecurringSchedule::get_recurrence()' );
-		return (int) $this->get_recurrence();
-	}
+    /**
+     * @return int
+     */
+    public function interval_in_seconds() {
+        _deprecated_function( __METHOD__, '3.0.0', '(int)ActionScheduler_Abstract_RecurringSchedule::get_recurrence()' );
+        return (int) $this->get_recurrence();
+    }
 
-	/**
-	 * Serialize interval schedules with data required prior to AS 3.0.0
-	 *
-	 * Prior to Action Scheduler 3.0.0, reccuring schedules used different property names to
-	 * refer to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
-	 * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
-	 * aligned properties and property names for better inheritance. To guard against the
-	 * possibility of infinite loops if downgrading to Action Scheduler < 3.0.0, we need to
-	 * also store the data with the old property names so if it's unserialized in AS < 3.0,
-	 * the schedule doesn't end up with a null/false/0 recurrence.
-	 *
-	 * @return array
-	 */
-	public function __sleep() {
+    /**
+     * Serialize interval schedules with data required prior to AS 3.0.0
+     *
+     * Prior to Action Scheduler 3.0.0, reccuring schedules used different property names to
+     * refer to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
+     * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
+     * aligned properties and property names for better inheritance. To guard against the
+     * possibility of infinite loops if downgrading to Action Scheduler < 3.0.0, we need to
+     * also store the data with the old property names so if it's unserialized in AS < 3.0,
+     * the schedule doesn't end up with a null/false/0 recurrence.
+     *
+     * @return array
+     */
+    public function __sleep() {
 
-		$sleep_params = parent::__sleep();
+        $sleep_params = parent::__sleep();
 
-		$this->start_timestamp     = $this->scheduled_timestamp;
-		$this->interval_in_seconds = $this->recurrence;
+        $this->start_timestamp     = $this->scheduled_timestamp;
+        $this->interval_in_seconds = $this->recurrence;
 
-		return array_merge( $sleep_params, array(
-			'start_timestamp',
-			'interval_in_seconds'
-		) );
-	}
+        return array_merge( $sleep_params, array(
+            'start_timestamp',
+            'interval_in_seconds'
+        ) );
+    }
 
-	/**
-	 * Unserialize interval schedules serialized/stored prior to AS 3.0.0
-	 *
-	 * For more background, @see ActionScheduler_Abstract_RecurringSchedule::__wakeup().
-	 */
-	public function __wakeup() {
-		if ( is_null( $this->scheduled_timestamp ) && ! is_null( $this->start_timestamp ) ) {
-			$this->scheduled_timestamp = $this->start_timestamp;
-			unset( $this->start_timestamp );
-		}
+    /**
+     * Unserialize interval schedules serialized/stored prior to AS 3.0.0
+     *
+     * For more background, @see ActionScheduler_Abstract_RecurringSchedule::__wakeup().
+     */
+    public function __wakeup() {
+        if ( is_null( $this->scheduled_timestamp ) && ! is_null( $this->start_timestamp ) ) {
+            $this->scheduled_timestamp = $this->start_timestamp;
+            unset( $this->start_timestamp );
+        }
 
-		if ( is_null( $this->recurrence ) && ! is_null( $this->interval_in_seconds ) ) {
-			$this->recurrence = $this->interval_in_seconds;
-			unset( $this->interval_in_seconds );
-		}
-		parent::__wakeup();
-	}
+        if ( is_null( $this->recurrence ) && ! is_null( $this->interval_in_seconds ) ) {
+            $this->recurrence = $this->interval_in_seconds;
+            unset( $this->interval_in_seconds );
+        }
+        parent::__wakeup();
+    }
 }

Spacing +9 added lines, -9 removed lines

@@ -22,8 +22,8 @@ 
 	 * @param DateTime $after
 	 * @return DateTime
 	 */
-	protected function calculate_next( DateTime $after ) {
-		$after->modify( '+' . (int) $this->get_recurrence() . ' seconds' );
+	protected function calculate_next(DateTime $after) {
+		$after->modify('+' . (int) $this->get_recurrence() . ' seconds');
 		return $after;
 	}
 
@@ -31,7 +31,7 @@ 
 	 * @return int
 	 */
 	public function interval_in_seconds() {
-		_deprecated_function( __METHOD__, '3.0.0', '(int)ActionScheduler_Abstract_RecurringSchedule::get_recurrence()' );
+		_deprecated_function(__METHOD__, '3.0.0', '(int)ActionScheduler_Abstract_RecurringSchedule::get_recurrence()');
 		return (int) $this->get_recurrence();
 	}
 
@@ -55,10 +55,10 @@ 
 		$this->start_timestamp     = $this->scheduled_timestamp;
 		$this->interval_in_seconds = $this->recurrence;
 
-		return array_merge( $sleep_params, array(
+		return array_merge($sleep_params, array(
 			'start_timestamp',
 			'interval_in_seconds'
-		) );
+		));
 	}
 
 	/**
@@ -67,14 +67,14 @@ 
 	 * For more background, @see ActionScheduler_Abstract_RecurringSchedule::__wakeup().
 	 */
 	public function __wakeup() {
-		if ( is_null( $this->scheduled_timestamp ) && ! is_null( $this->start_timestamp ) ) {
+		if (is_null($this->scheduled_timestamp) && !is_null($this->start_timestamp)) {
 			$this->scheduled_timestamp = $this->start_timestamp;
-			unset( $this->start_timestamp );
+			unset($this->start_timestamp);
 		}
 
-		if ( is_null( $this->recurrence ) && ! is_null( $this->interval_in_seconds ) ) {
+		if (is_null($this->recurrence) && !is_null($this->interval_in_seconds)) {
 			$this->recurrence = $this->interval_in_seconds;
-			unset( $this->interval_in_seconds );
+			unset($this->interval_in_seconds);
 		}
 		parent::__wakeup();
 	}

includes/libraries/action-scheduler/classes/schedules/ActionScheduler_SimpleSchedule.php

Indentation +55 added lines, -55 removed lines

@@ -5,67 +5,67 @@
  */
 class ActionScheduler_SimpleSchedule extends ActionScheduler_Abstract_Schedule {
 
-	/**
-	 * Deprecated property @see $this->__wakeup() for details.
-	 **/
-	private $timestamp = NULL;
+    /**
+     * Deprecated property @see $this->__wakeup() for details.
+     **/
+    private $timestamp = NULL;
 
-	/**
-	 * @param DateTime $after
-	 *
-	 * @return DateTime|null
-	 */
-	public function calculate_next( DateTime $after ) {
-		return null;
-	}
+    /**
+     * @param DateTime $after
+     *
+     * @return DateTime|null
+     */
+    public function calculate_next( DateTime $after ) {
+        return null;
+    }
 
-	/**
-	 * @return bool
-	 */
-	public function is_recurring() {
-		return false;
-	}
+    /**
+     * @return bool
+     */
+    public function is_recurring() {
+        return false;
+    }
 
-	/**
-	 * Serialize schedule with data required prior to AS 3.0.0
-	 *
-	 * Prior to Action Scheduler 3.0.0, schedules used different property names to refer
-	 * to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
-	 * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
-	 * aligned properties and property names for better inheritance. To guard against the
-	 * scheduled date for single actions always being seen as "now" if downgrading to
-	 * Action Scheduler < 3.0.0, we need to also store the data with the old property names
-	 * so if it's unserialized in AS < 3.0, the schedule doesn't end up with a null recurrence.
-	 *
-	 * @return array
-	 */
-	public function __sleep() {
+    /**
+     * Serialize schedule with data required prior to AS 3.0.0
+     *
+     * Prior to Action Scheduler 3.0.0, schedules used different property names to refer
+     * to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
+     * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
+     * aligned properties and property names for better inheritance. To guard against the
+     * scheduled date for single actions always being seen as "now" if downgrading to
+     * Action Scheduler < 3.0.0, we need to also store the data with the old property names
+     * so if it's unserialized in AS < 3.0, the schedule doesn't end up with a null recurrence.
+     *
+     * @return array
+     */
+    public function __sleep() {
 
-		$sleep_params = parent::__sleep();
+        $sleep_params = parent::__sleep();
 
-		$this->timestamp = $this->scheduled_timestamp;
+        $this->timestamp = $this->scheduled_timestamp;
 
-		return array_merge( $sleep_params, array(
-			'timestamp',
-		) );
-	}
+        return array_merge( $sleep_params, array(
+            'timestamp',
+        ) );
+    }
 
-	/**
-	 * Unserialize recurring schedules serialized/stored prior to AS 3.0.0
-	 *
-	 * Prior to Action Scheduler 3.0.0, schedules used different property names to refer
-	 * to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
-	 * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
-	 * aligned properties and property names for better inheritance. To maintain backward
-	 * compatibility with schedules serialized and stored prior to 3.0, we need to correctly
-	 * map the old property names with matching visibility.
-	 */
-	public function __wakeup() {
+    /**
+     * Unserialize recurring schedules serialized/stored prior to AS 3.0.0
+     *
+     * Prior to Action Scheduler 3.0.0, schedules used different property names to refer
+     * to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
+     * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
+     * aligned properties and property names for better inheritance. To maintain backward
+     * compatibility with schedules serialized and stored prior to 3.0, we need to correctly
+     * map the old property names with matching visibility.
+     */
+    public function __wakeup() {
 
-		if ( is_null( $this->scheduled_timestamp ) && ! is_null( $this->timestamp ) ) {
-			$this->scheduled_timestamp = $this->timestamp;
-			unset( $this->timestamp );
-		}
-		parent::__wakeup();
-	}
+        if ( is_null( $this->scheduled_timestamp ) && ! is_null( $this->timestamp ) ) {
+            $this->scheduled_timestamp = $this->timestamp;
+            unset( $this->timestamp );
+        }
+        parent::__wakeup();
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -15,7 +15,7 @@ 
 	 *
 	 * @return DateTime|null
 	 */
-	public function calculate_next( DateTime $after ) {
+	public function calculate_next(DateTime $after) {
 		return null;
 	}
 
@@ -45,9 +45,9 @@ 
 
 		$this->timestamp = $this->scheduled_timestamp;
 
-		return array_merge( $sleep_params, array(
+		return array_merge($sleep_params, array(
 			'timestamp',
-		) );
+		));
 	}
 
 	/**
@@ -62,9 +62,9 @@ 
 	 */
 	public function __wakeup() {
 
-		if ( is_null( $this->scheduled_timestamp ) && ! is_null( $this->timestamp ) ) {
+		if (is_null($this->scheduled_timestamp) && !is_null($this->timestamp)) {
 			$this->scheduled_timestamp = $this->timestamp;
-			unset( $this->timestamp );
+			unset($this->timestamp);
 		}
 		parent::__wakeup();
 	}

includes/libraries/action-scheduler/classes/schedules/ActionScheduler_Schedule.php

Indentation +10 added lines, -10 removed lines

@@ -4,15 +4,15 @@
  * Class ActionScheduler_Schedule
  */
 interface ActionScheduler_Schedule {
-	/**
-	 * @param DateTime $after
-	 * @return DateTime|null
-	 */
-	public function next( DateTime $after = NULL );
+    /**
+     * @param DateTime $after
+     * @return DateTime|null
+     */
+    public function next( DateTime $after = NULL );
 
-	/**
-	 * @return bool
-	 */
-	public function is_recurring();
+    /**
+     * @return bool
+     */
+    public function is_recurring();
 }
- 
\ No newline at end of file
+    
\ No newline at end of file

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 	 * @param DateTime $after
 	 * @return DateTime|null
 	 */
-	public function next( DateTime $after = NULL );
+	public function next(DateTime $after = NULL);
 
 	/**
 	 * @return bool

includes/libraries/action-scheduler/classes/schedules/ActionScheduler_NullSchedule.php

Indentation +18 added lines, -18 removed lines

@@ -5,24 +5,24 @@
  */
 class ActionScheduler_NullSchedule extends ActionScheduler_SimpleSchedule {
 
-	/**
-	 * Make the $date param optional and default to null.
-	 *
-	 * @param null $date The date & time to run the action.
-	 */
-	public function __construct( DateTime $date = null ) {
-		$this->scheduled_date = null;
-	}
+    /**
+     * Make the $date param optional and default to null.
+     *
+     * @param null $date The date & time to run the action.
+     */
+    public function __construct( DateTime $date = null ) {
+        $this->scheduled_date = null;
+    }
 
-	/**
-	 * This schedule has no scheduled DateTime, so we need to override the parent __sleep()
-	 * @return array
-	 */
-	public function __sleep() {
-		return array();
-	}
+    /**
+     * This schedule has no scheduled DateTime, so we need to override the parent __sleep()
+     * @return array
+     */
+    public function __sleep() {
+        return array();
+    }
 
-	public function __wakeup() {
-		$this->scheduled_date = null;
-	}
+    public function __wakeup() {
+        $this->scheduled_date = null;
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 	 *
 	 * @param null $date The date & time to run the action.
 	 */
-	public function __construct( DateTime $date = null ) {
+	public function __construct(DateTime $date = null) {
 		$this->scheduled_date = null;
 	}
 

includes/libraries/action-scheduler/classes/schedules/ActionScheduler_CanceledSchedule.php

Indentation +45 added lines, -45 removed lines

@@ -5,53 +5,53 @@
  */
 class ActionScheduler_CanceledSchedule extends ActionScheduler_SimpleSchedule {
 
-	/**
-	 * Deprecated property @see $this->__wakeup() for details.
-	 **/
-	private $timestamp = NULL;
+    /**
+     * Deprecated property @see $this->__wakeup() for details.
+     **/
+    private $timestamp = NULL;
 
-	/**
-	 * @param DateTime $after
-	 *
-	 * @return DateTime|null
-	 */
-	public function calculate_next( DateTime $after ) {
-		return null;
-	}
+    /**
+     * @param DateTime $after
+     *
+     * @return DateTime|null
+     */
+    public function calculate_next( DateTime $after ) {
+        return null;
+    }
 
-	/**
-	 * Cancelled actions should never have a next schedule, even if get_next()
-	 * is called with $after < $this->scheduled_date.
-	 *
-	 * @param DateTime $after
-	 * @return DateTime|null
-	 */
-	public function get_next( DateTime $after ) {
-		return null;
-	}
+    /**
+     * Cancelled actions should never have a next schedule, even if get_next()
+     * is called with $after < $this->scheduled_date.
+     *
+     * @param DateTime $after
+     * @return DateTime|null
+     */
+    public function get_next( DateTime $after ) {
+        return null;
+    }
 
-	/**
-	 * @return bool
-	 */
-	public function is_recurring() {
-		return false;
-	}
+    /**
+     * @return bool
+     */
+    public function is_recurring() {
+        return false;
+    }
 
-	/**
-	 * Unserialize recurring schedules serialized/stored prior to AS 3.0.0
-	 *
-	 * Prior to Action Scheduler 3.0.0, schedules used different property names to refer
-	 * to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
-	 * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
-	 * aligned properties and property names for better inheritance. To maintain backward
-	 * compatibility with schedules serialized and stored prior to 3.0, we need to correctly
-	 * map the old property names with matching visibility.
-	 */
-	public function __wakeup() {
-		if ( ! is_null( $this->timestamp ) ) {
-			$this->scheduled_timestamp = $this->timestamp;
-			unset( $this->timestamp );
-		}
-		parent::__wakeup();
-	}
+    /**
+     * Unserialize recurring schedules serialized/stored prior to AS 3.0.0
+     *
+     * Prior to Action Scheduler 3.0.0, schedules used different property names to refer
+     * to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
+     * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
+     * aligned properties and property names for better inheritance. To maintain backward
+     * compatibility with schedules serialized and stored prior to 3.0, we need to correctly
+     * map the old property names with matching visibility.
+     */
+    public function __wakeup() {
+        if ( ! is_null( $this->timestamp ) ) {
+            $this->scheduled_timestamp = $this->timestamp;
+            unset( $this->timestamp );
+        }
+        parent::__wakeup();
+    }
 }

Spacing +4 added lines, -4 removed lines

@@ -15,7 +15,7 @@ 
 	 *
 	 * @return DateTime|null
 	 */
-	public function calculate_next( DateTime $after ) {
+	public function calculate_next(DateTime $after) {
 		return null;
 	}
 
@@ -26,7 +26,7 @@ 
 	 * @param DateTime $after
 	 * @return DateTime|null
 	 */
-	public function get_next( DateTime $after ) {
+	public function get_next(DateTime $after) {
 		return null;
 	}
 
@@ -48,9 +48,9 @@ 
 	 * map the old property names with matching visibility.
 	 */
 	public function __wakeup() {
-		if ( ! is_null( $this->timestamp ) ) {
+		if (!is_null($this->timestamp)) {
 			$this->scheduled_timestamp = $this->timestamp;
-			unset( $this->timestamp );
+			unset($this->timestamp);
 		}
 		parent::__wakeup();
 	}

includes/libraries/action-scheduler/classes/schedules/ActionScheduler_CronSchedule.php

Indentation +93 added lines, -93 removed lines

@@ -5,98 +5,98 @@
  */
 class ActionScheduler_CronSchedule extends ActionScheduler_Abstract_RecurringSchedule implements ActionScheduler_Schedule {
 
-	/**
-	 * Deprecated property @see $this->__wakeup() for details.
-	 **/
-	private $start_timestamp = NULL;
-
-	/**
-	 * Deprecated property @see $this->__wakeup() for details.
-	 **/
-	private $cron = NULL;
-
-	/**
-	 * Wrapper for parent constructor to accept a cron expression string and map it to a CronExpression for this
-	 * objects $recurrence property.
-	 *
-	 * @param DateTime $start The date & time to run the action at or after. If $start aligns with the CronSchedule passed via $recurrence, it will be used. If it does not align, the first matching date after it will be used.
-	 * @param CronExpression|string $recurrence The CronExpression used to calculate the schedule's next instance.
-	 * @param DateTime|null $first (Optional) The date & time the first instance of this interval schedule ran. Default null, meaning this is the first instance.
-	 */
-	public function __construct( DateTime $start, $recurrence, DateTime $first = null ) {
-		if ( ! is_a( $recurrence, 'CronExpression' ) ) {
-			$recurrence = CronExpression::factory( $recurrence );
-		}
-
-		// For backward compatibility, we need to make sure the date is set to the first matching cron date, not whatever date is passed in. Importantly, by passing true as the 3rd param, if $start matches the cron expression, then it will be used. This was previously handled in the now deprecated next() method.
-		$date = $recurrence->getNextRunDate( $start, 0, true );
-
-		// parent::__construct() will set this to $date by default, but that may be different to $start now.
-		$first = empty( $first ) ? $start : $first;
-
-		parent::__construct( $date, $recurrence, $first );
-	}
-
-	/**
-	 * Calculate when an instance of this schedule would start based on a given
-	 * date & time using its the CronExpression.
-	 *
-	 * @param DateTime $after
-	 * @return DateTime
-	 */
-	protected function calculate_next( DateTime $after ) {
-		return $this->recurrence->getNextRunDate( $after, 0, false );
-	}
-
-	/**
-	 * @return string
-	 */
-	public function get_recurrence() {
-		return strval( $this->recurrence );
-	}
-
-	/**
-	 * Serialize cron schedules with data required prior to AS 3.0.0
-	 *
-	 * Prior to Action Scheduler 3.0.0, reccuring schedules used different property names to
-	 * refer to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
-	 * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
-	 * aligned properties and property names for better inheritance. To guard against the
-	 * possibility of infinite loops if downgrading to Action Scheduler < 3.0.0, we need to
-	 * also store the data with the old property names so if it's unserialized in AS < 3.0,
-	 * the schedule doesn't end up with a null recurrence.
-	 *
-	 * @return array
-	 */
-	public function __sleep() {
-
-		$sleep_params = parent::__sleep();
-
-		$this->start_timestamp = $this->scheduled_timestamp;
-		$this->cron            = $this->recurrence;
-
-		return array_merge( $sleep_params, array(
-			'start_timestamp',
-			'cron'
-		) );
-	}
-
-	/**
-	 * Unserialize cron schedules serialized/stored prior to AS 3.0.0
-	 *
-	 * For more background, @see ActionScheduler_Abstract_RecurringSchedule::__wakeup().
-	 */
-	public function __wakeup() {
-		if ( is_null( $this->scheduled_timestamp ) && ! is_null( $this->start_timestamp ) ) {
-			$this->scheduled_timestamp = $this->start_timestamp;
-			unset( $this->start_timestamp );
-		}
-
-		if ( is_null( $this->recurrence ) && ! is_null( $this->cron ) ) {
-			$this->recurrence = $this->cron;
-			unset( $this->cron );
-		}
-		parent::__wakeup();
-	}
+    /**
+     * Deprecated property @see $this->__wakeup() for details.
+     **/
+    private $start_timestamp = NULL;
+
+    /**
+     * Deprecated property @see $this->__wakeup() for details.
+     **/
+    private $cron = NULL;
+
+    /**
+     * Wrapper for parent constructor to accept a cron expression string and map it to a CronExpression for this
+     * objects $recurrence property.
+     *
+     * @param DateTime $start The date & time to run the action at or after. If $start aligns with the CronSchedule passed via $recurrence, it will be used. If it does not align, the first matching date after it will be used.
+     * @param CronExpression|string $recurrence The CronExpression used to calculate the schedule's next instance.
+     * @param DateTime|null $first (Optional) The date & time the first instance of this interval schedule ran. Default null, meaning this is the first instance.
+     */
+    public function __construct( DateTime $start, $recurrence, DateTime $first = null ) {
+        if ( ! is_a( $recurrence, 'CronExpression' ) ) {
+            $recurrence = CronExpression::factory( $recurrence );
+        }
+
+        // For backward compatibility, we need to make sure the date is set to the first matching cron date, not whatever date is passed in. Importantly, by passing true as the 3rd param, if $start matches the cron expression, then it will be used. This was previously handled in the now deprecated next() method.
+        $date = $recurrence->getNextRunDate( $start, 0, true );
+
+        // parent::__construct() will set this to $date by default, but that may be different to $start now.
+        $first = empty( $first ) ? $start : $first;
+
+        parent::__construct( $date, $recurrence, $first );
+    }
+
+    /**
+     * Calculate when an instance of this schedule would start based on a given
+     * date & time using its the CronExpression.
+     *
+     * @param DateTime $after
+     * @return DateTime
+     */
+    protected function calculate_next( DateTime $after ) {
+        return $this->recurrence->getNextRunDate( $after, 0, false );
+    }
+
+    /**
+     * @return string
+     */
+    public function get_recurrence() {
+        return strval( $this->recurrence );
+    }
+
+    /**
+     * Serialize cron schedules with data required prior to AS 3.0.0
+     *
+     * Prior to Action Scheduler 3.0.0, reccuring schedules used different property names to
+     * refer to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
+     * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
+     * aligned properties and property names for better inheritance. To guard against the
+     * possibility of infinite loops if downgrading to Action Scheduler < 3.0.0, we need to
+     * also store the data with the old property names so if it's unserialized in AS < 3.0,
+     * the schedule doesn't end up with a null recurrence.
+     *
+     * @return array
+     */
+    public function __sleep() {
+
+        $sleep_params = parent::__sleep();
+
+        $this->start_timestamp = $this->scheduled_timestamp;
+        $this->cron            = $this->recurrence;
+
+        return array_merge( $sleep_params, array(
+            'start_timestamp',
+            'cron'
+        ) );
+    }
+
+    /**
+     * Unserialize cron schedules serialized/stored prior to AS 3.0.0
+     *
+     * For more background, @see ActionScheduler_Abstract_RecurringSchedule::__wakeup().
+     */
+    public function __wakeup() {
+        if ( is_null( $this->scheduled_timestamp ) && ! is_null( $this->start_timestamp ) ) {
+            $this->scheduled_timestamp = $this->start_timestamp;
+            unset( $this->start_timestamp );
+        }
+
+        if ( is_null( $this->recurrence ) && ! is_null( $this->cron ) ) {
+            $this->recurrence = $this->cron;
+            unset( $this->cron );
+        }
+        parent::__wakeup();
+    }
 }
 

Spacing +15 added lines, -15 removed lines

@@ -23,18 +23,18 @@ 
 	 * @param CronExpression|string $recurrence The CronExpression used to calculate the schedule's next instance.
 	 * @param DateTime|null $first (Optional) The date & time the first instance of this interval schedule ran. Default null, meaning this is the first instance.
 	 */
-	public function __construct( DateTime $start, $recurrence, DateTime $first = null ) {
-		if ( ! is_a( $recurrence, 'CronExpression' ) ) {
-			$recurrence = CronExpression::factory( $recurrence );
+	public function __construct(DateTime $start, $recurrence, DateTime $first = null) {
+		if (!is_a($recurrence, 'CronExpression')) {
+			$recurrence = CronExpression::factory($recurrence);
 		}
 
 		// For backward compatibility, we need to make sure the date is set to the first matching cron date, not whatever date is passed in. Importantly, by passing true as the 3rd param, if $start matches the cron expression, then it will be used. This was previously handled in the now deprecated next() method.
-		$date = $recurrence->getNextRunDate( $start, 0, true );
+		$date = $recurrence->getNextRunDate($start, 0, true);
 
 		// parent::__construct() will set this to $date by default, but that may be different to $start now.
-		$first = empty( $first ) ? $start : $first;
+		$first = empty($first) ? $start : $first;
 
-		parent::__construct( $date, $recurrence, $first );
+		parent::__construct($date, $recurrence, $first);
 	}
 
 	/**
@@ -44,15 +44,15 @@ 
 	 * @param DateTime $after
 	 * @return DateTime
 	 */
-	protected function calculate_next( DateTime $after ) {
-		return $this->recurrence->getNextRunDate( $after, 0, false );
+	protected function calculate_next(DateTime $after) {
+		return $this->recurrence->getNextRunDate($after, 0, false);
 	}
 
 	/**
 	 * @return string
 	 */
 	public function get_recurrence() {
-		return strval( $this->recurrence );
+		return strval($this->recurrence);
 	}
 
 	/**
@@ -75,10 +75,10 @@ 
 		$this->start_timestamp = $this->scheduled_timestamp;
 		$this->cron            = $this->recurrence;
 
-		return array_merge( $sleep_params, array(
+		return array_merge($sleep_params, array(
 			'start_timestamp',
 			'cron'
-		) );
+		));
 	}
 
 	/**
@@ -87,14 +87,14 @@ 
 	 * For more background, @see ActionScheduler_Abstract_RecurringSchedule::__wakeup().
 	 */
 	public function __wakeup() {
-		if ( is_null( $this->scheduled_timestamp ) && ! is_null( $this->start_timestamp ) ) {
+		if (is_null($this->scheduled_timestamp) && !is_null($this->start_timestamp)) {
 			$this->scheduled_timestamp = $this->start_timestamp;
-			unset( $this->start_timestamp );
+			unset($this->start_timestamp);
 		}
 
-		if ( is_null( $this->recurrence ) && ! is_null( $this->cron ) ) {
+		if (is_null($this->recurrence) && !is_null($this->cron)) {
 			$this->recurrence = $this->cron;
-			unset( $this->cron );
+			unset($this->cron);
 		}
 		parent::__wakeup();
 	}

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

Indentation +602 added lines, -602 removed lines

@@ -6,606 +6,606 @@
  */
 class ActionScheduler_ListTable extends ActionScheduler_Abstract_ListTable {
 
-	/**
-	 * The package name.
-	 *
-	 * @var string
-	 */
-	protected $package = 'action-scheduler';
-
-	/**
-	 * Columns to show (name => label).
-	 *
-	 * @var array
-	 */
-	protected $columns = array();
-
-	/**
-	 * Actions (name => label).
-	 *
-	 * @var array
-	 */
-	protected $row_actions = array();
-
-	/**
-	 * The active data stores
-	 *
-	 * @var ActionScheduler_Store
-	 */
-	protected $store;
-
-	/**
-	 * A logger to use for getting action logs to display
-	 *
-	 * @var ActionScheduler_Logger
-	 */
-	protected $logger;
-
-	/**
-	 * A ActionScheduler_QueueRunner runner instance (or child class)
-	 *
-	 * @var ActionScheduler_QueueRunner
-	 */
-	protected $runner;
-
-	/**
-	 * Bulk actions. The key of the array is the method name of the implementation:
-	 *
-	 *     bulk_<key>(array $ids, string $sql_in).
-	 *
-	 * See the comments in the parent class for further details
-	 *
-	 * @var array
-	 */
-	protected $bulk_actions = array();
-
-	/**
-	 * Flag variable to render our notifications, if any, once.
-	 *
-	 * @var bool
-	 */
-	protected static $did_notification = false;
-
-	/**
-	 * Array of seconds for common time periods, like week or month, alongside an internationalised string representation, i.e. "Day" or "Days"
-	 *
-	 * @var array
-	 */
-	private static $time_periods;
-
-	/**
-	 * Sets the current data store object into `store->action` and initialises the object.
-	 *
-	 * @param ActionScheduler_Store $store
-	 * @param ActionScheduler_Logger $logger
-	 * @param ActionScheduler_QueueRunner $runner
-	 */
-	public function __construct( ActionScheduler_Store $store, ActionScheduler_Logger $logger, ActionScheduler_QueueRunner $runner ) {
-
-		$this->store  = $store;
-		$this->logger = $logger;
-		$this->runner = $runner;
-
-		$this->table_header = __( 'Scheduled Actions', 'action-scheduler' );
-
-		$this->bulk_actions = array(
-			'delete' => __( 'Delete', 'action-scheduler' ),
-		);
-
-		$this->columns = array(
-			'hook'        => __( 'Hook', 'action-scheduler' ),
-			'status'      => __( 'Status', 'action-scheduler' ),
-			'args'        => __( 'Arguments', 'action-scheduler' ),
-			'group'       => __( 'Group', 'action-scheduler' ),
-			'recurrence'  => __( 'Recurrence', 'action-scheduler' ),
-			'schedule'    => __( 'Scheduled Date', 'action-scheduler' ),
-			'log_entries' => __( 'Log', 'action-scheduler' ),
-		);
-
-		$this->sort_by = array(
-			'schedule',
-			'hook',
-			'group',
-		);
-
-		$this->search_by = array(
-			'hook',
-			'args',
-			'claim_id',
-		);
-
-		$request_status = $this->get_request_status();
-
-		if ( empty( $request_status ) ) {
-			$this->sort_by[] = 'status';
-		} elseif ( in_array( $request_status, array( 'in-progress', 'failed' ) ) ) {
-			$this->columns  += array( 'claim_id' => __( 'Claim ID', 'action-scheduler' ) );
-			$this->sort_by[] = 'claim_id';
-		}
-
-		$this->row_actions = array(
-			'hook' => array(
-				'run' => array(
-					'name'  => __( 'Run', 'action-scheduler' ),
-					'desc'  => __( 'Process the action now as if it were run as part of a queue', 'action-scheduler' ),
-				),
-				'cancel' => array(
-					'name'  => __( 'Cancel', 'action-scheduler' ),
-					'desc'  => __( 'Cancel the action now to avoid it being run in future', 'action-scheduler' ),
-					'class' => 'cancel trash',
-				),
-			),
-		);
-
-		self::$time_periods = array(
-			array(
-				'seconds' => YEAR_IN_SECONDS,
-				/* translators: %s: amount of time */
-				'names'   => _n_noop( '%s year', '%s years', 'action-scheduler' ),
-			),
-			array(
-				'seconds' => MONTH_IN_SECONDS,
-				/* translators: %s: amount of time */
-				'names'   => _n_noop( '%s month', '%s months', 'action-scheduler' ),
-			),
-			array(
-				'seconds' => WEEK_IN_SECONDS,
-				/* translators: %s: amount of time */
-				'names'   => _n_noop( '%s week', '%s weeks', 'action-scheduler' ),
-			),
-			array(
-				'seconds' => DAY_IN_SECONDS,
-				/* translators: %s: amount of time */
-				'names'   => _n_noop( '%s day', '%s days', 'action-scheduler' ),
-			),
-			array(
-				'seconds' => HOUR_IN_SECONDS,
-				/* translators: %s: amount of time */
-				'names'   => _n_noop( '%s hour', '%s hours', 'action-scheduler' ),
-			),
-			array(
-				'seconds' => MINUTE_IN_SECONDS,
-				/* translators: %s: amount of time */
-				'names'   => _n_noop( '%s minute', '%s minutes', 'action-scheduler' ),
-			),
-			array(
-				'seconds' => 1,
-				/* translators: %s: amount of time */
-				'names'   => _n_noop( '%s second', '%s seconds', 'action-scheduler' ),
-			),
-		);
-
-		parent::__construct( array(
-			'singular' => 'action-scheduler',
-			'plural'   => 'action-scheduler',
-			'ajax'     => false,
-		) );
-	}
-
-	/**
-	 * Convert an interval of seconds into a two part human friendly string.
-	 *
-	 * The WordPress human_time_diff() function only calculates the time difference to one degree, meaning
-	 * even if an action is 1 day and 11 hours away, it will display "1 day". This function goes one step
-	 * further to display two degrees of accuracy.
-	 *
-	 * Inspired by the Crontrol::interval() function by Edward Dale: https://wordpress.org/plugins/wp-crontrol/
-	 *
-	 * @param int $interval A interval in seconds.
-	 * @param int $periods_to_include Depth of time periods to include, e.g. for an interval of 70, and $periods_to_include of 2, both minutes and seconds would be included. With a value of 1, only minutes would be included.
-	 * @return string A human friendly string representation of the interval.
-	 */
-	private static function human_interval( $interval, $periods_to_include = 2 ) {
-
-		if ( $interval <= 0 ) {
-			return __( 'Now!', 'action-scheduler' );
-		}
-
-		$output = '';
-
-		for ( $time_period_index = 0, $periods_included = 0, $seconds_remaining = $interval; $time_period_index < count( self::$time_periods ) && $seconds_remaining > 0 && $periods_included < $periods_to_include; $time_period_index++ ) {
-
-			$periods_in_interval = floor( $seconds_remaining / self::$time_periods[ $time_period_index ]['seconds'] );
-
-			if ( $periods_in_interval > 0 ) {
-				if ( ! empty( $output ) ) {
-					$output .= ' ';
-				}
-				$output .= sprintf( _n( self::$time_periods[ $time_period_index ]['names'][0], self::$time_periods[ $time_period_index ]['names'][1], $periods_in_interval, 'action-scheduler' ), $periods_in_interval );
-				$seconds_remaining -= $periods_in_interval * self::$time_periods[ $time_period_index ]['seconds'];
-				$periods_included++;
-			}
-		}
-
-		return $output;
-	}
-
-	/**
-	 * Returns the recurrence of an action or 'Non-repeating'. The output is human readable.
-	 *
-	 * @param ActionScheduler_Action $action
-	 *
-	 * @return string
-	 */
-	protected function get_recurrence( $action ) {
-		$schedule = $action->get_schedule();
-		if ( $schedule->is_recurring() ) {
-			$recurrence = $schedule->get_recurrence();
-
-			if ( is_numeric( $recurrence ) ) {
-				/* translators: %s: time interval */
-				return sprintf( __( 'Every %s', 'action-scheduler' ), self::human_interval( $recurrence ) );
-			} else {
-				return $recurrence;
-			}
-		}
-
-		return __( 'Non-repeating', 'action-scheduler' );
-	}
-
-	/**
-	 * Serializes the argument of an action to render it in a human friendly format.
-	 *
-	 * @param array $row The array representation of the current row of the table
-	 *
-	 * @return string
-	 */
-	public function column_args( array $row ) {
-		if ( empty( $row['args'] ) ) {
-			return apply_filters( 'action_scheduler_list_table_column_args', '', $row );
-		}
-
-		$row_html = '<ul>';
-		foreach ( $row['args'] as $key => $value ) {
-			$row_html .= sprintf( '<li><code>%s => %s</code></li>', esc_html( var_export( $key, true ) ), esc_html( var_export( $value, true ) ) );
-		}
-		$row_html .= '</ul>';
-
-		return apply_filters( 'action_scheduler_list_table_column_args', $row_html, $row );
-	}
-
-	/**
-	 * Prints the logs entries inline. We do so to avoid loading Javascript and other hacks to show it in a modal.
-	 *
-	 * @param array $row Action array.
-	 * @return string
-	 */
-	public function column_log_entries( array $row ) {
-
-		$log_entries_html = '<ol>';
-
-		$timezone = new DateTimezone( 'UTC' );
-
-		foreach ( $row['log_entries'] as $log_entry ) {
-			$log_entries_html .= $this->get_log_entry_html( $log_entry, $timezone );
-		}
-
-		$log_entries_html .= '</ol>';
-
-		return $log_entries_html;
-	}
-
-	/**
-	 * Prints the logs entries inline. We do so to avoid loading Javascript and other hacks to show it in a modal.
-	 *
-	 * @param ActionScheduler_LogEntry $log_entry
-	 * @param DateTimezone $timezone
-	 * @return string
-	 */
-	protected function get_log_entry_html( ActionScheduler_LogEntry $log_entry, DateTimezone $timezone ) {
-		$date = $log_entry->get_date();
-		$date->setTimezone( $timezone );
-		return sprintf( '<li><strong>%s</strong><br/>%s</li>', esc_html( $date->format( 'Y-m-d H:i:s O' ) ), esc_html( $log_entry->get_message() ) );
-	}
-
-	/**
-	 * Only display row actions for pending actions.
-	 *
-	 * @param array  $row         Row to render
-	 * @param string $column_name Current row
-	 *
-	 * @return string
-	 */
-	protected function maybe_render_actions( $row, $column_name ) {
-		if ( 'pending' === strtolower( $row['status'] ) ) {
-			return parent::maybe_render_actions( $row, $column_name );
-		}
-
-		return '';
-	}
-
-	/**
-	 * Renders admin notifications
-	 *
-	 * Notifications:
-	 *  1. When the maximum number of tasks are being executed simultaneously.
-	 *  2. Notifications when a task is manually executed.
-	 *  3. Tables are missing.
-	 */
-	public function display_admin_notices() {
-		global $wpdb;
-
-		if ( ( is_a( $this->store, 'ActionScheduler_HybridStore' ) || is_a( $this->store, 'ActionScheduler_DBStore' ) ) && apply_filters( 'action_scheduler_enable_recreate_data_store', true ) ) {
-			$table_list = array(
-				'actionscheduler_actions',
-				'actionscheduler_logs',
-				'actionscheduler_groups',
-				'actionscheduler_claims',
-			);
-
-			$found_tables = $wpdb->get_col( "SHOW TABLES LIKE '{$wpdb->prefix}actionscheduler%'" ); // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
-			foreach ( $table_list as $table_name ) {
-				if ( ! in_array( $wpdb->prefix . $table_name, $found_tables ) ) {
-					$this->admin_notices[] = array(
-						'class'   => 'error',
-						'message' => __( 'It appears one or more database tables were missing. Attempting to re-create the missing table(s).' , 'action-scheduler' ),
-					);
-					$this->recreate_tables();
-					parent::display_admin_notices();
-
-					return;
-				}
-			}
-		}
-
-		if ( $this->runner->has_maximum_concurrent_batches() ) {
-			$claim_count           = $this->store->get_claim_count();
-			$this->admin_notices[] = array(
-				'class'   => 'updated',
-				'message' => sprintf(
-					/* translators: %s: amount of claims */
-					_n(
-						'Maximum simultaneous queues already in progress (%s queue). No additional queues will begin processing until the current queues are complete.',
-						'Maximum simultaneous queues already in progress (%s queues). No additional queues will begin processing until the current queues are complete.',
-						$claim_count,
-						'action-scheduler'
-					),
-					$claim_count
-				),
-			);
-		} elseif ( $this->store->has_pending_actions_due() ) {
-
-			$async_request_lock_expiration = ActionScheduler::lock()->get_expiration( 'async-request-runner' );
-
-			// No lock set or lock expired
-			if ( false === $async_request_lock_expiration || $async_request_lock_expiration < time() ) {
-				$in_progress_url       = add_query_arg( 'status', 'in-progress', remove_query_arg( 'status' ) );
-				/* translators: %s: process URL */
-				$async_request_message = sprintf( __( 'A new queue has begun processing. <a href="%s">View actions in-progress &raquo;</a>', 'action-scheduler' ), esc_url( $in_progress_url ) );
-			} else {
-				/* translators: %d: seconds */
-				$async_request_message = sprintf( __( 'The next queue will begin processing in approximately %d seconds.', 'action-scheduler' ), $async_request_lock_expiration - time() );
-			}
-
-			$this->admin_notices[] = array(
-				'class'   => 'notice notice-info',
-				'message' => $async_request_message,
-			);
-		}
-
-		$notification = get_transient( 'action_scheduler_admin_notice' );
-
-		if ( is_array( $notification ) ) {
-			delete_transient( 'action_scheduler_admin_notice' );
-
-			$action = $this->store->fetch_action( $notification['action_id'] );
-			$action_hook_html = '<strong><code>' . $action->get_hook() . '</code></strong>';
-			if ( 1 == $notification['success'] ) {
-				$class = 'updated';
-				switch ( $notification['row_action_type'] ) {
-					case 'run' :
-						/* translators: %s: action HTML */
-						$action_message_html = sprintf( __( 'Successfully executed action: %s', 'action-scheduler' ), $action_hook_html );
-						break;
-					case 'cancel' :
-						/* translators: %s: action HTML */
-						$action_message_html = sprintf( __( 'Successfully canceled action: %s', 'action-scheduler' ), $action_hook_html );
-						break;
-					default :
-						/* translators: %s: action HTML */
-						$action_message_html = sprintf( __( 'Successfully processed change for action: %s', 'action-scheduler' ), $action_hook_html );
-						break;
-				}
-			} else {
-				$class = 'error';
-				/* translators: 1: action HTML 2: action ID 3: error message */
-				$action_message_html = sprintf( __( 'Could not process change for action: "%1$s" (ID: %2$d). Error: %3$s', 'action-scheduler' ), $action_hook_html, esc_html( $notification['action_id'] ), esc_html( $notification['error_message'] ) );
-			}
-
-			$action_message_html = apply_filters( 'action_scheduler_admin_notice_html', $action_message_html, $action, $notification );
-
-			$this->admin_notices[] = array(
-				'class'   => $class,
-				'message' => $action_message_html,
-			);
-		}
-
-		parent::display_admin_notices();
-	}
-
-	/**
-	 * Prints the scheduled date in a human friendly format.
-	 *
-	 * @param array $row The array representation of the current row of the table
-	 *
-	 * @return string
-	 */
-	public function column_schedule( $row ) {
-		return $this->get_schedule_display_string( $row['schedule'] );
-	}
-
-	/**
-	 * Get the scheduled date in a human friendly format.
-	 *
-	 * @param ActionScheduler_Schedule $schedule
-	 * @return string
-	 */
-	protected function get_schedule_display_string( ActionScheduler_Schedule $schedule ) {
-
-		$schedule_display_string = '';
-
-		if ( ! $schedule->get_date() ) {
-			return '0000-00-00 00:00:00';
-		}
-
-		$next_timestamp = $schedule->get_date()->getTimestamp();
-
-		$schedule_display_string .= $schedule->get_date()->format( 'Y-m-d H:i:s O' );
-		$schedule_display_string .= '<br/>';
-
-		if ( gmdate( 'U' ) > $next_timestamp ) {
-			/* translators: %s: date interval */
-			$schedule_display_string .= sprintf( __( ' (%s ago)', 'action-scheduler' ), self::human_interval( gmdate( 'U' ) - $next_timestamp ) );
-		} else {
-			/* translators: %s: date interval */
-			$schedule_display_string .= sprintf( __( ' (%s)', 'action-scheduler' ), self::human_interval( $next_timestamp - gmdate( 'U' ) ) );
-		}
-
-		return $schedule_display_string;
-	}
-
-	/**
-	 * Bulk delete
-	 *
-	 * Deletes actions based on their ID. This is the handler for the bulk delete. It assumes the data
-	 * properly validated by the callee and it will delete the actions without any extra validation.
-	 *
-	 * @param array $ids
-	 * @param string $ids_sql Inherited and unused
-	 */
-	protected function bulk_delete( array $ids, $ids_sql ) {
-		foreach ( $ids as $id ) {
-			$this->store->delete_action( $id );
-		}
-	}
-
-	/**
-	 * Implements the logic behind running an action. ActionScheduler_Abstract_ListTable validates the request and their
-	 * parameters are valid.
-	 *
-	 * @param int $action_id
-	 */
-	protected function row_action_cancel( $action_id ) {
-		$this->process_row_action( $action_id, 'cancel' );
-	}
-
-	/**
-	 * Implements the logic behind running an action. ActionScheduler_Abstract_ListTable validates the request and their
-	 * parameters are valid.
-	 *
-	 * @param int $action_id
-	 */
-	protected function row_action_run( $action_id ) {
-		$this->process_row_action( $action_id, 'run' );
-	}
-
-	/**
-	 * Force the data store schema updates.
-	 */
-	protected function recreate_tables() {
-		if ( is_a( $this->store, 'ActionScheduler_HybridStore' ) ) {
-			$store = $this->store;
-		} else {
-			$store = new ActionScheduler_HybridStore();
-		}
-		add_action( 'action_scheduler/created_table', array( $store, 'set_autoincrement' ), 10, 2 );
-
-		$store_schema  = new ActionScheduler_StoreSchema();
-		$logger_schema = new ActionScheduler_LoggerSchema();
-		$store_schema->register_tables( true );
-		$logger_schema->register_tables( true );
-
-		remove_action( 'action_scheduler/created_table', array( $store, 'set_autoincrement' ), 10 );
-	}
-	/**
-	 * Implements the logic behind processing an action once an action link is clicked on the list table.
-	 *
-	 * @param int $action_id
-	 * @param string $row_action_type The type of action to perform on the action.
-	 */
-	protected function process_row_action( $action_id, $row_action_type ) {
-		try {
-			switch ( $row_action_type ) {
-				case 'run' :
-					$this->runner->process_action( $action_id, 'Admin List Table' );
-					break;
-				case 'cancel' :
-					$this->store->cancel_action( $action_id );
-					break;
-			}
-			$success = 1;
-			$error_message = '';
-		} catch ( Exception $e ) {
-			$success = 0;
-			$error_message = $e->getMessage();
-		}
-
-		set_transient( 'action_scheduler_admin_notice', compact( 'action_id', 'success', 'error_message', 'row_action_type' ), 30 );
-	}
-
-	/**
-	 * {@inheritDoc}
-	 */
-	public function prepare_items() {
-		$this->prepare_column_headers();
-
-		$per_page = $this->get_items_per_page( $this->package . '_items_per_page', $this->items_per_page );
-		$query = array(
-			'per_page' => $per_page,
-			'offset'   => $this->get_items_offset(),
-			'status'   => $this->get_request_status(),
-			'orderby'  => $this->get_request_orderby(),
-			'order'    => $this->get_request_order(),
-			'search'   => $this->get_request_search_query(),
-		);
-
-		$this->items = array();
-
-		$total_items = $this->store->query_actions( $query, 'count' );
-
-		$status_labels = $this->store->get_status_labels();
-
-		foreach ( $this->store->query_actions( $query ) as $action_id ) {
-			try {
-				$action = $this->store->fetch_action( $action_id );
-			} catch ( Exception $e ) {
-				continue;
-			}
-			if ( is_a( $action, 'ActionScheduler_NullAction' ) ) {
-				continue;
-			}
-			$this->items[ $action_id ] = array(
-				'ID'          => $action_id,
-				'hook'        => $action->get_hook(),
-				'status'      => $status_labels[ $this->store->get_status( $action_id ) ],
-				'args'        => $action->get_args(),
-				'group'       => $action->get_group(),
-				'log_entries' => $this->logger->get_logs( $action_id ),
-				'claim_id'    => $this->store->get_claim_id( $action_id ),
-				'recurrence'  => $this->get_recurrence( $action ),
-				'schedule'    => $action->get_schedule(),
-			);
-		}
-
-		$this->set_pagination_args( array(
-			'total_items' => $total_items,
-			'per_page'    => $per_page,
-			'total_pages' => ceil( $total_items / $per_page ),
-		) );
-	}
-
-	/**
-	 * Prints the available statuses so the user can click to filter.
-	 */
-	protected function display_filter_by_status() {
-		$this->status_counts = $this->store->action_counts();
-		parent::display_filter_by_status();
-	}
-
-	/**
-	 * Get the text to display in the search box on the list table.
-	 */
-	protected function get_search_box_button_text() {
-		return __( 'Search hook, args and claim ID', 'action-scheduler' );
-	}
+    /**
+     * The package name.
+     *
+     * @var string
+     */
+    protected $package = 'action-scheduler';
+
+    /**
+     * Columns to show (name => label).
+     *
+     * @var array
+     */
+    protected $columns = array();
+
+    /**
+     * Actions (name => label).
+     *
+     * @var array
+     */
+    protected $row_actions = array();
+
+    /**
+     * The active data stores
+     *
+     * @var ActionScheduler_Store
+     */
+    protected $store;
+
+    /**
+     * A logger to use for getting action logs to display
+     *
+     * @var ActionScheduler_Logger
+     */
+    protected $logger;
+
+    /**
+     * A ActionScheduler_QueueRunner runner instance (or child class)
+     *
+     * @var ActionScheduler_QueueRunner
+     */
+    protected $runner;
+
+    /**
+     * Bulk actions. The key of the array is the method name of the implementation:
+     *
+     *     bulk_<key>(array $ids, string $sql_in).
+     *
+     * See the comments in the parent class for further details
+     *
+     * @var array
+     */
+    protected $bulk_actions = array();
+
+    /**
+     * Flag variable to render our notifications, if any, once.
+     *
+     * @var bool
+     */
+    protected static $did_notification = false;
+
+    /**
+     * Array of seconds for common time periods, like week or month, alongside an internationalised string representation, i.e. "Day" or "Days"
+     *
+     * @var array
+     */
+    private static $time_periods;
+
+    /**
+     * Sets the current data store object into `store->action` and initialises the object.
+     *
+     * @param ActionScheduler_Store $store
+     * @param ActionScheduler_Logger $logger
+     * @param ActionScheduler_QueueRunner $runner
+     */
+    public function __construct( ActionScheduler_Store $store, ActionScheduler_Logger $logger, ActionScheduler_QueueRunner $runner ) {
+
+        $this->store  = $store;
+        $this->logger = $logger;
+        $this->runner = $runner;
+
+        $this->table_header = __( 'Scheduled Actions', 'action-scheduler' );
+
+        $this->bulk_actions = array(
+            'delete' => __( 'Delete', 'action-scheduler' ),
+        );
+
+        $this->columns = array(
+            'hook'        => __( 'Hook', 'action-scheduler' ),
+            'status'      => __( 'Status', 'action-scheduler' ),
+            'args'        => __( 'Arguments', 'action-scheduler' ),
+            'group'       => __( 'Group', 'action-scheduler' ),
+            'recurrence'  => __( 'Recurrence', 'action-scheduler' ),
+            'schedule'    => __( 'Scheduled Date', 'action-scheduler' ),
+            'log_entries' => __( 'Log', 'action-scheduler' ),
+        );
+
+        $this->sort_by = array(
+            'schedule',
+            'hook',
+            'group',
+        );
+
+        $this->search_by = array(
+            'hook',
+            'args',
+            'claim_id',
+        );
+
+        $request_status = $this->get_request_status();
+
+        if ( empty( $request_status ) ) {
+            $this->sort_by[] = 'status';
+        } elseif ( in_array( $request_status, array( 'in-progress', 'failed' ) ) ) {
+            $this->columns  += array( 'claim_id' => __( 'Claim ID', 'action-scheduler' ) );
+            $this->sort_by[] = 'claim_id';
+        }
+
+        $this->row_actions = array(
+            'hook' => array(
+                'run' => array(
+                    'name'  => __( 'Run', 'action-scheduler' ),
+                    'desc'  => __( 'Process the action now as if it were run as part of a queue', 'action-scheduler' ),
+                ),
+                'cancel' => array(
+                    'name'  => __( 'Cancel', 'action-scheduler' ),
+                    'desc'  => __( 'Cancel the action now to avoid it being run in future', 'action-scheduler' ),
+                    'class' => 'cancel trash',
+                ),
+            ),
+        );
+
+        self::$time_periods = array(
+            array(
+                'seconds' => YEAR_IN_SECONDS,
+                /* translators: %s: amount of time */
+                'names'   => _n_noop( '%s year', '%s years', 'action-scheduler' ),
+            ),
+            array(
+                'seconds' => MONTH_IN_SECONDS,
+                /* translators: %s: amount of time */
+                'names'   => _n_noop( '%s month', '%s months', 'action-scheduler' ),
+            ),
+            array(
+                'seconds' => WEEK_IN_SECONDS,
+                /* translators: %s: amount of time */
+                'names'   => _n_noop( '%s week', '%s weeks', 'action-scheduler' ),
+            ),
+            array(
+                'seconds' => DAY_IN_SECONDS,
+                /* translators: %s: amount of time */
+                'names'   => _n_noop( '%s day', '%s days', 'action-scheduler' ),
+            ),
+            array(
+                'seconds' => HOUR_IN_SECONDS,
+                /* translators: %s: amount of time */
+                'names'   => _n_noop( '%s hour', '%s hours', 'action-scheduler' ),
+            ),
+            array(
+                'seconds' => MINUTE_IN_SECONDS,
+                /* translators: %s: amount of time */
+                'names'   => _n_noop( '%s minute', '%s minutes', 'action-scheduler' ),
+            ),
+            array(
+                'seconds' => 1,
+                /* translators: %s: amount of time */
+                'names'   => _n_noop( '%s second', '%s seconds', 'action-scheduler' ),
+            ),
+        );
+
+        parent::__construct( array(
+            'singular' => 'action-scheduler',
+            'plural'   => 'action-scheduler',
+            'ajax'     => false,
+        ) );
+    }
+
+    /**
+     * Convert an interval of seconds into a two part human friendly string.
+     *
+     * The WordPress human_time_diff() function only calculates the time difference to one degree, meaning
+     * even if an action is 1 day and 11 hours away, it will display "1 day". This function goes one step
+     * further to display two degrees of accuracy.
+     *
+     * Inspired by the Crontrol::interval() function by Edward Dale: https://wordpress.org/plugins/wp-crontrol/
+     *
+     * @param int $interval A interval in seconds.
+     * @param int $periods_to_include Depth of time periods to include, e.g. for an interval of 70, and $periods_to_include of 2, both minutes and seconds would be included. With a value of 1, only minutes would be included.
+     * @return string A human friendly string representation of the interval.
+     */
+    private static function human_interval( $interval, $periods_to_include = 2 ) {
+
+        if ( $interval <= 0 ) {
+            return __( 'Now!', 'action-scheduler' );
+        }
+
+        $output = '';
+
+        for ( $time_period_index = 0, $periods_included = 0, $seconds_remaining = $interval; $time_period_index < count( self::$time_periods ) && $seconds_remaining > 0 && $periods_included < $periods_to_include; $time_period_index++ ) {
+
+            $periods_in_interval = floor( $seconds_remaining / self::$time_periods[ $time_period_index ]['seconds'] );
+
+            if ( $periods_in_interval > 0 ) {
+                if ( ! empty( $output ) ) {
+                    $output .= ' ';
+                }
+                $output .= sprintf( _n( self::$time_periods[ $time_period_index ]['names'][0], self::$time_periods[ $time_period_index ]['names'][1], $periods_in_interval, 'action-scheduler' ), $periods_in_interval );
+                $seconds_remaining -= $periods_in_interval * self::$time_periods[ $time_period_index ]['seconds'];
+                $periods_included++;
+            }
+        }
+
+        return $output;
+    }
+
+    /**
+     * Returns the recurrence of an action or 'Non-repeating'. The output is human readable.
+     *
+     * @param ActionScheduler_Action $action
+     *
+     * @return string
+     */
+    protected function get_recurrence( $action ) {
+        $schedule = $action->get_schedule();
+        if ( $schedule->is_recurring() ) {
+            $recurrence = $schedule->get_recurrence();
+
+            if ( is_numeric( $recurrence ) ) {
+                /* translators: %s: time interval */
+                return sprintf( __( 'Every %s', 'action-scheduler' ), self::human_interval( $recurrence ) );
+            } else {
+                return $recurrence;
+            }
+        }
+
+        return __( 'Non-repeating', 'action-scheduler' );
+    }
+
+    /**
+     * Serializes the argument of an action to render it in a human friendly format.
+     *
+     * @param array $row The array representation of the current row of the table
+     *
+     * @return string
+     */
+    public function column_args( array $row ) {
+        if ( empty( $row['args'] ) ) {
+            return apply_filters( 'action_scheduler_list_table_column_args', '', $row );
+        }
+
+        $row_html = '<ul>';
+        foreach ( $row['args'] as $key => $value ) {
+            $row_html .= sprintf( '<li><code>%s => %s</code></li>', esc_html( var_export( $key, true ) ), esc_html( var_export( $value, true ) ) );
+        }
+        $row_html .= '</ul>';
+
+        return apply_filters( 'action_scheduler_list_table_column_args', $row_html, $row );
+    }
+
+    /**
+     * Prints the logs entries inline. We do so to avoid loading Javascript and other hacks to show it in a modal.
+     *
+     * @param array $row Action array.
+     * @return string
+     */
+    public function column_log_entries( array $row ) {
+
+        $log_entries_html = '<ol>';
+
+        $timezone = new DateTimezone( 'UTC' );
+
+        foreach ( $row['log_entries'] as $log_entry ) {
+            $log_entries_html .= $this->get_log_entry_html( $log_entry, $timezone );
+        }
+
+        $log_entries_html .= '</ol>';
+
+        return $log_entries_html;
+    }
+
+    /**
+     * Prints the logs entries inline. We do so to avoid loading Javascript and other hacks to show it in a modal.
+     *
+     * @param ActionScheduler_LogEntry $log_entry
+     * @param DateTimezone $timezone
+     * @return string
+     */
+    protected function get_log_entry_html( ActionScheduler_LogEntry $log_entry, DateTimezone $timezone ) {
+        $date = $log_entry->get_date();
+        $date->setTimezone( $timezone );
+        return sprintf( '<li><strong>%s</strong><br/>%s</li>', esc_html( $date->format( 'Y-m-d H:i:s O' ) ), esc_html( $log_entry->get_message() ) );
+    }
+
+    /**
+     * Only display row actions for pending actions.
+     *
+     * @param array  $row         Row to render
+     * @param string $column_name Current row
+     *
+     * @return string
+     */
+    protected function maybe_render_actions( $row, $column_name ) {
+        if ( 'pending' === strtolower( $row['status'] ) ) {
+            return parent::maybe_render_actions( $row, $column_name );
+        }
+
+        return '';
+    }
+
+    /**
+     * Renders admin notifications
+     *
+     * Notifications:
+     *  1. When the maximum number of tasks are being executed simultaneously.
+     *  2. Notifications when a task is manually executed.
+     *  3. Tables are missing.
+     */
+    public function display_admin_notices() {
+        global $wpdb;
+
+        if ( ( is_a( $this->store, 'ActionScheduler_HybridStore' ) || is_a( $this->store, 'ActionScheduler_DBStore' ) ) && apply_filters( 'action_scheduler_enable_recreate_data_store', true ) ) {
+            $table_list = array(
+                'actionscheduler_actions',
+                'actionscheduler_logs',
+                'actionscheduler_groups',
+                'actionscheduler_claims',
+            );
+
+            $found_tables = $wpdb->get_col( "SHOW TABLES LIKE '{$wpdb->prefix}actionscheduler%'" ); // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
+            foreach ( $table_list as $table_name ) {
+                if ( ! in_array( $wpdb->prefix . $table_name, $found_tables ) ) {
+                    $this->admin_notices[] = array(
+                        'class'   => 'error',
+                        'message' => __( 'It appears one or more database tables were missing. Attempting to re-create the missing table(s).' , 'action-scheduler' ),
+                    );
+                    $this->recreate_tables();
+                    parent::display_admin_notices();
+
+                    return;
+                }
+            }
+        }
+
+        if ( $this->runner->has_maximum_concurrent_batches() ) {
+            $claim_count           = $this->store->get_claim_count();
+            $this->admin_notices[] = array(
+                'class'   => 'updated',
+                'message' => sprintf(
+                    /* translators: %s: amount of claims */
+                    _n(
+                        'Maximum simultaneous queues already in progress (%s queue). No additional queues will begin processing until the current queues are complete.',
+                        'Maximum simultaneous queues already in progress (%s queues). No additional queues will begin processing until the current queues are complete.',
+                        $claim_count,
+                        'action-scheduler'
+                    ),
+                    $claim_count
+                ),
+            );
+        } elseif ( $this->store->has_pending_actions_due() ) {
+
+            $async_request_lock_expiration = ActionScheduler::lock()->get_expiration( 'async-request-runner' );
+
+            // No lock set or lock expired
+            if ( false === $async_request_lock_expiration || $async_request_lock_expiration < time() ) {
+                $in_progress_url       = add_query_arg( 'status', 'in-progress', remove_query_arg( 'status' ) );
+                /* translators: %s: process URL */
+                $async_request_message = sprintf( __( 'A new queue has begun processing. <a href="%s">View actions in-progress &raquo;</a>', 'action-scheduler' ), esc_url( $in_progress_url ) );
+            } else {
+                /* translators: %d: seconds */
+                $async_request_message = sprintf( __( 'The next queue will begin processing in approximately %d seconds.', 'action-scheduler' ), $async_request_lock_expiration - time() );
+            }
+
+            $this->admin_notices[] = array(
+                'class'   => 'notice notice-info',
+                'message' => $async_request_message,
+            );
+        }
+
+        $notification = get_transient( 'action_scheduler_admin_notice' );
+
+        if ( is_array( $notification ) ) {
+            delete_transient( 'action_scheduler_admin_notice' );
+
+            $action = $this->store->fetch_action( $notification['action_id'] );
+            $action_hook_html = '<strong><code>' . $action->get_hook() . '</code></strong>';
+            if ( 1 == $notification['success'] ) {
+                $class = 'updated';
+                switch ( $notification['row_action_type'] ) {
+                    case 'run' :
+                        /* translators: %s: action HTML */
+                        $action_message_html = sprintf( __( 'Successfully executed action: %s', 'action-scheduler' ), $action_hook_html );
+                        break;
+                    case 'cancel' :
+                        /* translators: %s: action HTML */
+                        $action_message_html = sprintf( __( 'Successfully canceled action: %s', 'action-scheduler' ), $action_hook_html );
+                        break;
+                    default :
+                        /* translators: %s: action HTML */
+                        $action_message_html = sprintf( __( 'Successfully processed change for action: %s', 'action-scheduler' ), $action_hook_html );
+                        break;
+                }
+            } else {
+                $class = 'error';
+                /* translators: 1: action HTML 2: action ID 3: error message */
+                $action_message_html = sprintf( __( 'Could not process change for action: "%1$s" (ID: %2$d). Error: %3$s', 'action-scheduler' ), $action_hook_html, esc_html( $notification['action_id'] ), esc_html( $notification['error_message'] ) );
+            }
+
+            $action_message_html = apply_filters( 'action_scheduler_admin_notice_html', $action_message_html, $action, $notification );
+
+            $this->admin_notices[] = array(
+                'class'   => $class,
+                'message' => $action_message_html,
+            );
+        }
+
+        parent::display_admin_notices();
+    }
+
+    /**
+     * Prints the scheduled date in a human friendly format.
+     *
+     * @param array $row The array representation of the current row of the table
+     *
+     * @return string
+     */
+    public function column_schedule( $row ) {
+        return $this->get_schedule_display_string( $row['schedule'] );
+    }
+
+    /**
+     * Get the scheduled date in a human friendly format.
+     *
+     * @param ActionScheduler_Schedule $schedule
+     * @return string
+     */
+    protected function get_schedule_display_string( ActionScheduler_Schedule $schedule ) {
+
+        $schedule_display_string = '';
+
+        if ( ! $schedule->get_date() ) {
+            return '0000-00-00 00:00:00';
+        }
+
+        $next_timestamp = $schedule->get_date()->getTimestamp();
+
+        $schedule_display_string .= $schedule->get_date()->format( 'Y-m-d H:i:s O' );
+        $schedule_display_string .= '<br/>';
+
+        if ( gmdate( 'U' ) > $next_timestamp ) {
+            /* translators: %s: date interval */
+            $schedule_display_string .= sprintf( __( ' (%s ago)', 'action-scheduler' ), self::human_interval( gmdate( 'U' ) - $next_timestamp ) );
+        } else {
+            /* translators: %s: date interval */
+            $schedule_display_string .= sprintf( __( ' (%s)', 'action-scheduler' ), self::human_interval( $next_timestamp - gmdate( 'U' ) ) );
+        }
+
+        return $schedule_display_string;
+    }
+
+    /**
+     * Bulk delete
+     *
+     * Deletes actions based on their ID. This is the handler for the bulk delete. It assumes the data
+     * properly validated by the callee and it will delete the actions without any extra validation.
+     *
+     * @param array $ids
+     * @param string $ids_sql Inherited and unused
+     */
+    protected function bulk_delete( array $ids, $ids_sql ) {
+        foreach ( $ids as $id ) {
+            $this->store->delete_action( $id );
+        }
+    }
+
+    /**
+     * Implements the logic behind running an action. ActionScheduler_Abstract_ListTable validates the request and their
+     * parameters are valid.
+     *
+     * @param int $action_id
+     */
+    protected function row_action_cancel( $action_id ) {
+        $this->process_row_action( $action_id, 'cancel' );
+    }
+
+    /**
+     * Implements the logic behind running an action. ActionScheduler_Abstract_ListTable validates the request and their
+     * parameters are valid.
+     *
+     * @param int $action_id
+     */
+    protected function row_action_run( $action_id ) {
+        $this->process_row_action( $action_id, 'run' );
+    }
+
+    /**
+     * Force the data store schema updates.
+     */
+    protected function recreate_tables() {
+        if ( is_a( $this->store, 'ActionScheduler_HybridStore' ) ) {
+            $store = $this->store;
+        } else {
+            $store = new ActionScheduler_HybridStore();
+        }
+        add_action( 'action_scheduler/created_table', array( $store, 'set_autoincrement' ), 10, 2 );
+
+        $store_schema  = new ActionScheduler_StoreSchema();
+        $logger_schema = new ActionScheduler_LoggerSchema();
+        $store_schema->register_tables( true );
+        $logger_schema->register_tables( true );
+
+        remove_action( 'action_scheduler/created_table', array( $store, 'set_autoincrement' ), 10 );
+    }
+    /**
+     * Implements the logic behind processing an action once an action link is clicked on the list table.
+     *
+     * @param int $action_id
+     * @param string $row_action_type The type of action to perform on the action.
+     */
+    protected function process_row_action( $action_id, $row_action_type ) {
+        try {
+            switch ( $row_action_type ) {
+                case 'run' :
+                    $this->runner->process_action( $action_id, 'Admin List Table' );
+                    break;
+                case 'cancel' :
+                    $this->store->cancel_action( $action_id );
+                    break;
+            }
+            $success = 1;
+            $error_message = '';
+        } catch ( Exception $e ) {
+            $success = 0;
+            $error_message = $e->getMessage();
+        }
+
+        set_transient( 'action_scheduler_admin_notice', compact( 'action_id', 'success', 'error_message', 'row_action_type' ), 30 );
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function prepare_items() {
+        $this->prepare_column_headers();
+
+        $per_page = $this->get_items_per_page( $this->package . '_items_per_page', $this->items_per_page );
+        $query = array(
+            'per_page' => $per_page,
+            'offset'   => $this->get_items_offset(),
+            'status'   => $this->get_request_status(),
+            'orderby'  => $this->get_request_orderby(),
+            'order'    => $this->get_request_order(),
+            'search'   => $this->get_request_search_query(),
+        );
+
+        $this->items = array();
+
+        $total_items = $this->store->query_actions( $query, 'count' );
+
+        $status_labels = $this->store->get_status_labels();
+
+        foreach ( $this->store->query_actions( $query ) as $action_id ) {
+            try {
+                $action = $this->store->fetch_action( $action_id );
+            } catch ( Exception $e ) {
+                continue;
+            }
+            if ( is_a( $action, 'ActionScheduler_NullAction' ) ) {
+                continue;
+            }
+            $this->items[ $action_id ] = array(
+                'ID'          => $action_id,
+                'hook'        => $action->get_hook(),
+                'status'      => $status_labels[ $this->store->get_status( $action_id ) ],
+                'args'        => $action->get_args(),
+                'group'       => $action->get_group(),
+                'log_entries' => $this->logger->get_logs( $action_id ),
+                'claim_id'    => $this->store->get_claim_id( $action_id ),
+                'recurrence'  => $this->get_recurrence( $action ),
+                'schedule'    => $action->get_schedule(),
+            );
+        }
+
+        $this->set_pagination_args( array(
+            'total_items' => $total_items,
+            'per_page'    => $per_page,
+            'total_pages' => ceil( $total_items / $per_page ),
+        ) );
+    }
+
+    /**
+     * Prints the available statuses so the user can click to filter.
+     */
+    protected function display_filter_by_status() {
+        $this->status_counts = $this->store->action_counts();
+        parent::display_filter_by_status();
+    }
+
+    /**
+     * Get the text to display in the search box on the list table.
+     */
+    protected function get_search_box_button_text() {
+        return __( 'Search hook, args and claim ID', 'action-scheduler' );
+    }
 }

Spacing +120 added lines, -120 removed lines

@@ -80,26 +80,26 @@ 
 	 * @param ActionScheduler_Logger $logger
 	 * @param ActionScheduler_QueueRunner $runner
 	 */
-	public function __construct( ActionScheduler_Store $store, ActionScheduler_Logger $logger, ActionScheduler_QueueRunner $runner ) {
+	public function __construct(ActionScheduler_Store $store, ActionScheduler_Logger $logger, ActionScheduler_QueueRunner $runner) {
 
 		$this->store  = $store;
 		$this->logger = $logger;
 		$this->runner = $runner;
 
-		$this->table_header = __( 'Scheduled Actions', 'action-scheduler' );
+		$this->table_header = __('Scheduled Actions', 'action-scheduler');
 
 		$this->bulk_actions = array(
-			'delete' => __( 'Delete', 'action-scheduler' ),
+			'delete' => __('Delete', 'action-scheduler'),
 		);
 
 		$this->columns = array(
-			'hook'        => __( 'Hook', 'action-scheduler' ),
-			'status'      => __( 'Status', 'action-scheduler' ),
-			'args'        => __( 'Arguments', 'action-scheduler' ),
-			'group'       => __( 'Group', 'action-scheduler' ),
-			'recurrence'  => __( 'Recurrence', 'action-scheduler' ),
-			'schedule'    => __( 'Scheduled Date', 'action-scheduler' ),
-			'log_entries' => __( 'Log', 'action-scheduler' ),
+			'hook'        => __('Hook', 'action-scheduler'),
+			'status'      => __('Status', 'action-scheduler'),
+			'args'        => __('Arguments', 'action-scheduler'),
+			'group'       => __('Group', 'action-scheduler'),
+			'recurrence'  => __('Recurrence', 'action-scheduler'),
+			'schedule'    => __('Scheduled Date', 'action-scheduler'),
+			'log_entries' => __('Log', 'action-scheduler'),
 		);
 
 		$this->sort_by = array(
@@ -116,22 +116,22 @@ 
 
 		$request_status = $this->get_request_status();
 
-		if ( empty( $request_status ) ) {
+		if (empty($request_status)) {
 			$this->sort_by[] = 'status';
-		} elseif ( in_array( $request_status, array( 'in-progress', 'failed' ) ) ) {
-			$this->columns  += array( 'claim_id' => __( 'Claim ID', 'action-scheduler' ) );
+		} elseif (in_array($request_status, array('in-progress', 'failed'))) {
+			$this->columns  += array('claim_id' => __('Claim ID', 'action-scheduler'));
 			$this->sort_by[] = 'claim_id';
 		}
 
 		$this->row_actions = array(
 			'hook' => array(
 				'run' => array(
-					'name'  => __( 'Run', 'action-scheduler' ),
-					'desc'  => __( 'Process the action now as if it were run as part of a queue', 'action-scheduler' ),
+					'name'  => __('Run', 'action-scheduler'),
+					'desc'  => __('Process the action now as if it were run as part of a queue', 'action-scheduler'),
 				),
 				'cancel' => array(
-					'name'  => __( 'Cancel', 'action-scheduler' ),
-					'desc'  => __( 'Cancel the action now to avoid it being run in future', 'action-scheduler' ),
+					'name'  => __('Cancel', 'action-scheduler'),
+					'desc'  => __('Cancel the action now to avoid it being run in future', 'action-scheduler'),
 					'class' => 'cancel trash',
 				),
 			),
@@ -141,45 +141,45 @@ 
 			array(
 				'seconds' => YEAR_IN_SECONDS,
 				/* translators: %s: amount of time */
-				'names'   => _n_noop( '%s year', '%s years', 'action-scheduler' ),
+				'names'   => _n_noop('%s year', '%s years', 'action-scheduler'),
 			),
 			array(
 				'seconds' => MONTH_IN_SECONDS,
 				/* translators: %s: amount of time */
-				'names'   => _n_noop( '%s month', '%s months', 'action-scheduler' ),
+				'names'   => _n_noop('%s month', '%s months', 'action-scheduler'),
 			),
 			array(
 				'seconds' => WEEK_IN_SECONDS,
 				/* translators: %s: amount of time */
-				'names'   => _n_noop( '%s week', '%s weeks', 'action-scheduler' ),
+				'names'   => _n_noop('%s week', '%s weeks', 'action-scheduler'),
 			),
 			array(
 				'seconds' => DAY_IN_SECONDS,
 				/* translators: %s: amount of time */
-				'names'   => _n_noop( '%s day', '%s days', 'action-scheduler' ),
+				'names'   => _n_noop('%s day', '%s days', 'action-scheduler'),
 			),
 			array(
 				'seconds' => HOUR_IN_SECONDS,
 				/* translators: %s: amount of time */
-				'names'   => _n_noop( '%s hour', '%s hours', 'action-scheduler' ),
+				'names'   => _n_noop('%s hour', '%s hours', 'action-scheduler'),
 			),
 			array(
 				'seconds' => MINUTE_IN_SECONDS,
 				/* translators: %s: amount of time */
-				'names'   => _n_noop( '%s minute', '%s minutes', 'action-scheduler' ),
+				'names'   => _n_noop('%s minute', '%s minutes', 'action-scheduler'),
 			),
 			array(
 				'seconds' => 1,
 				/* translators: %s: amount of time */
-				'names'   => _n_noop( '%s second', '%s seconds', 'action-scheduler' ),
+				'names'   => _n_noop('%s second', '%s seconds', 'action-scheduler'),
 			),
 		);
 
-		parent::__construct( array(
+		parent::__construct(array(
 			'singular' => 'action-scheduler',
 			'plural'   => 'action-scheduler',
 			'ajax'     => false,
-		) );
+		));
 	}
 
 	/**
@@ -195,24 +195,24 @@ 
 	 * @param int $periods_to_include Depth of time periods to include, e.g. for an interval of 70, and $periods_to_include of 2, both minutes and seconds would be included. With a value of 1, only minutes would be included.
 	 * @return string A human friendly string representation of the interval.
 	 */
-	private static function human_interval( $interval, $periods_to_include = 2 ) {
+	private static function human_interval($interval, $periods_to_include = 2) {
 
-		if ( $interval <= 0 ) {
-			return __( 'Now!', 'action-scheduler' );
+		if ($interval <= 0) {
+			return __('Now!', 'action-scheduler');
 		}
 
 		$output = '';
 
-		for ( $time_period_index = 0, $periods_included = 0, $seconds_remaining = $interval; $time_period_index < count( self::$time_periods ) && $seconds_remaining > 0 && $periods_included < $periods_to_include; $time_period_index++ ) {
+		for ($time_period_index = 0, $periods_included = 0, $seconds_remaining = $interval; $time_period_index < count(self::$time_periods) && $seconds_remaining > 0 && $periods_included < $periods_to_include; $time_period_index++) {
 
-			$periods_in_interval = floor( $seconds_remaining / self::$time_periods[ $time_period_index ]['seconds'] );
+			$periods_in_interval = floor($seconds_remaining / self::$time_periods[$time_period_index]['seconds']);
 
-			if ( $periods_in_interval > 0 ) {
-				if ( ! empty( $output ) ) {
+			if ($periods_in_interval > 0) {
+				if (!empty($output)) {
 					$output .= ' ';
 				}
-				$output .= sprintf( _n( self::$time_periods[ $time_period_index ]['names'][0], self::$time_periods[ $time_period_index ]['names'][1], $periods_in_interval, 'action-scheduler' ), $periods_in_interval );
-				$seconds_remaining -= $periods_in_interval * self::$time_periods[ $time_period_index ]['seconds'];
+				$output .= sprintf(_n(self::$time_periods[$time_period_index]['names'][0], self::$time_periods[$time_period_index]['names'][1], $periods_in_interval, 'action-scheduler'), $periods_in_interval);
+				$seconds_remaining -= $periods_in_interval * self::$time_periods[$time_period_index]['seconds'];
 				$periods_included++;
 			}
 		}
@@ -227,20 +227,20 @@ 
 	 *
 	 * @return string
 	 */
-	protected function get_recurrence( $action ) {
+	protected function get_recurrence($action) {
 		$schedule = $action->get_schedule();
-		if ( $schedule->is_recurring() ) {
+		if ($schedule->is_recurring()) {
 			$recurrence = $schedule->get_recurrence();
 
-			if ( is_numeric( $recurrence ) ) {
+			if (is_numeric($recurrence)) {
 				/* translators: %s: time interval */
-				return sprintf( __( 'Every %s', 'action-scheduler' ), self::human_interval( $recurrence ) );
+				return sprintf(__('Every %s', 'action-scheduler'), self::human_interval($recurrence));
 			} else {
 				return $recurrence;
 			}
 		}
 
-		return __( 'Non-repeating', 'action-scheduler' );
+		return __('Non-repeating', 'action-scheduler');
 	}
 
 	/**
@@ -250,18 +250,18 @@ 
 	 *
 	 * @return string
 	 */
-	public function column_args( array $row ) {
-		if ( empty( $row['args'] ) ) {
-			return apply_filters( 'action_scheduler_list_table_column_args', '', $row );
+	public function column_args(array $row) {
+		if (empty($row['args'])) {
+			return apply_filters('action_scheduler_list_table_column_args', '', $row);
 		}
 
 		$row_html = '<ul>';
-		foreach ( $row['args'] as $key => $value ) {
-			$row_html .= sprintf( '<li><code>%s => %s</code></li>', esc_html( var_export( $key, true ) ), esc_html( var_export( $value, true ) ) );
+		foreach ($row['args'] as $key => $value) {
+			$row_html .= sprintf('<li><code>%s => %s</code></li>', esc_html(var_export($key, true)), esc_html(var_export($value, true)));
 		}
 		$row_html .= '</ul>';
 
-		return apply_filters( 'action_scheduler_list_table_column_args', $row_html, $row );
+		return apply_filters('action_scheduler_list_table_column_args', $row_html, $row);
 	}
 
 	/**
@@ -270,14 +270,14 @@ 
 	 * @param array $row Action array.
 	 * @return string
 	 */
-	public function column_log_entries( array $row ) {
+	public function column_log_entries(array $row) {
 
 		$log_entries_html = '<ol>';
 
-		$timezone = new DateTimezone( 'UTC' );
+		$timezone = new DateTimezone('UTC');
 
-		foreach ( $row['log_entries'] as $log_entry ) {
-			$log_entries_html .= $this->get_log_entry_html( $log_entry, $timezone );
+		foreach ($row['log_entries'] as $log_entry) {
+			$log_entries_html .= $this->get_log_entry_html($log_entry, $timezone);
 		}
 
 		$log_entries_html .= '</ol>';
@@ -292,10 +292,10 @@ 
 	 * @param DateTimezone $timezone
 	 * @return string
 	 */
-	protected function get_log_entry_html( ActionScheduler_LogEntry $log_entry, DateTimezone $timezone ) {
+	protected function get_log_entry_html(ActionScheduler_LogEntry $log_entry, DateTimezone $timezone) {
 		$date = $log_entry->get_date();
-		$date->setTimezone( $timezone );
-		return sprintf( '<li><strong>%s</strong><br/>%s</li>', esc_html( $date->format( 'Y-m-d H:i:s O' ) ), esc_html( $log_entry->get_message() ) );
+		$date->setTimezone($timezone);
+		return sprintf('<li><strong>%s</strong><br/>%s</li>', esc_html($date->format('Y-m-d H:i:s O')), esc_html($log_entry->get_message()));
 	}
 
 	/**
@@ -306,9 +306,9 @@ 
 	 *
 	 * @return string
 	 */
-	protected function maybe_render_actions( $row, $column_name ) {
-		if ( 'pending' === strtolower( $row['status'] ) ) {
-			return parent::maybe_render_actions( $row, $column_name );
+	protected function maybe_render_actions($row, $column_name) {
+		if ('pending' === strtolower($row['status'])) {
+			return parent::maybe_render_actions($row, $column_name);
 		}
 
 		return '';
@@ -325,7 +325,7 @@ 
 	public function display_admin_notices() {
 		global $wpdb;
 
-		if ( ( is_a( $this->store, 'ActionScheduler_HybridStore' ) || is_a( $this->store, 'ActionScheduler_DBStore' ) ) && apply_filters( 'action_scheduler_enable_recreate_data_store', true ) ) {
+		if ((is_a($this->store, 'ActionScheduler_HybridStore') || is_a($this->store, 'ActionScheduler_DBStore')) && apply_filters('action_scheduler_enable_recreate_data_store', true)) {
 			$table_list = array(
 				'actionscheduler_actions',
 				'actionscheduler_logs',
@@ -333,12 +333,12 @@ 
 				'actionscheduler_claims',
 			);
 
-			$found_tables = $wpdb->get_col( "SHOW TABLES LIKE '{$wpdb->prefix}actionscheduler%'" ); // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
-			foreach ( $table_list as $table_name ) {
-				if ( ! in_array( $wpdb->prefix . $table_name, $found_tables ) ) {
+			$found_tables = $wpdb->get_col("SHOW TABLES LIKE '{$wpdb->prefix}actionscheduler%'"); // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
+			foreach ($table_list as $table_name) {
+				if (!in_array($wpdb->prefix . $table_name, $found_tables)) {
 					$this->admin_notices[] = array(
 						'class'   => 'error',
-						'message' => __( 'It appears one or more database tables were missing. Attempting to re-create the missing table(s).' , 'action-scheduler' ),
+						'message' => __('It appears one or more database tables were missing. Attempting to re-create the missing table(s).', 'action-scheduler'),
 					);
 					$this->recreate_tables();
 					parent::display_admin_notices();
@@ -348,7 +348,7 @@ 
 			}
 		}
 
-		if ( $this->runner->has_maximum_concurrent_batches() ) {
+		if ($this->runner->has_maximum_concurrent_batches()) {
 			$claim_count           = $this->store->get_claim_count();
 			$this->admin_notices[] = array(
 				'class'   => 'updated',
@@ -363,18 +363,18 @@ 
 					$claim_count
 				),
 			);
-		} elseif ( $this->store->has_pending_actions_due() ) {
+		} elseif ($this->store->has_pending_actions_due()) {
 
-			$async_request_lock_expiration = ActionScheduler::lock()->get_expiration( 'async-request-runner' );
+			$async_request_lock_expiration = ActionScheduler::lock()->get_expiration('async-request-runner');
 
 			// No lock set or lock expired
-			if ( false === $async_request_lock_expiration || $async_request_lock_expiration < time() ) {
-				$in_progress_url       = add_query_arg( 'status', 'in-progress', remove_query_arg( 'status' ) );
+			if (false === $async_request_lock_expiration || $async_request_lock_expiration < time()) {
+				$in_progress_url       = add_query_arg('status', 'in-progress', remove_query_arg('status'));
 				/* translators: %s: process URL */
-				$async_request_message = sprintf( __( 'A new queue has begun processing. <a href="%s">View actions in-progress &raquo;</a>', 'action-scheduler' ), esc_url( $in_progress_url ) );
+				$async_request_message = sprintf(__('A new queue has begun processing. <a href="%s">View actions in-progress &raquo;</a>', 'action-scheduler'), esc_url($in_progress_url));
 			} else {
 				/* translators: %d: seconds */
-				$async_request_message = sprintf( __( 'The next queue will begin processing in approximately %d seconds.', 'action-scheduler' ), $async_request_lock_expiration - time() );
+				$async_request_message = sprintf(__('The next queue will begin processing in approximately %d seconds.', 'action-scheduler'), $async_request_lock_expiration - time());
 			}
 
 			$this->admin_notices[] = array(
@@ -383,36 +383,36 @@ 
 			);
 		}
 
-		$notification = get_transient( 'action_scheduler_admin_notice' );
+		$notification = get_transient('action_scheduler_admin_notice');
 
-		if ( is_array( $notification ) ) {
-			delete_transient( 'action_scheduler_admin_notice' );
+		if (is_array($notification)) {
+			delete_transient('action_scheduler_admin_notice');
 
-			$action = $this->store->fetch_action( $notification['action_id'] );
+			$action = $this->store->fetch_action($notification['action_id']);
 			$action_hook_html = '<strong><code>' . $action->get_hook() . '</code></strong>';
-			if ( 1 == $notification['success'] ) {
+			if (1 == $notification['success']) {
 				$class = 'updated';
-				switch ( $notification['row_action_type'] ) {
+				switch ($notification['row_action_type']) {
 					case 'run' :
 						/* translators: %s: action HTML */
-						$action_message_html = sprintf( __( 'Successfully executed action: %s', 'action-scheduler' ), $action_hook_html );
+						$action_message_html = sprintf(__('Successfully executed action: %s', 'action-scheduler'), $action_hook_html);
 						break;
 					case 'cancel' :
 						/* translators: %s: action HTML */
-						$action_message_html = sprintf( __( 'Successfully canceled action: %s', 'action-scheduler' ), $action_hook_html );
+						$action_message_html = sprintf(__('Successfully canceled action: %s', 'action-scheduler'), $action_hook_html);
 						break;
 					default :
 						/* translators: %s: action HTML */
-						$action_message_html = sprintf( __( 'Successfully processed change for action: %s', 'action-scheduler' ), $action_hook_html );
+						$action_message_html = sprintf(__('Successfully processed change for action: %s', 'action-scheduler'), $action_hook_html);
 						break;
 				}
 			} else {
 				$class = 'error';
 				/* translators: 1: action HTML 2: action ID 3: error message */
-				$action_message_html = sprintf( __( 'Could not process change for action: "%1$s" (ID: %2$d). Error: %3$s', 'action-scheduler' ), $action_hook_html, esc_html( $notification['action_id'] ), esc_html( $notification['error_message'] ) );
+				$action_message_html = sprintf(__('Could not process change for action: "%1$s" (ID: %2$d). Error: %3$s', 'action-scheduler'), $action_hook_html, esc_html($notification['action_id']), esc_html($notification['error_message']));
 			}
 
-			$action_message_html = apply_filters( 'action_scheduler_admin_notice_html', $action_message_html, $action, $notification );
+			$action_message_html = apply_filters('action_scheduler_admin_notice_html', $action_message_html, $action, $notification);
 
 			$this->admin_notices[] = array(
 				'class'   => $class,
@@ -430,8 +430,8 @@ 
 	 *
 	 * @return string
 	 */
-	public function column_schedule( $row ) {
-		return $this->get_schedule_display_string( $row['schedule'] );
+	public function column_schedule($row) {
+		return $this->get_schedule_display_string($row['schedule']);
 	}
 
 	/**
@@ -440,25 +440,25 @@ 
 	 * @param ActionScheduler_Schedule $schedule
 	 * @return string
 	 */
-	protected function get_schedule_display_string( ActionScheduler_Schedule $schedule ) {
+	protected function get_schedule_display_string(ActionScheduler_Schedule $schedule) {
 
 		$schedule_display_string = '';
 
-		if ( ! $schedule->get_date() ) {
+		if (!$schedule->get_date()) {
 			return '0000-00-00 00:00:00';
 		}
 
 		$next_timestamp = $schedule->get_date()->getTimestamp();
 
-		$schedule_display_string .= $schedule->get_date()->format( 'Y-m-d H:i:s O' );
+		$schedule_display_string .= $schedule->get_date()->format('Y-m-d H:i:s O');
 		$schedule_display_string .= '<br/>';
 
-		if ( gmdate( 'U' ) > $next_timestamp ) {
+		if (gmdate('U') > $next_timestamp) {
 			/* translators: %s: date interval */
-			$schedule_display_string .= sprintf( __( ' (%s ago)', 'action-scheduler' ), self::human_interval( gmdate( 'U' ) - $next_timestamp ) );
+			$schedule_display_string .= sprintf(__(' (%s ago)', 'action-scheduler'), self::human_interval(gmdate('U') - $next_timestamp));
 		} else {
 			/* translators: %s: date interval */
-			$schedule_display_string .= sprintf( __( ' (%s)', 'action-scheduler' ), self::human_interval( $next_timestamp - gmdate( 'U' ) ) );
+			$schedule_display_string .= sprintf(__(' (%s)', 'action-scheduler'), self::human_interval($next_timestamp - gmdate('U')));
 		}
 
 		return $schedule_display_string;
@@ -473,9 +473,9 @@ 
 	 * @param array $ids
 	 * @param string $ids_sql Inherited and unused
 	 */
-	protected function bulk_delete( array $ids, $ids_sql ) {
-		foreach ( $ids as $id ) {
-			$this->store->delete_action( $id );
+	protected function bulk_delete(array $ids, $ids_sql) {
+		foreach ($ids as $id) {
+			$this->store->delete_action($id);
 		}
 	}
 
@@ -485,8 +485,8 @@ 
 	 *
 	 * @param int $action_id
 	 */
-	protected function row_action_cancel( $action_id ) {
-		$this->process_row_action( $action_id, 'cancel' );
+	protected function row_action_cancel($action_id) {
+		$this->process_row_action($action_id, 'cancel');
 	}
 
 	/**
@@ -495,27 +495,27 @@ 
 	 *
 	 * @param int $action_id
 	 */
-	protected function row_action_run( $action_id ) {
-		$this->process_row_action( $action_id, 'run' );
+	protected function row_action_run($action_id) {
+		$this->process_row_action($action_id, 'run');
 	}
 
 	/**
 	 * Force the data store schema updates.
 	 */
 	protected function recreate_tables() {
-		if ( is_a( $this->store, 'ActionScheduler_HybridStore' ) ) {
+		if (is_a($this->store, 'ActionScheduler_HybridStore')) {
 			$store = $this->store;
 		} else {
 			$store = new ActionScheduler_HybridStore();
 		}
-		add_action( 'action_scheduler/created_table', array( $store, 'set_autoincrement' ), 10, 2 );
+		add_action('action_scheduler/created_table', array($store, 'set_autoincrement'), 10, 2);
 
 		$store_schema  = new ActionScheduler_StoreSchema();
 		$logger_schema = new ActionScheduler_LoggerSchema();
-		$store_schema->register_tables( true );
-		$logger_schema->register_tables( true );
+		$store_schema->register_tables(true);
+		$logger_schema->register_tables(true);
 
-		remove_action( 'action_scheduler/created_table', array( $store, 'set_autoincrement' ), 10 );
+		remove_action('action_scheduler/created_table', array($store, 'set_autoincrement'), 10);
 	}
 	/**
 	 * Implements the logic behind processing an action once an action link is clicked on the list table.
@@ -523,24 +523,24 @@ 
 	 * @param int $action_id
 	 * @param string $row_action_type The type of action to perform on the action.
 	 */
-	protected function process_row_action( $action_id, $row_action_type ) {
+	protected function process_row_action($action_id, $row_action_type) {
 		try {
-			switch ( $row_action_type ) {
+			switch ($row_action_type) {
 				case 'run' :
-					$this->runner->process_action( $action_id, 'Admin List Table' );
+					$this->runner->process_action($action_id, 'Admin List Table');
 					break;
 				case 'cancel' :
-					$this->store->cancel_action( $action_id );
+					$this->store->cancel_action($action_id);
 					break;
 			}
 			$success = 1;
 			$error_message = '';
-		} catch ( Exception $e ) {
+		} catch (Exception $e) {
 			$success = 0;
 			$error_message = $e->getMessage();
 		}
 
-		set_transient( 'action_scheduler_admin_notice', compact( 'action_id', 'success', 'error_message', 'row_action_type' ), 30 );
+		set_transient('action_scheduler_admin_notice', compact('action_id', 'success', 'error_message', 'row_action_type'), 30);
 	}
 
 	/**
@@ -549,7 +549,7 @@ 
 	public function prepare_items() {
 		$this->prepare_column_headers();
 
-		$per_page = $this->get_items_per_page( $this->package . '_items_per_page', $this->items_per_page );
+		$per_page = $this->get_items_per_page($this->package . '_items_per_page', $this->items_per_page);
 		$query = array(
 			'per_page' => $per_page,
 			'offset'   => $this->get_items_offset(),
@@ -561,37 +561,37 @@ 
 
 		$this->items = array();
 
-		$total_items = $this->store->query_actions( $query, 'count' );
+		$total_items = $this->store->query_actions($query, 'count');
 
 		$status_labels = $this->store->get_status_labels();
 
-		foreach ( $this->store->query_actions( $query ) as $action_id ) {
+		foreach ($this->store->query_actions($query) as $action_id) {
 			try {
-				$action = $this->store->fetch_action( $action_id );
-			} catch ( Exception $e ) {
+				$action = $this->store->fetch_action($action_id);
+			} catch (Exception $e) {
 				continue;
 			}
-			if ( is_a( $action, 'ActionScheduler_NullAction' ) ) {
+			if (is_a($action, 'ActionScheduler_NullAction')) {
 				continue;
 			}
-			$this->items[ $action_id ] = array(
+			$this->items[$action_id] = array(
 				'ID'          => $action_id,
 				'hook'        => $action->get_hook(),
-				'status'      => $status_labels[ $this->store->get_status( $action_id ) ],
+				'status'      => $status_labels[$this->store->get_status($action_id)],
 				'args'        => $action->get_args(),
 				'group'       => $action->get_group(),
-				'log_entries' => $this->logger->get_logs( $action_id ),
-				'claim_id'    => $this->store->get_claim_id( $action_id ),
-				'recurrence'  => $this->get_recurrence( $action ),
+				'log_entries' => $this->logger->get_logs($action_id),
+				'claim_id'    => $this->store->get_claim_id($action_id),
+				'recurrence'  => $this->get_recurrence($action),
 				'schedule'    => $action->get_schedule(),
 			);
 		}
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $total_items,
 			'per_page'    => $per_page,
-			'total_pages' => ceil( $total_items / $per_page ),
-		) );
+			'total_pages' => ceil($total_items / $per_page),
+		));
 	}
 
 	/**
@@ -606,6 +606,6 @@ 
 	 * Get the text to display in the search box on the list table.
 	 */
 	protected function get_search_box_button_text() {
-		return __( 'Search hook, args and claim ID', 'action-scheduler' );
+		return __('Search hook, args and claim ID', 'action-scheduler');
 	}
 }

includes/libraries/action-scheduler/classes/WP_CLI/ProgressBar.php

Indentation +99 added lines, -99 removed lines

@@ -17,103 +17,103 @@
  */
 class ProgressBar {
 
-	/** @var integer */
-	protected $total_ticks;
-
-	/** @var integer */
-	protected $count;
-
-	/** @var integer */
-	protected $interval;
-
-	/** @var string */
-	protected $message;
-
-	/** @var \cli\progress\Bar */
-	protected $progress_bar;
-
-	/**
-	 * ProgressBar constructor.
-	 *
-	 * @param string  $message    Text to display before the progress bar.
-	 * @param integer $count      Total number of ticks to be performed.
-	 * @param integer $interval   Optional. The interval in milliseconds between updates. Default 100.
- 	 *
-	 * @throws Exception When this is not run within WP CLI
-	 */
-	public function __construct( $message, $count, $interval = 100 ) {
-		if ( ! ( defined( 'WP_CLI' ) && WP_CLI ) ) {
-			/* translators: %s php class name */
-			throw new \Exception( sprintf( __( 'The %s class can only be run within WP CLI.', 'action-scheduler' ), __CLASS__ ) );
-		}
-
-		$this->total_ticks = 0;
-		$this->message     = $message;
-		$this->count       = $count;
-		$this->interval    = $interval;
-	}
-
-	/**
-	 * Increment the progress bar ticks.
-	 */
-	public function tick() {
-		if ( null === $this->progress_bar ) {
-			$this->setup_progress_bar();
-		}
-
-		$this->progress_bar->tick();
-		$this->total_ticks++;
-
-		do_action( 'action_scheduler/progress_tick', $this->total_ticks );
-	}
-
-	/**
-	 * Get the progress bar tick count.
-	 *
-	 * @return int
-	 */
-	public function current() {
-		return $this->progress_bar ? $this->progress_bar->current() : 0;
-	}
-
-	/**
-	 * Finish the current progress bar.
-	 */
-	public function finish() {
-		if ( null !== $this->progress_bar ) {
-			$this->progress_bar->finish();
-		}
-
-		$this->progress_bar = null;
-	}
-
-	/**
-	 * Set the message used when creating the progress bar.
-	 *
-	 * @param string $message The message to be used when the next progress bar is created.
-	 */
-	public function set_message( $message ) {
-		$this->message = $message;
-	}
-
-	/**
-	 * Set the count for a new progress bar.
-	 *
-	 * @param integer $count The total number of ticks expected to complete.
-	 */
-	public function set_count( $count ) {
-		$this->count = $count;
-		$this->finish();
-	}
-
-	/**
-	 * Set up the progress bar.
-	 */
-	protected function setup_progress_bar() {
-		$this->progress_bar = \WP_CLI\Utils\make_progress_bar(
-			$this->message,
-			$this->count,
-			$this->interval
-		);
-	}
+    /** @var integer */
+    protected $total_ticks;
+
+    /** @var integer */
+    protected $count;
+
+    /** @var integer */
+    protected $interval;
+
+    /** @var string */
+    protected $message;
+
+    /** @var \cli\progress\Bar */
+    protected $progress_bar;
+
+    /**
+     * ProgressBar constructor.
+     *
+     * @param string  $message    Text to display before the progress bar.
+     * @param integer $count      Total number of ticks to be performed.
+     * @param integer $interval   Optional. The interval in milliseconds between updates. Default 100.
+     *
+     * @throws Exception When this is not run within WP CLI
+     */
+    public function __construct( $message, $count, $interval = 100 ) {
+        if ( ! ( defined( 'WP_CLI' ) && WP_CLI ) ) {
+            /* translators: %s php class name */
+            throw new \Exception( sprintf( __( 'The %s class can only be run within WP CLI.', 'action-scheduler' ), __CLASS__ ) );
+        }
+
+        $this->total_ticks = 0;
+        $this->message     = $message;
+        $this->count       = $count;
+        $this->interval    = $interval;
+    }
+
+    /**
+     * Increment the progress bar ticks.
+     */
+    public function tick() {
+        if ( null === $this->progress_bar ) {
+            $this->setup_progress_bar();
+        }
+
+        $this->progress_bar->tick();
+        $this->total_ticks++;
+
+        do_action( 'action_scheduler/progress_tick', $this->total_ticks );
+    }
+
+    /**
+     * Get the progress bar tick count.
+     *
+     * @return int
+     */
+    public function current() {
+        return $this->progress_bar ? $this->progress_bar->current() : 0;
+    }
+
+    /**
+     * Finish the current progress bar.
+     */
+    public function finish() {
+        if ( null !== $this->progress_bar ) {
+            $this->progress_bar->finish();
+        }
+
+        $this->progress_bar = null;
+    }
+
+    /**
+     * Set the message used when creating the progress bar.
+     *
+     * @param string $message The message to be used when the next progress bar is created.
+     */
+    public function set_message( $message ) {
+        $this->message = $message;
+    }
+
+    /**
+     * Set the count for a new progress bar.
+     *
+     * @param integer $count The total number of ticks expected to complete.
+     */
+    public function set_count( $count ) {
+        $this->count = $count;
+        $this->finish();
+    }
+
+    /**
+     * Set up the progress bar.
+     */
+    protected function setup_progress_bar() {
+        $this->progress_bar = \WP_CLI\Utils\make_progress_bar(
+            $this->message,
+            $this->count,
+            $this->interval
+        );
+    }
 }

Spacing +8 added lines, -8 removed lines

@@ -41,10 +41,10 @@ 
  	 *
 	 * @throws Exception When this is not run within WP CLI
 	 */
-	public function __construct( $message, $count, $interval = 100 ) {
-		if ( ! ( defined( 'WP_CLI' ) && WP_CLI ) ) {
+	public function __construct($message, $count, $interval = 100) {
+		if (!(defined('WP_CLI') && WP_CLI)) {
 			/* translators: %s php class name */
-			throw new \Exception( sprintf( __( 'The %s class can only be run within WP CLI.', 'action-scheduler' ), __CLASS__ ) );
+			throw new \Exception(sprintf(__('The %s class can only be run within WP CLI.', 'action-scheduler'), __CLASS__));
 		}
 
 		$this->total_ticks = 0;
@@ -57,14 +57,14 @@ 
 	 * Increment the progress bar ticks.
 	 */
 	public function tick() {
-		if ( null === $this->progress_bar ) {
+		if (null === $this->progress_bar) {
 			$this->setup_progress_bar();
 		}
 
 		$this->progress_bar->tick();
 		$this->total_ticks++;
 
-		do_action( 'action_scheduler/progress_tick', $this->total_ticks );
+		do_action('action_scheduler/progress_tick', $this->total_ticks);
 	}
 
 	/**
@@ -80,7 +80,7 @@ 
 	 * Finish the current progress bar.
 	 */
 	public function finish() {
-		if ( null !== $this->progress_bar ) {
+		if (null !== $this->progress_bar) {
 			$this->progress_bar->finish();
 		}
 
@@ -92,7 +92,7 @@ 
 	 *
 	 * @param string $message The message to be used when the next progress bar is created.
 	 */
-	public function set_message( $message ) {
+	public function set_message($message) {
 		$this->message = $message;
 	}
 
@@ -101,7 +101,7 @@ 
 	 *
 	 * @param integer $count The total number of ticks expected to complete.
 	 */
-	public function set_count( $count ) {
+	public function set_count($count) {
 		$this->count = $count;
 		$this->finish();
 	}