AyeCode / invoicing

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

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
 	private $id = '';
 	private $action_ids = array();
 
-	public function __construct( $id, array $action_ids ) {
+	public function __construct($id, array $action_ids) {
 		$this->id = $id;
 		$this->action_ids = $action_ids;
 	}

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

Spacing +34 added lines, -34 removed lines

@@ -19,7 +19,7 @@ 
 	 * @codeCoverageIgnore
 	 */
 	public static function instance() {
-		if ( empty(self::$runner) ) {
+		if (empty(self::$runner)) {
 			$class = apply_filters('action_scheduler_queue_runner_class', 'ActionScheduler_QueueRunner');
 			self::$runner = new $class();
 		}
@@ -33,11 +33,11 @@ 
 	 * @param ActionScheduler_FatalErrorMonitor $monitor
 	 * @param ActionScheduler_QueueCleaner      $cleaner
 	 */
-	public function __construct( ActionScheduler_Store $store = null, ActionScheduler_FatalErrorMonitor $monitor = null, ActionScheduler_QueueCleaner $cleaner = null, ActionScheduler_AsyncRequest_QueueRunner $async_request = null ) {
-		parent::__construct( $store, $monitor, $cleaner );
+	public function __construct(ActionScheduler_Store $store = null, ActionScheduler_FatalErrorMonitor $monitor = null, ActionScheduler_QueueCleaner $cleaner = null, ActionScheduler_AsyncRequest_QueueRunner $async_request = null) {
+		parent::__construct($store, $monitor, $cleaner);
 
-		if ( is_null( $async_request ) ) {
-			$async_request = new ActionScheduler_AsyncRequest_QueueRunner( $this->store );
+		if (is_null($async_request)) {
+			$async_request = new ActionScheduler_AsyncRequest_QueueRunner($this->store);
 		}
 
 		$this->async_request = $async_request;
@@ -48,23 +48,23 @@ 
 	 */
 	public function init() {
 
-		add_filter( 'cron_schedules', array( self::instance(), 'add_wp_cron_schedule' ) );
+		add_filter('cron_schedules', array(self::instance(), 'add_wp_cron_schedule'));
 
-		$cron_context = array( 'WP Cron' );
+		$cron_context = array('WP Cron');
 
-		if ( ! wp_next_scheduled( self::WP_CRON_HOOK, $cron_context ) ) {
+		if (!wp_next_scheduled(self::WP_CRON_HOOK, $cron_context)) {
 
 			// Check for and remove any WP Cron hook scheduled by Action Scheduler < 3.0.0, which didn't include the $context param
-			$next_timestamp = wp_next_scheduled( self::WP_CRON_HOOK );
-			if ( $next_timestamp ) {
-				wp_unschedule_event( $next_timestamp, self::WP_CRON_HOOK );
+			$next_timestamp = wp_next_scheduled(self::WP_CRON_HOOK);
+			if ($next_timestamp) {
+				wp_unschedule_event($next_timestamp, self::WP_CRON_HOOK);
 			}
 
-			$schedule = apply_filters( 'action_scheduler_run_schedule', self::WP_CRON_SCHEDULE );
-			wp_schedule_event( time(), $schedule, self::WP_CRON_HOOK, $cron_context );
+			$schedule = apply_filters('action_scheduler_run_schedule', self::WP_CRON_SCHEDULE);
+			wp_schedule_event(time(), $schedule, self::WP_CRON_HOOK, $cron_context);
 		}
 
-		add_action( self::WP_CRON_HOOK, array( self::instance(), 'run' ) );
+		add_action(self::WP_CRON_HOOK, array(self::instance(), 'run'));
 		$this->hook_dispatch_async_request();
 	}
 
@@ -72,14 +72,14 @@ 
 	 * Hook check for dispatching an async request.
 	 */
 	public function hook_dispatch_async_request() {
-		add_action( 'shutdown', array( $this, 'maybe_dispatch_async_request' ) );
+		add_action('shutdown', array($this, 'maybe_dispatch_async_request'));
 	}
 
 	/**
 	 * Unhook check for dispatching an async request.
 	 */
 	public function unhook_dispatch_async_request() {
-		remove_action( 'shutdown', array( $this, 'maybe_dispatch_async_request' ) );
+		remove_action('shutdown', array($this, 'maybe_dispatch_async_request'));
 	}
 
 	/**
@@ -101,9 +101,9 @@ 
 	 * should dispatch a request to process pending actions.
 	 */
 	public function maybe_dispatch_async_request() {
-		if ( is_admin() && ! ActionScheduler::lock()->is_locked( 'async-request-runner' ) ) {
+		if (is_admin() && !ActionScheduler::lock()->is_locked('async-request-runner')) {
 			// Only start an async queue at most once every 60 seconds
-			ActionScheduler::lock()->set( 'async-request-runner' );
+			ActionScheduler::lock()->set('async-request-runner');
 			$this->async_request->maybe_dispatch();
 		}
 	}
@@ -121,21 +121,21 @@ 
 	 *        Generally, this should be capitalised and not localised as it's a proper noun.
 	 * @return int The number of actions processed.
 	 */
-	public function run( $context = 'WP Cron' ) {
+	public function run($context = 'WP Cron') {
 		ActionScheduler_Compatibility::raise_memory_limit();
-		ActionScheduler_Compatibility::raise_time_limit( $this->get_time_limit() );
-		do_action( 'action_scheduler_before_process_queue' );
+		ActionScheduler_Compatibility::raise_time_limit($this->get_time_limit());
+		do_action('action_scheduler_before_process_queue');
 		$this->run_cleanup();
 		$processed_actions = 0;
-		if ( false === $this->has_maximum_concurrent_batches() ) {
-			$batch_size = apply_filters( 'action_scheduler_queue_runner_batch_size', 25 );
+		if (false === $this->has_maximum_concurrent_batches()) {
+			$batch_size = apply_filters('action_scheduler_queue_runner_batch_size', 25);
 			do {
-				$processed_actions_in_batch = $this->do_batch( $batch_size, $context );
+				$processed_actions_in_batch = $this->do_batch($batch_size, $context);
 				$processed_actions         += $processed_actions_in_batch;
-			} while ( $processed_actions_in_batch > 0 && ! $this->batch_limits_exceeded( $processed_actions ) ); // keep going until we run out of actions, time, or memory
+			} while ($processed_actions_in_batch > 0 && !$this->batch_limits_exceeded($processed_actions)); // keep going until we run out of actions, time, or memory
 		}
 
-		do_action( 'action_scheduler_after_process_queue' );
+		do_action('action_scheduler_after_process_queue');
 		return $processed_actions;
 	}
 
@@ -150,20 +150,20 @@ 
 	 *        Generally, this should be capitalised and not localised as it's a proper noun.
 	 * @return int The number of actions processed.
 	 */
-	protected function do_batch( $size = 100, $context = '' ) {
+	protected function do_batch($size = 100, $context = '') {
 		$claim = $this->store->stake_claim($size);
 		$this->monitor->attach($claim);
 		$processed_actions = 0;
 
-		foreach ( $claim->get_actions() as $action_id ) {
+		foreach ($claim->get_actions() as $action_id) {
 			// bail if we lost the claim
-			if ( ! in_array( $action_id, $this->store->find_actions_by_claim_id( $claim->get_id() ) ) ) {
+			if (!in_array($action_id, $this->store->find_actions_by_claim_id($claim->get_id()))) {
 				break;
 			}
-			$this->process_action( $action_id, $context );
+			$this->process_action($action_id, $context);
 			$processed_actions++;
 
-			if ( $this->batch_limits_exceeded( $processed_actions ) ) {
+			if ($this->batch_limits_exceeded($processed_actions)) {
 				break;
 			}
 		}
@@ -182,15 +182,15 @@ 
 	 * add_filter( 'action_scheduler_queue_runner_flush_cache', '__return_true' );
 	 */
 	protected function clear_caches() {
-		if ( ! wp_using_ext_object_cache() || apply_filters( 'action_scheduler_queue_runner_flush_cache', false ) ) {
+		if (!wp_using_ext_object_cache() || apply_filters('action_scheduler_queue_runner_flush_cache', false)) {
 			wp_cache_flush();
 		}
 	}
 
-	public function add_wp_cron_schedule( $schedules ) {
+	public function add_wp_cron_schedule($schedules) {
 		$schedules['every_minute'] = array(
 			'interval' => 60, // in seconds
-			'display'  => __( 'Every minute', 'action-scheduler' ),
+			'display'  => __('Every minute', 'action-scheduler'),
 		);
 
 		return $schedules;

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

Spacing +9 added lines, -9 removed lines

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

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

Spacing +20 added lines, -20 removed lines

@@ -34,20 +34,20 @@ 
 	 * Initialize the class and attach callbacks.
 	 */
 	public static function init() {
-		if ( empty( self::$wp_comment_logger ) ) {
+		if (empty(self::$wp_comment_logger)) {
 			self::$wp_comment_logger = new ActionScheduler_wpCommentLogger();
 		}
 
-		add_action( self::$cleanup_hook, array( __CLASS__, 'delete_all_action_comments' ) );
+		add_action(self::$cleanup_hook, array(__CLASS__, 'delete_all_action_comments'));
 
 		// While there are orphaned logs left in the comments table, we need to attach the callbacks which filter comment counts.
-		add_action( 'pre_get_comments', array( self::$wp_comment_logger, 'filter_comment_queries' ), 10, 1 );
-		add_action( 'wp_count_comments', array( self::$wp_comment_logger, 'filter_comment_count' ), 20, 2 ); // run after WC_Comments::wp_count_comments() to make sure we exclude order notes and action logs
-		add_action( 'comment_feed_where', array( self::$wp_comment_logger, 'filter_comment_feed' ), 10, 2 );
+		add_action('pre_get_comments', array(self::$wp_comment_logger, 'filter_comment_queries'), 10, 1);
+		add_action('wp_count_comments', array(self::$wp_comment_logger, 'filter_comment_count'), 20, 2); // run after WC_Comments::wp_count_comments() to make sure we exclude order notes and action logs
+		add_action('comment_feed_where', array(self::$wp_comment_logger, 'filter_comment_feed'), 10, 2);
 
 		// Action Scheduler may be displayed as a Tools screen or WooCommerce > Status administration screen
-		add_action( 'load-tools_page_action-scheduler', array( __CLASS__, 'register_admin_notice' ) );
-		add_action( 'load-woocommerce_page_wc-status', array( __CLASS__, 'register_admin_notice' ) );
+		add_action('load-tools_page_action-scheduler', array(__CLASS__, 'register_admin_notice'));
+		add_action('load-woocommerce_page_wc-status', array(__CLASS__, 'register_admin_notice'));
 	}
 
 	/**
@@ -58,7 +58,7 @@ 
 	 * @return boolean Whether there are scheduled action comments in the comments table.
 	 */
 	public static function has_logs() {
-		return 'yes' === get_option( self::$has_logs_option_key );
+		return 'yes' === get_option(self::$has_logs_option_key);
 	}
 
 	/**
@@ -66,11 +66,11 @@ 
 	 * Attached to the migration complete hook 'action_scheduler/migration_complete'.
 	 */
 	public static function maybe_schedule_cleanup() {
-		if ( (bool) get_comments( array( 'type' => ActionScheduler_wpCommentLogger::TYPE, 'number' => 1, 'fields' => 'ids' ) ) ) {
-			update_option( self::$has_logs_option_key, 'yes' );
+		if ((bool) get_comments(array('type' => ActionScheduler_wpCommentLogger::TYPE, 'number' => 1, 'fields' => 'ids'))) {
+			update_option(self::$has_logs_option_key, 'yes');
 
-			if ( ! as_next_scheduled_action( self::$cleanup_hook ) ) {
-				as_schedule_single_action( gmdate( 'U' ) + ( 6 * MONTH_IN_SECONDS ), self::$cleanup_hook );
+			if (!as_next_scheduled_action(self::$cleanup_hook)) {
+				as_schedule_single_action(gmdate('U') + (6 * MONTH_IN_SECONDS), self::$cleanup_hook);
 			}
 		}
 	}
@@ -80,15 +80,15 @@ 
 	 */
 	public static function delete_all_action_comments() {
 		global $wpdb;
-		$wpdb->delete( $wpdb->comments, array( 'comment_type' => ActionScheduler_wpCommentLogger::TYPE, 'comment_agent' => ActionScheduler_wpCommentLogger::AGENT ) );
-		delete_option( self::$has_logs_option_key );
+		$wpdb->delete($wpdb->comments, array('comment_type' => ActionScheduler_wpCommentLogger::TYPE, 'comment_agent' => ActionScheduler_wpCommentLogger::AGENT));
+		delete_option(self::$has_logs_option_key);
 	}
 
 	/**
 	 * Registers admin notices about the orphaned action logs.
 	 */
 	public static function register_admin_notice() {
-		add_action( 'admin_notices', array( __CLASS__, 'print_admin_notice' ) );
+		add_action('admin_notices', array(__CLASS__, 'print_admin_notice'));
 	}
 	
 	/**
@@ -96,20 +96,20 @@ 
 	 */
 	public static function print_admin_notice() {
 		$next_cleanup_message        = '';
-		$next_scheduled_cleanup_hook = as_next_scheduled_action( self::$cleanup_hook );
+		$next_scheduled_cleanup_hook = as_next_scheduled_action(self::$cleanup_hook);
 
-		if ( $next_scheduled_cleanup_hook ) {
+		if ($next_scheduled_cleanup_hook) {
 			/* translators: %s: date interval */
-			$next_cleanup_message = sprintf( __( 'This data will be deleted in %s.', 'action-scheduler' ), human_time_diff( gmdate( 'U' ), $next_scheduled_cleanup_hook ) );
+			$next_cleanup_message = sprintf(__('This data will be deleted in %s.', 'action-scheduler'), human_time_diff(gmdate('U'), $next_scheduled_cleanup_hook));
 		}
 
 		$notice = sprintf(
 			/* translators: 1: next cleanup message 2: github issue URL */
-			__( 'Action Scheduler has migrated data to custom tables; however, orphaned log entries exist in the WordPress Comments table. %1$s <a href="%2$s">Learn more &raquo;</a>', 'action-scheduler' ),
+			__('Action Scheduler has migrated data to custom tables; however, orphaned log entries exist in the WordPress Comments table. %1$s <a href="%2$s">Learn more &raquo;</a>', 'action-scheduler'),
 			$next_cleanup_message,
 			'https://github.com/woocommerce/action-scheduler/issues/368'
 		);
 
-		echo '<div class="notice notice-warning"><p>' . wp_kses_post( $notice ) . '</p></div>';
+		echo '<div class="notice notice-warning"><p>' . wp_kses_post($notice) . '</p></div>';
 	}
 }

includes/libraries/action-scheduler/classes/actions/ActionScheduler_CanceledAction.php

Spacing +4 added lines, -4 removed lines

@@ -14,10 +14,10 @@
 	 * @param ActionScheduler_Schedule $schedule
 	 * @param string $group
 	 */
-	public function __construct( $hook, array $args = array(), ActionScheduler_Schedule $schedule = null, $group = '' ) {
-		parent::__construct( $hook, $args, $schedule, $group );
-		if ( is_null( $schedule ) ) {
-			$this->set_schedule( new ActionScheduler_NullSchedule() );
+	public function __construct($hook, array $args = array(), ActionScheduler_Schedule $schedule = null, $group = '') {
+		parent::__construct($hook, $args, $schedule, $group);
+		if (is_null($schedule)) {
+			$this->set_schedule(new ActionScheduler_NullSchedule());
 		}
 	}
 }

includes/libraries/action-scheduler/classes/actions/ActionScheduler_Action.php

Spacing +6 added lines, -6 removed lines

@@ -10,8 +10,8 @@ 
 	protected $schedule = NULL;
 	protected $group = '';
 
-	public function __construct( $hook, array $args = array(), ActionScheduler_Schedule $schedule = NULL, $group = '' ) {
-		$schedule = empty( $schedule ) ? new ActionScheduler_NullSchedule() : $schedule;
+	public function __construct($hook, array $args = array(), ActionScheduler_Schedule $schedule = NULL, $group = '') {
+		$schedule = empty($schedule) ? new ActionScheduler_NullSchedule() : $schedule;
 		$this->set_hook($hook);
 		$this->set_schedule($schedule);
 		$this->set_args($args);
@@ -25,7 +25,7 @@ 
 	/**
 	 * @param string $hook
 	 */
-	protected function set_hook( $hook ) {
+	protected function set_hook($hook) {
 		$this->hook = $hook;
 	}
 
@@ -33,7 +33,7 @@ 
 		return $this->hook;
 	}
 
-	protected function set_schedule( ActionScheduler_Schedule $schedule ) {
+	protected function set_schedule(ActionScheduler_Schedule $schedule) {
 		$this->schedule = $schedule;
 	}
 
@@ -44,7 +44,7 @@ 
 		return $this->schedule;
 	}
 
-	protected function set_args( array $args ) {
+	protected function set_args(array $args) {
 		$this->args = $args;
 	}
 
@@ -55,7 +55,7 @@ 
 	/**
 	 * @param string $group
 	 */
-	protected function set_group( $group ) {
+	protected function set_group($group) {
 		$this->group = $group;
 	}
 

includes/libraries/action-scheduler/classes/actions/ActionScheduler_NullAction.php

Spacing +2 added lines, -2 removed lines

@@ -5,8 +5,8 @@
  */
 class ActionScheduler_NullAction extends ActionScheduler_Action {
 
-	public function __construct( $hook = '', array $args = array(), ActionScheduler_Schedule $schedule = NULL ) {
-		$this->set_schedule( new ActionScheduler_NullSchedule() );
+	public function __construct($hook = '', array $args = array(), ActionScheduler_Schedule $schedule = NULL) {
+		$this->set_schedule(new ActionScheduler_NullSchedule());
 	}
 
 	public function execute() {

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