AyeCode / invoicing

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

Spacing +6 added lines, -6 removed lines

@@ -11,8 +11,8 @@ 
 
 	private $versions = array();
 
-	public function register( $version_string, $initialization_callback ) {
-		if ( isset($this->versions[$version_string]) ) {
+	public function register($version_string, $initialization_callback) {
+		if (isset($this->versions[$version_string])) {
 			return FALSE;
 		}
 		$this->versions[$version_string] = $initialization_callback;
@@ -25,16 +25,16 @@ 
 
 	public function latest_version() {
 		$keys = array_keys($this->versions);
-		if ( empty($keys) ) {
+		if (empty($keys)) {
 			return false;
 		}
-		uasort( $keys, 'version_compare' );
+		uasort($keys, 'version_compare');
 		return end($keys);
 	}
 
 	public function latest_version_callback() {
 		$latest = $this->latest_version();
-		if ( empty($latest) || !isset($this->versions[$latest]) ) {
+		if (empty($latest) || !isset($this->versions[$latest])) {
 			return '__return_null';
 		}
 		return $this->versions[$latest];
@@ -45,7 +45,7 @@ 
 	 * @codeCoverageIgnore
 	 */
 	public static function instance() {
-		if ( empty(self::$instance) ) {
+		if (empty(self::$instance)) {
 			self::$instance = new self();
 		}
 		return self::$instance;

includes/libraries/action-scheduler/classes/abstracts/ActionScheduler_Abstract_QueueRunner.php

Spacing +46 added lines, -46 removed lines

@@ -31,13 +31,13 @@ 
 	 * @param ActionScheduler_FatalErrorMonitor $monitor
 	 * @param ActionScheduler_QueueCleaner      $cleaner
 	 */
-	public function __construct( ActionScheduler_Store $store = null, ActionScheduler_FatalErrorMonitor $monitor = null, ActionScheduler_QueueCleaner $cleaner = null ) {
+	public function __construct(ActionScheduler_Store $store = null, ActionScheduler_FatalErrorMonitor $monitor = null, ActionScheduler_QueueCleaner $cleaner = null) {
 
-		$this->created_time = microtime( true );
+		$this->created_time = microtime(true);
 
 		$this->store   = $store ? $store : ActionScheduler_Store::instance();
-		$this->monitor = $monitor ? $monitor : new ActionScheduler_FatalErrorMonitor( $this->store );
-		$this->cleaner = $cleaner ? $cleaner : new ActionScheduler_QueueCleaner( $this->store );
+		$this->monitor = $monitor ? $monitor : new ActionScheduler_FatalErrorMonitor($this->store);
+		$this->cleaner = $cleaner ? $cleaner : new ActionScheduler_QueueCleaner($this->store);
 	}
 
 	/**
@@ -47,35 +47,35 @@ 
 	 * @param string $context Optional identifer for the context in which this action is being processed, e.g. 'WP CLI' or 'WP Cron'
 	 *        Generally, this should be capitalised and not localised as it's a proper noun.
 	 */
-	public function process_action( $action_id, $context = '' ) {
+	public function process_action($action_id, $context = '') {
 		try {
 			$valid_action = false;
-			do_action( 'action_scheduler_before_execute', $action_id, $context );
+			do_action('action_scheduler_before_execute', $action_id, $context);
 
-			if ( ActionScheduler_Store::STATUS_PENDING !== $this->store->get_status( $action_id ) ) {
-				do_action( 'action_scheduler_execution_ignored', $action_id, $context );
+			if (ActionScheduler_Store::STATUS_PENDING !== $this->store->get_status($action_id)) {
+				do_action('action_scheduler_execution_ignored', $action_id, $context);
 				return;
 			}
 
 			$valid_action = true;
-			do_action( 'action_scheduler_begin_execute', $action_id, $context );
+			do_action('action_scheduler_begin_execute', $action_id, $context);
 
-			$action = $this->store->fetch_action( $action_id );
-			$this->store->log_execution( $action_id );
+			$action = $this->store->fetch_action($action_id);
+			$this->store->log_execution($action_id);
 			$action->execute();
-			do_action( 'action_scheduler_after_execute', $action_id, $action, $context );
-			$this->store->mark_complete( $action_id );
-		} catch ( Exception $e ) {
-			if ( $valid_action ) {
-				$this->store->mark_failure( $action_id );
-				do_action( 'action_scheduler_failed_execution', $action_id, $e, $context );
+			do_action('action_scheduler_after_execute', $action_id, $action, $context);
+			$this->store->mark_complete($action_id);
+		} catch (Exception $e) {
+			if ($valid_action) {
+				$this->store->mark_failure($action_id);
+				do_action('action_scheduler_failed_execution', $action_id, $e, $context);
 			} else {
-				do_action( 'action_scheduler_failed_validation', $action_id, $e, $context );
+				do_action('action_scheduler_failed_validation', $action_id, $e, $context);
 			}
 		}
 
-		if ( isset( $action ) && is_a( $action, 'ActionScheduler_Action' ) && $action->get_schedule()->is_recurring() ) {
-			$this->schedule_next_instance( $action, $action_id );
+		if (isset($action) && is_a($action, 'ActionScheduler_Action') && $action->get_schedule()->is_recurring()) {
+			$this->schedule_next_instance($action, $action_id);
 		}
 	}
 
@@ -85,11 +85,11 @@ 
 	 * @param ActionScheduler_Action $action
 	 * @param int $action_id
 	 */
-	protected function schedule_next_instance( ActionScheduler_Action $action, $action_id ) {
+	protected function schedule_next_instance(ActionScheduler_Action $action, $action_id) {
 		try {
-			ActionScheduler::factory()->repeat( $action );
-		} catch ( Exception $e ) {
-			do_action( 'action_scheduler_failed_to_schedule_next_instance', $action_id, $e, $action );
+			ActionScheduler::factory()->repeat($action);
+		} catch (Exception $e) {
+			do_action('action_scheduler_failed_to_schedule_next_instance', $action_id, $e, $action);
 		}
 	}
 
@@ -99,7 +99,7 @@ 
 	 * @author Jeremy Pry
 	 */
 	protected function run_cleanup() {
-		$this->cleaner->clean( 10 * $this->get_time_limit() );
+		$this->cleaner->clean(10 * $this->get_time_limit());
 	}
 
 	/**
@@ -108,7 +108,7 @@ 
 	 * @return int
 	 */
 	public function get_allowed_concurrent_batches() {
-		return apply_filters( 'action_scheduler_queue_runner_concurrent_batches', 1 );
+		return apply_filters('action_scheduler_queue_runner_concurrent_batches', 1);
 	}
 
 	/**
@@ -130,12 +130,12 @@ 
 		$time_limit = 30;
 
 		// Apply deprecated filter from deprecated get_maximum_execution_time() method
-		if ( has_filter( 'action_scheduler_maximum_execution_time' ) ) {
-			_deprecated_function( 'action_scheduler_maximum_execution_time', '2.1.1', 'action_scheduler_queue_runner_time_limit' );
-			$time_limit = apply_filters( 'action_scheduler_maximum_execution_time', $time_limit );
+		if (has_filter('action_scheduler_maximum_execution_time')) {
+			_deprecated_function('action_scheduler_maximum_execution_time', '2.1.1', 'action_scheduler_queue_runner_time_limit');
+			$time_limit = apply_filters('action_scheduler_maximum_execution_time', $time_limit);
 		}
 
-		return absint( apply_filters( 'action_scheduler_queue_runner_time_limit', $time_limit ) );
+		return absint(apply_filters('action_scheduler_queue_runner_time_limit', $time_limit));
 	}
 
 	/**
@@ -144,14 +144,14 @@ 
 	 * @return int The number of seconds.
 	 */
 	protected function get_execution_time() {
-		$execution_time = microtime( true ) - $this->created_time;
+		$execution_time = microtime(true) - $this->created_time;
 
 		// Get the CPU time if the hosting environment uses it rather than wall-clock time to calculate a process's execution time.
-		if ( function_exists( 'getrusage' ) && apply_filters( 'action_scheduler_use_cpu_execution_time', defined( 'PANTHEON_ENVIRONMENT' ) ) ) {
+		if (function_exists('getrusage') && apply_filters('action_scheduler_use_cpu_execution_time', defined('PANTHEON_ENVIRONMENT'))) {
 			$resource_usages = getrusage();
 
-			if ( isset( $resource_usages['ru_stime.tv_usec'], $resource_usages['ru_stime.tv_usec'] ) ) {
-				$execution_time = $resource_usages['ru_stime.tv_sec'] + ( $resource_usages['ru_stime.tv_usec'] / 1000000 );
+			if (isset($resource_usages['ru_stime.tv_usec'], $resource_usages['ru_stime.tv_usec'])) {
+				$execution_time = $resource_usages['ru_stime.tv_sec'] + ($resource_usages['ru_stime.tv_usec'] / 1000000);
 			}
 		}
 
@@ -164,15 +164,15 @@ 
 	 * @param int $processed_actions The number of actions processed so far - used to determine the likelihood of exceeding the time limit if processing another action
 	 * @return bool
 	 */
-	protected function time_likely_to_be_exceeded( $processed_actions ) {
+	protected function time_likely_to_be_exceeded($processed_actions) {
 
 		$execution_time        = $this->get_execution_time();
 		$max_execution_time    = $this->get_time_limit();
 		$time_per_action       = $execution_time / $processed_actions;
-		$estimated_time        = $execution_time + ( $time_per_action * 3 );
+		$estimated_time        = $execution_time + ($time_per_action * 3);
 		$likely_to_be_exceeded = $estimated_time > $max_execution_time;
 
-		return apply_filters( 'action_scheduler_maximum_execution_time_likely_to_be_exceeded', $likely_to_be_exceeded, $this, $processed_actions, $execution_time, $max_execution_time );
+		return apply_filters('action_scheduler_maximum_execution_time_likely_to_be_exceeded', $likely_to_be_exceeded, $this, $processed_actions, $execution_time, $max_execution_time);
 	}
 
 	/**
@@ -183,18 +183,18 @@ 
 	 * @return int
 	 */
 	protected function get_memory_limit() {
-		if ( function_exists( 'ini_get' ) ) {
-			$memory_limit = ini_get( 'memory_limit' );
+		if (function_exists('ini_get')) {
+			$memory_limit = ini_get('memory_limit');
 		} else {
 			$memory_limit = '128M'; // Sensible default, and minimum required by WooCommerce
 		}
 
-		if ( ! $memory_limit || -1 === $memory_limit || '-1' === $memory_limit ) {
+		if (!$memory_limit || -1 === $memory_limit || '-1' === $memory_limit) {
 			// Unlimited, set to 32GB.
 			$memory_limit = '32G';
 		}
 
-		return ActionScheduler_Compatibility::convert_hr_to_bytes( $memory_limit );
+		return ActionScheduler_Compatibility::convert_hr_to_bytes($memory_limit);
 	}
 
 	/**
@@ -209,10 +209,10 @@ 
 	protected function memory_exceeded() {
 
 		$memory_limit    = $this->get_memory_limit() * 0.90;
-		$current_memory  = memory_get_usage( true );
+		$current_memory  = memory_get_usage(true);
 		$memory_exceeded = $current_memory >= $memory_limit;
 
-		return apply_filters( 'action_scheduler_memory_exceeded', $memory_exceeded, $this );
+		return apply_filters('action_scheduler_memory_exceeded', $memory_exceeded, $this);
 	}
 
 	/**
@@ -224,8 +224,8 @@ 
 	 * @param int $processed_actions The number of actions processed so far - used to determine the likelihood of exceeding the time limit if processing another action
 	 * @return bool
 	 */
-	protected function batch_limits_exceeded( $processed_actions ) {
-		return $this->memory_exceeded() || $this->time_likely_to_be_exceeded( $processed_actions );
+	protected function batch_limits_exceeded($processed_actions) {
+		return $this->memory_exceeded() || $this->time_likely_to_be_exceeded($processed_actions);
 	}
 
 	/**
@@ -236,5 +236,5 @@ 
 	 *        Generally, this should be capitalised and not localised as it's a proper noun.
 	 * @return int The number of actions processed.
 	 */
-	abstract public function run( $context = '' );
+	abstract public function run($context = '');
 }

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

Spacing +122 added lines, -122 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once( ABSPATH . 'wp-admin/includes/class-wp-list-table.php' );
+if (!class_exists('WP_List_Table')) {
+	require_once(ABSPATH . 'wp-admin/includes/class-wp-list-table.php');
 }
 
 /**
@@ -108,7 +108,7 @@ 
 	 * 
 	 * @deprecated 3.0.0
 	 */
-	protected function translate( $text, $context = '' ) {
+	protected function translate($text, $context = '') {
 		return $text;
 	}
 
@@ -120,12 +120,12 @@ 
 	protected function get_bulk_actions() {
 		$actions = array();
 
-		foreach ( $this->bulk_actions as $action => $label ) {
-			if ( ! is_callable( array( $this, 'bulk_' . $action ) ) ) {
-				throw new RuntimeException( "The bulk action $action does not have a callback method" );
+		foreach ($this->bulk_actions as $action => $label) {
+			if (!is_callable(array($this, 'bulk_' . $action))) {
+				throw new RuntimeException("The bulk action $action does not have a callback method");
 			}
 
-			$actions[ $action ] = $label;
+			$actions[$action] = $label;
 		}
 
 		return $actions;
@@ -140,22 +140,22 @@ 
 		global $wpdb;
 		// Detect when a bulk action is being triggered.
 		$action = $this->current_action();
-		if ( ! $action ) {
+		if (!$action) {
 			return;
 		}
 
-		check_admin_referer( 'bulk-' . $this->_args['plural'] );
+		check_admin_referer('bulk-' . $this->_args['plural']);
 
 		$method   = 'bulk_' . $action;
-		if ( array_key_exists( $action, $this->bulk_actions ) && is_callable( array( $this, $method ) ) && ! empty( $_GET['ID'] ) && is_array( $_GET['ID'] ) ) {
-			$ids_sql = '(' . implode( ',', array_fill( 0, count( $_GET['ID'] ), '%s' ) ) . ')';
-			$this->$method( $_GET['ID'], $wpdb->prepare( $ids_sql, $_GET['ID'] ) );
+		if (array_key_exists($action, $this->bulk_actions) && is_callable(array($this, $method)) && !empty($_GET['ID']) && is_array($_GET['ID'])) {
+			$ids_sql = '(' . implode(',', array_fill(0, count($_GET['ID']), '%s')) . ')';
+			$this->$method($_GET['ID'], $wpdb->prepare($ids_sql, $_GET['ID']));
 		}
 
-		wp_redirect( remove_query_arg(
-			array( '_wp_http_referer', '_wpnonce', 'ID', 'action', 'action2' ),
-			wp_unslash( $_SERVER['REQUEST_URI'] )
-		) );
+		wp_redirect(remove_query_arg(
+			array('_wp_http_referer', '_wpnonce', 'ID', 'action', 'action2'),
+			wp_unslash($_SERVER['REQUEST_URI'])
+		));
 		exit;
 	}
 
@@ -163,10 +163,10 @@ 
 	 * Default code for deleting entries.
 	 * validated already by process_bulk_action()
 	 */
-	protected function bulk_delete( array $ids, $ids_sql ) {
+	protected function bulk_delete(array $ids, $ids_sql) {
 		$store = ActionScheduler::store();
-		foreach ( $ids as $action_id ) {
-			$store->delete( $action_id );
+		foreach ($ids as $action_id) {
+			$store->delete($action_id);
 		}
 	}
 
@@ -188,8 +188,8 @@ 
 	 */
 	public function get_sortable_columns() {
 		$sort_by = array();
-		foreach ( $this->sort_by as $column ) {
-			$sort_by[ $column ] = array( $column, true );
+		foreach ($this->sort_by as $column) {
+			$sort_by[$column] = array($column, true);
 		}
 		return $sort_by;
 	}
@@ -200,7 +200,7 @@ 
 	 */
 	public function get_columns() {
 		$columns = array_merge(
-			array( 'cb' => '<input type="checkbox" />' ),
+			array('cb' => '<input type="checkbox" />'),
 			$this->columns
 		);
 
@@ -217,8 +217,8 @@ 
 	protected function get_items_query_limit() {
 		global $wpdb;
 
-		$per_page = $this->get_items_per_page( $this->package . '_items_per_page', $this->items_per_page );
-		return $wpdb->prepare( 'LIMIT %d', $per_page );
+		$per_page = $this->get_items_per_page($this->package . '_items_per_page', $this->items_per_page);
+		return $wpdb->prepare('LIMIT %d', $per_page);
 	}
 
 	/**
@@ -227,10 +227,10 @@ 
 	 * @return int
 	 */
 	protected function get_items_offset() {
-		$per_page = $this->get_items_per_page( $this->package . '_items_per_page', $this->items_per_page );
+		$per_page = $this->get_items_per_page($this->package . '_items_per_page', $this->items_per_page);
 		$current_page = $this->get_pagenum();
-		if ( 1 < $current_page ) {
-			$offset = $per_page * ( $current_page - 1 );
+		if (1 < $current_page) {
+			$offset = $per_page * ($current_page - 1);
 		} else {
 			$offset = 0;
 		}
@@ -248,7 +248,7 @@ 
 	protected function get_items_query_offset() {
 		global $wpdb;
 
-		return $wpdb->prepare( 'OFFSET %d', $this->get_items_offset() );
+		return $wpdb->prepare('OFFSET %d', $this->get_items_offset());
 	}
 
 	/**
@@ -257,12 +257,12 @@ 
 	 * column and sortable. It will also use order (ASC|DESC) using DESC by default.
 	 */
 	protected function get_items_query_order() {
-		if ( empty( $this->sort_by ) ) {
+		if (empty($this->sort_by)) {
 			return '';
 		}
 
-		$orderby = esc_sql( $this->get_request_orderby() );
-		$order   = esc_sql( $this->get_request_order() );
+		$orderby = esc_sql($this->get_request_orderby());
+		$order   = esc_sql($this->get_request_order());
 
 		return "ORDER BY {$orderby} {$order}";
 	}
@@ -274,10 +274,10 @@ 
 	 */
 	protected function get_request_orderby() {
 
-		$valid_sortable_columns = array_values( $this->sort_by );
+		$valid_sortable_columns = array_values($this->sort_by);
 
-		if ( ! empty( $_GET['orderby'] ) && in_array( $_GET['orderby'], $valid_sortable_columns ) ) {
-			$orderby = sanitize_text_field( $_GET['orderby'] );
+		if (!empty($_GET['orderby']) && in_array($_GET['orderby'], $valid_sortable_columns)) {
+			$orderby = sanitize_text_field($_GET['orderby']);
 		} else {
 			$orderby = $valid_sortable_columns[0];
 		}
@@ -292,7 +292,7 @@ 
 	 */
 	protected function get_request_order() {
 
-		if ( ! empty( $_GET['order'] ) && 'desc' === strtolower( $_GET['order'] ) ) {
+		if (!empty($_GET['order']) && 'desc' === strtolower($_GET['order'])) {
 			$order = 'DESC';
 		} else {
 			$order = 'ASC';
@@ -307,7 +307,7 @@ 
 	 * @return string
 	 */
 	protected function get_request_status() {
-		$status = ( ! empty( $_GET['status'] ) ) ? $_GET['status'] : '';
+		$status = (!empty($_GET['status'])) ? $_GET['status'] : '';
 		return $status;
 	}
 
@@ -317,7 +317,7 @@ 
 	 * @return string
 	 */
 	protected function get_request_search_query() {
-		$search_query = ( ! empty( $_GET['s'] ) ) ? $_GET['s'] : '';
+		$search_query = (!empty($_GET['s'])) ? $_GET['s'] : '';
 		return $search_query;
 	}
 
@@ -328,8 +328,8 @@ 
 	 * @return array
 	 */
 	protected function get_table_columns() {
-		$columns = array_keys( $this->columns );
-		if ( ! in_array( $this->ID, $columns ) ) {
+		$columns = array_keys($this->columns);
+		if (!in_array($this->ID, $columns)) {
 			$columns[] = $this->ID;
 		}
 
@@ -351,15 +351,15 @@ 
 	protected function get_items_query_search() {
 		global $wpdb;
 
-		if ( empty( $_GET['s'] ) || empty( $this->search_by ) ) {
+		if (empty($_GET['s']) || empty($this->search_by)) {
 			return '';
 		}
 
-		$filter  = array();
-		foreach ( $this->search_by as $column ) {
-			$filter[] = $wpdb->prepare('`' . $column . '` like "%%s%"', $wpdb->esc_like( $_GET['s'] ));
+		$filter = array();
+		foreach ($this->search_by as $column) {
+			$filter[] = $wpdb->prepare('`' . $column . '` like "%%s%"', $wpdb->esc_like($_GET['s']));
 		}
-		return implode( ' OR ', $filter );
+		return implode(' OR ', $filter);
 	}
 
 	/**
@@ -369,21 +369,21 @@ 
 	protected function get_items_query_filters() {
 		global $wpdb;
 
-		if ( ! $this->filter_by || empty( $_GET['filter_by'] ) || ! is_array( $_GET['filter_by'] ) ) {
+		if (!$this->filter_by || empty($_GET['filter_by']) || !is_array($_GET['filter_by'])) {
 			return '';
 		}
 
 		$filter = array();
 
-		foreach ( $this->filter_by as $column => $options ) {
-			if ( empty( $_GET['filter_by'][ $column ] ) || empty( $options[ $_GET['filter_by'][ $column ] ] ) ) {
+		foreach ($this->filter_by as $column => $options) {
+			if (empty($_GET['filter_by'][$column]) || empty($options[$_GET['filter_by'][$column]])) {
 				continue;
 			}
 
-			$filter[] = $wpdb->prepare( "`$column` = %s", $_GET['filter_by'][ $column ] );
+			$filter[] = $wpdb->prepare("`$column` = %s", $_GET['filter_by'][$column]);
 		}
 
-		return implode( ' AND ', $filter );
+		return implode(' AND ', $filter);
 
 	}
 
@@ -403,9 +403,9 @@ 
 
 		$this->process_row_actions();
 
-		if ( ! empty( $_REQUEST['_wp_http_referer'] ) ) {
+		if (!empty($_REQUEST['_wp_http_referer'])) {
 			// _wp_http_referer is used only on bulk actions, we remove it to keep the $_GET shorter
-			wp_redirect( remove_query_arg( array( '_wp_http_referer', '_wpnonce' ), wp_unslash( $_SERVER['REQUEST_URI'] ) ) );
+			wp_redirect(remove_query_arg(array('_wp_http_referer', '_wpnonce'), wp_unslash($_SERVER['REQUEST_URI'])));
 			exit;
 		}
 
@@ -418,53 +418,53 @@ 
 			$this->get_items_query_search(),
 			$this->get_items_query_filters(),
 		));
-		$columns = '`' . implode( '`, `', $this->get_table_columns() ) . '`';
+		$columns = '`' . implode('`, `', $this->get_table_columns()) . '`';
 
-		if ( ! empty( $where ) ) {
-			$where = 'WHERE ('. implode( ') AND (', $where ) . ')';
+		if (!empty($where)) {
+			$where = 'WHERE (' . implode(') AND (', $where) . ')';
 		} else {
 			$where = '';
 		}
 
 		$sql = "SELECT $columns FROM {$this->table_name} {$where} {$order} {$limit} {$offset}";
 
-		$this->set_items( $wpdb->get_results( $sql, ARRAY_A ) );
+		$this->set_items($wpdb->get_results($sql, ARRAY_A));
 
 		$query_count = "SELECT COUNT({$this->ID}) FROM {$this->table_name} {$where}";
-		$total_items = $wpdb->get_var( $query_count );
-		$per_page    = $this->get_items_per_page( $this->package . '_items_per_page', $this->items_per_page );
-		$this->set_pagination_args( array(
+		$total_items = $wpdb->get_var($query_count);
+		$per_page    = $this->get_items_per_page($this->package . '_items_per_page', $this->items_per_page);
+		$this->set_pagination_args(array(
 			'total_items' => $total_items,
 			'per_page'    => $per_page,
-			'total_pages' => ceil( $total_items / $per_page ),
-		) );
+			'total_pages' => ceil($total_items / $per_page),
+		));
 	}
 
-	public function extra_tablenav( $which ) {
-		if ( ! $this->filter_by || 'top' !== $which ) {
+	public function extra_tablenav($which) {
+		if (!$this->filter_by || 'top' !== $which) {
 			return;
 		}
 
 		echo '<div class="alignleft actions">';
 
-		foreach ( $this->filter_by as $id => $options ) {
-			$default = ! empty( $_GET['filter_by'][ $id ] ) ? $_GET['filter_by'][ $id ] : '';
-			if ( empty( $options[ $default ] ) ) {
+		foreach ($this->filter_by as $id => $options) {
+			$default = !empty($_GET['filter_by'][$id]) ? $_GET['filter_by'][$id] : '';
+			if (empty($options[$default])) {
 				$default = '';
 			}
 
-			echo '<select name="filter_by[' . esc_attr( $id ) . ']" class="first" id="filter-by-' . esc_attr( $id ) . '">';
+			echo '<select name="filter_by[' . esc_attr($id) . ']" class="first" id="filter-by-' . esc_attr($id) . '">';
 
-			foreach ( $options as $value => $label ) {
-				echo '<option value="' . esc_attr( $value ) . '" ' . esc_html( $value == $default ? 'selected' : '' )  .'>'
-					. esc_html( $label )
+			foreach ($options as $value => $label) {
+				echo '<option value="' . esc_attr($value) . '" ' . esc_html($value == $default ? 'selected' : '') . '>'
+					. esc_html($label)
 				. '</option>';
 			}
 
 			echo '</select>';
 		}
 
-		submit_button( esc_html__( 'Filter', 'action-scheduler' ), '', 'filter_action', false, array( 'id' => 'post-query-submit' ) );
+		submit_button(esc_html__('Filter', 'action-scheduler'), '', 'filter_action', false, array('id' => 'post-query-submit'));
 		echo '</div>';
 	}
 
@@ -472,10 +472,10 @@ 
 	 * Set the data for displaying. It will attempt to unserialize (There is a chance that some columns
 	 * are serialized). This can be override in child classes for futher data transformation.
 	 */
-	protected function set_items( array $items ) {
+	protected function set_items(array $items) {
 		$this->items = array();
-		foreach ( $items as $item ) {
-			$this->items[ $item[ $this->ID ] ] = array_map( 'maybe_unserialize', $item );
+		foreach ($items as $item) {
+			$this->items[$item[$this->ID]] = array_map('maybe_unserialize', $item);
 		}
 	}
 
@@ -484,8 +484,8 @@ 
 	 * of how the primary key is named (to keep the code simpler). The bulk actions will do the proper
 	 * name transformation though using `$this->ID`.
 	 */
-	public function column_cb( $row ) {
-		return '<input name="ID[]" type="checkbox" value="' . esc_attr( $row[ $this->ID ] ) .'" />';
+	public function column_cb($row) {
+		return '<input name="ID[]" type="checkbox" value="' . esc_attr($row[$this->ID]) . '" />';
 	}
 
 	/**
@@ -498,62 +498,62 @@ 
 	 * @param $column_name   Current row
 	 * @return
 	 */
-	protected function maybe_render_actions( $row, $column_name ) {
-		if ( empty( $this->row_actions[ $column_name ] ) ) {
+	protected function maybe_render_actions($row, $column_name) {
+		if (empty($this->row_actions[$column_name])) {
 			return;
 		}
 
-		$row_id = $row[ $this->ID ];
+		$row_id = $row[$this->ID];
 
 		$actions = '<div class="row-actions">';
 		$action_count = 0;
-		foreach ( $this->row_actions[ $column_name ] as $action_key => $action ) {
+		foreach ($this->row_actions[$column_name] as $action_key => $action) {
 
 			$action_count++;
 
-			if ( ! method_exists( $this, 'row_action_' . $action_key ) ) {
+			if (!method_exists($this, 'row_action_' . $action_key)) {
 				continue;
 			}
 
-			$action_link = ! empty( $action['link'] ) ? $action['link'] : add_query_arg( array( 'row_action' => $action_key, 'row_id' => $row_id, 'nonce'  => wp_create_nonce( $action_key . '::' . $row_id ) ) );
-			$span_class  = ! empty( $action['class'] ) ? $action['class'] : $action_key;
-			$separator   = ( $action_count < count( $this->row_actions[ $column_name ] ) ) ? ' | ' : '';
+			$action_link = !empty($action['link']) ? $action['link'] : add_query_arg(array('row_action' => $action_key, 'row_id' => $row_id, 'nonce'  => wp_create_nonce($action_key . '::' . $row_id)));
+			$span_class  = !empty($action['class']) ? $action['class'] : $action_key;
+			$separator   = ($action_count < count($this->row_actions[$column_name])) ? ' | ' : '';
 
-			$actions .= sprintf( '<span class="%s">', esc_attr( $span_class ) );
-			$actions .= sprintf( '<a href="%1$s" title="%2$s">%3$s</a>', esc_url( $action_link ), esc_attr( $action['desc'] ), esc_html( $action['name'] ) );
-			$actions .= sprintf( '%s</span>', $separator );
+			$actions .= sprintf('<span class="%s">', esc_attr($span_class));
+			$actions .= sprintf('<a href="%1$s" title="%2$s">%3$s</a>', esc_url($action_link), esc_attr($action['desc']), esc_html($action['name']));
+			$actions .= sprintf('%s</span>', $separator);
 		}
 		$actions .= '</div>';
 		return $actions;
 	}
 
 	protected function process_row_actions() {
-		$parameters = array( 'row_action', 'row_id', 'nonce' );
-		foreach ( $parameters as $parameter ) {
-			if ( empty( $_REQUEST[ $parameter ] ) ) {
+		$parameters = array('row_action', 'row_id', 'nonce');
+		foreach ($parameters as $parameter) {
+			if (empty($_REQUEST[$parameter])) {
 				return;
 			}
 		}
 
 		$method = 'row_action_' . $_REQUEST['row_action'];
 
-		if ( $_REQUEST['nonce'] === wp_create_nonce( $_REQUEST[ 'row_action' ] . '::' . $_REQUEST[ 'row_id' ] ) && method_exists( $this, $method ) ) {
-			$this->$method( $_REQUEST['row_id'] );
+		if ($_REQUEST['nonce'] === wp_create_nonce($_REQUEST['row_action'] . '::' . $_REQUEST['row_id']) && method_exists($this, $method)) {
+			$this->$method($_REQUEST['row_id']);
 		}
 
-		wp_redirect( remove_query_arg(
-			array( 'row_id', 'row_action', 'nonce' ),
-			wp_unslash( $_SERVER['REQUEST_URI'] )
-		) );
+		wp_redirect(remove_query_arg(
+			array('row_id', 'row_action', 'nonce'),
+			wp_unslash($_SERVER['REQUEST_URI'])
+		));
 		exit;
 	}
 
 	/**
 	 * Default column formatting, it will escape everythig for security.
 	 */
-	public function column_default( $item, $column_name ) {
-		$column_html = esc_html( $item[ $column_name ] );
-		$column_html .= $this->maybe_render_actions( $item, $column_name );
+	public function column_default($item, $column_name) {
+		$column_html = esc_html($item[$column_name]);
+		$column_html .= $this->maybe_render_actions($item, $column_name);
 		return $column_html;
 	}
 
@@ -561,10 +561,10 @@ 
 	 * Display the table heading and search query, if any
 	 */
 	protected function display_header() {
-		echo '<h1 class="wp-heading-inline">' . esc_attr( $this->table_header ) . '</h1>';
-		if ( $this->get_request_search_query() ) {
+		echo '<h1 class="wp-heading-inline">' . esc_attr($this->table_header) . '</h1>';
+		if ($this->get_request_search_query()) {
 			/* translators: %s: search query */
-			echo '<span class="subtitle">' . esc_attr( sprintf( __( 'Search results for "%s"', 'action-scheduler' ), $this->get_request_search_query() ) ) . '</span>';
+			echo '<span class="subtitle">' . esc_attr(sprintf(__('Search results for "%s"', 'action-scheduler'), $this->get_request_search_query())) . '</span>';
 		}
 		echo '<hr class="wp-header-end">';
 	}
@@ -573,9 +573,9 @@ 
 	 * Display the table heading and search query, if any
 	 */
 	protected function display_admin_notices() {
-		foreach ( $this->admin_notices as $notice ) {
+		foreach ($this->admin_notices as $notice) {
 			echo '<div id="message" class="' . $notice['class'] . '">';
-			echo '	<p>' . wp_kses_post( $notice['message'] ) . '</p>';
+			echo '	<p>' . wp_kses_post($notice['message']) . '</p>';
 			echo '</div>';
 		}
 	}
@@ -589,30 +589,30 @@ 
 		$request_status    = $this->get_request_status();
 
 		// Helper to set 'all' filter when not set on status counts passed in
-		if ( ! isset( $this->status_counts['all'] ) ) {
-			$this->status_counts = array( 'all' => array_sum( $this->status_counts ) ) + $this->status_counts;
+		if (!isset($this->status_counts['all'])) {
+			$this->status_counts = array('all' => array_sum($this->status_counts)) + $this->status_counts;
 		}
 
-		foreach ( $this->status_counts as $status_name => $count ) {
+		foreach ($this->status_counts as $status_name => $count) {
 
-			if ( 0 === $count ) {
+			if (0 === $count) {
 				continue;
 			}
 
-			if ( $status_name === $request_status || ( empty( $request_status ) && 'all' === $status_name ) ) {
+			if ($status_name === $request_status || (empty($request_status) && 'all' === $status_name)) {
 				$status_list_item = '<li class="%1$s"><strong>%3$s</strong> (%4$d)</li>';
 			} else {
 				$status_list_item = '<li class="%1$s"><a href="%2$s">%3$s</a> (%4$d)</li>';
 			}
 
-			$status_filter_url   = ( 'all' === $status_name ) ? remove_query_arg( 'status' ) : add_query_arg( 'status', $status_name );
-			$status_filter_url   = remove_query_arg( array( 'paged', 's' ), $status_filter_url );
-			$status_list_items[] = sprintf( $status_list_item, esc_attr( $status_name ), esc_url( $status_filter_url ), esc_html( ucfirst( $status_name ) ), absint( $count ) );
+			$status_filter_url   = ('all' === $status_name) ? remove_query_arg('status') : add_query_arg('status', $status_name);
+			$status_filter_url   = remove_query_arg(array('paged', 's'), $status_filter_url);
+			$status_list_items[] = sprintf($status_list_item, esc_attr($status_name), esc_url($status_filter_url), esc_html(ucfirst($status_name)), absint($count));
 		}
 
-		if ( $status_list_items ) {
+		if ($status_list_items) {
 			echo '<ul class="subsubsub">';
-			echo implode( " | \n", $status_list_items );
+			echo implode(" | \n", $status_list_items);
 			echo '</ul>';
 		}
 	}
@@ -623,15 +623,15 @@ 
 	 * forget about any extra HTML.
 	 */
 	protected function display_table() {
-		echo '<form id="' . esc_attr( $this->_args['plural'] ) . '-filter" method="get">';
-		foreach ( $_GET as $key => $value ) {
-			if ( '_' === $key[0] || 'paged' === $key ) {
+		echo '<form id="' . esc_attr($this->_args['plural']) . '-filter" method="get">';
+		foreach ($_GET as $key => $value) {
+			if ('_' === $key[0] || 'paged' === $key) {
 				continue;
 			}
-			echo '<input type="hidden" name="' . esc_attr( $key ) . '" value="' . esc_attr( $value ) . '" />';
+			echo '<input type="hidden" name="' . esc_attr($key) . '" value="' . esc_attr($value) . '" />';
 		}
-		if ( ! empty( $this->search_by ) ) {
-			echo $this->search_box( $this->get_search_box_button_text(), 'plugin' ); // WPCS: XSS OK
+		if (!empty($this->search_by)) {
+			echo $this->search_box($this->get_search_box_button_text(), 'plugin'); // WPCS: XSS OK
 		}
 		parent::display();
 		echo '</form>';
@@ -644,9 +644,9 @@ 
 		$this->process_bulk_action();
 		$this->process_row_actions();
 
-		if ( ! empty( $_REQUEST['_wp_http_referer'] ) ) {
+		if (!empty($_REQUEST['_wp_http_referer'])) {
 			// _wp_http_referer is used only on bulk actions, we remove it to keep the $_GET shorter
-			wp_redirect( remove_query_arg( array( '_wp_http_referer', '_wpnonce' ), wp_unslash( $_SERVER['REQUEST_URI'] ) ) );
+			wp_redirect(remove_query_arg(array('_wp_http_referer', '_wpnonce'), wp_unslash($_SERVER['REQUEST_URI'])));
 			exit;
 		}
 	}
@@ -669,6 +669,6 @@ 
 	 * Get the text to display in the search box on the list table.
 	 */
 	protected function get_search_box_placeholder() {
-		return esc_html__( 'Search', 'action-scheduler' );
+		return esc_html__('Search', 'action-scheduler');
 	}
 }

includes/libraries/action-scheduler/classes/abstracts/ActionScheduler_Abstract_RecurringSchedule.php

Spacing +7 added lines, -7 removed lines

@@ -39,9 +39,9 @@ 
 	 * @param mixed $recurrence The data used to determine the schedule's recurrance.
 	 * @param DateTime|null $first (Optional) The date & time the first instance of this interval schedule ran. Default null, meaning this is the first instance.
 	 */
-	public function __construct( DateTime $date, $recurrence, DateTime $first = null ) {
-		parent::__construct( $date );
-		$this->first_date = empty( $first ) ? $date : $first;
+	public function __construct(DateTime $date, $recurrence, DateTime $first = null) {
+		parent::__construct($date);
+		$this->first_date = empty($first) ? $date : $first;
 		$this->recurrence = $recurrence;
 	}
 
@@ -75,10 +75,10 @@ 
 	public function __sleep() {
 		$sleep_params = parent::__sleep();
 		$this->first_timestamp = $this->first_date->getTimestamp();
-		return array_merge( $sleep_params, array(
+		return array_merge($sleep_params, array(
 			'first_timestamp',
 			'recurrence'
-		) );
+		));
 	}
 
 	/**
@@ -93,8 +93,8 @@ 
 	 */
 	public function __wakeup() {
 		parent::__wakeup();
-		if ( $this->first_timestamp > 0 ) {
-			$this->first_date = as_get_datetime_object( $this->first_timestamp );
+		if ($this->first_timestamp > 0) {
+			$this->first_date = as_get_datetime_object($this->first_timestamp);
 		} else {
 			$this->first_date = $this->get_date();
 		}

includes/libraries/action-scheduler/classes/abstracts/ActionScheduler_TimezoneHelper.php

Spacing +37 added lines, -37 removed lines

@@ -15,17 +15,17 @@ 
 	 * @param DateTime $date
 	 * @return ActionScheduler_DateTime
 	 */
-	public static function set_local_timezone( DateTime $date ) {
+	public static function set_local_timezone(DateTime $date) {
 
 		// Accept a DateTime for easier backward compatibility, even though we require methods on ActionScheduler_DateTime
-		if ( ! is_a( $date, 'ActionScheduler_DateTime' ) ) {
-			$date = as_get_datetime_object( $date->format( 'U' ) );
+		if (!is_a($date, 'ActionScheduler_DateTime')) {
+			$date = as_get_datetime_object($date->format('U'));
 		}
 
-		if ( get_option( 'timezone_string' ) ) {
-			$date->setTimezone( new DateTimeZone( self::get_local_timezone_string() ) );
+		if (get_option('timezone_string')) {
+			$date->setTimezone(new DateTimeZone(self::get_local_timezone_string()));
 		} else {
-			$date->setUtcOffset( self::get_local_timezone_offset() );
+			$date->setUtcOffset(self::get_local_timezone_offset());
 		}
 
 		return $date;
@@ -44,16 +44,16 @@ 
 	 * @since 2.1.0
 	 * @return string PHP timezone string for the site or empty if no timezone string is available.
 	 */
-	protected static function get_local_timezone_string( $reset = false ) {
+	protected static function get_local_timezone_string($reset = false) {
 		// If site timezone string exists, return it.
-		$timezone = get_option( 'timezone_string' );
-		if ( $timezone ) {
+		$timezone = get_option('timezone_string');
+		if ($timezone) {
 			return $timezone;
 		}
 
 		// Get UTC offset, if it isn't set then return UTC.
-		$utc_offset = intval( get_option( 'gmt_offset', 0 ) );
-		if ( 0 === $utc_offset ) {
+		$utc_offset = intval(get_option('gmt_offset', 0));
+		if (0 === $utc_offset) {
 			return 'UTC';
 		}
 
@@ -61,15 +61,15 @@ 
 		$utc_offset *= 3600;
 
 		// Attempt to guess the timezone string from the UTC offset.
-		$timezone = timezone_name_from_abbr( '', $utc_offset );
-		if ( $timezone ) {
+		$timezone = timezone_name_from_abbr('', $utc_offset);
+		if ($timezone) {
 			return $timezone;
 		}
 
 		// Last try, guess timezone string manually.
-		foreach ( timezone_abbreviations_list() as $abbr ) {
-			foreach ( $abbr as $city ) {
-				if ( (bool) date( 'I' ) === (bool) $city['dst'] && $city['timezone_id'] && intval( $city['offset'] ) === $utc_offset ) {
+		foreach (timezone_abbreviations_list() as $abbr) {
+			foreach ($abbr as $city) {
+				if ((bool) date('I') === (bool) $city['dst'] && $city['timezone_id'] && intval($city['offset']) === $utc_offset) {
 					return $city['timezone_id'];
 				}
 			}
@@ -86,48 +86,48 @@ 
 	 * @return float
 	 */
 	protected static function get_local_timezone_offset() {
-		$timezone = get_option( 'timezone_string' );
+		$timezone = get_option('timezone_string');
 
-		if ( $timezone ) {
-			$timezone_object = new DateTimeZone( $timezone );
-			return $timezone_object->getOffset( new DateTime( 'now' ) );
+		if ($timezone) {
+			$timezone_object = new DateTimeZone($timezone);
+			return $timezone_object->getOffset(new DateTime('now'));
 		} else {
-			return floatval( get_option( 'gmt_offset', 0 ) ) * HOUR_IN_SECONDS;
+			return floatval(get_option('gmt_offset', 0)) * HOUR_IN_SECONDS;
 		}
 	}
 
 	/**
 	 * @deprecated 2.1.0
 	 */
-	public static function get_local_timezone( $reset = FALSE ) {
-		_deprecated_function( __FUNCTION__, '2.1.0', 'ActionScheduler_TimezoneHelper::set_local_timezone()' );
-		if ( $reset ) {
+	public static function get_local_timezone($reset = FALSE) {
+		_deprecated_function(__FUNCTION__, '2.1.0', 'ActionScheduler_TimezoneHelper::set_local_timezone()');
+		if ($reset) {
 			self::$local_timezone = NULL;
 		}
-		if ( !isset(self::$local_timezone) ) {
+		if (!isset(self::$local_timezone)) {
 			$tzstring = get_option('timezone_string');
 
-			if ( empty($tzstring) ) {
+			if (empty($tzstring)) {
 				$gmt_offset = get_option('gmt_offset');
-				if ( $gmt_offset == 0 ) {
+				if ($gmt_offset == 0) {
 					$tzstring = 'UTC';
 				} else {
 					$gmt_offset *= HOUR_IN_SECONDS;
-					$tzstring   = timezone_name_from_abbr( '', $gmt_offset, 1 );
+					$tzstring = timezone_name_from_abbr('', $gmt_offset, 1);
 
 					// If there's no timezone string, try again with no DST.
-					if ( false === $tzstring ) {
-						$tzstring = timezone_name_from_abbr( '', $gmt_offset, 0 );
+					if (false === $tzstring) {
+						$tzstring = timezone_name_from_abbr('', $gmt_offset, 0);
 					}
 
 					// Try mapping to the first abbreviation we can find.
-					if ( false === $tzstring ) {
-						$is_dst = date( 'I' );
-						foreach ( timezone_abbreviations_list() as $abbr ) {
-							foreach ( $abbr as $city ) {
-								if ( $city['dst'] == $is_dst && $city['offset'] == $gmt_offset ) {
+					if (false === $tzstring) {
+						$is_dst = date('I');
+						foreach (timezone_abbreviations_list() as $abbr) {
+							foreach ($abbr as $city) {
+								if ($city['dst'] == $is_dst && $city['offset'] == $gmt_offset) {
 									// If there's no valid timezone ID, keep looking.
-									if ( null === $city['timezone_id'] ) {
+									if (null === $city['timezone_id']) {
 										continue;
 									}
 
@@ -139,7 +139,7 @@ 
 					}
 
 					// If we still have no valid string, then fall back to UTC.
-					if ( false === $tzstring ) {
+					if (false === $tzstring) {
 						$tzstring = 'UTC';
 					}
 				}

includes/libraries/action-scheduler/classes/abstracts/ActionScheduler_Store.php

Spacing +58 added lines, -58 removed lines

@@ -26,21 +26,21 @@ 
 	 *
 	 * @return string The action ID
 	 */
-	abstract public function save_action( ActionScheduler_Action $action, DateTime $scheduled_date = NULL );
+	abstract public function save_action(ActionScheduler_Action $action, DateTime $scheduled_date = NULL);
 
 	/**
 	 * @param string $action_id
 	 *
 	 * @return ActionScheduler_Action
 	 */
-	abstract public function fetch_action( $action_id );
+	abstract public function fetch_action($action_id);
 
 	/**
 	 * @param string $hook Hook name/slug.
 	 * @param array  $params Hook arguments.
 	 * @return string ID of the next action matching the criteria.
 	 */
-	abstract public function find_action( $hook, $params = array() );
+	abstract public function find_action($hook, $params = array());
 
 	/**
 	 * @param array  $query Query parameters.
@@ -48,7 +48,7 @@ 
 	 *
 	 * @return array|int The IDs of or count of actions matching the query.
 	 */
-	abstract public function query_actions( $query = array(), $query_type = 'select' );
+	abstract public function query_actions($query = array(), $query_type = 'select');
 
 	/**
 	 * Get a count of all actions in the store, grouped by status
@@ -60,19 +60,19 @@ 
 	/**
 	 * @param string $action_id
 	 */
-	abstract public function cancel_action( $action_id );
+	abstract public function cancel_action($action_id);
 
 	/**
 	 * @param string $action_id
 	 */
-	abstract public function delete_action( $action_id );
+	abstract public function delete_action($action_id);
 
 	/**
 	 * @param string $action_id
 	 *
 	 * @return DateTime The date the action is schedule to run, or the date that it ran.
 	 */
-	abstract public function get_date( $action_id );
+	abstract public function get_date($action_id);
 
 
 	/**
@@ -83,7 +83,7 @@ 
 	 *
 	 * @return ActionScheduler_ActionClaim
 	 */
-	abstract public function stake_claim( $max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '' );
+	abstract public function stake_claim($max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '');
 
 	/**
 	 * @return int
@@ -93,53 +93,53 @@ 
 	/**
 	 * @param ActionScheduler_ActionClaim $claim
 	 */
-	abstract public function release_claim( ActionScheduler_ActionClaim $claim );
+	abstract public function release_claim(ActionScheduler_ActionClaim $claim);
 
 	/**
 	 * @param string $action_id
 	 */
-	abstract public function unclaim_action( $action_id );
+	abstract public function unclaim_action($action_id);
 
 	/**
 	 * @param string $action_id
 	 */
-	abstract public function mark_failure( $action_id );
+	abstract public function mark_failure($action_id);
 
 	/**
 	 * @param string $action_id
 	 */
-	abstract public function log_execution( $action_id );
+	abstract public function log_execution($action_id);
 
 	/**
 	 * @param string $action_id
 	 */
-	abstract public function mark_complete( $action_id );
+	abstract public function mark_complete($action_id);
 
 	/**
 	 * @param string $action_id
 	 *
 	 * @return string
 	 */
-	abstract public function get_status( $action_id );
+	abstract public function get_status($action_id);
 
 	/**
 	 * @param string $action_id
 	 * @return mixed
 	 */
-	abstract public function get_claim_id( $action_id );
+	abstract public function get_claim_id($action_id);
 
 	/**
 	 * @param string $claim_id
 	 * @return array
 	 */
-	abstract public function find_actions_by_claim_id( $claim_id );
+	abstract public function find_actions_by_claim_id($claim_id);
 
 	/**
 	 * @param string $comparison_operator
 	 * @return string
 	 */
-	protected function validate_sql_comparator( $comparison_operator ) {
-		if ( in_array( $comparison_operator, array('!=', '>', '>=', '<', '<=', '=') ) ) {
+	protected function validate_sql_comparator($comparison_operator) {
+		if (in_array($comparison_operator, array('!=', '>', '>=', '<', '<=', '='))) {
 			return $comparison_operator;
 		}
 		return '=';
@@ -152,14 +152,14 @@ 
 	 * @param DateTime $scheduled_date (optional)
 	 * @return string
 	 */
-	protected function get_scheduled_date_string( ActionScheduler_Action $action, DateTime $scheduled_date = NULL ) {
+	protected function get_scheduled_date_string(ActionScheduler_Action $action, DateTime $scheduled_date = NULL) {
 		$next = null === $scheduled_date ? $action->get_schedule()->get_date() : $scheduled_date;
-		if ( ! $next ) {
+		if (!$next) {
 			return '0000-00-00 00:00:00';
 		}
-		$next->setTimezone( new DateTimeZone( 'UTC' ) );
+		$next->setTimezone(new DateTimeZone('UTC'));
 
-		return $next->format( 'Y-m-d H:i:s' );
+		return $next->format('Y-m-d H:i:s');
 	}
 
 	/**
@@ -169,14 +169,14 @@ 
 	 * @param DateTime $scheduled_date (optional)
 	 * @return string
 	 */
-	protected function get_scheduled_date_string_local( ActionScheduler_Action $action, DateTime $scheduled_date = NULL ) {
+	protected function get_scheduled_date_string_local(ActionScheduler_Action $action, DateTime $scheduled_date = NULL) {
 		$next = null === $scheduled_date ? $action->get_schedule()->get_date() : $scheduled_date;
-		if ( ! $next ) {
+		if (!$next) {
 			return '0000-00-00 00:00:00';
 		}
 
-		ActionScheduler_TimezoneHelper::set_local_timezone( $next );
-		return $next->format( 'Y-m-d H:i:s' );
+		ActionScheduler_TimezoneHelper::set_local_timezone($next);
+		return $next->format('Y-m-d H:i:s');
 	}
 
 	/**
@@ -187,15 +187,15 @@ 
 	 *
 	 * @throws ActionScheduler_InvalidActionException When the decoded arguments are invalid.
 	 */
-	protected function validate_args( $args, $action_id ) {
+	protected function validate_args($args, $action_id) {
 		// Ensure we have an array of args.
-		if ( ! is_array( $args ) ) {
-			throw ActionScheduler_InvalidActionException::from_decoding_args( $action_id );
+		if (!is_array($args)) {
+			throw ActionScheduler_InvalidActionException::from_decoding_args($action_id);
 		}
 
 		// Validate JSON decoding if possible.
-		if ( function_exists( 'json_last_error' ) && JSON_ERROR_NONE !== json_last_error() ) {
-			throw ActionScheduler_InvalidActionException::from_decoding_args( $action_id, $args );
+		if (function_exists('json_last_error') && JSON_ERROR_NONE !== json_last_error()) {
+			throw ActionScheduler_InvalidActionException::from_decoding_args($action_id, $args);
 		}
 	}
 
@@ -207,9 +207,9 @@ 
 	 *
 	 * @throws ActionScheduler_InvalidActionException When the schedule is invalid.
 	 */
-	protected function validate_schedule( $schedule, $action_id ) {
-		if ( empty( $schedule ) || ! is_a( $schedule, 'ActionScheduler_Schedule' ) ) {
-			throw ActionScheduler_InvalidActionException::from_schedule( $action_id, $schedule );
+	protected function validate_schedule($schedule, $action_id) {
+		if (empty($schedule) || !is_a($schedule, 'ActionScheduler_Schedule')) {
+			throw ActionScheduler_InvalidActionException::from_schedule($action_id, $schedule);
 		}
 	}
 
@@ -222,9 +222,9 @@ 
 	 * @param  ActionScheduler_Action $action Action to be validated.
 	 * @throws InvalidArgumentException When json encoded args is too long.
 	 */
-	protected function validate_action( ActionScheduler_Action $action ) {
-		if ( strlen( json_encode( $action->get_args() ) ) > static::$max_args_length ) {
-			throw new InvalidArgumentException( sprintf( __( 'ActionScheduler_Action::$args too long. To ensure the args column can be indexed, action args should not be more than %d characters when encoded as JSON.', 'action-scheduler' ), static::$max_args_length ) );
+	protected function validate_action(ActionScheduler_Action $action) {
+		if (strlen(json_encode($action->get_args())) > static::$max_args_length) {
+			throw new InvalidArgumentException(sprintf(__('ActionScheduler_Action::$args too long. To ensure the args column can be indexed, action args should not be more than %d characters when encoded as JSON.', 'action-scheduler'), static::$max_args_length));
 		}
 	}
 
@@ -237,9 +237,9 @@ 
 	 *
 	 * @return void
 	 */
-	public function cancel_actions_by_hook( $hook ) {
+	public function cancel_actions_by_hook($hook) {
 		$action_ids = true;
-		while ( ! empty( $action_ids ) ) {
+		while (!empty($action_ids)) {
 			$action_ids = $this->query_actions(
 				array(
 					'hook' => $hook,
@@ -248,7 +248,7 @@ 
 				)
 			);
 
-			$this->bulk_cancel_actions( $action_ids );
+			$this->bulk_cancel_actions($action_ids);
 		}
 	}
 
@@ -261,9 +261,9 @@ 
 	 *
 	 * @return void
 	 */
-	public function cancel_actions_by_group( $group ) {
+	public function cancel_actions_by_group($group) {
 		$action_ids = true;
-		while ( ! empty( $action_ids ) ) {
+		while (!empty($action_ids)) {
 			$action_ids = $this->query_actions(
 				array(
 					'group' => $group,
@@ -272,7 +272,7 @@ 
 				)
 			);
 
-			$this->bulk_cancel_actions( $action_ids );
+			$this->bulk_cancel_actions($action_ids);
 		}
 	}
 
@@ -285,12 +285,12 @@ 
 	 *
 	 * @return void
 	 */
-	private function bulk_cancel_actions( $action_ids ) {
-		foreach ( $action_ids as $action_id ) {
-			$this->cancel_action( $action_id );
+	private function bulk_cancel_actions($action_ids) {
+		foreach ($action_ids as $action_id) {
+			$this->cancel_action($action_id);
 		}
 
-		do_action( 'action_scheduler_bulk_cancel_actions', $action_ids );
+		do_action('action_scheduler_bulk_cancel_actions', $action_ids);
 	}
 
 	/**
@@ -298,11 +298,11 @@ 
 	 */
 	public function get_status_labels() {
 		return array(
-			self::STATUS_COMPLETE => __( 'Complete', 'action-scheduler' ),
-			self::STATUS_PENDING  => __( 'Pending', 'action-scheduler' ),
-			self::STATUS_RUNNING  => __( 'In-progress', 'action-scheduler' ),
-			self::STATUS_FAILED   => __( 'Failed', 'action-scheduler' ),
-			self::STATUS_CANCELED => __( 'Canceled', 'action-scheduler' ),
+			self::STATUS_COMPLETE => __('Complete', 'action-scheduler'),
+			self::STATUS_PENDING  => __('Pending', 'action-scheduler'),
+			self::STATUS_RUNNING  => __('In-progress', 'action-scheduler'),
+			self::STATUS_FAILED   => __('Failed', 'action-scheduler'),
+			self::STATUS_CANCELED => __('Canceled', 'action-scheduler'),
 		);
 	}
 
@@ -314,12 +314,12 @@ 
 	 * @return string
 	 */
 	public function has_pending_actions_due() {
-		$pending_actions = $this->query_actions( array(
+		$pending_actions = $this->query_actions(array(
 			'date'   => as_get_datetime_object(),
 			'status' => ActionScheduler_Store::STATUS_PENDING,
-		) );
+		));
 
-		return ! empty( $pending_actions );
+		return !empty($pending_actions);
 	}
 
 	/**
@@ -330,14 +330,14 @@ 
 	/**
 	 * Callable function to mark an action as migrated optionally overridden in derived classes.
 	 */
-	public function mark_migrated( $action_id ) {}
+	public function mark_migrated($action_id) {}
 
 	/**
 	 * @return ActionScheduler_Store
 	 */
 	public static function instance() {
-		if ( empty( self::$store ) ) {
-			$class = apply_filters( 'action_scheduler_store_class', self::DEFAULT_CLASS );
+		if (empty(self::$store)) {
+			$class = apply_filters('action_scheduler_store_class', self::DEFAULT_CLASS);
 			self::$store = new $class();
 		}
 		return self::$store;

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

Spacing +7 added lines, -7 removed lines

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

includes/libraries/action-scheduler/classes/abstracts/ActionScheduler_Logger.php

Spacing +59 added lines, -59 removed lines

@@ -11,7 +11,7 @@ 
 	 * @return ActionScheduler_Logger
 	 */
 	public static function instance() {
-		if ( empty(self::$logger) ) {
+		if (empty(self::$logger)) {
 			$class = apply_filters('action_scheduler_logger_class', 'ActionScheduler_wpCommentLogger');
 			self::$logger = new $class();
 		}
@@ -25,21 +25,21 @@ 
 	 *
 	 * @return string The log entry ID
 	 */
-	abstract public function log( $action_id, $message, DateTime $date = NULL );
+	abstract public function log($action_id, $message, DateTime $date = NULL);
 
 	/**
 	 * @param string $entry_id
 	 *
 	 * @return ActionScheduler_LogEntry
 	 */
-	abstract public function get_entry( $entry_id );
+	abstract public function get_entry($entry_id);
 
 	/**
 	 * @param string $action_id
 	 *
 	 * @return ActionScheduler_LogEntry[]
 	 */
-	abstract public function get_logs( $action_id );
+	abstract public function get_logs($action_id);
 
 
 	/**
@@ -47,90 +47,90 @@ 
 	 */
 	public function init() {
 		$this->hook_stored_action();
-		add_action( 'action_scheduler_canceled_action', array( $this, 'log_canceled_action' ), 10, 1 );
-		add_action( 'action_scheduler_begin_execute', array( $this, 'log_started_action' ), 10, 2 );
-		add_action( 'action_scheduler_after_execute', array( $this, 'log_completed_action' ), 10, 3 );
-		add_action( 'action_scheduler_failed_execution', array( $this, 'log_failed_action' ), 10, 3 );
-		add_action( 'action_scheduler_failed_action', array( $this, 'log_timed_out_action' ), 10, 2 );
-		add_action( 'action_scheduler_unexpected_shutdown', array( $this, 'log_unexpected_shutdown' ), 10, 2 );
-		add_action( 'action_scheduler_reset_action', array( $this, 'log_reset_action' ), 10, 1 );
-		add_action( 'action_scheduler_execution_ignored', array( $this, 'log_ignored_action' ), 10, 2 );
-		add_action( 'action_scheduler_failed_fetch_action', array( $this, 'log_failed_fetch_action' ), 10, 2 );
-		add_action( 'action_scheduler_failed_to_schedule_next_instance', array( $this, 'log_failed_schedule_next_instance' ), 10, 2 );
-		add_action( 'action_scheduler_bulk_cancel_actions', array( $this, 'bulk_log_cancel_actions' ), 10, 1 );
+		add_action('action_scheduler_canceled_action', array($this, 'log_canceled_action'), 10, 1);
+		add_action('action_scheduler_begin_execute', array($this, 'log_started_action'), 10, 2);
+		add_action('action_scheduler_after_execute', array($this, 'log_completed_action'), 10, 3);
+		add_action('action_scheduler_failed_execution', array($this, 'log_failed_action'), 10, 3);
+		add_action('action_scheduler_failed_action', array($this, 'log_timed_out_action'), 10, 2);
+		add_action('action_scheduler_unexpected_shutdown', array($this, 'log_unexpected_shutdown'), 10, 2);
+		add_action('action_scheduler_reset_action', array($this, 'log_reset_action'), 10, 1);
+		add_action('action_scheduler_execution_ignored', array($this, 'log_ignored_action'), 10, 2);
+		add_action('action_scheduler_failed_fetch_action', array($this, 'log_failed_fetch_action'), 10, 2);
+		add_action('action_scheduler_failed_to_schedule_next_instance', array($this, 'log_failed_schedule_next_instance'), 10, 2);
+		add_action('action_scheduler_bulk_cancel_actions', array($this, 'bulk_log_cancel_actions'), 10, 1);
 	}
 
 	public function hook_stored_action() {
-		add_action( 'action_scheduler_stored_action', array( $this, 'log_stored_action' ) );
+		add_action('action_scheduler_stored_action', array($this, 'log_stored_action'));
 	}
 
 	public function unhook_stored_action() {
-		remove_action( 'action_scheduler_stored_action', array( $this, 'log_stored_action' ) );
+		remove_action('action_scheduler_stored_action', array($this, 'log_stored_action'));
 	}
 
-	public function log_stored_action( $action_id ) {
-		$this->log( $action_id, __( 'action created', 'action-scheduler' ) );
+	public function log_stored_action($action_id) {
+		$this->log($action_id, __('action created', 'action-scheduler'));
 	}
 
-	public function log_canceled_action( $action_id ) {
-		$this->log( $action_id, __( 'action canceled', 'action-scheduler' ) );
+	public function log_canceled_action($action_id) {
+		$this->log($action_id, __('action canceled', 'action-scheduler'));
 	}
 
-	public function log_started_action( $action_id, $context = '' ) {
-		if ( ! empty( $context ) ) {
+	public function log_started_action($action_id, $context = '') {
+		if (!empty($context)) {
 			/* translators: %s: context */
-			$message = sprintf( __( 'action started via %s', 'action-scheduler' ), $context );
+			$message = sprintf(__('action started via %s', 'action-scheduler'), $context);
 		} else {
-			$message = __( 'action started', 'action-scheduler' );
+			$message = __('action started', 'action-scheduler');
 		}
-		$this->log( $action_id, $message );
+		$this->log($action_id, $message);
 	}
 
-	public function log_completed_action( $action_id, $action = NULL, $context = '' ) {
-		if ( ! empty( $context ) ) {
+	public function log_completed_action($action_id, $action = NULL, $context = '') {
+		if (!empty($context)) {
 			/* translators: %s: context */
-			$message = sprintf( __( 'action complete via %s', 'action-scheduler' ), $context );
+			$message = sprintf(__('action complete via %s', 'action-scheduler'), $context);
 		} else {
-			$message = __( 'action complete', 'action-scheduler' );
+			$message = __('action complete', 'action-scheduler');
 		}
-		$this->log( $action_id, $message );
+		$this->log($action_id, $message);
 	}
 
-	public function log_failed_action( $action_id, Exception $exception, $context = '' ) {
-		if ( ! empty( $context ) ) {
+	public function log_failed_action($action_id, Exception $exception, $context = '') {
+		if (!empty($context)) {
 			/* translators: 1: context 2: exception message */
-			$message = sprintf( __( 'action failed via %1$s: %2$s', 'action-scheduler' ), $context, $exception->getMessage() );
+			$message = sprintf(__('action failed via %1$s: %2$s', 'action-scheduler'), $context, $exception->getMessage());
 		} else {
 			/* translators: %s: exception message */
-			$message = sprintf( __( 'action failed: %s', 'action-scheduler' ), $exception->getMessage() );
+			$message = sprintf(__('action failed: %s', 'action-scheduler'), $exception->getMessage());
 		}
-		$this->log( $action_id, $message );
+		$this->log($action_id, $message);
 	}
 
-	public function log_timed_out_action( $action_id, $timeout ) {
+	public function log_timed_out_action($action_id, $timeout) {
 		/* translators: %s: amount of time */
-		$this->log( $action_id, sprintf( __( 'action timed out after %s seconds', 'action-scheduler' ), $timeout ) );
+		$this->log($action_id, sprintf(__('action timed out after %s seconds', 'action-scheduler'), $timeout));
 	}
 
-	public function log_unexpected_shutdown( $action_id, $error ) {
-		if ( ! empty( $error ) ) {
+	public function log_unexpected_shutdown($action_id, $error) {
+		if (!empty($error)) {
 			/* translators: 1: error message 2: filename 3: line */
-			$this->log( $action_id, sprintf( __( 'unexpected shutdown: PHP Fatal error %1$s in %2$s on line %3$s', 'action-scheduler' ), $error['message'], $error['file'], $error['line'] ) );
+			$this->log($action_id, sprintf(__('unexpected shutdown: PHP Fatal error %1$s in %2$s on line %3$s', 'action-scheduler'), $error['message'], $error['file'], $error['line']));
 		}
 	}
 
-	public function log_reset_action( $action_id ) {
-		$this->log( $action_id, __( 'action reset', 'action-scheduler' ) );
+	public function log_reset_action($action_id) {
+		$this->log($action_id, __('action reset', 'action-scheduler'));
 	}
 
-	public function log_ignored_action( $action_id, $context = '' ) {
-		if ( ! empty( $context ) ) {
+	public function log_ignored_action($action_id, $context = '') {
+		if (!empty($context)) {
 			/* translators: %s: context */
-			$message = sprintf( __( 'action ignored via %s', 'action-scheduler' ), $context );
+			$message = sprintf(__('action ignored via %s', 'action-scheduler'), $context);
 		} else {
-			$message = __( 'action ignored', 'action-scheduler' );
+			$message = __('action ignored', 'action-scheduler');
 		}
-		$this->log( $action_id, $message );
+		$this->log($action_id, $message);
 	}
 
 	/**
@@ -139,21 +139,21 @@ 
 	 *
 	 * @return ActionScheduler_LogEntry[]
 	 */
-	public function log_failed_fetch_action( $action_id, Exception $exception = NULL ) {
+	public function log_failed_fetch_action($action_id, Exception $exception = NULL) {
 
-		if ( ! is_null( $exception ) ) {
+		if (!is_null($exception)) {
 			/* translators: %s: exception message */
-			$log_message = sprintf( __( 'There was a failure fetching this action: %s', 'action-scheduler' ), $exception->getMessage() );
+			$log_message = sprintf(__('There was a failure fetching this action: %s', 'action-scheduler'), $exception->getMessage());
 		} else {
-			$log_message = __( 'There was a failure fetching this action', 'action-scheduler' );
+			$log_message = __('There was a failure fetching this action', 'action-scheduler');
 		}
 
-		$this->log( $action_id, $log_message );
+		$this->log($action_id, $log_message);
 	}
 
-	public function log_failed_schedule_next_instance( $action_id, Exception $exception ) {
+	public function log_failed_schedule_next_instance($action_id, Exception $exception) {
 		/* translators: %s: exception message */
-		$this->log( $action_id, sprintf( __( 'There was a failure scheduling the next instance of this action: %s', 'action-scheduler' ), $exception->getMessage() ) );
+		$this->log($action_id, sprintf(__('There was a failure scheduling the next instance of this action: %s', 'action-scheduler'), $exception->getMessage()));
 	}
 
 	/**
@@ -164,13 +164,13 @@ 
 	 *
 	 * @param array $action_ids List of action ID.
 	 */
-	public function bulk_log_cancel_actions( $action_ids ) {
-		if ( empty( $action_ids ) ) {
+	public function bulk_log_cancel_actions($action_ids) {
+		if (empty($action_ids)) {
 			return;
 		}
 
-		foreach ( $action_ids as $action_id ) {
-			$this->log_canceled_action( $action_id );
+		foreach ($action_ids as $action_id) {
+			$this->log_canceled_action($action_id);
 		}
 	}
 }

includes/libraries/action-scheduler/classes/abstracts/ActionScheduler_Lock.php

Spacing +8 added lines, -8 removed lines

@@ -19,8 +19,8 @@ 
 	 * @param string $lock_type A string to identify different lock types.
 	 * @return bool
 	 */
-	public function is_locked( $lock_type ) {
-		return ( $this->get_expiration( $lock_type ) >= time() );
+	public function is_locked($lock_type) {
+		return ($this->get_expiration($lock_type) >= time());
 	}
 
 	/**
@@ -29,7 +29,7 @@ 
 	 * @param string $lock_type A string to identify different lock types.
 	 * @return bool
 	 */
-	abstract public function set( $lock_type );
+	abstract public function set($lock_type);
 
 	/**
 	 * If a lock is set, return the timestamp it was set to expiry.
@@ -37,7 +37,7 @@ 
 	 * @param string $lock_type A string to identify different lock types.
 	 * @return bool|int False if no lock is set, otherwise the timestamp for when the lock is set to expire.
 	 */
-	abstract public function get_expiration( $lock_type );
+	abstract public function get_expiration($lock_type);
 
 	/**
 	 * Get the amount of time to set for a given lock. 60 seconds by default.
@@ -45,16 +45,16 @@ 
 	 * @param string $lock_type A string to identify different lock types.
 	 * @return int
 	 */
-	protected function get_duration( $lock_type ) {
-		return apply_filters( 'action_scheduler_lock_duration', self::$lock_duration, $lock_type );
+	protected function get_duration($lock_type) {
+		return apply_filters('action_scheduler_lock_duration', self::$lock_duration, $lock_type);
 	}
 
 	/**
 	 * @return ActionScheduler_Lock
 	 */
 	public static function instance() {
-		if ( empty( self::$locker ) ) {
-			$class = apply_filters( 'action_scheduler_lock_class', 'ActionScheduler_OptionLock' );
+		if (empty(self::$locker)) {
+			$class = apply_filters('action_scheduler_lock_class', 'ActionScheduler_OptionLock');
 			self::$locker = new $class();
 		}
 		return self::$locker;