insideout10 / wordlift-plugin

src/ext/action-scheduler/classes/data-stores/ActionScheduler_wpPostStore_TaxonomyRegistrar.php

Indentation +15 added lines, -15 removed lines

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

Spacing +3 added lines, -3 removed lines

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

src/ext/action-scheduler/classes/ActionScheduler_FatalErrorMonitor.php

Indentation +42 added lines, -42 removed lines

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

Spacing +19 added lines, -19 removed lines

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

src/ext/action-scheduler/classes/ActionScheduler_QueueCleaner.php

Indentation +156 added lines, -156 removed lines

@@ -5,160 +5,160 @@
  */
 class ActionScheduler_QueueCleaner {
 
-	/** @var int */
-	protected $batch_size;
-
-	/** @var ActionScheduler_Store */
-	private $store = null;
-
-	/**
-	 * 31 days in seconds.
-	 *
-	 * @var int
-	 */
-	private $month_in_seconds = 2678400;
-
-	/**
-	 * ActionScheduler_QueueCleaner constructor.
-	 *
-	 * @param ActionScheduler_Store $store      The store instance.
-	 * @param int                   $batch_size The batch size.
-	 */
-	public function __construct( ActionScheduler_Store $store = null, $batch_size = 20 ) {
-		$this->store      = $store ? $store : ActionScheduler_Store::instance();
-		$this->batch_size = $batch_size;
-	}
-
-	public function delete_old_actions() {
-		$lifespan = apply_filters( 'action_scheduler_retention_period', $this->month_in_seconds );
-		$cutoff   = as_get_datetime_object( $lifespan . ' seconds ago' );
-
-		$statuses_to_purge = array(
-			ActionScheduler_Store::STATUS_COMPLETE,
-			ActionScheduler_Store::STATUS_CANCELED,
-		);
-
-		foreach ( $statuses_to_purge as $status ) {
-			$actions_to_delete = $this->store->query_actions(
-				array(
-					'status'           => $status,
-					'modified'         => $cutoff,
-					'modified_compare' => '<=',
-					'per_page'         => $this->get_batch_size(),
-					'orderby'          => 'none',
-				)
-			);
-
-			foreach ( $actions_to_delete as $action_id ) {
-				try {
-					$this->store->delete_action( $action_id );
-				} catch ( Exception $e ) {
-
-					/**
-					 * Notify 3rd party code of exceptions when deleting a completed action older than the retention period
-					 *
-					 * This hook provides a way for 3rd party code to log or otherwise handle exceptions relating to their
-					 * actions.
-					 *
-					 * @since 2.0.0
-					 *
-					 * @param int $action_id The scheduled actions ID in the data store
-					 * @param Exception $e The exception thrown when attempting to delete the action from the data store
-					 * @param int $lifespan The retention period, in seconds, for old actions
-					 * @param int $count_of_actions_to_delete The number of old actions being deleted in this batch
-					 */
-					do_action( 'action_scheduler_failed_old_action_deletion', $action_id, $e, $lifespan, count( $actions_to_delete ) );
-				}
-			}
-		}
-	}
-
-	/**
-	 * Unclaim pending actions that have not been run within a given time limit.
-	 *
-	 * When called by ActionScheduler_Abstract_QueueRunner::run_cleanup(), the time limit passed
-	 * as a parameter is 10x the time limit used for queue processing.
-	 *
-	 * @param int $time_limit The number of seconds to allow a queue to run before unclaiming its pending actions. Default 300 (5 minutes).
-	 */
-	public function reset_timeouts( $time_limit = 300 ) {
-		$timeout = apply_filters( 'action_scheduler_timeout_period', $time_limit );
-		if ( $timeout < 0 ) {
-			return;
-		}
-		$cutoff           = as_get_datetime_object( $timeout . ' seconds ago' );
-		$actions_to_reset = $this->store->query_actions(
-			array(
-				'status'           => ActionScheduler_Store::STATUS_PENDING,
-				'modified'         => $cutoff,
-				'modified_compare' => '<=',
-				'claimed'          => true,
-				'per_page'         => $this->get_batch_size(),
-				'orderby'          => 'none',
-			)
-		);
-
-		foreach ( $actions_to_reset as $action_id ) {
-			$this->store->unclaim_action( $action_id );
-			do_action( 'action_scheduler_reset_action', $action_id );
-		}
-	}
-
-	/**
-	 * Mark actions that have been running for more than a given time limit as failed, based on
-	 * the assumption some uncatachable and unloggable fatal error occurred during processing.
-	 *
-	 * When called by ActionScheduler_Abstract_QueueRunner::run_cleanup(), the time limit passed
-	 * as a parameter is 10x the time limit used for queue processing.
-	 *
-	 * @param int $time_limit The number of seconds to allow an action to run before it is considered to have failed. Default 300 (5 minutes).
-	 */
-	public function mark_failures( $time_limit = 300 ) {
-		$timeout = apply_filters( 'action_scheduler_failure_period', $time_limit );
-		if ( $timeout < 0 ) {
-			return;
-		}
-		$cutoff           = as_get_datetime_object( $timeout . ' seconds ago' );
-		$actions_to_reset = $this->store->query_actions(
-			array(
-				'status'           => ActionScheduler_Store::STATUS_RUNNING,
-				'modified'         => $cutoff,
-				'modified_compare' => '<=',
-				'per_page'         => $this->get_batch_size(),
-				'orderby'          => 'none',
-			)
-		);
-
-		foreach ( $actions_to_reset as $action_id ) {
-			$this->store->mark_failure( $action_id );
-			do_action( 'action_scheduler_failed_action', $action_id, $timeout );
-		}
-	}
-
-	/**
-	 * Do all of the cleaning actions.
-	 *
-	 * @param int $time_limit The number of seconds to use as the timeout and failure period. Default 300 (5 minutes).
-	 * @author Jeremy Pry
-	 */
-	public function clean( $time_limit = 300 ) {
-		$this->delete_old_actions();
-		$this->reset_timeouts( $time_limit );
-		$this->mark_failures( $time_limit );
-	}
-
-	/**
-	 * Get the batch size for cleaning the queue.
-	 *
-	 * @author Jeremy Pry
-	 * @return int
-	 */
-	protected function get_batch_size() {
-		/**
-		 * Filter the batch size when cleaning the queue.
-		 *
-		 * @param int $batch_size The number of actions to clean in one batch.
-		 */
-		return absint( apply_filters( 'action_scheduler_cleanup_batch_size', $this->batch_size ) );
-	}
+    /** @var int */
+    protected $batch_size;
+
+    /** @var ActionScheduler_Store */
+    private $store = null;
+
+    /**
+     * 31 days in seconds.
+     *
+     * @var int
+     */
+    private $month_in_seconds = 2678400;
+
+    /**
+     * ActionScheduler_QueueCleaner constructor.
+     *
+     * @param ActionScheduler_Store $store      The store instance.
+     * @param int                   $batch_size The batch size.
+     */
+    public function __construct( ActionScheduler_Store $store = null, $batch_size = 20 ) {
+        $this->store      = $store ? $store : ActionScheduler_Store::instance();
+        $this->batch_size = $batch_size;
+    }
+
+    public function delete_old_actions() {
+        $lifespan = apply_filters( 'action_scheduler_retention_period', $this->month_in_seconds );
+        $cutoff   = as_get_datetime_object( $lifespan . ' seconds ago' );
+
+        $statuses_to_purge = array(
+            ActionScheduler_Store::STATUS_COMPLETE,
+            ActionScheduler_Store::STATUS_CANCELED,
+        );
+
+        foreach ( $statuses_to_purge as $status ) {
+            $actions_to_delete = $this->store->query_actions(
+                array(
+                    'status'           => $status,
+                    'modified'         => $cutoff,
+                    'modified_compare' => '<=',
+                    'per_page'         => $this->get_batch_size(),
+                    'orderby'          => 'none',
+                )
+            );
+
+            foreach ( $actions_to_delete as $action_id ) {
+                try {
+                    $this->store->delete_action( $action_id );
+                } catch ( Exception $e ) {
+
+                    /**
+                     * Notify 3rd party code of exceptions when deleting a completed action older than the retention period
+                     *
+                     * This hook provides a way for 3rd party code to log or otherwise handle exceptions relating to their
+                     * actions.
+                     *
+                     * @since 2.0.0
+                     *
+                     * @param int $action_id The scheduled actions ID in the data store
+                     * @param Exception $e The exception thrown when attempting to delete the action from the data store
+                     * @param int $lifespan The retention period, in seconds, for old actions
+                     * @param int $count_of_actions_to_delete The number of old actions being deleted in this batch
+                     */
+                    do_action( 'action_scheduler_failed_old_action_deletion', $action_id, $e, $lifespan, count( $actions_to_delete ) );
+                }
+            }
+        }
+    }
+
+    /**
+     * Unclaim pending actions that have not been run within a given time limit.
+     *
+     * When called by ActionScheduler_Abstract_QueueRunner::run_cleanup(), the time limit passed
+     * as a parameter is 10x the time limit used for queue processing.
+     *
+     * @param int $time_limit The number of seconds to allow a queue to run before unclaiming its pending actions. Default 300 (5 minutes).
+     */
+    public function reset_timeouts( $time_limit = 300 ) {
+        $timeout = apply_filters( 'action_scheduler_timeout_period', $time_limit );
+        if ( $timeout < 0 ) {
+            return;
+        }
+        $cutoff           = as_get_datetime_object( $timeout . ' seconds ago' );
+        $actions_to_reset = $this->store->query_actions(
+            array(
+                'status'           => ActionScheduler_Store::STATUS_PENDING,
+                'modified'         => $cutoff,
+                'modified_compare' => '<=',
+                'claimed'          => true,
+                'per_page'         => $this->get_batch_size(),
+                'orderby'          => 'none',
+            )
+        );
+
+        foreach ( $actions_to_reset as $action_id ) {
+            $this->store->unclaim_action( $action_id );
+            do_action( 'action_scheduler_reset_action', $action_id );
+        }
+    }
+
+    /**
+     * Mark actions that have been running for more than a given time limit as failed, based on
+     * the assumption some uncatachable and unloggable fatal error occurred during processing.
+     *
+     * When called by ActionScheduler_Abstract_QueueRunner::run_cleanup(), the time limit passed
+     * as a parameter is 10x the time limit used for queue processing.
+     *
+     * @param int $time_limit The number of seconds to allow an action to run before it is considered to have failed. Default 300 (5 minutes).
+     */
+    public function mark_failures( $time_limit = 300 ) {
+        $timeout = apply_filters( 'action_scheduler_failure_period', $time_limit );
+        if ( $timeout < 0 ) {
+            return;
+        }
+        $cutoff           = as_get_datetime_object( $timeout . ' seconds ago' );
+        $actions_to_reset = $this->store->query_actions(
+            array(
+                'status'           => ActionScheduler_Store::STATUS_RUNNING,
+                'modified'         => $cutoff,
+                'modified_compare' => '<=',
+                'per_page'         => $this->get_batch_size(),
+                'orderby'          => 'none',
+            )
+        );
+
+        foreach ( $actions_to_reset as $action_id ) {
+            $this->store->mark_failure( $action_id );
+            do_action( 'action_scheduler_failed_action', $action_id, $timeout );
+        }
+    }
+
+    /**
+     * Do all of the cleaning actions.
+     *
+     * @param int $time_limit The number of seconds to use as the timeout and failure period. Default 300 (5 minutes).
+     * @author Jeremy Pry
+     */
+    public function clean( $time_limit = 300 ) {
+        $this->delete_old_actions();
+        $this->reset_timeouts( $time_limit );
+        $this->mark_failures( $time_limit );
+    }
+
+    /**
+     * Get the batch size for cleaning the queue.
+     *
+     * @author Jeremy Pry
+     * @return int
+     */
+    protected function get_batch_size() {
+        /**
+         * Filter the batch size when cleaning the queue.
+         *
+         * @param int $batch_size The number of actions to clean in one batch.
+         */
+        return absint( apply_filters( 'action_scheduler_cleanup_batch_size', $this->batch_size ) );
+    }
 }

Spacing +26 added lines, -26 removed lines

@@ -24,21 +24,21 @@ 
 	 * @param ActionScheduler_Store $store      The store instance.
 	 * @param int                   $batch_size The batch size.
 	 */
-	public function __construct( ActionScheduler_Store $store = null, $batch_size = 20 ) {
+	public function __construct(ActionScheduler_Store $store = null, $batch_size = 20) {
 		$this->store      = $store ? $store : ActionScheduler_Store::instance();
 		$this->batch_size = $batch_size;
 	}
 
 	public function delete_old_actions() {
-		$lifespan = apply_filters( 'action_scheduler_retention_period', $this->month_in_seconds );
-		$cutoff   = as_get_datetime_object( $lifespan . ' seconds ago' );
+		$lifespan = apply_filters('action_scheduler_retention_period', $this->month_in_seconds);
+		$cutoff   = as_get_datetime_object($lifespan.' seconds ago');
 
 		$statuses_to_purge = array(
 			ActionScheduler_Store::STATUS_COMPLETE,
 			ActionScheduler_Store::STATUS_CANCELED,
 		);
 
-		foreach ( $statuses_to_purge as $status ) {
+		foreach ($statuses_to_purge as $status) {
 			$actions_to_delete = $this->store->query_actions(
 				array(
 					'status'           => $status,
@@ -49,10 +49,10 @@ 
 				)
 			);
 
-			foreach ( $actions_to_delete as $action_id ) {
+			foreach ($actions_to_delete as $action_id) {
 				try {
-					$this->store->delete_action( $action_id );
-				} catch ( Exception $e ) {
+					$this->store->delete_action($action_id);
+				} catch (Exception $e) {
 
 					/**
 					 * Notify 3rd party code of exceptions when deleting a completed action older than the retention period
@@ -67,7 +67,7 @@ 
 					 * @param int $lifespan The retention period, in seconds, for old actions
 					 * @param int $count_of_actions_to_delete The number of old actions being deleted in this batch
 					 */
-					do_action( 'action_scheduler_failed_old_action_deletion', $action_id, $e, $lifespan, count( $actions_to_delete ) );
+					do_action('action_scheduler_failed_old_action_deletion', $action_id, $e, $lifespan, count($actions_to_delete));
 				}
 			}
 		}
@@ -81,12 +81,12 @@ 
 	 *
 	 * @param int $time_limit The number of seconds to allow a queue to run before unclaiming its pending actions. Default 300 (5 minutes).
 	 */
-	public function reset_timeouts( $time_limit = 300 ) {
-		$timeout = apply_filters( 'action_scheduler_timeout_period', $time_limit );
-		if ( $timeout < 0 ) {
+	public function reset_timeouts($time_limit = 300) {
+		$timeout = apply_filters('action_scheduler_timeout_period', $time_limit);
+		if ($timeout < 0) {
 			return;
 		}
-		$cutoff           = as_get_datetime_object( $timeout . ' seconds ago' );
+		$cutoff           = as_get_datetime_object($timeout.' seconds ago');
 		$actions_to_reset = $this->store->query_actions(
 			array(
 				'status'           => ActionScheduler_Store::STATUS_PENDING,
@@ -98,9 +98,9 @@ 
 			)
 		);
 
-		foreach ( $actions_to_reset as $action_id ) {
-			$this->store->unclaim_action( $action_id );
-			do_action( 'action_scheduler_reset_action', $action_id );
+		foreach ($actions_to_reset as $action_id) {
+			$this->store->unclaim_action($action_id);
+			do_action('action_scheduler_reset_action', $action_id);
 		}
 	}
 
@@ -113,12 +113,12 @@ 
 	 *
 	 * @param int $time_limit The number of seconds to allow an action to run before it is considered to have failed. Default 300 (5 minutes).
 	 */
-	public function mark_failures( $time_limit = 300 ) {
-		$timeout = apply_filters( 'action_scheduler_failure_period', $time_limit );
-		if ( $timeout < 0 ) {
+	public function mark_failures($time_limit = 300) {
+		$timeout = apply_filters('action_scheduler_failure_period', $time_limit);
+		if ($timeout < 0) {
 			return;
 		}
-		$cutoff           = as_get_datetime_object( $timeout . ' seconds ago' );
+		$cutoff           = as_get_datetime_object($timeout.' seconds ago');
 		$actions_to_reset = $this->store->query_actions(
 			array(
 				'status'           => ActionScheduler_Store::STATUS_RUNNING,
@@ -129,9 +129,9 @@ 
 			)
 		);
 
-		foreach ( $actions_to_reset as $action_id ) {
-			$this->store->mark_failure( $action_id );
-			do_action( 'action_scheduler_failed_action', $action_id, $timeout );
+		foreach ($actions_to_reset as $action_id) {
+			$this->store->mark_failure($action_id);
+			do_action('action_scheduler_failed_action', $action_id, $timeout);
 		}
 	}
 
@@ -141,10 +141,10 @@ 
 	 * @param int $time_limit The number of seconds to use as the timeout and failure period. Default 300 (5 minutes).
 	 * @author Jeremy Pry
 	 */
-	public function clean( $time_limit = 300 ) {
+	public function clean($time_limit = 300) {
 		$this->delete_old_actions();
-		$this->reset_timeouts( $time_limit );
-		$this->mark_failures( $time_limit );
+		$this->reset_timeouts($time_limit);
+		$this->mark_failures($time_limit);
 	}
 
 	/**
@@ -159,6 +159,6 @@ 
 		 *
 		 * @param int $batch_size The number of actions to clean in one batch.
 		 */
-		return absint( apply_filters( 'action_scheduler_cleanup_batch_size', $this->batch_size ) );
+		return absint(apply_filters('action_scheduler_cleanup_batch_size', $this->batch_size));
 	}
 }

src/ext/action-scheduler/classes/ActionScheduler_AdminView.php

Indentation +238 added lines, -238 removed lines

@@ -7,242 +7,242 @@
  */
 class ActionScheduler_AdminView extends ActionScheduler_AdminView_Deprecated {
 
-	private static $admin_view = null;
-
-	private static $screen_id = 'tools_page_action-scheduler';
-
-	/** @var ActionScheduler_ListTable */
-	protected $list_table;
-
-	/**
-	 * @return ActionScheduler_AdminView
-	 * @codeCoverageIgnore
-	 */
-	public static function instance() {
-
-		if ( empty( self::$admin_view ) ) {
-			$class            = apply_filters( 'action_scheduler_admin_view_class', 'ActionScheduler_AdminView' );
-			self::$admin_view = new $class();
-		}
-
-		return self::$admin_view;
-	}
-
-	/**
-	 * @codeCoverageIgnore
-	 */
-	public function init() {
-		if ( is_admin() && ( ! defined( 'DOING_AJAX' ) || false == DOING_AJAX ) ) {
-
-			if ( class_exists( 'WooCommerce' ) ) {
-				add_action( 'woocommerce_admin_status_content_action-scheduler', array( $this, 'render_admin_ui' ) );
-				add_action( 'woocommerce_system_status_report', array( $this, 'system_status_report' ) );
-				add_filter( 'woocommerce_admin_status_tabs', array( $this, 'register_system_status_tab' ) );
-			}
-
-			add_action( 'admin_menu', array( $this, 'register_menu' ) );
-			add_action( 'admin_notices', array( $this, 'maybe_check_pastdue_actions' ) );
-			add_action( 'current_screen', array( $this, 'add_help_tabs' ) );
-		}
-	}
-
-	public function system_status_report() {
-		$table = new ActionScheduler_wcSystemStatus( ActionScheduler::store() );
-		$table->render();
-	}
-
-	/**
-	 * Registers action-scheduler into WooCommerce > System status.
-	 *
-	 * @param array $tabs An associative array of tab key => label.
-	 * @return array $tabs An associative array of tab key => label, including Action Scheduler's tabs
-	 */
-	public function register_system_status_tab( array $tabs ) {
-		$tabs['action-scheduler'] = __( 'Scheduled Actions', 'action-scheduler' );
-
-		return $tabs;
-	}
-
-	/**
-	 * Include Action Scheduler's administration under the Tools menu.
-	 *
-	 * A menu under the Tools menu is important for backward compatibility (as that's
-	 * where it started), and also provides more convenient access than the WooCommerce
-	 * System Status page, and for sites where WooCommerce isn't active.
-	 */
-	public function register_menu() {
-		$hook_suffix = add_submenu_page(
-			'tools.php',
-			__( 'Scheduled Actions', 'action-scheduler' ),
-			__( 'Scheduled Actions', 'action-scheduler' ),
-			'manage_options',
-			'action-scheduler',
-			array( $this, 'render_admin_ui' )
-		);
-		add_action( 'load-' . $hook_suffix, array( $this, 'process_admin_ui' ) );
-	}
-
-	/**
-	 * Triggers processing of any pending actions.
-	 */
-	public function process_admin_ui() {
-		$this->get_list_table();
-	}
-
-	/**
-	 * Renders the Admin UI
-	 */
-	public function render_admin_ui() {
-		$table = $this->get_list_table();
-		$table->display_page();
-	}
-
-	/**
-	 * Get the admin UI object and process any requested actions.
-	 *
-	 * @return ActionScheduler_ListTable
-	 */
-	protected function get_list_table() {
-		if ( null === $this->list_table ) {
-			$this->list_table = new ActionScheduler_ListTable( ActionScheduler::store(), ActionScheduler::logger(), ActionScheduler::runner() );
-			$this->list_table->process_actions();
-		}
-
-		return $this->list_table;
-	}
-
-	/**
-	 * Action: admin_notices
-	 *
-	 * Maybe check past-due actions, and print notice.
-	 *
-	 * @uses $this->check_pastdue_actions()
-	 */
-	public function maybe_check_pastdue_actions() {
-
-		// Filter to prevent checking actions (ex: inappropriate user).
-		if ( ! apply_filters( 'action_scheduler_check_pastdue_actions', current_user_can( 'manage_options' ) ) ) {
-			return;
-		}
-
-		// Get last check transient.
-		$last_check = get_transient( 'action_scheduler_last_pastdue_actions_check' );
-
-		// If transient exists, we're within interval, so bail.
-		if ( ! empty( $last_check ) ) {
-			return;
-		}
-
-		// Perform the check.
-		$this->check_pastdue_actions();
-	}
-
-	/**
-	 * Check past-due actions, and print notice.
-	 *
-	 * @todo update $link_url to "Past-due" filter when released (see issue #510, PR #511)
-	 */
-	protected function check_pastdue_actions() {
-
-		// Set thresholds.
-		$threshold_seconds = (int) apply_filters( 'action_scheduler_pastdue_actions_seconds', DAY_IN_SECONDS );
-		$threshhold_min    = (int) apply_filters( 'action_scheduler_pastdue_actions_min', 1 );
-
-		// Allow third-parties to preempt the default check logic.
-		$check = apply_filters( 'action_scheduler_pastdue_actions_check_pre', null );
-
-		// Scheduled actions query arguments.
-		$query_args = array(
-			'date'     => as_get_datetime_object( time() - $threshold_seconds ),
-			'status'   => ActionScheduler_Store::STATUS_PENDING,
-			'per_page' => $threshhold_min,
-		);
-
-		// If no third-party preempted, run default check.
-		if ( $check === null ) {
-			$store               = ActionScheduler_Store::instance();
-			$num_pastdue_actions = (int) $store->query_actions( $query_args, 'count' );
-
-			// Check if past-due actions count is greater than or equal to threshold.
-			$check = ( $num_pastdue_actions >= $threshhold_min );
-			$check = (bool) apply_filters( 'action_scheduler_pastdue_actions_check', $check, $num_pastdue_actions, $threshold_seconds, $threshhold_min );
-		}
-
-		// If check failed, set transient and abort.
-		if ( ! boolval( $check ) ) {
-			$interval = apply_filters( 'action_scheduler_pastdue_actions_check_interval', round( $threshold_seconds / 4 ), $threshold_seconds );
-			set_transient( 'action_scheduler_last_pastdue_actions_check', time(), $interval );
-
-			return;
-		}
-
-		$actions_url = add_query_arg(
-			array(
-				'page'   => 'action-scheduler',
-				'status' => 'past-due',
-				'order'  => 'asc',
-			),
-			admin_url( 'tools.php' )
-		);
-
-		// Print notice.
-		echo '<div class="notice notice-warning"><p>';
-		printf(
-			_n(
-				// translators: 1) is the number of affected actions, 2) is a link to an admin screen.
-				'<strong>Action Scheduler:</strong> %1$d <a href="%2$s">past-due action</a> found; something may be wrong. <a href="https://actionscheduler.org/faq/#my-site-has-past-due-actions-what-can-i-do" target="_blank">Read documentation &raquo;</a>',
-				'<strong>Action Scheduler:</strong> %1$d <a href="%2$s">past-due actions</a> found; something may be wrong. <a href="https://actionscheduler.org/faq/#my-site-has-past-due-actions-what-can-i-do" target="_blank">Read documentation &raquo;</a>',
-				$num_pastdue_actions,
-				'action-scheduler'
-			),
-			$num_pastdue_actions,
-			esc_attr( esc_url( $actions_url ) )
-		);
-		echo '</p></div>';
-
-		// Facilitate third-parties to evaluate and print notices.
-		do_action( 'action_scheduler_pastdue_actions_extra_notices', $query_args );
-	}
-
-	/**
-	 * Provide more information about the screen and its data in the help tab.
-	 */
-	public function add_help_tabs() {
-		$screen = get_current_screen();
-
-		if ( ! $screen || self::$screen_id != $screen->id ) {
-			return;
-		}
-
-		$as_version = ActionScheduler_Versions::instance()->latest_version();
-		$screen->add_help_tab(
-			array(
-				'id'      => 'action_scheduler_about',
-				'title'   => __( 'About', 'action-scheduler' ),
-				'content' =>
-					'<h2>' . sprintf( __( 'About Action Scheduler %s', 'action-scheduler' ), $as_version ) . '</h2>' .
-					'<p>' .
-						__( 'Action Scheduler is a scalable, traceable job queue for background processing large sets of actions. Action Scheduler works by triggering an action hook to run at some time in the future. Scheduled actions can also be scheduled to run on a recurring schedule.', 'action-scheduler' ) .
-					'</p>',
-			)
-		);
-
-		$screen->add_help_tab(
-			array(
-				'id'      => 'action_scheduler_columns',
-				'title'   => __( 'Columns', 'action-scheduler' ),
-				'content' =>
-					'<h2>' . __( 'Scheduled Action Columns', 'action-scheduler' ) . '</h2>' .
-					'<ul>' .
-					sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Hook', 'action-scheduler' ), __( 'Name of the action hook that will be triggered.', 'action-scheduler' ) ) .
-					sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Status', 'action-scheduler' ), __( 'Action statuses are Pending, Complete, Canceled, Failed', 'action-scheduler' ) ) .
-					sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Arguments', 'action-scheduler' ), __( 'Optional data array passed to the action hook.', 'action-scheduler' ) ) .
-					sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Group', 'action-scheduler' ), __( 'Optional action group.', 'action-scheduler' ) ) .
-					sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Recurrence', 'action-scheduler' ), __( 'The action\'s schedule frequency.', 'action-scheduler' ) ) .
-					sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Scheduled', 'action-scheduler' ), __( 'The date/time the action is/was scheduled to run.', 'action-scheduler' ) ) .
-					sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Log', 'action-scheduler' ), __( 'Activity log for the action.', 'action-scheduler' ) ) .
-					'</ul>',
-			)
-		);
-	}
+    private static $admin_view = null;
+
+    private static $screen_id = 'tools_page_action-scheduler';
+
+    /** @var ActionScheduler_ListTable */
+    protected $list_table;
+
+    /**
+     * @return ActionScheduler_AdminView
+     * @codeCoverageIgnore
+     */
+    public static function instance() {
+
+        if ( empty( self::$admin_view ) ) {
+            $class            = apply_filters( 'action_scheduler_admin_view_class', 'ActionScheduler_AdminView' );
+            self::$admin_view = new $class();
+        }
+
+        return self::$admin_view;
+    }
+
+    /**
+     * @codeCoverageIgnore
+     */
+    public function init() {
+        if ( is_admin() && ( ! defined( 'DOING_AJAX' ) || false == DOING_AJAX ) ) {
+
+            if ( class_exists( 'WooCommerce' ) ) {
+                add_action( 'woocommerce_admin_status_content_action-scheduler', array( $this, 'render_admin_ui' ) );
+                add_action( 'woocommerce_system_status_report', array( $this, 'system_status_report' ) );
+                add_filter( 'woocommerce_admin_status_tabs', array( $this, 'register_system_status_tab' ) );
+            }
+
+            add_action( 'admin_menu', array( $this, 'register_menu' ) );
+            add_action( 'admin_notices', array( $this, 'maybe_check_pastdue_actions' ) );
+            add_action( 'current_screen', array( $this, 'add_help_tabs' ) );
+        }
+    }
+
+    public function system_status_report() {
+        $table = new ActionScheduler_wcSystemStatus( ActionScheduler::store() );
+        $table->render();
+    }
+
+    /**
+     * Registers action-scheduler into WooCommerce > System status.
+     *
+     * @param array $tabs An associative array of tab key => label.
+     * @return array $tabs An associative array of tab key => label, including Action Scheduler's tabs
+     */
+    public function register_system_status_tab( array $tabs ) {
+        $tabs['action-scheduler'] = __( 'Scheduled Actions', 'action-scheduler' );
+
+        return $tabs;
+    }
+
+    /**
+     * Include Action Scheduler's administration under the Tools menu.
+     *
+     * A menu under the Tools menu is important for backward compatibility (as that's
+     * where it started), and also provides more convenient access than the WooCommerce
+     * System Status page, and for sites where WooCommerce isn't active.
+     */
+    public function register_menu() {
+        $hook_suffix = add_submenu_page(
+            'tools.php',
+            __( 'Scheduled Actions', 'action-scheduler' ),
+            __( 'Scheduled Actions', 'action-scheduler' ),
+            'manage_options',
+            'action-scheduler',
+            array( $this, 'render_admin_ui' )
+        );
+        add_action( 'load-' . $hook_suffix, array( $this, 'process_admin_ui' ) );
+    }
+
+    /**
+     * Triggers processing of any pending actions.
+     */
+    public function process_admin_ui() {
+        $this->get_list_table();
+    }
+
+    /**
+     * Renders the Admin UI
+     */
+    public function render_admin_ui() {
+        $table = $this->get_list_table();
+        $table->display_page();
+    }
+
+    /**
+     * Get the admin UI object and process any requested actions.
+     *
+     * @return ActionScheduler_ListTable
+     */
+    protected function get_list_table() {
+        if ( null === $this->list_table ) {
+            $this->list_table = new ActionScheduler_ListTable( ActionScheduler::store(), ActionScheduler::logger(), ActionScheduler::runner() );
+            $this->list_table->process_actions();
+        }
+
+        return $this->list_table;
+    }
+
+    /**
+     * Action: admin_notices
+     *
+     * Maybe check past-due actions, and print notice.
+     *
+     * @uses $this->check_pastdue_actions()
+     */
+    public function maybe_check_pastdue_actions() {
+
+        // Filter to prevent checking actions (ex: inappropriate user).
+        if ( ! apply_filters( 'action_scheduler_check_pastdue_actions', current_user_can( 'manage_options' ) ) ) {
+            return;
+        }
+
+        // Get last check transient.
+        $last_check = get_transient( 'action_scheduler_last_pastdue_actions_check' );
+
+        // If transient exists, we're within interval, so bail.
+        if ( ! empty( $last_check ) ) {
+            return;
+        }
+
+        // Perform the check.
+        $this->check_pastdue_actions();
+    }
+
+    /**
+     * Check past-due actions, and print notice.
+     *
+     * @todo update $link_url to "Past-due" filter when released (see issue #510, PR #511)
+     */
+    protected function check_pastdue_actions() {
+
+        // Set thresholds.
+        $threshold_seconds = (int) apply_filters( 'action_scheduler_pastdue_actions_seconds', DAY_IN_SECONDS );
+        $threshhold_min    = (int) apply_filters( 'action_scheduler_pastdue_actions_min', 1 );
+
+        // Allow third-parties to preempt the default check logic.
+        $check = apply_filters( 'action_scheduler_pastdue_actions_check_pre', null );
+
+        // Scheduled actions query arguments.
+        $query_args = array(
+            'date'     => as_get_datetime_object( time() - $threshold_seconds ),
+            'status'   => ActionScheduler_Store::STATUS_PENDING,
+            'per_page' => $threshhold_min,
+        );
+
+        // If no third-party preempted, run default check.
+        if ( $check === null ) {
+            $store               = ActionScheduler_Store::instance();
+            $num_pastdue_actions = (int) $store->query_actions( $query_args, 'count' );
+
+            // Check if past-due actions count is greater than or equal to threshold.
+            $check = ( $num_pastdue_actions >= $threshhold_min );
+            $check = (bool) apply_filters( 'action_scheduler_pastdue_actions_check', $check, $num_pastdue_actions, $threshold_seconds, $threshhold_min );
+        }
+
+        // If check failed, set transient and abort.
+        if ( ! boolval( $check ) ) {
+            $interval = apply_filters( 'action_scheduler_pastdue_actions_check_interval', round( $threshold_seconds / 4 ), $threshold_seconds );
+            set_transient( 'action_scheduler_last_pastdue_actions_check', time(), $interval );
+
+            return;
+        }
+
+        $actions_url = add_query_arg(
+            array(
+                'page'   => 'action-scheduler',
+                'status' => 'past-due',
+                'order'  => 'asc',
+            ),
+            admin_url( 'tools.php' )
+        );
+
+        // Print notice.
+        echo '<div class="notice notice-warning"><p>';
+        printf(
+            _n(
+                // translators: 1) is the number of affected actions, 2) is a link to an admin screen.
+                '<strong>Action Scheduler:</strong> %1$d <a href="%2$s">past-due action</a> found; something may be wrong. <a href="https://actionscheduler.org/faq/#my-site-has-past-due-actions-what-can-i-do" target="_blank">Read documentation &raquo;</a>',
+                '<strong>Action Scheduler:</strong> %1$d <a href="%2$s">past-due actions</a> found; something may be wrong. <a href="https://actionscheduler.org/faq/#my-site-has-past-due-actions-what-can-i-do" target="_blank">Read documentation &raquo;</a>',
+                $num_pastdue_actions,
+                'action-scheduler'
+            ),
+            $num_pastdue_actions,
+            esc_attr( esc_url( $actions_url ) )
+        );
+        echo '</p></div>';
+
+        // Facilitate third-parties to evaluate and print notices.
+        do_action( 'action_scheduler_pastdue_actions_extra_notices', $query_args );
+    }
+
+    /**
+     * Provide more information about the screen and its data in the help tab.
+     */
+    public function add_help_tabs() {
+        $screen = get_current_screen();
+
+        if ( ! $screen || self::$screen_id != $screen->id ) {
+            return;
+        }
+
+        $as_version = ActionScheduler_Versions::instance()->latest_version();
+        $screen->add_help_tab(
+            array(
+                'id'      => 'action_scheduler_about',
+                'title'   => __( 'About', 'action-scheduler' ),
+                'content' =>
+                    '<h2>' . sprintf( __( 'About Action Scheduler %s', 'action-scheduler' ), $as_version ) . '</h2>' .
+                    '<p>' .
+                        __( 'Action Scheduler is a scalable, traceable job queue for background processing large sets of actions. Action Scheduler works by triggering an action hook to run at some time in the future. Scheduled actions can also be scheduled to run on a recurring schedule.', 'action-scheduler' ) .
+                    '</p>',
+            )
+        );
+
+        $screen->add_help_tab(
+            array(
+                'id'      => 'action_scheduler_columns',
+                'title'   => __( 'Columns', 'action-scheduler' ),
+                'content' =>
+                    '<h2>' . __( 'Scheduled Action Columns', 'action-scheduler' ) . '</h2>' .
+                    '<ul>' .
+                    sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Hook', 'action-scheduler' ), __( 'Name of the action hook that will be triggered.', 'action-scheduler' ) ) .
+                    sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Status', 'action-scheduler' ), __( 'Action statuses are Pending, Complete, Canceled, Failed', 'action-scheduler' ) ) .
+                    sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Arguments', 'action-scheduler' ), __( 'Optional data array passed to the action hook.', 'action-scheduler' ) ) .
+                    sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Group', 'action-scheduler' ), __( 'Optional action group.', 'action-scheduler' ) ) .
+                    sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Recurrence', 'action-scheduler' ), __( 'The action\'s schedule frequency.', 'action-scheduler' ) ) .
+                    sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Scheduled', 'action-scheduler' ), __( 'The date/time the action is/was scheduled to run.', 'action-scheduler' ) ) .
+                    sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Log', 'action-scheduler' ), __( 'Activity log for the action.', 'action-scheduler' ) ) .
+                    '</ul>',
+            )
+        );
+    }
 }

Spacing +51 added lines, -51 removed lines

@@ -20,8 +20,8 @@ 
 	 */
 	public static function instance() {
 
-		if ( empty( self::$admin_view ) ) {
-			$class            = apply_filters( 'action_scheduler_admin_view_class', 'ActionScheduler_AdminView' );
+		if (empty(self::$admin_view)) {
+			$class            = apply_filters('action_scheduler_admin_view_class', 'ActionScheduler_AdminView');
 			self::$admin_view = new $class();
 		}
 
@@ -32,22 +32,22 @@ 
 	 * @codeCoverageIgnore
 	 */
 	public function init() {
-		if ( is_admin() && ( ! defined( 'DOING_AJAX' ) || false == DOING_AJAX ) ) {
+		if (is_admin() && ( ! defined('DOING_AJAX') || false == DOING_AJAX)) {
 
-			if ( class_exists( 'WooCommerce' ) ) {
-				add_action( 'woocommerce_admin_status_content_action-scheduler', array( $this, 'render_admin_ui' ) );
-				add_action( 'woocommerce_system_status_report', array( $this, 'system_status_report' ) );
-				add_filter( 'woocommerce_admin_status_tabs', array( $this, 'register_system_status_tab' ) );
+			if (class_exists('WooCommerce')) {
+				add_action('woocommerce_admin_status_content_action-scheduler', array($this, 'render_admin_ui'));
+				add_action('woocommerce_system_status_report', array($this, 'system_status_report'));
+				add_filter('woocommerce_admin_status_tabs', array($this, 'register_system_status_tab'));
 			}
 
-			add_action( 'admin_menu', array( $this, 'register_menu' ) );
-			add_action( 'admin_notices', array( $this, 'maybe_check_pastdue_actions' ) );
-			add_action( 'current_screen', array( $this, 'add_help_tabs' ) );
+			add_action('admin_menu', array($this, 'register_menu'));
+			add_action('admin_notices', array($this, 'maybe_check_pastdue_actions'));
+			add_action('current_screen', array($this, 'add_help_tabs'));
 		}
 	}
 
 	public function system_status_report() {
-		$table = new ActionScheduler_wcSystemStatus( ActionScheduler::store() );
+		$table = new ActionScheduler_wcSystemStatus(ActionScheduler::store());
 		$table->render();
 	}
 
@@ -57,8 +57,8 @@ 
 	 * @param array $tabs An associative array of tab key => label.
 	 * @return array $tabs An associative array of tab key => label, including Action Scheduler's tabs
 	 */
-	public function register_system_status_tab( array $tabs ) {
-		$tabs['action-scheduler'] = __( 'Scheduled Actions', 'action-scheduler' );
+	public function register_system_status_tab(array $tabs) {
+		$tabs['action-scheduler'] = __('Scheduled Actions', 'action-scheduler');
 
 		return $tabs;
 	}
@@ -73,13 +73,13 @@ 
 	public function register_menu() {
 		$hook_suffix = add_submenu_page(
 			'tools.php',
-			__( 'Scheduled Actions', 'action-scheduler' ),
-			__( 'Scheduled Actions', 'action-scheduler' ),
+			__('Scheduled Actions', 'action-scheduler'),
+			__('Scheduled Actions', 'action-scheduler'),
 			'manage_options',
 			'action-scheduler',
-			array( $this, 'render_admin_ui' )
+			array($this, 'render_admin_ui')
 		);
-		add_action( 'load-' . $hook_suffix, array( $this, 'process_admin_ui' ) );
+		add_action('load-'.$hook_suffix, array($this, 'process_admin_ui'));
 	}
 
 	/**
@@ -103,8 +103,8 @@ 
 	 * @return ActionScheduler_ListTable
 	 */
 	protected function get_list_table() {
-		if ( null === $this->list_table ) {
-			$this->list_table = new ActionScheduler_ListTable( ActionScheduler::store(), ActionScheduler::logger(), ActionScheduler::runner() );
+		if (null === $this->list_table) {
+			$this->list_table = new ActionScheduler_ListTable(ActionScheduler::store(), ActionScheduler::logger(), ActionScheduler::runner());
 			$this->list_table->process_actions();
 		}
 
@@ -121,15 +121,15 @@ 
 	public function maybe_check_pastdue_actions() {
 
 		// Filter to prevent checking actions (ex: inappropriate user).
-		if ( ! apply_filters( 'action_scheduler_check_pastdue_actions', current_user_can( 'manage_options' ) ) ) {
+		if ( ! apply_filters('action_scheduler_check_pastdue_actions', current_user_can('manage_options'))) {
 			return;
 		}
 
 		// Get last check transient.
-		$last_check = get_transient( 'action_scheduler_last_pastdue_actions_check' );
+		$last_check = get_transient('action_scheduler_last_pastdue_actions_check');
 
 		// If transient exists, we're within interval, so bail.
-		if ( ! empty( $last_check ) ) {
+		if ( ! empty($last_check)) {
 			return;
 		}
 
@@ -145,33 +145,33 @@ 
 	protected function check_pastdue_actions() {
 
 		// Set thresholds.
-		$threshold_seconds = (int) apply_filters( 'action_scheduler_pastdue_actions_seconds', DAY_IN_SECONDS );
-		$threshhold_min    = (int) apply_filters( 'action_scheduler_pastdue_actions_min', 1 );
+		$threshold_seconds = (int) apply_filters('action_scheduler_pastdue_actions_seconds', DAY_IN_SECONDS);
+		$threshhold_min    = (int) apply_filters('action_scheduler_pastdue_actions_min', 1);
 
 		// Allow third-parties to preempt the default check logic.
-		$check = apply_filters( 'action_scheduler_pastdue_actions_check_pre', null );
+		$check = apply_filters('action_scheduler_pastdue_actions_check_pre', null);
 
 		// Scheduled actions query arguments.
 		$query_args = array(
-			'date'     => as_get_datetime_object( time() - $threshold_seconds ),
+			'date'     => as_get_datetime_object(time() - $threshold_seconds),
 			'status'   => ActionScheduler_Store::STATUS_PENDING,
 			'per_page' => $threshhold_min,
 		);
 
 		// If no third-party preempted, run default check.
-		if ( $check === null ) {
+		if ($check === null) {
 			$store               = ActionScheduler_Store::instance();
-			$num_pastdue_actions = (int) $store->query_actions( $query_args, 'count' );
+			$num_pastdue_actions = (int) $store->query_actions($query_args, 'count');
 
 			// Check if past-due actions count is greater than or equal to threshold.
-			$check = ( $num_pastdue_actions >= $threshhold_min );
-			$check = (bool) apply_filters( 'action_scheduler_pastdue_actions_check', $check, $num_pastdue_actions, $threshold_seconds, $threshhold_min );
+			$check = ($num_pastdue_actions >= $threshhold_min);
+			$check = (bool) apply_filters('action_scheduler_pastdue_actions_check', $check, $num_pastdue_actions, $threshold_seconds, $threshhold_min);
 		}
 
 		// If check failed, set transient and abort.
-		if ( ! boolval( $check ) ) {
-			$interval = apply_filters( 'action_scheduler_pastdue_actions_check_interval', round( $threshold_seconds / 4 ), $threshold_seconds );
-			set_transient( 'action_scheduler_last_pastdue_actions_check', time(), $interval );
+		if ( ! boolval($check)) {
+			$interval = apply_filters('action_scheduler_pastdue_actions_check_interval', round($threshold_seconds / 4), $threshold_seconds);
+			set_transient('action_scheduler_last_pastdue_actions_check', time(), $interval);
 
 			return;
 		}
@@ -182,7 +182,7 @@ 
 				'status' => 'past-due',
 				'order'  => 'asc',
 			),
-			admin_url( 'tools.php' )
+			admin_url('tools.php')
 		);
 
 		// Print notice.
@@ -196,12 +196,12 @@ 
 				'action-scheduler'
 			),
 			$num_pastdue_actions,
-			esc_attr( esc_url( $actions_url ) )
+			esc_attr(esc_url($actions_url))
 		);
 		echo '</p></div>';
 
 		// Facilitate third-parties to evaluate and print notices.
-		do_action( 'action_scheduler_pastdue_actions_extra_notices', $query_args );
+		do_action('action_scheduler_pastdue_actions_extra_notices', $query_args);
 	}
 
 	/**
@@ -210,7 +210,7 @@ 
 	public function add_help_tabs() {
 		$screen = get_current_screen();
 
-		if ( ! $screen || self::$screen_id != $screen->id ) {
+		if ( ! $screen || self::$screen_id != $screen->id) {
 			return;
 		}
 
@@ -218,11 +218,11 @@ 
 		$screen->add_help_tab(
 			array(
 				'id'      => 'action_scheduler_about',
-				'title'   => __( 'About', 'action-scheduler' ),
+				'title'   => __('About', 'action-scheduler'),
 				'content' =>
-					'<h2>' . sprintf( __( 'About Action Scheduler %s', 'action-scheduler' ), $as_version ) . '</h2>' .
-					'<p>' .
-						__( 'Action Scheduler is a scalable, traceable job queue for background processing large sets of actions. Action Scheduler works by triggering an action hook to run at some time in the future. Scheduled actions can also be scheduled to run on a recurring schedule.', 'action-scheduler' ) .
+					'<h2>'.sprintf(__('About Action Scheduler %s', 'action-scheduler'), $as_version).'</h2>'.
+					'<p>'.
+						__('Action Scheduler is a scalable, traceable job queue for background processing large sets of actions. Action Scheduler works by triggering an action hook to run at some time in the future. Scheduled actions can also be scheduled to run on a recurring schedule.', 'action-scheduler').
 					'</p>',
 			)
 		);
@@ -230,17 +230,17 @@ 
 		$screen->add_help_tab(
 			array(
 				'id'      => 'action_scheduler_columns',
-				'title'   => __( 'Columns', 'action-scheduler' ),
+				'title'   => __('Columns', 'action-scheduler'),
 				'content' =>
-					'<h2>' . __( 'Scheduled Action Columns', 'action-scheduler' ) . '</h2>' .
-					'<ul>' .
-					sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Hook', 'action-scheduler' ), __( 'Name of the action hook that will be triggered.', 'action-scheduler' ) ) .
-					sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Status', 'action-scheduler' ), __( 'Action statuses are Pending, Complete, Canceled, Failed', 'action-scheduler' ) ) .
-					sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Arguments', 'action-scheduler' ), __( 'Optional data array passed to the action hook.', 'action-scheduler' ) ) .
-					sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Group', 'action-scheduler' ), __( 'Optional action group.', 'action-scheduler' ) ) .
-					sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Recurrence', 'action-scheduler' ), __( 'The action\'s schedule frequency.', 'action-scheduler' ) ) .
-					sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Scheduled', 'action-scheduler' ), __( 'The date/time the action is/was scheduled to run.', 'action-scheduler' ) ) .
-					sprintf( '<li><strong>%1$s</strong>: %2$s</li>', __( 'Log', 'action-scheduler' ), __( 'Activity log for the action.', 'action-scheduler' ) ) .
+					'<h2>'.__('Scheduled Action Columns', 'action-scheduler').'</h2>'.
+					'<ul>'.
+					sprintf('<li><strong>%1$s</strong>: %2$s</li>', __('Hook', 'action-scheduler'), __('Name of the action hook that will be triggered.', 'action-scheduler')).
+					sprintf('<li><strong>%1$s</strong>: %2$s</li>', __('Status', 'action-scheduler'), __('Action statuses are Pending, Complete, Canceled, Failed', 'action-scheduler')).
+					sprintf('<li><strong>%1$s</strong>: %2$s</li>', __('Arguments', 'action-scheduler'), __('Optional data array passed to the action hook.', 'action-scheduler')).
+					sprintf('<li><strong>%1$s</strong>: %2$s</li>', __('Group', 'action-scheduler'), __('Optional action group.', 'action-scheduler')).
+					sprintf('<li><strong>%1$s</strong>: %2$s</li>', __('Recurrence', 'action-scheduler'), __('The action\'s schedule frequency.', 'action-scheduler')).
+					sprintf('<li><strong>%1$s</strong>: %2$s</li>', __('Scheduled', 'action-scheduler'), __('The date/time the action is/was scheduled to run.', 'action-scheduler')).
+					sprintf('<li><strong>%1$s</strong>: %2$s</li>', __('Log', 'action-scheduler'), __('Activity log for the action.', 'action-scheduler')).
 					'</ul>',
 			)
 		);

src/ext/action-scheduler/lib/cron-expression/CronExpression_DayOfMonthField.php

Indentation +72 added lines, -72 removed lines

@@ -20,87 +20,87 @@
  */
 class CronExpression_DayOfMonthField extends CronExpression_AbstractField {
 
-	/**
-	 * Get the nearest day of the week for a given day in a month
-	 *
-	 * @param int $currentYear  Current year
-	 * @param int $currentMonth Current month
-	 * @param int $targetDay    Target day of the month
-	 *
-	 * @return DateTime Returns the nearest date
-	 */
-	private static function getNearestWeekday( $currentYear, $currentMonth, $targetDay ) {
-		$tday           = str_pad( $targetDay, 2, '0', STR_PAD_LEFT );
-		$target         = new DateTime( "$currentYear-$currentMonth-$tday" );
-		$currentWeekday = (int) $target->format( 'N' );
+    /**
+     * Get the nearest day of the week for a given day in a month
+     *
+     * @param int $currentYear  Current year
+     * @param int $currentMonth Current month
+     * @param int $targetDay    Target day of the month
+     *
+     * @return DateTime Returns the nearest date
+     */
+    private static function getNearestWeekday( $currentYear, $currentMonth, $targetDay ) {
+        $tday           = str_pad( $targetDay, 2, '0', STR_PAD_LEFT );
+        $target         = new DateTime( "$currentYear-$currentMonth-$tday" );
+        $currentWeekday = (int) $target->format( 'N' );
 
-		if ( $currentWeekday < 6 ) {
-			return $target;
-		}
+        if ( $currentWeekday < 6 ) {
+            return $target;
+        }
 
-		$lastDayOfMonth = $target->format( 't' );
+        $lastDayOfMonth = $target->format( 't' );
 
-		foreach ( array( -1, 1, -2, 2 ) as $i ) {
-			$adjusted = $targetDay + $i;
-			if ( $adjusted > 0 && $adjusted <= $lastDayOfMonth ) {
-				$target->setDate( $currentYear, $currentMonth, $adjusted );
-				if ( $target->format( 'N' ) < 6 && $target->format( 'm' ) == $currentMonth ) {
-					return $target;
-				}
-			}
-		}
-	}
+        foreach ( array( -1, 1, -2, 2 ) as $i ) {
+            $adjusted = $targetDay + $i;
+            if ( $adjusted > 0 && $adjusted <= $lastDayOfMonth ) {
+                $target->setDate( $currentYear, $currentMonth, $adjusted );
+                if ( $target->format( 'N' ) < 6 && $target->format( 'm' ) == $currentMonth ) {
+                    return $target;
+                }
+            }
+        }
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function isSatisfiedBy( DateTime $date, $value ) {
-		// ? states that the field value is to be skipped
-		if ( $value == '?' ) {
-			return true;
-		}
+    /**
+     * {@inheritdoc}
+     */
+    public function isSatisfiedBy( DateTime $date, $value ) {
+        // ? states that the field value is to be skipped
+        if ( $value == '?' ) {
+            return true;
+        }
 
-		$fieldValue = $date->format( 'd' );
+        $fieldValue = $date->format( 'd' );
 
-		// Check to see if this is the last day of the month
-		if ( $value == 'L' ) {
-			return $fieldValue == $date->format( 't' );
-		}
+        // Check to see if this is the last day of the month
+        if ( $value == 'L' ) {
+            return $fieldValue == $date->format( 't' );
+        }
 
-		// Check to see if this is the nearest weekday to a particular value
-		if ( strpos( $value, 'W' ) ) {
-			// Parse the target day
-			$targetDay = substr( $value, 0, strpos( $value, 'W' ) );
-			// Find out if the current day is the nearest day of the week
-			return $date->format( 'j' ) == self::getNearestWeekday(
-				$date->format( 'Y' ),
-				$date->format( 'm' ),
-				$targetDay
-			)->format( 'j' );
-		}
+        // Check to see if this is the nearest weekday to a particular value
+        if ( strpos( $value, 'W' ) ) {
+            // Parse the target day
+            $targetDay = substr( $value, 0, strpos( $value, 'W' ) );
+            // Find out if the current day is the nearest day of the week
+            return $date->format( 'j' ) == self::getNearestWeekday(
+                $date->format( 'Y' ),
+                $date->format( 'm' ),
+                $targetDay
+            )->format( 'j' );
+        }
 
-		return $this->isSatisfied( $date->format( 'd' ), $value );
-	}
+        return $this->isSatisfied( $date->format( 'd' ), $value );
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function increment( DateTime $date, $invert = false ) {
-		if ( $invert ) {
-			$date->modify( 'previous day' );
-			$date->setTime( 23, 59 );
-		} else {
-			$date->modify( 'next day' );
-			$date->setTime( 0, 0 );
-		}
+    /**
+     * {@inheritdoc}
+     */
+    public function increment( DateTime $date, $invert = false ) {
+        if ( $invert ) {
+            $date->modify( 'previous day' );
+            $date->setTime( 23, 59 );
+        } else {
+            $date->modify( 'next day' );
+            $date->setTime( 0, 0 );
+        }
 
-		return $this;
-	}
+        return $this;
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function validate( $value ) {
-		return (bool) preg_match( '/[\*,\/\-\?LW0-9A-Za-z]+/', $value );
-	}
+    /**
+     * {@inheritdoc}
+     */
+    public function validate( $value ) {
+        return (bool) preg_match( '/[\*,\/\-\?LW0-9A-Za-z]+/', $value );
+    }
 }

Spacing +30 added lines, -30 removed lines

@@ -29,22 +29,22 @@ 
 	 *
 	 * @return DateTime Returns the nearest date
 	 */
-	private static function getNearestWeekday( $currentYear, $currentMonth, $targetDay ) {
-		$tday           = str_pad( $targetDay, 2, '0', STR_PAD_LEFT );
-		$target         = new DateTime( "$currentYear-$currentMonth-$tday" );
-		$currentWeekday = (int) $target->format( 'N' );
+	private static function getNearestWeekday($currentYear, $currentMonth, $targetDay) {
+		$tday           = str_pad($targetDay, 2, '0', STR_PAD_LEFT);
+		$target         = new DateTime("$currentYear-$currentMonth-$tday");
+		$currentWeekday = (int) $target->format('N');
 
-		if ( $currentWeekday < 6 ) {
+		if ($currentWeekday < 6) {
 			return $target;
 		}
 
-		$lastDayOfMonth = $target->format( 't' );
+		$lastDayOfMonth = $target->format('t');
 
-		foreach ( array( -1, 1, -2, 2 ) as $i ) {
+		foreach (array( -1, 1, -2, 2 ) as $i) {
 			$adjusted = $targetDay + $i;
-			if ( $adjusted > 0 && $adjusted <= $lastDayOfMonth ) {
-				$target->setDate( $currentYear, $currentMonth, $adjusted );
-				if ( $target->format( 'N' ) < 6 && $target->format( 'm' ) == $currentMonth ) {
+			if ($adjusted > 0 && $adjusted <= $lastDayOfMonth) {
+				$target->setDate($currentYear, $currentMonth, $adjusted);
+				if ($target->format('N') < 6 && $target->format('m') == $currentMonth) {
 					return $target;
 				}
 			}
@@ -54,44 +54,44 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function isSatisfiedBy( DateTime $date, $value ) {
+	public function isSatisfiedBy(DateTime $date, $value) {
 		// ? states that the field value is to be skipped
-		if ( $value == '?' ) {
+		if ($value == '?') {
 			return true;
 		}
 
-		$fieldValue = $date->format( 'd' );
+		$fieldValue = $date->format('d');
 
 		// Check to see if this is the last day of the month
-		if ( $value == 'L' ) {
-			return $fieldValue == $date->format( 't' );
+		if ($value == 'L') {
+			return $fieldValue == $date->format('t');
 		}
 
 		// Check to see if this is the nearest weekday to a particular value
-		if ( strpos( $value, 'W' ) ) {
+		if (strpos($value, 'W')) {
 			// Parse the target day
-			$targetDay = substr( $value, 0, strpos( $value, 'W' ) );
+			$targetDay = substr($value, 0, strpos($value, 'W'));
 			// Find out if the current day is the nearest day of the week
-			return $date->format( 'j' ) == self::getNearestWeekday(
-				$date->format( 'Y' ),
-				$date->format( 'm' ),
+			return $date->format('j') == self::getNearestWeekday(
+				$date->format('Y'),
+				$date->format('m'),
 				$targetDay
-			)->format( 'j' );
+			)->format('j');
 		}
 
-		return $this->isSatisfied( $date->format( 'd' ), $value );
+		return $this->isSatisfied($date->format('d'), $value);
 	}
 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function increment( DateTime $date, $invert = false ) {
-		if ( $invert ) {
-			$date->modify( 'previous day' );
-			$date->setTime( 23, 59 );
+	public function increment(DateTime $date, $invert = false) {
+		if ($invert) {
+			$date->modify('previous day');
+			$date->setTime(23, 59);
 		} else {
-			$date->modify( 'next day' );
-			$date->setTime( 0, 0 );
+			$date->modify('next day');
+			$date->setTime(0, 0);
 		}
 
 		return $this;
@@ -100,7 +100,7 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function validate( $value ) {
-		return (bool) preg_match( '/[\*,\/\-\?LW0-9A-Za-z]+/', $value );
+	public function validate($value) {
+		return (bool) preg_match('/[\*,\/\-\?LW0-9A-Za-z]+/', $value);
 	}
 }

src/ext/action-scheduler/lib/cron-expression/CronExpression_YearField.php

Indentation +27 added lines, -27 removed lines

@@ -7,34 +7,34 @@
  */
 class CronExpression_YearField extends CronExpression_AbstractField {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function isSatisfiedBy( DateTime $date, $value ) {
-		return $this->isSatisfied( $date->format( 'Y' ), $value );
-	}
+    /**
+     * {@inheritdoc}
+     */
+    public function isSatisfiedBy( DateTime $date, $value ) {
+        return $this->isSatisfied( $date->format( 'Y' ), $value );
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function increment( DateTime $date, $invert = false ) {
-		if ( $invert ) {
-			$date->modify( '-1 year' );
-			$date->setDate( $date->format( 'Y' ), 12, 31 );
-			$date->setTime( 23, 59, 0 );
-		} else {
-			$date->modify( '+1 year' );
-			$date->setDate( $date->format( 'Y' ), 1, 1 );
-			$date->setTime( 0, 0, 0 );
-		}
+    /**
+     * {@inheritdoc}
+     */
+    public function increment( DateTime $date, $invert = false ) {
+        if ( $invert ) {
+            $date->modify( '-1 year' );
+            $date->setDate( $date->format( 'Y' ), 12, 31 );
+            $date->setTime( 23, 59, 0 );
+        } else {
+            $date->modify( '+1 year' );
+            $date->setDate( $date->format( 'Y' ), 1, 1 );
+            $date->setTime( 0, 0, 0 );
+        }
 
-		return $this;
-	}
+        return $this;
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function validate( $value ) {
-		return (bool) preg_match( '/[\*,\/\-0-9]+/', $value );
-	}
+    /**
+     * {@inheritdoc}
+     */
+    public function validate( $value ) {
+        return (bool) preg_match( '/[\*,\/\-0-9]+/', $value );
+    }
 }

Spacing +12 added lines, -12 removed lines

@@ -10,22 +10,22 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function isSatisfiedBy( DateTime $date, $value ) {
-		return $this->isSatisfied( $date->format( 'Y' ), $value );
+	public function isSatisfiedBy(DateTime $date, $value) {
+		return $this->isSatisfied($date->format('Y'), $value);
 	}
 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function increment( DateTime $date, $invert = false ) {
-		if ( $invert ) {
-			$date->modify( '-1 year' );
-			$date->setDate( $date->format( 'Y' ), 12, 31 );
-			$date->setTime( 23, 59, 0 );
+	public function increment(DateTime $date, $invert = false) {
+		if ($invert) {
+			$date->modify('-1 year');
+			$date->setDate($date->format('Y'), 12, 31);
+			$date->setTime(23, 59, 0);
 		} else {
-			$date->modify( '+1 year' );
-			$date->setDate( $date->format( 'Y' ), 1, 1 );
-			$date->setTime( 0, 0, 0 );
+			$date->modify('+1 year');
+			$date->setDate($date->format('Y'), 1, 1);
+			$date->setTime(0, 0, 0);
 		}
 
 		return $this;
@@ -34,7 +34,7 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function validate( $value ) {
-		return (bool) preg_match( '/[\*,\/\-0-9]+/', $value );
+	public function validate($value) {
+		return (bool) preg_match('/[\*,\/\-0-9]+/', $value);
 	}
 }

src/ext/action-scheduler/lib/cron-expression/CronExpression_HoursField.php

Indentation +31 added lines, -31 removed lines

@@ -7,38 +7,38 @@
  */
 class CronExpression_HoursField extends CronExpression_AbstractField {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function isSatisfiedBy( DateTime $date, $value ) {
-		return $this->isSatisfied( $date->format( 'H' ), $value );
-	}
+    /**
+     * {@inheritdoc}
+     */
+    public function isSatisfiedBy( DateTime $date, $value ) {
+        return $this->isSatisfied( $date->format( 'H' ), $value );
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function increment( DateTime $date, $invert = false ) {
-		// Change timezone to UTC temporarily. This will
-		// allow us to go back or forwards and hour even
-		// if DST will be changed between the hours.
-		$timezone = $date->getTimezone();
-		$date->setTimezone( new DateTimeZone( 'UTC' ) );
-		if ( $invert ) {
-			$date->modify( '-1 hour' );
-			$date->setTime( $date->format( 'H' ), 59 );
-		} else {
-			$date->modify( '+1 hour' );
-			$date->setTime( $date->format( 'H' ), 0 );
-		}
-		$date->setTimezone( $timezone );
+    /**
+     * {@inheritdoc}
+     */
+    public function increment( DateTime $date, $invert = false ) {
+        // Change timezone to UTC temporarily. This will
+        // allow us to go back or forwards and hour even
+        // if DST will be changed between the hours.
+        $timezone = $date->getTimezone();
+        $date->setTimezone( new DateTimeZone( 'UTC' ) );
+        if ( $invert ) {
+            $date->modify( '-1 hour' );
+            $date->setTime( $date->format( 'H' ), 59 );
+        } else {
+            $date->modify( '+1 hour' );
+            $date->setTime( $date->format( 'H' ), 0 );
+        }
+        $date->setTimezone( $timezone );
 
-		return $this;
-	}
+        return $this;
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function validate( $value ) {
-		return (bool) preg_match( '/[\*,\/\-0-9]+/', $value );
-	}
+    /**
+     * {@inheritdoc}
+     */
+    public function validate( $value ) {
+        return (bool) preg_match( '/[\*,\/\-0-9]+/', $value );
+    }
 }

Spacing +12 added lines, -12 removed lines

@@ -10,27 +10,27 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function isSatisfiedBy( DateTime $date, $value ) {
-		return $this->isSatisfied( $date->format( 'H' ), $value );
+	public function isSatisfiedBy(DateTime $date, $value) {
+		return $this->isSatisfied($date->format('H'), $value);
 	}
 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function increment( DateTime $date, $invert = false ) {
+	public function increment(DateTime $date, $invert = false) {
 		// Change timezone to UTC temporarily. This will
 		// allow us to go back or forwards and hour even
 		// if DST will be changed between the hours.
 		$timezone = $date->getTimezone();
-		$date->setTimezone( new DateTimeZone( 'UTC' ) );
-		if ( $invert ) {
-			$date->modify( '-1 hour' );
-			$date->setTime( $date->format( 'H' ), 59 );
+		$date->setTimezone(new DateTimeZone('UTC'));
+		if ($invert) {
+			$date->modify('-1 hour');
+			$date->setTime($date->format('H'), 59);
 		} else {
-			$date->modify( '+1 hour' );
-			$date->setTime( $date->format( 'H' ), 0 );
+			$date->modify('+1 hour');
+			$date->setTime($date->format('H'), 0);
 		}
-		$date->setTimezone( $timezone );
+		$date->setTimezone($timezone);
 
 		return $this;
 	}
@@ -38,7 +38,7 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function validate( $value ) {
-		return (bool) preg_match( '/[\*,\/\-0-9]+/', $value );
+	public function validate($value) {
+		return (bool) preg_match('/[\*,\/\-0-9]+/', $value);
 	}
 }

src/ext/action-scheduler/lib/cron-expression/CronExpression_DayOfWeekField.php

Indentation +88 added lines, -88 removed lines

@@ -17,105 +17,105 @@
  */
 class CronExpression_DayOfWeekField extends CronExpression_AbstractField {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function isSatisfiedBy( DateTime $date, $value ) {
-		if ( $value == '?' ) {
-			return true;
-		}
+    /**
+     * {@inheritdoc}
+     */
+    public function isSatisfiedBy( DateTime $date, $value ) {
+        if ( $value == '?' ) {
+            return true;
+        }
 
-		// Convert text day of the week values to integers
-		$value = str_ireplace(
-			array( 'SUN', 'MON', 'TUE', 'WED', 'THU', 'FRI', 'SAT' ),
-			range( 0, 6 ),
-			$value
-		);
+        // Convert text day of the week values to integers
+        $value = str_ireplace(
+            array( 'SUN', 'MON', 'TUE', 'WED', 'THU', 'FRI', 'SAT' ),
+            range( 0, 6 ),
+            $value
+        );
 
-		$currentYear    = $date->format( 'Y' );
-		$currentMonth   = $date->format( 'm' );
-		$lastDayOfMonth = $date->format( 't' );
+        $currentYear    = $date->format( 'Y' );
+        $currentMonth   = $date->format( 'm' );
+        $lastDayOfMonth = $date->format( 't' );
 
-		// Find out if this is the last specific weekday of the month
-		if ( strpos( $value, 'L' ) ) {
-			$weekday = str_replace( '7', '0', substr( $value, 0, strpos( $value, 'L' ) ) );
-			$tdate   = clone $date;
-			$tdate->setDate( $currentYear, $currentMonth, $lastDayOfMonth );
-			while ( $tdate->format( 'w' ) != $weekday ) {
-				$tdate->setDate( $currentYear, $currentMonth, --$lastDayOfMonth );
-			}
+        // Find out if this is the last specific weekday of the month
+        if ( strpos( $value, 'L' ) ) {
+            $weekday = str_replace( '7', '0', substr( $value, 0, strpos( $value, 'L' ) ) );
+            $tdate   = clone $date;
+            $tdate->setDate( $currentYear, $currentMonth, $lastDayOfMonth );
+            while ( $tdate->format( 'w' ) != $weekday ) {
+                $tdate->setDate( $currentYear, $currentMonth, --$lastDayOfMonth );
+            }
 
-			return $date->format( 'j' ) == $lastDayOfMonth;
-		}
+            return $date->format( 'j' ) == $lastDayOfMonth;
+        }
 
-		// Handle # hash tokens
-		if ( strpos( $value, '#' ) ) {
-			list($weekday, $nth) = explode( '#', $value );
-			// Validate the hash fields
-			if ( $weekday < 1 || $weekday > 5 ) {
-				throw new InvalidArgumentException( "Weekday must be a value between 1 and 5. {$weekday} given" );
-			}
-			if ( $nth > 5 ) {
-				throw new InvalidArgumentException( 'There are never more than 5 of a given weekday in a month' );
-			}
-			// The current weekday must match the targeted weekday to proceed
-			if ( $date->format( 'N' ) != $weekday ) {
-				return false;
-			}
+        // Handle # hash tokens
+        if ( strpos( $value, '#' ) ) {
+            list($weekday, $nth) = explode( '#', $value );
+            // Validate the hash fields
+            if ( $weekday < 1 || $weekday > 5 ) {
+                throw new InvalidArgumentException( "Weekday must be a value between 1 and 5. {$weekday} given" );
+            }
+            if ( $nth > 5 ) {
+                throw new InvalidArgumentException( 'There are never more than 5 of a given weekday in a month' );
+            }
+            // The current weekday must match the targeted weekday to proceed
+            if ( $date->format( 'N' ) != $weekday ) {
+                return false;
+            }
 
-			$tdate = clone $date;
-			$tdate->setDate( $currentYear, $currentMonth, 1 );
-			$dayCount   = 0;
-			$currentDay = 1;
-			while ( $currentDay < $lastDayOfMonth + 1 ) {
-				if ( $tdate->format( 'N' ) == $weekday ) {
-					if ( ++$dayCount >= $nth ) {
-						break;
-					}
-				}
-				$tdate->setDate( $currentYear, $currentMonth, ++$currentDay );
-			}
+            $tdate = clone $date;
+            $tdate->setDate( $currentYear, $currentMonth, 1 );
+            $dayCount   = 0;
+            $currentDay = 1;
+            while ( $currentDay < $lastDayOfMonth + 1 ) {
+                if ( $tdate->format( 'N' ) == $weekday ) {
+                    if ( ++$dayCount >= $nth ) {
+                        break;
+                    }
+                }
+                $tdate->setDate( $currentYear, $currentMonth, ++$currentDay );
+            }
 
-			return $date->format( 'j' ) == $currentDay;
-		}
+            return $date->format( 'j' ) == $currentDay;
+        }
 
-		// Handle day of the week values
-		if ( strpos( $value, '-' ) ) {
-			$parts = explode( '-', $value );
-			if ( $parts[0] == '7' ) {
-				$parts[0] = '0';
-			} elseif ( $parts[1] == '0' ) {
-				$parts[1] = '7';
-			}
-			$value = implode( '-', $parts );
-		}
+        // Handle day of the week values
+        if ( strpos( $value, '-' ) ) {
+            $parts = explode( '-', $value );
+            if ( $parts[0] == '7' ) {
+                $parts[0] = '0';
+            } elseif ( $parts[1] == '0' ) {
+                $parts[1] = '7';
+            }
+            $value = implode( '-', $parts );
+        }
 
-		// Test to see which Sunday to use -- 0 == 7 == Sunday
-		$format     = in_array( 7, str_split( $value ) ) ? 'N' : 'w';
-		$fieldValue = $date->format( $format );
+        // Test to see which Sunday to use -- 0 == 7 == Sunday
+        $format     = in_array( 7, str_split( $value ) ) ? 'N' : 'w';
+        $fieldValue = $date->format( $format );
 
-		return $this->isSatisfied( $fieldValue, $value );
-	}
+        return $this->isSatisfied( $fieldValue, $value );
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function increment( DateTime $date, $invert = false ) {
-		if ( $invert ) {
-			$date->modify( '-1 day' );
-			$date->setTime( 23, 59, 0 );
-		} else {
-			$date->modify( '+1 day' );
-			$date->setTime( 0, 0, 0 );
-		}
+    /**
+     * {@inheritdoc}
+     */
+    public function increment( DateTime $date, $invert = false ) {
+        if ( $invert ) {
+            $date->modify( '-1 day' );
+            $date->setTime( 23, 59, 0 );
+        } else {
+            $date->modify( '+1 day' );
+            $date->setTime( 0, 0, 0 );
+        }
 
-		return $this;
-	}
+        return $this;
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function validate( $value ) {
-		return (bool) preg_match( '/[\*,\/\-0-9A-Z]+/', $value );
-	}
+    /**
+     * {@inheritdoc}
+     */
+    public function validate( $value ) {
+        return (bool) preg_match( '/[\*,\/\-0-9A-Z]+/', $value );
+    }
 }

Spacing +41 added lines, -41 removed lines

@@ -20,93 +20,93 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function isSatisfiedBy( DateTime $date, $value ) {
-		if ( $value == '?' ) {
+	public function isSatisfiedBy(DateTime $date, $value) {
+		if ($value == '?') {
 			return true;
 		}
 
 		// Convert text day of the week values to integers
 		$value = str_ireplace(
-			array( 'SUN', 'MON', 'TUE', 'WED', 'THU', 'FRI', 'SAT' ),
-			range( 0, 6 ),
+			array('SUN', 'MON', 'TUE', 'WED', 'THU', 'FRI', 'SAT'),
+			range(0, 6),
 			$value
 		);
 
-		$currentYear    = $date->format( 'Y' );
-		$currentMonth   = $date->format( 'm' );
-		$lastDayOfMonth = $date->format( 't' );
+		$currentYear    = $date->format('Y');
+		$currentMonth   = $date->format('m');
+		$lastDayOfMonth = $date->format('t');
 
 		// Find out if this is the last specific weekday of the month
-		if ( strpos( $value, 'L' ) ) {
-			$weekday = str_replace( '7', '0', substr( $value, 0, strpos( $value, 'L' ) ) );
+		if (strpos($value, 'L')) {
+			$weekday = str_replace('7', '0', substr($value, 0, strpos($value, 'L')));
 			$tdate   = clone $date;
-			$tdate->setDate( $currentYear, $currentMonth, $lastDayOfMonth );
-			while ( $tdate->format( 'w' ) != $weekday ) {
-				$tdate->setDate( $currentYear, $currentMonth, --$lastDayOfMonth );
+			$tdate->setDate($currentYear, $currentMonth, $lastDayOfMonth);
+			while ($tdate->format('w') != $weekday) {
+				$tdate->setDate($currentYear, $currentMonth, --$lastDayOfMonth);
 			}
 
-			return $date->format( 'j' ) == $lastDayOfMonth;
+			return $date->format('j') == $lastDayOfMonth;
 		}
 
 		// Handle # hash tokens
-		if ( strpos( $value, '#' ) ) {
-			list($weekday, $nth) = explode( '#', $value );
+		if (strpos($value, '#')) {
+			list($weekday, $nth) = explode('#', $value);
 			// Validate the hash fields
-			if ( $weekday < 1 || $weekday > 5 ) {
-				throw new InvalidArgumentException( "Weekday must be a value between 1 and 5. {$weekday} given" );
+			if ($weekday < 1 || $weekday > 5) {
+				throw new InvalidArgumentException("Weekday must be a value between 1 and 5. {$weekday} given");
 			}
-			if ( $nth > 5 ) {
-				throw new InvalidArgumentException( 'There are never more than 5 of a given weekday in a month' );
+			if ($nth > 5) {
+				throw new InvalidArgumentException('There are never more than 5 of a given weekday in a month');
 			}
 			// The current weekday must match the targeted weekday to proceed
-			if ( $date->format( 'N' ) != $weekday ) {
+			if ($date->format('N') != $weekday) {
 				return false;
 			}
 
 			$tdate = clone $date;
-			$tdate->setDate( $currentYear, $currentMonth, 1 );
+			$tdate->setDate($currentYear, $currentMonth, 1);
 			$dayCount   = 0;
 			$currentDay = 1;
-			while ( $currentDay < $lastDayOfMonth + 1 ) {
-				if ( $tdate->format( 'N' ) == $weekday ) {
+			while ($currentDay < $lastDayOfMonth + 1) {
+				if ($tdate->format('N') == $weekday) {
 					if ( ++$dayCount >= $nth ) {
 						break;
 					}
 				}
-				$tdate->setDate( $currentYear, $currentMonth, ++$currentDay );
+				$tdate->setDate($currentYear, $currentMonth, ++$currentDay);
 			}
 
-			return $date->format( 'j' ) == $currentDay;
+			return $date->format('j') == $currentDay;
 		}
 
 		// Handle day of the week values
-		if ( strpos( $value, '-' ) ) {
-			$parts = explode( '-', $value );
-			if ( $parts[0] == '7' ) {
+		if (strpos($value, '-')) {
+			$parts = explode('-', $value);
+			if ($parts[0] == '7') {
 				$parts[0] = '0';
-			} elseif ( $parts[1] == '0' ) {
+			} elseif ($parts[1] == '0') {
 				$parts[1] = '7';
 			}
-			$value = implode( '-', $parts );
+			$value = implode('-', $parts);
 		}
 
 		// Test to see which Sunday to use -- 0 == 7 == Sunday
-		$format     = in_array( 7, str_split( $value ) ) ? 'N' : 'w';
-		$fieldValue = $date->format( $format );
+		$format     = in_array(7, str_split($value)) ? 'N' : 'w';
+		$fieldValue = $date->format($format);
 
-		return $this->isSatisfied( $fieldValue, $value );
+		return $this->isSatisfied($fieldValue, $value);
 	}
 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function increment( DateTime $date, $invert = false ) {
-		if ( $invert ) {
-			$date->modify( '-1 day' );
-			$date->setTime( 23, 59, 0 );
+	public function increment(DateTime $date, $invert = false) {
+		if ($invert) {
+			$date->modify('-1 day');
+			$date->setTime(23, 59, 0);
 		} else {
-			$date->modify( '+1 day' );
-			$date->setTime( 0, 0, 0 );
+			$date->modify('+1 day');
+			$date->setTime(0, 0, 0);
 		}
 
 		return $this;
@@ -115,7 +115,7 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function validate( $value ) {
-		return (bool) preg_match( '/[\*,\/\-0-9A-Z]+/', $value );
+	public function validate($value) {
+		return (bool) preg_match('/[\*,\/\-0-9A-Z]+/', $value);
 	}
 }

src/ext/action-scheduler/lib/cron-expression/CronExpression_MinutesField.php

Indentation +23 added lines, -23 removed lines

@@ -7,30 +7,30 @@
  */
 class CronExpression_MinutesField extends CronExpression_AbstractField {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function isSatisfiedBy( DateTime $date, $value ) {
-		return $this->isSatisfied( $date->format( 'i' ), $value );
-	}
+    /**
+     * {@inheritdoc}
+     */
+    public function isSatisfiedBy( DateTime $date, $value ) {
+        return $this->isSatisfied( $date->format( 'i' ), $value );
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function increment( DateTime $date, $invert = false ) {
-		if ( $invert ) {
-			$date->modify( '-1 minute' );
-		} else {
-			$date->modify( '+1 minute' );
-		}
+    /**
+     * {@inheritdoc}
+     */
+    public function increment( DateTime $date, $invert = false ) {
+        if ( $invert ) {
+            $date->modify( '-1 minute' );
+        } else {
+            $date->modify( '+1 minute' );
+        }
 
-		return $this;
-	}
+        return $this;
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function validate( $value ) {
-		return (bool) preg_match( '/[\*,\/\-0-9]+/', $value );
-	}
+    /**
+     * {@inheritdoc}
+     */
+    public function validate( $value ) {
+        return (bool) preg_match( '/[\*,\/\-0-9]+/', $value );
+    }
 }

Spacing +8 added lines, -8 removed lines

@@ -10,18 +10,18 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function isSatisfiedBy( DateTime $date, $value ) {
-		return $this->isSatisfied( $date->format( 'i' ), $value );
+	public function isSatisfiedBy(DateTime $date, $value) {
+		return $this->isSatisfied($date->format('i'), $value);
 	}
 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function increment( DateTime $date, $invert = false ) {
-		if ( $invert ) {
-			$date->modify( '-1 minute' );
+	public function increment(DateTime $date, $invert = false) {
+		if ($invert) {
+			$date->modify('-1 minute');
 		} else {
-			$date->modify( '+1 minute' );
+			$date->modify('+1 minute');
 		}
 
 		return $this;
@@ -30,7 +30,7 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function validate( $value ) {
-		return (bool) preg_match( '/[\*,\/\-0-9]+/', $value );
+	public function validate($value) {
+		return (bool) preg_match('/[\*,\/\-0-9]+/', $value);
 	}
 }