insideout10 / wordlift-plugin

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

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;
 	}
 

Indentation +19 added lines, -19 removed lines

@@ -5,25 +5,25 @@
  */
 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;
+    }
 }

src/libraries/action-scheduler/classes/ActionScheduler_InvalidActionException.php

Indentation +32 added lines, -32 removed lines

@@ -9,39 +9,39 @@
  */
 class ActionScheduler_InvalidActionException extends \InvalidArgumentException implements ActionScheduler_Exception {
 
-	/**
-	 * Create a new exception when the action's schedule cannot be fetched.
-	 *
-	 * @param string $action_id The action ID with bad args.
-	 * @return static
-	 */
-	public static function from_schedule( $action_id, $schedule ) {
-		$message = sprintf(
-			/* translators: 1: action ID 2: schedule */
-			__( 'Action [%1$s] has an invalid schedule: %2$s', 'action-scheduler' ),
-			$action_id,
-			var_export( $schedule, true )
-		);
+    /**
+     * Create a new exception when the action's schedule cannot be fetched.
+     *
+     * @param string $action_id The action ID with bad args.
+     * @return static
+     */
+    public static function from_schedule( $action_id, $schedule ) {
+        $message = sprintf(
+            /* translators: 1: action ID 2: schedule */
+            __( 'Action [%1$s] has an invalid schedule: %2$s', 'action-scheduler' ),
+            $action_id,
+            var_export( $schedule, true )
+        );
 
-		return new static( $message );
-	}
+        return new static( $message );
+    }
 
-	/**
-	 * Create a new exception when the action's args cannot be decoded to an array.
-	 *
-	 * @author Jeremy Pry
-	 *
-	 * @param string $action_id The action ID with bad args.
-	 * @return static
-	 */
-	public static function from_decoding_args( $action_id, $args = array() ) {
-		$message = sprintf(
-			/* translators: 1: action ID 2: arguments */
-			__( 'Action [%1$s] has invalid arguments. It cannot be JSON decoded to an array. $args = %2$s', 'action-scheduler' ),
-			$action_id,
-			var_export( $args, true )
-		);
+    /**
+     * Create a new exception when the action's args cannot be decoded to an array.
+     *
+     * @author Jeremy Pry
+     *
+     * @param string $action_id The action ID with bad args.
+     * @return static
+     */
+    public static function from_decoding_args( $action_id, $args = array() ) {
+        $message = sprintf(
+            /* translators: 1: action ID 2: arguments */
+            __( 'Action [%1$s] has invalid arguments. It cannot be JSON decoded to an array. $args = %2$s', 'action-scheduler' ),
+            $action_id,
+            var_export( $args, true )
+        );
 
-		return new static( $message );
-	}
+        return new static( $message );
+    }
 }

Spacing +8 added lines, -8 removed lines

@@ -15,15 +15,15 @@ 
 	 * @param string $action_id The action ID with bad args.
 	 * @return static
 	 */
-	public static function from_schedule( $action_id, $schedule ) {
+	public static function from_schedule($action_id, $schedule) {
 		$message = sprintf(
 			/* translators: 1: action ID 2: schedule */
-			__( 'Action [%1$s] has an invalid schedule: %2$s', 'action-scheduler' ),
+			__('Action [%1$s] has an invalid schedule: %2$s', 'action-scheduler'),
 			$action_id,
-			var_export( $schedule, true )
+			var_export($schedule, true)
 		);
 
-		return new static( $message );
+		return new static($message);
 	}
 
 	/**
@@ -34,14 +34,14 @@ 
 	 * @param string $action_id The action ID with bad args.
 	 * @return static
 	 */
-	public static function from_decoding_args( $action_id, $args = array() ) {
+	public static function from_decoding_args($action_id, $args = array()) {
 		$message = sprintf(
 			/* translators: 1: action ID 2: arguments */
-			__( 'Action [%1$s] has invalid arguments. It cannot be JSON decoded to an array. $args = %2$s', 'action-scheduler' ),
+			__('Action [%1$s] has invalid arguments. It cannot be JSON decoded to an array. $args = %2$s', 'action-scheduler'),
 			$action_id,
-			var_export( $args, true )
+			var_export($args, true)
 		);
 
-		return new static( $message );
+		return new static($message);
 	}
 }

src/libraries/action-scheduler/classes/ActionScheduler_OptionLock.php

Indentation +35 added lines, -35 removed lines

@@ -9,41 +9,41 @@
  */
 class ActionScheduler_OptionLock extends ActionScheduler_Lock {
 
-	/**
-	 * Set a lock using options for a given amount of time (60 seconds by default).
-	 *
-	 * Using an autoloaded option avoids running database queries or other resource intensive tasks
-	 * on frequently triggered hooks, like 'init' or 'shutdown'.
-	 *
-	 * For example, ActionScheduler_QueueRunner->maybe_dispatch_async_request() uses a lock to avoid
-	 * calling ActionScheduler_QueueRunner->has_maximum_concurrent_batches() every time the 'shutdown',
-	 * hook is triggered, because that method calls ActionScheduler_QueueRunner->store->get_claim_count()
-	 * to find the current number of claims in the database.
-	 *
-	 * @param string $lock_type A string to identify different lock types.
-	 * @bool True if lock value has changed, false if not or if set failed.
-	 */
-	public function set( $lock_type ) {
-		return update_option( $this->get_key( $lock_type ), time() + $this->get_duration( $lock_type ) );
-	}
+    /**
+     * Set a lock using options for a given amount of time (60 seconds by default).
+     *
+     * Using an autoloaded option avoids running database queries or other resource intensive tasks
+     * on frequently triggered hooks, like 'init' or 'shutdown'.
+     *
+     * For example, ActionScheduler_QueueRunner->maybe_dispatch_async_request() uses a lock to avoid
+     * calling ActionScheduler_QueueRunner->has_maximum_concurrent_batches() every time the 'shutdown',
+     * hook is triggered, because that method calls ActionScheduler_QueueRunner->store->get_claim_count()
+     * to find the current number of claims in the database.
+     *
+     * @param string $lock_type A string to identify different lock types.
+     * @bool True if lock value has changed, false if not or if set failed.
+     */
+    public function set( $lock_type ) {
+        return update_option( $this->get_key( $lock_type ), time() + $this->get_duration( $lock_type ) );
+    }
 
-	/**
-	 * If a lock is set, return the timestamp it was set to expiry.
-	 *
-	 * @param string $lock_type A string to identify different lock types.
-	 * @return bool|int False if no lock is set, otherwise the timestamp for when the lock is set to expire.
-	 */
-	public function get_expiration( $lock_type ) {
-		return get_option( $this->get_key( $lock_type ) );
-	}
+    /**
+     * If a lock is set, return the timestamp it was set to expiry.
+     *
+     * @param string $lock_type A string to identify different lock types.
+     * @return bool|int False if no lock is set, otherwise the timestamp for when the lock is set to expire.
+     */
+    public function get_expiration( $lock_type ) {
+        return get_option( $this->get_key( $lock_type ) );
+    }
 
-	/**
-	 * Get the key to use for storing the lock in the transient
-	 *
-	 * @param string $lock_type A string to identify different lock types.
-	 * @return string
-	 */
-	protected function get_key( $lock_type ) {
-		return sprintf( 'action_scheduler_lock_%s', $lock_type );
-	}
+    /**
+     * Get the key to use for storing the lock in the transient
+     *
+     * @param string $lock_type A string to identify different lock types.
+     * @return string
+     */
+    protected function get_key( $lock_type ) {
+        return sprintf( 'action_scheduler_lock_%s', $lock_type );
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -23,8 +23,8 @@ 
 	 * @param string $lock_type A string to identify different lock types.
 	 * @bool True if lock value has changed, false if not or if set failed.
 	 */
-	public function set( $lock_type ) {
-		return update_option( $this->get_key( $lock_type ), time() + $this->get_duration( $lock_type ) );
+	public function set($lock_type) {
+		return update_option($this->get_key($lock_type), time() + $this->get_duration($lock_type));
 	}
 
 	/**
@@ -33,8 +33,8 @@ 
 	 * @param string $lock_type A string to identify different lock types.
 	 * @return bool|int False if no lock is set, otherwise the timestamp for when the lock is set to expire.
 	 */
-	public function get_expiration( $lock_type ) {
-		return get_option( $this->get_key( $lock_type ) );
+	public function get_expiration($lock_type) {
+		return get_option($this->get_key($lock_type));
 	}
 
 	/**
@@ -43,7 +43,7 @@ 
 	 * @param string $lock_type A string to identify different lock types.
 	 * @return string
 	 */
-	protected function get_key( $lock_type ) {
-		return sprintf( 'action_scheduler_lock_%s', $lock_type );
+	protected function get_key($lock_type) {
+		return sprintf('action_scheduler_lock_%s', $lock_type);
 	}
 }

src/libraries/action-scheduler/classes/ActionScheduler_AsyncRequest_QueueRunner.php

Indentation +84 added lines, -84 removed lines

@@ -10,88 +10,88 @@
  */
 class ActionScheduler_AsyncRequest_QueueRunner extends WP_Async_Request {
 
-	/**
-	 * Data store for querying actions
-	 *
-	 * @var ActionScheduler_Store
-	 * @access protected
-	 */
-	protected $store;
-
-	/**
-	 * Prefix for ajax hooks
-	 *
-	 * @var string
-	 * @access protected
-	 */
-	protected $prefix = 'as';
-
-	/**
-	 * Action for ajax hooks
-	 *
-	 * @var string
-	 * @access protected
-	 */
-	protected $action = 'async_request_queue_runner';
-
-	/**
-	 * Initiate new async request
-	 */
-	public function __construct( ActionScheduler_Store $store ) {
-		parent::__construct();
-		$this->store = $store;
-	}
-
-	/**
-	 * Handle async requests
-	 *
-	 * Run a queue, and maybe dispatch another async request to run another queue
-	 * if there are still pending actions after completing a queue in this request.
-	 */
-	protected function handle() {
-		do_action( 'action_scheduler_run_queue', 'Async Request' ); // run a queue in the same way as WP Cron, but declare the Async Request context
-
-		$sleep_seconds = $this->get_sleep_seconds();
-
-		if ( $sleep_seconds ) {
-			sleep( $sleep_seconds );
-		}
-
-		$this->maybe_dispatch();
-	}
-
-	/**
-	 * If the async request runner is needed and allowed to run, dispatch a request.
-	 */
-	public function maybe_dispatch() {
-		if ( ! $this->allow() ) {
-			return;
-		}
-
-		$this->dispatch();
-		ActionScheduler_QueueRunner::instance()->unhook_dispatch_async_request();
-	}
-
-	/**
-	 * Only allow async requests when needed.
-	 *
-	 * Also allow 3rd party code to disable running actions via async requests.
-	 */
-	protected function allow() {
-
-		if ( ! has_action( 'action_scheduler_run_queue' ) || ActionScheduler::runner()->has_maximum_concurrent_batches() || ! $this->store->has_pending_actions_due() ) {
-			$allow = false;
-		} else {
-			$allow = true;
-		}
-
-		return apply_filters( 'action_scheduler_allow_async_request_runner', $allow );
-	}
-
-	/**
-	 * Chaining async requests can crash MySQL. A brief sleep call in PHP prevents that.
-	 */
-	protected function get_sleep_seconds() {
-		return apply_filters( 'action_scheduler_async_request_sleep_seconds', 5, $this );
-	}
+    /**
+     * Data store for querying actions
+     *
+     * @var ActionScheduler_Store
+     * @access protected
+     */
+    protected $store;
+
+    /**
+     * Prefix for ajax hooks
+     *
+     * @var string
+     * @access protected
+     */
+    protected $prefix = 'as';
+
+    /**
+     * Action for ajax hooks
+     *
+     * @var string
+     * @access protected
+     */
+    protected $action = 'async_request_queue_runner';
+
+    /**
+     * Initiate new async request
+     */
+    public function __construct( ActionScheduler_Store $store ) {
+        parent::__construct();
+        $this->store = $store;
+    }
+
+    /**
+     * Handle async requests
+     *
+     * Run a queue, and maybe dispatch another async request to run another queue
+     * if there are still pending actions after completing a queue in this request.
+     */
+    protected function handle() {
+        do_action( 'action_scheduler_run_queue', 'Async Request' ); // run a queue in the same way as WP Cron, but declare the Async Request context
+
+        $sleep_seconds = $this->get_sleep_seconds();
+
+        if ( $sleep_seconds ) {
+            sleep( $sleep_seconds );
+        }
+
+        $this->maybe_dispatch();
+    }
+
+    /**
+     * If the async request runner is needed and allowed to run, dispatch a request.
+     */
+    public function maybe_dispatch() {
+        if ( ! $this->allow() ) {
+            return;
+        }
+
+        $this->dispatch();
+        ActionScheduler_QueueRunner::instance()->unhook_dispatch_async_request();
+    }
+
+    /**
+     * Only allow async requests when needed.
+     *
+     * Also allow 3rd party code to disable running actions via async requests.
+     */
+    protected function allow() {
+
+        if ( ! has_action( 'action_scheduler_run_queue' ) || ActionScheduler::runner()->has_maximum_concurrent_batches() || ! $this->store->has_pending_actions_due() ) {
+            $allow = false;
+        } else {
+            $allow = true;
+        }
+
+        return apply_filters( 'action_scheduler_allow_async_request_runner', $allow );
+    }
+
+    /**
+     * Chaining async requests can crash MySQL. A brief sleep call in PHP prevents that.
+     */
+    protected function get_sleep_seconds() {
+        return apply_filters( 'action_scheduler_async_request_sleep_seconds', 5, $this );
+    }
 }

Spacing +9 added lines, -9 removed lines

@@ -3,7 +3,7 @@ 
  * ActionScheduler_AsyncRequest_QueueRunner
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * ActionScheduler_AsyncRequest_QueueRunner class.
@@ -37,7 +37,7 @@ 
 	/**
 	 * Initiate new async request
 	 */
-	public function __construct( ActionScheduler_Store $store ) {
+	public function __construct(ActionScheduler_Store $store) {
 		parent::__construct();
 		$this->store = $store;
 	}
@@ -49,12 +49,12 @@ 
 	 * if there are still pending actions after completing a queue in this request.
 	 */
 	protected function handle() {
-		do_action( 'action_scheduler_run_queue', 'Async Request' ); // run a queue in the same way as WP Cron, but declare the Async Request context
+		do_action('action_scheduler_run_queue', 'Async Request'); // run a queue in the same way as WP Cron, but declare the Async Request context
 
 		$sleep_seconds = $this->get_sleep_seconds();
 
-		if ( $sleep_seconds ) {
-			sleep( $sleep_seconds );
+		if ($sleep_seconds) {
+			sleep($sleep_seconds);
 		}
 
 		$this->maybe_dispatch();
@@ -64,7 +64,7 @@ 
 	 * If the async request runner is needed and allowed to run, dispatch a request.
 	 */
 	public function maybe_dispatch() {
-		if ( ! $this->allow() ) {
+		if ( ! $this->allow()) {
 			return;
 		}
 
@@ -79,19 +79,19 @@ 
 	 */
 	protected function allow() {
 
-		if ( ! has_action( 'action_scheduler_run_queue' ) || ActionScheduler::runner()->has_maximum_concurrent_batches() || ! $this->store->has_pending_actions_due() ) {
+		if ( ! has_action('action_scheduler_run_queue') || ActionScheduler::runner()->has_maximum_concurrent_batches() || ! $this->store->has_pending_actions_due()) {
 			$allow = false;
 		} else {
 			$allow = true;
 		}
 
-		return apply_filters( 'action_scheduler_allow_async_request_runner', $allow );
+		return apply_filters('action_scheduler_allow_async_request_runner', $allow);
 	}
 
 	/**
 	 * Chaining async requests can crash MySQL. A brief sleep call in PHP prevents that.
 	 */
 	protected function get_sleep_seconds() {
-		return apply_filters( 'action_scheduler_async_request_sleep_seconds', 5, $this );
+		return apply_filters('action_scheduler_async_request_sleep_seconds', 5, $this);
 	}
 }

src/libraries/action-scheduler/classes/ActionScheduler_DateTime.php

Indentation +63 added lines, -63 removed lines

@@ -7,73 +7,73 @@
  */
 class ActionScheduler_DateTime extends DateTime {
 
-	/**
-	 * UTC offset.
-	 *
-	 * Only used when a timezone is not set. When a timezone string is
-	 * used, this will be set to 0.
-	 *
-	 * @var int
-	 */
-	protected $utcOffset = 0;
+    /**
+     * UTC offset.
+     *
+     * Only used when a timezone is not set. When a timezone string is
+     * used, this will be set to 0.
+     *
+     * @var int
+     */
+    protected $utcOffset = 0;
 
-	/**
-	 * Get the unix timestamp of the current object.
-	 *
-	 * Missing in PHP 5.2 so just here so it can be supported consistently.
-	 *
-	 * @return int
-	 */
-	#[\ReturnTypeWillChange]
-	public function getTimestamp() {
-		return method_exists( 'DateTime', 'getTimestamp' ) ? parent::getTimestamp() : $this->format( 'U' );
-	}
+    /**
+     * Get the unix timestamp of the current object.
+     *
+     * Missing in PHP 5.2 so just here so it can be supported consistently.
+     *
+     * @return int
+     */
+    #[\ReturnTypeWillChange]
+    public function getTimestamp() {
+        return method_exists( 'DateTime', 'getTimestamp' ) ? parent::getTimestamp() : $this->format( 'U' );
+    }
 
-	/**
-	 * Set the UTC offset.
-	 *
-	 * This represents a fixed offset instead of a timezone setting.
-	 *
-	 * @param $offset
-	 */
-	public function setUtcOffset( $offset ) {
-		$this->utcOffset = intval( $offset );
-	}
+    /**
+     * Set the UTC offset.
+     *
+     * This represents a fixed offset instead of a timezone setting.
+     *
+     * @param $offset
+     */
+    public function setUtcOffset( $offset ) {
+        $this->utcOffset = intval( $offset );
+    }
 
-	/**
-	 * Returns the timezone offset.
-	 *
-	 * @return int
-	 * @link http://php.net/manual/en/datetime.getoffset.php
-	 */
-	#[\ReturnTypeWillChange]
-	public function getOffset() {
-		return $this->utcOffset ? $this->utcOffset : parent::getOffset();
-	}
+    /**
+     * Returns the timezone offset.
+     *
+     * @return int
+     * @link http://php.net/manual/en/datetime.getoffset.php
+     */
+    #[\ReturnTypeWillChange]
+    public function getOffset() {
+        return $this->utcOffset ? $this->utcOffset : parent::getOffset();
+    }
 
-	/**
-	 * Set the TimeZone associated with the DateTime
-	 *
-	 * @param DateTimeZone $timezone
-	 *
-	 * @return static
-	 * @link http://php.net/manual/en/datetime.settimezone.php
-	 */
-	#[\ReturnTypeWillChange]
-	public function setTimezone( $timezone ) {
-		$this->utcOffset = 0;
-		parent::setTimezone( $timezone );
+    /**
+     * Set the TimeZone associated with the DateTime
+     *
+     * @param DateTimeZone $timezone
+     *
+     * @return static
+     * @link http://php.net/manual/en/datetime.settimezone.php
+     */
+    #[\ReturnTypeWillChange]
+    public function setTimezone( $timezone ) {
+        $this->utcOffset = 0;
+        parent::setTimezone( $timezone );
 
-		return $this;
-	}
+        return $this;
+    }
 
-	/**
-	 * Get the timestamp with the WordPress timezone offset added or subtracted.
-	 *
-	 * @since  3.0.0
-	 * @return int
-	 */
-	public function getOffsetTimestamp() {
-		return $this->getTimestamp() + $this->getOffset();
-	}
+    /**
+     * Get the timestamp with the WordPress timezone offset added or subtracted.
+     *
+     * @since  3.0.0
+     * @return int
+     */
+    public function getOffsetTimestamp() {
+        return $this->getTimestamp() + $this->getOffset();
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -26,7 +26,7 @@ 
 	 */
 	#[\ReturnTypeWillChange]
 	public function getTimestamp() {
-		return method_exists( 'DateTime', 'getTimestamp' ) ? parent::getTimestamp() : $this->format( 'U' );
+		return method_exists('DateTime', 'getTimestamp') ? parent::getTimestamp() : $this->format('U');
 	}
 
 	/**
@@ -36,8 +36,8 @@ 
 	 *
 	 * @param $offset
 	 */
-	public function setUtcOffset( $offset ) {
-		$this->utcOffset = intval( $offset );
+	public function setUtcOffset($offset) {
+		$this->utcOffset = intval($offset);
 	}
 
 	/**
@@ -60,9 +60,9 @@ 
 	 * @link http://php.net/manual/en/datetime.settimezone.php
 	 */
 	#[\ReturnTypeWillChange]
-	public function setTimezone( $timezone ) {
+	public function setTimezone($timezone) {
 		$this->utcOffset = 0;
-		parent::setTimezone( $timezone );
+		parent::setTimezone($timezone);
 
 		return $this;
 	}

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

Indentation +130 added lines, -130 removed lines

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

Spacing +33 added lines, -33 removed lines

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

src/libraries/action-scheduler/classes/abstracts/ActionScheduler_Abstract_ListTable.php

Indentation +741 added lines, -741 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+    require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -23,744 +23,744 @@ 
  */
 abstract class ActionScheduler_Abstract_ListTable extends WP_List_Table {
 
-	/**
-	 * The table name
-	 *
-	 * @var string
-	 */
-	protected $table_name;
-
-	/**
-	 * Package name, used to get options from WP_List_Table::get_items_per_page.
-	 *
-	 * @var string
-	 */
-	protected $package;
-
-	/**
-	 * How many items do we render per page?
-	 *
-	 * @var int
-	 */
-	protected $items_per_page = 10;
-
-	/**
-	 * Enables search in this table listing. If this array
-	 * is empty it means the listing is not searchable.
-	 *
-	 * @var array
-	 */
-	protected $search_by = array();
-
-	/**
-	 * Columns to show in the table listing. It is a key => value pair. The
-	 * key must much the table column name and the value is the label, which is
-	 * automatically translated.
-	 *
-	 * @var array
-	 */
-	protected $columns = array();
-
-	/**
-	 * Defines the row-actions. It expects an array where the key
-	 * is the column name and the value is an array of actions.
-	 *
-	 * The array of actions are key => value, where key is the method name
-	 * (with the prefix row_action_<key>) and the value is the label
-	 * and title.
-	 *
-	 * @var array
-	 */
-	protected $row_actions = array();
-
-	/**
-	 * The Primary key of our table
-	 *
-	 * @var string
-	 */
-	protected $ID = 'ID';
-
-	/**
-	 * Enables sorting, it expects an array
-	 * of columns (the column names are the values)
-	 *
-	 * @var array
-	 */
-	protected $sort_by = array();
-
-	/**
-	 * The default sort order
-	 *
-	 * @var string
-	 */
-	protected $filter_by = array();
-
-	/**
-	 * The status name => count combinations for this table's items. Used to display status filters.
-	 *
-	 * @var array
-	 */
-	protected $status_counts = array();
-
-	/**
-	 * Notices to display when loading the table. Array of arrays of form array( 'class' => {updated|error}, 'message' => 'This is the notice text display.' ).
-	 *
-	 * @var array
-	 */
-	protected $admin_notices = array();
-
-	/**
-	 * Localised string displayed in the <h1> element above the able.
-	 *
-	 * @var string
-	 */
-	protected $table_header;
-
-	/**
-	 * Enables bulk actions. It must be an array where the key is the action name
-	 * and the value is the label (which is translated automatically). It is important
-	 * to notice that it will check that the method exists (`bulk_$name`) and will throw
-	 * an exception if it does not exists.
-	 *
-	 * This class will automatically check if the current request has a bulk action, will do the
-	 * validations and afterwards will execute the bulk method, with two arguments. The first argument
-	 * is the array with primary keys, the second argument is a string with a list of the primary keys,
-	 * escaped and ready to use (with `IN`).
-	 *
-	 * @var array
-	 */
-	protected $bulk_actions = array();
-
-	/**
-	 * Makes translation easier, it basically just wraps
-	 * `_x` with some default (the package name).
-	 *
-	 * @param string $text The new text to translate.
-	 * @param string $context The context of the text.
-	 * @return string|void The translated text.
-	 *
-	 * @deprecated 3.0.0 Use `_x()` instead.
-	 */
-	protected function translate( $text, $context = '' ) {
-		return $text;
-	}
-
-	/**
-	 * Reads `$this->bulk_actions` and returns an array that WP_List_Table understands. It
-	 * also validates that the bulk method handler exists. It throws an exception because
-	 * this is a library meant for developers and missing a bulk method is a development-time error.
-	 *
-	 * @return array
-	 *
-	 * @throws RuntimeException Throws RuntimeException when the bulk action does not have a callback method.
-	 */
-	protected function get_bulk_actions() {
-		$actions = array();
-
-		foreach ( $this->bulk_actions as $action => $label ) {
-			if ( ! is_callable( array( $this, 'bulk_' . $action ) ) ) {
-				throw new RuntimeException( "The bulk action $action does not have a callback method" );
-			}
-
-			$actions[ $action ] = $label;
-		}
-
-		return $actions;
-	}
-
-	/**
-	 * Checks if the current request has a bulk action. If that is the case it will validate and will
-	 * execute the bulk method handler. Regardless if the action is valid or not it will redirect to
-	 * the previous page removing the current arguments that makes this request a bulk action.
-	 */
-	protected function process_bulk_action() {
-		global $wpdb;
-		// Detect when a bulk action is being triggered.
-		$action = $this->current_action();
-		if ( ! $action ) {
-			return;
-		}
-
-		check_admin_referer( 'bulk-' . $this->_args['plural'] );
-
-		$method = 'bulk_' . $action;
-		if ( array_key_exists( $action, $this->bulk_actions ) && is_callable( array( $this, $method ) ) && ! empty( $_GET['ID'] ) && is_array( $_GET['ID'] ) ) {
-			$ids_sql = '(' . implode( ',', array_fill( 0, count( $_GET['ID'] ), '%s' ) ) . ')';
-			$id      = array_map( 'absint', $_GET['ID'] );
-			$this->$method( $id, $wpdb->prepare( $ids_sql, $id ) ); //phpcs:ignore WordPress.DB.PreparedSQL
-		}
-
-		if ( isset( $_SERVER['REQUEST_URI'] ) ) {
-			wp_safe_redirect(
-				remove_query_arg(
-					array( '_wp_http_referer', '_wpnonce', 'ID', 'action', 'action2' ),
-					esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'] ) )
-				)
-			);
-			exit;
-		}
-	}
-
-	/**
-	 * Default code for deleting entries.
-	 * validated already by process_bulk_action()
-	 *
-	 * @param array  $ids ids of the items to delete.
-	 * @param string $ids_sql the sql for the ids.
-	 * @return void
-	 */
-	protected function bulk_delete( array $ids, $ids_sql ) {
-		$store = ActionScheduler::store();
-		foreach ( $ids as $action_id ) {
-			$store->delete( $action_id );
-		}
-	}
-
-	/**
-	 * Prepares the _column_headers property which is used by WP_Table_List at rendering.
-	 * It merges the columns and the sortable columns.
-	 */
-	protected function prepare_column_headers() {
-		$this->_column_headers = array(
-			$this->get_columns(),
-			get_hidden_columns( $this->screen ),
-			$this->get_sortable_columns(),
-		);
-	}
-
-	/**
-	 * Reads $this->sort_by and returns the columns name in a format that WP_Table_List
-	 * expects
-	 */
-	public function get_sortable_columns() {
-		$sort_by = array();
-		foreach ( $this->sort_by as $column ) {
-			$sort_by[ $column ] = array( $column, true );
-		}
-		return $sort_by;
-	}
-
-	/**
-	 * Returns the columns names for rendering. It adds a checkbox for selecting everything
-	 * as the first column
-	 */
-	public function get_columns() {
-		$columns = array_merge(
-			array( 'cb' => '<input type="checkbox" />' ),
-			$this->columns
-		);
-
-		return $columns;
-	}
-
-	/**
-	 * Get prepared LIMIT clause for items query
-	 *
-	 * @global wpdb $wpdb
-	 *
-	 * @return string Prepared LIMIT clause for items query.
-	 */
-	protected function get_items_query_limit() {
-		global $wpdb;
-
-		$per_page = $this->get_items_per_page( $this->get_per_page_option_name(), $this->items_per_page );
-		return $wpdb->prepare( 'LIMIT %d', $per_page );
-	}
-
-	/**
-	 * Returns the number of items to offset/skip for this current view.
-	 *
-	 * @return int
-	 */
-	protected function get_items_offset() {
-		$per_page     = $this->get_items_per_page( $this->get_per_page_option_name(), $this->items_per_page );
-		$current_page = $this->get_pagenum();
-		if ( 1 < $current_page ) {
-			$offset = $per_page * ( $current_page - 1 );
-		} else {
-			$offset = 0;
-		}
-
-		return $offset;
-	}
-
-	/**
-	 * Get prepared OFFSET clause for items query
-	 *
-	 * @global wpdb $wpdb
-	 *
-	 * @return string Prepared OFFSET clause for items query.
-	 */
-	protected function get_items_query_offset() {
-		global $wpdb;
-
-		return $wpdb->prepare( 'OFFSET %d', $this->get_items_offset() );
-	}
-
-	/**
-	 * Prepares the ORDER BY sql statement. It uses `$this->sort_by` to know which
-	 * columns are sortable. This requests validates the orderby $_GET parameter is a valid
-	 * column and sortable. It will also use order (ASC|DESC) using DESC by default.
-	 */
-	protected function get_items_query_order() {
-		if ( empty( $this->sort_by ) ) {
-			return '';
-		}
-
-		$orderby = esc_sql( $this->get_request_orderby() );
-		$order   = esc_sql( $this->get_request_order() );
-
-		return "ORDER BY {$orderby} {$order}";
-	}
-
-	/**
-	 * Return the sortable column specified for this request to order the results by, if any.
-	 *
-	 * @return string
-	 */
-	protected function get_request_orderby() {
-
-		$valid_sortable_columns = array_values( $this->sort_by );
-
-		if ( ! empty( $_GET['orderby'] ) && in_array( $_GET['orderby'], $valid_sortable_columns, true ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			$orderby = sanitize_text_field( wp_unslash( $_GET['orderby'] ) ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-		} else {
-			$orderby = $valid_sortable_columns[0];
-		}
-
-		return $orderby;
-	}
-
-	/**
-	 * Return the sortable column order specified for this request.
-	 *
-	 * @return string
-	 */
-	protected function get_request_order() {
-
-		if ( ! empty( $_GET['order'] ) && 'desc' === strtolower( sanitize_text_field( wp_unslash( $_GET['order'] ) ) ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			$order = 'DESC';
-		} else {
-			$order = 'ASC';
-		}
-
-		return $order;
-	}
-
-	/**
-	 * Return the status filter for this request, if any.
-	 *
-	 * @return string
-	 */
-	protected function get_request_status() {
-		$status = ( ! empty( $_GET['status'] ) ) ? sanitize_text_field( wp_unslash( $_GET['status'] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-		return $status;
-	}
-
-	/**
-	 * Return the search filter for this request, if any.
-	 *
-	 * @return string
-	 */
-	protected function get_request_search_query() {
-		$search_query = ( ! empty( $_GET['s'] ) ) ? sanitize_text_field( wp_unslash( $_GET['s'] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-		return $search_query;
-	}
-
-	/**
-	 * Process and return the columns name. This is meant for using with SQL, this means it
-	 * always includes the primary key.
-	 *
-	 * @return array
-	 */
-	protected function get_table_columns() {
-		$columns = array_keys( $this->columns );
-		if ( ! in_array( $this->ID, $columns, true ) ) {
-			$columns[] = $this->ID;
-		}
-
-		return $columns;
-	}
-
-	/**
-	 * Check if the current request is doing a "full text" search. If that is the case
-	 * prepares the SQL to search texts using LIKE.
-	 *
-	 * If the current request does not have any search or if this list table does not support
-	 * that feature it will return an empty string.
-	 *
-	 * @return string
-	 */
-	protected function get_items_query_search() {
-		global $wpdb;
-
-		if ( empty( $_GET['s'] ) || empty( $this->search_by ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			return '';
-		}
-
-		$search_string = sanitize_text_field( wp_unslash( $_GET['s'] ) ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-
-		$filter = array();
-		foreach ( $this->search_by as $column ) {
-			$wild     = '%';
-			$sql_like = $wild . $wpdb->esc_like( $search_string ) . $wild;
-			$filter[] = $wpdb->prepare( '`' . $column . '` LIKE %s', $sql_like ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.DB.PreparedSQL.NotPrepared
-		}
-		return implode( ' OR ', $filter );
-	}
-
-	/**
-	 * Prepares the SQL to filter rows by the options defined at `$this->filter_by`. Before trusting
-	 * any data sent by the user it validates that it is a valid option.
-	 */
-	protected function get_items_query_filters() {
-		global $wpdb;
-
-		if ( ! $this->filter_by || empty( $_GET['filter_by'] ) || ! is_array( $_GET['filter_by'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			return '';
-		}
-
-		$filter = array();
-
-		foreach ( $this->filter_by as $column => $options ) {
-			if ( empty( $_GET['filter_by'][ $column ] ) || empty( $options[ $_GET['filter_by'][ $column ] ] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-				continue;
-			}
-
-			$filter[] = $wpdb->prepare( "`$column` = %s", sanitize_text_field( wp_unslash( $_GET['filter_by'][ $column ] ) ) ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.DB.PreparedSQL.InterpolatedNotPrepared
-		}
-
-		return implode( ' AND ', $filter );
-
-	}
-
-	/**
-	 * Prepares the data to feed WP_Table_List.
-	 *
-	 * This has the core for selecting, sorting and filting data. To keep the code simple
-	 * its logic is split among many methods (get_items_query_*).
-	 *
-	 * Beside populating the items this function will also count all the records that matches
-	 * the filtering criteria and will do fill the pagination variables.
-	 */
-	public function prepare_items() {
-		global $wpdb;
-
-		$this->process_bulk_action();
-
-		$this->process_row_actions();
-
-		if ( ! empty( $_REQUEST['_wp_http_referer'] && ! empty( $_SERVER['REQUEST_URI'] ) ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			// _wp_http_referer is used only on bulk actions, we remove it to keep the $_GET shorter
-			wp_safe_redirect( remove_query_arg( array( '_wp_http_referer', '_wpnonce' ), esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'] ) ) ) );
-			exit;
-		}
-
-		$this->prepare_column_headers();
-
-		$limit   = $this->get_items_query_limit();
-		$offset  = $this->get_items_query_offset();
-		$order   = $this->get_items_query_order();
-		$where   = array_filter(
-			array(
-				$this->get_items_query_search(),
-				$this->get_items_query_filters(),
-			)
-		);
-		$columns = '`' . implode( '`, `', $this->get_table_columns() ) . '`';
-
-		if ( ! empty( $where ) ) {
-			$where = 'WHERE (' . implode( ') AND (', $where ) . ')';
-		} else {
-			$where = '';
-		}
-
-		$sql = "SELECT $columns FROM {$this->table_name} {$where} {$order} {$limit} {$offset}";
-
-		$this->set_items( $wpdb->get_results( $sql, ARRAY_A ) ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
-
-		$query_count = "SELECT COUNT({$this->ID}) FROM {$this->table_name} {$where}";
-		$total_items = $wpdb->get_var( $query_count ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
-		$per_page    = $this->get_items_per_page( $this->get_per_page_option_name(), $this->items_per_page );
-		$this->set_pagination_args(
-			array(
-				'total_items' => $total_items,
-				'per_page'    => $per_page,
-				'total_pages' => ceil( $total_items / $per_page ),
-			)
-		);
-	}
-
-	/**
-	 * Display the table.
-	 *
-	 * @param string $which The name of the table.
-	 */
-	public function extra_tablenav( $which ) {
-		if ( ! $this->filter_by || 'top' !== $which ) {
-			return;
-		}
-
-		echo '<div class="alignleft actions">';
-
-		foreach ( $this->filter_by as $id => $options ) {
-			$default = ! empty( $_GET['filter_by'][ $id ] ) ? sanitize_text_field( wp_unslash( $_GET['filter_by'][ $id ] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			if ( empty( $options[ $default ] ) ) {
-				$default = '';
-			}
-
-			echo '<select name="filter_by[' . esc_attr( $id ) . ']" class="first" id="filter-by-' . esc_attr( $id ) . '">';
-
-			foreach ( $options as $value => $label ) {
-				echo '<option value="' . esc_attr( $value ) . '" ' . esc_html( $value === $default ? 'selected' : '' ) . '>'
-					. esc_html( $label )
-				. '</option>';
-			}
-
-			echo '</select>';
-		}
-
-		submit_button( esc_html__( 'Filter', 'action-scheduler' ), '', 'filter_action', false, array( 'id' => 'post-query-submit' ) );
-		echo '</div>';
-	}
-
-	/**
-	 * Set the data for displaying. It will attempt to unserialize (There is a chance that some columns
-	 * are serialized). This can be override in child classes for futher data transformation.
-	 *
-	 * @param array $items Items array.
-	 */
-	protected function set_items( array $items ) {
-		$this->items = array();
-		foreach ( $items as $item ) {
-			$this->items[ $item[ $this->ID ] ] = array_map( 'maybe_unserialize', $item );
-		}
-	}
-
-	/**
-	 * Renders the checkbox for each row, this is the first column and it is named ID regardless
-	 * of how the primary key is named (to keep the code simpler). The bulk actions will do the proper
-	 * name transformation though using `$this->ID`.
-	 *
-	 * @param array $row The row to render.
-	 */
-	public function column_cb( $row ) {
-		return '<input name="ID[]" type="checkbox" value="' . esc_attr( $row[ $this->ID ] ) . '" />';
-	}
-
-	/**
-	 * Renders the row-actions.
-	 *
-	 * This method renders the action menu, it reads the definition from the $row_actions property,
-	 * and it checks that the row action method exists before rendering it.
-	 *
-	 * @param array  $row Row to be rendered.
-	 * @param string $column_name Column name.
-	 * @return string
-	 */
-	protected function maybe_render_actions( $row, $column_name ) {
-		if ( empty( $this->row_actions[ $column_name ] ) ) {
-			return;
-		}
-
-		$row_id = $row[ $this->ID ];
-
-		$actions      = '<div class="row-actions">';
-		$action_count = 0;
-		foreach ( $this->row_actions[ $column_name ] as $action_key => $action ) {
-
-			$action_count++;
-
-			if ( ! method_exists( $this, 'row_action_' . $action_key ) ) {
-				continue;
-			}
-
-			$action_link = ! empty( $action['link'] ) ? $action['link'] : add_query_arg(
-				array(
-					'row_action' => $action_key,
-					'row_id'     => $row_id,
-					'nonce'      => wp_create_nonce( $action_key . '::' . $row_id ),
-				)
-			);
-			$span_class  = ! empty( $action['class'] ) ? $action['class'] : $action_key;
-			$separator   = ( $action_count < count( $this->row_actions[ $column_name ] ) ) ? ' | ' : '';
-
-			$actions .= sprintf( '<span class="%s">', esc_attr( $span_class ) );
-			$actions .= sprintf( '<a href="%1$s" title="%2$s">%3$s</a>', esc_url( $action_link ), esc_attr( $action['desc'] ), esc_html( $action['name'] ) );
-			$actions .= sprintf( '%s</span>', $separator );
-		}
-		$actions .= '</div>';
-		return $actions;
-	}
-
-	/**
-	 * Process the bulk actions.
-	 *
-	 * @return void
-	 */
-	protected function process_row_actions() {
-		$parameters = array( 'row_action', 'row_id', 'nonce' );
-		foreach ( $parameters as $parameter ) {
-			if ( empty( $_REQUEST[ $parameter ] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-				return;
-			}
-		}
-
-		$action = sanitize_text_field( wp_unslash( $_REQUEST['row_action'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.Security.ValidatedSanitizedInput.InputNotValidated
-		$row_id = sanitize_text_field( wp_unslash( $_REQUEST['row_id'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.Security.ValidatedSanitizedInput.InputNotValidated
-		$nonce  = sanitize_text_field( wp_unslash( $_REQUEST['nonce'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.Security.ValidatedSanitizedInput.InputNotValidated
-		$method = 'row_action_' . $action; // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-
-		if ( wp_verify_nonce( $nonce, $action . '::' . $row_id ) && method_exists( $this, $method ) ) {
-			$this->$method( sanitize_text_field( wp_unslash( $row_id ) ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-		}
-
-		if ( isset( $_SERVER['REQUEST_URI'] ) ) {
-			wp_safe_redirect(
-				remove_query_arg(
-					array( 'row_id', 'row_action', 'nonce' ),
-					esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'] ) )
-				)
-			);
-			exit;
-		}
-	}
-
-	/**
-	 * Default column formatting, it will escape everythig for security.
-	 *
-	 * @param array  $item The item array.
-	 * @param string $column_name Column name to display.
-	 *
-	 * @return string
-	 */
-	public function column_default( $item, $column_name ) {
-		$column_html  = esc_html( $item[ $column_name ] );
-		$column_html .= $this->maybe_render_actions( $item, $column_name );
-		return $column_html;
-	}
-
-	/**
-	 * Display the table heading and search query, if any
-	 */
-	protected function display_header() {
-		echo '<h1 class="wp-heading-inline">' . esc_attr( $this->table_header ) . '</h1>';
-		if ( $this->get_request_search_query() ) {
-			/* translators: %s: search query */
-			echo '<span class="subtitle">' . esc_attr( sprintf( __( 'Search results for "%s"', 'action-scheduler' ), $this->get_request_search_query() ) ) . '</span>';
-		}
-		echo '<hr class="wp-header-end">';
-	}
-
-	/**
-	 * Display the table heading and search query, if any
-	 */
-	protected function display_admin_notices() {
-		foreach ( $this->admin_notices as $notice ) {
-			echo '<div id="message" class="' . esc_attr( $notice['class'] ) . '">';
-			echo '	<p>' . wp_kses_post( $notice['message'] ) . '</p>';
-			echo '</div>';
-		}
-	}
-
-	/**
-	 * Prints the available statuses so the user can click to filter.
-	 */
-	protected function display_filter_by_status() {
-
-		$status_list_items = array();
-		$request_status    = $this->get_request_status();
-
-		// Helper to set 'all' filter when not set on status counts passed in.
-		if ( ! isset( $this->status_counts['all'] ) ) {
-			$this->status_counts = array( 'all' => array_sum( $this->status_counts ) ) + $this->status_counts;
-		}
-
-		foreach ( $this->status_counts as $status_name => $count ) {
-
-			if ( 0 === $count ) {
-				continue;
-			}
-
-			if ( $status_name === $request_status || ( empty( $request_status ) && 'all' === $status_name ) ) {
-				$status_list_item = '<li class="%1$s"><a href="%2$s" class="current">%3$s</a> (%4$d)</li>';
-			} else {
-				$status_list_item = '<li class="%1$s"><a href="%2$s">%3$s</a> (%4$d)</li>';
-			}
-
-			$status_filter_url   = ( 'all' === $status_name ) ? remove_query_arg( 'status' ) : add_query_arg( 'status', $status_name );
-			$status_filter_url   = remove_query_arg( array( 'paged', 's' ), $status_filter_url );
-			$status_list_items[] = sprintf( $status_list_item, esc_attr( $status_name ), esc_url( $status_filter_url ), esc_html( ucfirst( $status_name ) ), absint( $count ) );
-		}
-
-		if ( $status_list_items ) {
-			echo '<ul class="subsubsub">';
-			echo implode( " | \n", $status_list_items ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
-			echo '</ul>';
-		}
-	}
-
-	/**
-	 * Renders the table list, we override the original class to render the table inside a form
-	 * and to render any needed HTML (like the search box). By doing so the callee of a function can simple
-	 * forget about any extra HTML.
-	 */
-	protected function display_table() {
-		echo '<form id="' . esc_attr( $this->_args['plural'] ) . '-filter" method="get">';
-		foreach ( $_GET as $key => $value ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			if ( '_' === $key[0] || 'paged' === $key || 'ID' === $key ) {
-				continue;
-			}
-			echo '<input type="hidden" name="' . esc_attr( $key ) . '" value="' . esc_attr( $value ) . '" />';
-		}
-		if ( ! empty( $this->search_by ) ) {
-			echo $this->search_box( $this->get_search_box_button_text(), 'plugin' ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
-		}
-		parent::display();
-		echo '</form>';
-	}
-
-	/**
-	 * Process any pending actions.
-	 */
-	public function process_actions() {
-		$this->process_bulk_action();
-		$this->process_row_actions();
-
-		if ( ! empty( $_REQUEST['_wp_http_referer'] ) && ! empty( $_SERVER['REQUEST_URI'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			// _wp_http_referer is used only on bulk actions, we remove it to keep the $_GET shorter
-			wp_safe_redirect( remove_query_arg( array( '_wp_http_referer', '_wpnonce' ), esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'] ) ) ) );
-			exit;
-		}
-	}
-
-	/**
-	 * Render the list table page, including header, notices, status filters and table.
-	 */
-	public function display_page() {
-		$this->prepare_items();
-
-		echo '<div class="wrap">';
-		$this->display_header();
-		$this->display_admin_notices();
-		$this->display_filter_by_status();
-		$this->display_table();
-		echo '</div>';
-	}
-
-	/**
-	 * Get the text to display in the search box on the list table.
-	 */
-	protected function get_search_box_placeholder() {
-		return esc_html__( 'Search', 'action-scheduler' );
-	}
-
-	/**
-	 * Gets the screen per_page option name.
-	 *
-	 * @return string
-	 */
-	protected function get_per_page_option_name() {
-		return $this->package . '_items_per_page';
-	}
+    /**
+     * The table name
+     *
+     * @var string
+     */
+    protected $table_name;
+
+    /**
+     * Package name, used to get options from WP_List_Table::get_items_per_page.
+     *
+     * @var string
+     */
+    protected $package;
+
+    /**
+     * How many items do we render per page?
+     *
+     * @var int
+     */
+    protected $items_per_page = 10;
+
+    /**
+     * Enables search in this table listing. If this array
+     * is empty it means the listing is not searchable.
+     *
+     * @var array
+     */
+    protected $search_by = array();
+
+    /**
+     * Columns to show in the table listing. It is a key => value pair. The
+     * key must much the table column name and the value is the label, which is
+     * automatically translated.
+     *
+     * @var array
+     */
+    protected $columns = array();
+
+    /**
+     * Defines the row-actions. It expects an array where the key
+     * is the column name and the value is an array of actions.
+     *
+     * The array of actions are key => value, where key is the method name
+     * (with the prefix row_action_<key>) and the value is the label
+     * and title.
+     *
+     * @var array
+     */
+    protected $row_actions = array();
+
+    /**
+     * The Primary key of our table
+     *
+     * @var string
+     */
+    protected $ID = 'ID';
+
+    /**
+     * Enables sorting, it expects an array
+     * of columns (the column names are the values)
+     *
+     * @var array
+     */
+    protected $sort_by = array();
+
+    /**
+     * The default sort order
+     *
+     * @var string
+     */
+    protected $filter_by = array();
+
+    /**
+     * The status name => count combinations for this table's items. Used to display status filters.
+     *
+     * @var array
+     */
+    protected $status_counts = array();
+
+    /**
+     * Notices to display when loading the table. Array of arrays of form array( 'class' => {updated|error}, 'message' => 'This is the notice text display.' ).
+     *
+     * @var array
+     */
+    protected $admin_notices = array();
+
+    /**
+     * Localised string displayed in the <h1> element above the able.
+     *
+     * @var string
+     */
+    protected $table_header;
+
+    /**
+     * Enables bulk actions. It must be an array where the key is the action name
+     * and the value is the label (which is translated automatically). It is important
+     * to notice that it will check that the method exists (`bulk_$name`) and will throw
+     * an exception if it does not exists.
+     *
+     * This class will automatically check if the current request has a bulk action, will do the
+     * validations and afterwards will execute the bulk method, with two arguments. The first argument
+     * is the array with primary keys, the second argument is a string with a list of the primary keys,
+     * escaped and ready to use (with `IN`).
+     *
+     * @var array
+     */
+    protected $bulk_actions = array();
+
+    /**
+     * Makes translation easier, it basically just wraps
+     * `_x` with some default (the package name).
+     *
+     * @param string $text The new text to translate.
+     * @param string $context The context of the text.
+     * @return string|void The translated text.
+     *
+     * @deprecated 3.0.0 Use `_x()` instead.
+     */
+    protected function translate( $text, $context = '' ) {
+        return $text;
+    }
+
+    /**
+     * Reads `$this->bulk_actions` and returns an array that WP_List_Table understands. It
+     * also validates that the bulk method handler exists. It throws an exception because
+     * this is a library meant for developers and missing a bulk method is a development-time error.
+     *
+     * @return array
+     *
+     * @throws RuntimeException Throws RuntimeException when the bulk action does not have a callback method.
+     */
+    protected function get_bulk_actions() {
+        $actions = array();
+
+        foreach ( $this->bulk_actions as $action => $label ) {
+            if ( ! is_callable( array( $this, 'bulk_' . $action ) ) ) {
+                throw new RuntimeException( "The bulk action $action does not have a callback method" );
+            }
+
+            $actions[ $action ] = $label;
+        }
+
+        return $actions;
+    }
+
+    /**
+     * Checks if the current request has a bulk action. If that is the case it will validate and will
+     * execute the bulk method handler. Regardless if the action is valid or not it will redirect to
+     * the previous page removing the current arguments that makes this request a bulk action.
+     */
+    protected function process_bulk_action() {
+        global $wpdb;
+        // Detect when a bulk action is being triggered.
+        $action = $this->current_action();
+        if ( ! $action ) {
+            return;
+        }
+
+        check_admin_referer( 'bulk-' . $this->_args['plural'] );
+
+        $method = 'bulk_' . $action;
+        if ( array_key_exists( $action, $this->bulk_actions ) && is_callable( array( $this, $method ) ) && ! empty( $_GET['ID'] ) && is_array( $_GET['ID'] ) ) {
+            $ids_sql = '(' . implode( ',', array_fill( 0, count( $_GET['ID'] ), '%s' ) ) . ')';
+            $id      = array_map( 'absint', $_GET['ID'] );
+            $this->$method( $id, $wpdb->prepare( $ids_sql, $id ) ); //phpcs:ignore WordPress.DB.PreparedSQL
+        }
+
+        if ( isset( $_SERVER['REQUEST_URI'] ) ) {
+            wp_safe_redirect(
+                remove_query_arg(
+                    array( '_wp_http_referer', '_wpnonce', 'ID', 'action', 'action2' ),
+                    esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'] ) )
+                )
+            );
+            exit;
+        }
+    }
+
+    /**
+     * Default code for deleting entries.
+     * validated already by process_bulk_action()
+     *
+     * @param array  $ids ids of the items to delete.
+     * @param string $ids_sql the sql for the ids.
+     * @return void
+     */
+    protected function bulk_delete( array $ids, $ids_sql ) {
+        $store = ActionScheduler::store();
+        foreach ( $ids as $action_id ) {
+            $store->delete( $action_id );
+        }
+    }
+
+    /**
+     * Prepares the _column_headers property which is used by WP_Table_List at rendering.
+     * It merges the columns and the sortable columns.
+     */
+    protected function prepare_column_headers() {
+        $this->_column_headers = array(
+            $this->get_columns(),
+            get_hidden_columns( $this->screen ),
+            $this->get_sortable_columns(),
+        );
+    }
+
+    /**
+     * Reads $this->sort_by and returns the columns name in a format that WP_Table_List
+     * expects
+     */
+    public function get_sortable_columns() {
+        $sort_by = array();
+        foreach ( $this->sort_by as $column ) {
+            $sort_by[ $column ] = array( $column, true );
+        }
+        return $sort_by;
+    }
+
+    /**
+     * Returns the columns names for rendering. It adds a checkbox for selecting everything
+     * as the first column
+     */
+    public function get_columns() {
+        $columns = array_merge(
+            array( 'cb' => '<input type="checkbox" />' ),
+            $this->columns
+        );
+
+        return $columns;
+    }
+
+    /**
+     * Get prepared LIMIT clause for items query
+     *
+     * @global wpdb $wpdb
+     *
+     * @return string Prepared LIMIT clause for items query.
+     */
+    protected function get_items_query_limit() {
+        global $wpdb;
+
+        $per_page = $this->get_items_per_page( $this->get_per_page_option_name(), $this->items_per_page );
+        return $wpdb->prepare( 'LIMIT %d', $per_page );
+    }
+
+    /**
+     * Returns the number of items to offset/skip for this current view.
+     *
+     * @return int
+     */
+    protected function get_items_offset() {
+        $per_page     = $this->get_items_per_page( $this->get_per_page_option_name(), $this->items_per_page );
+        $current_page = $this->get_pagenum();
+        if ( 1 < $current_page ) {
+            $offset = $per_page * ( $current_page - 1 );
+        } else {
+            $offset = 0;
+        }
+
+        return $offset;
+    }
+
+    /**
+     * Get prepared OFFSET clause for items query
+     *
+     * @global wpdb $wpdb
+     *
+     * @return string Prepared OFFSET clause for items query.
+     */
+    protected function get_items_query_offset() {
+        global $wpdb;
+
+        return $wpdb->prepare( 'OFFSET %d', $this->get_items_offset() );
+    }
+
+    /**
+     * Prepares the ORDER BY sql statement. It uses `$this->sort_by` to know which
+     * columns are sortable. This requests validates the orderby $_GET parameter is a valid
+     * column and sortable. It will also use order (ASC|DESC) using DESC by default.
+     */
+    protected function get_items_query_order() {
+        if ( empty( $this->sort_by ) ) {
+            return '';
+        }
+
+        $orderby = esc_sql( $this->get_request_orderby() );
+        $order   = esc_sql( $this->get_request_order() );
+
+        return "ORDER BY {$orderby} {$order}";
+    }
+
+    /**
+     * Return the sortable column specified for this request to order the results by, if any.
+     *
+     * @return string
+     */
+    protected function get_request_orderby() {
+
+        $valid_sortable_columns = array_values( $this->sort_by );
+
+        if ( ! empty( $_GET['orderby'] ) && in_array( $_GET['orderby'], $valid_sortable_columns, true ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            $orderby = sanitize_text_field( wp_unslash( $_GET['orderby'] ) ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+        } else {
+            $orderby = $valid_sortable_columns[0];
+        }
+
+        return $orderby;
+    }
+
+    /**
+     * Return the sortable column order specified for this request.
+     *
+     * @return string
+     */
+    protected function get_request_order() {
+
+        if ( ! empty( $_GET['order'] ) && 'desc' === strtolower( sanitize_text_field( wp_unslash( $_GET['order'] ) ) ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            $order = 'DESC';
+        } else {
+            $order = 'ASC';
+        }
+
+        return $order;
+    }
+
+    /**
+     * Return the status filter for this request, if any.
+     *
+     * @return string
+     */
+    protected function get_request_status() {
+        $status = ( ! empty( $_GET['status'] ) ) ? sanitize_text_field( wp_unslash( $_GET['status'] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+        return $status;
+    }
+
+    /**
+     * Return the search filter for this request, if any.
+     *
+     * @return string
+     */
+    protected function get_request_search_query() {
+        $search_query = ( ! empty( $_GET['s'] ) ) ? sanitize_text_field( wp_unslash( $_GET['s'] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+        return $search_query;
+    }
+
+    /**
+     * Process and return the columns name. This is meant for using with SQL, this means it
+     * always includes the primary key.
+     *
+     * @return array
+     */
+    protected function get_table_columns() {
+        $columns = array_keys( $this->columns );
+        if ( ! in_array( $this->ID, $columns, true ) ) {
+            $columns[] = $this->ID;
+        }
+
+        return $columns;
+    }
+
+    /**
+     * Check if the current request is doing a "full text" search. If that is the case
+     * prepares the SQL to search texts using LIKE.
+     *
+     * If the current request does not have any search or if this list table does not support
+     * that feature it will return an empty string.
+     *
+     * @return string
+     */
+    protected function get_items_query_search() {
+        global $wpdb;
+
+        if ( empty( $_GET['s'] ) || empty( $this->search_by ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            return '';
+        }
+
+        $search_string = sanitize_text_field( wp_unslash( $_GET['s'] ) ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+
+        $filter = array();
+        foreach ( $this->search_by as $column ) {
+            $wild     = '%';
+            $sql_like = $wild . $wpdb->esc_like( $search_string ) . $wild;
+            $filter[] = $wpdb->prepare( '`' . $column . '` LIKE %s', $sql_like ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.DB.PreparedSQL.NotPrepared
+        }
+        return implode( ' OR ', $filter );
+    }
+
+    /**
+     * Prepares the SQL to filter rows by the options defined at `$this->filter_by`. Before trusting
+     * any data sent by the user it validates that it is a valid option.
+     */
+    protected function get_items_query_filters() {
+        global $wpdb;
+
+        if ( ! $this->filter_by || empty( $_GET['filter_by'] ) || ! is_array( $_GET['filter_by'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            return '';
+        }
+
+        $filter = array();
+
+        foreach ( $this->filter_by as $column => $options ) {
+            if ( empty( $_GET['filter_by'][ $column ] ) || empty( $options[ $_GET['filter_by'][ $column ] ] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+                continue;
+            }
+
+            $filter[] = $wpdb->prepare( "`$column` = %s", sanitize_text_field( wp_unslash( $_GET['filter_by'][ $column ] ) ) ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.DB.PreparedSQL.InterpolatedNotPrepared
+        }
+
+        return implode( ' AND ', $filter );
+
+    }
+
+    /**
+     * Prepares the data to feed WP_Table_List.
+     *
+     * This has the core for selecting, sorting and filting data. To keep the code simple
+     * its logic is split among many methods (get_items_query_*).
+     *
+     * Beside populating the items this function will also count all the records that matches
+     * the filtering criteria and will do fill the pagination variables.
+     */
+    public function prepare_items() {
+        global $wpdb;
+
+        $this->process_bulk_action();
+
+        $this->process_row_actions();
+
+        if ( ! empty( $_REQUEST['_wp_http_referer'] && ! empty( $_SERVER['REQUEST_URI'] ) ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            // _wp_http_referer is used only on bulk actions, we remove it to keep the $_GET shorter
+            wp_safe_redirect( remove_query_arg( array( '_wp_http_referer', '_wpnonce' ), esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'] ) ) ) );
+            exit;
+        }
+
+        $this->prepare_column_headers();
+
+        $limit   = $this->get_items_query_limit();
+        $offset  = $this->get_items_query_offset();
+        $order   = $this->get_items_query_order();
+        $where   = array_filter(
+            array(
+                $this->get_items_query_search(),
+                $this->get_items_query_filters(),
+            )
+        );
+        $columns = '`' . implode( '`, `', $this->get_table_columns() ) . '`';
+
+        if ( ! empty( $where ) ) {
+            $where = 'WHERE (' . implode( ') AND (', $where ) . ')';
+        } else {
+            $where = '';
+        }
+
+        $sql = "SELECT $columns FROM {$this->table_name} {$where} {$order} {$limit} {$offset}";
+
+        $this->set_items( $wpdb->get_results( $sql, ARRAY_A ) ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+
+        $query_count = "SELECT COUNT({$this->ID}) FROM {$this->table_name} {$where}";
+        $total_items = $wpdb->get_var( $query_count ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+        $per_page    = $this->get_items_per_page( $this->get_per_page_option_name(), $this->items_per_page );
+        $this->set_pagination_args(
+            array(
+                'total_items' => $total_items,
+                'per_page'    => $per_page,
+                'total_pages' => ceil( $total_items / $per_page ),
+            )
+        );
+    }
+
+    /**
+     * Display the table.
+     *
+     * @param string $which The name of the table.
+     */
+    public function extra_tablenav( $which ) {
+        if ( ! $this->filter_by || 'top' !== $which ) {
+            return;
+        }
+
+        echo '<div class="alignleft actions">';
+
+        foreach ( $this->filter_by as $id => $options ) {
+            $default = ! empty( $_GET['filter_by'][ $id ] ) ? sanitize_text_field( wp_unslash( $_GET['filter_by'][ $id ] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            if ( empty( $options[ $default ] ) ) {
+                $default = '';
+            }
+
+            echo '<select name="filter_by[' . esc_attr( $id ) . ']" class="first" id="filter-by-' . esc_attr( $id ) . '">';
+
+            foreach ( $options as $value => $label ) {
+                echo '<option value="' . esc_attr( $value ) . '" ' . esc_html( $value === $default ? 'selected' : '' ) . '>'
+                    . esc_html( $label )
+                . '</option>';
+            }
+
+            echo '</select>';
+        }
+
+        submit_button( esc_html__( 'Filter', 'action-scheduler' ), '', 'filter_action', false, array( 'id' => 'post-query-submit' ) );
+        echo '</div>';
+    }
+
+    /**
+     * Set the data for displaying. It will attempt to unserialize (There is a chance that some columns
+     * are serialized). This can be override in child classes for futher data transformation.
+     *
+     * @param array $items Items array.
+     */
+    protected function set_items( array $items ) {
+        $this->items = array();
+        foreach ( $items as $item ) {
+            $this->items[ $item[ $this->ID ] ] = array_map( 'maybe_unserialize', $item );
+        }
+    }
+
+    /**
+     * Renders the checkbox for each row, this is the first column and it is named ID regardless
+     * of how the primary key is named (to keep the code simpler). The bulk actions will do the proper
+     * name transformation though using `$this->ID`.
+     *
+     * @param array $row The row to render.
+     */
+    public function column_cb( $row ) {
+        return '<input name="ID[]" type="checkbox" value="' . esc_attr( $row[ $this->ID ] ) . '" />';
+    }
+
+    /**
+     * Renders the row-actions.
+     *
+     * This method renders the action menu, it reads the definition from the $row_actions property,
+     * and it checks that the row action method exists before rendering it.
+     *
+     * @param array  $row Row to be rendered.
+     * @param string $column_name Column name.
+     * @return string
+     */
+    protected function maybe_render_actions( $row, $column_name ) {
+        if ( empty( $this->row_actions[ $column_name ] ) ) {
+            return;
+        }
+
+        $row_id = $row[ $this->ID ];
+
+        $actions      = '<div class="row-actions">';
+        $action_count = 0;
+        foreach ( $this->row_actions[ $column_name ] as $action_key => $action ) {
+
+            $action_count++;
+
+            if ( ! method_exists( $this, 'row_action_' . $action_key ) ) {
+                continue;
+            }
+
+            $action_link = ! empty( $action['link'] ) ? $action['link'] : add_query_arg(
+                array(
+                    'row_action' => $action_key,
+                    'row_id'     => $row_id,
+                    'nonce'      => wp_create_nonce( $action_key . '::' . $row_id ),
+                )
+            );
+            $span_class  = ! empty( $action['class'] ) ? $action['class'] : $action_key;
+            $separator   = ( $action_count < count( $this->row_actions[ $column_name ] ) ) ? ' | ' : '';
+
+            $actions .= sprintf( '<span class="%s">', esc_attr( $span_class ) );
+            $actions .= sprintf( '<a href="%1$s" title="%2$s">%3$s</a>', esc_url( $action_link ), esc_attr( $action['desc'] ), esc_html( $action['name'] ) );
+            $actions .= sprintf( '%s</span>', $separator );
+        }
+        $actions .= '</div>';
+        return $actions;
+    }
+
+    /**
+     * Process the bulk actions.
+     *
+     * @return void
+     */
+    protected function process_row_actions() {
+        $parameters = array( 'row_action', 'row_id', 'nonce' );
+        foreach ( $parameters as $parameter ) {
+            if ( empty( $_REQUEST[ $parameter ] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+                return;
+            }
+        }
+
+        $action = sanitize_text_field( wp_unslash( $_REQUEST['row_action'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.Security.ValidatedSanitizedInput.InputNotValidated
+        $row_id = sanitize_text_field( wp_unslash( $_REQUEST['row_id'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.Security.ValidatedSanitizedInput.InputNotValidated
+        $nonce  = sanitize_text_field( wp_unslash( $_REQUEST['nonce'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.Security.ValidatedSanitizedInput.InputNotValidated
+        $method = 'row_action_' . $action; // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+
+        if ( wp_verify_nonce( $nonce, $action . '::' . $row_id ) && method_exists( $this, $method ) ) {
+            $this->$method( sanitize_text_field( wp_unslash( $row_id ) ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+        }
+
+        if ( isset( $_SERVER['REQUEST_URI'] ) ) {
+            wp_safe_redirect(
+                remove_query_arg(
+                    array( 'row_id', 'row_action', 'nonce' ),
+                    esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'] ) )
+                )
+            );
+            exit;
+        }
+    }
+
+    /**
+     * Default column formatting, it will escape everythig for security.
+     *
+     * @param array  $item The item array.
+     * @param string $column_name Column name to display.
+     *
+     * @return string
+     */
+    public function column_default( $item, $column_name ) {
+        $column_html  = esc_html( $item[ $column_name ] );
+        $column_html .= $this->maybe_render_actions( $item, $column_name );
+        return $column_html;
+    }
+
+    /**
+     * Display the table heading and search query, if any
+     */
+    protected function display_header() {
+        echo '<h1 class="wp-heading-inline">' . esc_attr( $this->table_header ) . '</h1>';
+        if ( $this->get_request_search_query() ) {
+            /* translators: %s: search query */
+            echo '<span class="subtitle">' . esc_attr( sprintf( __( 'Search results for "%s"', 'action-scheduler' ), $this->get_request_search_query() ) ) . '</span>';
+        }
+        echo '<hr class="wp-header-end">';
+    }
+
+    /**
+     * Display the table heading and search query, if any
+     */
+    protected function display_admin_notices() {
+        foreach ( $this->admin_notices as $notice ) {
+            echo '<div id="message" class="' . esc_attr( $notice['class'] ) . '">';
+            echo '	<p>' . wp_kses_post( $notice['message'] ) . '</p>';
+            echo '</div>';
+        }
+    }
+
+    /**
+     * Prints the available statuses so the user can click to filter.
+     */
+    protected function display_filter_by_status() {
+
+        $status_list_items = array();
+        $request_status    = $this->get_request_status();
+
+        // Helper to set 'all' filter when not set on status counts passed in.
+        if ( ! isset( $this->status_counts['all'] ) ) {
+            $this->status_counts = array( 'all' => array_sum( $this->status_counts ) ) + $this->status_counts;
+        }
+
+        foreach ( $this->status_counts as $status_name => $count ) {
+
+            if ( 0 === $count ) {
+                continue;
+            }
+
+            if ( $status_name === $request_status || ( empty( $request_status ) && 'all' === $status_name ) ) {
+                $status_list_item = '<li class="%1$s"><a href="%2$s" class="current">%3$s</a> (%4$d)</li>';
+            } else {
+                $status_list_item = '<li class="%1$s"><a href="%2$s">%3$s</a> (%4$d)</li>';
+            }
+
+            $status_filter_url   = ( 'all' === $status_name ) ? remove_query_arg( 'status' ) : add_query_arg( 'status', $status_name );
+            $status_filter_url   = remove_query_arg( array( 'paged', 's' ), $status_filter_url );
+            $status_list_items[] = sprintf( $status_list_item, esc_attr( $status_name ), esc_url( $status_filter_url ), esc_html( ucfirst( $status_name ) ), absint( $count ) );
+        }
+
+        if ( $status_list_items ) {
+            echo '<ul class="subsubsub">';
+            echo implode( " | \n", $status_list_items ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
+            echo '</ul>';
+        }
+    }
+
+    /**
+     * Renders the table list, we override the original class to render the table inside a form
+     * and to render any needed HTML (like the search box). By doing so the callee of a function can simple
+     * forget about any extra HTML.
+     */
+    protected function display_table() {
+        echo '<form id="' . esc_attr( $this->_args['plural'] ) . '-filter" method="get">';
+        foreach ( $_GET as $key => $value ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            if ( '_' === $key[0] || 'paged' === $key || 'ID' === $key ) {
+                continue;
+            }
+            echo '<input type="hidden" name="' . esc_attr( $key ) . '" value="' . esc_attr( $value ) . '" />';
+        }
+        if ( ! empty( $this->search_by ) ) {
+            echo $this->search_box( $this->get_search_box_button_text(), 'plugin' ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
+        }
+        parent::display();
+        echo '</form>';
+    }
+
+    /**
+     * Process any pending actions.
+     */
+    public function process_actions() {
+        $this->process_bulk_action();
+        $this->process_row_actions();
+
+        if ( ! empty( $_REQUEST['_wp_http_referer'] ) && ! empty( $_SERVER['REQUEST_URI'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            // _wp_http_referer is used only on bulk actions, we remove it to keep the $_GET shorter
+            wp_safe_redirect( remove_query_arg( array( '_wp_http_referer', '_wpnonce' ), esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'] ) ) ) );
+            exit;
+        }
+    }
+
+    /**
+     * Render the list table page, including header, notices, status filters and table.
+     */
+    public function display_page() {
+        $this->prepare_items();
+
+        echo '<div class="wrap">';
+        $this->display_header();
+        $this->display_admin_notices();
+        $this->display_filter_by_status();
+        $this->display_table();
+        echo '</div>';
+    }
+
+    /**
+     * Get the text to display in the search box on the list table.
+     */
+    protected function get_search_box_placeholder() {
+        return esc_html__( 'Search', 'action-scheduler' );
+    }
+
+    /**
+     * Gets the screen per_page option name.
+     *
+     * @return string
+     */
+    protected function get_per_page_option_name() {
+        return $this->package . '_items_per_page';
+    }
 }

Spacing +129 added lines, -129 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -141,7 +141,7 @@ 
 	 *
 	 * @deprecated 3.0.0 Use `_x()` instead.
 	 */
-	protected function translate( $text, $context = '' ) {
+	protected function translate($text, $context = '') {
 		return $text;
 	}
 
@@ -157,12 +157,12 @@ 
 	protected function get_bulk_actions() {
 		$actions = array();
 
-		foreach ( $this->bulk_actions as $action => $label ) {
-			if ( ! is_callable( array( $this, 'bulk_' . $action ) ) ) {
-				throw new RuntimeException( "The bulk action $action does not have a callback method" );
+		foreach ($this->bulk_actions as $action => $label) {
+			if ( ! is_callable(array($this, 'bulk_'.$action))) {
+				throw new RuntimeException("The bulk action $action does not have a callback method");
 			}
 
-			$actions[ $action ] = $label;
+			$actions[$action] = $label;
 		}
 
 		return $actions;
@@ -177,24 +177,24 @@ 
 		global $wpdb;
 		// Detect when a bulk action is being triggered.
 		$action = $this->current_action();
-		if ( ! $action ) {
+		if ( ! $action) {
 			return;
 		}
 
-		check_admin_referer( 'bulk-' . $this->_args['plural'] );
+		check_admin_referer('bulk-'.$this->_args['plural']);
 
-		$method = 'bulk_' . $action;
-		if ( array_key_exists( $action, $this->bulk_actions ) && is_callable( array( $this, $method ) ) && ! empty( $_GET['ID'] ) && is_array( $_GET['ID'] ) ) {
-			$ids_sql = '(' . implode( ',', array_fill( 0, count( $_GET['ID'] ), '%s' ) ) . ')';
-			$id      = array_map( 'absint', $_GET['ID'] );
-			$this->$method( $id, $wpdb->prepare( $ids_sql, $id ) ); //phpcs:ignore WordPress.DB.PreparedSQL
+		$method = 'bulk_'.$action;
+		if (array_key_exists($action, $this->bulk_actions) && is_callable(array($this, $method)) && ! empty($_GET['ID']) && is_array($_GET['ID'])) {
+			$ids_sql = '('.implode(',', array_fill(0, count($_GET['ID']), '%s')).')';
+			$id      = array_map('absint', $_GET['ID']);
+			$this->$method($id, $wpdb->prepare($ids_sql, $id)); //phpcs:ignore WordPress.DB.PreparedSQL
 		}
 
-		if ( isset( $_SERVER['REQUEST_URI'] ) ) {
+		if (isset($_SERVER['REQUEST_URI'])) {
 			wp_safe_redirect(
 				remove_query_arg(
-					array( '_wp_http_referer', '_wpnonce', 'ID', 'action', 'action2' ),
-					esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'] ) )
+					array('_wp_http_referer', '_wpnonce', 'ID', 'action', 'action2'),
+					esc_url_raw(wp_unslash($_SERVER['REQUEST_URI']))
 				)
 			);
 			exit;
@@ -209,10 +209,10 @@ 
 	 * @param string $ids_sql the sql for the ids.
 	 * @return void
 	 */
-	protected function bulk_delete( array $ids, $ids_sql ) {
+	protected function bulk_delete(array $ids, $ids_sql) {
 		$store = ActionScheduler::store();
-		foreach ( $ids as $action_id ) {
-			$store->delete( $action_id );
+		foreach ($ids as $action_id) {
+			$store->delete($action_id);
 		}
 	}
 
@@ -223,7 +223,7 @@ 
 	protected function prepare_column_headers() {
 		$this->_column_headers = array(
 			$this->get_columns(),
-			get_hidden_columns( $this->screen ),
+			get_hidden_columns($this->screen),
 			$this->get_sortable_columns(),
 		);
 	}
@@ -234,8 +234,8 @@ 
 	 */
 	public function get_sortable_columns() {
 		$sort_by = array();
-		foreach ( $this->sort_by as $column ) {
-			$sort_by[ $column ] = array( $column, true );
+		foreach ($this->sort_by as $column) {
+			$sort_by[$column] = array($column, true);
 		}
 		return $sort_by;
 	}
@@ -246,7 +246,7 @@ 
 	 */
 	public function get_columns() {
 		$columns = array_merge(
-			array( 'cb' => '<input type="checkbox" />' ),
+			array('cb' => '<input type="checkbox" />'),
 			$this->columns
 		);
 
@@ -263,8 +263,8 @@ 
 	protected function get_items_query_limit() {
 		global $wpdb;
 
-		$per_page = $this->get_items_per_page( $this->get_per_page_option_name(), $this->items_per_page );
-		return $wpdb->prepare( 'LIMIT %d', $per_page );
+		$per_page = $this->get_items_per_page($this->get_per_page_option_name(), $this->items_per_page);
+		return $wpdb->prepare('LIMIT %d', $per_page);
 	}
 
 	/**
@@ -273,10 +273,10 @@ 
 	 * @return int
 	 */
 	protected function get_items_offset() {
-		$per_page     = $this->get_items_per_page( $this->get_per_page_option_name(), $this->items_per_page );
+		$per_page     = $this->get_items_per_page($this->get_per_page_option_name(), $this->items_per_page);
 		$current_page = $this->get_pagenum();
-		if ( 1 < $current_page ) {
-			$offset = $per_page * ( $current_page - 1 );
+		if (1 < $current_page) {
+			$offset = $per_page * ($current_page - 1);
 		} else {
 			$offset = 0;
 		}
@@ -294,7 +294,7 @@ 
 	protected function get_items_query_offset() {
 		global $wpdb;
 
-		return $wpdb->prepare( 'OFFSET %d', $this->get_items_offset() );
+		return $wpdb->prepare('OFFSET %d', $this->get_items_offset());
 	}
 
 	/**
@@ -303,12 +303,12 @@ 
 	 * column and sortable. It will also use order (ASC|DESC) using DESC by default.
 	 */
 	protected function get_items_query_order() {
-		if ( empty( $this->sort_by ) ) {
+		if (empty($this->sort_by)) {
 			return '';
 		}
 
-		$orderby = esc_sql( $this->get_request_orderby() );
-		$order   = esc_sql( $this->get_request_order() );
+		$orderby = esc_sql($this->get_request_orderby());
+		$order   = esc_sql($this->get_request_order());
 
 		return "ORDER BY {$orderby} {$order}";
 	}
@@ -320,10 +320,10 @@ 
 	 */
 	protected function get_request_orderby() {
 
-		$valid_sortable_columns = array_values( $this->sort_by );
+		$valid_sortable_columns = array_values($this->sort_by);
 
-		if ( ! empty( $_GET['orderby'] ) && in_array( $_GET['orderby'], $valid_sortable_columns, true ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			$orderby = sanitize_text_field( wp_unslash( $_GET['orderby'] ) ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		if ( ! empty($_GET['orderby']) && in_array($_GET['orderby'], $valid_sortable_columns, true)) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+			$orderby = sanitize_text_field(wp_unslash($_GET['orderby'])); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 		} else {
 			$orderby = $valid_sortable_columns[0];
 		}
@@ -338,7 +338,7 @@ 
 	 */
 	protected function get_request_order() {
 
-		if ( ! empty( $_GET['order'] ) && 'desc' === strtolower( sanitize_text_field( wp_unslash( $_GET['order'] ) ) ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		if ( ! empty($_GET['order']) && 'desc' === strtolower(sanitize_text_field(wp_unslash($_GET['order'])))) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 			$order = 'DESC';
 		} else {
 			$order = 'ASC';
@@ -353,7 +353,7 @@ 
 	 * @return string
 	 */
 	protected function get_request_status() {
-		$status = ( ! empty( $_GET['status'] ) ) ? sanitize_text_field( wp_unslash( $_GET['status'] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		$status = ( ! empty($_GET['status'])) ? sanitize_text_field(wp_unslash($_GET['status'])) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 		return $status;
 	}
 
@@ -363,7 +363,7 @@ 
 	 * @return string
 	 */
 	protected function get_request_search_query() {
-		$search_query = ( ! empty( $_GET['s'] ) ) ? sanitize_text_field( wp_unslash( $_GET['s'] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		$search_query = ( ! empty($_GET['s'])) ? sanitize_text_field(wp_unslash($_GET['s'])) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 		return $search_query;
 	}
 
@@ -374,8 +374,8 @@ 
 	 * @return array
 	 */
 	protected function get_table_columns() {
-		$columns = array_keys( $this->columns );
-		if ( ! in_array( $this->ID, $columns, true ) ) {
+		$columns = array_keys($this->columns);
+		if ( ! in_array($this->ID, $columns, true)) {
 			$columns[] = $this->ID;
 		}
 
@@ -394,19 +394,19 @@ 
 	protected function get_items_query_search() {
 		global $wpdb;
 
-		if ( empty( $_GET['s'] ) || empty( $this->search_by ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		if (empty($_GET['s']) || empty($this->search_by)) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 			return '';
 		}
 
-		$search_string = sanitize_text_field( wp_unslash( $_GET['s'] ) ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		$search_string = sanitize_text_field(wp_unslash($_GET['s'])); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 
 		$filter = array();
-		foreach ( $this->search_by as $column ) {
+		foreach ($this->search_by as $column) {
 			$wild     = '%';
-			$sql_like = $wild . $wpdb->esc_like( $search_string ) . $wild;
-			$filter[] = $wpdb->prepare( '`' . $column . '` LIKE %s', $sql_like ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.DB.PreparedSQL.NotPrepared
+			$sql_like = $wild.$wpdb->esc_like($search_string).$wild;
+			$filter[] = $wpdb->prepare('`'.$column.'` LIKE %s', $sql_like); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.DB.PreparedSQL.NotPrepared
 		}
-		return implode( ' OR ', $filter );
+		return implode(' OR ', $filter);
 	}
 
 	/**
@@ -416,21 +416,21 @@ 
 	protected function get_items_query_filters() {
 		global $wpdb;
 
-		if ( ! $this->filter_by || empty( $_GET['filter_by'] ) || ! is_array( $_GET['filter_by'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		if ( ! $this->filter_by || empty($_GET['filter_by']) || ! is_array($_GET['filter_by'])) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 			return '';
 		}
 
 		$filter = array();
 
-		foreach ( $this->filter_by as $column => $options ) {
-			if ( empty( $_GET['filter_by'][ $column ] ) || empty( $options[ $_GET['filter_by'][ $column ] ] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		foreach ($this->filter_by as $column => $options) {
+			if (empty($_GET['filter_by'][$column]) || empty($options[$_GET['filter_by'][$column]])) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 				continue;
 			}
 
-			$filter[] = $wpdb->prepare( "`$column` = %s", sanitize_text_field( wp_unslash( $_GET['filter_by'][ $column ] ) ) ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.DB.PreparedSQL.InterpolatedNotPrepared
+			$filter[] = $wpdb->prepare("`$column` = %s", sanitize_text_field(wp_unslash($_GET['filter_by'][$column]))); //phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.DB.PreparedSQL.InterpolatedNotPrepared
 		}
 
-		return implode( ' AND ', $filter );
+		return implode(' AND ', $filter);
 
 	}
 
@@ -450,9 +450,9 @@ 
 
 		$this->process_row_actions();
 
-		if ( ! empty( $_REQUEST['_wp_http_referer'] && ! empty( $_SERVER['REQUEST_URI'] ) ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		if ( ! empty($_REQUEST['_wp_http_referer'] && ! empty($_SERVER['REQUEST_URI']))) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 			// _wp_http_referer is used only on bulk actions, we remove it to keep the $_GET shorter
-			wp_safe_redirect( remove_query_arg( array( '_wp_http_referer', '_wpnonce' ), esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'] ) ) ) );
+			wp_safe_redirect(remove_query_arg(array('_wp_http_referer', '_wpnonce'), esc_url_raw(wp_unslash($_SERVER['REQUEST_URI']))));
 			exit;
 		}
 
@@ -467,26 +467,26 @@ 
 				$this->get_items_query_filters(),
 			)
 		);
-		$columns = '`' . implode( '`, `', $this->get_table_columns() ) . '`';
+		$columns = '`'.implode('`, `', $this->get_table_columns()).'`';
 
-		if ( ! empty( $where ) ) {
-			$where = 'WHERE (' . implode( ') AND (', $where ) . ')';
+		if ( ! empty($where)) {
+			$where = 'WHERE ('.implode(') AND (', $where).')';
 		} else {
 			$where = '';
 		}
 
 		$sql = "SELECT $columns FROM {$this->table_name} {$where} {$order} {$limit} {$offset}";
 
-		$this->set_items( $wpdb->get_results( $sql, ARRAY_A ) ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		$this->set_items($wpdb->get_results($sql, ARRAY_A)); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
 
 		$query_count = "SELECT COUNT({$this->ID}) FROM {$this->table_name} {$where}";
-		$total_items = $wpdb->get_var( $query_count ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
-		$per_page    = $this->get_items_per_page( $this->get_per_page_option_name(), $this->items_per_page );
+		$total_items = $wpdb->get_var($query_count); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		$per_page    = $this->get_items_per_page($this->get_per_page_option_name(), $this->items_per_page);
 		$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),
 			)
 		);
 	}
@@ -496,31 +496,31 @@ 
 	 *
 	 * @param string $which The name of the table.
 	 */
-	public function extra_tablenav( $which ) {
-		if ( ! $this->filter_by || 'top' !== $which ) {
+	public function extra_tablenav($which) {
+		if ( ! $this->filter_by || 'top' !== $which) {
 			return;
 		}
 
 		echo '<div class="alignleft actions">';
 
-		foreach ( $this->filter_by as $id => $options ) {
-			$default = ! empty( $_GET['filter_by'][ $id ] ) ? sanitize_text_field( wp_unslash( $_GET['filter_by'][ $id ] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			if ( empty( $options[ $default ] ) ) {
+		foreach ($this->filter_by as $id => $options) {
+			$default = ! empty($_GET['filter_by'][$id]) ? sanitize_text_field(wp_unslash($_GET['filter_by'][$id])) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+			if (empty($options[$default])) {
 				$default = '';
 			}
 
-			echo '<select name="filter_by[' . esc_attr( $id ) . ']" class="first" id="filter-by-' . esc_attr( $id ) . '">';
+			echo '<select name="filter_by['.esc_attr($id).']" class="first" id="filter-by-'.esc_attr($id).'">';
 
-			foreach ( $options as $value => $label ) {
-				echo '<option value="' . esc_attr( $value ) . '" ' . esc_html( $value === $default ? 'selected' : '' ) . '>'
-					. esc_html( $label )
+			foreach ($options as $value => $label) {
+				echo '<option value="'.esc_attr($value).'" '.esc_html($value === $default ? 'selected' : '').'>'
+					. esc_html($label)
 				. '</option>';
 			}
 
 			echo '</select>';
 		}
 
-		submit_button( esc_html__( 'Filter', 'action-scheduler' ), '', 'filter_action', false, array( 'id' => 'post-query-submit' ) );
+		submit_button(esc_html__('Filter', 'action-scheduler'), '', 'filter_action', false, array('id' => 'post-query-submit'));
 		echo '</div>';
 	}
 
@@ -530,10 +530,10 @@ 
 	 *
 	 * @param array $items Items array.
 	 */
-	protected function set_items( array $items ) {
+	protected function set_items(array $items) {
 		$this->items = array();
-		foreach ( $items as $item ) {
-			$this->items[ $item[ $this->ID ] ] = array_map( 'maybe_unserialize', $item );
+		foreach ($items as $item) {
+			$this->items[$item[$this->ID]] = array_map('maybe_unserialize', $item);
 		}
 	}
 
@@ -544,8 +544,8 @@ 
 	 *
 	 * @param array $row The row to render.
 	 */
-	public function column_cb( $row ) {
-		return '<input name="ID[]" type="checkbox" value="' . esc_attr( $row[ $this->ID ] ) . '" />';
+	public function column_cb($row) {
+		return '<input name="ID[]" type="checkbox" value="'.esc_attr($row[$this->ID]).'" />';
 	}
 
 	/**
@@ -558,36 +558,36 @@ 
 	 * @param string $column_name Column name.
 	 * @return string
 	 */
-	protected function maybe_render_actions( $row, $column_name ) {
-		if ( empty( $this->row_actions[ $column_name ] ) ) {
+	protected function maybe_render_actions($row, $column_name) {
+		if (empty($this->row_actions[$column_name])) {
 			return;
 		}
 
-		$row_id = $row[ $this->ID ];
+		$row_id = $row[$this->ID];
 
 		$actions      = '<div class="row-actions">';
 		$action_count = 0;
-		foreach ( $this->row_actions[ $column_name ] as $action_key => $action ) {
+		foreach ($this->row_actions[$column_name] as $action_key => $action) {
 
 			$action_count++;
 
-			if ( ! method_exists( $this, 'row_action_' . $action_key ) ) {
+			if ( ! method_exists($this, 'row_action_'.$action_key)) {
 				continue;
 			}
 
-			$action_link = ! empty( $action['link'] ) ? $action['link'] : add_query_arg(
+			$action_link = ! empty($action['link']) ? $action['link'] : add_query_arg(
 				array(
 					'row_action' => $action_key,
 					'row_id'     => $row_id,
-					'nonce'      => wp_create_nonce( $action_key . '::' . $row_id ),
+					'nonce'      => wp_create_nonce($action_key.'::'.$row_id),
 				)
 			);
-			$span_class  = ! empty( $action['class'] ) ? $action['class'] : $action_key;
-			$separator   = ( $action_count < count( $this->row_actions[ $column_name ] ) ) ? ' | ' : '';
+			$span_class  = ! empty($action['class']) ? $action['class'] : $action_key;
+			$separator   = ($action_count < count($this->row_actions[$column_name])) ? ' | ' : '';
 
-			$actions .= sprintf( '<span class="%s">', esc_attr( $span_class ) );
-			$actions .= sprintf( '<a href="%1$s" title="%2$s">%3$s</a>', esc_url( $action_link ), esc_attr( $action['desc'] ), esc_html( $action['name'] ) );
-			$actions .= sprintf( '%s</span>', $separator );
+			$actions .= sprintf('<span class="%s">', esc_attr($span_class));
+			$actions .= sprintf('<a href="%1$s" title="%2$s">%3$s</a>', esc_url($action_link), esc_attr($action['desc']), esc_html($action['name']));
+			$actions .= sprintf('%s</span>', $separator);
 		}
 		$actions .= '</div>';
 		return $actions;
@@ -599,27 +599,27 @@ 
 	 * @return void
 	 */
 	protected function process_row_actions() {
-		$parameters = array( 'row_action', 'row_id', 'nonce' );
-		foreach ( $parameters as $parameter ) {
-			if ( empty( $_REQUEST[ $parameter ] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		$parameters = array('row_action', 'row_id', 'nonce');
+		foreach ($parameters as $parameter) {
+			if (empty($_REQUEST[$parameter])) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
 				return;
 			}
 		}
 
-		$action = sanitize_text_field( wp_unslash( $_REQUEST['row_action'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.Security.ValidatedSanitizedInput.InputNotValidated
-		$row_id = sanitize_text_field( wp_unslash( $_REQUEST['row_id'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.Security.ValidatedSanitizedInput.InputNotValidated
-		$nonce  = sanitize_text_field( wp_unslash( $_REQUEST['nonce'] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.Security.ValidatedSanitizedInput.InputNotValidated
-		$method = 'row_action_' . $action; // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		$action = sanitize_text_field(wp_unslash($_REQUEST['row_action'])); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.Security.ValidatedSanitizedInput.InputNotValidated
+		$row_id = sanitize_text_field(wp_unslash($_REQUEST['row_id'])); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.Security.ValidatedSanitizedInput.InputNotValidated
+		$nonce  = sanitize_text_field(wp_unslash($_REQUEST['nonce'])); // phpcs:ignore WordPress.Security.NonceVerification.Recommended, WordPress.Security.ValidatedSanitizedInput.InputNotValidated
+		$method = 'row_action_'.$action; // phpcs:ignore WordPress.Security.NonceVerification.Recommended
 
-		if ( wp_verify_nonce( $nonce, $action . '::' . $row_id ) && method_exists( $this, $method ) ) {
-			$this->$method( sanitize_text_field( wp_unslash( $row_id ) ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		if (wp_verify_nonce($nonce, $action.'::'.$row_id) && method_exists($this, $method)) {
+			$this->$method(sanitize_text_field(wp_unslash($row_id))); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
 		}
 
-		if ( isset( $_SERVER['REQUEST_URI'] ) ) {
+		if (isset($_SERVER['REQUEST_URI'])) {
 			wp_safe_redirect(
 				remove_query_arg(
-					array( 'row_id', 'row_action', 'nonce' ),
-					esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'] ) )
+					array('row_id', 'row_action', 'nonce'),
+					esc_url_raw(wp_unslash($_SERVER['REQUEST_URI']))
 				)
 			);
 			exit;
@@ -634,9 +634,9 @@ 
 	 *
 	 * @return string
 	 */
-	public function column_default( $item, $column_name ) {
-		$column_html  = esc_html( $item[ $column_name ] );
-		$column_html .= $this->maybe_render_actions( $item, $column_name );
+	public function column_default($item, $column_name) {
+		$column_html  = esc_html($item[$column_name]);
+		$column_html .= $this->maybe_render_actions($item, $column_name);
 		return $column_html;
 	}
 
@@ -644,10 +644,10 @@ 
 	 * Display the table heading and search query, if any
 	 */
 	protected function display_header() {
-		echo '<h1 class="wp-heading-inline">' . esc_attr( $this->table_header ) . '</h1>';
-		if ( $this->get_request_search_query() ) {
+		echo '<h1 class="wp-heading-inline">'.esc_attr($this->table_header).'</h1>';
+		if ($this->get_request_search_query()) {
 			/* translators: %s: search query */
-			echo '<span class="subtitle">' . esc_attr( sprintf( __( 'Search results for "%s"', 'action-scheduler' ), $this->get_request_search_query() ) ) . '</span>';
+			echo '<span class="subtitle">'.esc_attr(sprintf(__('Search results for "%s"', 'action-scheduler'), $this->get_request_search_query())).'</span>';
 		}
 		echo '<hr class="wp-header-end">';
 	}
@@ -656,9 +656,9 @@ 
 	 * Display the table heading and search query, if any
 	 */
 	protected function display_admin_notices() {
-		foreach ( $this->admin_notices as $notice ) {
-			echo '<div id="message" class="' . esc_attr( $notice['class'] ) . '">';
-			echo '	<p>' . wp_kses_post( $notice['message'] ) . '</p>';
+		foreach ($this->admin_notices as $notice) {
+			echo '<div id="message" class="'.esc_attr($notice['class']).'">';
+			echo '	<p>'.wp_kses_post($notice['message']).'</p>';
 			echo '</div>';
 		}
 	}
@@ -672,30 +672,30 @@ 
 		$request_status    = $this->get_request_status();
 
 		// Helper to set 'all' filter when not set on status counts passed in.
-		if ( ! isset( $this->status_counts['all'] ) ) {
-			$this->status_counts = array( 'all' => array_sum( $this->status_counts ) ) + $this->status_counts;
+		if ( ! isset($this->status_counts['all'])) {
+			$this->status_counts = array('all' => array_sum($this->status_counts)) + $this->status_counts;
 		}
 
-		foreach ( $this->status_counts as $status_name => $count ) {
+		foreach ($this->status_counts as $status_name => $count) {
 
-			if ( 0 === $count ) {
+			if (0 === $count) {
 				continue;
 			}
 
-			if ( $status_name === $request_status || ( empty( $request_status ) && 'all' === $status_name ) ) {
+			if ($status_name === $request_status || (empty($request_status) && 'all' === $status_name)) {
 				$status_list_item = '<li class="%1$s"><a href="%2$s" class="current">%3$s</a> (%4$d)</li>';
 			} else {
 				$status_list_item = '<li class="%1$s"><a href="%2$s">%3$s</a> (%4$d)</li>';
 			}
 
-			$status_filter_url   = ( 'all' === $status_name ) ? remove_query_arg( 'status' ) : add_query_arg( 'status', $status_name );
-			$status_filter_url   = remove_query_arg( array( 'paged', 's' ), $status_filter_url );
-			$status_list_items[] = sprintf( $status_list_item, esc_attr( $status_name ), esc_url( $status_filter_url ), esc_html( ucfirst( $status_name ) ), absint( $count ) );
+			$status_filter_url   = ('all' === $status_name) ? remove_query_arg('status') : add_query_arg('status', $status_name);
+			$status_filter_url   = remove_query_arg(array('paged', 's'), $status_filter_url);
+			$status_list_items[] = sprintf($status_list_item, esc_attr($status_name), esc_url($status_filter_url), esc_html(ucfirst($status_name)), absint($count));
 		}
 
-		if ( $status_list_items ) {
+		if ($status_list_items) {
 			echo '<ul class="subsubsub">';
-			echo implode( " | \n", $status_list_items ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
+			echo implode(" | \n", $status_list_items); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
 			echo '</ul>';
 		}
 	}
@@ -706,15 +706,15 @@ 
 	 * forget about any extra HTML.
 	 */
 	protected function display_table() {
-		echo '<form id="' . esc_attr( $this->_args['plural'] ) . '-filter" method="get">';
-		foreach ( $_GET as $key => $value ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			if ( '_' === $key[0] || 'paged' === $key || 'ID' === $key ) {
+		echo '<form id="'.esc_attr($this->_args['plural']).'-filter" method="get">';
+		foreach ($_GET as $key => $value) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+			if ('_' === $key[0] || 'paged' === $key || 'ID' === $key) {
 				continue;
 			}
-			echo '<input type="hidden" name="' . esc_attr( $key ) . '" value="' . esc_attr( $value ) . '" />';
+			echo '<input type="hidden" name="'.esc_attr($key).'" value="'.esc_attr($value).'" />';
 		}
-		if ( ! empty( $this->search_by ) ) {
-			echo $this->search_box( $this->get_search_box_button_text(), 'plugin' ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
+		if ( ! empty($this->search_by)) {
+			echo $this->search_box($this->get_search_box_button_text(), 'plugin'); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
 		}
 		parent::display();
 		echo '</form>';
@@ -727,9 +727,9 @@ 
 		$this->process_bulk_action();
 		$this->process_row_actions();
 
-		if ( ! empty( $_REQUEST['_wp_http_referer'] ) && ! empty( $_SERVER['REQUEST_URI'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		if ( ! empty($_REQUEST['_wp_http_referer']) && ! empty($_SERVER['REQUEST_URI'])) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
 			// _wp_http_referer is used only on bulk actions, we remove it to keep the $_GET shorter
-			wp_safe_redirect( remove_query_arg( array( '_wp_http_referer', '_wpnonce' ), esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'] ) ) ) );
+			wp_safe_redirect(remove_query_arg(array('_wp_http_referer', '_wpnonce'), esc_url_raw(wp_unslash($_SERVER['REQUEST_URI']))));
 			exit;
 		}
 	}
@@ -752,7 +752,7 @@ 
 	 * Get the text to display in the search box on the list table.
 	 */
 	protected function get_search_box_placeholder() {
-		return esc_html__( 'Search', 'action-scheduler' );
+		return esc_html__('Search', 'action-scheduler');
 	}
 
 	/**
@@ -761,6 +761,6 @@ 
 	 * @return string
 	 */
 	protected function get_per_page_option_name() {
-		return $this->package . '_items_per_page';
+		return $this->package.'_items_per_page';
 	}
 }

src/libraries/action-scheduler/classes/abstracts/ActionScheduler_Abstract_Schedule.php

Spacing +7 added lines, -7 removed lines

@@ -22,7 +22,7 @@ 
 	/**
 	 * @param DateTime $date The date & time to run the action.
 	 */
-	public function __construct( DateTime $date ) {
+	public function __construct(DateTime $date) {
 		$this->scheduled_date = $date;
 	}
 
@@ -39,7 +39,7 @@ 
 	 * @param DateTime $after
 	 * @return DateTime
 	 */
-	abstract protected function calculate_next( DateTime $after );
+	abstract protected function calculate_next(DateTime $after);
 
 	/**
 	 * Get the next date & time when this schedule should run after a given date & time.
@@ -47,10 +47,10 @@ 
 	 * @param DateTime $after
 	 * @return DateTime|null
 	 */
-	public function get_next( DateTime $after ) {
+	public function get_next(DateTime $after) {
 		$after = clone $after;
-		if ( $after > $this->scheduled_date ) {
-			$after = $this->calculate_next( $after );
+		if ($after > $this->scheduled_date) {
+			$after = $this->calculate_next($after);
 			return $after;
 		}
 		return clone $this->scheduled_date;
@@ -77,7 +77,7 @@ 
 	}
 
 	public function __wakeup() {
-		$this->scheduled_date = as_get_datetime_object( $this->scheduled_timestamp );
-		unset( $this->scheduled_timestamp );
+		$this->scheduled_date = as_get_datetime_object($this->scheduled_timestamp);
+		unset($this->scheduled_timestamp);
 	}
 }

Indentation +68 added lines, -68 removed lines

@@ -5,80 +5,80 @@
  */
 abstract class ActionScheduler_Abstract_Schedule extends ActionScheduler_Schedule_Deprecated {
 
-	/**
-	 * The date & time the schedule is set to run.
-	 *
-	 * @var DateTime
-	 */
-	private $scheduled_date = null;
+    /**
+     * The date & time the schedule is set to run.
+     *
+     * @var DateTime
+     */
+    private $scheduled_date = null;
 
-	/**
-	 * Timestamp equivalent of @see $this->scheduled_date
-	 *
-	 * @var int
-	 */
-	protected $scheduled_timestamp = null;
+    /**
+     * Timestamp equivalent of @see $this->scheduled_date
+     *
+     * @var int
+     */
+    protected $scheduled_timestamp = null;
 
-	/**
-	 * @param DateTime $date The date & time to run the action.
-	 */
-	public function __construct( DateTime $date ) {
-		$this->scheduled_date = $date;
-	}
+    /**
+     * @param DateTime $date The date & time to run the action.
+     */
+    public function __construct( DateTime $date ) {
+        $this->scheduled_date = $date;
+    }
 
-	/**
-	 * Check if a schedule should recur.
-	 *
-	 * @return bool
-	 */
-	abstract public function is_recurring();
+    /**
+     * Check if a schedule should recur.
+     *
+     * @return bool
+     */
+    abstract public function is_recurring();
 
-	/**
-	 * Calculate when the next instance of this schedule would run based on a given date & time.
-	 *
-	 * @param DateTime $after
-	 * @return DateTime
-	 */
-	abstract protected function calculate_next( DateTime $after );
+    /**
+     * Calculate when the next instance of this schedule would run based on a given date & time.
+     *
+     * @param DateTime $after
+     * @return DateTime
+     */
+    abstract protected function calculate_next( DateTime $after );
 
-	/**
-	 * Get the next date & time when this schedule should run after a given date & time.
-	 *
-	 * @param DateTime $after
-	 * @return DateTime|null
-	 */
-	public function get_next( DateTime $after ) {
-		$after = clone $after;
-		if ( $after > $this->scheduled_date ) {
-			$after = $this->calculate_next( $after );
-			return $after;
-		}
-		return clone $this->scheduled_date;
-	}
+    /**
+     * Get the next date & time when this schedule should run after a given date & time.
+     *
+     * @param DateTime $after
+     * @return DateTime|null
+     */
+    public function get_next( DateTime $after ) {
+        $after = clone $after;
+        if ( $after > $this->scheduled_date ) {
+            $after = $this->calculate_next( $after );
+            return $after;
+        }
+        return clone $this->scheduled_date;
+    }
 
-	/**
-	 * Get the date & time the schedule is set to run.
-	 *
-	 * @return DateTime|null
-	 */
-	public function get_date() {
-		return $this->scheduled_date;
-	}
+    /**
+     * Get the date & time the schedule is set to run.
+     *
+     * @return DateTime|null
+     */
+    public function get_date() {
+        return $this->scheduled_date;
+    }
 
-	/**
-	 * For PHP 5.2 compat, since DateTime objects can't be serialized
-	 *
-	 * @return array
-	 */
-	public function __sleep() {
-		$this->scheduled_timestamp = $this->scheduled_date->getTimestamp();
-		return array(
-			'scheduled_timestamp',
-		);
-	}
+    /**
+     * For PHP 5.2 compat, since DateTime objects can't be serialized
+     *
+     * @return array
+     */
+    public function __sleep() {
+        $this->scheduled_timestamp = $this->scheduled_date->getTimestamp();
+        return array(
+            'scheduled_timestamp',
+        );
+    }
 
-	public function __wakeup() {
-		$this->scheduled_date = as_get_datetime_object( $this->scheduled_timestamp );
-		unset( $this->scheduled_timestamp );
-	}
+    public function __wakeup() {
+        $this->scheduled_date = as_get_datetime_object( $this->scheduled_timestamp );
+        unset( $this->scheduled_timestamp );
+    }
 }

src/libraries/action-scheduler/classes/WP_CLI/ActionScheduler_WPCLI_Scheduler_command.php

Indentation +180 added lines, -180 removed lines

@@ -5,184 +5,184 @@
  */
 class ActionScheduler_WPCLI_Scheduler_command extends WP_CLI_Command {
 
-	/**
-	 * Force tables schema creation for Action Scheduler
-	 *
-	 * ## OPTIONS
-	 *
-	 * @param array $args Positional arguments.
-	 * @param array $assoc_args Keyed arguments.
-	 *
-	 * @subcommand fix-schema
-	 */
-	public function fix_schema( $args, $assoc_args ) {
-		$schema_classes = array( ActionScheduler_LoggerSchema::class, ActionScheduler_StoreSchema::class );
-
-		foreach ( $schema_classes as $classname ) {
-			if ( is_subclass_of( $classname, ActionScheduler_Abstract_Schema::class ) ) {
-				$obj = new $classname();
-				$obj->init();
-				$obj->register_tables( true );
-
-				WP_CLI::success(
-					sprintf(
-						/* translators: %s refers to the schema name*/
-						__( 'Registered schema for %s', 'action-scheduler' ),
-						$classname
-					)
-				);
-			}
-		}
-	}
-
-	/**
-	 * Run the Action Scheduler
-	 *
-	 * ## OPTIONS
-	 *
-	 * [--batch-size=<size>]
-	 * : The maximum number of actions to run. Defaults to 100.
-	 *
-	 * [--batches=<size>]
-	 * : Limit execution to a number of batches. Defaults to 0, meaning batches will continue being executed until all actions are complete.
-	 *
-	 * [--cleanup-batch-size=<size>]
-	 * : The maximum number of actions to clean up. Defaults to the value of --batch-size.
-	 *
-	 * [--hooks=<hooks>]
-	 * : Only run actions with the specified hook. Omitting this option runs actions with any hook. Define multiple hooks as a comma separated string (without spaces), e.g. `--hooks=hook_one,hook_two,hook_three`
-	 *
-	 * [--group=<group>]
-	 * : Only run actions from the specified group. Omitting this option runs actions from all groups.
-	 *
-	 * [--free-memory-on=<count>]
-	 * : The number of actions to process between freeing memory. 0 disables freeing memory. Default 50.
-	 *
-	 * [--pause=<seconds>]
-	 * : The number of seconds to pause when freeing memory. Default no pause.
-	 *
-	 * [--force]
-	 * : Whether to force execution despite the maximum number of concurrent processes being exceeded.
-	 *
-	 * @param array $args Positional arguments.
-	 * @param array $assoc_args Keyed arguments.
-	 * @throws \WP_CLI\ExitException When an error occurs.
-	 *
-	 * @subcommand run
-	 */
-	public function run( $args, $assoc_args ) {
-		// Handle passed arguments.
-		$batch   = absint( \WP_CLI\Utils\get_flag_value( $assoc_args, 'batch-size', 100 ) );
-		$batches = absint( \WP_CLI\Utils\get_flag_value( $assoc_args, 'batches', 0 ) );
-		$clean   = absint( \WP_CLI\Utils\get_flag_value( $assoc_args, 'cleanup-batch-size', $batch ) );
-		$hooks   = explode( ',', WP_CLI\Utils\get_flag_value( $assoc_args, 'hooks', '' ) );
-		$hooks   = array_filter( array_map( 'trim', $hooks ) );
-		$group   = \WP_CLI\Utils\get_flag_value( $assoc_args, 'group', '' );
-		$free_on = \WP_CLI\Utils\get_flag_value( $assoc_args, 'free-memory-on', 50 );
-		$sleep   = \WP_CLI\Utils\get_flag_value( $assoc_args, 'pause', 0 );
-		$force   = \WP_CLI\Utils\get_flag_value( $assoc_args, 'force', false );
-
-		ActionScheduler_DataController::set_free_ticks( $free_on );
-		ActionScheduler_DataController::set_sleep_time( $sleep );
-
-		$batches_completed = 0;
-		$actions_completed = 0;
-		$unlimited         = $batches === 0;
-
-		try {
-			// Custom queue cleaner instance.
-			$cleaner = new ActionScheduler_QueueCleaner( null, $clean );
-
-			// Get the queue runner instance
-			$runner = new ActionScheduler_WPCLI_QueueRunner( null, null, $cleaner );
-
-			// Determine how many tasks will be run in the first batch.
-			$total = $runner->setup( $batch, $hooks, $group, $force );
-
-			// Run actions for as long as possible.
-			while ( $total > 0 ) {
-				$this->print_total_actions( $total );
-				$actions_completed += $runner->run();
-				$batches_completed++;
-
-				// Maybe set up tasks for the next batch.
-				$total = ( $unlimited || $batches_completed < $batches ) ? $runner->setup( $batch, $hooks, $group, $force ) : 0;
-			}
-		} catch ( Exception $e ) {
-			$this->print_error( $e );
-		}
-
-		$this->print_total_batches( $batches_completed );
-		$this->print_success( $actions_completed );
-	}
-
-	/**
-	 * Print WP CLI message about how many actions are about to be processed.
-	 *
-	 * @author Jeremy Pry
-	 *
-	 * @param int $total
-	 */
-	protected function print_total_actions( $total ) {
-		WP_CLI::log(
-			sprintf(
-				/* translators: %d refers to how many scheduled taks were found to run */
-				_n( 'Found %d scheduled task', 'Found %d scheduled tasks', $total, 'action-scheduler' ),
-				number_format_i18n( $total )
-			)
-		);
-	}
-
-	/**
-	 * Print WP CLI message about how many batches of actions were processed.
-	 *
-	 * @author Jeremy Pry
-	 *
-	 * @param int $batches_completed
-	 */
-	protected function print_total_batches( $batches_completed ) {
-		WP_CLI::log(
-			sprintf(
-				/* translators: %d refers to the total number of batches executed */
-				_n( '%d batch executed.', '%d batches executed.', $batches_completed, 'action-scheduler' ),
-				number_format_i18n( $batches_completed )
-			)
-		);
-	}
-
-	/**
-	 * Convert an exception into a WP CLI error.
-	 *
-	 * @author Jeremy Pry
-	 *
-	 * @param Exception $e The error object.
-	 *
-	 * @throws \WP_CLI\ExitException
-	 */
-	protected function print_error( Exception $e ) {
-		WP_CLI::error(
-			sprintf(
-				/* translators: %s refers to the exception error message */
-				__( 'There was an error running the action scheduler: %s', 'action-scheduler' ),
-				$e->getMessage()
-			)
-		);
-	}
-
-	/**
-	 * Print a success message with the number of completed actions.
-	 *
-	 * @author Jeremy Pry
-	 *
-	 * @param int $actions_completed
-	 */
-	protected function print_success( $actions_completed ) {
-		WP_CLI::success(
-			sprintf(
-				/* translators: %d refers to the total number of taskes completed */
-				_n( '%d scheduled task completed.', '%d scheduled tasks completed.', $actions_completed, 'action-scheduler' ),
-				number_format_i18n( $actions_completed )
-			)
-		);
-	}
+    /**
+     * Force tables schema creation for Action Scheduler
+     *
+     * ## OPTIONS
+     *
+     * @param array $args Positional arguments.
+     * @param array $assoc_args Keyed arguments.
+     *
+     * @subcommand fix-schema
+     */
+    public function fix_schema( $args, $assoc_args ) {
+        $schema_classes = array( ActionScheduler_LoggerSchema::class, ActionScheduler_StoreSchema::class );
+
+        foreach ( $schema_classes as $classname ) {
+            if ( is_subclass_of( $classname, ActionScheduler_Abstract_Schema::class ) ) {
+                $obj = new $classname();
+                $obj->init();
+                $obj->register_tables( true );
+
+                WP_CLI::success(
+                    sprintf(
+                        /* translators: %s refers to the schema name*/
+                        __( 'Registered schema for %s', 'action-scheduler' ),
+                        $classname
+                    )
+                );
+            }
+        }
+    }
+
+    /**
+     * Run the Action Scheduler
+     *
+     * ## OPTIONS
+     *
+     * [--batch-size=<size>]
+     * : The maximum number of actions to run. Defaults to 100.
+     *
+     * [--batches=<size>]
+     * : Limit execution to a number of batches. Defaults to 0, meaning batches will continue being executed until all actions are complete.
+     *
+     * [--cleanup-batch-size=<size>]
+     * : The maximum number of actions to clean up. Defaults to the value of --batch-size.
+     *
+     * [--hooks=<hooks>]
+     * : Only run actions with the specified hook. Omitting this option runs actions with any hook. Define multiple hooks as a comma separated string (without spaces), e.g. `--hooks=hook_one,hook_two,hook_three`
+     *
+     * [--group=<group>]
+     * : Only run actions from the specified group. Omitting this option runs actions from all groups.
+     *
+     * [--free-memory-on=<count>]
+     * : The number of actions to process between freeing memory. 0 disables freeing memory. Default 50.
+     *
+     * [--pause=<seconds>]
+     * : The number of seconds to pause when freeing memory. Default no pause.
+     *
+     * [--force]
+     * : Whether to force execution despite the maximum number of concurrent processes being exceeded.
+     *
+     * @param array $args Positional arguments.
+     * @param array $assoc_args Keyed arguments.
+     * @throws \WP_CLI\ExitException When an error occurs.
+     *
+     * @subcommand run
+     */
+    public function run( $args, $assoc_args ) {
+        // Handle passed arguments.
+        $batch   = absint( \WP_CLI\Utils\get_flag_value( $assoc_args, 'batch-size', 100 ) );
+        $batches = absint( \WP_CLI\Utils\get_flag_value( $assoc_args, 'batches', 0 ) );
+        $clean   = absint( \WP_CLI\Utils\get_flag_value( $assoc_args, 'cleanup-batch-size', $batch ) );
+        $hooks   = explode( ',', WP_CLI\Utils\get_flag_value( $assoc_args, 'hooks', '' ) );
+        $hooks   = array_filter( array_map( 'trim', $hooks ) );
+        $group   = \WP_CLI\Utils\get_flag_value( $assoc_args, 'group', '' );
+        $free_on = \WP_CLI\Utils\get_flag_value( $assoc_args, 'free-memory-on', 50 );
+        $sleep   = \WP_CLI\Utils\get_flag_value( $assoc_args, 'pause', 0 );
+        $force   = \WP_CLI\Utils\get_flag_value( $assoc_args, 'force', false );
+
+        ActionScheduler_DataController::set_free_ticks( $free_on );
+        ActionScheduler_DataController::set_sleep_time( $sleep );
+
+        $batches_completed = 0;
+        $actions_completed = 0;
+        $unlimited         = $batches === 0;
+
+        try {
+            // Custom queue cleaner instance.
+            $cleaner = new ActionScheduler_QueueCleaner( null, $clean );
+
+            // Get the queue runner instance
+            $runner = new ActionScheduler_WPCLI_QueueRunner( null, null, $cleaner );
+
+            // Determine how many tasks will be run in the first batch.
+            $total = $runner->setup( $batch, $hooks, $group, $force );
+
+            // Run actions for as long as possible.
+            while ( $total > 0 ) {
+                $this->print_total_actions( $total );
+                $actions_completed += $runner->run();
+                $batches_completed++;
+
+                // Maybe set up tasks for the next batch.
+                $total = ( $unlimited || $batches_completed < $batches ) ? $runner->setup( $batch, $hooks, $group, $force ) : 0;
+            }
+        } catch ( Exception $e ) {
+            $this->print_error( $e );
+        }
+
+        $this->print_total_batches( $batches_completed );
+        $this->print_success( $actions_completed );
+    }
+
+    /**
+     * Print WP CLI message about how many actions are about to be processed.
+     *
+     * @author Jeremy Pry
+     *
+     * @param int $total
+     */
+    protected function print_total_actions( $total ) {
+        WP_CLI::log(
+            sprintf(
+                /* translators: %d refers to how many scheduled taks were found to run */
+                _n( 'Found %d scheduled task', 'Found %d scheduled tasks', $total, 'action-scheduler' ),
+                number_format_i18n( $total )
+            )
+        );
+    }
+
+    /**
+     * Print WP CLI message about how many batches of actions were processed.
+     *
+     * @author Jeremy Pry
+     *
+     * @param int $batches_completed
+     */
+    protected function print_total_batches( $batches_completed ) {
+        WP_CLI::log(
+            sprintf(
+                /* translators: %d refers to the total number of batches executed */
+                _n( '%d batch executed.', '%d batches executed.', $batches_completed, 'action-scheduler' ),
+                number_format_i18n( $batches_completed )
+            )
+        );
+    }
+
+    /**
+     * Convert an exception into a WP CLI error.
+     *
+     * @author Jeremy Pry
+     *
+     * @param Exception $e The error object.
+     *
+     * @throws \WP_CLI\ExitException
+     */
+    protected function print_error( Exception $e ) {
+        WP_CLI::error(
+            sprintf(
+                /* translators: %s refers to the exception error message */
+                __( 'There was an error running the action scheduler: %s', 'action-scheduler' ),
+                $e->getMessage()
+            )
+        );
+    }
+
+    /**
+     * Print a success message with the number of completed actions.
+     *
+     * @author Jeremy Pry
+     *
+     * @param int $actions_completed
+     */
+    protected function print_success( $actions_completed ) {
+        WP_CLI::success(
+            sprintf(
+                /* translators: %d refers to the total number of taskes completed */
+                _n( '%d scheduled task completed.', '%d scheduled tasks completed.', $actions_completed, 'action-scheduler' ),
+                number_format_i18n( $actions_completed )
+            )
+        );
+    }
 }

Spacing +40 added lines, -40 removed lines

@@ -15,19 +15,19 @@ 
 	 *
 	 * @subcommand fix-schema
 	 */
-	public function fix_schema( $args, $assoc_args ) {
-		$schema_classes = array( ActionScheduler_LoggerSchema::class, ActionScheduler_StoreSchema::class );
+	public function fix_schema($args, $assoc_args) {
+		$schema_classes = array(ActionScheduler_LoggerSchema::class, ActionScheduler_StoreSchema::class);
 
-		foreach ( $schema_classes as $classname ) {
-			if ( is_subclass_of( $classname, ActionScheduler_Abstract_Schema::class ) ) {
+		foreach ($schema_classes as $classname) {
+			if (is_subclass_of($classname, ActionScheduler_Abstract_Schema::class)) {
 				$obj = new $classname();
 				$obj->init();
-				$obj->register_tables( true );
+				$obj->register_tables(true);
 
 				WP_CLI::success(
 					sprintf(
 						/* translators: %s refers to the schema name*/
-						__( 'Registered schema for %s', 'action-scheduler' ),
+						__('Registered schema for %s', 'action-scheduler'),
 						$classname
 					)
 				);
@@ -70,20 +70,20 @@ 
 	 *
 	 * @subcommand run
 	 */
-	public function run( $args, $assoc_args ) {
+	public function run($args, $assoc_args) {
 		// Handle passed arguments.
-		$batch   = absint( \WP_CLI\Utils\get_flag_value( $assoc_args, 'batch-size', 100 ) );
-		$batches = absint( \WP_CLI\Utils\get_flag_value( $assoc_args, 'batches', 0 ) );
-		$clean   = absint( \WP_CLI\Utils\get_flag_value( $assoc_args, 'cleanup-batch-size', $batch ) );
-		$hooks   = explode( ',', WP_CLI\Utils\get_flag_value( $assoc_args, 'hooks', '' ) );
-		$hooks   = array_filter( array_map( 'trim', $hooks ) );
-		$group   = \WP_CLI\Utils\get_flag_value( $assoc_args, 'group', '' );
-		$free_on = \WP_CLI\Utils\get_flag_value( $assoc_args, 'free-memory-on', 50 );
-		$sleep   = \WP_CLI\Utils\get_flag_value( $assoc_args, 'pause', 0 );
-		$force   = \WP_CLI\Utils\get_flag_value( $assoc_args, 'force', false );
-
-		ActionScheduler_DataController::set_free_ticks( $free_on );
-		ActionScheduler_DataController::set_sleep_time( $sleep );
+		$batch   = absint(\WP_CLI\Utils\get_flag_value($assoc_args, 'batch-size', 100));
+		$batches = absint(\WP_CLI\Utils\get_flag_value($assoc_args, 'batches', 0));
+		$clean   = absint(\WP_CLI\Utils\get_flag_value($assoc_args, 'cleanup-batch-size', $batch));
+		$hooks   = explode(',', WP_CLI\Utils\get_flag_value($assoc_args, 'hooks', ''));
+		$hooks   = array_filter(array_map('trim', $hooks));
+		$group   = \WP_CLI\Utils\get_flag_value($assoc_args, 'group', '');
+		$free_on = \WP_CLI\Utils\get_flag_value($assoc_args, 'free-memory-on', 50);
+		$sleep   = \WP_CLI\Utils\get_flag_value($assoc_args, 'pause', 0);
+		$force   = \WP_CLI\Utils\get_flag_value($assoc_args, 'force', false);
+
+		ActionScheduler_DataController::set_free_ticks($free_on);
+		ActionScheduler_DataController::set_sleep_time($sleep);
 
 		$batches_completed = 0;
 		$actions_completed = 0;
@@ -91,29 +91,29 @@ 
 
 		try {
 			// Custom queue cleaner instance.
-			$cleaner = new ActionScheduler_QueueCleaner( null, $clean );
+			$cleaner = new ActionScheduler_QueueCleaner(null, $clean);
 
 			// Get the queue runner instance
-			$runner = new ActionScheduler_WPCLI_QueueRunner( null, null, $cleaner );
+			$runner = new ActionScheduler_WPCLI_QueueRunner(null, null, $cleaner);
 
 			// Determine how many tasks will be run in the first batch.
-			$total = $runner->setup( $batch, $hooks, $group, $force );
+			$total = $runner->setup($batch, $hooks, $group, $force);
 
 			// Run actions for as long as possible.
-			while ( $total > 0 ) {
-				$this->print_total_actions( $total );
+			while ($total > 0) {
+				$this->print_total_actions($total);
 				$actions_completed += $runner->run();
 				$batches_completed++;
 
 				// Maybe set up tasks for the next batch.
-				$total = ( $unlimited || $batches_completed < $batches ) ? $runner->setup( $batch, $hooks, $group, $force ) : 0;
+				$total = ($unlimited || $batches_completed < $batches) ? $runner->setup($batch, $hooks, $group, $force) : 0;
 			}
-		} catch ( Exception $e ) {
-			$this->print_error( $e );
+		} catch (Exception $e) {
+			$this->print_error($e);
 		}
 
-		$this->print_total_batches( $batches_completed );
-		$this->print_success( $actions_completed );
+		$this->print_total_batches($batches_completed);
+		$this->print_success($actions_completed);
 	}
 
 	/**
@@ -123,12 +123,12 @@ 
 	 *
 	 * @param int $total
 	 */
-	protected function print_total_actions( $total ) {
+	protected function print_total_actions($total) {
 		WP_CLI::log(
 			sprintf(
 				/* translators: %d refers to how many scheduled taks were found to run */
-				_n( 'Found %d scheduled task', 'Found %d scheduled tasks', $total, 'action-scheduler' ),
-				number_format_i18n( $total )
+				_n('Found %d scheduled task', 'Found %d scheduled tasks', $total, 'action-scheduler'),
+				number_format_i18n($total)
 			)
 		);
 	}
@@ -140,12 +140,12 @@ 
 	 *
 	 * @param int $batches_completed
 	 */
-	protected function print_total_batches( $batches_completed ) {
+	protected function print_total_batches($batches_completed) {
 		WP_CLI::log(
 			sprintf(
 				/* translators: %d refers to the total number of batches executed */
-				_n( '%d batch executed.', '%d batches executed.', $batches_completed, 'action-scheduler' ),
-				number_format_i18n( $batches_completed )
+				_n('%d batch executed.', '%d batches executed.', $batches_completed, 'action-scheduler'),
+				number_format_i18n($batches_completed)
 			)
 		);
 	}
@@ -159,11 +159,11 @@ 
 	 *
 	 * @throws \WP_CLI\ExitException
 	 */
-	protected function print_error( Exception $e ) {
+	protected function print_error(Exception $e) {
 		WP_CLI::error(
 			sprintf(
 				/* translators: %s refers to the exception error message */
-				__( 'There was an error running the action scheduler: %s', 'action-scheduler' ),
+				__('There was an error running the action scheduler: %s', 'action-scheduler'),
 				$e->getMessage()
 			)
 		);
@@ -176,12 +176,12 @@ 
 	 *
 	 * @param int $actions_completed
 	 */
-	protected function print_success( $actions_completed ) {
+	protected function print_success($actions_completed) {
 		WP_CLI::success(
 			sprintf(
 				/* translators: %d refers to the total number of taskes completed */
-				_n( '%d scheduled task completed.', '%d scheduled tasks completed.', $actions_completed, 'action-scheduler' ),
-				number_format_i18n( $actions_completed )
+				_n('%d scheduled task completed.', '%d scheduled tasks completed.', $actions_completed, 'action-scheduler'),
+				number_format_i18n($actions_completed)
 			)
 		);
 	}