AyeCode / invoicing

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

Spacing +30 added lines, -30 removed lines

@@ -19,7 +19,7 @@ 
 	 */
 	public static function instance() {
 
-		if ( empty( self::$admin_view ) ) {
+		if (empty(self::$admin_view)) {
 			$class = apply_filters('action_scheduler_admin_view_class', 'ActionScheduler_AdminView');
 			self::$admin_view = new $class();
 		}
@@ -31,22 +31,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_menu', array($this, 'register_menu'));
 
-			add_action( 'current_screen', array( $this, 'add_help_tabs' ) );
+			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();
 	}
 
@@ -56,8 +56,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;
 	}
@@ -72,13 +72,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'));
 	}
 
 	/**
@@ -102,8 +102,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();
 		}
 
@@ -116,7 +116,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;
 		}
 
@@ -124,11 +124,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>' .
+					'<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' ) .
+						__('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>',
 			)
 		);
@@ -136,17 +136,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>' .
+					'<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' ) ) .
+					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>',
 			)
 		);

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

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

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

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

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

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

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

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

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