insideout10 / wordlift-plugin

src/libraries/action-scheduler/classes/data-stores/ActionScheduler_wpPostStore.php

Indentation +1059 added lines, -1059 removed lines

@@ -4,1072 +4,1072 @@
  * Class ActionScheduler_wpPostStore
  */
 class ActionScheduler_wpPostStore extends ActionScheduler_Store {
-	const POST_TYPE         = 'scheduled-action';
-	const GROUP_TAXONOMY    = 'action-group';
-	const SCHEDULE_META_KEY = '_action_manager_schedule';
-	const DEPENDENCIES_MET  = 'as-post-store-dependencies-met';
-
-	/**
-	 * Used to share information about the before_date property of claims internally.
-	 *
-	 * This is used in preference to passing the same information as a method param
-	 * for backwards-compatibility reasons.
-	 *
-	 * @var DateTime|null
-	 */
-	private $claim_before_date = null;
-
-	/**
-	 * Local Timezone.
-	 *
-	 * @var DateTimeZone
-	 */
-	protected $local_timezone = null;
-
-	/**
-	 * Save action.
-	 *
-	 * @param ActionScheduler_Action $action Scheduled Action.
-	 * @param DateTime               $scheduled_date Scheduled Date.
-	 *
-	 * @throws RuntimeException Throws an exception if the action could not be saved.
-	 * @return int
-	 */
-	public function save_action( ActionScheduler_Action $action, DateTime $scheduled_date = null ) {
-		try {
-			$this->validate_action( $action );
-			$post_array = $this->create_post_array( $action, $scheduled_date );
-			$post_id    = $this->save_post_array( $post_array );
-			$this->save_post_schedule( $post_id, $action->get_schedule() );
-			$this->save_action_group( $post_id, $action->get_group() );
-			do_action( 'action_scheduler_stored_action', $post_id );
-			return $post_id;
-		} catch ( Exception $e ) {
-			/* translators: %s: action error message */
-			throw new RuntimeException( sprintf( __( 'Error saving action: %s', 'action-scheduler' ), $e->getMessage() ), 0 );
-		}
-	}
-
-	/**
-	 * Create post array.
-	 *
-	 * @param ActionScheduler_Action $action Scheduled Action.
-	 * @param DateTime               $scheduled_date Scheduled Date.
-	 *
-	 * @return array Returns an array of post data.
-	 */
-	protected function create_post_array( ActionScheduler_Action $action, DateTime $scheduled_date = null ) {
-		$post = array(
-			'post_type'     => self::POST_TYPE,
-			'post_title'    => $action->get_hook(),
-			'post_content'  => wp_json_encode( $action->get_args() ),
-			'post_status'   => ( $action->is_finished() ? 'publish' : 'pending' ),
-			'post_date_gmt' => $this->get_scheduled_date_string( $action, $scheduled_date ),
-			'post_date'     => $this->get_scheduled_date_string_local( $action, $scheduled_date ),
-		);
-		return $post;
-	}
-
-	/**
-	 * Save post array.
-	 *
-	 * @param array $post_array Post array.
-	 * @return int Returns the post ID.
-	 * @throws RuntimeException Throws an exception if the action could not be saved.
-	 */
-	protected function save_post_array( $post_array ) {
-		add_filter( 'wp_insert_post_data', array( $this, 'filter_insert_post_data' ), 10, 1 );
-		add_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10, 5 );
-
-		$has_kses = false !== has_filter( 'content_save_pre', 'wp_filter_post_kses' );
-
-		if ( $has_kses ) {
-			// Prevent KSES from corrupting JSON in post_content.
-			kses_remove_filters();
-		}
-
-		$post_id = wp_insert_post( $post_array );
-
-		if ( $has_kses ) {
-			kses_init_filters();
-		}
-
-		remove_filter( 'wp_insert_post_data', array( $this, 'filter_insert_post_data' ), 10 );
-		remove_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10 );
-
-		if ( is_wp_error( $post_id ) || empty( $post_id ) ) {
-			throw new RuntimeException( __( 'Unable to save action.', 'action-scheduler' ) );
-		}
-		return $post_id;
-	}
-
-	/**
-	 * Filter insert post data.
-	 *
-	 * @param array $postdata Post data to filter.
-	 *
-	 * @return array
-	 */
-	public function filter_insert_post_data( $postdata ) {
-		if ( self::POST_TYPE === $postdata['post_type'] ) {
-			$postdata['post_author'] = 0;
-			if ( 'future' === $postdata['post_status'] ) {
-				$postdata['post_status'] = 'publish';
-			}
-		}
-		return $postdata;
-	}
-
-	/**
-	 * Create a (probably unique) post name for scheduled actions in a more performant manner than wp_unique_post_slug().
-	 *
-	 * When an action's post status is transitioned to something other than 'draft', 'pending' or 'auto-draft, like 'publish'
-	 * or 'failed' or 'trash', WordPress will find a unique slug (stored in post_name column) using the wp_unique_post_slug()
-	 * function. This is done to ensure URL uniqueness. The approach taken by wp_unique_post_slug() is to iterate over existing
-	 * post_name values that match, and append a number 1 greater than the largest. This makes sense when manually creating a
-	 * post from the Edit Post screen. It becomes a bottleneck when automatically processing thousands of actions, with a
-	 * database containing thousands of related post_name values.
-	 *
-	 * WordPress 5.1 introduces the 'pre_wp_unique_post_slug' filter for plugins to address this issue.
-	 *
-	 * We can short-circuit WordPress's wp_unique_post_slug() approach using the 'pre_wp_unique_post_slug' filter. This
-	 * method is available to be used as a callback on that filter. It provides a more scalable approach to generating a
-	 * post_name/slug that is probably unique. Because Action Scheduler never actually uses the post_name field, or an
-	 * action's slug, being probably unique is good enough.
-	 *
-	 * For more backstory on this issue, see:
-	 * - https://github.com/woocommerce/action-scheduler/issues/44 and
-	 * - https://core.trac.wordpress.org/ticket/21112
-	 *
-	 * @param string $override_slug Short-circuit return value.
-	 * @param string $slug          The desired slug (post_name).
-	 * @param int    $post_ID       Post ID.
-	 * @param string $post_status   The post status.
-	 * @param string $post_type     Post type.
-	 * @return string
-	 */
-	public function set_unique_post_slug( $override_slug, $slug, $post_ID, $post_status, $post_type ) {
-		if ( self::POST_TYPE === $post_type ) {
-			$override_slug = uniqid( self::POST_TYPE . '-', true ) . '-' . wp_generate_password( 32, false );
-		}
-		return $override_slug;
-	}
-
-	/**
-	 * Save post schedule.
-	 *
-	 * @param int    $post_id  Post ID of the scheduled action.
-	 * @param string $schedule Schedule to save.
-	 *
-	 * @return void
-	 */
-	protected function save_post_schedule( $post_id, $schedule ) {
-		update_post_meta( $post_id, self::SCHEDULE_META_KEY, $schedule );
-	}
-
-	/**
-	 * Save action group.
-	 *
-	 * @param int    $post_id Post ID.
-	 * @param string $group   Group to save.
-	 * @return void
-	 */
-	protected function save_action_group( $post_id, $group ) {
-		if ( empty( $group ) ) {
-			wp_set_object_terms( $post_id, array(), self::GROUP_TAXONOMY, false );
-		} else {
-			wp_set_object_terms( $post_id, array( $group ), self::GROUP_TAXONOMY, false );
-		}
-	}
-
-	/**
-	 * Fetch actions.
-	 *
-	 * @param int $action_id Action ID.
-	 * @return object
-	 */
-	public function fetch_action( $action_id ) {
-		$post = $this->get_post( $action_id );
-		if ( empty( $post ) || self::POST_TYPE !== $post->post_type ) {
-			return $this->get_null_action();
-		}
-
-		try {
-			$action = $this->make_action_from_post( $post );
-		} catch ( ActionScheduler_InvalidActionException $exception ) {
-			do_action( 'action_scheduler_failed_fetch_action', $post->ID, $exception );
-			return $this->get_null_action();
-		}
-
-		return $action;
-	}
-
-	/**
-	 * Get post.
-	 *
-	 * @param string $action_id - Action ID.
-	 * @return WP_Post|null
-	 */
-	protected function get_post( $action_id ) {
-		if ( empty( $action_id ) ) {
-			return null;
-		}
-		return get_post( $action_id );
-	}
-
-	/**
-	 * Get NULL action.
-	 *
-	 * @return ActionScheduler_NullAction
-	 */
-	protected function get_null_action() {
-		return new ActionScheduler_NullAction();
-	}
-
-	/**
-	 * Make action from post.
-	 *
-	 * @param WP_Post $post Post object.
-	 * @return WP_Post
-	 */
-	protected function make_action_from_post( $post ) {
-		$hook = $post->post_title;
-
-		$args = json_decode( $post->post_content, true );
-		$this->validate_args( $args, $post->ID );
-
-		$schedule = get_post_meta( $post->ID, self::SCHEDULE_META_KEY, true );
-		$this->validate_schedule( $schedule, $post->ID );
-
-		$group = wp_get_object_terms( $post->ID, self::GROUP_TAXONOMY, array( 'fields' => 'names' ) );
-		$group = empty( $group ) ? '' : reset( $group );
-
-		return ActionScheduler::factory()->get_stored_action( $this->get_action_status_by_post_status( $post->post_status ), $hook, $args, $schedule, $group );
-	}
-
-	/**
-	 * Get action status by post status.
-	 *
-	 * @param string $post_status Post status.
-	 *
-	 * @throws InvalidArgumentException Throw InvalidArgumentException if $post_status not in known status fields returned by $this->get_status_labels().
-	 * @return string
-	 */
-	protected function get_action_status_by_post_status( $post_status ) {
-
-		switch ( $post_status ) {
-			case 'publish':
-				$action_status = self::STATUS_COMPLETE;
-				break;
-			case 'trash':
-				$action_status = self::STATUS_CANCELED;
-				break;
-			default:
-				if ( ! array_key_exists( $post_status, $this->get_status_labels() ) ) {
-					throw new InvalidArgumentException( sprintf( 'Invalid post status: "%s". No matching action status available.', $post_status ) );
-				}
-				$action_status = $post_status;
-				break;
-		}
-
-		return $action_status;
-	}
-
-	/**
-	 * Get post status by action status.
-	 *
-	 * @param string $action_status Action status.
-	 *
-	 * @throws InvalidArgumentException Throws InvalidArgumentException if $post_status not in known status fields returned by $this->get_status_labels().
-	 * @return string
-	 */
-	protected function get_post_status_by_action_status( $action_status ) {
-
-		switch ( $action_status ) {
-			case self::STATUS_COMPLETE:
-				$post_status = 'publish';
-				break;
-			case self::STATUS_CANCELED:
-				$post_status = 'trash';
-				break;
-			default:
-				if ( ! array_key_exists( $action_status, $this->get_status_labels() ) ) {
-					throw new InvalidArgumentException( sprintf( 'Invalid action status: "%s".', $action_status ) );
-				}
-				$post_status = $action_status;
-				break;
-		}
-
-		return $post_status;
-	}
-
-	/**
-	 * Returns the SQL statement to query (or count) actions.
-	 *
-	 * @param array  $query            - Filtering options.
-	 * @param string $select_or_count  - Whether the SQL should select and return the IDs or just the row count.
-	 *
-	 * @throws InvalidArgumentException - Throw InvalidArgumentException if $select_or_count not count or select.
-	 * @return string SQL statement. The returned SQL is already properly escaped.
-	 */
-	protected function get_query_actions_sql( array $query, $select_or_count = 'select' ) {
-
-		if ( ! in_array( $select_or_count, array( 'select', 'count' ), true ) ) {
-			throw new InvalidArgumentException( __( 'Invalid schedule. Cannot save action.', 'action-scheduler' ) );
-		}
-
-		$query = wp_parse_args(
-			$query,
-			array(
-				'hook'             => '',
-				'args'             => null,
-				'date'             => null,
-				'date_compare'     => '<=',
-				'modified'         => null,
-				'modified_compare' => '<=',
-				'group'            => '',
-				'status'           => '',
-				'claimed'          => null,
-				'per_page'         => 5,
-				'offset'           => 0,
-				'orderby'          => 'date',
-				'order'            => 'ASC',
-				'search'           => '',
-			)
-		);
-
-		/**
-		 * Global wpdb object.
-		 *
-		 * @var wpdb $wpdb
-		 */
-		global $wpdb;
-		$sql        = ( 'count' === $select_or_count ) ? 'SELECT count(p.ID)' : 'SELECT p.ID ';
-		$sql       .= "FROM {$wpdb->posts} p";
-		$sql_params = array();
-		if ( empty( $query['group'] ) && 'group' === $query['orderby'] ) {
-			$sql .= " LEFT JOIN {$wpdb->term_relationships} tr ON tr.object_id=p.ID";
-			$sql .= " LEFT JOIN {$wpdb->term_taxonomy} tt ON tr.term_taxonomy_id=tt.term_taxonomy_id";
-			$sql .= " LEFT JOIN {$wpdb->terms} t ON tt.term_id=t.term_id";
-		} elseif ( ! empty( $query['group'] ) ) {
-			$sql         .= " INNER JOIN {$wpdb->term_relationships} tr ON tr.object_id=p.ID";
-			$sql         .= " INNER JOIN {$wpdb->term_taxonomy} tt ON tr.term_taxonomy_id=tt.term_taxonomy_id";
-			$sql         .= " INNER JOIN {$wpdb->terms} t ON tt.term_id=t.term_id";
-			$sql         .= ' AND t.slug=%s';
-			$sql_params[] = $query['group'];
-		}
-		$sql         .= ' WHERE post_type=%s';
-		$sql_params[] = self::POST_TYPE;
-		if ( $query['hook'] ) {
-			$sql         .= ' AND p.post_title=%s';
-			$sql_params[] = $query['hook'];
-		}
-		if ( $query['args'] !== null ) {
-			$sql         .= ' AND p.post_content=%s';
-			$sql_params[] = wp_json_encode( $query['args'] );
-		}
-
-		if ( $query['status'] ) {
-			$post_statuses = array_map( array( $this, 'get_post_status_by_action_status' ), (array) $query['status'] );
-			$placeholders  = array_fill( 0, count( $post_statuses ), '%s' );
-			$sql          .= ' AND p.post_status IN (' . join( ', ', $placeholders ) . ')';
-			$sql_params    = array_merge( $sql_params, array_values( $post_statuses ) );
-		}
-
-		if ( $query['date'] instanceof DateTime ) {
-			$date = clone $query['date'];
-			$date->setTimezone( new DateTimeZone( 'UTC' ) );
-			$date_string  = $date->format( 'Y-m-d H:i:s' );
-			$comparator   = $this->validate_sql_comparator( $query['date_compare'] );
-			$sql         .= " AND p.post_date_gmt $comparator %s";
-			$sql_params[] = $date_string;
-		}
-
-		if ( $query['modified'] instanceof DateTime ) {
-			$modified = clone $query['modified'];
-			$modified->setTimezone( new DateTimeZone( 'UTC' ) );
-			$date_string  = $modified->format( 'Y-m-d H:i:s' );
-			$comparator   = $this->validate_sql_comparator( $query['modified_compare'] );
-			$sql         .= " AND p.post_modified_gmt $comparator %s";
-			$sql_params[] = $date_string;
-		}
-
-		if ( true === $query['claimed'] ) {
-			$sql .= " AND p.post_password != ''";
-		} elseif ( false === $query['claimed'] ) {
-			$sql .= " AND p.post_password = ''";
-		} elseif ( $query['claimed'] !== null ) {
-			$sql         .= ' AND p.post_password = %s';
-			$sql_params[] = $query['claimed'];
-		}
-
-		if ( ! empty( $query['search'] ) ) {
-			$sql .= ' AND (p.post_title LIKE %s OR p.post_content LIKE %s OR p.post_password LIKE %s)';
-			for ( $i = 0; $i < 3; $i++ ) {
-				$sql_params[] = sprintf( '%%%s%%', $query['search'] );
-			}
-		}
-
-		if ( 'select' === $select_or_count ) {
-			switch ( $query['orderby'] ) {
-				case 'hook':
-					$orderby = 'p.post_title';
-					break;
-				case 'group':
-					$orderby = 't.name';
-					break;
-				case 'status':
-					$orderby = 'p.post_status';
-					break;
-				case 'modified':
-					$orderby = 'p.post_modified';
-					break;
-				case 'claim_id':
-					$orderby = 'p.post_password';
-					break;
-				case 'schedule':
-				case 'date':
-				default:
-					$orderby = 'p.post_date_gmt';
-					break;
-			}
-			if ( 'ASC' === strtoupper( $query['order'] ) ) {
-				$order = 'ASC';
-			} else {
-				$order = 'DESC';
-			}
-			$sql .= " ORDER BY $orderby $order";
-			if ( $query['per_page'] > 0 ) {
-				$sql         .= ' LIMIT %d, %d';
-				$sql_params[] = $query['offset'];
-				$sql_params[] = $query['per_page'];
-			}
-		}
-
-		return $wpdb->prepare( $sql, $sql_params ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
-	}
-
-	/**
-	 * Query for action count or list of action IDs.
-	 *
-	 * @since 3.3.0 $query['status'] accepts array of statuses instead of a single status.
-	 *
-	 * @see ActionScheduler_Store::query_actions for $query arg usage.
-	 *
-	 * @param array  $query      Query filtering options.
-	 * @param string $query_type Whether to select or count the results. Defaults to select.
-	 *
-	 * @return string|array|null The IDs of actions matching the query. Null on failure.
-	 */
-	public function query_actions( $query = array(), $query_type = 'select' ) {
-		/**
-		 * Global $wpdb object.
-		 *
-		 * @var wpdb $wpdb
-		 */
-		global $wpdb;
-
-		$sql = $this->get_query_actions_sql( $query, $query_type );
-
-		return ( 'count' === $query_type ) ? $wpdb->get_var( $sql ) : $wpdb->get_col( $sql ); // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery,WordPress.DB.DirectDatabaseQuery.NoCaching,WordPress.DB.PreparedSQL.NotPrepared
-	}
-
-	/**
-	 * Get a count of all actions in the store, grouped by status
-	 *
-	 * @return array
-	 */
-	public function action_counts() {
-
-		$action_counts_by_status = array();
-		$action_stati_and_labels = $this->get_status_labels();
-		$posts_count_by_status   = (array) wp_count_posts( self::POST_TYPE, 'readable' );
-
-		foreach ( $posts_count_by_status as $post_status_name => $count ) {
-
-			try {
-				$action_status_name = $this->get_action_status_by_post_status( $post_status_name );
-			} catch ( Exception $e ) {
-				// Ignore any post statuses that aren't for actions.
-				continue;
-			}
-			if ( array_key_exists( $action_status_name, $action_stati_and_labels ) ) {
-				$action_counts_by_status[ $action_status_name ] = $count;
-			}
-		}
-
-		return $action_counts_by_status;
-	}
-
-	/**
-	 * Cancel action.
-	 *
-	 * @param int $action_id Action ID.
-	 *
-	 * @throws InvalidArgumentException If $action_id is not identified.
-	 */
-	public function cancel_action( $action_id ) {
-		$post = get_post( $action_id );
-		if ( empty( $post ) || ( self::POST_TYPE !== $post->post_type ) ) {
-			/* translators: %s is the action ID */
-			throw new InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
-		}
-		do_action( 'action_scheduler_canceled_action', $action_id );
-		add_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10, 5 );
-		wp_trash_post( $action_id );
-		remove_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10 );
-	}
-
-	/**
-	 * Delete action.
-	 *
-	 * @param int $action_id Action ID.
-	 * @return void
-	 * @throws InvalidArgumentException If action is not identified.
-	 */
-	public function delete_action( $action_id ) {
-		$post = get_post( $action_id );
-		if ( empty( $post ) || ( self::POST_TYPE !== $post->post_type ) ) {
-			/* translators: %s is the action ID */
-			throw new InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
-		}
-		do_action( 'action_scheduler_deleted_action', $action_id );
-
-		wp_delete_post( $action_id, true );
-	}
-
-	/**
-	 * Get date for claim id.
-	 *
-	 * @param int $action_id Action ID.
-	 * @return ActionScheduler_DateTime The date the action is schedule to run, or the date that it ran.
-	 */
-	public function get_date( $action_id ) {
-		$next = $this->get_date_gmt( $action_id );
-		return ActionScheduler_TimezoneHelper::set_local_timezone( $next );
-	}
-
-	/**
-	 * Get Date GMT.
-	 *
-	 * @param int $action_id Action ID.
-	 *
-	 * @throws InvalidArgumentException If $action_id is not identified.
-	 * @return ActionScheduler_DateTime The date the action is schedule to run, or the date that it ran.
-	 */
-	public function get_date_gmt( $action_id ) {
-		$post = get_post( $action_id );
-		if ( empty( $post ) || ( self::POST_TYPE !== $post->post_type ) ) {
-			/* translators: %s is the action ID */
-			throw new InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
-		}
-		if ( 'publish' === $post->post_status ) {
-			return as_get_datetime_object( $post->post_modified_gmt );
-		} else {
-			return as_get_datetime_object( $post->post_date_gmt );
-		}
-	}
-
-	/**
-	 * Stake claim.
-	 *
-	 * @param int      $max_actions Maximum number of actions.
-	 * @param DateTime $before_date Jobs must be schedule before this date. Defaults to now.
-	 * @param array    $hooks       Claim only actions with a hook or hooks.
-	 * @param string   $group       Claim only actions in the given group.
-	 *
-	 * @return ActionScheduler_ActionClaim
-	 * @throws RuntimeException When there is an error staking a claim.
-	 * @throws InvalidArgumentException When the given group is not valid.
-	 */
-	public function stake_claim( $max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '' ) {
-		$this->claim_before_date = $before_date;
-		$claim_id                = $this->generate_claim_id();
-		$this->claim_actions( $claim_id, $max_actions, $before_date, $hooks, $group );
-		$action_ids              = $this->find_actions_by_claim_id( $claim_id );
-		$this->claim_before_date = null;
-
-		return new ActionScheduler_ActionClaim( $claim_id, $action_ids );
-	}
-
-	/**
-	 * Get claim count.
-	 *
-	 * @return int
-	 */
-	public function get_claim_count() {
-		global $wpdb;
-
-		// phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery,WordPress.DB.DirectDatabaseQuery.NoCaching
-		return $wpdb->get_var(
-			$wpdb->prepare(
-				"SELECT COUNT(DISTINCT post_password) FROM {$wpdb->posts} WHERE post_password != '' AND post_type = %s AND post_status IN ('in-progress','pending')",
-				array( self::POST_TYPE )
-			)
-		);
-	}
-
-	/**
-	 * Generate claim id.
-	 *
-	 * @return string
-	 */
-	protected function generate_claim_id() {
-		$claim_id = md5( microtime( true ) . wp_rand( 0, 1000 ) );
-		return substr( $claim_id, 0, 20 ); // to fit in db field with 20 char limit.
-	}
-
-	/**
-	 * Claim actions.
-	 *
-	 * @param string   $claim_id    Claim ID.
-	 * @param int      $limit       Limit.
-	 * @param DateTime $before_date Should use UTC timezone.
-	 * @param array    $hooks       Claim only actions with a hook or hooks.
-	 * @param string   $group       Claim only actions in the given group.
-	 *
-	 * @return int The number of actions that were claimed.
-	 * @throws RuntimeException  When there is a database error.
-	 */
-	protected function claim_actions( $claim_id, $limit, DateTime $before_date = null, $hooks = array(), $group = '' ) {
-		// Set up initial variables.
-		$date      = null === $before_date ? as_get_datetime_object() : clone $before_date;
-		$limit_ids = ! empty( $group );
-		$ids       = $limit_ids ? $this->get_actions_by_group( $group, $limit, $date ) : array();
-
-		// If limiting by IDs and no posts found, then return early since we have nothing to update.
-		if ( $limit_ids && 0 === count( $ids ) ) {
-			return 0;
-		}
-
-		/**
-		 * Global wpdb object.
-		 *
-		 * @var wpdb $wpdb
-		 */
-		global $wpdb;
-
-		/*
+    const POST_TYPE         = 'scheduled-action';
+    const GROUP_TAXONOMY    = 'action-group';
+    const SCHEDULE_META_KEY = '_action_manager_schedule';
+    const DEPENDENCIES_MET  = 'as-post-store-dependencies-met';
+
+    /**
+     * Used to share information about the before_date property of claims internally.
+     *
+     * This is used in preference to passing the same information as a method param
+     * for backwards-compatibility reasons.
+     *
+     * @var DateTime|null
+     */
+    private $claim_before_date = null;
+
+    /**
+     * Local Timezone.
+     *
+     * @var DateTimeZone
+     */
+    protected $local_timezone = null;
+
+    /**
+     * Save action.
+     *
+     * @param ActionScheduler_Action $action Scheduled Action.
+     * @param DateTime               $scheduled_date Scheduled Date.
+     *
+     * @throws RuntimeException Throws an exception if the action could not be saved.
+     * @return int
+     */
+    public function save_action( ActionScheduler_Action $action, DateTime $scheduled_date = null ) {
+        try {
+            $this->validate_action( $action );
+            $post_array = $this->create_post_array( $action, $scheduled_date );
+            $post_id    = $this->save_post_array( $post_array );
+            $this->save_post_schedule( $post_id, $action->get_schedule() );
+            $this->save_action_group( $post_id, $action->get_group() );
+            do_action( 'action_scheduler_stored_action', $post_id );
+            return $post_id;
+        } catch ( Exception $e ) {
+            /* translators: %s: action error message */
+            throw new RuntimeException( sprintf( __( 'Error saving action: %s', 'action-scheduler' ), $e->getMessage() ), 0 );
+        }
+    }
+
+    /**
+     * Create post array.
+     *
+     * @param ActionScheduler_Action $action Scheduled Action.
+     * @param DateTime               $scheduled_date Scheduled Date.
+     *
+     * @return array Returns an array of post data.
+     */
+    protected function create_post_array( ActionScheduler_Action $action, DateTime $scheduled_date = null ) {
+        $post = array(
+            'post_type'     => self::POST_TYPE,
+            'post_title'    => $action->get_hook(),
+            'post_content'  => wp_json_encode( $action->get_args() ),
+            'post_status'   => ( $action->is_finished() ? 'publish' : 'pending' ),
+            'post_date_gmt' => $this->get_scheduled_date_string( $action, $scheduled_date ),
+            'post_date'     => $this->get_scheduled_date_string_local( $action, $scheduled_date ),
+        );
+        return $post;
+    }
+
+    /**
+     * Save post array.
+     *
+     * @param array $post_array Post array.
+     * @return int Returns the post ID.
+     * @throws RuntimeException Throws an exception if the action could not be saved.
+     */
+    protected function save_post_array( $post_array ) {
+        add_filter( 'wp_insert_post_data', array( $this, 'filter_insert_post_data' ), 10, 1 );
+        add_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10, 5 );
+
+        $has_kses = false !== has_filter( 'content_save_pre', 'wp_filter_post_kses' );
+
+        if ( $has_kses ) {
+            // Prevent KSES from corrupting JSON in post_content.
+            kses_remove_filters();
+        }
+
+        $post_id = wp_insert_post( $post_array );
+
+        if ( $has_kses ) {
+            kses_init_filters();
+        }
+
+        remove_filter( 'wp_insert_post_data', array( $this, 'filter_insert_post_data' ), 10 );
+        remove_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10 );
+
+        if ( is_wp_error( $post_id ) || empty( $post_id ) ) {
+            throw new RuntimeException( __( 'Unable to save action.', 'action-scheduler' ) );
+        }
+        return $post_id;
+    }
+
+    /**
+     * Filter insert post data.
+     *
+     * @param array $postdata Post data to filter.
+     *
+     * @return array
+     */
+    public function filter_insert_post_data( $postdata ) {
+        if ( self::POST_TYPE === $postdata['post_type'] ) {
+            $postdata['post_author'] = 0;
+            if ( 'future' === $postdata['post_status'] ) {
+                $postdata['post_status'] = 'publish';
+            }
+        }
+        return $postdata;
+    }
+
+    /**
+     * Create a (probably unique) post name for scheduled actions in a more performant manner than wp_unique_post_slug().
+     *
+     * When an action's post status is transitioned to something other than 'draft', 'pending' or 'auto-draft, like 'publish'
+     * or 'failed' or 'trash', WordPress will find a unique slug (stored in post_name column) using the wp_unique_post_slug()
+     * function. This is done to ensure URL uniqueness. The approach taken by wp_unique_post_slug() is to iterate over existing
+     * post_name values that match, and append a number 1 greater than the largest. This makes sense when manually creating a
+     * post from the Edit Post screen. It becomes a bottleneck when automatically processing thousands of actions, with a
+     * database containing thousands of related post_name values.
+     *
+     * WordPress 5.1 introduces the 'pre_wp_unique_post_slug' filter for plugins to address this issue.
+     *
+     * We can short-circuit WordPress's wp_unique_post_slug() approach using the 'pre_wp_unique_post_slug' filter. This
+     * method is available to be used as a callback on that filter. It provides a more scalable approach to generating a
+     * post_name/slug that is probably unique. Because Action Scheduler never actually uses the post_name field, or an
+     * action's slug, being probably unique is good enough.
+     *
+     * For more backstory on this issue, see:
+     * - https://github.com/woocommerce/action-scheduler/issues/44 and
+     * - https://core.trac.wordpress.org/ticket/21112
+     *
+     * @param string $override_slug Short-circuit return value.
+     * @param string $slug          The desired slug (post_name).
+     * @param int    $post_ID       Post ID.
+     * @param string $post_status   The post status.
+     * @param string $post_type     Post type.
+     * @return string
+     */
+    public function set_unique_post_slug( $override_slug, $slug, $post_ID, $post_status, $post_type ) {
+        if ( self::POST_TYPE === $post_type ) {
+            $override_slug = uniqid( self::POST_TYPE . '-', true ) . '-' . wp_generate_password( 32, false );
+        }
+        return $override_slug;
+    }
+
+    /**
+     * Save post schedule.
+     *
+     * @param int    $post_id  Post ID of the scheduled action.
+     * @param string $schedule Schedule to save.
+     *
+     * @return void
+     */
+    protected function save_post_schedule( $post_id, $schedule ) {
+        update_post_meta( $post_id, self::SCHEDULE_META_KEY, $schedule );
+    }
+
+    /**
+     * Save action group.
+     *
+     * @param int    $post_id Post ID.
+     * @param string $group   Group to save.
+     * @return void
+     */
+    protected function save_action_group( $post_id, $group ) {
+        if ( empty( $group ) ) {
+            wp_set_object_terms( $post_id, array(), self::GROUP_TAXONOMY, false );
+        } else {
+            wp_set_object_terms( $post_id, array( $group ), self::GROUP_TAXONOMY, false );
+        }
+    }
+
+    /**
+     * Fetch actions.
+     *
+     * @param int $action_id Action ID.
+     * @return object
+     */
+    public function fetch_action( $action_id ) {
+        $post = $this->get_post( $action_id );
+        if ( empty( $post ) || self::POST_TYPE !== $post->post_type ) {
+            return $this->get_null_action();
+        }
+
+        try {
+            $action = $this->make_action_from_post( $post );
+        } catch ( ActionScheduler_InvalidActionException $exception ) {
+            do_action( 'action_scheduler_failed_fetch_action', $post->ID, $exception );
+            return $this->get_null_action();
+        }
+
+        return $action;
+    }
+
+    /**
+     * Get post.
+     *
+     * @param string $action_id - Action ID.
+     * @return WP_Post|null
+     */
+    protected function get_post( $action_id ) {
+        if ( empty( $action_id ) ) {
+            return null;
+        }
+        return get_post( $action_id );
+    }
+
+    /**
+     * Get NULL action.
+     *
+     * @return ActionScheduler_NullAction
+     */
+    protected function get_null_action() {
+        return new ActionScheduler_NullAction();
+    }
+
+    /**
+     * Make action from post.
+     *
+     * @param WP_Post $post Post object.
+     * @return WP_Post
+     */
+    protected function make_action_from_post( $post ) {
+        $hook = $post->post_title;
+
+        $args = json_decode( $post->post_content, true );
+        $this->validate_args( $args, $post->ID );
+
+        $schedule = get_post_meta( $post->ID, self::SCHEDULE_META_KEY, true );
+        $this->validate_schedule( $schedule, $post->ID );
+
+        $group = wp_get_object_terms( $post->ID, self::GROUP_TAXONOMY, array( 'fields' => 'names' ) );
+        $group = empty( $group ) ? '' : reset( $group );
+
+        return ActionScheduler::factory()->get_stored_action( $this->get_action_status_by_post_status( $post->post_status ), $hook, $args, $schedule, $group );
+    }
+
+    /**
+     * Get action status by post status.
+     *
+     * @param string $post_status Post status.
+     *
+     * @throws InvalidArgumentException Throw InvalidArgumentException if $post_status not in known status fields returned by $this->get_status_labels().
+     * @return string
+     */
+    protected function get_action_status_by_post_status( $post_status ) {
+
+        switch ( $post_status ) {
+            case 'publish':
+                $action_status = self::STATUS_COMPLETE;
+                break;
+            case 'trash':
+                $action_status = self::STATUS_CANCELED;
+                break;
+            default:
+                if ( ! array_key_exists( $post_status, $this->get_status_labels() ) ) {
+                    throw new InvalidArgumentException( sprintf( 'Invalid post status: "%s". No matching action status available.', $post_status ) );
+                }
+                $action_status = $post_status;
+                break;
+        }
+
+        return $action_status;
+    }
+
+    /**
+     * Get post status by action status.
+     *
+     * @param string $action_status Action status.
+     *
+     * @throws InvalidArgumentException Throws InvalidArgumentException if $post_status not in known status fields returned by $this->get_status_labels().
+     * @return string
+     */
+    protected function get_post_status_by_action_status( $action_status ) {
+
+        switch ( $action_status ) {
+            case self::STATUS_COMPLETE:
+                $post_status = 'publish';
+                break;
+            case self::STATUS_CANCELED:
+                $post_status = 'trash';
+                break;
+            default:
+                if ( ! array_key_exists( $action_status, $this->get_status_labels() ) ) {
+                    throw new InvalidArgumentException( sprintf( 'Invalid action status: "%s".', $action_status ) );
+                }
+                $post_status = $action_status;
+                break;
+        }
+
+        return $post_status;
+    }
+
+    /**
+     * Returns the SQL statement to query (or count) actions.
+     *
+     * @param array  $query            - Filtering options.
+     * @param string $select_or_count  - Whether the SQL should select and return the IDs or just the row count.
+     *
+     * @throws InvalidArgumentException - Throw InvalidArgumentException if $select_or_count not count or select.
+     * @return string SQL statement. The returned SQL is already properly escaped.
+     */
+    protected function get_query_actions_sql( array $query, $select_or_count = 'select' ) {
+
+        if ( ! in_array( $select_or_count, array( 'select', 'count' ), true ) ) {
+            throw new InvalidArgumentException( __( 'Invalid schedule. Cannot save action.', 'action-scheduler' ) );
+        }
+
+        $query = wp_parse_args(
+            $query,
+            array(
+                'hook'             => '',
+                'args'             => null,
+                'date'             => null,
+                'date_compare'     => '<=',
+                'modified'         => null,
+                'modified_compare' => '<=',
+                'group'            => '',
+                'status'           => '',
+                'claimed'          => null,
+                'per_page'         => 5,
+                'offset'           => 0,
+                'orderby'          => 'date',
+                'order'            => 'ASC',
+                'search'           => '',
+            )
+        );
+
+        /**
+         * Global wpdb object.
+         *
+         * @var wpdb $wpdb
+         */
+        global $wpdb;
+        $sql        = ( 'count' === $select_or_count ) ? 'SELECT count(p.ID)' : 'SELECT p.ID ';
+        $sql       .= "FROM {$wpdb->posts} p";
+        $sql_params = array();
+        if ( empty( $query['group'] ) && 'group' === $query['orderby'] ) {
+            $sql .= " LEFT JOIN {$wpdb->term_relationships} tr ON tr.object_id=p.ID";
+            $sql .= " LEFT JOIN {$wpdb->term_taxonomy} tt ON tr.term_taxonomy_id=tt.term_taxonomy_id";
+            $sql .= " LEFT JOIN {$wpdb->terms} t ON tt.term_id=t.term_id";
+        } elseif ( ! empty( $query['group'] ) ) {
+            $sql         .= " INNER JOIN {$wpdb->term_relationships} tr ON tr.object_id=p.ID";
+            $sql         .= " INNER JOIN {$wpdb->term_taxonomy} tt ON tr.term_taxonomy_id=tt.term_taxonomy_id";
+            $sql         .= " INNER JOIN {$wpdb->terms} t ON tt.term_id=t.term_id";
+            $sql         .= ' AND t.slug=%s';
+            $sql_params[] = $query['group'];
+        }
+        $sql         .= ' WHERE post_type=%s';
+        $sql_params[] = self::POST_TYPE;
+        if ( $query['hook'] ) {
+            $sql         .= ' AND p.post_title=%s';
+            $sql_params[] = $query['hook'];
+        }
+        if ( $query['args'] !== null ) {
+            $sql         .= ' AND p.post_content=%s';
+            $sql_params[] = wp_json_encode( $query['args'] );
+        }
+
+        if ( $query['status'] ) {
+            $post_statuses = array_map( array( $this, 'get_post_status_by_action_status' ), (array) $query['status'] );
+            $placeholders  = array_fill( 0, count( $post_statuses ), '%s' );
+            $sql          .= ' AND p.post_status IN (' . join( ', ', $placeholders ) . ')';
+            $sql_params    = array_merge( $sql_params, array_values( $post_statuses ) );
+        }
+
+        if ( $query['date'] instanceof DateTime ) {
+            $date = clone $query['date'];
+            $date->setTimezone( new DateTimeZone( 'UTC' ) );
+            $date_string  = $date->format( 'Y-m-d H:i:s' );
+            $comparator   = $this->validate_sql_comparator( $query['date_compare'] );
+            $sql         .= " AND p.post_date_gmt $comparator %s";
+            $sql_params[] = $date_string;
+        }
+
+        if ( $query['modified'] instanceof DateTime ) {
+            $modified = clone $query['modified'];
+            $modified->setTimezone( new DateTimeZone( 'UTC' ) );
+            $date_string  = $modified->format( 'Y-m-d H:i:s' );
+            $comparator   = $this->validate_sql_comparator( $query['modified_compare'] );
+            $sql         .= " AND p.post_modified_gmt $comparator %s";
+            $sql_params[] = $date_string;
+        }
+
+        if ( true === $query['claimed'] ) {
+            $sql .= " AND p.post_password != ''";
+        } elseif ( false === $query['claimed'] ) {
+            $sql .= " AND p.post_password = ''";
+        } elseif ( $query['claimed'] !== null ) {
+            $sql         .= ' AND p.post_password = %s';
+            $sql_params[] = $query['claimed'];
+        }
+
+        if ( ! empty( $query['search'] ) ) {
+            $sql .= ' AND (p.post_title LIKE %s OR p.post_content LIKE %s OR p.post_password LIKE %s)';
+            for ( $i = 0; $i < 3; $i++ ) {
+                $sql_params[] = sprintf( '%%%s%%', $query['search'] );
+            }
+        }
+
+        if ( 'select' === $select_or_count ) {
+            switch ( $query['orderby'] ) {
+                case 'hook':
+                    $orderby = 'p.post_title';
+                    break;
+                case 'group':
+                    $orderby = 't.name';
+                    break;
+                case 'status':
+                    $orderby = 'p.post_status';
+                    break;
+                case 'modified':
+                    $orderby = 'p.post_modified';
+                    break;
+                case 'claim_id':
+                    $orderby = 'p.post_password';
+                    break;
+                case 'schedule':
+                case 'date':
+                default:
+                    $orderby = 'p.post_date_gmt';
+                    break;
+            }
+            if ( 'ASC' === strtoupper( $query['order'] ) ) {
+                $order = 'ASC';
+            } else {
+                $order = 'DESC';
+            }
+            $sql .= " ORDER BY $orderby $order";
+            if ( $query['per_page'] > 0 ) {
+                $sql         .= ' LIMIT %d, %d';
+                $sql_params[] = $query['offset'];
+                $sql_params[] = $query['per_page'];
+            }
+        }
+
+        return $wpdb->prepare( $sql, $sql_params ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+    }
+
+    /**
+     * Query for action count or list of action IDs.
+     *
+     * @since 3.3.0 $query['status'] accepts array of statuses instead of a single status.
+     *
+     * @see ActionScheduler_Store::query_actions for $query arg usage.
+     *
+     * @param array  $query      Query filtering options.
+     * @param string $query_type Whether to select or count the results. Defaults to select.
+     *
+     * @return string|array|null The IDs of actions matching the query. Null on failure.
+     */
+    public function query_actions( $query = array(), $query_type = 'select' ) {
+        /**
+         * Global $wpdb object.
+         *
+         * @var wpdb $wpdb
+         */
+        global $wpdb;
+
+        $sql = $this->get_query_actions_sql( $query, $query_type );
+
+        return ( 'count' === $query_type ) ? $wpdb->get_var( $sql ) : $wpdb->get_col( $sql ); // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery,WordPress.DB.DirectDatabaseQuery.NoCaching,WordPress.DB.PreparedSQL.NotPrepared
+    }
+
+    /**
+     * Get a count of all actions in the store, grouped by status
+     *
+     * @return array
+     */
+    public function action_counts() {
+
+        $action_counts_by_status = array();
+        $action_stati_and_labels = $this->get_status_labels();
+        $posts_count_by_status   = (array) wp_count_posts( self::POST_TYPE, 'readable' );
+
+        foreach ( $posts_count_by_status as $post_status_name => $count ) {
+
+            try {
+                $action_status_name = $this->get_action_status_by_post_status( $post_status_name );
+            } catch ( Exception $e ) {
+                // Ignore any post statuses that aren't for actions.
+                continue;
+            }
+            if ( array_key_exists( $action_status_name, $action_stati_and_labels ) ) {
+                $action_counts_by_status[ $action_status_name ] = $count;
+            }
+        }
+
+        return $action_counts_by_status;
+    }
+
+    /**
+     * Cancel action.
+     *
+     * @param int $action_id Action ID.
+     *
+     * @throws InvalidArgumentException If $action_id is not identified.
+     */
+    public function cancel_action( $action_id ) {
+        $post = get_post( $action_id );
+        if ( empty( $post ) || ( self::POST_TYPE !== $post->post_type ) ) {
+            /* translators: %s is the action ID */
+            throw new InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+        }
+        do_action( 'action_scheduler_canceled_action', $action_id );
+        add_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10, 5 );
+        wp_trash_post( $action_id );
+        remove_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10 );
+    }
+
+    /**
+     * Delete action.
+     *
+     * @param int $action_id Action ID.
+     * @return void
+     * @throws InvalidArgumentException If action is not identified.
+     */
+    public function delete_action( $action_id ) {
+        $post = get_post( $action_id );
+        if ( empty( $post ) || ( self::POST_TYPE !== $post->post_type ) ) {
+            /* translators: %s is the action ID */
+            throw new InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+        }
+        do_action( 'action_scheduler_deleted_action', $action_id );
+
+        wp_delete_post( $action_id, true );
+    }
+
+    /**
+     * Get date for claim id.
+     *
+     * @param int $action_id Action ID.
+     * @return ActionScheduler_DateTime The date the action is schedule to run, or the date that it ran.
+     */
+    public function get_date( $action_id ) {
+        $next = $this->get_date_gmt( $action_id );
+        return ActionScheduler_TimezoneHelper::set_local_timezone( $next );
+    }
+
+    /**
+     * Get Date GMT.
+     *
+     * @param int $action_id Action ID.
+     *
+     * @throws InvalidArgumentException If $action_id is not identified.
+     * @return ActionScheduler_DateTime The date the action is schedule to run, or the date that it ran.
+     */
+    public function get_date_gmt( $action_id ) {
+        $post = get_post( $action_id );
+        if ( empty( $post ) || ( self::POST_TYPE !== $post->post_type ) ) {
+            /* translators: %s is the action ID */
+            throw new InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+        }
+        if ( 'publish' === $post->post_status ) {
+            return as_get_datetime_object( $post->post_modified_gmt );
+        } else {
+            return as_get_datetime_object( $post->post_date_gmt );
+        }
+    }
+
+    /**
+     * Stake claim.
+     *
+     * @param int      $max_actions Maximum number of actions.
+     * @param DateTime $before_date Jobs must be schedule before this date. Defaults to now.
+     * @param array    $hooks       Claim only actions with a hook or hooks.
+     * @param string   $group       Claim only actions in the given group.
+     *
+     * @return ActionScheduler_ActionClaim
+     * @throws RuntimeException When there is an error staking a claim.
+     * @throws InvalidArgumentException When the given group is not valid.
+     */
+    public function stake_claim( $max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '' ) {
+        $this->claim_before_date = $before_date;
+        $claim_id                = $this->generate_claim_id();
+        $this->claim_actions( $claim_id, $max_actions, $before_date, $hooks, $group );
+        $action_ids              = $this->find_actions_by_claim_id( $claim_id );
+        $this->claim_before_date = null;
+
+        return new ActionScheduler_ActionClaim( $claim_id, $action_ids );
+    }
+
+    /**
+     * Get claim count.
+     *
+     * @return int
+     */
+    public function get_claim_count() {
+        global $wpdb;
+
+        // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery,WordPress.DB.DirectDatabaseQuery.NoCaching
+        return $wpdb->get_var(
+            $wpdb->prepare(
+                "SELECT COUNT(DISTINCT post_password) FROM {$wpdb->posts} WHERE post_password != '' AND post_type = %s AND post_status IN ('in-progress','pending')",
+                array( self::POST_TYPE )
+            )
+        );
+    }
+
+    /**
+     * Generate claim id.
+     *
+     * @return string
+     */
+    protected function generate_claim_id() {
+        $claim_id = md5( microtime( true ) . wp_rand( 0, 1000 ) );
+        return substr( $claim_id, 0, 20 ); // to fit in db field with 20 char limit.
+    }
+
+    /**
+     * Claim actions.
+     *
+     * @param string   $claim_id    Claim ID.
+     * @param int      $limit       Limit.
+     * @param DateTime $before_date Should use UTC timezone.
+     * @param array    $hooks       Claim only actions with a hook or hooks.
+     * @param string   $group       Claim only actions in the given group.
+     *
+     * @return int The number of actions that were claimed.
+     * @throws RuntimeException  When there is a database error.
+     */
+    protected function claim_actions( $claim_id, $limit, DateTime $before_date = null, $hooks = array(), $group = '' ) {
+        // Set up initial variables.
+        $date      = null === $before_date ? as_get_datetime_object() : clone $before_date;
+        $limit_ids = ! empty( $group );
+        $ids       = $limit_ids ? $this->get_actions_by_group( $group, $limit, $date ) : array();
+
+        // If limiting by IDs and no posts found, then return early since we have nothing to update.
+        if ( $limit_ids && 0 === count( $ids ) ) {
+            return 0;
+        }
+
+        /**
+         * Global wpdb object.
+         *
+         * @var wpdb $wpdb
+         */
+        global $wpdb;
+
+        /*
 		 * Build up custom query to update the affected posts. Parameters are built as a separate array
 		 * to make it easier to identify where they are in the query.
 		 *
 		 * We can't use $wpdb->update() here because of the "ID IN ..." clause.
 		 */
-		$update = "UPDATE {$wpdb->posts} SET post_password = %s, post_modified_gmt = %s, post_modified = %s";
-		$params = array(
-			$claim_id,
-			current_time( 'mysql', true ),
-			current_time( 'mysql' ),
-		);
-
-		// Build initial WHERE clause.
-		$where    = "WHERE post_type = %s AND post_status = %s AND post_password = ''";
-		$params[] = self::POST_TYPE;
-		$params[] = ActionScheduler_Store::STATUS_PENDING;
-
-		if ( ! empty( $hooks ) ) {
-			$placeholders = array_fill( 0, count( $hooks ), '%s' );
-			$where       .= ' AND post_title IN (' . join( ', ', $placeholders ) . ')';
-			$params       = array_merge( $params, array_values( $hooks ) );
-		}
-
-		/*
+        $update = "UPDATE {$wpdb->posts} SET post_password = %s, post_modified_gmt = %s, post_modified = %s";
+        $params = array(
+            $claim_id,
+            current_time( 'mysql', true ),
+            current_time( 'mysql' ),
+        );
+
+        // Build initial WHERE clause.
+        $where    = "WHERE post_type = %s AND post_status = %s AND post_password = ''";
+        $params[] = self::POST_TYPE;
+        $params[] = ActionScheduler_Store::STATUS_PENDING;
+
+        if ( ! empty( $hooks ) ) {
+            $placeholders = array_fill( 0, count( $hooks ), '%s' );
+            $where       .= ' AND post_title IN (' . join( ', ', $placeholders ) . ')';
+            $params       = array_merge( $params, array_values( $hooks ) );
+        }
+
+        /*
 		 * Add the IDs to the WHERE clause. IDs not escaped because they came directly from a prior DB query.
 		 *
 		 * If we're not limiting by IDs, then include the post_date_gmt clause.
 		 */
-		if ( $limit_ids ) {
-			$where .= ' AND ID IN (' . join( ',', $ids ) . ')';
-		} else {
-			$where   .= ' AND post_date_gmt <= %s';
-			$params[] = $date->format( 'Y-m-d H:i:s' );
-		}
-
-		// Add the ORDER BY clause and,ms limit.
-		$order    = 'ORDER BY menu_order ASC, post_date_gmt ASC, ID ASC LIMIT %d';
-		$params[] = $limit;
-
-		// Run the query and gather results.
-		$rows_affected = $wpdb->query( $wpdb->prepare( "{$update} {$where} {$order}", $params ) ); // phpcs:ignore // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared, WordPress.DB.PreparedSQLPlaceholders.UnfinishedPrepare
-
-		if ( false === $rows_affected ) {
-			throw new RuntimeException( __( 'Unable to claim actions. Database error.', 'action-scheduler' ) );
-		}
-
-		return (int) $rows_affected;
-	}
-
-	/**
-	 * Get IDs of actions within a certain group and up to a certain date/time.
-	 *
-	 * @param string   $group The group to use in finding actions.
-	 * @param int      $limit The number of actions to retrieve.
-	 * @param DateTime $date  DateTime object representing cutoff time for actions. Actions retrieved will be
-	 *                        up to and including this DateTime.
-	 *
-	 * @return array IDs of actions in the appropriate group and before the appropriate time.
-	 * @throws InvalidArgumentException When the group does not exist.
-	 */
-	protected function get_actions_by_group( $group, $limit, DateTime $date ) {
-		// Ensure the group exists before continuing.
-		if ( ! term_exists( $group, self::GROUP_TAXONOMY ) ) {
-			/* translators: %s is the group name */
-			throw new InvalidArgumentException( sprintf( __( 'The group "%s" does not exist.', 'action-scheduler' ), $group ) );
-		}
-
-		// Set up a query for post IDs to use later.
-		$query      = new WP_Query();
-		$query_args = array(
-			'fields'           => 'ids',
-			'post_type'        => self::POST_TYPE,
-			'post_status'      => ActionScheduler_Store::STATUS_PENDING,
-			'has_password'     => false,
-			'posts_per_page'   => $limit * 3,
-			'suppress_filters' => true,
-			'no_found_rows'    => true,
-			'orderby'          => array(
-				'menu_order' => 'ASC',
-				'date'       => 'ASC',
-				'ID'         => 'ASC',
-			),
-			'date_query'       => array(
-				'column'    => 'post_date_gmt',
-				'before'    => $date->format( 'Y-m-d H:i' ),
-				'inclusive' => true,
-			),
-			'tax_query'        => array( // phpcs:ignore WordPress.DB.SlowDBQuery
-				array(
-					'taxonomy'         => self::GROUP_TAXONOMY,
-					'field'            => 'slug',
-					'terms'            => $group,
-					'include_children' => false,
-				),
-			),
-		);
-
-		return $query->query( $query_args );
-	}
-
-	/**
-	 * Find actions by claim ID.
-	 *
-	 * @param string $claim_id Claim ID.
-	 * @return array
-	 */
-	public function find_actions_by_claim_id( $claim_id ) {
-		/**
-		 * Global wpdb object.
-		 *
-		 * @var wpdb $wpdb
-		 */
-		global $wpdb;
-
-		$action_ids  = array();
-		$before_date = isset( $this->claim_before_date ) ? $this->claim_before_date : as_get_datetime_object();
-		$cut_off     = $before_date->format( 'Y-m-d H:i:s' );
-
-		// phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
-		$results = $wpdb->get_results(
-			$wpdb->prepare(
-				"SELECT ID, post_date_gmt FROM {$wpdb->posts} WHERE post_type = %s AND post_password = %s",
-				array(
-					self::POST_TYPE,
-					$claim_id,
-				)
-			)
-		);
-
-		// Verify that the scheduled date for each action is within the expected bounds (in some unusual
-		// cases, we cannot depend on MySQL to honor all of the WHERE conditions we specify).
-		foreach ( $results as $claimed_action ) {
-			if ( $claimed_action->post_date_gmt <= $cut_off ) {
-				$action_ids[] = absint( $claimed_action->ID );
-			}
-		}
-
-		return $action_ids;
-	}
-
-	/**
-	 * Release claim.
-	 *
-	 * @param ActionScheduler_ActionClaim $claim Claim object to release.
-	 * @return void
-	 * @throws RuntimeException When the claim is not unlocked.
-	 */
-	public function release_claim( ActionScheduler_ActionClaim $claim ) {
-		$action_ids = $this->find_actions_by_claim_id( $claim->get_id() );
-		if ( empty( $action_ids ) ) {
-			return; // nothing to do.
-		}
-		$action_id_string = implode( ',', array_map( 'intval', $action_ids ) );
-		/**
-		 * Global wpdb object.
-		 *
-		 * @var wpdb $wpdb
-		 */
-		global $wpdb;
-
-		// phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
-		$result = $wpdb->query(
-			$wpdb->prepare(
-				"UPDATE {$wpdb->posts} SET post_password = '' WHERE ID IN ($action_id_string) AND post_password = %s", //phpcs:ignore
-				array(
-					$claim->get_id(),
-				)
-			)
-		);
-		if ( false === $result ) {
-			/* translators: %s: claim ID */
-			throw new RuntimeException( sprintf( __( 'Unable to unlock claim %s. Database error.', 'action-scheduler' ), $claim->get_id() ) );
-		}
-	}
-
-	/**
-	 * Unclaim action.
-	 *
-	 * @param string $action_id Action ID.
-	 * @throws RuntimeException When unable to unlock claim on action ID.
-	 */
-	public function unclaim_action( $action_id ) {
-		/**
-		 * Global wpdb object.
-		 *
-		 * @var wpdb $wpdb
-		 */
-		global $wpdb;
-
-		//phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
-		$result = $wpdb->query(
-			$wpdb->prepare(
-				"UPDATE {$wpdb->posts} SET post_password = '' WHERE ID = %d AND post_type = %s",
-				$action_id,
-				self::POST_TYPE
-			)
-		);
-		if ( false === $result ) {
-			/* translators: %s: action ID */
-			throw new RuntimeException( sprintf( __( 'Unable to unlock claim on action %s. Database error.', 'action-scheduler' ), $action_id ) );
-		}
-	}
-
-	/**
-	 * Mark failure on action.
-	 *
-	 * @param int $action_id Action ID.
-	 *
-	 * @return void
-	 * @throws RuntimeException When unable to mark failure on action ID.
-	 */
-	public function mark_failure( $action_id ) {
-		/**
-		 * Global wpdb object.
-		 *
-		 * @var wpdb $wpdb
-		 */
-		global $wpdb;
-
-		// phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
-		$result = $wpdb->query(
-			$wpdb->prepare( "UPDATE {$wpdb->posts} SET post_status = %s WHERE ID = %d AND post_type = %s", self::STATUS_FAILED, $action_id, self::POST_TYPE )
-		);
-		if ( false === $result ) {
-			/* translators: %s: action ID */
-			throw new RuntimeException( sprintf( __( 'Unable to mark failure on action %s. Database error.', 'action-scheduler' ), $action_id ) );
-		}
-	}
-
-	/**
-	 * Return an action's claim ID, as stored in the post password column
-	 *
-	 * @param int $action_id Action ID.
-	 * @return mixed
-	 */
-	public function get_claim_id( $action_id ) {
-		return $this->get_post_column( $action_id, 'post_password' );
-	}
-
-	/**
-	 * Return an action's status, as stored in the post status column
-	 *
-	 * @param int $action_id Action ID.
-	 *
-	 * @return mixed
-	 * @throws InvalidArgumentException When the action ID is invalid.
-	 */
-	public function get_status( $action_id ) {
-		$status = $this->get_post_column( $action_id, 'post_status' );
-
-		if ( null === $status ) {
-			throw new InvalidArgumentException( __( 'Invalid action ID. No status found.', 'action-scheduler' ) );
-		}
-
-		return $this->get_action_status_by_post_status( $status );
-	}
-
-	/**
-	 * Get post column
-	 *
-	 * @param string $action_id Action ID.
-	 * @param string $column_name Column Name.
-	 *
-	 * @return string|null
-	 */
-	private function get_post_column( $action_id, $column_name ) {
-		/**
-		 * Global wpdb object.
-		 *
-		 * @var wpdb $wpdb
-		 */
-		global $wpdb;
-
-		// phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
-		return $wpdb->get_var(
-			$wpdb->prepare(
-				"SELECT {$column_name} FROM {$wpdb->posts} WHERE ID=%d AND post_type=%s", // phpcs:ignore
-				$action_id,
-				self::POST_TYPE
-			)
-		);
-	}
-
-	/**
-	 * Log Execution.
-	 *
-	 * @param string $action_id Action ID.
-	 */
-	public function log_execution( $action_id ) {
-		/**
-		 * Global wpdb object.
-		 *
-		 * @var wpdb $wpdb
-		 */
-		global $wpdb;
-
-		// phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
-		$wpdb->query(
-			$wpdb->prepare(
-				"UPDATE {$wpdb->posts} SET menu_order = menu_order+1, post_status=%s, post_modified_gmt = %s, post_modified = %s WHERE ID = %d AND post_type = %s",
-				self::STATUS_RUNNING,
-				current_time( 'mysql', true ),
-				current_time( 'mysql' ),
-				$action_id,
-				self::POST_TYPE
-			)
-		);
-	}
-
-	/**
-	 * Record that an action was completed.
-	 *
-	 * @param string $action_id ID of the completed action.
-	 *
-	 * @throws InvalidArgumentException When the action ID is invalid.
-	 * @throws RuntimeException         When there was an error executing the action.
-	 */
-	public function mark_complete( $action_id ) {
-		$post = get_post( $action_id );
-		if ( empty( $post ) || ( self::POST_TYPE !== $post->post_type ) ) {
-			/* translators: %s is the action ID */
-			throw new InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
-		}
-		add_filter( 'wp_insert_post_data', array( $this, 'filter_insert_post_data' ), 10, 1 );
-		add_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10, 5 );
-		$result = wp_update_post(
-			array(
-				'ID'          => $action_id,
-				'post_status' => 'publish',
-			),
-			true
-		);
-		remove_filter( 'wp_insert_post_data', array( $this, 'filter_insert_post_data' ), 10 );
-		remove_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10 );
-		if ( is_wp_error( $result ) ) {
-			throw new RuntimeException( $result->get_error_message() );
-		}
-
-		/**
-		 * Fires after a scheduled action has been completed.
-		 *
-		 * @since 3.4.2
-		 *
-		 * @param int $action_id Action ID.
-		 */
-		do_action( 'action_scheduler_completed_action', $action_id );
-	}
-
-	/**
-	 * Mark action as migrated when there is an error deleting the action.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function mark_migrated( $action_id ) {
-		wp_update_post(
-			array(
-				'ID'          => $action_id,
-				'post_status' => 'migrated',
-			)
-		);
-	}
-
-	/**
-	 * Determine whether the post store can be migrated.
-	 *
-	 * @param [type] $setting - Setting value.
-	 * @return bool
-	 */
-	public function migration_dependencies_met( $setting ) {
-		global $wpdb;
-
-		$dependencies_met = get_transient( self::DEPENDENCIES_MET );
-		if ( empty( $dependencies_met ) ) {
-			$maximum_args_length = apply_filters( 'action_scheduler_maximum_args_length', 191 );
-			$found_action        = $wpdb->get_var( // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
-				$wpdb->prepare(
-					"SELECT ID FROM {$wpdb->posts} WHERE post_type = %s AND CHAR_LENGTH(post_content) > %d LIMIT 1",
-					$maximum_args_length,
-					self::POST_TYPE
-				)
-			);
-			$dependencies_met    = $found_action ? 'no' : 'yes';
-			set_transient( self::DEPENDENCIES_MET, $dependencies_met, DAY_IN_SECONDS );
-		}
-
-		return 'yes' === $dependencies_met ? $setting : false;
-	}
-
-	/**
-	 * InnoDB indexes have a maximum size of 767 bytes by default, which is only 191 characters with utf8mb4.
-	 *
-	 * Previously, AS wasn't concerned about args length, as we used the (unindex) post_content column. However,
-	 * as we prepare to move to custom tables, and can use an indexed VARCHAR column instead, we want to warn
-	 * developers of this impending requirement.
-	 *
-	 * @param ActionScheduler_Action $action Action object.
-	 */
-	protected function validate_action( ActionScheduler_Action $action ) {
-		try {
-			parent::validate_action( $action );
-		} catch ( Exception $e ) {
-			/* translators: %s is the error message */
-			$message = sprintf( __( '%s Support for strings longer than this will be removed in a future version.', 'action-scheduler' ), $e->getMessage() );
-			_doing_it_wrong( 'ActionScheduler_Action::$args', esc_html( $message ), '2.1.0' );
-		}
-	}
-
-	/**
-	 * (@codeCoverageIgnore)
-	 */
-	public function init() {
-		add_filter( 'action_scheduler_migration_dependencies_met', array( $this, 'migration_dependencies_met' ) );
-
-		$post_type_registrar = new ActionScheduler_wpPostStore_PostTypeRegistrar();
-		$post_type_registrar->register();
-
-		$post_status_registrar = new ActionScheduler_wpPostStore_PostStatusRegistrar();
-		$post_status_registrar->register();
-
-		$taxonomy_registrar = new ActionScheduler_wpPostStore_TaxonomyRegistrar();
-		$taxonomy_registrar->register();
-	}
+        if ( $limit_ids ) {
+            $where .= ' AND ID IN (' . join( ',', $ids ) . ')';
+        } else {
+            $where   .= ' AND post_date_gmt <= %s';
+            $params[] = $date->format( 'Y-m-d H:i:s' );
+        }
+
+        // Add the ORDER BY clause and,ms limit.
+        $order    = 'ORDER BY menu_order ASC, post_date_gmt ASC, ID ASC LIMIT %d';
+        $params[] = $limit;
+
+        // Run the query and gather results.
+        $rows_affected = $wpdb->query( $wpdb->prepare( "{$update} {$where} {$order}", $params ) ); // phpcs:ignore // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared, WordPress.DB.PreparedSQLPlaceholders.UnfinishedPrepare
+
+        if ( false === $rows_affected ) {
+            throw new RuntimeException( __( 'Unable to claim actions. Database error.', 'action-scheduler' ) );
+        }
+
+        return (int) $rows_affected;
+    }
+
+    /**
+     * Get IDs of actions within a certain group and up to a certain date/time.
+     *
+     * @param string   $group The group to use in finding actions.
+     * @param int      $limit The number of actions to retrieve.
+     * @param DateTime $date  DateTime object representing cutoff time for actions. Actions retrieved will be
+     *                        up to and including this DateTime.
+     *
+     * @return array IDs of actions in the appropriate group and before the appropriate time.
+     * @throws InvalidArgumentException When the group does not exist.
+     */
+    protected function get_actions_by_group( $group, $limit, DateTime $date ) {
+        // Ensure the group exists before continuing.
+        if ( ! term_exists( $group, self::GROUP_TAXONOMY ) ) {
+            /* translators: %s is the group name */
+            throw new InvalidArgumentException( sprintf( __( 'The group "%s" does not exist.', 'action-scheduler' ), $group ) );
+        }
+
+        // Set up a query for post IDs to use later.
+        $query      = new WP_Query();
+        $query_args = array(
+            'fields'           => 'ids',
+            'post_type'        => self::POST_TYPE,
+            'post_status'      => ActionScheduler_Store::STATUS_PENDING,
+            'has_password'     => false,
+            'posts_per_page'   => $limit * 3,
+            'suppress_filters' => true,
+            'no_found_rows'    => true,
+            'orderby'          => array(
+                'menu_order' => 'ASC',
+                'date'       => 'ASC',
+                'ID'         => 'ASC',
+            ),
+            'date_query'       => array(
+                'column'    => 'post_date_gmt',
+                'before'    => $date->format( 'Y-m-d H:i' ),
+                'inclusive' => true,
+            ),
+            'tax_query'        => array( // phpcs:ignore WordPress.DB.SlowDBQuery
+                array(
+                    'taxonomy'         => self::GROUP_TAXONOMY,
+                    'field'            => 'slug',
+                    'terms'            => $group,
+                    'include_children' => false,
+                ),
+            ),
+        );
+
+        return $query->query( $query_args );
+    }
+
+    /**
+     * Find actions by claim ID.
+     *
+     * @param string $claim_id Claim ID.
+     * @return array
+     */
+    public function find_actions_by_claim_id( $claim_id ) {
+        /**
+         * Global wpdb object.
+         *
+         * @var wpdb $wpdb
+         */
+        global $wpdb;
+
+        $action_ids  = array();
+        $before_date = isset( $this->claim_before_date ) ? $this->claim_before_date : as_get_datetime_object();
+        $cut_off     = $before_date->format( 'Y-m-d H:i:s' );
+
+        // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
+        $results = $wpdb->get_results(
+            $wpdb->prepare(
+                "SELECT ID, post_date_gmt FROM {$wpdb->posts} WHERE post_type = %s AND post_password = %s",
+                array(
+                    self::POST_TYPE,
+                    $claim_id,
+                )
+            )
+        );
+
+        // Verify that the scheduled date for each action is within the expected bounds (in some unusual
+        // cases, we cannot depend on MySQL to honor all of the WHERE conditions we specify).
+        foreach ( $results as $claimed_action ) {
+            if ( $claimed_action->post_date_gmt <= $cut_off ) {
+                $action_ids[] = absint( $claimed_action->ID );
+            }
+        }
+
+        return $action_ids;
+    }
+
+    /**
+     * Release claim.
+     *
+     * @param ActionScheduler_ActionClaim $claim Claim object to release.
+     * @return void
+     * @throws RuntimeException When the claim is not unlocked.
+     */
+    public function release_claim( ActionScheduler_ActionClaim $claim ) {
+        $action_ids = $this->find_actions_by_claim_id( $claim->get_id() );
+        if ( empty( $action_ids ) ) {
+            return; // nothing to do.
+        }
+        $action_id_string = implode( ',', array_map( 'intval', $action_ids ) );
+        /**
+         * Global wpdb object.
+         *
+         * @var wpdb $wpdb
+         */
+        global $wpdb;
+
+        // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
+        $result = $wpdb->query(
+            $wpdb->prepare(
+                "UPDATE {$wpdb->posts} SET post_password = '' WHERE ID IN ($action_id_string) AND post_password = %s", //phpcs:ignore
+                array(
+                    $claim->get_id(),
+                )
+            )
+        );
+        if ( false === $result ) {
+            /* translators: %s: claim ID */
+            throw new RuntimeException( sprintf( __( 'Unable to unlock claim %s. Database error.', 'action-scheduler' ), $claim->get_id() ) );
+        }
+    }
+
+    /**
+     * Unclaim action.
+     *
+     * @param string $action_id Action ID.
+     * @throws RuntimeException When unable to unlock claim on action ID.
+     */
+    public function unclaim_action( $action_id ) {
+        /**
+         * Global wpdb object.
+         *
+         * @var wpdb $wpdb
+         */
+        global $wpdb;
+
+        //phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
+        $result = $wpdb->query(
+            $wpdb->prepare(
+                "UPDATE {$wpdb->posts} SET post_password = '' WHERE ID = %d AND post_type = %s",
+                $action_id,
+                self::POST_TYPE
+            )
+        );
+        if ( false === $result ) {
+            /* translators: %s: action ID */
+            throw new RuntimeException( sprintf( __( 'Unable to unlock claim on action %s. Database error.', 'action-scheduler' ), $action_id ) );
+        }
+    }
+
+    /**
+     * Mark failure on action.
+     *
+     * @param int $action_id Action ID.
+     *
+     * @return void
+     * @throws RuntimeException When unable to mark failure on action ID.
+     */
+    public function mark_failure( $action_id ) {
+        /**
+         * Global wpdb object.
+         *
+         * @var wpdb $wpdb
+         */
+        global $wpdb;
+
+        // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
+        $result = $wpdb->query(
+            $wpdb->prepare( "UPDATE {$wpdb->posts} SET post_status = %s WHERE ID = %d AND post_type = %s", self::STATUS_FAILED, $action_id, self::POST_TYPE )
+        );
+        if ( false === $result ) {
+            /* translators: %s: action ID */
+            throw new RuntimeException( sprintf( __( 'Unable to mark failure on action %s. Database error.', 'action-scheduler' ), $action_id ) );
+        }
+    }
+
+    /**
+     * Return an action's claim ID, as stored in the post password column
+     *
+     * @param int $action_id Action ID.
+     * @return mixed
+     */
+    public function get_claim_id( $action_id ) {
+        return $this->get_post_column( $action_id, 'post_password' );
+    }
+
+    /**
+     * Return an action's status, as stored in the post status column
+     *
+     * @param int $action_id Action ID.
+     *
+     * @return mixed
+     * @throws InvalidArgumentException When the action ID is invalid.
+     */
+    public function get_status( $action_id ) {
+        $status = $this->get_post_column( $action_id, 'post_status' );
+
+        if ( null === $status ) {
+            throw new InvalidArgumentException( __( 'Invalid action ID. No status found.', 'action-scheduler' ) );
+        }
+
+        return $this->get_action_status_by_post_status( $status );
+    }
+
+    /**
+     * Get post column
+     *
+     * @param string $action_id Action ID.
+     * @param string $column_name Column Name.
+     *
+     * @return string|null
+     */
+    private function get_post_column( $action_id, $column_name ) {
+        /**
+         * Global wpdb object.
+         *
+         * @var wpdb $wpdb
+         */
+        global $wpdb;
+
+        // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
+        return $wpdb->get_var(
+            $wpdb->prepare(
+                "SELECT {$column_name} FROM {$wpdb->posts} WHERE ID=%d AND post_type=%s", // phpcs:ignore
+                $action_id,
+                self::POST_TYPE
+            )
+        );
+    }
+
+    /**
+     * Log Execution.
+     *
+     * @param string $action_id Action ID.
+     */
+    public function log_execution( $action_id ) {
+        /**
+         * Global wpdb object.
+         *
+         * @var wpdb $wpdb
+         */
+        global $wpdb;
+
+        // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
+        $wpdb->query(
+            $wpdb->prepare(
+                "UPDATE {$wpdb->posts} SET menu_order = menu_order+1, post_status=%s, post_modified_gmt = %s, post_modified = %s WHERE ID = %d AND post_type = %s",
+                self::STATUS_RUNNING,
+                current_time( 'mysql', true ),
+                current_time( 'mysql' ),
+                $action_id,
+                self::POST_TYPE
+            )
+        );
+    }
+
+    /**
+     * Record that an action was completed.
+     *
+     * @param string $action_id ID of the completed action.
+     *
+     * @throws InvalidArgumentException When the action ID is invalid.
+     * @throws RuntimeException         When there was an error executing the action.
+     */
+    public function mark_complete( $action_id ) {
+        $post = get_post( $action_id );
+        if ( empty( $post ) || ( self::POST_TYPE !== $post->post_type ) ) {
+            /* translators: %s is the action ID */
+            throw new InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+        }
+        add_filter( 'wp_insert_post_data', array( $this, 'filter_insert_post_data' ), 10, 1 );
+        add_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10, 5 );
+        $result = wp_update_post(
+            array(
+                'ID'          => $action_id,
+                'post_status' => 'publish',
+            ),
+            true
+        );
+        remove_filter( 'wp_insert_post_data', array( $this, 'filter_insert_post_data' ), 10 );
+        remove_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10 );
+        if ( is_wp_error( $result ) ) {
+            throw new RuntimeException( $result->get_error_message() );
+        }
+
+        /**
+         * Fires after a scheduled action has been completed.
+         *
+         * @since 3.4.2
+         *
+         * @param int $action_id Action ID.
+         */
+        do_action( 'action_scheduler_completed_action', $action_id );
+    }
+
+    /**
+     * Mark action as migrated when there is an error deleting the action.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function mark_migrated( $action_id ) {
+        wp_update_post(
+            array(
+                'ID'          => $action_id,
+                'post_status' => 'migrated',
+            )
+        );
+    }
+
+    /**
+     * Determine whether the post store can be migrated.
+     *
+     * @param [type] $setting - Setting value.
+     * @return bool
+     */
+    public function migration_dependencies_met( $setting ) {
+        global $wpdb;
+
+        $dependencies_met = get_transient( self::DEPENDENCIES_MET );
+        if ( empty( $dependencies_met ) ) {
+            $maximum_args_length = apply_filters( 'action_scheduler_maximum_args_length', 191 );
+            $found_action        = $wpdb->get_var( // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
+                $wpdb->prepare(
+                    "SELECT ID FROM {$wpdb->posts} WHERE post_type = %s AND CHAR_LENGTH(post_content) > %d LIMIT 1",
+                    $maximum_args_length,
+                    self::POST_TYPE
+                )
+            );
+            $dependencies_met    = $found_action ? 'no' : 'yes';
+            set_transient( self::DEPENDENCIES_MET, $dependencies_met, DAY_IN_SECONDS );
+        }
+
+        return 'yes' === $dependencies_met ? $setting : false;
+    }
+
+    /**
+     * InnoDB indexes have a maximum size of 767 bytes by default, which is only 191 characters with utf8mb4.
+     *
+     * Previously, AS wasn't concerned about args length, as we used the (unindex) post_content column. However,
+     * as we prepare to move to custom tables, and can use an indexed VARCHAR column instead, we want to warn
+     * developers of this impending requirement.
+     *
+     * @param ActionScheduler_Action $action Action object.
+     */
+    protected function validate_action( ActionScheduler_Action $action ) {
+        try {
+            parent::validate_action( $action );
+        } catch ( Exception $e ) {
+            /* translators: %s is the error message */
+            $message = sprintf( __( '%s Support for strings longer than this will be removed in a future version.', 'action-scheduler' ), $e->getMessage() );
+            _doing_it_wrong( 'ActionScheduler_Action::$args', esc_html( $message ), '2.1.0' );
+        }
+    }
+
+    /**
+     * (@codeCoverageIgnore)
+     */
+    public function init() {
+        add_filter( 'action_scheduler_migration_dependencies_met', array( $this, 'migration_dependencies_met' ) );
+
+        $post_type_registrar = new ActionScheduler_wpPostStore_PostTypeRegistrar();
+        $post_type_registrar->register();
+
+        $post_status_registrar = new ActionScheduler_wpPostStore_PostStatusRegistrar();
+        $post_status_registrar->register();
+
+        $taxonomy_registrar = new ActionScheduler_wpPostStore_TaxonomyRegistrar();
+        $taxonomy_registrar->register();
+    }
 }

Spacing +210 added lines, -210 removed lines

@@ -35,18 +35,18 @@ 
 	 * @throws RuntimeException Throws an exception if the action could not be saved.
 	 * @return int
 	 */
-	public function save_action( ActionScheduler_Action $action, DateTime $scheduled_date = null ) {
+	public function save_action(ActionScheduler_Action $action, DateTime $scheduled_date = null) {
 		try {
-			$this->validate_action( $action );
-			$post_array = $this->create_post_array( $action, $scheduled_date );
-			$post_id    = $this->save_post_array( $post_array );
-			$this->save_post_schedule( $post_id, $action->get_schedule() );
-			$this->save_action_group( $post_id, $action->get_group() );
-			do_action( 'action_scheduler_stored_action', $post_id );
+			$this->validate_action($action);
+			$post_array = $this->create_post_array($action, $scheduled_date);
+			$post_id    = $this->save_post_array($post_array);
+			$this->save_post_schedule($post_id, $action->get_schedule());
+			$this->save_action_group($post_id, $action->get_group());
+			do_action('action_scheduler_stored_action', $post_id);
 			return $post_id;
-		} catch ( Exception $e ) {
+		} catch (Exception $e) {
 			/* translators: %s: action error message */
-			throw new RuntimeException( sprintf( __( 'Error saving action: %s', 'action-scheduler' ), $e->getMessage() ), 0 );
+			throw new RuntimeException(sprintf(__('Error saving action: %s', 'action-scheduler'), $e->getMessage()), 0);
 		}
 	}
 
@@ -58,14 +58,14 @@ 
 	 *
 	 * @return array Returns an array of post data.
 	 */
-	protected function create_post_array( ActionScheduler_Action $action, DateTime $scheduled_date = null ) {
+	protected function create_post_array(ActionScheduler_Action $action, DateTime $scheduled_date = null) {
 		$post = array(
 			'post_type'     => self::POST_TYPE,
 			'post_title'    => $action->get_hook(),
-			'post_content'  => wp_json_encode( $action->get_args() ),
-			'post_status'   => ( $action->is_finished() ? 'publish' : 'pending' ),
-			'post_date_gmt' => $this->get_scheduled_date_string( $action, $scheduled_date ),
-			'post_date'     => $this->get_scheduled_date_string_local( $action, $scheduled_date ),
+			'post_content'  => wp_json_encode($action->get_args()),
+			'post_status'   => ($action->is_finished() ? 'publish' : 'pending'),
+			'post_date_gmt' => $this->get_scheduled_date_string($action, $scheduled_date),
+			'post_date'     => $this->get_scheduled_date_string_local($action, $scheduled_date),
 		);
 		return $post;
 	}
@@ -77,28 +77,28 @@ 
 	 * @return int Returns the post ID.
 	 * @throws RuntimeException Throws an exception if the action could not be saved.
 	 */
-	protected function save_post_array( $post_array ) {
-		add_filter( 'wp_insert_post_data', array( $this, 'filter_insert_post_data' ), 10, 1 );
-		add_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10, 5 );
+	protected function save_post_array($post_array) {
+		add_filter('wp_insert_post_data', array($this, 'filter_insert_post_data'), 10, 1);
+		add_filter('pre_wp_unique_post_slug', array($this, 'set_unique_post_slug'), 10, 5);
 
-		$has_kses = false !== has_filter( 'content_save_pre', 'wp_filter_post_kses' );
+		$has_kses = false !== has_filter('content_save_pre', 'wp_filter_post_kses');
 
-		if ( $has_kses ) {
+		if ($has_kses) {
 			// Prevent KSES from corrupting JSON in post_content.
 			kses_remove_filters();
 		}
 
-		$post_id = wp_insert_post( $post_array );
+		$post_id = wp_insert_post($post_array);
 
-		if ( $has_kses ) {
+		if ($has_kses) {
 			kses_init_filters();
 		}
 
-		remove_filter( 'wp_insert_post_data', array( $this, 'filter_insert_post_data' ), 10 );
-		remove_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10 );
+		remove_filter('wp_insert_post_data', array($this, 'filter_insert_post_data'), 10);
+		remove_filter('pre_wp_unique_post_slug', array($this, 'set_unique_post_slug'), 10);
 
-		if ( is_wp_error( $post_id ) || empty( $post_id ) ) {
-			throw new RuntimeException( __( 'Unable to save action.', 'action-scheduler' ) );
+		if (is_wp_error($post_id) || empty($post_id)) {
+			throw new RuntimeException(__('Unable to save action.', 'action-scheduler'));
 		}
 		return $post_id;
 	}
@@ -110,10 +110,10 @@ 
 	 *
 	 * @return array
 	 */
-	public function filter_insert_post_data( $postdata ) {
-		if ( self::POST_TYPE === $postdata['post_type'] ) {
+	public function filter_insert_post_data($postdata) {
+		if (self::POST_TYPE === $postdata['post_type']) {
 			$postdata['post_author'] = 0;
-			if ( 'future' === $postdata['post_status'] ) {
+			if ('future' === $postdata['post_status']) {
 				$postdata['post_status'] = 'publish';
 			}
 		}
@@ -148,9 +148,9 @@ 
 	 * @param string $post_type     Post type.
 	 * @return string
 	 */
-	public function set_unique_post_slug( $override_slug, $slug, $post_ID, $post_status, $post_type ) {
-		if ( self::POST_TYPE === $post_type ) {
-			$override_slug = uniqid( self::POST_TYPE . '-', true ) . '-' . wp_generate_password( 32, false );
+	public function set_unique_post_slug($override_slug, $slug, $post_ID, $post_status, $post_type) {
+		if (self::POST_TYPE === $post_type) {
+			$override_slug = uniqid(self::POST_TYPE.'-', true).'-'.wp_generate_password(32, false);
 		}
 		return $override_slug;
 	}
@@ -163,8 +163,8 @@ 
 	 *
 	 * @return void
 	 */
-	protected function save_post_schedule( $post_id, $schedule ) {
-		update_post_meta( $post_id, self::SCHEDULE_META_KEY, $schedule );
+	protected function save_post_schedule($post_id, $schedule) {
+		update_post_meta($post_id, self::SCHEDULE_META_KEY, $schedule);
 	}
 
 	/**
@@ -174,11 +174,11 @@ 
 	 * @param string $group   Group to save.
 	 * @return void
 	 */
-	protected function save_action_group( $post_id, $group ) {
-		if ( empty( $group ) ) {
-			wp_set_object_terms( $post_id, array(), self::GROUP_TAXONOMY, false );
+	protected function save_action_group($post_id, $group) {
+		if (empty($group)) {
+			wp_set_object_terms($post_id, array(), self::GROUP_TAXONOMY, false);
 		} else {
-			wp_set_object_terms( $post_id, array( $group ), self::GROUP_TAXONOMY, false );
+			wp_set_object_terms($post_id, array($group), self::GROUP_TAXONOMY, false);
 		}
 	}
 
@@ -188,16 +188,16 @@ 
 	 * @param int $action_id Action ID.
 	 * @return object
 	 */
-	public function fetch_action( $action_id ) {
-		$post = $this->get_post( $action_id );
-		if ( empty( $post ) || self::POST_TYPE !== $post->post_type ) {
+	public function fetch_action($action_id) {
+		$post = $this->get_post($action_id);
+		if (empty($post) || self::POST_TYPE !== $post->post_type) {
 			return $this->get_null_action();
 		}
 
 		try {
-			$action = $this->make_action_from_post( $post );
-		} catch ( ActionScheduler_InvalidActionException $exception ) {
-			do_action( 'action_scheduler_failed_fetch_action', $post->ID, $exception );
+			$action = $this->make_action_from_post($post);
+		} catch (ActionScheduler_InvalidActionException $exception) {
+			do_action('action_scheduler_failed_fetch_action', $post->ID, $exception);
 			return $this->get_null_action();
 		}
 
@@ -210,11 +210,11 @@ 
 	 * @param string $action_id - Action ID.
 	 * @return WP_Post|null
 	 */
-	protected function get_post( $action_id ) {
-		if ( empty( $action_id ) ) {
+	protected function get_post($action_id) {
+		if (empty($action_id)) {
 			return null;
 		}
-		return get_post( $action_id );
+		return get_post($action_id);
 	}
 
 	/**
@@ -232,19 +232,19 @@ 
 	 * @param WP_Post $post Post object.
 	 * @return WP_Post
 	 */
-	protected function make_action_from_post( $post ) {
+	protected function make_action_from_post($post) {
 		$hook = $post->post_title;
 
-		$args = json_decode( $post->post_content, true );
-		$this->validate_args( $args, $post->ID );
+		$args = json_decode($post->post_content, true);
+		$this->validate_args($args, $post->ID);
 
-		$schedule = get_post_meta( $post->ID, self::SCHEDULE_META_KEY, true );
-		$this->validate_schedule( $schedule, $post->ID );
+		$schedule = get_post_meta($post->ID, self::SCHEDULE_META_KEY, true);
+		$this->validate_schedule($schedule, $post->ID);
 
-		$group = wp_get_object_terms( $post->ID, self::GROUP_TAXONOMY, array( 'fields' => 'names' ) );
-		$group = empty( $group ) ? '' : reset( $group );
+		$group = wp_get_object_terms($post->ID, self::GROUP_TAXONOMY, array('fields' => 'names'));
+		$group = empty($group) ? '' : reset($group);
 
-		return ActionScheduler::factory()->get_stored_action( $this->get_action_status_by_post_status( $post->post_status ), $hook, $args, $schedule, $group );
+		return ActionScheduler::factory()->get_stored_action($this->get_action_status_by_post_status($post->post_status), $hook, $args, $schedule, $group);
 	}
 
 	/**
@@ -255,9 +255,9 @@ 
 	 * @throws InvalidArgumentException Throw InvalidArgumentException if $post_status not in known status fields returned by $this->get_status_labels().
 	 * @return string
 	 */
-	protected function get_action_status_by_post_status( $post_status ) {
+	protected function get_action_status_by_post_status($post_status) {
 
-		switch ( $post_status ) {
+		switch ($post_status) {
 			case 'publish':
 				$action_status = self::STATUS_COMPLETE;
 				break;
@@ -265,8 +265,8 @@ 
 				$action_status = self::STATUS_CANCELED;
 				break;
 			default:
-				if ( ! array_key_exists( $post_status, $this->get_status_labels() ) ) {
-					throw new InvalidArgumentException( sprintf( 'Invalid post status: "%s". No matching action status available.', $post_status ) );
+				if ( ! array_key_exists($post_status, $this->get_status_labels())) {
+					throw new InvalidArgumentException(sprintf('Invalid post status: "%s". No matching action status available.', $post_status));
 				}
 				$action_status = $post_status;
 				break;
@@ -283,9 +283,9 @@ 
 	 * @throws InvalidArgumentException Throws InvalidArgumentException if $post_status not in known status fields returned by $this->get_status_labels().
 	 * @return string
 	 */
-	protected function get_post_status_by_action_status( $action_status ) {
+	protected function get_post_status_by_action_status($action_status) {
 
-		switch ( $action_status ) {
+		switch ($action_status) {
 			case self::STATUS_COMPLETE:
 				$post_status = 'publish';
 				break;
@@ -293,8 +293,8 @@ 
 				$post_status = 'trash';
 				break;
 			default:
-				if ( ! array_key_exists( $action_status, $this->get_status_labels() ) ) {
-					throw new InvalidArgumentException( sprintf( 'Invalid action status: "%s".', $action_status ) );
+				if ( ! array_key_exists($action_status, $this->get_status_labels())) {
+					throw new InvalidArgumentException(sprintf('Invalid action status: "%s".', $action_status));
 				}
 				$post_status = $action_status;
 				break;
@@ -312,10 +312,10 @@ 
 	 * @throws InvalidArgumentException - Throw InvalidArgumentException if $select_or_count not count or select.
 	 * @return string SQL statement. The returned SQL is already properly escaped.
 	 */
-	protected function get_query_actions_sql( array $query, $select_or_count = 'select' ) {
+	protected function get_query_actions_sql(array $query, $select_or_count = 'select') {
 
-		if ( ! in_array( $select_or_count, array( 'select', 'count' ), true ) ) {
-			throw new InvalidArgumentException( __( 'Invalid schedule. Cannot save action.', 'action-scheduler' ) );
+		if ( ! in_array($select_or_count, array('select', 'count'), true)) {
+			throw new InvalidArgumentException(__('Invalid schedule. Cannot save action.', 'action-scheduler'));
 		}
 
 		$query = wp_parse_args(
@@ -344,14 +344,14 @@ 
 		 * @var wpdb $wpdb
 		 */
 		global $wpdb;
-		$sql        = ( 'count' === $select_or_count ) ? 'SELECT count(p.ID)' : 'SELECT p.ID ';
+		$sql        = ('count' === $select_or_count) ? 'SELECT count(p.ID)' : 'SELECT p.ID ';
 		$sql       .= "FROM {$wpdb->posts} p";
 		$sql_params = array();
-		if ( empty( $query['group'] ) && 'group' === $query['orderby'] ) {
+		if (empty($query['group']) && 'group' === $query['orderby']) {
 			$sql .= " LEFT JOIN {$wpdb->term_relationships} tr ON tr.object_id=p.ID";
 			$sql .= " LEFT JOIN {$wpdb->term_taxonomy} tt ON tr.term_taxonomy_id=tt.term_taxonomy_id";
 			$sql .= " LEFT JOIN {$wpdb->terms} t ON tt.term_id=t.term_id";
-		} elseif ( ! empty( $query['group'] ) ) {
+		} elseif ( ! empty($query['group'])) {
 			$sql         .= " INNER JOIN {$wpdb->term_relationships} tr ON tr.object_id=p.ID";
 			$sql         .= " INNER JOIN {$wpdb->term_taxonomy} tt ON tr.term_taxonomy_id=tt.term_taxonomy_id";
 			$sql         .= " INNER JOIN {$wpdb->terms} t ON tt.term_id=t.term_id";
@@ -360,58 +360,58 @@ 
 		}
 		$sql         .= ' WHERE post_type=%s';
 		$sql_params[] = self::POST_TYPE;
-		if ( $query['hook'] ) {
+		if ($query['hook']) {
 			$sql         .= ' AND p.post_title=%s';
 			$sql_params[] = $query['hook'];
 		}
-		if ( $query['args'] !== null ) {
+		if ($query['args'] !== null) {
 			$sql         .= ' AND p.post_content=%s';
-			$sql_params[] = wp_json_encode( $query['args'] );
+			$sql_params[] = wp_json_encode($query['args']);
 		}
 
-		if ( $query['status'] ) {
-			$post_statuses = array_map( array( $this, 'get_post_status_by_action_status' ), (array) $query['status'] );
-			$placeholders  = array_fill( 0, count( $post_statuses ), '%s' );
-			$sql          .= ' AND p.post_status IN (' . join( ', ', $placeholders ) . ')';
-			$sql_params    = array_merge( $sql_params, array_values( $post_statuses ) );
+		if ($query['status']) {
+			$post_statuses = array_map(array($this, 'get_post_status_by_action_status'), (array) $query['status']);
+			$placeholders  = array_fill(0, count($post_statuses), '%s');
+			$sql          .= ' AND p.post_status IN ('.join(', ', $placeholders).')';
+			$sql_params    = array_merge($sql_params, array_values($post_statuses));
 		}
 
-		if ( $query['date'] instanceof DateTime ) {
+		if ($query['date'] instanceof DateTime) {
 			$date = clone $query['date'];
-			$date->setTimezone( new DateTimeZone( 'UTC' ) );
-			$date_string  = $date->format( 'Y-m-d H:i:s' );
-			$comparator   = $this->validate_sql_comparator( $query['date_compare'] );
+			$date->setTimezone(new DateTimeZone('UTC'));
+			$date_string  = $date->format('Y-m-d H:i:s');
+			$comparator   = $this->validate_sql_comparator($query['date_compare']);
 			$sql         .= " AND p.post_date_gmt $comparator %s";
 			$sql_params[] = $date_string;
 		}
 
-		if ( $query['modified'] instanceof DateTime ) {
+		if ($query['modified'] instanceof DateTime) {
 			$modified = clone $query['modified'];
-			$modified->setTimezone( new DateTimeZone( 'UTC' ) );
-			$date_string  = $modified->format( 'Y-m-d H:i:s' );
-			$comparator   = $this->validate_sql_comparator( $query['modified_compare'] );
+			$modified->setTimezone(new DateTimeZone('UTC'));
+			$date_string  = $modified->format('Y-m-d H:i:s');
+			$comparator   = $this->validate_sql_comparator($query['modified_compare']);
 			$sql         .= " AND p.post_modified_gmt $comparator %s";
 			$sql_params[] = $date_string;
 		}
 
-		if ( true === $query['claimed'] ) {
+		if (true === $query['claimed']) {
 			$sql .= " AND p.post_password != ''";
-		} elseif ( false === $query['claimed'] ) {
+		} elseif (false === $query['claimed']) {
 			$sql .= " AND p.post_password = ''";
-		} elseif ( $query['claimed'] !== null ) {
+		} elseif ($query['claimed'] !== null) {
 			$sql         .= ' AND p.post_password = %s';
 			$sql_params[] = $query['claimed'];
 		}
 
-		if ( ! empty( $query['search'] ) ) {
+		if ( ! empty($query['search'])) {
 			$sql .= ' AND (p.post_title LIKE %s OR p.post_content LIKE %s OR p.post_password LIKE %s)';
-			for ( $i = 0; $i < 3; $i++ ) {
-				$sql_params[] = sprintf( '%%%s%%', $query['search'] );
+			for ($i = 0; $i < 3; $i++) {
+				$sql_params[] = sprintf('%%%s%%', $query['search']);
 			}
 		}
 
-		if ( 'select' === $select_or_count ) {
-			switch ( $query['orderby'] ) {
+		if ('select' === $select_or_count) {
+			switch ($query['orderby']) {
 				case 'hook':
 					$orderby = 'p.post_title';
 					break;
@@ -433,20 +433,20 @@ 
 					$orderby = 'p.post_date_gmt';
 					break;
 			}
-			if ( 'ASC' === strtoupper( $query['order'] ) ) {
+			if ('ASC' === strtoupper($query['order'])) {
 				$order = 'ASC';
 			} else {
 				$order = 'DESC';
 			}
 			$sql .= " ORDER BY $orderby $order";
-			if ( $query['per_page'] > 0 ) {
+			if ($query['per_page'] > 0) {
 				$sql         .= ' LIMIT %d, %d';
 				$sql_params[] = $query['offset'];
 				$sql_params[] = $query['per_page'];
 			}
 		}
 
-		return $wpdb->prepare( $sql, $sql_params ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		return $wpdb->prepare($sql, $sql_params); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
 	}
 
 	/**
@@ -461,7 +461,7 @@ 
 	 *
 	 * @return string|array|null The IDs of actions matching the query. Null on failure.
 	 */
-	public function query_actions( $query = array(), $query_type = 'select' ) {
+	public function query_actions($query = array(), $query_type = 'select') {
 		/**
 		 * Global $wpdb object.
 		 *
@@ -469,9 +469,9 @@ 
 		 */
 		global $wpdb;
 
-		$sql = $this->get_query_actions_sql( $query, $query_type );
+		$sql = $this->get_query_actions_sql($query, $query_type);
 
-		return ( 'count' === $query_type ) ? $wpdb->get_var( $sql ) : $wpdb->get_col( $sql ); // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery,WordPress.DB.DirectDatabaseQuery.NoCaching,WordPress.DB.PreparedSQL.NotPrepared
+		return ('count' === $query_type) ? $wpdb->get_var($sql) : $wpdb->get_col($sql); // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery,WordPress.DB.DirectDatabaseQuery.NoCaching,WordPress.DB.PreparedSQL.NotPrepared
 	}
 
 	/**
@@ -483,18 +483,18 @@ 
 
 		$action_counts_by_status = array();
 		$action_stati_and_labels = $this->get_status_labels();
-		$posts_count_by_status   = (array) wp_count_posts( self::POST_TYPE, 'readable' );
+		$posts_count_by_status   = (array) wp_count_posts(self::POST_TYPE, 'readable');
 
-		foreach ( $posts_count_by_status as $post_status_name => $count ) {
+		foreach ($posts_count_by_status as $post_status_name => $count) {
 
 			try {
-				$action_status_name = $this->get_action_status_by_post_status( $post_status_name );
-			} catch ( Exception $e ) {
+				$action_status_name = $this->get_action_status_by_post_status($post_status_name);
+			} catch (Exception $e) {
 				// Ignore any post statuses that aren't for actions.
 				continue;
 			}
-			if ( array_key_exists( $action_status_name, $action_stati_and_labels ) ) {
-				$action_counts_by_status[ $action_status_name ] = $count;
+			if (array_key_exists($action_status_name, $action_stati_and_labels)) {
+				$action_counts_by_status[$action_status_name] = $count;
 			}
 		}
 
@@ -508,16 +508,16 @@ 
 	 *
 	 * @throws InvalidArgumentException If $action_id is not identified.
 	 */
-	public function cancel_action( $action_id ) {
-		$post = get_post( $action_id );
-		if ( empty( $post ) || ( self::POST_TYPE !== $post->post_type ) ) {
+	public function cancel_action($action_id) {
+		$post = get_post($action_id);
+		if (empty($post) || (self::POST_TYPE !== $post->post_type)) {
 			/* translators: %s is the action ID */
-			throw new InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+			throw new InvalidArgumentException(sprintf(__('Unidentified action %s', 'action-scheduler'), $action_id));
 		}
-		do_action( 'action_scheduler_canceled_action', $action_id );
-		add_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10, 5 );
-		wp_trash_post( $action_id );
-		remove_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10 );
+		do_action('action_scheduler_canceled_action', $action_id);
+		add_filter('pre_wp_unique_post_slug', array($this, 'set_unique_post_slug'), 10, 5);
+		wp_trash_post($action_id);
+		remove_filter('pre_wp_unique_post_slug', array($this, 'set_unique_post_slug'), 10);
 	}
 
 	/**
@@ -527,15 +527,15 @@ 
 	 * @return void
 	 * @throws InvalidArgumentException If action is not identified.
 	 */
-	public function delete_action( $action_id ) {
-		$post = get_post( $action_id );
-		if ( empty( $post ) || ( self::POST_TYPE !== $post->post_type ) ) {
+	public function delete_action($action_id) {
+		$post = get_post($action_id);
+		if (empty($post) || (self::POST_TYPE !== $post->post_type)) {
 			/* translators: %s is the action ID */
-			throw new InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+			throw new InvalidArgumentException(sprintf(__('Unidentified action %s', 'action-scheduler'), $action_id));
 		}
-		do_action( 'action_scheduler_deleted_action', $action_id );
+		do_action('action_scheduler_deleted_action', $action_id);
 
-		wp_delete_post( $action_id, true );
+		wp_delete_post($action_id, true);
 	}
 
 	/**
@@ -544,9 +544,9 @@ 
 	 * @param int $action_id Action ID.
 	 * @return ActionScheduler_DateTime The date the action is schedule to run, or the date that it ran.
 	 */
-	public function get_date( $action_id ) {
-		$next = $this->get_date_gmt( $action_id );
-		return ActionScheduler_TimezoneHelper::set_local_timezone( $next );
+	public function get_date($action_id) {
+		$next = $this->get_date_gmt($action_id);
+		return ActionScheduler_TimezoneHelper::set_local_timezone($next);
 	}
 
 	/**
@@ -557,16 +557,16 @@ 
 	 * @throws InvalidArgumentException If $action_id is not identified.
 	 * @return ActionScheduler_DateTime The date the action is schedule to run, or the date that it ran.
 	 */
-	public function get_date_gmt( $action_id ) {
-		$post = get_post( $action_id );
-		if ( empty( $post ) || ( self::POST_TYPE !== $post->post_type ) ) {
+	public function get_date_gmt($action_id) {
+		$post = get_post($action_id);
+		if (empty($post) || (self::POST_TYPE !== $post->post_type)) {
 			/* translators: %s is the action ID */
-			throw new InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+			throw new InvalidArgumentException(sprintf(__('Unidentified action %s', 'action-scheduler'), $action_id));
 		}
-		if ( 'publish' === $post->post_status ) {
-			return as_get_datetime_object( $post->post_modified_gmt );
+		if ('publish' === $post->post_status) {
+			return as_get_datetime_object($post->post_modified_gmt);
 		} else {
-			return as_get_datetime_object( $post->post_date_gmt );
+			return as_get_datetime_object($post->post_date_gmt);
 		}
 	}
 
@@ -582,14 +582,14 @@ 
 	 * @throws RuntimeException When there is an error staking a claim.
 	 * @throws InvalidArgumentException When the given group is not valid.
 	 */
-	public function stake_claim( $max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '' ) {
+	public function stake_claim($max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '') {
 		$this->claim_before_date = $before_date;
 		$claim_id                = $this->generate_claim_id();
-		$this->claim_actions( $claim_id, $max_actions, $before_date, $hooks, $group );
-		$action_ids              = $this->find_actions_by_claim_id( $claim_id );
+		$this->claim_actions($claim_id, $max_actions, $before_date, $hooks, $group);
+		$action_ids              = $this->find_actions_by_claim_id($claim_id);
 		$this->claim_before_date = null;
 
-		return new ActionScheduler_ActionClaim( $claim_id, $action_ids );
+		return new ActionScheduler_ActionClaim($claim_id, $action_ids);
 	}
 
 	/**
@@ -604,7 +604,7 @@ 
 		return $wpdb->get_var(
 			$wpdb->prepare(
 				"SELECT COUNT(DISTINCT post_password) FROM {$wpdb->posts} WHERE post_password != '' AND post_type = %s AND post_status IN ('in-progress','pending')",
-				array( self::POST_TYPE )
+				array(self::POST_TYPE)
 			)
 		);
 	}
@@ -615,8 +615,8 @@ 
 	 * @return string
 	 */
 	protected function generate_claim_id() {
-		$claim_id = md5( microtime( true ) . wp_rand( 0, 1000 ) );
-		return substr( $claim_id, 0, 20 ); // to fit in db field with 20 char limit.
+		$claim_id = md5(microtime(true).wp_rand(0, 1000));
+		return substr($claim_id, 0, 20); // to fit in db field with 20 char limit.
 	}
 
 	/**
@@ -631,14 +631,14 @@ 
 	 * @return int The number of actions that were claimed.
 	 * @throws RuntimeException  When there is a database error.
 	 */
-	protected function claim_actions( $claim_id, $limit, DateTime $before_date = null, $hooks = array(), $group = '' ) {
+	protected function claim_actions($claim_id, $limit, DateTime $before_date = null, $hooks = array(), $group = '') {
 		// Set up initial variables.
 		$date      = null === $before_date ? as_get_datetime_object() : clone $before_date;
-		$limit_ids = ! empty( $group );
-		$ids       = $limit_ids ? $this->get_actions_by_group( $group, $limit, $date ) : array();
+		$limit_ids = ! empty($group);
+		$ids       = $limit_ids ? $this->get_actions_by_group($group, $limit, $date) : array();
 
 		// If limiting by IDs and no posts found, then return early since we have nothing to update.
-		if ( $limit_ids && 0 === count( $ids ) ) {
+		if ($limit_ids && 0 === count($ids)) {
 			return 0;
 		}
 
@@ -658,8 +658,8 @@ 
 		$update = "UPDATE {$wpdb->posts} SET post_password = %s, post_modified_gmt = %s, post_modified = %s";
 		$params = array(
 			$claim_id,
-			current_time( 'mysql', true ),
-			current_time( 'mysql' ),
+			current_time('mysql', true),
+			current_time('mysql'),
 		);
 
 		// Build initial WHERE clause.
@@ -667,10 +667,10 @@ 
 		$params[] = self::POST_TYPE;
 		$params[] = ActionScheduler_Store::STATUS_PENDING;
 
-		if ( ! empty( $hooks ) ) {
-			$placeholders = array_fill( 0, count( $hooks ), '%s' );
-			$where       .= ' AND post_title IN (' . join( ', ', $placeholders ) . ')';
-			$params       = array_merge( $params, array_values( $hooks ) );
+		if ( ! empty($hooks)) {
+			$placeholders = array_fill(0, count($hooks), '%s');
+			$where       .= ' AND post_title IN ('.join(', ', $placeholders).')';
+			$params       = array_merge($params, array_values($hooks));
 		}
 
 		/*
@@ -678,11 +678,11 @@ 
 		 *
 		 * If we're not limiting by IDs, then include the post_date_gmt clause.
 		 */
-		if ( $limit_ids ) {
-			$where .= ' AND ID IN (' . join( ',', $ids ) . ')';
+		if ($limit_ids) {
+			$where .= ' AND ID IN ('.join(',', $ids).')';
 		} else {
 			$where   .= ' AND post_date_gmt <= %s';
-			$params[] = $date->format( 'Y-m-d H:i:s' );
+			$params[] = $date->format('Y-m-d H:i:s');
 		}
 
 		// Add the ORDER BY clause and,ms limit.
@@ -690,10 +690,10 @@ 
 		$params[] = $limit;
 
 		// Run the query and gather results.
-		$rows_affected = $wpdb->query( $wpdb->prepare( "{$update} {$where} {$order}", $params ) ); // phpcs:ignore // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared, WordPress.DB.PreparedSQLPlaceholders.UnfinishedPrepare
+		$rows_affected = $wpdb->query($wpdb->prepare("{$update} {$where} {$order}", $params)); // phpcs:ignore // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared, WordPress.DB.PreparedSQLPlaceholders.UnfinishedPrepare
 
-		if ( false === $rows_affected ) {
-			throw new RuntimeException( __( 'Unable to claim actions. Database error.', 'action-scheduler' ) );
+		if (false === $rows_affected) {
+			throw new RuntimeException(__('Unable to claim actions. Database error.', 'action-scheduler'));
 		}
 
 		return (int) $rows_affected;
@@ -710,11 +710,11 @@ 
 	 * @return array IDs of actions in the appropriate group and before the appropriate time.
 	 * @throws InvalidArgumentException When the group does not exist.
 	 */
-	protected function get_actions_by_group( $group, $limit, DateTime $date ) {
+	protected function get_actions_by_group($group, $limit, DateTime $date) {
 		// Ensure the group exists before continuing.
-		if ( ! term_exists( $group, self::GROUP_TAXONOMY ) ) {
+		if ( ! term_exists($group, self::GROUP_TAXONOMY)) {
 			/* translators: %s is the group name */
-			throw new InvalidArgumentException( sprintf( __( 'The group "%s" does not exist.', 'action-scheduler' ), $group ) );
+			throw new InvalidArgumentException(sprintf(__('The group "%s" does not exist.', 'action-scheduler'), $group));
 		}
 
 		// Set up a query for post IDs to use later.
@@ -734,7 +734,7 @@ 
 			),
 			'date_query'       => array(
 				'column'    => 'post_date_gmt',
-				'before'    => $date->format( 'Y-m-d H:i' ),
+				'before'    => $date->format('Y-m-d H:i'),
 				'inclusive' => true,
 			),
 			'tax_query'        => array( // phpcs:ignore WordPress.DB.SlowDBQuery
@@ -747,7 +747,7 @@ 
 			),
 		);
 
-		return $query->query( $query_args );
+		return $query->query($query_args);
 	}
 
 	/**
@@ -756,7 +756,7 @@ 
 	 * @param string $claim_id Claim ID.
 	 * @return array
 	 */
-	public function find_actions_by_claim_id( $claim_id ) {
+	public function find_actions_by_claim_id($claim_id) {
 		/**
 		 * Global wpdb object.
 		 *
@@ -765,8 +765,8 @@ 
 		global $wpdb;
 
 		$action_ids  = array();
-		$before_date = isset( $this->claim_before_date ) ? $this->claim_before_date : as_get_datetime_object();
-		$cut_off     = $before_date->format( 'Y-m-d H:i:s' );
+		$before_date = isset($this->claim_before_date) ? $this->claim_before_date : as_get_datetime_object();
+		$cut_off     = $before_date->format('Y-m-d H:i:s');
 
 		// phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
 		$results = $wpdb->get_results(
@@ -781,9 +781,9 @@ 
 
 		// Verify that the scheduled date for each action is within the expected bounds (in some unusual
 		// cases, we cannot depend on MySQL to honor all of the WHERE conditions we specify).
-		foreach ( $results as $claimed_action ) {
-			if ( $claimed_action->post_date_gmt <= $cut_off ) {
-				$action_ids[] = absint( $claimed_action->ID );
+		foreach ($results as $claimed_action) {
+			if ($claimed_action->post_date_gmt <= $cut_off) {
+				$action_ids[] = absint($claimed_action->ID);
 			}
 		}
 
@@ -797,12 +797,12 @@ 
 	 * @return void
 	 * @throws RuntimeException When the claim is not unlocked.
 	 */
-	public function release_claim( ActionScheduler_ActionClaim $claim ) {
-		$action_ids = $this->find_actions_by_claim_id( $claim->get_id() );
-		if ( empty( $action_ids ) ) {
+	public function release_claim(ActionScheduler_ActionClaim $claim) {
+		$action_ids = $this->find_actions_by_claim_id($claim->get_id());
+		if (empty($action_ids)) {
 			return; // nothing to do.
 		}
-		$action_id_string = implode( ',', array_map( 'intval', $action_ids ) );
+		$action_id_string = implode(',', array_map('intval', $action_ids));
 		/**
 		 * Global wpdb object.
 		 *
@@ -819,9 +819,9 @@ 
 				)
 			)
 		);
-		if ( false === $result ) {
+		if (false === $result) {
 			/* translators: %s: claim ID */
-			throw new RuntimeException( sprintf( __( 'Unable to unlock claim %s. Database error.', 'action-scheduler' ), $claim->get_id() ) );
+			throw new RuntimeException(sprintf(__('Unable to unlock claim %s. Database error.', 'action-scheduler'), $claim->get_id()));
 		}
 	}
 
@@ -831,7 +831,7 @@ 
 	 * @param string $action_id Action ID.
 	 * @throws RuntimeException When unable to unlock claim on action ID.
 	 */
-	public function unclaim_action( $action_id ) {
+	public function unclaim_action($action_id) {
 		/**
 		 * Global wpdb object.
 		 *
@@ -847,9 +847,9 @@ 
 				self::POST_TYPE
 			)
 		);
-		if ( false === $result ) {
+		if (false === $result) {
 			/* translators: %s: action ID */
-			throw new RuntimeException( sprintf( __( 'Unable to unlock claim on action %s. Database error.', 'action-scheduler' ), $action_id ) );
+			throw new RuntimeException(sprintf(__('Unable to unlock claim on action %s. Database error.', 'action-scheduler'), $action_id));
 		}
 	}
 
@@ -861,7 +861,7 @@ 
 	 * @return void
 	 * @throws RuntimeException When unable to mark failure on action ID.
 	 */
-	public function mark_failure( $action_id ) {
+	public function mark_failure($action_id) {
 		/**
 		 * Global wpdb object.
 		 *
@@ -871,11 +871,11 @@ 
 
 		// phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
 		$result = $wpdb->query(
-			$wpdb->prepare( "UPDATE {$wpdb->posts} SET post_status = %s WHERE ID = %d AND post_type = %s", self::STATUS_FAILED, $action_id, self::POST_TYPE )
+			$wpdb->prepare("UPDATE {$wpdb->posts} SET post_status = %s WHERE ID = %d AND post_type = %s", self::STATUS_FAILED, $action_id, self::POST_TYPE)
 		);
-		if ( false === $result ) {
+		if (false === $result) {
 			/* translators: %s: action ID */
-			throw new RuntimeException( sprintf( __( 'Unable to mark failure on action %s. Database error.', 'action-scheduler' ), $action_id ) );
+			throw new RuntimeException(sprintf(__('Unable to mark failure on action %s. Database error.', 'action-scheduler'), $action_id));
 		}
 	}
 
@@ -885,8 +885,8 @@ 
 	 * @param int $action_id Action ID.
 	 * @return mixed
 	 */
-	public function get_claim_id( $action_id ) {
-		return $this->get_post_column( $action_id, 'post_password' );
+	public function get_claim_id($action_id) {
+		return $this->get_post_column($action_id, 'post_password');
 	}
 
 	/**
@@ -897,14 +897,14 @@ 
 	 * @return mixed
 	 * @throws InvalidArgumentException When the action ID is invalid.
 	 */
-	public function get_status( $action_id ) {
-		$status = $this->get_post_column( $action_id, 'post_status' );
+	public function get_status($action_id) {
+		$status = $this->get_post_column($action_id, 'post_status');
 
-		if ( null === $status ) {
-			throw new InvalidArgumentException( __( 'Invalid action ID. No status found.', 'action-scheduler' ) );
+		if (null === $status) {
+			throw new InvalidArgumentException(__('Invalid action ID. No status found.', 'action-scheduler'));
 		}
 
-		return $this->get_action_status_by_post_status( $status );
+		return $this->get_action_status_by_post_status($status);
 	}
 
 	/**
@@ -915,7 +915,7 @@ 
 	 *
 	 * @return string|null
 	 */
-	private function get_post_column( $action_id, $column_name ) {
+	private function get_post_column($action_id, $column_name) {
 		/**
 		 * Global wpdb object.
 		 *
@@ -938,7 +938,7 @@ 
 	 *
 	 * @param string $action_id Action ID.
 	 */
-	public function log_execution( $action_id ) {
+	public function log_execution($action_id) {
 		/**
 		 * Global wpdb object.
 		 *
@@ -951,8 +951,8 @@ 
 			$wpdb->prepare(
 				"UPDATE {$wpdb->posts} SET menu_order = menu_order+1, post_status=%s, post_modified_gmt = %s, post_modified = %s WHERE ID = %d AND post_type = %s",
 				self::STATUS_RUNNING,
-				current_time( 'mysql', true ),
-				current_time( 'mysql' ),
+				current_time('mysql', true),
+				current_time('mysql'),
 				$action_id,
 				self::POST_TYPE
 			)
@@ -967,14 +967,14 @@ 
 	 * @throws InvalidArgumentException When the action ID is invalid.
 	 * @throws RuntimeException         When there was an error executing the action.
 	 */
-	public function mark_complete( $action_id ) {
-		$post = get_post( $action_id );
-		if ( empty( $post ) || ( self::POST_TYPE !== $post->post_type ) ) {
+	public function mark_complete($action_id) {
+		$post = get_post($action_id);
+		if (empty($post) || (self::POST_TYPE !== $post->post_type)) {
 			/* translators: %s is the action ID */
-			throw new InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) );
+			throw new InvalidArgumentException(sprintf(__('Unidentified action %s', 'action-scheduler'), $action_id));
 		}
-		add_filter( 'wp_insert_post_data', array( $this, 'filter_insert_post_data' ), 10, 1 );
-		add_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10, 5 );
+		add_filter('wp_insert_post_data', array($this, 'filter_insert_post_data'), 10, 1);
+		add_filter('pre_wp_unique_post_slug', array($this, 'set_unique_post_slug'), 10, 5);
 		$result = wp_update_post(
 			array(
 				'ID'          => $action_id,
@@ -982,10 +982,10 @@ 
 			),
 			true
 		);
-		remove_filter( 'wp_insert_post_data', array( $this, 'filter_insert_post_data' ), 10 );
-		remove_filter( 'pre_wp_unique_post_slug', array( $this, 'set_unique_post_slug' ), 10 );
-		if ( is_wp_error( $result ) ) {
-			throw new RuntimeException( $result->get_error_message() );
+		remove_filter('wp_insert_post_data', array($this, 'filter_insert_post_data'), 10);
+		remove_filter('pre_wp_unique_post_slug', array($this, 'set_unique_post_slug'), 10);
+		if (is_wp_error($result)) {
+			throw new RuntimeException($result->get_error_message());
 		}
 
 		/**
@@ -995,7 +995,7 @@ 
 		 *
 		 * @param int $action_id Action ID.
 		 */
-		do_action( 'action_scheduler_completed_action', $action_id );
+		do_action('action_scheduler_completed_action', $action_id);
 	}
 
 	/**
@@ -1003,7 +1003,7 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function mark_migrated( $action_id ) {
+	public function mark_migrated($action_id) {
 		wp_update_post(
 			array(
 				'ID'          => $action_id,
@@ -1018,12 +1018,12 @@ 
 	 * @param [type] $setting - Setting value.
 	 * @return bool
 	 */
-	public function migration_dependencies_met( $setting ) {
+	public function migration_dependencies_met($setting) {
 		global $wpdb;
 
-		$dependencies_met = get_transient( self::DEPENDENCIES_MET );
-		if ( empty( $dependencies_met ) ) {
-			$maximum_args_length = apply_filters( 'action_scheduler_maximum_args_length', 191 );
+		$dependencies_met = get_transient(self::DEPENDENCIES_MET);
+		if (empty($dependencies_met)) {
+			$maximum_args_length = apply_filters('action_scheduler_maximum_args_length', 191);
 			$found_action        = $wpdb->get_var( // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
 				$wpdb->prepare(
 					"SELECT ID FROM {$wpdb->posts} WHERE post_type = %s AND CHAR_LENGTH(post_content) > %d LIMIT 1",
@@ -1031,8 +1031,8 @@ 
 					self::POST_TYPE
 				)
 			);
-			$dependencies_met    = $found_action ? 'no' : 'yes';
-			set_transient( self::DEPENDENCIES_MET, $dependencies_met, DAY_IN_SECONDS );
+			$dependencies_met = $found_action ? 'no' : 'yes';
+			set_transient(self::DEPENDENCIES_MET, $dependencies_met, DAY_IN_SECONDS);
 		}
 
 		return 'yes' === $dependencies_met ? $setting : false;
@@ -1047,13 +1047,13 @@ 
 	 *
 	 * @param ActionScheduler_Action $action Action object.
 	 */
-	protected function validate_action( ActionScheduler_Action $action ) {
+	protected function validate_action(ActionScheduler_Action $action) {
 		try {
-			parent::validate_action( $action );
-		} catch ( Exception $e ) {
+			parent::validate_action($action);
+		} catch (Exception $e) {
 			/* translators: %s is the error message */
-			$message = sprintf( __( '%s Support for strings longer than this will be removed in a future version.', 'action-scheduler' ), $e->getMessage() );
-			_doing_it_wrong( 'ActionScheduler_Action::$args', esc_html( $message ), '2.1.0' );
+			$message = sprintf(__('%s Support for strings longer than this will be removed in a future version.', 'action-scheduler'), $e->getMessage());
+			_doing_it_wrong('ActionScheduler_Action::$args', esc_html($message), '2.1.0');
 		}
 	}
 
@@ -1061,7 +1061,7 @@ 
 	 * (@codeCoverageIgnore)
 	 */
 	public function init() {
-		add_filter( 'action_scheduler_migration_dependencies_met', array( $this, 'migration_dependencies_met' ) );
+		add_filter('action_scheduler_migration_dependencies_met', array($this, 'migration_dependencies_met'));
 
 		$post_type_registrar = new ActionScheduler_wpPostStore_PostTypeRegistrar();
 		$post_type_registrar->register();

src/libraries/action-scheduler/classes/data-stores/ActionScheduler_HybridStore.php

Indentation +406 added lines, -406 removed lines

@@ -13,415 +13,415 @@
  * @since 3.0.0
  */
 class ActionScheduler_HybridStore extends Store {
-	const DEMARKATION_OPTION = 'action_scheduler_hybrid_store_demarkation';
-
-	private $primary_store;
-	private $secondary_store;
-	private $migration_runner;
-
-	/**
-	 * @var int The dividing line between IDs of actions created
-	 *          by the primary and secondary stores.
-	 *
-	 * Methods that accept an action ID will compare the ID against
-	 * this to determine which store will contain that ID. In almost
-	 * all cases, the ID should come from the primary store, but if
-	 * client code is bypassing the API functions and fetching IDs
-	 * from elsewhere, then there is a chance that an unmigrated ID
-	 * might be requested.
-	 */
-	private $demarkation_id = 0;
-
-	/**
-	 * ActionScheduler_HybridStore constructor.
-	 *
-	 * @param Config $config Migration config object.
-	 */
-	public function __construct( Config $config = null ) {
-		$this->demarkation_id = (int) get_option( self::DEMARKATION_OPTION, 0 );
-		if ( empty( $config ) ) {
-			$config = Controller::instance()->get_migration_config_object();
-		}
-		$this->primary_store    = $config->get_destination_store();
-		$this->secondary_store  = $config->get_source_store();
-		$this->migration_runner = new Runner( $config );
-	}
-
-	/**
-	 * Initialize the table data store tables.
-	 *
-	 * @codeCoverageIgnore
-	 */
-	public function init() {
-		add_action( 'action_scheduler/created_table', array( $this, 'set_autoincrement' ), 10, 2 );
-		$this->primary_store->init();
-		$this->secondary_store->init();
-		remove_action( 'action_scheduler/created_table', array( $this, 'set_autoincrement' ), 10 );
-	}
-
-	/**
-	 * When the actions table is created, set its autoincrement
-	 * value to be one higher than the posts table to ensure that
-	 * there are no ID collisions.
-	 *
-	 * @param string $table_name
-	 * @param string $table_suffix
-	 *
-	 * @return void
-	 * @codeCoverageIgnore
-	 */
-	public function set_autoincrement( $table_name, $table_suffix ) {
-		if ( ActionScheduler_StoreSchema::ACTIONS_TABLE === $table_suffix ) {
-			if ( empty( $this->demarkation_id ) ) {
-				$this->demarkation_id = $this->set_demarkation_id();
-			}
-			/** @var \wpdb $wpdb */
-			global $wpdb;
-			/**
-			 * A default date of '0000-00-00 00:00:00' is invalid in MySQL 5.7 when configured with
-			 * sql_mode including both STRICT_TRANS_TABLES and NO_ZERO_DATE.
-			 */
-			$default_date = new DateTime( 'tomorrow' );
-			$null_action  = new ActionScheduler_NullAction();
-			$date_gmt     = $this->get_scheduled_date_string( $null_action, $default_date );
-			$date_local   = $this->get_scheduled_date_string_local( $null_action, $default_date );
-
-			$row_count = $wpdb->insert(
-				$wpdb->{ActionScheduler_StoreSchema::ACTIONS_TABLE},
-				array(
-					'action_id'            => $this->demarkation_id,
-					'hook'                 => '',
-					'status'               => '',
-					'scheduled_date_gmt'   => $date_gmt,
-					'scheduled_date_local' => $date_local,
-					'last_attempt_gmt'     => $date_gmt,
-					'last_attempt_local'   => $date_local,
-				)
-			);
-			if ( $row_count > 0 ) {
-				$wpdb->delete(
-					$wpdb->{ActionScheduler_StoreSchema::ACTIONS_TABLE},
-					array( 'action_id' => $this->demarkation_id )
-				);
-			}
-		}
-	}
-
-	/**
-	 * Store the demarkation id in WP options.
-	 *
-	 * @param int $id The ID to set as the demarkation point between the two stores
-	 *                Leave null to use the next ID from the WP posts table.
-	 *
-	 * @return int The new ID.
-	 *
-	 * @codeCoverageIgnore
-	 */
-	private function set_demarkation_id( $id = null ) {
-		if ( empty( $id ) ) {
-			/** @var \wpdb $wpdb */
-			global $wpdb;
-			$id = (int) $wpdb->get_var( "SELECT MAX(ID) FROM $wpdb->posts" );
-			$id ++;
-		}
-		update_option( self::DEMARKATION_OPTION, $id );
-
-		return $id;
-	}
-
-	/**
-	 * Find the first matching action from the secondary store.
-	 * If it exists, migrate it to the primary store immediately.
-	 * After it migrates, the secondary store will logically contain
-	 * the next matching action, so return the result thence.
-	 *
-	 * @param string $hook
-	 * @param array  $params
-	 *
-	 * @return string
-	 */
-	public function find_action( $hook, $params = array() ) {
-		$found_unmigrated_action = $this->secondary_store->find_action( $hook, $params );
-		if ( ! empty( $found_unmigrated_action ) ) {
-			$this->migrate( array( $found_unmigrated_action ) );
-		}
-
-		return $this->primary_store->find_action( $hook, $params );
-	}
-
-	/**
-	 * Find actions matching the query in the secondary source first.
-	 * If any are found, migrate them immediately. Then the secondary
-	 * store will contain the canonical results.
-	 *
-	 * @param array  $query
-	 * @param string $query_type Whether to select or count the results. Default, select.
-	 *
-	 * @return int[]
-	 */
-	public function query_actions( $query = array(), $query_type = 'select' ) {
-		$found_unmigrated_actions = $this->secondary_store->query_actions( $query, 'select' );
-		if ( ! empty( $found_unmigrated_actions ) ) {
-			$this->migrate( $found_unmigrated_actions );
-		}
-
-		return $this->primary_store->query_actions( $query, $query_type );
-	}
-
-	/**
-	 * Get a count of all actions in the store, grouped by status
-	 *
-	 * @return array Set of 'status' => int $count pairs for statuses with 1 or more actions of that status.
-	 */
-	public function action_counts() {
-		$unmigrated_actions_count = $this->secondary_store->action_counts();
-		$migrated_actions_count   = $this->primary_store->action_counts();
-		$actions_count_by_status  = array();
-
-		foreach ( $this->get_status_labels() as $status_key => $status_label ) {
-
-			$count = 0;
-
-			if ( isset( $unmigrated_actions_count[ $status_key ] ) ) {
-				$count += $unmigrated_actions_count[ $status_key ];
-			}
-
-			if ( isset( $migrated_actions_count[ $status_key ] ) ) {
-				$count += $migrated_actions_count[ $status_key ];
-			}
-
-			$actions_count_by_status[ $status_key ] = $count;
-		}
-
-		$actions_count_by_status = array_filter( $actions_count_by_status );
-
-		return $actions_count_by_status;
-	}
-
-	/**
-	 * If any actions would have been claimed by the secondary store,
-	 * migrate them immediately, then ask the primary store for the
-	 * canonical claim.
-	 *
-	 * @param int           $max_actions
-	 * @param DateTime|null $before_date
-	 *
-	 * @return ActionScheduler_ActionClaim
-	 */
-	public function stake_claim( $max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '' ) {
-		$claim = $this->secondary_store->stake_claim( $max_actions, $before_date, $hooks, $group );
-
-		$claimed_actions = $claim->get_actions();
-		if ( ! empty( $claimed_actions ) ) {
-			$this->migrate( $claimed_actions );
-		}
-
-		$this->secondary_store->release_claim( $claim );
-
-		return $this->primary_store->stake_claim( $max_actions, $before_date, $hooks, $group );
-	}
-
-	/**
-	 * Migrate a list of actions to the table data store.
-	 *
-	 * @param array $action_ids List of action IDs.
-	 */
-	private function migrate( $action_ids ) {
-		$this->migration_runner->migrate_actions( $action_ids );
-	}
-
-	/**
-	 * Save an action to the primary store.
-	 *
-	 * @param ActionScheduler_Action $action Action object to be saved.
-	 * @param DateTime               $date Optional. Schedule date. Default null.
-	 *
-	 * @return int The action ID
-	 */
-	public function save_action( ActionScheduler_Action $action, DateTime $date = null ) {
-		return $this->primary_store->save_action( $action, $date );
-	}
-
-	/**
-	 * Retrieve an existing action whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function fetch_action( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id, true );
-		if ( $store ) {
-			return $store->fetch_action( $action_id );
-		} else {
-			return new ActionScheduler_NullAction();
-		}
-	}
-
-	/**
-	 * Cancel an existing action whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function cancel_action( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id );
-		if ( $store ) {
-			$store->cancel_action( $action_id );
-		}
-	}
-
-	/**
-	 * Delete an existing action whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function delete_action( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id );
-		if ( $store ) {
-			$store->delete_action( $action_id );
-		}
-	}
-
-	/**
-	 * Get the schedule date an existing action whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function get_date( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id );
-		if ( $store ) {
-			return $store->get_date( $action_id );
-		} else {
-			return null;
-		}
-	}
-
-	/**
-	 * Mark an existing action as failed whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function mark_failure( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id );
-		if ( $store ) {
-			$store->mark_failure( $action_id );
-		}
-	}
-
-	/**
-	 * Log the execution of an existing action whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function log_execution( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id );
-		if ( $store ) {
-			$store->log_execution( $action_id );
-		}
-	}
-
-	/**
-	 * Mark an existing action complete whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function mark_complete( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id );
-		if ( $store ) {
-			$store->mark_complete( $action_id );
-		}
-	}
-
-	/**
-	 * Get an existing action status whether migrated or not.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function get_status( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id );
-		if ( $store ) {
-			return $store->get_status( $action_id );
-		}
-		return null;
-	}
-
-	/**
-	 * Return which store an action is stored in.
-	 *
-	 * @param int  $action_id ID of the action.
-	 * @param bool $primary_first Optional flag indicating search the primary store first.
-	 * @return ActionScheduler_Store
-	 */
-	protected function get_store_from_action_id( $action_id, $primary_first = false ) {
-		if ( $primary_first ) {
-			$stores = array(
-				$this->primary_store,
-				$this->secondary_store,
-			);
-		} elseif ( $action_id < $this->demarkation_id ) {
-			$stores = array(
-				$this->secondary_store,
-				$this->primary_store,
-			);
-		} else {
-			$stores = array(
-				$this->primary_store,
-			);
-		}
-
-		foreach ( $stores as $store ) {
-			$action = $store->fetch_action( $action_id );
-			if ( ! is_a( $action, 'ActionScheduler_NullAction' ) ) {
-				return $store;
-			}
-		}
-		return null;
-	}
-
-	/*
+    const DEMARKATION_OPTION = 'action_scheduler_hybrid_store_demarkation';
+
+    private $primary_store;
+    private $secondary_store;
+    private $migration_runner;
+
+    /**
+     * @var int The dividing line between IDs of actions created
+     *          by the primary and secondary stores.
+     *
+     * Methods that accept an action ID will compare the ID against
+     * this to determine which store will contain that ID. In almost
+     * all cases, the ID should come from the primary store, but if
+     * client code is bypassing the API functions and fetching IDs
+     * from elsewhere, then there is a chance that an unmigrated ID
+     * might be requested.
+     */
+    private $demarkation_id = 0;
+
+    /**
+     * ActionScheduler_HybridStore constructor.
+     *
+     * @param Config $config Migration config object.
+     */
+    public function __construct( Config $config = null ) {
+        $this->demarkation_id = (int) get_option( self::DEMARKATION_OPTION, 0 );
+        if ( empty( $config ) ) {
+            $config = Controller::instance()->get_migration_config_object();
+        }
+        $this->primary_store    = $config->get_destination_store();
+        $this->secondary_store  = $config->get_source_store();
+        $this->migration_runner = new Runner( $config );
+    }
+
+    /**
+     * Initialize the table data store tables.
+     *
+     * @codeCoverageIgnore
+     */
+    public function init() {
+        add_action( 'action_scheduler/created_table', array( $this, 'set_autoincrement' ), 10, 2 );
+        $this->primary_store->init();
+        $this->secondary_store->init();
+        remove_action( 'action_scheduler/created_table', array( $this, 'set_autoincrement' ), 10 );
+    }
+
+    /**
+     * When the actions table is created, set its autoincrement
+     * value to be one higher than the posts table to ensure that
+     * there are no ID collisions.
+     *
+     * @param string $table_name
+     * @param string $table_suffix
+     *
+     * @return void
+     * @codeCoverageIgnore
+     */
+    public function set_autoincrement( $table_name, $table_suffix ) {
+        if ( ActionScheduler_StoreSchema::ACTIONS_TABLE === $table_suffix ) {
+            if ( empty( $this->demarkation_id ) ) {
+                $this->demarkation_id = $this->set_demarkation_id();
+            }
+            /** @var \wpdb $wpdb */
+            global $wpdb;
+            /**
+             * A default date of '0000-00-00 00:00:00' is invalid in MySQL 5.7 when configured with
+             * sql_mode including both STRICT_TRANS_TABLES and NO_ZERO_DATE.
+             */
+            $default_date = new DateTime( 'tomorrow' );
+            $null_action  = new ActionScheduler_NullAction();
+            $date_gmt     = $this->get_scheduled_date_string( $null_action, $default_date );
+            $date_local   = $this->get_scheduled_date_string_local( $null_action, $default_date );
+
+            $row_count = $wpdb->insert(
+                $wpdb->{ActionScheduler_StoreSchema::ACTIONS_TABLE},
+                array(
+                    'action_id'            => $this->demarkation_id,
+                    'hook'                 => '',
+                    'status'               => '',
+                    'scheduled_date_gmt'   => $date_gmt,
+                    'scheduled_date_local' => $date_local,
+                    'last_attempt_gmt'     => $date_gmt,
+                    'last_attempt_local'   => $date_local,
+                )
+            );
+            if ( $row_count > 0 ) {
+                $wpdb->delete(
+                    $wpdb->{ActionScheduler_StoreSchema::ACTIONS_TABLE},
+                    array( 'action_id' => $this->demarkation_id )
+                );
+            }
+        }
+    }
+
+    /**
+     * Store the demarkation id in WP options.
+     *
+     * @param int $id The ID to set as the demarkation point between the two stores
+     *                Leave null to use the next ID from the WP posts table.
+     *
+     * @return int The new ID.
+     *
+     * @codeCoverageIgnore
+     */
+    private function set_demarkation_id( $id = null ) {
+        if ( empty( $id ) ) {
+            /** @var \wpdb $wpdb */
+            global $wpdb;
+            $id = (int) $wpdb->get_var( "SELECT MAX(ID) FROM $wpdb->posts" );
+            $id ++;
+        }
+        update_option( self::DEMARKATION_OPTION, $id );
+
+        return $id;
+    }
+
+    /**
+     * Find the first matching action from the secondary store.
+     * If it exists, migrate it to the primary store immediately.
+     * After it migrates, the secondary store will logically contain
+     * the next matching action, so return the result thence.
+     *
+     * @param string $hook
+     * @param array  $params
+     *
+     * @return string
+     */
+    public function find_action( $hook, $params = array() ) {
+        $found_unmigrated_action = $this->secondary_store->find_action( $hook, $params );
+        if ( ! empty( $found_unmigrated_action ) ) {
+            $this->migrate( array( $found_unmigrated_action ) );
+        }
+
+        return $this->primary_store->find_action( $hook, $params );
+    }
+
+    /**
+     * Find actions matching the query in the secondary source first.
+     * If any are found, migrate them immediately. Then the secondary
+     * store will contain the canonical results.
+     *
+     * @param array  $query
+     * @param string $query_type Whether to select or count the results. Default, select.
+     *
+     * @return int[]
+     */
+    public function query_actions( $query = array(), $query_type = 'select' ) {
+        $found_unmigrated_actions = $this->secondary_store->query_actions( $query, 'select' );
+        if ( ! empty( $found_unmigrated_actions ) ) {
+            $this->migrate( $found_unmigrated_actions );
+        }
+
+        return $this->primary_store->query_actions( $query, $query_type );
+    }
+
+    /**
+     * Get a count of all actions in the store, grouped by status
+     *
+     * @return array Set of 'status' => int $count pairs for statuses with 1 or more actions of that status.
+     */
+    public function action_counts() {
+        $unmigrated_actions_count = $this->secondary_store->action_counts();
+        $migrated_actions_count   = $this->primary_store->action_counts();
+        $actions_count_by_status  = array();
+
+        foreach ( $this->get_status_labels() as $status_key => $status_label ) {
+
+            $count = 0;
+
+            if ( isset( $unmigrated_actions_count[ $status_key ] ) ) {
+                $count += $unmigrated_actions_count[ $status_key ];
+            }
+
+            if ( isset( $migrated_actions_count[ $status_key ] ) ) {
+                $count += $migrated_actions_count[ $status_key ];
+            }
+
+            $actions_count_by_status[ $status_key ] = $count;
+        }
+
+        $actions_count_by_status = array_filter( $actions_count_by_status );
+
+        return $actions_count_by_status;
+    }
+
+    /**
+     * If any actions would have been claimed by the secondary store,
+     * migrate them immediately, then ask the primary store for the
+     * canonical claim.
+     *
+     * @param int           $max_actions
+     * @param DateTime|null $before_date
+     *
+     * @return ActionScheduler_ActionClaim
+     */
+    public function stake_claim( $max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '' ) {
+        $claim = $this->secondary_store->stake_claim( $max_actions, $before_date, $hooks, $group );
+
+        $claimed_actions = $claim->get_actions();
+        if ( ! empty( $claimed_actions ) ) {
+            $this->migrate( $claimed_actions );
+        }
+
+        $this->secondary_store->release_claim( $claim );
+
+        return $this->primary_store->stake_claim( $max_actions, $before_date, $hooks, $group );
+    }
+
+    /**
+     * Migrate a list of actions to the table data store.
+     *
+     * @param array $action_ids List of action IDs.
+     */
+    private function migrate( $action_ids ) {
+        $this->migration_runner->migrate_actions( $action_ids );
+    }
+
+    /**
+     * Save an action to the primary store.
+     *
+     * @param ActionScheduler_Action $action Action object to be saved.
+     * @param DateTime               $date Optional. Schedule date. Default null.
+     *
+     * @return int The action ID
+     */
+    public function save_action( ActionScheduler_Action $action, DateTime $date = null ) {
+        return $this->primary_store->save_action( $action, $date );
+    }
+
+    /**
+     * Retrieve an existing action whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function fetch_action( $action_id ) {
+        $store = $this->get_store_from_action_id( $action_id, true );
+        if ( $store ) {
+            return $store->fetch_action( $action_id );
+        } else {
+            return new ActionScheduler_NullAction();
+        }
+    }
+
+    /**
+     * Cancel an existing action whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function cancel_action( $action_id ) {
+        $store = $this->get_store_from_action_id( $action_id );
+        if ( $store ) {
+            $store->cancel_action( $action_id );
+        }
+    }
+
+    /**
+     * Delete an existing action whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function delete_action( $action_id ) {
+        $store = $this->get_store_from_action_id( $action_id );
+        if ( $store ) {
+            $store->delete_action( $action_id );
+        }
+    }
+
+    /**
+     * Get the schedule date an existing action whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function get_date( $action_id ) {
+        $store = $this->get_store_from_action_id( $action_id );
+        if ( $store ) {
+            return $store->get_date( $action_id );
+        } else {
+            return null;
+        }
+    }
+
+    /**
+     * Mark an existing action as failed whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function mark_failure( $action_id ) {
+        $store = $this->get_store_from_action_id( $action_id );
+        if ( $store ) {
+            $store->mark_failure( $action_id );
+        }
+    }
+
+    /**
+     * Log the execution of an existing action whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function log_execution( $action_id ) {
+        $store = $this->get_store_from_action_id( $action_id );
+        if ( $store ) {
+            $store->log_execution( $action_id );
+        }
+    }
+
+    /**
+     * Mark an existing action complete whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function mark_complete( $action_id ) {
+        $store = $this->get_store_from_action_id( $action_id );
+        if ( $store ) {
+            $store->mark_complete( $action_id );
+        }
+    }
+
+    /**
+     * Get an existing action status whether migrated or not.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function get_status( $action_id ) {
+        $store = $this->get_store_from_action_id( $action_id );
+        if ( $store ) {
+            return $store->get_status( $action_id );
+        }
+        return null;
+    }
+
+    /**
+     * Return which store an action is stored in.
+     *
+     * @param int  $action_id ID of the action.
+     * @param bool $primary_first Optional flag indicating search the primary store first.
+     * @return ActionScheduler_Store
+     */
+    protected function get_store_from_action_id( $action_id, $primary_first = false ) {
+        if ( $primary_first ) {
+            $stores = array(
+                $this->primary_store,
+                $this->secondary_store,
+            );
+        } elseif ( $action_id < $this->demarkation_id ) {
+            $stores = array(
+                $this->secondary_store,
+                $this->primary_store,
+            );
+        } else {
+            $stores = array(
+                $this->primary_store,
+            );
+        }
+
+        foreach ( $stores as $store ) {
+            $action = $store->fetch_action( $action_id );
+            if ( ! is_a( $action, 'ActionScheduler_NullAction' ) ) {
+                return $store;
+            }
+        }
+        return null;
+    }
+
+    /*
 	 * * * * * * * * * * * * * * * * * * * * * * * * * *
 	 * All claim-related functions should operate solely
 	 * on the primary store.
 	 * * * * * * * * * * * * * * * * * * * * * * * * * * */
 
-	/**
-	 * Get the claim count from the table data store.
-	 */
-	public function get_claim_count() {
-		return $this->primary_store->get_claim_count();
-	}
-
-	/**
-	 * Retrieve the claim ID for an action from the table data store.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function get_claim_id( $action_id ) {
-		return $this->primary_store->get_claim_id( $action_id );
-	}
-
-	/**
-	 * Release a claim in the table data store.
-	 *
-	 * @param ActionScheduler_ActionClaim $claim Claim object.
-	 */
-	public function release_claim( ActionScheduler_ActionClaim $claim ) {
-		$this->primary_store->release_claim( $claim );
-	}
-
-	/**
-	 * Release claims on an action in the table data store.
-	 *
-	 * @param int $action_id Action ID.
-	 */
-	public function unclaim_action( $action_id ) {
-		$this->primary_store->unclaim_action( $action_id );
-	}
-
-	/**
-	 * Retrieve a list of action IDs by claim.
-	 *
-	 * @param int $claim_id Claim ID.
-	 */
-	public function find_actions_by_claim_id( $claim_id ) {
-		return $this->primary_store->find_actions_by_claim_id( $claim_id );
-	}
+    /**
+     * Get the claim count from the table data store.
+     */
+    public function get_claim_count() {
+        return $this->primary_store->get_claim_count();
+    }
+
+    /**
+     * Retrieve the claim ID for an action from the table data store.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function get_claim_id( $action_id ) {
+        return $this->primary_store->get_claim_id( $action_id );
+    }
+
+    /**
+     * Release a claim in the table data store.
+     *
+     * @param ActionScheduler_ActionClaim $claim Claim object.
+     */
+    public function release_claim( ActionScheduler_ActionClaim $claim ) {
+        $this->primary_store->release_claim( $claim );
+    }
+
+    /**
+     * Release claims on an action in the table data store.
+     *
+     * @param int $action_id Action ID.
+     */
+    public function unclaim_action( $action_id ) {
+        $this->primary_store->unclaim_action( $action_id );
+    }
+
+    /**
+     * Retrieve a list of action IDs by claim.
+     *
+     * @param int $claim_id Claim ID.
+     */
+    public function find_actions_by_claim_id( $claim_id ) {
+        return $this->primary_store->find_actions_by_claim_id( $claim_id );
+    }
 }

Spacing +92 added lines, -92 removed lines

@@ -37,14 +37,14 @@ 
 	 *
 	 * @param Config $config Migration config object.
 	 */
-	public function __construct( Config $config = null ) {
-		$this->demarkation_id = (int) get_option( self::DEMARKATION_OPTION, 0 );
-		if ( empty( $config ) ) {
+	public function __construct(Config $config = null) {
+		$this->demarkation_id = (int) get_option(self::DEMARKATION_OPTION, 0);
+		if (empty($config)) {
 			$config = Controller::instance()->get_migration_config_object();
 		}
 		$this->primary_store    = $config->get_destination_store();
 		$this->secondary_store  = $config->get_source_store();
-		$this->migration_runner = new Runner( $config );
+		$this->migration_runner = new Runner($config);
 	}
 
 	/**
@@ -53,10 +53,10 @@ 
 	 * @codeCoverageIgnore
 	 */
 	public function init() {
-		add_action( 'action_scheduler/created_table', array( $this, 'set_autoincrement' ), 10, 2 );
+		add_action('action_scheduler/created_table', array($this, 'set_autoincrement'), 10, 2);
 		$this->primary_store->init();
 		$this->secondary_store->init();
-		remove_action( 'action_scheduler/created_table', array( $this, 'set_autoincrement' ), 10 );
+		remove_action('action_scheduler/created_table', array($this, 'set_autoincrement'), 10);
 	}
 
 	/**
@@ -70,9 +70,9 @@ 
 	 * @return void
 	 * @codeCoverageIgnore
 	 */
-	public function set_autoincrement( $table_name, $table_suffix ) {
-		if ( ActionScheduler_StoreSchema::ACTIONS_TABLE === $table_suffix ) {
-			if ( empty( $this->demarkation_id ) ) {
+	public function set_autoincrement($table_name, $table_suffix) {
+		if (ActionScheduler_StoreSchema::ACTIONS_TABLE === $table_suffix) {
+			if (empty($this->demarkation_id)) {
 				$this->demarkation_id = $this->set_demarkation_id();
 			}
 			/** @var \wpdb $wpdb */
@@ -81,10 +81,10 @@ 
 			 * A default date of '0000-00-00 00:00:00' is invalid in MySQL 5.7 when configured with
 			 * sql_mode including both STRICT_TRANS_TABLES and NO_ZERO_DATE.
 			 */
-			$default_date = new DateTime( 'tomorrow' );
+			$default_date = new DateTime('tomorrow');
 			$null_action  = new ActionScheduler_NullAction();
-			$date_gmt     = $this->get_scheduled_date_string( $null_action, $default_date );
-			$date_local   = $this->get_scheduled_date_string_local( $null_action, $default_date );
+			$date_gmt     = $this->get_scheduled_date_string($null_action, $default_date);
+			$date_local   = $this->get_scheduled_date_string_local($null_action, $default_date);
 
 			$row_count = $wpdb->insert(
 				$wpdb->{ActionScheduler_StoreSchema::ACTIONS_TABLE},
@@ -98,10 +98,10 @@ 
 					'last_attempt_local'   => $date_local,
 				)
 			);
-			if ( $row_count > 0 ) {
+			if ($row_count > 0) {
 				$wpdb->delete(
 					$wpdb->{ActionScheduler_StoreSchema::ACTIONS_TABLE},
-					array( 'action_id' => $this->demarkation_id )
+					array('action_id' => $this->demarkation_id)
 				);
 			}
 		}
@@ -117,14 +117,14 @@ 
 	 *
 	 * @codeCoverageIgnore
 	 */
-	private function set_demarkation_id( $id = null ) {
-		if ( empty( $id ) ) {
+	private function set_demarkation_id($id = null) {
+		if (empty($id)) {
 			/** @var \wpdb $wpdb */
 			global $wpdb;
-			$id = (int) $wpdb->get_var( "SELECT MAX(ID) FROM $wpdb->posts" );
-			$id ++;
+			$id = (int) $wpdb->get_var("SELECT MAX(ID) FROM $wpdb->posts");
+			$id++;
 		}
-		update_option( self::DEMARKATION_OPTION, $id );
+		update_option(self::DEMARKATION_OPTION, $id);
 
 		return $id;
 	}
@@ -140,13 +140,13 @@ 
 	 *
 	 * @return string
 	 */
-	public function find_action( $hook, $params = array() ) {
-		$found_unmigrated_action = $this->secondary_store->find_action( $hook, $params );
-		if ( ! empty( $found_unmigrated_action ) ) {
-			$this->migrate( array( $found_unmigrated_action ) );
+	public function find_action($hook, $params = array()) {
+		$found_unmigrated_action = $this->secondary_store->find_action($hook, $params);
+		if ( ! empty($found_unmigrated_action)) {
+			$this->migrate(array($found_unmigrated_action));
 		}
 
-		return $this->primary_store->find_action( $hook, $params );
+		return $this->primary_store->find_action($hook, $params);
 	}
 
 	/**
@@ -159,13 +159,13 @@ 
 	 *
 	 * @return int[]
 	 */
-	public function query_actions( $query = array(), $query_type = 'select' ) {
-		$found_unmigrated_actions = $this->secondary_store->query_actions( $query, 'select' );
-		if ( ! empty( $found_unmigrated_actions ) ) {
-			$this->migrate( $found_unmigrated_actions );
+	public function query_actions($query = array(), $query_type = 'select') {
+		$found_unmigrated_actions = $this->secondary_store->query_actions($query, 'select');
+		if ( ! empty($found_unmigrated_actions)) {
+			$this->migrate($found_unmigrated_actions);
 		}
 
-		return $this->primary_store->query_actions( $query, $query_type );
+		return $this->primary_store->query_actions($query, $query_type);
 	}
 
 	/**
@@ -178,22 +178,22 @@ 
 		$migrated_actions_count   = $this->primary_store->action_counts();
 		$actions_count_by_status  = array();
 
-		foreach ( $this->get_status_labels() as $status_key => $status_label ) {
+		foreach ($this->get_status_labels() as $status_key => $status_label) {
 
 			$count = 0;
 
-			if ( isset( $unmigrated_actions_count[ $status_key ] ) ) {
-				$count += $unmigrated_actions_count[ $status_key ];
+			if (isset($unmigrated_actions_count[$status_key])) {
+				$count += $unmigrated_actions_count[$status_key];
 			}
 
-			if ( isset( $migrated_actions_count[ $status_key ] ) ) {
-				$count += $migrated_actions_count[ $status_key ];
+			if (isset($migrated_actions_count[$status_key])) {
+				$count += $migrated_actions_count[$status_key];
 			}
 
-			$actions_count_by_status[ $status_key ] = $count;
+			$actions_count_by_status[$status_key] = $count;
 		}
 
-		$actions_count_by_status = array_filter( $actions_count_by_status );
+		$actions_count_by_status = array_filter($actions_count_by_status);
 
 		return $actions_count_by_status;
 	}
@@ -208,17 +208,17 @@ 
 	 *
 	 * @return ActionScheduler_ActionClaim
 	 */
-	public function stake_claim( $max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '' ) {
-		$claim = $this->secondary_store->stake_claim( $max_actions, $before_date, $hooks, $group );
+	public function stake_claim($max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '') {
+		$claim = $this->secondary_store->stake_claim($max_actions, $before_date, $hooks, $group);
 
 		$claimed_actions = $claim->get_actions();
-		if ( ! empty( $claimed_actions ) ) {
-			$this->migrate( $claimed_actions );
+		if ( ! empty($claimed_actions)) {
+			$this->migrate($claimed_actions);
 		}
 
-		$this->secondary_store->release_claim( $claim );
+		$this->secondary_store->release_claim($claim);
 
-		return $this->primary_store->stake_claim( $max_actions, $before_date, $hooks, $group );
+		return $this->primary_store->stake_claim($max_actions, $before_date, $hooks, $group);
 	}
 
 	/**
@@ -226,8 +226,8 @@ 
 	 *
 	 * @param array $action_ids List of action IDs.
 	 */
-	private function migrate( $action_ids ) {
-		$this->migration_runner->migrate_actions( $action_ids );
+	private function migrate($action_ids) {
+		$this->migration_runner->migrate_actions($action_ids);
 	}
 
 	/**
@@ -238,8 +238,8 @@ 
 	 *
 	 * @return int The action ID
 	 */
-	public function save_action( ActionScheduler_Action $action, DateTime $date = null ) {
-		return $this->primary_store->save_action( $action, $date );
+	public function save_action(ActionScheduler_Action $action, DateTime $date = null) {
+		return $this->primary_store->save_action($action, $date);
 	}
 
 	/**
@@ -247,10 +247,10 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function fetch_action( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id, true );
-		if ( $store ) {
-			return $store->fetch_action( $action_id );
+	public function fetch_action($action_id) {
+		$store = $this->get_store_from_action_id($action_id, true);
+		if ($store) {
+			return $store->fetch_action($action_id);
 		} else {
 			return new ActionScheduler_NullAction();
 		}
@@ -261,10 +261,10 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function cancel_action( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id );
-		if ( $store ) {
-			$store->cancel_action( $action_id );
+	public function cancel_action($action_id) {
+		$store = $this->get_store_from_action_id($action_id);
+		if ($store) {
+			$store->cancel_action($action_id);
 		}
 	}
 
@@ -273,10 +273,10 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function delete_action( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id );
-		if ( $store ) {
-			$store->delete_action( $action_id );
+	public function delete_action($action_id) {
+		$store = $this->get_store_from_action_id($action_id);
+		if ($store) {
+			$store->delete_action($action_id);
 		}
 	}
 
@@ -285,10 +285,10 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function get_date( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id );
-		if ( $store ) {
-			return $store->get_date( $action_id );
+	public function get_date($action_id) {
+		$store = $this->get_store_from_action_id($action_id);
+		if ($store) {
+			return $store->get_date($action_id);
 		} else {
 			return null;
 		}
@@ -299,10 +299,10 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function mark_failure( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id );
-		if ( $store ) {
-			$store->mark_failure( $action_id );
+	public function mark_failure($action_id) {
+		$store = $this->get_store_from_action_id($action_id);
+		if ($store) {
+			$store->mark_failure($action_id);
 		}
 	}
 
@@ -311,10 +311,10 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function log_execution( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id );
-		if ( $store ) {
-			$store->log_execution( $action_id );
+	public function log_execution($action_id) {
+		$store = $this->get_store_from_action_id($action_id);
+		if ($store) {
+			$store->log_execution($action_id);
 		}
 	}
 
@@ -323,10 +323,10 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function mark_complete( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id );
-		if ( $store ) {
-			$store->mark_complete( $action_id );
+	public function mark_complete($action_id) {
+		$store = $this->get_store_from_action_id($action_id);
+		if ($store) {
+			$store->mark_complete($action_id);
 		}
 	}
 
@@ -335,10 +335,10 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function get_status( $action_id ) {
-		$store = $this->get_store_from_action_id( $action_id );
-		if ( $store ) {
-			return $store->get_status( $action_id );
+	public function get_status($action_id) {
+		$store = $this->get_store_from_action_id($action_id);
+		if ($store) {
+			return $store->get_status($action_id);
 		}
 		return null;
 	}
@@ -350,13 +350,13 @@ 
 	 * @param bool $primary_first Optional flag indicating search the primary store first.
 	 * @return ActionScheduler_Store
 	 */
-	protected function get_store_from_action_id( $action_id, $primary_first = false ) {
-		if ( $primary_first ) {
+	protected function get_store_from_action_id($action_id, $primary_first = false) {
+		if ($primary_first) {
 			$stores = array(
 				$this->primary_store,
 				$this->secondary_store,
 			);
-		} elseif ( $action_id < $this->demarkation_id ) {
+		} elseif ($action_id < $this->demarkation_id) {
 			$stores = array(
 				$this->secondary_store,
 				$this->primary_store,
@@ -367,9 +367,9 @@ 
 			);
 		}
 
-		foreach ( $stores as $store ) {
-			$action = $store->fetch_action( $action_id );
-			if ( ! is_a( $action, 'ActionScheduler_NullAction' ) ) {
+		foreach ($stores as $store) {
+			$action = $store->fetch_action($action_id);
+			if ( ! is_a($action, 'ActionScheduler_NullAction')) {
 				return $store;
 			}
 		}
@@ -394,8 +394,8 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function get_claim_id( $action_id ) {
-		return $this->primary_store->get_claim_id( $action_id );
+	public function get_claim_id($action_id) {
+		return $this->primary_store->get_claim_id($action_id);
 	}
 
 	/**
@@ -403,8 +403,8 @@ 
 	 *
 	 * @param ActionScheduler_ActionClaim $claim Claim object.
 	 */
-	public function release_claim( ActionScheduler_ActionClaim $claim ) {
-		$this->primary_store->release_claim( $claim );
+	public function release_claim(ActionScheduler_ActionClaim $claim) {
+		$this->primary_store->release_claim($claim);
 	}
 
 	/**
@@ -412,8 +412,8 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function unclaim_action( $action_id ) {
-		$this->primary_store->unclaim_action( $action_id );
+	public function unclaim_action($action_id) {
+		$this->primary_store->unclaim_action($action_id);
 	}
 
 	/**
@@ -421,7 +421,7 @@ 
 	 *
 	 * @param int $claim_id Claim ID.
 	 */
-	public function find_actions_by_claim_id( $claim_id ) {
-		return $this->primary_store->find_actions_by_claim_id( $claim_id );
+	public function find_actions_by_claim_id($claim_id) {
+		return $this->primary_store->find_actions_by_claim_id($claim_id);
 	}
 }

src/libraries/action-scheduler/classes/data-stores/ActionScheduler_wpPostStore_PostTypeRegistrar.php

Indentation +39 added lines, -39 removed lines

@@ -6,46 +6,46 @@
  * @codeCoverageIgnore
  */
 class ActionScheduler_wpPostStore_PostTypeRegistrar {
-	public function register() {
-		register_post_type( ActionScheduler_wpPostStore::POST_TYPE, $this->post_type_args() );
-	}
+    public function register() {
+        register_post_type( ActionScheduler_wpPostStore::POST_TYPE, $this->post_type_args() );
+    }
 
-	/**
-	 * Build the args array for the post type definition
-	 *
-	 * @return array
-	 */
-	protected function post_type_args() {
-		$args = array(
-			'label'        => __( 'Scheduled Actions', 'action-scheduler' ),
-			'description'  => __( 'Scheduled actions are hooks triggered on a cetain date and time.', 'action-scheduler' ),
-			'public'       => false,
-			'map_meta_cap' => true,
-			'hierarchical' => false,
-			'supports'     => array( 'title', 'editor', 'comments' ),
-			'rewrite'      => false,
-			'query_var'    => false,
-			'can_export'   => true,
-			'ep_mask'      => EP_NONE,
-			'labels'       => array(
-				'name'               => __( 'Scheduled Actions', 'action-scheduler' ),
-				'singular_name'      => __( 'Scheduled Action', 'action-scheduler' ),
-				'menu_name'          => _x( 'Scheduled Actions', 'Admin menu name', 'action-scheduler' ),
-				'add_new'            => __( 'Add', 'action-scheduler' ),
-				'add_new_item'       => __( 'Add New Scheduled Action', 'action-scheduler' ),
-				'edit'               => __( 'Edit', 'action-scheduler' ),
-				'edit_item'          => __( 'Edit Scheduled Action', 'action-scheduler' ),
-				'new_item'           => __( 'New Scheduled Action', 'action-scheduler' ),
-				'view'               => __( 'View Action', 'action-scheduler' ),
-				'view_item'          => __( 'View Action', 'action-scheduler' ),
-				'search_items'       => __( 'Search Scheduled Actions', 'action-scheduler' ),
-				'not_found'          => __( 'No actions found', 'action-scheduler' ),
-				'not_found_in_trash' => __( 'No actions found in trash', 'action-scheduler' ),
-			),
-		);
+    /**
+     * Build the args array for the post type definition
+     *
+     * @return array
+     */
+    protected function post_type_args() {
+        $args = array(
+            'label'        => __( 'Scheduled Actions', 'action-scheduler' ),
+            'description'  => __( 'Scheduled actions are hooks triggered on a cetain date and time.', 'action-scheduler' ),
+            'public'       => false,
+            'map_meta_cap' => true,
+            'hierarchical' => false,
+            'supports'     => array( 'title', 'editor', 'comments' ),
+            'rewrite'      => false,
+            'query_var'    => false,
+            'can_export'   => true,
+            'ep_mask'      => EP_NONE,
+            'labels'       => array(
+                'name'               => __( 'Scheduled Actions', 'action-scheduler' ),
+                'singular_name'      => __( 'Scheduled Action', 'action-scheduler' ),
+                'menu_name'          => _x( 'Scheduled Actions', 'Admin menu name', 'action-scheduler' ),
+                'add_new'            => __( 'Add', 'action-scheduler' ),
+                'add_new_item'       => __( 'Add New Scheduled Action', 'action-scheduler' ),
+                'edit'               => __( 'Edit', 'action-scheduler' ),
+                'edit_item'          => __( 'Edit Scheduled Action', 'action-scheduler' ),
+                'new_item'           => __( 'New Scheduled Action', 'action-scheduler' ),
+                'view'               => __( 'View Action', 'action-scheduler' ),
+                'view_item'          => __( 'View Action', 'action-scheduler' ),
+                'search_items'       => __( 'Search Scheduled Actions', 'action-scheduler' ),
+                'not_found'          => __( 'No actions found', 'action-scheduler' ),
+                'not_found_in_trash' => __( 'No actions found in trash', 'action-scheduler' ),
+            ),
+        );
 
-		$args = apply_filters( 'action_scheduler_post_type_args', $args );
-		return $args;
-	}
+        $args = apply_filters( 'action_scheduler_post_type_args', $args );
+        return $args;
+    }
 }
 

Spacing +18 added lines, -18 removed lines

@@ -7,7 +7,7 @@ 
  */
 class ActionScheduler_wpPostStore_PostTypeRegistrar {
 	public function register() {
-		register_post_type( ActionScheduler_wpPostStore::POST_TYPE, $this->post_type_args() );
+		register_post_type(ActionScheduler_wpPostStore::POST_TYPE, $this->post_type_args());
 	}
 
 	/**
@@ -17,34 +17,34 @@ 
 	 */
 	protected function post_type_args() {
 		$args = array(
-			'label'        => __( 'Scheduled Actions', 'action-scheduler' ),
-			'description'  => __( 'Scheduled actions are hooks triggered on a cetain date and time.', 'action-scheduler' ),
+			'label'        => __('Scheduled Actions', 'action-scheduler'),
+			'description'  => __('Scheduled actions are hooks triggered on a cetain date and time.', 'action-scheduler'),
 			'public'       => false,
 			'map_meta_cap' => true,
 			'hierarchical' => false,
-			'supports'     => array( 'title', 'editor', 'comments' ),
+			'supports'     => array('title', 'editor', 'comments'),
 			'rewrite'      => false,
 			'query_var'    => false,
 			'can_export'   => true,
 			'ep_mask'      => EP_NONE,
 			'labels'       => array(
-				'name'               => __( 'Scheduled Actions', 'action-scheduler' ),
-				'singular_name'      => __( 'Scheduled Action', 'action-scheduler' ),
-				'menu_name'          => _x( 'Scheduled Actions', 'Admin menu name', 'action-scheduler' ),
-				'add_new'            => __( 'Add', 'action-scheduler' ),
-				'add_new_item'       => __( 'Add New Scheduled Action', 'action-scheduler' ),
-				'edit'               => __( 'Edit', 'action-scheduler' ),
-				'edit_item'          => __( 'Edit Scheduled Action', 'action-scheduler' ),
-				'new_item'           => __( 'New Scheduled Action', 'action-scheduler' ),
-				'view'               => __( 'View Action', 'action-scheduler' ),
-				'view_item'          => __( 'View Action', 'action-scheduler' ),
-				'search_items'       => __( 'Search Scheduled Actions', 'action-scheduler' ),
-				'not_found'          => __( 'No actions found', 'action-scheduler' ),
-				'not_found_in_trash' => __( 'No actions found in trash', 'action-scheduler' ),
+				'name'               => __('Scheduled Actions', 'action-scheduler'),
+				'singular_name'      => __('Scheduled Action', 'action-scheduler'),
+				'menu_name'          => _x('Scheduled Actions', 'Admin menu name', 'action-scheduler'),
+				'add_new'            => __('Add', 'action-scheduler'),
+				'add_new_item'       => __('Add New Scheduled Action', 'action-scheduler'),
+				'edit'               => __('Edit', 'action-scheduler'),
+				'edit_item'          => __('Edit Scheduled Action', 'action-scheduler'),
+				'new_item'           => __('New Scheduled Action', 'action-scheduler'),
+				'view'               => __('View Action', 'action-scheduler'),
+				'view_item'          => __('View Action', 'action-scheduler'),
+				'search_items'       => __('Search Scheduled Actions', 'action-scheduler'),
+				'not_found'          => __('No actions found', 'action-scheduler'),
+				'not_found_in_trash' => __('No actions found in trash', 'action-scheduler'),
 			),
 		);
 
-		$args = apply_filters( 'action_scheduler_post_type_args', $args );
+		$args = apply_filters('action_scheduler_post_type_args', $args);
 		return $args;
 	}
 }

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

Indentation +15 added lines, -15 removed lines

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

Spacing +3 added lines, -3 removed lines

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

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

Indentation +42 added lines, -42 removed lines

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

Spacing +19 added lines, -19 removed lines

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

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

Indentation +156 added lines, -156 removed lines

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

Spacing +26 added lines, -26 removed lines

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

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

Indentation +238 added lines, -238 removed lines

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

Spacing +51 added lines, -51 removed lines

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

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

Indentation +72 added lines, -72 removed lines

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

Spacing +30 added lines, -30 removed lines

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

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

Indentation +27 added lines, -27 removed lines

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

Spacing +12 added lines, -12 removed lines

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