AyeCode / invoicing

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

Indentation +159 added lines, -159 removed lines

@@ -5,175 +5,175 @@
  */
 class ActionScheduler_ActionFactory {
 
-	/**
-	 * @param string $status The action's status in the data store
-	 * @param string $hook The hook to trigger when this action runs
-	 * @param array $args Args to pass to callbacks when the hook is triggered
-	 * @param ActionScheduler_Schedule $schedule The action's schedule
-	 * @param string $group A group to put the action in
-	 *
-	 * @return ActionScheduler_Action An instance of the stored action
-	 */
-	public function get_stored_action( $status, $hook, array $args = array(), ActionScheduler_Schedule $schedule = null, $group = '' ) {
+    /**
+     * @param string $status The action's status in the data store
+     * @param string $hook The hook to trigger when this action runs
+     * @param array $args Args to pass to callbacks when the hook is triggered
+     * @param ActionScheduler_Schedule $schedule The action's schedule
+     * @param string $group A group to put the action in
+     *
+     * @return ActionScheduler_Action An instance of the stored action
+     */
+    public function get_stored_action( $status, $hook, array $args = array(), ActionScheduler_Schedule $schedule = null, $group = '' ) {
 
-		switch ( $status ) {
-			case ActionScheduler_Store::STATUS_PENDING :
-				$action_class = 'ActionScheduler_Action';
-				break;
-			case ActionScheduler_Store::STATUS_CANCELED :
-				$action_class = 'ActionScheduler_CanceledAction';
-				if ( ! is_null( $schedule ) && ! is_a( $schedule, 'ActionScheduler_CanceledSchedule' ) ) {
-					$schedule = new ActionScheduler_CanceledSchedule( $schedule->get_date() );
-				}
-				break;
-			default :
-				$action_class = 'ActionScheduler_FinishedAction';
-				break;
-		}
+        switch ( $status ) {
+            case ActionScheduler_Store::STATUS_PENDING :
+                $action_class = 'ActionScheduler_Action';
+                break;
+            case ActionScheduler_Store::STATUS_CANCELED :
+                $action_class = 'ActionScheduler_CanceledAction';
+                if ( ! is_null( $schedule ) && ! is_a( $schedule, 'ActionScheduler_CanceledSchedule' ) ) {
+                    $schedule = new ActionScheduler_CanceledSchedule( $schedule->get_date() );
+                }
+                break;
+            default :
+                $action_class = 'ActionScheduler_FinishedAction';
+                break;
+        }
 
-		$action_class = apply_filters( 'action_scheduler_stored_action_class', $action_class, $status, $hook, $args, $schedule, $group );
+        $action_class = apply_filters( 'action_scheduler_stored_action_class', $action_class, $status, $hook, $args, $schedule, $group );
 
-		$action = new $action_class( $hook, $args, $schedule, $group );
+        $action = new $action_class( $hook, $args, $schedule, $group );
 
-		/**
-		 * Allow 3rd party code to change the instantiated action for a given hook, args, schedule and group.
-		 *
-		 * @param ActionScheduler_Action $action The instantiated action.
-		 * @param string $hook The instantiated action's hook.
-		 * @param array $args The instantiated action's args.
-		 * @param ActionScheduler_Schedule $schedule The instantiated action's schedule.
-		 * @param string $group The instantiated action's group.
-		 */
-		return apply_filters( 'action_scheduler_stored_action_instance', $action, $hook, $args, $schedule, $group );
-	}
+        /**
+         * Allow 3rd party code to change the instantiated action for a given hook, args, schedule and group.
+         *
+         * @param ActionScheduler_Action $action The instantiated action.
+         * @param string $hook The instantiated action's hook.
+         * @param array $args The instantiated action's args.
+         * @param ActionScheduler_Schedule $schedule The instantiated action's schedule.
+         * @param string $group The instantiated action's group.
+         */
+        return apply_filters( 'action_scheduler_stored_action_instance', $action, $hook, $args, $schedule, $group );
+    }
 
-	/**
-	 * Enqueue an action to run one time, as soon as possible (rather a specific scheduled time).
-	 *
-	 * This method creates a new action with the NULLSchedule. This schedule maps to a MySQL datetime string of
-	 * 0000-00-00 00:00:00. This is done to create a psuedo "async action" type that is fully backward compatible.
-	 * Existing queries to claim actions claim by date, meaning actions scheduled for 0000-00-00 00:00:00 will
-	 * always be claimed prior to actions scheduled for a specific date. This makes sure that any async action is
-	 * given priority in queue processing. This has the added advantage of making sure async actions can be
-	 * claimed by both the existing WP Cron and WP CLI runners, as well as a new async request runner.
-	 *
-	 * @param string $hook The hook to trigger when this action runs
-	 * @param array $args Args to pass when the hook is triggered
-	 * @param string $group A group to put the action in
-	 *
-	 * @return string The ID of the stored action
-	 */
-	public function async( $hook, $args = array(), $group = '' ) {
-		$schedule = new ActionScheduler_NullSchedule();
-		$action = new ActionScheduler_Action( $hook, $args, $schedule, $group );
-		return $this->store( $action );
-	}
+    /**
+     * Enqueue an action to run one time, as soon as possible (rather a specific scheduled time).
+     *
+     * This method creates a new action with the NULLSchedule. This schedule maps to a MySQL datetime string of
+     * 0000-00-00 00:00:00. This is done to create a psuedo "async action" type that is fully backward compatible.
+     * Existing queries to claim actions claim by date, meaning actions scheduled for 0000-00-00 00:00:00 will
+     * always be claimed prior to actions scheduled for a specific date. This makes sure that any async action is
+     * given priority in queue processing. This has the added advantage of making sure async actions can be
+     * claimed by both the existing WP Cron and WP CLI runners, as well as a new async request runner.
+     *
+     * @param string $hook The hook to trigger when this action runs
+     * @param array $args Args to pass when the hook is triggered
+     * @param string $group A group to put the action in
+     *
+     * @return string The ID of the stored action
+     */
+    public function async( $hook, $args = array(), $group = '' ) {
+        $schedule = new ActionScheduler_NullSchedule();
+        $action = new ActionScheduler_Action( $hook, $args, $schedule, $group );
+        return $this->store( $action );
+    }
 
-	/**
-	 * @param string $hook The hook to trigger when this action runs
-	 * @param array $args Args to pass when the hook is triggered
-	 * @param int $when Unix timestamp when the action will run
-	 * @param string $group A group to put the action in
-	 *
-	 * @return string The ID of the stored action
-	 */
-	public function single( $hook, $args = array(), $when = null, $group = '' ) {
-		$date = as_get_datetime_object( $when );
-		$schedule = new ActionScheduler_SimpleSchedule( $date );
-		$action = new ActionScheduler_Action( $hook, $args, $schedule, $group );
-		return $this->store( $action );
-	}
+    /**
+     * @param string $hook The hook to trigger when this action runs
+     * @param array $args Args to pass when the hook is triggered
+     * @param int $when Unix timestamp when the action will run
+     * @param string $group A group to put the action in
+     *
+     * @return string The ID of the stored action
+     */
+    public function single( $hook, $args = array(), $when = null, $group = '' ) {
+        $date = as_get_datetime_object( $when );
+        $schedule = new ActionScheduler_SimpleSchedule( $date );
+        $action = new ActionScheduler_Action( $hook, $args, $schedule, $group );
+        return $this->store( $action );
+    }
 
-	/**
-	 * Create the first instance of an action recurring on a given interval.
-	 *
-	 * @param string $hook The hook to trigger when this action runs
-	 * @param array $args Args to pass when the hook is triggered
-	 * @param int $first Unix timestamp for the first run
-	 * @param int $interval Seconds between runs
-	 * @param string $group A group to put the action in
-	 *
-	 * @return string The ID of the stored action
-	 */
-	public function recurring( $hook, $args = array(), $first = null, $interval = null, $group = '' ) {
-		if ( empty($interval) ) {
-			return $this->single( $hook, $args, $first, $group );
-		}
-		$date = as_get_datetime_object( $first );
-		$schedule = new ActionScheduler_IntervalSchedule( $date, $interval );
-		$action = new ActionScheduler_Action( $hook, $args, $schedule, $group );
-		return $this->store( $action );
-	}
+    /**
+     * Create the first instance of an action recurring on a given interval.
+     *
+     * @param string $hook The hook to trigger when this action runs
+     * @param array $args Args to pass when the hook is triggered
+     * @param int $first Unix timestamp for the first run
+     * @param int $interval Seconds between runs
+     * @param string $group A group to put the action in
+     *
+     * @return string The ID of the stored action
+     */
+    public function recurring( $hook, $args = array(), $first = null, $interval = null, $group = '' ) {
+        if ( empty($interval) ) {
+            return $this->single( $hook, $args, $first, $group );
+        }
+        $date = as_get_datetime_object( $first );
+        $schedule = new ActionScheduler_IntervalSchedule( $date, $interval );
+        $action = new ActionScheduler_Action( $hook, $args, $schedule, $group );
+        return $this->store( $action );
+    }
 
-	/**
-	 * Create the first instance of an action recurring on a Cron schedule.
-	 *
-	 * @param string $hook The hook to trigger when this action runs
-	 * @param array $args Args to pass when the hook is triggered
-	 * @param int $base_timestamp The first instance of the action will be scheduled
-	 *        to run at a time calculated after this timestamp matching the cron
-	 *        expression. This can be used to delay the first instance of the action.
-	 * @param int $schedule A cron definition string
-	 * @param string $group A group to put the action in
-	 *
-	 * @return string The ID of the stored action
-	 */
-	public function cron( $hook, $args = array(), $base_timestamp = null, $schedule = null, $group = '' ) {
-		if ( empty($schedule) ) {
-			return $this->single( $hook, $args, $base_timestamp, $group );
-		}
-		$date = as_get_datetime_object( $base_timestamp );
-		$cron = CronExpression::factory( $schedule );
-		$schedule = new ActionScheduler_CronSchedule( $date, $cron );
-		$action = new ActionScheduler_Action( $hook, $args, $schedule, $group );
-		return $this->store( $action );
-	}
+    /**
+     * Create the first instance of an action recurring on a Cron schedule.
+     *
+     * @param string $hook The hook to trigger when this action runs
+     * @param array $args Args to pass when the hook is triggered
+     * @param int $base_timestamp The first instance of the action will be scheduled
+     *        to run at a time calculated after this timestamp matching the cron
+     *        expression. This can be used to delay the first instance of the action.
+     * @param int $schedule A cron definition string
+     * @param string $group A group to put the action in
+     *
+     * @return string The ID of the stored action
+     */
+    public function cron( $hook, $args = array(), $base_timestamp = null, $schedule = null, $group = '' ) {
+        if ( empty($schedule) ) {
+            return $this->single( $hook, $args, $base_timestamp, $group );
+        }
+        $date = as_get_datetime_object( $base_timestamp );
+        $cron = CronExpression::factory( $schedule );
+        $schedule = new ActionScheduler_CronSchedule( $date, $cron );
+        $action = new ActionScheduler_Action( $hook, $args, $schedule, $group );
+        return $this->store( $action );
+    }
 
-	/**
-	 * Create a successive instance of a recurring or cron action.
-	 *
-	 * Importantly, the action will be rescheduled to run based on the current date/time.
-	 * That means when the action is scheduled to run in the past, the next scheduled date
-	 * will be pushed forward. For example, if a recurring action set to run every hour
-	 * was scheduled to run 5 seconds ago, it will be next scheduled for 1 hour in the
-	 * future, which is 1 hour and 5 seconds from when it was last scheduled to run.
-	 *
-	 * Alternatively, if the action is scheduled to run in the future, and is run early,
-	 * likely via manual intervention, then its schedule will change based on the time now.
-	 * For example, if a recurring action set to run every day, and is run 12 hours early,
-	 * it will run again in 24 hours, not 36 hours.
-	 *
-	 * This slippage is less of an issue with Cron actions, as the specific run time can
-	 * be set for them to run, e.g. 1am each day. In those cases, and entire period would
-	 * need to be missed before there was any change is scheduled, e.g. in the case of an
-	 * action scheduled for 1am each day, the action would need to run an entire day late.
-	 *
-	 * @param ActionScheduler_Action $action The existing action.
-	 *
-	 * @return string The ID of the stored action
-	 * @throws InvalidArgumentException If $action is not a recurring action.
-	 */
-	public function repeat( $action ) {
-		$schedule = $action->get_schedule();
-		$next     = $schedule->get_next( as_get_datetime_object() );
+    /**
+     * Create a successive instance of a recurring or cron action.
+     *
+     * Importantly, the action will be rescheduled to run based on the current date/time.
+     * That means when the action is scheduled to run in the past, the next scheduled date
+     * will be pushed forward. For example, if a recurring action set to run every hour
+     * was scheduled to run 5 seconds ago, it will be next scheduled for 1 hour in the
+     * future, which is 1 hour and 5 seconds from when it was last scheduled to run.
+     *
+     * Alternatively, if the action is scheduled to run in the future, and is run early,
+     * likely via manual intervention, then its schedule will change based on the time now.
+     * For example, if a recurring action set to run every day, and is run 12 hours early,
+     * it will run again in 24 hours, not 36 hours.
+     *
+     * This slippage is less of an issue with Cron actions, as the specific run time can
+     * be set for them to run, e.g. 1am each day. In those cases, and entire period would
+     * need to be missed before there was any change is scheduled, e.g. in the case of an
+     * action scheduled for 1am each day, the action would need to run an entire day late.
+     *
+     * @param ActionScheduler_Action $action The existing action.
+     *
+     * @return string The ID of the stored action
+     * @throws InvalidArgumentException If $action is not a recurring action.
+     */
+    public function repeat( $action ) {
+        $schedule = $action->get_schedule();
+        $next     = $schedule->get_next( as_get_datetime_object() );
 
-		if ( is_null( $next ) || ! $schedule->is_recurring() ) {
-			throw new InvalidArgumentException( __( 'Invalid action - must be a recurring action.', 'action-scheduler' ) );
-		}
+        if ( is_null( $next ) || ! $schedule->is_recurring() ) {
+            throw new InvalidArgumentException( __( 'Invalid action - must be a recurring action.', 'action-scheduler' ) );
+        }
 
-		$schedule_class = get_class( $schedule );
-		$new_schedule = new $schedule( $next, $schedule->get_recurrence(), $schedule->get_first_date() );
-		$new_action = new ActionScheduler_Action( $action->get_hook(), $action->get_args(), $new_schedule, $action->get_group() );
-		return $this->store( $new_action );
-	}
+        $schedule_class = get_class( $schedule );
+        $new_schedule = new $schedule( $next, $schedule->get_recurrence(), $schedule->get_first_date() );
+        $new_action = new ActionScheduler_Action( $action->get_hook(), $action->get_args(), $new_schedule, $action->get_group() );
+        return $this->store( $new_action );
+    }
 
-	/**
-	 * @param ActionScheduler_Action $action
-	 *
-	 * @return string The ID of the stored action
-	 */
-	protected function store( ActionScheduler_Action $action ) {
-		$store = ActionScheduler_Store::instance();
-		return $store->save_action( $action );
-	}
+    /**
+     * @param ActionScheduler_Action $action
+     *
+     * @return string The ID of the stored action
+     */
+    protected function store( ActionScheduler_Action $action ) {
+        $store = ActionScheduler_Store::instance();
+        return $store->save_action( $action );
+    }
 }

Spacing +40 added lines, -40 removed lines

@@ -14,16 +14,16 @@ 
 	 *
 	 * @return ActionScheduler_Action An instance of the stored action
 	 */
-	public function get_stored_action( $status, $hook, array $args = array(), ActionScheduler_Schedule $schedule = null, $group = '' ) {
+	public function get_stored_action($status, $hook, array $args = array(), ActionScheduler_Schedule $schedule = null, $group = '') {
 
-		switch ( $status ) {
+		switch ($status) {
 			case ActionScheduler_Store::STATUS_PENDING :
 				$action_class = 'ActionScheduler_Action';
 				break;
 			case ActionScheduler_Store::STATUS_CANCELED :
 				$action_class = 'ActionScheduler_CanceledAction';
-				if ( ! is_null( $schedule ) && ! is_a( $schedule, 'ActionScheduler_CanceledSchedule' ) ) {
-					$schedule = new ActionScheduler_CanceledSchedule( $schedule->get_date() );
+				if (!is_null($schedule) && !is_a($schedule, 'ActionScheduler_CanceledSchedule')) {
+					$schedule = new ActionScheduler_CanceledSchedule($schedule->get_date());
 				}
 				break;
 			default :
@@ -31,9 +31,9 @@ 
 				break;
 		}
 
-		$action_class = apply_filters( 'action_scheduler_stored_action_class', $action_class, $status, $hook, $args, $schedule, $group );
+		$action_class = apply_filters('action_scheduler_stored_action_class', $action_class, $status, $hook, $args, $schedule, $group);
 
-		$action = new $action_class( $hook, $args, $schedule, $group );
+		$action = new $action_class($hook, $args, $schedule, $group);
 
 		/**
 		 * Allow 3rd party code to change the instantiated action for a given hook, args, schedule and group.
@@ -44,7 +44,7 @@ 
 		 * @param ActionScheduler_Schedule $schedule The instantiated action's schedule.
 		 * @param string $group The instantiated action's group.
 		 */
-		return apply_filters( 'action_scheduler_stored_action_instance', $action, $hook, $args, $schedule, $group );
+		return apply_filters('action_scheduler_stored_action_instance', $action, $hook, $args, $schedule, $group);
 	}
 
 	/**
@@ -63,10 +63,10 @@ 
 	 *
 	 * @return string The ID of the stored action
 	 */
-	public function async( $hook, $args = array(), $group = '' ) {
+	public function async($hook, $args = array(), $group = '') {
 		$schedule = new ActionScheduler_NullSchedule();
-		$action = new ActionScheduler_Action( $hook, $args, $schedule, $group );
-		return $this->store( $action );
+		$action = new ActionScheduler_Action($hook, $args, $schedule, $group);
+		return $this->store($action);
 	}
 
 	/**
@@ -77,11 +77,11 @@ 
 	 *
 	 * @return string The ID of the stored action
 	 */
-	public function single( $hook, $args = array(), $when = null, $group = '' ) {
-		$date = as_get_datetime_object( $when );
-		$schedule = new ActionScheduler_SimpleSchedule( $date );
-		$action = new ActionScheduler_Action( $hook, $args, $schedule, $group );
-		return $this->store( $action );
+	public function single($hook, $args = array(), $when = null, $group = '') {
+		$date = as_get_datetime_object($when);
+		$schedule = new ActionScheduler_SimpleSchedule($date);
+		$action = new ActionScheduler_Action($hook, $args, $schedule, $group);
+		return $this->store($action);
 	}
 
 	/**
@@ -95,14 +95,14 @@ 
 	 *
 	 * @return string The ID of the stored action
 	 */
-	public function recurring( $hook, $args = array(), $first = null, $interval = null, $group = '' ) {
-		if ( empty($interval) ) {
-			return $this->single( $hook, $args, $first, $group );
+	public function recurring($hook, $args = array(), $first = null, $interval = null, $group = '') {
+		if (empty($interval)) {
+			return $this->single($hook, $args, $first, $group);
 		}
-		$date = as_get_datetime_object( $first );
-		$schedule = new ActionScheduler_IntervalSchedule( $date, $interval );
-		$action = new ActionScheduler_Action( $hook, $args, $schedule, $group );
-		return $this->store( $action );
+		$date = as_get_datetime_object($first);
+		$schedule = new ActionScheduler_IntervalSchedule($date, $interval);
+		$action = new ActionScheduler_Action($hook, $args, $schedule, $group);
+		return $this->store($action);
 	}
 
 	/**
@@ -118,15 +118,15 @@ 
 	 *
 	 * @return string The ID of the stored action
 	 */
-	public function cron( $hook, $args = array(), $base_timestamp = null, $schedule = null, $group = '' ) {
-		if ( empty($schedule) ) {
-			return $this->single( $hook, $args, $base_timestamp, $group );
+	public function cron($hook, $args = array(), $base_timestamp = null, $schedule = null, $group = '') {
+		if (empty($schedule)) {
+			return $this->single($hook, $args, $base_timestamp, $group);
 		}
-		$date = as_get_datetime_object( $base_timestamp );
-		$cron = CronExpression::factory( $schedule );
-		$schedule = new ActionScheduler_CronSchedule( $date, $cron );
-		$action = new ActionScheduler_Action( $hook, $args, $schedule, $group );
-		return $this->store( $action );
+		$date = as_get_datetime_object($base_timestamp);
+		$cron = CronExpression::factory($schedule);
+		$schedule = new ActionScheduler_CronSchedule($date, $cron);
+		$action = new ActionScheduler_Action($hook, $args, $schedule, $group);
+		return $this->store($action);
 	}
 
 	/**
@@ -153,18 +153,18 @@ 
 	 * @return string The ID of the stored action
 	 * @throws InvalidArgumentException If $action is not a recurring action.
 	 */
-	public function repeat( $action ) {
+	public function repeat($action) {
 		$schedule = $action->get_schedule();
-		$next     = $schedule->get_next( as_get_datetime_object() );
+		$next     = $schedule->get_next(as_get_datetime_object());
 
-		if ( is_null( $next ) || ! $schedule->is_recurring() ) {
-			throw new InvalidArgumentException( __( 'Invalid action - must be a recurring action.', 'action-scheduler' ) );
+		if (is_null($next) || !$schedule->is_recurring()) {
+			throw new InvalidArgumentException(__('Invalid action - must be a recurring action.', 'action-scheduler'));
 		}
 
-		$schedule_class = get_class( $schedule );
-		$new_schedule = new $schedule( $next, $schedule->get_recurrence(), $schedule->get_first_date() );
-		$new_action = new ActionScheduler_Action( $action->get_hook(), $action->get_args(), $new_schedule, $action->get_group() );
-		return $this->store( $new_action );
+		$schedule_class = get_class($schedule);
+		$new_schedule = new $schedule($next, $schedule->get_recurrence(), $schedule->get_first_date());
+		$new_action = new ActionScheduler_Action($action->get_hook(), $action->get_args(), $new_schedule, $action->get_group());
+		return $this->store($new_action);
 	}
 
 	/**
@@ -172,8 +172,8 @@ 
 	 *
 	 * @return string The ID of the stored action
 	 */
-	protected function store( ActionScheduler_Action $action ) {
+	protected function store(ActionScheduler_Action $action) {
 		$store = ActionScheduler_Store::instance();
-		return $store->save_action( $action );
+		return $store->save_action($action);
 	}
 }

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

Indentation +61 added lines, -61 removed lines

@@ -16,71 +16,71 @@
  * @codeCoverageIgnore
  */
 class BatchFetcher {
-	/** var ActionScheduler_Store */
-	private $store;
+    /** var ActionScheduler_Store */
+    private $store;
 
-	/**
-	 * BatchFetcher constructor.
-	 *
-	 * @param ActionScheduler_Store $source_store Source store object.
-	 */
-	public function __construct( Store $source_store ) {
-		$this->store = $source_store;
-	}
+    /**
+     * BatchFetcher constructor.
+     *
+     * @param ActionScheduler_Store $source_store Source store object.
+     */
+    public function __construct( Store $source_store ) {
+        $this->store = $source_store;
+    }
 
-	/**
-	 * Retrieve a list of actions.
-	 *
-	 * @param int $count The number of actions to retrieve
-	 *
-	 * @return int[] A list of action IDs
-	 */
-	public function fetch( $count = 10 ) {
-		foreach ( $this->get_query_strategies( $count ) as $query ) {
-			$action_ids = $this->store->query_actions( $query );
-			if ( ! empty( $action_ids ) ) {
-				return $action_ids;
-			}
-		}
+    /**
+     * Retrieve a list of actions.
+     *
+     * @param int $count The number of actions to retrieve
+     *
+     * @return int[] A list of action IDs
+     */
+    public function fetch( $count = 10 ) {
+        foreach ( $this->get_query_strategies( $count ) as $query ) {
+            $action_ids = $this->store->query_actions( $query );
+            if ( ! empty( $action_ids ) ) {
+                return $action_ids;
+            }
+        }
 
-		return [];
-	}
+        return [];
+    }
 
-	/**
-	 * Generate a list of prioritized of action search parameters.
-	 *
-	 * @param int $count Number of actions to find.
-	 *
-	 * @return array
-	 */
-	private function get_query_strategies( $count ) {
-		$now  = as_get_datetime_object();
-		$args = [
-			'date'     => $now,
-			'per_page' => $count,
-			'offset'   => 0,
-			'orderby'  => 'date',
-			'order'    => 'ASC',
-		];
+    /**
+     * Generate a list of prioritized of action search parameters.
+     *
+     * @param int $count Number of actions to find.
+     *
+     * @return array
+     */
+    private function get_query_strategies( $count ) {
+        $now  = as_get_datetime_object();
+        $args = [
+            'date'     => $now,
+            'per_page' => $count,
+            'offset'   => 0,
+            'orderby'  => 'date',
+            'order'    => 'ASC',
+        ];
 
-		$priorities = [
-			Store::STATUS_PENDING,
-			Store::STATUS_FAILED,
-			Store::STATUS_CANCELED,
-			Store::STATUS_COMPLETE,
-			Store::STATUS_RUNNING,
-			'', // any other unanticipated status
-		];
+        $priorities = [
+            Store::STATUS_PENDING,
+            Store::STATUS_FAILED,
+            Store::STATUS_CANCELED,
+            Store::STATUS_COMPLETE,
+            Store::STATUS_RUNNING,
+            '', // any other unanticipated status
+        ];
 
-		foreach ( $priorities as $status ) {
-			yield wp_parse_args( [
-				'status'       => $status,
-				'date_compare' => '<=',
-			], $args );
-			yield wp_parse_args( [
-				'status'       => $status,
-				'date_compare' => '>=',
-			], $args );
-		}
-	}
+        foreach ( $priorities as $status ) {
+            yield wp_parse_args( [
+                'status'       => $status,
+                'date_compare' => '<=',
+            ], $args );
+            yield wp_parse_args( [
+                'status'       => $status,
+                'date_compare' => '>=',
+            ], $args );
+        }
+    }
 }
\ No newline at end of file

Spacing +11 added lines, -11 removed lines

@@ -24,7 +24,7 @@ 
 	 *
 	 * @param ActionScheduler_Store $source_store Source store object.
 	 */
-	public function __construct( Store $source_store ) {
+	public function __construct(Store $source_store) {
 		$this->store = $source_store;
 	}
 
@@ -35,10 +35,10 @@ 
 	 *
 	 * @return int[] A list of action IDs
 	 */
-	public function fetch( $count = 10 ) {
-		foreach ( $this->get_query_strategies( $count ) as $query ) {
-			$action_ids = $this->store->query_actions( $query );
-			if ( ! empty( $action_ids ) ) {
+	public function fetch($count = 10) {
+		foreach ($this->get_query_strategies($count) as $query) {
+			$action_ids = $this->store->query_actions($query);
+			if (!empty($action_ids)) {
 				return $action_ids;
 			}
 		}
@@ -53,7 +53,7 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_query_strategies( $count ) {
+	private function get_query_strategies($count) {
 		$now  = as_get_datetime_object();
 		$args = [
 			'date'     => $now,
@@ -72,15 +72,15 @@ 
 			'', // any other unanticipated status
 		];
 
-		foreach ( $priorities as $status ) {
-			yield wp_parse_args( [
+		foreach ($priorities as $status) {
+			yield wp_parse_args([
 				'status'       => $status,
 				'date_compare' => '<=',
-			], $args );
-			yield wp_parse_args( [
+			], $args);
+			yield wp_parse_args([
 				'status'       => $status,
 				'date_compare' => '>=',
-			], $args );
+			], $args);
 		}
 	}
 }
\ No newline at end of file

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

Indentation +148 added lines, -148 removed lines

@@ -17,152 +17,152 @@
  * A config builder for the ActionScheduler\Migration\Runner class
  */
 class Config {
-	/** @var ActionScheduler_Store */
-	private $source_store;
-
-	/** @var ActionScheduler_Logger */
-	private $source_logger;
-
-	/** @var ActionScheduler_Store */
-	private $destination_store;
-
-	/** @var ActionScheduler_Logger */
-	private $destination_logger;
-
-	/** @var Progress bar */
-	private $progress_bar;
-
-	/** @var bool */
-	private $dry_run = false;
-
-	/**
-	 * Config constructor.
-	 */
-	public function __construct() {
-
-	}
-
-	/**
-	 * Get the configured source store.
-	 *
-	 * @return ActionScheduler_Store
-	 */
-	public function get_source_store() {
-		if ( empty( $this->source_store ) ) {
-			throw new \RuntimeException( __( 'Source store must be configured before running a migration', 'action-scheduler' ) );
-		}
-
-		return $this->source_store;
-	}
-
-	/**
-	 * Set the configured source store.
-	 *
-	 * @param ActionScheduler_Store $store Source store object.
-	 */
-	public function set_source_store( Store $store ) {
-		$this->source_store = $store;
-	}
-
-	/**
-	 * Get the configured source loger.
-	 *
-	 * @return ActionScheduler_Logger
-	 */
-	public function get_source_logger() {
-		if ( empty( $this->source_logger ) ) {
-			throw new \RuntimeException( __( 'Source logger must be configured before running a migration', 'action-scheduler' ) );
-		}
-
-		return $this->source_logger;
-	}
-
-	/**
-	 * Set the configured source logger.
-	 *
-	 * @param ActionScheduler_Logger $logger
-	 */
-	public function set_source_logger( Logger $logger ) {
-		$this->source_logger = $logger;
-	}
-
-	/**
-	 * Get the configured destination store.
-	 *
-	 * @return ActionScheduler_Store
-	 */
-	public function get_destination_store() {
-		if ( empty( $this->destination_store ) ) {
-			throw new \RuntimeException( __( 'Destination store must be configured before running a migration', 'action-scheduler' ) );
-		}
-
-		return $this->destination_store;
-	}
-
-	/**
-	 * Set the configured destination store.
-	 *
-	 * @param ActionScheduler_Store $store
-	 */
-	public function set_destination_store( Store $store ) {
-		$this->destination_store = $store;
-	}
-
-	/**
-	 * Get the configured destination logger.
-	 *
-	 * @return ActionScheduler_Logger
-	 */
-	public function get_destination_logger() {
-		if ( empty( $this->destination_logger ) ) {
-			throw new \RuntimeException( __( 'Destination logger must be configured before running a migration', 'action-scheduler' ) );
-		}
-
-		return $this->destination_logger;
-	}
-
-	/**
-	 * Set the configured destination logger.
-	 *
-	 * @param ActionScheduler_Logger $logger
-	 */
-	public function set_destination_logger( Logger $logger ) {
-		$this->destination_logger = $logger;
-	}
-
-	/**
-	 * Get flag indicating whether it's a dry run.
-	 *
-	 * @return bool
-	 */
-	public function get_dry_run() {
-		return $this->dry_run;
-	}
-
-	/**
-	 * Set flag indicating whether it's a dry run.
-	 *
-	 * @param bool $dry_run
-	 */
-	public function set_dry_run( $dry_run ) {
-		$this->dry_run = (bool) $dry_run;
-	}
-
-	/**
-	 * Get progress bar object.
-	 *
-	 * @return ActionScheduler\WPCLI\ProgressBar
-	 */
-	public function get_progress_bar() {
-		return $this->progress_bar;
-	}
-
-	/**
-	 * Set progress bar object.
-	 *
-	 * @param ActionScheduler\WPCLI\ProgressBar $progress_bar
-	 */
-	public function set_progress_bar( ProgressBar $progress_bar ) {
-		$this->progress_bar = $progress_bar;
-	}
+    /** @var ActionScheduler_Store */
+    private $source_store;
+
+    /** @var ActionScheduler_Logger */
+    private $source_logger;
+
+    /** @var ActionScheduler_Store */
+    private $destination_store;
+
+    /** @var ActionScheduler_Logger */
+    private $destination_logger;
+
+    /** @var Progress bar */
+    private $progress_bar;
+
+    /** @var bool */
+    private $dry_run = false;
+
+    /**
+     * Config constructor.
+     */
+    public function __construct() {
+
+    }
+
+    /**
+     * Get the configured source store.
+     *
+     * @return ActionScheduler_Store
+     */
+    public function get_source_store() {
+        if ( empty( $this->source_store ) ) {
+            throw new \RuntimeException( __( 'Source store must be configured before running a migration', 'action-scheduler' ) );
+        }
+
+        return $this->source_store;
+    }
+
+    /**
+     * Set the configured source store.
+     *
+     * @param ActionScheduler_Store $store Source store object.
+     */
+    public function set_source_store( Store $store ) {
+        $this->source_store = $store;
+    }
+
+    /**
+     * Get the configured source loger.
+     *
+     * @return ActionScheduler_Logger
+     */
+    public function get_source_logger() {
+        if ( empty( $this->source_logger ) ) {
+            throw new \RuntimeException( __( 'Source logger must be configured before running a migration', 'action-scheduler' ) );
+        }
+
+        return $this->source_logger;
+    }
+
+    /**
+     * Set the configured source logger.
+     *
+     * @param ActionScheduler_Logger $logger
+     */
+    public function set_source_logger( Logger $logger ) {
+        $this->source_logger = $logger;
+    }
+
+    /**
+     * Get the configured destination store.
+     *
+     * @return ActionScheduler_Store
+     */
+    public function get_destination_store() {
+        if ( empty( $this->destination_store ) ) {
+            throw new \RuntimeException( __( 'Destination store must be configured before running a migration', 'action-scheduler' ) );
+        }
+
+        return $this->destination_store;
+    }
+
+    /**
+     * Set the configured destination store.
+     *
+     * @param ActionScheduler_Store $store
+     */
+    public function set_destination_store( Store $store ) {
+        $this->destination_store = $store;
+    }
+
+    /**
+     * Get the configured destination logger.
+     *
+     * @return ActionScheduler_Logger
+     */
+    public function get_destination_logger() {
+        if ( empty( $this->destination_logger ) ) {
+            throw new \RuntimeException( __( 'Destination logger must be configured before running a migration', 'action-scheduler' ) );
+        }
+
+        return $this->destination_logger;
+    }
+
+    /**
+     * Set the configured destination logger.
+     *
+     * @param ActionScheduler_Logger $logger
+     */
+    public function set_destination_logger( Logger $logger ) {
+        $this->destination_logger = $logger;
+    }
+
+    /**
+     * Get flag indicating whether it's a dry run.
+     *
+     * @return bool
+     */
+    public function get_dry_run() {
+        return $this->dry_run;
+    }
+
+    /**
+     * Set flag indicating whether it's a dry run.
+     *
+     * @param bool $dry_run
+     */
+    public function set_dry_run( $dry_run ) {
+        $this->dry_run = (bool) $dry_run;
+    }
+
+    /**
+     * Get progress bar object.
+     *
+     * @return ActionScheduler\WPCLI\ProgressBar
+     */
+    public function get_progress_bar() {
+        return $this->progress_bar;
+    }
+
+    /**
+     * Set progress bar object.
+     *
+     * @param ActionScheduler\WPCLI\ProgressBar $progress_bar
+     */
+    public function set_progress_bar( ProgressBar $progress_bar ) {
+        $this->progress_bar = $progress_bar;
+    }
 }

Spacing +14 added lines, -14 removed lines

@@ -48,8 +48,8 @@ 
 	 * @return ActionScheduler_Store
 	 */
 	public function get_source_store() {
-		if ( empty( $this->source_store ) ) {
-			throw new \RuntimeException( __( 'Source store must be configured before running a migration', 'action-scheduler' ) );
+		if (empty($this->source_store)) {
+			throw new \RuntimeException(__('Source store must be configured before running a migration', 'action-scheduler'));
 		}
 
 		return $this->source_store;
@@ -60,7 +60,7 @@ 
 	 *
 	 * @param ActionScheduler_Store $store Source store object.
 	 */
-	public function set_source_store( Store $store ) {
+	public function set_source_store(Store $store) {
 		$this->source_store = $store;
 	}
 
@@ -70,8 +70,8 @@ 
 	 * @return ActionScheduler_Logger
 	 */
 	public function get_source_logger() {
-		if ( empty( $this->source_logger ) ) {
-			throw new \RuntimeException( __( 'Source logger must be configured before running a migration', 'action-scheduler' ) );
+		if (empty($this->source_logger)) {
+			throw new \RuntimeException(__('Source logger must be configured before running a migration', 'action-scheduler'));
 		}
 
 		return $this->source_logger;
@@ -82,7 +82,7 @@ 
 	 *
 	 * @param ActionScheduler_Logger $logger
 	 */
-	public function set_source_logger( Logger $logger ) {
+	public function set_source_logger(Logger $logger) {
 		$this->source_logger = $logger;
 	}
 
@@ -92,8 +92,8 @@ 
 	 * @return ActionScheduler_Store
 	 */
 	public function get_destination_store() {
-		if ( empty( $this->destination_store ) ) {
-			throw new \RuntimeException( __( 'Destination store must be configured before running a migration', 'action-scheduler' ) );
+		if (empty($this->destination_store)) {
+			throw new \RuntimeException(__('Destination store must be configured before running a migration', 'action-scheduler'));
 		}
 
 		return $this->destination_store;
@@ -104,7 +104,7 @@ 
 	 *
 	 * @param ActionScheduler_Store $store
 	 */
-	public function set_destination_store( Store $store ) {
+	public function set_destination_store(Store $store) {
 		$this->destination_store = $store;
 	}
 
@@ -114,8 +114,8 @@ 
 	 * @return ActionScheduler_Logger
 	 */
 	public function get_destination_logger() {
-		if ( empty( $this->destination_logger ) ) {
-			throw new \RuntimeException( __( 'Destination logger must be configured before running a migration', 'action-scheduler' ) );
+		if (empty($this->destination_logger)) {
+			throw new \RuntimeException(__('Destination logger must be configured before running a migration', 'action-scheduler'));
 		}
 
 		return $this->destination_logger;
@@ -126,7 +126,7 @@ 
 	 *
 	 * @param ActionScheduler_Logger $logger
 	 */
-	public function set_destination_logger( Logger $logger ) {
+	public function set_destination_logger(Logger $logger) {
 		$this->destination_logger = $logger;
 	}
 
@@ -144,7 +144,7 @@ 
 	 *
 	 * @param bool $dry_run
 	 */
-	public function set_dry_run( $dry_run ) {
+	public function set_dry_run($dry_run) {
 		$this->dry_run = (bool) $dry_run;
 	}
 
@@ -162,7 +162,7 @@ 
 	 *
 	 * @param ActionScheduler\WPCLI\ProgressBar $progress_bar
 	 */
-	public function set_progress_bar( ProgressBar $progress_bar ) {
+	public function set_progress_bar(ProgressBar $progress_bar) {
 		$this->progress_bar = $progress_bar;
 	}
 }

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

Indentation +120 added lines, -120 removed lines

@@ -13,124 +13,124 @@
  * @codeCoverageIgnore
  */
 class Runner {
-	/** @var ActionScheduler_Store */
-	private $source_store;
-
-	/** @var ActionScheduler_Store */
-	private $destination_store;
-
-	/** @var ActionScheduler_Logger */
-	private $source_logger;
-
-	/** @var ActionScheduler_Logger */
-	private $destination_logger;
-
-	/** @var BatchFetcher */
-	private $batch_fetcher;
-
-	/** @var ActionMigrator */
-	private $action_migrator;
-
-	/** @var LogMigrator */
-	private $log_migrator;
-
-	/** @var ProgressBar */
-	private $progress_bar;
-
-	/**
-	 * Runner constructor.
-	 *
-	 * @param Config $config Migration configuration object.
-	 */
-	public function __construct( Config $config ) {
-		$this->source_store       = $config->get_source_store();
-		$this->destination_store  = $config->get_destination_store();
-		$this->source_logger      = $config->get_source_logger();
-		$this->destination_logger = $config->get_destination_logger();
-
-		$this->batch_fetcher = new BatchFetcher( $this->source_store );
-		if ( $config->get_dry_run() ) {
-			$this->log_migrator    = new DryRun_LogMigrator( $this->source_logger, $this->destination_logger );
-			$this->action_migrator = new DryRun_ActionMigrator( $this->source_store, $this->destination_store, $this->log_migrator );
-		} else {
-			$this->log_migrator    = new LogMigrator( $this->source_logger, $this->destination_logger );
-			$this->action_migrator = new ActionMigrator( $this->source_store, $this->destination_store, $this->log_migrator );
-		}
-
-		if ( defined( 'WP_CLI' ) && WP_CLI ) {
-			$this->progress_bar = $config->get_progress_bar();
-		}
-	}
-
-	/**
-	 * Run migration batch.
-	 *
-	 * @param int $batch_size Optional batch size. Default 10.
-	 *
-	 * @return int Size of batch processed.
-	 */
-	public function run( $batch_size = 10 ) {
-		$batch = $this->batch_fetcher->fetch( $batch_size );
-		$batch_size = count( $batch );
-
-		if ( ! $batch_size ) {
-			return 0;
-		}
-
-		if ( $this->progress_bar ) {
-			/* translators: %d: amount of actions */
-			$this->progress_bar->set_message( sprintf( _n( 'Migrating %d action', 'Migrating %d actions', $batch_size, 'action-scheduler' ), number_format_i18n( $batch_size ) ) );
-			$this->progress_bar->set_count( $batch_size );
-		}
-
-		$this->migrate_actions( $batch );
-
-		return $batch_size;
-	}
-
-	/**
-	 * Migration a batch of actions.
-	 *
-	 * @param array $action_ids List of action IDs to migrate.
-	 */
-	public function migrate_actions( array $action_ids ) {
-		do_action( 'action_scheduler/migration_batch_starting', $action_ids );
-
-		\ActionScheduler::logger()->unhook_stored_action();
-		$this->destination_logger->unhook_stored_action();
-
-		foreach ( $action_ids as $source_action_id ) {
-			$destination_action_id = $this->action_migrator->migrate( $source_action_id );
-			if ( $destination_action_id ) {
-				$this->destination_logger->log( $destination_action_id, sprintf(
-					/* translators: 1: source action ID 2: source store class 3: destination action ID 4: destination store class */
-					__( 'Migrated action with ID %1$d in %2$s to ID %3$d in %4$s', 'action-scheduler' ),
-					$source_action_id,
-					get_class( $this->source_store ),
-					$destination_action_id,
-					get_class( $this->destination_store )
-				) );
-			}
-
-			if ( $this->progress_bar ) {
-				$this->progress_bar->tick();
-			}
-		}
-
-		if ( $this->progress_bar ) {
-			$this->progress_bar->finish();
-		}
-
-		\ActionScheduler::logger()->hook_stored_action();
-
-		do_action( 'action_scheduler/migration_batch_complete', $action_ids );
-	}
-
-	/**
-	 * Initialize destination store and logger.
-	 */
-	public function init_destination() {
-		$this->destination_store->init();
-		$this->destination_logger->init();
-	}
+    /** @var ActionScheduler_Store */
+    private $source_store;
+
+    /** @var ActionScheduler_Store */
+    private $destination_store;
+
+    /** @var ActionScheduler_Logger */
+    private $source_logger;
+
+    /** @var ActionScheduler_Logger */
+    private $destination_logger;
+
+    /** @var BatchFetcher */
+    private $batch_fetcher;
+
+    /** @var ActionMigrator */
+    private $action_migrator;
+
+    /** @var LogMigrator */
+    private $log_migrator;
+
+    /** @var ProgressBar */
+    private $progress_bar;
+
+    /**
+     * Runner constructor.
+     *
+     * @param Config $config Migration configuration object.
+     */
+    public function __construct( Config $config ) {
+        $this->source_store       = $config->get_source_store();
+        $this->destination_store  = $config->get_destination_store();
+        $this->source_logger      = $config->get_source_logger();
+        $this->destination_logger = $config->get_destination_logger();
+
+        $this->batch_fetcher = new BatchFetcher( $this->source_store );
+        if ( $config->get_dry_run() ) {
+            $this->log_migrator    = new DryRun_LogMigrator( $this->source_logger, $this->destination_logger );
+            $this->action_migrator = new DryRun_ActionMigrator( $this->source_store, $this->destination_store, $this->log_migrator );
+        } else {
+            $this->log_migrator    = new LogMigrator( $this->source_logger, $this->destination_logger );
+            $this->action_migrator = new ActionMigrator( $this->source_store, $this->destination_store, $this->log_migrator );
+        }
+
+        if ( defined( 'WP_CLI' ) && WP_CLI ) {
+            $this->progress_bar = $config->get_progress_bar();
+        }
+    }
+
+    /**
+     * Run migration batch.
+     *
+     * @param int $batch_size Optional batch size. Default 10.
+     *
+     * @return int Size of batch processed.
+     */
+    public function run( $batch_size = 10 ) {
+        $batch = $this->batch_fetcher->fetch( $batch_size );
+        $batch_size = count( $batch );
+
+        if ( ! $batch_size ) {
+            return 0;
+        }
+
+        if ( $this->progress_bar ) {
+            /* translators: %d: amount of actions */
+            $this->progress_bar->set_message( sprintf( _n( 'Migrating %d action', 'Migrating %d actions', $batch_size, 'action-scheduler' ), number_format_i18n( $batch_size ) ) );
+            $this->progress_bar->set_count( $batch_size );
+        }
+
+        $this->migrate_actions( $batch );
+
+        return $batch_size;
+    }
+
+    /**
+     * Migration a batch of actions.
+     *
+     * @param array $action_ids List of action IDs to migrate.
+     */
+    public function migrate_actions( array $action_ids ) {
+        do_action( 'action_scheduler/migration_batch_starting', $action_ids );
+
+        \ActionScheduler::logger()->unhook_stored_action();
+        $this->destination_logger->unhook_stored_action();
+
+        foreach ( $action_ids as $source_action_id ) {
+            $destination_action_id = $this->action_migrator->migrate( $source_action_id );
+            if ( $destination_action_id ) {
+                $this->destination_logger->log( $destination_action_id, sprintf(
+                    /* translators: 1: source action ID 2: source store class 3: destination action ID 4: destination store class */
+                    __( 'Migrated action with ID %1$d in %2$s to ID %3$d in %4$s', 'action-scheduler' ),
+                    $source_action_id,
+                    get_class( $this->source_store ),
+                    $destination_action_id,
+                    get_class( $this->destination_store )
+                ) );
+            }
+
+            if ( $this->progress_bar ) {
+                $this->progress_bar->tick();
+            }
+        }
+
+        if ( $this->progress_bar ) {
+            $this->progress_bar->finish();
+        }
+
+        \ActionScheduler::logger()->hook_stored_action();
+
+        do_action( 'action_scheduler/migration_batch_complete', $action_ids );
+    }
+
+    /**
+     * Initialize destination store and logger.
+     */
+    public function init_destination() {
+        $this->destination_store->init();
+        $this->destination_logger->init();
+    }
 }

Spacing +29 added lines, -29 removed lines

@@ -42,22 +42,22 @@ 
 	 *
 	 * @param Config $config Migration configuration object.
 	 */
-	public function __construct( Config $config ) {
+	public function __construct(Config $config) {
 		$this->source_store       = $config->get_source_store();
 		$this->destination_store  = $config->get_destination_store();
 		$this->source_logger      = $config->get_source_logger();
 		$this->destination_logger = $config->get_destination_logger();
 
-		$this->batch_fetcher = new BatchFetcher( $this->source_store );
-		if ( $config->get_dry_run() ) {
-			$this->log_migrator    = new DryRun_LogMigrator( $this->source_logger, $this->destination_logger );
-			$this->action_migrator = new DryRun_ActionMigrator( $this->source_store, $this->destination_store, $this->log_migrator );
+		$this->batch_fetcher = new BatchFetcher($this->source_store);
+		if ($config->get_dry_run()) {
+			$this->log_migrator    = new DryRun_LogMigrator($this->source_logger, $this->destination_logger);
+			$this->action_migrator = new DryRun_ActionMigrator($this->source_store, $this->destination_store, $this->log_migrator);
 		} else {
-			$this->log_migrator    = new LogMigrator( $this->source_logger, $this->destination_logger );
-			$this->action_migrator = new ActionMigrator( $this->source_store, $this->destination_store, $this->log_migrator );
+			$this->log_migrator    = new LogMigrator($this->source_logger, $this->destination_logger);
+			$this->action_migrator = new ActionMigrator($this->source_store, $this->destination_store, $this->log_migrator);
 		}
 
-		if ( defined( 'WP_CLI' ) && WP_CLI ) {
+		if (defined('WP_CLI') && WP_CLI) {
 			$this->progress_bar = $config->get_progress_bar();
 		}
 	}
@@ -69,21 +69,21 @@ 
 	 *
 	 * @return int Size of batch processed.
 	 */
-	public function run( $batch_size = 10 ) {
-		$batch = $this->batch_fetcher->fetch( $batch_size );
-		$batch_size = count( $batch );
+	public function run($batch_size = 10) {
+		$batch = $this->batch_fetcher->fetch($batch_size);
+		$batch_size = count($batch);
 
-		if ( ! $batch_size ) {
+		if (!$batch_size) {
 			return 0;
 		}
 
-		if ( $this->progress_bar ) {
+		if ($this->progress_bar) {
 			/* translators: %d: amount of actions */
-			$this->progress_bar->set_message( sprintf( _n( 'Migrating %d action', 'Migrating %d actions', $batch_size, 'action-scheduler' ), number_format_i18n( $batch_size ) ) );
-			$this->progress_bar->set_count( $batch_size );
+			$this->progress_bar->set_message(sprintf(_n('Migrating %d action', 'Migrating %d actions', $batch_size, 'action-scheduler'), number_format_i18n($batch_size)));
+			$this->progress_bar->set_count($batch_size);
 		}
 
-		$this->migrate_actions( $batch );
+		$this->migrate_actions($batch);
 
 		return $batch_size;
 	}
@@ -93,37 +93,37 @@ 
 	 *
 	 * @param array $action_ids List of action IDs to migrate.
 	 */
-	public function migrate_actions( array $action_ids ) {
-		do_action( 'action_scheduler/migration_batch_starting', $action_ids );
+	public function migrate_actions(array $action_ids) {
+		do_action('action_scheduler/migration_batch_starting', $action_ids);
 
 		\ActionScheduler::logger()->unhook_stored_action();
 		$this->destination_logger->unhook_stored_action();
 
-		foreach ( $action_ids as $source_action_id ) {
-			$destination_action_id = $this->action_migrator->migrate( $source_action_id );
-			if ( $destination_action_id ) {
-				$this->destination_logger->log( $destination_action_id, sprintf(
+		foreach ($action_ids as $source_action_id) {
+			$destination_action_id = $this->action_migrator->migrate($source_action_id);
+			if ($destination_action_id) {
+				$this->destination_logger->log($destination_action_id, sprintf(
 					/* translators: 1: source action ID 2: source store class 3: destination action ID 4: destination store class */
-					__( 'Migrated action with ID %1$d in %2$s to ID %3$d in %4$s', 'action-scheduler' ),
+					__('Migrated action with ID %1$d in %2$s to ID %3$d in %4$s', 'action-scheduler'),
 					$source_action_id,
-					get_class( $this->source_store ),
+					get_class($this->source_store),
 					$destination_action_id,
-					get_class( $this->destination_store )
-				) );
+					get_class($this->destination_store)
+				));
 			}
 
-			if ( $this->progress_bar ) {
+			if ($this->progress_bar) {
 				$this->progress_bar->tick();
 			}
 		}
 
-		if ( $this->progress_bar ) {
+		if ($this->progress_bar) {
 			$this->progress_bar->finish();
 		}
 
 		\ActionScheduler::logger()->hook_stored_action();
 
-		do_action( 'action_scheduler/migration_batch_complete', $action_ids );
+		do_action('action_scheduler/migration_batch_complete', $action_ids);
 	}
 
 	/**

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

Indentation +28 added lines, -28 removed lines

@@ -15,35 +15,35 @@
  * @codeCoverageIgnore
  */
 class LogMigrator {
-	/** @var ActionScheduler_Logger */
-	private $source;
+    /** @var ActionScheduler_Logger */
+    private $source;
 
-	/** @var ActionScheduler_Logger */
-	private $destination;
+    /** @var ActionScheduler_Logger */
+    private $destination;
 
-	/**
-	 * ActionMigrator constructor.
-	 *
-	 * @param ActionScheduler_Logger $source_logger Source logger object.
-	 * @param ActionScheduler_Logger $destination_Logger Destination logger object.
-	 */
-	public function __construct( ActionScheduler_Logger $source_logger, ActionScheduler_Logger $destination_Logger ) {
-		$this->source      = $source_logger;
-		$this->destination = $destination_Logger;
-	}
+    /**
+     * ActionMigrator constructor.
+     *
+     * @param ActionScheduler_Logger $source_logger Source logger object.
+     * @param ActionScheduler_Logger $destination_Logger Destination logger object.
+     */
+    public function __construct( ActionScheduler_Logger $source_logger, ActionScheduler_Logger $destination_Logger ) {
+        $this->source      = $source_logger;
+        $this->destination = $destination_Logger;
+    }
 
-	/**
-	 * Migrate an action log.
-	 *
-	 * @param int $source_action_id Source logger object.
-	 * @param int $destination_action_id Destination logger object.
-	 */
-	public function migrate( $source_action_id, $destination_action_id ) {
-		$logs = $this->source->get_logs( $source_action_id );
-		foreach ( $logs as $log ) {
-			if ( $log->get_action_id() == $source_action_id ) {
-				$this->destination->log( $destination_action_id, $log->get_message(), $log->get_date() );
-			}
-		}
-	}
+    /**
+     * Migrate an action log.
+     *
+     * @param int $source_action_id Source logger object.
+     * @param int $destination_action_id Destination logger object.
+     */
+    public function migrate( $source_action_id, $destination_action_id ) {
+        $logs = $this->source->get_logs( $source_action_id );
+        foreach ( $logs as $log ) {
+            if ( $log->get_action_id() == $source_action_id ) {
+                $this->destination->log( $destination_action_id, $log->get_message(), $log->get_date() );
+            }
+        }
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -27,7 +27,7 @@ 
 	 * @param ActionScheduler_Logger $source_logger Source logger object.
 	 * @param ActionScheduler_Logger $destination_Logger Destination logger object.
 	 */
-	public function __construct( ActionScheduler_Logger $source_logger, ActionScheduler_Logger $destination_Logger ) {
+	public function __construct(ActionScheduler_Logger $source_logger, ActionScheduler_Logger $destination_Logger) {
 		$this->source      = $source_logger;
 		$this->destination = $destination_Logger;
 	}
@@ -38,11 +38,11 @@ 
 	 * @param int $source_action_id Source logger object.
 	 * @param int $destination_action_id Destination logger object.
 	 */
-	public function migrate( $source_action_id, $destination_action_id ) {
-		$logs = $this->source->get_logs( $source_action_id );
-		foreach ( $logs as $log ) {
-			if ( $log->get_action_id() == $source_action_id ) {
-				$this->destination->log( $destination_action_id, $log->get_message(), $log->get_date() );
+	public function migrate($source_action_id, $destination_action_id) {
+		$logs = $this->source->get_logs($source_action_id);
+		foreach ($logs as $log) {
+			if ($log->get_action_id() == $source_action_id) {
+				$this->destination->log($destination_action_id, $log->get_message(), $log->get_date());
 			}
 		}
 	}

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

Indentation +111 added lines, -111 removed lines

@@ -13,116 +13,116 @@
  * @codeCoverageIgnore
  */
 class Scheduler {
-	/** Migration action hook. */
-	const HOOK            = 'action_scheduler/migration_hook';
-
-	/** Migration action group. */
-	const GROUP           = 'action-scheduler-migration';
-
-	/**
-	 * Set up the callback for the scheduled job.
-	 */
-	public function hook() {
-		add_action( self::HOOK, array( $this, 'run_migration' ), 10, 0 );
-	}
-
-	/**
-	 * Remove the callback for the scheduled job.
-	 */
-	public function unhook() {
-		remove_action( self::HOOK, array( $this, 'run_migration' ), 10 );
-	}
-
-	/**
-	 * The migration callback.
-	 */
-	public function run_migration() {
-		$migration_runner = $this->get_migration_runner();
-		$count            = $migration_runner->run( $this->get_batch_size() );
-
-		if ( $count === 0 ) {
-			$this->mark_complete();
-		} else {
-			$this->schedule_migration( time() + $this->get_schedule_interval() );
-		}
-	}
-
-	/**
-	 * Mark the migration complete.
-	 */
-	public function mark_complete() {
-		$this->unschedule_migration();
-
-		\ActionScheduler_DataController::mark_migration_complete();
-		do_action( 'action_scheduler/migration_complete' );
-	}
-
-	/**
-	 * Get a flag indicating whether the migration is scheduled.
-	 *
-	 * @return bool Whether there is a pending action in the store to handle the migration
-	 */
-	public function is_migration_scheduled() {
-		$next = as_next_scheduled_action( self::HOOK );
-
-		return ! empty( $next );
-	}
-
-	/**
-	 * Schedule the migration.
-	 *
-	 * @param int $when Optional timestamp to run the next migration batch. Defaults to now.
-	 *
-	 * @return string The action ID
-	 */
-	public function schedule_migration( $when = 0 ) {
-		$next = as_next_scheduled_action( self::HOOK );
-
-		if ( ! empty( $next ) ) {
-			return $next;
-		}
-
-		if ( empty( $when ) ) {
-			$when = time();
-		}
-
-		return as_schedule_single_action( $when, self::HOOK, array(), self::GROUP );
-	}
-
-	/**
-	 * Remove the scheduled migration action.
-	 */
-	public function unschedule_migration() {
-		as_unschedule_action( self::HOOK, null, self::GROUP );
-	}
-
-	/**
-	 * Get migration batch schedule interval.
-	 *
-	 * @return int Seconds between migration runs. Defaults to 0 seconds to allow chaining migration via Async Runners.
-	 */
-	private function get_schedule_interval() {
-		return (int) apply_filters( 'action_scheduler/migration_interval', 0 );
-	}
-
-	/**
-	 * Get migration batch size.
-	 *
-	 * @return int Number of actions to migrate in each batch. Defaults to 250.
-	 */
-	private function get_batch_size() {
-		return (int) apply_filters( 'action_scheduler/migration_batch_size', 250 );
-	}
-
-	/**
-	 * Get migration runner object.
-	 *
-	 * @return Runner
-	 */
-	private function get_migration_runner() {
-		$config = Controller::instance()->get_migration_config_object();
-
-		return new Runner( $config );
-	}
+    /** Migration action hook. */
+    const HOOK            = 'action_scheduler/migration_hook';
+
+    /** Migration action group. */
+    const GROUP           = 'action-scheduler-migration';
+
+    /**
+     * Set up the callback for the scheduled job.
+     */
+    public function hook() {
+        add_action( self::HOOK, array( $this, 'run_migration' ), 10, 0 );
+    }
+
+    /**
+     * Remove the callback for the scheduled job.
+     */
+    public function unhook() {
+        remove_action( self::HOOK, array( $this, 'run_migration' ), 10 );
+    }
+
+    /**
+     * The migration callback.
+     */
+    public function run_migration() {
+        $migration_runner = $this->get_migration_runner();
+        $count            = $migration_runner->run( $this->get_batch_size() );
+
+        if ( $count === 0 ) {
+            $this->mark_complete();
+        } else {
+            $this->schedule_migration( time() + $this->get_schedule_interval() );
+        }
+    }
+
+    /**
+     * Mark the migration complete.
+     */
+    public function mark_complete() {
+        $this->unschedule_migration();
+
+        \ActionScheduler_DataController::mark_migration_complete();
+        do_action( 'action_scheduler/migration_complete' );
+    }
+
+    /**
+     * Get a flag indicating whether the migration is scheduled.
+     *
+     * @return bool Whether there is a pending action in the store to handle the migration
+     */
+    public function is_migration_scheduled() {
+        $next = as_next_scheduled_action( self::HOOK );
+
+        return ! empty( $next );
+    }
+
+    /**
+     * Schedule the migration.
+     *
+     * @param int $when Optional timestamp to run the next migration batch. Defaults to now.
+     *
+     * @return string The action ID
+     */
+    public function schedule_migration( $when = 0 ) {
+        $next = as_next_scheduled_action( self::HOOK );
+
+        if ( ! empty( $next ) ) {
+            return $next;
+        }
+
+        if ( empty( $when ) ) {
+            $when = time();
+        }
+
+        return as_schedule_single_action( $when, self::HOOK, array(), self::GROUP );
+    }
+
+    /**
+     * Remove the scheduled migration action.
+     */
+    public function unschedule_migration() {
+        as_unschedule_action( self::HOOK, null, self::GROUP );
+    }
+
+    /**
+     * Get migration batch schedule interval.
+     *
+     * @return int Seconds between migration runs. Defaults to 0 seconds to allow chaining migration via Async Runners.
+     */
+    private function get_schedule_interval() {
+        return (int) apply_filters( 'action_scheduler/migration_interval', 0 );
+    }
+
+    /**
+     * Get migration batch size.
+     *
+     * @return int Number of actions to migrate in each batch. Defaults to 250.
+     */
+    private function get_batch_size() {
+        return (int) apply_filters( 'action_scheduler/migration_batch_size', 250 );
+    }
+
+    /**
+     * Get migration runner object.
+     *
+     * @return Runner
+     */
+    private function get_migration_runner() {
+        $config = Controller::instance()->get_migration_config_object();
+
+        return new Runner( $config );
+    }
 
 }

Spacing +17 added lines, -17 removed lines

@@ -23,14 +23,14 @@ 
 	 * Set up the callback for the scheduled job.
 	 */
 	public function hook() {
-		add_action( self::HOOK, array( $this, 'run_migration' ), 10, 0 );
+		add_action(self::HOOK, array($this, 'run_migration'), 10, 0);
 	}
 
 	/**
 	 * Remove the callback for the scheduled job.
 	 */
 	public function unhook() {
-		remove_action( self::HOOK, array( $this, 'run_migration' ), 10 );
+		remove_action(self::HOOK, array($this, 'run_migration'), 10);
 	}
 
 	/**
@@ -38,12 +38,12 @@ 
 	 */
 	public function run_migration() {
 		$migration_runner = $this->get_migration_runner();
-		$count            = $migration_runner->run( $this->get_batch_size() );
+		$count            = $migration_runner->run($this->get_batch_size());
 
-		if ( $count === 0 ) {
+		if ($count === 0) {
 			$this->mark_complete();
 		} else {
-			$this->schedule_migration( time() + $this->get_schedule_interval() );
+			$this->schedule_migration(time() + $this->get_schedule_interval());
 		}
 	}
 
@@ -54,7 +54,7 @@ 
 		$this->unschedule_migration();
 
 		\ActionScheduler_DataController::mark_migration_complete();
-		do_action( 'action_scheduler/migration_complete' );
+		do_action('action_scheduler/migration_complete');
 	}
 
 	/**
@@ -63,9 +63,9 @@ 
 	 * @return bool Whether there is a pending action in the store to handle the migration
 	 */
 	public function is_migration_scheduled() {
-		$next = as_next_scheduled_action( self::HOOK );
+		$next = as_next_scheduled_action(self::HOOK);
 
-		return ! empty( $next );
+		return !empty($next);
 	}
 
 	/**
@@ -75,25 +75,25 @@ 
 	 *
 	 * @return string The action ID
 	 */
-	public function schedule_migration( $when = 0 ) {
-		$next = as_next_scheduled_action( self::HOOK );
+	public function schedule_migration($when = 0) {
+		$next = as_next_scheduled_action(self::HOOK);
 
-		if ( ! empty( $next ) ) {
+		if (!empty($next)) {
 			return $next;
 		}
 
-		if ( empty( $when ) ) {
+		if (empty($when)) {
 			$when = time();
 		}
 
-		return as_schedule_single_action( $when, self::HOOK, array(), self::GROUP );
+		return as_schedule_single_action($when, self::HOOK, array(), self::GROUP);
 	}
 
 	/**
 	 * Remove the scheduled migration action.
 	 */
 	public function unschedule_migration() {
-		as_unschedule_action( self::HOOK, null, self::GROUP );
+		as_unschedule_action(self::HOOK, null, self::GROUP);
 	}
 
 	/**
@@ -102,7 +102,7 @@ 
 	 * @return int Seconds between migration runs. Defaults to 0 seconds to allow chaining migration via Async Runners.
 	 */
 	private function get_schedule_interval() {
-		return (int) apply_filters( 'action_scheduler/migration_interval', 0 );
+		return (int) apply_filters('action_scheduler/migration_interval', 0);
 	}
 
 	/**
@@ -111,7 +111,7 @@ 
 	 * @return int Number of actions to migrate in each batch. Defaults to 250.
 	 */
 	private function get_batch_size() {
-		return (int) apply_filters( 'action_scheduler/migration_batch_size', 250 );
+		return (int) apply_filters('action_scheduler/migration_batch_size', 250);
 	}
 
 	/**
@@ -122,7 +122,7 @@ 
 	private function get_migration_runner() {
 		$config = Controller::instance()->get_migration_config_object();
 
-		return new Runner( $config );
+		return new Runner($config);
 	}
 
 }

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

Indentation +11 added lines, -11 removed lines

@@ -13,16 +13,16 @@
  * @codeCoverageIgnore
  */
 class DryRun_ActionMigrator extends ActionMigrator {
-	/**
-	 * Simulate migrating an action.
-	 *
-	 * @param int $source_action_id Action ID.
-	 *
-	 * @return int
-	 */
-	public function migrate( $source_action_id ) {
-		do_action( 'action_scheduler/migrate_action_dry_run', $source_action_id );
+    /**
+     * Simulate migrating an action.
+     *
+     * @param int $source_action_id Action ID.
+     *
+     * @return int
+     */
+    public function migrate( $source_action_id ) {
+        do_action( 'action_scheduler/migrate_action_dry_run', $source_action_id );
 
-		return 0;
-	}
+        return 0;
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -20,8 +20,8 @@
 	 *
 	 * @return int
 	 */
-	public function migrate( $source_action_id ) {
-		do_action( 'action_scheduler/migrate_action_dry_run', $source_action_id );
+	public function migrate($source_action_id) {
+		do_action('action_scheduler/migrate_action_dry_run', $source_action_id);
 
 		return 0;
 	}

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

Indentation +9 added lines, -9 removed lines

@@ -11,13 +11,13 @@
  * @codeCoverageIgnore
  */
 class DryRun_LogMigrator extends LogMigrator {
-	/**
-	 * Simulate migrating an action log.
-	 *
-	 * @param int $source_action_id Source logger object.
-	 * @param int $destination_action_id Destination logger object.
-	 */
-	public function migrate( $source_action_id, $destination_action_id ) {
-		// no-op
-	}
+    /**
+     * Simulate migrating an action log.
+     *
+     * @param int $source_action_id Source logger object.
+     * @param int $destination_action_id Destination logger object.
+     */
+    public function migrate( $source_action_id, $destination_action_id ) {
+        // no-op
+    }
 }
\ No newline at end of file

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 	 * @param int $source_action_id Source logger object.
 	 * @param int $destination_action_id Destination logger object.
 	 */
-	public function migrate( $source_action_id, $destination_action_id ) {
+	public function migrate($source_action_id, $destination_action_id) {
 		// no-op
 	}
 }
\ No newline at end of file

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

Indentation +31 added lines, -31 removed lines

@@ -9,39 +9,39 @@
  */
 class ActionScheduler_DBStoreMigrator extends ActionScheduler_DBStore {
 
-	/**
-	 * Save an action with optional last attempt date.
-	 *
-	 * Normally, saving an action sets its attempted date to 0000-00-00 00:00:00 because when an action is first saved,
-	 * it can't have been attempted yet, but migrated completed actions will have an attempted date, so we need to save
-	 * that when first saving the action.
-	 *
-	 * @param ActionScheduler_Action $action
-	 * @param \DateTime $scheduled_date Optional date of the first instance to store.
-	 * @param \DateTime $last_attempt_date Optional date the action was last attempted.
-	 *
-	 * @return string The action ID
-	 * @throws \RuntimeException When the action is not saved.
-	 */
-	public function save_action( ActionScheduler_Action $action, \DateTime $scheduled_date = null, \DateTime $last_attempt_date = null ){
-		try {
-			/** @var \wpdb $wpdb */
-			global $wpdb;
+    /**
+     * Save an action with optional last attempt date.
+     *
+     * Normally, saving an action sets its attempted date to 0000-00-00 00:00:00 because when an action is first saved,
+     * it can't have been attempted yet, but migrated completed actions will have an attempted date, so we need to save
+     * that when first saving the action.
+     *
+     * @param ActionScheduler_Action $action
+     * @param \DateTime $scheduled_date Optional date of the first instance to store.
+     * @param \DateTime $last_attempt_date Optional date the action was last attempted.
+     *
+     * @return string The action ID
+     * @throws \RuntimeException When the action is not saved.
+     */
+    public function save_action( ActionScheduler_Action $action, \DateTime $scheduled_date = null, \DateTime $last_attempt_date = null ){
+        try {
+            /** @var \wpdb $wpdb */
+            global $wpdb;
 
-			$action_id = parent::save_action( $action, $scheduled_date );
+            $action_id = parent::save_action( $action, $scheduled_date );
 
-			if ( null !== $last_attempt_date ) {
-				$data = [
-					'last_attempt_gmt'   => $this->get_scheduled_date_string( $action, $last_attempt_date ),
-					'last_attempt_local' => $this->get_scheduled_date_string_local( $action, $last_attempt_date ),
-				];
+            if ( null !== $last_attempt_date ) {
+                $data = [
+                    'last_attempt_gmt'   => $this->get_scheduled_date_string( $action, $last_attempt_date ),
+                    'last_attempt_local' => $this->get_scheduled_date_string_local( $action, $last_attempt_date ),
+                ];
 
-				$wpdb->update( $wpdb->actionscheduler_actions, $data, array( 'action_id' => $action_id ), array( '%s', '%s' ), array( '%d' ) );
-			}
+                $wpdb->update( $wpdb->actionscheduler_actions, $data, array( 'action_id' => $action_id ), array( '%s', '%s' ), array( '%d' ) );
+            }
 
-			return $action_id;
-		} catch ( \Exception $e ) {
-			throw new \RuntimeException( sprintf( __( 'Error saving action: %s', 'action-scheduler' ), $e->getMessage() ), 0 );
-		}
-	}
+            return $action_id;
+        } catch ( \Exception $e ) {
+            throw new \RuntimeException( sprintf( __( 'Error saving action: %s', 'action-scheduler' ), $e->getMessage() ), 0 );
+        }
+    }
 }

Spacing +8 added lines, -8 removed lines

@@ -23,25 +23,25 @@
 	 * @return string The action ID
 	 * @throws \RuntimeException When the action is not saved.
 	 */
-	public function save_action( ActionScheduler_Action $action, \DateTime $scheduled_date = null, \DateTime $last_attempt_date = null ){
+	public function save_action(ActionScheduler_Action $action, \DateTime $scheduled_date = null, \DateTime $last_attempt_date = null) {
 		try {
 			/** @var \wpdb $wpdb */
 			global $wpdb;
 
-			$action_id = parent::save_action( $action, $scheduled_date );
+			$action_id = parent::save_action($action, $scheduled_date);
 
-			if ( null !== $last_attempt_date ) {
+			if (null !== $last_attempt_date) {
 				$data = [
-					'last_attempt_gmt'   => $this->get_scheduled_date_string( $action, $last_attempt_date ),
-					'last_attempt_local' => $this->get_scheduled_date_string_local( $action, $last_attempt_date ),
+					'last_attempt_gmt'   => $this->get_scheduled_date_string($action, $last_attempt_date),
+					'last_attempt_local' => $this->get_scheduled_date_string_local($action, $last_attempt_date),
 				];
 
-				$wpdb->update( $wpdb->actionscheduler_actions, $data, array( 'action_id' => $action_id ), array( '%s', '%s' ), array( '%d' ) );
+				$wpdb->update($wpdb->actionscheduler_actions, $data, array('action_id' => $action_id), array('%s', '%s'), array('%d'));
 			}
 
 			return $action_id;
-		} catch ( \Exception $e ) {
-			throw new \RuntimeException( sprintf( __( 'Error saving action: %s', 'action-scheduler' ), $e->getMessage() ), 0 );
+		} catch (\Exception $e) {
+			throw new \RuntimeException(sprintf(__('Error saving action: %s', 'action-scheduler'), $e->getMessage()), 0);
 		}
 	}
 }