iqltechnologies / brighty

plugins/woocommerce/packages/action-scheduler/classes/data-stores/ActionScheduler_DBStore.php

Spacing +193 added lines, -193 removed lines

@@ -45,45 +45,45 @@ 
 	 * @return int Action ID.
 	 * @throws RuntimeException     Throws exception when saving the action fails.
 	 */
-	public function save_action( ActionScheduler_Action $action, \DateTime $date = null ) {
+	public function save_action(ActionScheduler_Action $action, \DateTime $date = null) {
 		try {
 
-			$this->validate_action( $action );
+			$this->validate_action($action);
 
 			/** @var \wpdb $wpdb */
 			global $wpdb;
 			$data = array(
 				'hook'                 => $action->get_hook(),
-				'status'               => ( $action->is_finished() ? self::STATUS_COMPLETE : self::STATUS_PENDING ),
-				'scheduled_date_gmt'   => $this->get_scheduled_date_string( $action, $date ),
-				'scheduled_date_local' => $this->get_scheduled_date_string_local( $action, $date ),
-				'schedule'             => serialize( $action->get_schedule() ), // phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.serialize_serialize
-				'group_id'             => $this->get_group_id( $action->get_group() ),
+				'status'               => ($action->is_finished() ? self::STATUS_COMPLETE : self::STATUS_PENDING),
+				'scheduled_date_gmt'   => $this->get_scheduled_date_string($action, $date),
+				'scheduled_date_local' => $this->get_scheduled_date_string_local($action, $date),
+				'schedule'             => serialize($action->get_schedule()), // phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.serialize_serialize
+				'group_id'             => $this->get_group_id($action->get_group()),
 			);
-			$args = wp_json_encode( $action->get_args() );
-			if ( strlen( $args ) <= static::$max_index_length ) {
+			$args = wp_json_encode($action->get_args());
+			if (strlen($args) <= static::$max_index_length) {
 				$data['args'] = $args;
 			} else {
-				$data['args']          = $this->hash_args( $args );
+				$data['args']          = $this->hash_args($args);
 				$data['extended_args'] = $args;
 			}
 
-			$table_name = ! empty( $wpdb->actionscheduler_actions ) ? $wpdb->actionscheduler_actions : $wpdb->prefix . 'actionscheduler_actions';
-			$wpdb->insert( $table_name, $data );
+			$table_name = !empty($wpdb->actionscheduler_actions) ? $wpdb->actionscheduler_actions : $wpdb->prefix . 'actionscheduler_actions';
+			$wpdb->insert($table_name, $data);
 			$action_id = $wpdb->insert_id;
 
-			if ( is_wp_error( $action_id ) ) {
-				throw new \RuntimeException( $action_id->get_error_message() );
-			} elseif ( empty( $action_id ) ) {
-				throw new \RuntimeException( $wpdb->last_error ? $wpdb->last_error : __( 'Database error.', 'woocommerce' ) );
+			if (is_wp_error($action_id)) {
+				throw new \RuntimeException($action_id->get_error_message());
+			} elseif (empty($action_id)) {
+				throw new \RuntimeException($wpdb->last_error ? $wpdb->last_error : __('Database error.', 'woocommerce'));
 			}
 
-			do_action( 'action_scheduler_stored_action', $action_id );
+			do_action('action_scheduler_stored_action', $action_id);
 
 			return $action_id;
-		} catch ( \Exception $e ) {
+		} catch (\Exception $e) {
 			/* translators: %s: error message */
-			throw new \RuntimeException( sprintf( __( 'Error saving action: %s', 'woocommerce' ), $e->getMessage() ), 0 );
+			throw new \RuntimeException(sprintf(__('Error saving action: %s', 'woocommerce'), $e->getMessage()), 0);
 		}
 	}
 
@@ -93,8 +93,8 @@ 
 	 * @param string $args JSON encoded action args.
 	 * @return string
 	 */
-	protected function hash_args( $args ) {
-		return md5( $args );
+	protected function hash_args($args) {
+		return md5($args);
 	}
 
 	/**
@@ -103,12 +103,12 @@ 
 	 * @param array $args Action args.
 	 * @return string
 	 */
-	protected function get_args_for_query( $args ) {
-		$encoded = wp_json_encode( $args );
-		if ( strlen( $encoded ) <= static::$max_index_length ) {
+	protected function get_args_for_query($args) {
+		$encoded = wp_json_encode($args);
+		if (strlen($encoded) <= static::$max_index_length) {
 			return $encoded;
 		}
-		return $this->hash_args( $encoded );
+		return $this->hash_args($encoded);
 	}
 	/**
 	 * Get a group's ID based on its name/slug.
@@ -118,15 +118,15 @@ 
 	 *
 	 * @return int The group's ID, if it exists or is created, or 0 if it does not exist and is not created.
 	 */
-	protected function get_group_id( $slug, $create_if_not_exists = true ) {
-		if ( empty( $slug ) ) {
+	protected function get_group_id($slug, $create_if_not_exists = true) {
+		if (empty($slug)) {
 			return 0;
 		}
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$group_id = (int) $wpdb->get_var( $wpdb->prepare( "SELECT group_id FROM {$wpdb->actionscheduler_groups} WHERE slug=%s", $slug ) );
-		if ( empty( $group_id ) && $create_if_not_exists ) {
-			$group_id = $this->create_group( $slug );
+		$group_id = (int) $wpdb->get_var($wpdb->prepare("SELECT group_id FROM {$wpdb->actionscheduler_groups} WHERE slug=%s", $slug));
+		if (empty($group_id) && $create_if_not_exists) {
+			$group_id = $this->create_group($slug);
 		}
 
 		return $group_id;
@@ -139,10 +139,10 @@ 
 	 *
 	 * @return int Group ID.
 	 */
-	protected function create_group( $slug ) {
+	protected function create_group($slug) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$wpdb->insert( $wpdb->actionscheduler_groups, array( 'slug' => $slug ) );
+		$wpdb->insert($wpdb->actionscheduler_groups, array('slug' => $slug));
 
 		return (int) $wpdb->insert_id;
 	}
@@ -154,7 +154,7 @@ 
 	 *
 	 * @return ActionScheduler_Action
 	 */
-	public function fetch_action( $action_id ) {
+	public function fetch_action($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 		$data = $wpdb->get_row(
@@ -164,13 +164,13 @@ 
 			)
 		);
 
-		if ( empty( $data ) ) {
+		if (empty($data)) {
 			return $this->get_null_action();
 		}
 
-		if ( ! empty( $data->extended_args ) ) {
+		if (!empty($data->extended_args)) {
 			$data->args = $data->extended_args;
-			unset( $data->extended_args );
+			unset($data->extended_args);
 		}
 
 		// Convert NULL dates to zero dates.
@@ -180,16 +180,16 @@ 
 			'last_attempt_gmt',
 			'last_attempt_gmt',
 		);
-		foreach ( $date_fields as $date_field ) {
-			if ( is_null( $data->$date_field ) ) {
+		foreach ($date_fields as $date_field) {
+			if (is_null($data->$date_field)) {
 				$data->$date_field = ActionScheduler_StoreSchema::DEFAULT_DATE;
 			}
 		}
 
 		try {
-			$action = $this->make_action_from_db_record( $data );
-		} catch ( ActionScheduler_InvalidActionException $exception ) {
-			do_action( 'action_scheduler_failed_fetch_action', $action_id, $exception );
+			$action = $this->make_action_from_db_record($data);
+		} catch (ActionScheduler_InvalidActionException $exception) {
+			do_action('action_scheduler_failed_fetch_action', $action_id, $exception);
 			return $this->get_null_action();
 		}
 
@@ -212,21 +212,21 @@ 
 	 *
 	 * @return ActionScheduler_Action|ActionScheduler_CanceledAction|ActionScheduler_FinishedAction
 	 */
-	protected function make_action_from_db_record( $data ) {
+	protected function make_action_from_db_record($data) {
 
 		$hook     = $data->hook;
-		$args     = json_decode( $data->args, true );
-		$schedule = unserialize( $data->schedule ); // phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.serialize_unserialize
+		$args     = json_decode($data->args, true);
+		$schedule = unserialize($data->schedule); // phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.serialize_unserialize
 
-		$this->validate_args( $args, $data->action_id );
-		$this->validate_schedule( $schedule, $data->action_id );
+		$this->validate_args($args, $data->action_id);
+		$this->validate_schedule($schedule, $data->action_id);
 
-		if ( empty( $schedule ) ) {
+		if (empty($schedule)) {
 			$schedule = new ActionScheduler_NullSchedule();
 		}
 		$group = $data->group ? $data->group : '';
 
-		return ActionScheduler::factory()->get_stored_action( $data->status, $data->hook, $args, $schedule, $group );
+		return ActionScheduler::factory()->get_stored_action($data->status, $data->hook, $args, $schedule, $group);
 	}
 
 	/**
@@ -240,10 +240,10 @@ 
 	 * @return string SQL statement already properly escaped.
 	 * @throws InvalidArgumentException If the query is invalid.
 	 */
-	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 value for select or count parameter. Cannot query actions.', 'woocommerce' ) );
+		if (!in_array($select_or_count, array('select', 'count'), true)) {
+			throw new InvalidArgumentException(__('Invalid value for select or count parameter. Cannot query actions.', 'woocommerce'));
 		}
 
 		$query = wp_parse_args(
@@ -267,72 +267,72 @@ 
 
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$sql        = ( 'count' === $select_or_count ) ? 'SELECT count(a.action_id)' : 'SELECT a.action_id';
+		$sql        = ('count' === $select_or_count) ? 'SELECT count(a.action_id)' : 'SELECT a.action_id';
 		$sql       .= " FROM {$wpdb->actionscheduler_actions} a";
 		$sql_params = array();
 
-		if ( ! empty( $query['group'] ) || 'group' === $query['orderby'] ) {
+		if (!empty($query['group']) || 'group' === $query['orderby']) {
 			$sql .= " LEFT JOIN {$wpdb->actionscheduler_groups} g ON g.group_id=a.group_id";
 		}
 
 		$sql .= ' WHERE 1=1';
 
-		if ( ! empty( $query['group'] ) ) {
+		if (!empty($query['group'])) {
 			$sql         .= ' AND g.slug=%s';
 			$sql_params[] = $query['group'];
 		}
 
-		if ( $query['hook'] ) {
+		if ($query['hook']) {
 			$sql         .= ' AND a.hook=%s';
 			$sql_params[] = $query['hook'];
 		}
-		if ( ! is_null( $query['args'] ) ) {
+		if (!is_null($query['args'])) {
 			$sql         .= ' AND a.args=%s';
-			$sql_params[] = $this->get_args_for_query( $query['args'] );
+			$sql_params[] = $this->get_args_for_query($query['args']);
 		}
 
-		if ( $query['status'] ) {
+		if ($query['status']) {
 			$statuses     = (array) $query['status'];
-			$placeholders = array_fill( 0, count( $statuses ), '%s' );
-			$sql         .= ' AND a.status IN (' . join( ', ', $placeholders ) . ')';
-			$sql_params   = array_merge( $sql_params, array_values( $statuses ) );
+			$placeholders = array_fill(0, count($statuses), '%s');
+			$sql         .= ' AND a.status IN (' . join(', ', $placeholders) . ')';
+			$sql_params   = array_merge($sql_params, array_values($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 a.scheduled_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 a.last_attempt_gmt $comparator %s";
 			$sql_params[] = $date_string;
 		}
 
-		if ( true === $query['claimed'] ) {
+		if (true === $query['claimed']) {
 			$sql .= ' AND a.claim_id != 0';
-		} elseif ( false === $query['claimed'] ) {
+		} elseif (false === $query['claimed']) {
 			$sql .= ' AND a.claim_id = 0';
-		} elseif ( ! is_null( $query['claimed'] ) ) {
+		} elseif (!is_null($query['claimed'])) {
 			$sql         .= ' AND a.claim_id = %d';
 			$sql_params[] = $query['claimed'];
 		}
 
-		if ( ! empty( $query['search'] ) ) {
+		if (!empty($query['search'])) {
 			$sql .= ' AND (a.hook LIKE %s OR (a.extended_args IS NULL AND a.args LIKE %s) OR a.extended_args 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']);
 			}
 
 			$search_claim_id = (int) $query['search'];
-			if ( $search_claim_id ) {
+			if ($search_claim_id) {
 				$sql         .= ' OR a.claim_id = %d';
 				$sql_params[] = $search_claim_id;
 			}
@@ -340,13 +340,13 @@ 
 			$sql .= ')';
 		}
 
-		if ( 'select' === $select_or_count ) {
-			if ( 'ASC' === strtoupper( $query['order'] ) ) {
+		if ('select' === $select_or_count) {
+			if ('ASC' === strtoupper($query['order'])) {
 				$order = 'ASC';
 			} else {
 				$order = 'DESC';
 			}
-			switch ( $query['orderby'] ) {
+			switch ($query['orderby']) {
 				case 'hook':
 					$sql .= " ORDER BY a.hook $order";
 					break;
@@ -367,15 +367,15 @@ 
 					break;
 			}
 
-			if ( $query['per_page'] > 0 ) {
+			if ($query['per_page'] > 0) {
 				$sql         .= ' LIMIT %d, %d';
 				$sql_params[] = $query['offset'];
 				$sql_params[] = $query['per_page'];
 			}
 		}
 
-		if ( ! empty( $sql_params ) ) {
-			$sql = $wpdb->prepare( $sql, $sql_params ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		if (!empty($sql_params)) {
+			$sql = $wpdb->prepare($sql, $sql_params); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
 		}
 
 		return $sql;
@@ -393,13 +393,13 @@ 
 	 *
 	 * @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') {
 		/** @var wpdb $wpdb */
 		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.PreparedSQL.NotPrepared, WordPress.DB.DirectDatabaseQuery.NoSql, WordPress.DB.DirectDatabaseQuery.NoCaching
+		return ('count' === $query_type) ? $wpdb->get_var($sql) : $wpdb->get_col($sql); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared, WordPress.DB.DirectDatabaseQuery.NoSql, WordPress.DB.DirectDatabaseQuery.NoCaching
 	}
 
 	/**
@@ -417,10 +417,10 @@ 
 		$actions_count_by_status = array();
 		$action_stati_and_labels = $this->get_status_labels();
 
-		foreach ( $wpdb->get_results( $sql ) as $action_data ) { // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		foreach ($wpdb->get_results($sql) as $action_data) { // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
 			// Ignore any actions with invalid status.
-			if ( array_key_exists( $action_data->status, $action_stati_and_labels ) ) {
-				$actions_count_by_status[ $action_data->status ] = $action_data->count;
+			if (array_key_exists($action_data->status, $action_stati_and_labels)) {
+				$actions_count_by_status[$action_data->status] = $action_data->count;
 			}
 		}
 
@@ -435,22 +435,22 @@ 
 	 * @return void
 	 * @throws \InvalidArgumentException If the action update failed.
 	 */
-	public function cancel_action( $action_id ) {
+	public function cancel_action($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 
 		$updated = $wpdb->update(
 			$wpdb->actionscheduler_actions,
-			array( 'status' => self::STATUS_CANCELED ),
-			array( 'action_id' => $action_id ),
-			array( '%s' ),
-			array( '%d' )
+			array('status' => self::STATUS_CANCELED),
+			array('action_id' => $action_id),
+			array('%s'),
+			array('%d')
 		);
-		if ( false === $updated ) {
+		if (false === $updated) {
 			/* translators: %s: action ID */
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'woocommerce' ), $action_id ) );
+			throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'woocommerce'), $action_id));
 		}
-		do_action( 'action_scheduler_canceled_action', $action_id );
+		do_action('action_scheduler_canceled_action', $action_id);
 	}
 
 	/**
@@ -462,8 +462,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function cancel_actions_by_hook( $hook ) {
-		$this->bulk_cancel_actions( array( 'hook' => $hook ) );
+	public function cancel_actions_by_hook($hook) {
+		$this->bulk_cancel_actions(array('hook' => $hook));
 	}
 
 	/**
@@ -473,8 +473,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function cancel_actions_by_group( $group ) {
-		$this->bulk_cancel_actions( array( 'group' => $group ) );
+	public function cancel_actions_by_group($group) {
+		$this->bulk_cancel_actions(array('group' => $group));
 	}
 
 	/**
@@ -484,16 +484,16 @@ 
 	 *
 	 * @param array $query_args Query parameters.
 	 */
-	protected function bulk_cancel_actions( $query_args ) {
+	protected function bulk_cancel_actions($query_args) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 
-		if ( ! is_array( $query_args ) ) {
+		if (!is_array($query_args)) {
 			return;
 		}
 
 		// Don't cancel actions that are already canceled.
-		if ( isset( $query_args['status'] ) && self::STATUS_CANCELED === $query_args['status'] ) {
+		if (isset($query_args['status']) && self::STATUS_CANCELED === $query_args['status']) {
 			return;
 		}
 
@@ -507,16 +507,16 @@ 
 			)
 		);
 
-		while ( $action_ids ) {
-			$action_ids = $this->query_actions( $query_args );
-			if ( empty( $action_ids ) ) {
+		while ($action_ids) {
+			$action_ids = $this->query_actions($query_args);
+			if (empty($action_ids)) {
 				break;
 			}
 
-			$format     = array_fill( 0, count( $action_ids ), '%d' );
-			$query_in   = '(' . implode( ',', $format ) . ')';
+			$format     = array_fill(0, count($action_ids), '%d');
+			$query_in   = '(' . implode(',', $format) . ')';
 			$parameters = $action_ids;
-			array_unshift( $parameters, self::STATUS_CANCELED );
+			array_unshift($parameters, self::STATUS_CANCELED);
 
 			$wpdb->query(
 				$wpdb->prepare(
@@ -525,7 +525,7 @@ 
 				)
 			);
 
-			do_action( 'action_scheduler_bulk_cancel_actions', $action_ids );
+			do_action('action_scheduler_bulk_cancel_actions', $action_ids);
 		}
 	}
 
@@ -535,14 +535,14 @@ 
 	 * @param int $action_id Action ID.
 	 * @throws \InvalidArgumentException If the action deletion failed.
 	 */
-	public function delete_action( $action_id ) {
+	public function delete_action($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$deleted = $wpdb->delete( $wpdb->actionscheduler_actions, array( 'action_id' => $action_id ), array( '%d' ) );
-		if ( empty( $deleted ) ) {
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'woocommerce' ), $action_id ) ); //phpcs:ignore WordPress.WP.I18n.MissingTranslatorsComment
+		$deleted = $wpdb->delete($wpdb->actionscheduler_actions, array('action_id' => $action_id), array('%d'));
+		if (empty($deleted)) {
+			throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'woocommerce'), $action_id)); //phpcs:ignore WordPress.WP.I18n.MissingTranslatorsComment
 		}
-		do_action( 'action_scheduler_deleted_action', $action_id );
+		do_action('action_scheduler_deleted_action', $action_id);
 	}
 
 	/**
@@ -552,9 +552,9 @@ 
 	 *
 	 * @return \DateTime The local date the action is scheduled to run, or the date that it ran.
 	 */
-	public function get_date( $action_id ) {
-		$date = $this->get_date_gmt( $action_id );
-		ActionScheduler_TimezoneHelper::set_local_timezone( $date );
+	public function get_date($action_id) {
+		$date = $this->get_date_gmt($action_id);
+		ActionScheduler_TimezoneHelper::set_local_timezone($date);
 		return $date;
 	}
 
@@ -566,17 +566,17 @@ 
 	 * @throws \InvalidArgumentException If action cannot be identified.
 	 * @return \DateTime The GMT date the action is scheduled to run, or the date that it ran.
 	 */
-	protected function get_date_gmt( $action_id ) {
+	protected function get_date_gmt($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$record = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d", $action_id ) );
-		if ( empty( $record ) ) {
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'woocommerce' ), $action_id ) ); //phpcs:ignore WordPress.WP.I18n.MissingTranslatorsComment
+		$record = $wpdb->get_row($wpdb->prepare("SELECT * FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d", $action_id));
+		if (empty($record)) {
+			throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'woocommerce'), $action_id)); //phpcs:ignore WordPress.WP.I18n.MissingTranslatorsComment
 		}
-		if ( self::STATUS_PENDING === $record->status ) {
-			return as_get_datetime_object( $record->scheduled_date_gmt );
+		if (self::STATUS_PENDING === $record->status) {
+			return as_get_datetime_object($record->scheduled_date_gmt);
 		} else {
-			return as_get_datetime_object( $record->last_attempt_gmt );
+			return as_get_datetime_object($record->last_attempt_gmt);
 		}
 	}
 
@@ -590,15 +590,15 @@ 
 	 *
 	 * @return ActionScheduler_ActionClaim
 	 */
-	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 = '') {
 		$claim_id = $this->generate_claim_id();
 
 		$this->claim_before_date = $before_date;
-		$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);
 	}
 
 	/**
@@ -610,7 +610,7 @@ 
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 		$now = as_get_datetime_object();
-		$wpdb->insert( $wpdb->actionscheduler_claims, array( 'date_created_gmt' => $now->format( 'Y-m-d H:i:s' ) ) );
+		$wpdb->insert($wpdb->actionscheduler_claims, array('date_created_gmt' => $now->format('Y-m-d H:i:s')));
 
 		return $wpdb->insert_id;
 	}
@@ -628,39 +628,39 @@ 
 	 * @throws \InvalidArgumentException Throws InvalidArgumentException if group doesn't exist.
 	 * @throws \RuntimeException Throws RuntimeException if unable to claim action.
 	 */
-	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 = '') {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 
 		$now  = as_get_datetime_object();
-		$date = is_null( $before_date ) ? $now : clone $before_date;
+		$date = is_null($before_date) ? $now : clone $before_date;
 
 		// can't use $wpdb->update() because of the <= condition.
 		$update = "UPDATE {$wpdb->actionscheduler_actions} SET claim_id=%d, last_attempt_gmt=%s, last_attempt_local=%s";
 		$params = array(
 			$claim_id,
-			$now->format( 'Y-m-d H:i:s' ),
-			current_time( 'mysql' ),
+			$now->format('Y-m-d H:i:s'),
+			current_time('mysql'),
 		);
 
 		$where    = 'WHERE claim_id = 0 AND scheduled_date_gmt <= %s AND status=%s';
-		$params[] = $date->format( 'Y-m-d H:i:s' );
+		$params[] = $date->format('Y-m-d H:i:s');
 		$params[] = self::STATUS_PENDING;
 
-		if ( ! empty( $hooks ) ) {
-			$placeholders = array_fill( 0, count( $hooks ), '%s' );
-			$where       .= ' AND hook IN (' . join( ', ', $placeholders ) . ')';
-			$params       = array_merge( $params, array_values( $hooks ) );
+		if (!empty($hooks)) {
+			$placeholders = array_fill(0, count($hooks), '%s');
+			$where       .= ' AND hook IN (' . join(', ', $placeholders) . ')';
+			$params       = array_merge($params, array_values($hooks));
 		}
 
-		if ( ! empty( $group ) ) {
+		if (!empty($group)) {
 
-			$group_id = $this->get_group_id( $group, false );
+			$group_id = $this->get_group_id($group, false);
 
 			// throw exception if no matching group found, this matches ActionScheduler_wpPostStore's behaviour.
-			if ( empty( $group_id ) ) {
+			if (empty($group_id)) {
 				/* translators: %s: group name */
-				throw new InvalidArgumentException( sprintf( __( 'The group "%s" does not exist.', 'woocommerce' ), $group ) );
+				throw new InvalidArgumentException(sprintf(__('The group "%s" does not exist.', 'woocommerce'), $group));
 			}
 
 			$where   .= ' AND group_id = %d';
@@ -674,13 +674,13 @@ 
 		 *
 		 * @param string $order_by_sql
 		 */
-		$order    = apply_filters( 'action_scheduler_claim_actions_order_by', 'ORDER BY attempts ASC, scheduled_date_gmt ASC, action_id ASC' );
+		$order    = apply_filters('action_scheduler_claim_actions_order_by', 'ORDER BY attempts ASC, scheduled_date_gmt ASC, action_id ASC');
 		$params[] = $limit;
 
-		$sql           = $wpdb->prepare( "{$update} {$where} {$order} LIMIT %d", $params ); // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared, WordPress.DB.PreparedSQLPlaceholders
-		$rows_affected = $wpdb->query( $sql ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared, WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
-		if ( false === $rows_affected ) {
-			throw new \RuntimeException( __( 'Unable to claim actions. Database error.', 'woocommerce' ) );
+		$sql           = $wpdb->prepare("{$update} {$where} {$order} LIMIT %d", $params); // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared, WordPress.DB.PreparedSQLPlaceholders
+		$rows_affected = $wpdb->query($sql); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared, WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.NoCaching
+		if (false === $rows_affected) {
+			throw new \RuntimeException(__('Unable to claim actions. Database error.', 'woocommerce'));
 		}
 
 		return (int) $rows_affected;
@@ -695,9 +695,9 @@ 
 		global $wpdb;
 
 		$sql = "SELECT COUNT(DISTINCT claim_id) FROM {$wpdb->actionscheduler_actions} WHERE claim_id != 0 AND status IN ( %s, %s)";
-		$sql = $wpdb->prepare( $sql, array( self::STATUS_PENDING, self::STATUS_RUNNING ) ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		$sql = $wpdb->prepare($sql, array(self::STATUS_PENDING, self::STATUS_RUNNING)); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
 
-		return (int) $wpdb->get_var( $sql ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		return (int) $wpdb->get_var($sql); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
 	}
 
 	/**
@@ -706,14 +706,14 @@ 
 	 * @param string $action_id Action ID.
 	 * @return mixed
 	 */
-	public function get_claim_id( $action_id ) {
+	public function get_claim_id($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 
 		$sql = "SELECT claim_id FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d";
-		$sql = $wpdb->prepare( $sql, $action_id ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		$sql = $wpdb->prepare($sql, $action_id); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
 
-		return (int) $wpdb->get_var( $sql ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		return (int) $wpdb->get_var($sql); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
 	}
 
 	/**
@@ -722,13 +722,13 @@ 
 	 * @param  int $claim_id Claim ID.
 	 * @return int[]
 	 */
-	public function find_actions_by_claim_id( $claim_id ) {
+	public function find_actions_by_claim_id($claim_id) {
 		/** @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' );
+		$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');
 
 		$sql = $wpdb->prepare(
 			"SELECT action_id, scheduled_date_gmt FROM {$wpdb->actionscheduler_actions} WHERE claim_id = %d",
@@ -737,9 +737,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 ( $wpdb->get_results( $sql ) as $claimed_action ) { // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
-			if ( $claimed_action->scheduled_date_gmt <= $cut_off ) {
-				$action_ids[] = absint( $claimed_action->action_id );
+		foreach ($wpdb->get_results($sql) as $claimed_action) { // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+			if ($claimed_action->scheduled_date_gmt <= $cut_off) {
+				$action_ids[] = absint($claimed_action->action_id);
 			}
 		}
 
@@ -751,11 +751,11 @@ 
 	 *
 	 * @param ActionScheduler_ActionClaim $claim Claim object.
 	 */
-	public function release_claim( ActionScheduler_ActionClaim $claim ) {
+	public function release_claim(ActionScheduler_ActionClaim $claim) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$wpdb->update( $wpdb->actionscheduler_actions, array( 'claim_id' => 0 ), array( 'claim_id' => $claim->get_id() ), array( '%d' ), array( '%d' ) );
-		$wpdb->delete( $wpdb->actionscheduler_claims, array( 'claim_id' => $claim->get_id() ), array( '%d' ) );
+		$wpdb->update($wpdb->actionscheduler_actions, array('claim_id' => 0), array('claim_id' => $claim->get_id()), array('%d'), array('%d'));
+		$wpdb->delete($wpdb->actionscheduler_claims, array('claim_id' => $claim->get_id()), array('%d'));
 	}
 
 	/**
@@ -765,15 +765,15 @@ 
 	 *
 	 * @return void
 	 */
-	public function unclaim_action( $action_id ) {
+	public function unclaim_action($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 		$wpdb->update(
 			$wpdb->actionscheduler_actions,
-			array( 'claim_id' => 0 ),
-			array( 'action_id' => $action_id ),
-			array( '%s' ),
-			array( '%d' )
+			array('claim_id' => 0),
+			array('action_id' => $action_id),
+			array('%s'),
+			array('%d')
 		);
 	}
 
@@ -783,18 +783,18 @@ 
 	 * @param int $action_id Action ID.
 	 * @throws \InvalidArgumentException Throw an exception if action was not updated.
 	 */
-	public function mark_failure( $action_id ) {
+	public function mark_failure($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 		$updated = $wpdb->update(
 			$wpdb->actionscheduler_actions,
-			array( 'status' => self::STATUS_FAILED ),
-			array( 'action_id' => $action_id ),
-			array( '%s' ),
-			array( '%d' )
+			array('status' => self::STATUS_FAILED),
+			array('action_id' => $action_id),
+			array('%s'),
+			array('%d')
 		);
-		if ( empty( $updated ) ) {
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'woocommerce' ), $action_id ) ); //phpcs:ignore WordPress.WP.I18n.MissingTranslatorsComment
+		if (empty($updated)) {
+			throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'woocommerce'), $action_id)); //phpcs:ignore WordPress.WP.I18n.MissingTranslatorsComment
 		}
 	}
 
@@ -805,13 +805,13 @@ 
 	 *
 	 * @return void
 	 */
-	public function log_execution( $action_id ) {
+	public function log_execution($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 
 		$sql = "UPDATE {$wpdb->actionscheduler_actions} SET attempts = attempts+1, status=%s, last_attempt_gmt = %s, last_attempt_local = %s WHERE action_id = %d";
-		$sql = $wpdb->prepare( $sql, self::STATUS_RUNNING, current_time( 'mysql', true ), current_time( 'mysql' ), $action_id ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
-		$wpdb->query( $sql ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		$sql = $wpdb->prepare($sql, self::STATUS_RUNNING, current_time('mysql', true), current_time('mysql'), $action_id); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		$wpdb->query($sql); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
 	}
 
 	/**
@@ -822,22 +822,22 @@ 
 	 * @return void
 	 * @throws \InvalidArgumentException Throw an exception if action was not updated.
 	 */
-	public function mark_complete( $action_id ) {
+	public function mark_complete($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 		$updated = $wpdb->update(
 			$wpdb->actionscheduler_actions,
 			array(
 				'status'             => self::STATUS_COMPLETE,
-				'last_attempt_gmt'   => current_time( 'mysql', true ),
-				'last_attempt_local' => current_time( 'mysql' ),
+				'last_attempt_gmt'   => current_time('mysql', true),
+				'last_attempt_local' => current_time('mysql'),
 			),
-			array( 'action_id' => $action_id ),
-			array( '%s' ),
-			array( '%d' )
+			array('action_id' => $action_id),
+			array('%s'),
+			array('%d')
 		);
-		if ( empty( $updated ) ) {
-			throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'woocommerce' ), $action_id ) ); //phpcs:ignore WordPress.WP.I18n.MissingTranslatorsComment
+		if (empty($updated)) {
+			throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'woocommerce'), $action_id)); //phpcs:ignore WordPress.WP.I18n.MissingTranslatorsComment
 		}
 
 		/**
@@ -847,7 +847,7 @@ 
 		 *
 		 * @param int $action_id Action ID.
 		 */
-		do_action( 'action_scheduler_completed_action', $action_id );
+		do_action('action_scheduler_completed_action', $action_id);
 	}
 
 	/**
@@ -859,17 +859,17 @@ 
 	 * @throws \InvalidArgumentException Throw an exception if not status was found for action_id.
 	 * @throws \RuntimeException Throw an exception if action status could not be retrieved.
 	 */
-	public function get_status( $action_id ) {
+	public function get_status($action_id) {
 		/** @var \wpdb $wpdb */
 		global $wpdb;
 		$sql    = "SELECT status FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d";
-		$sql    = $wpdb->prepare( $sql, $action_id ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
-		$status = $wpdb->get_var( $sql ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		$sql    = $wpdb->prepare($sql, $action_id); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		$status = $wpdb->get_var($sql); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
 
-		if ( null === $status ) {
-			throw new \InvalidArgumentException( __( 'Invalid action ID. No status found.', 'woocommerce' ) );
-		} elseif ( empty( $status ) ) {
-			throw new \RuntimeException( __( 'Unknown status found for action.', 'woocommerce' ) );
+		if (null === $status) {
+			throw new \InvalidArgumentException(__('Invalid action ID. No status found.', 'woocommerce'));
+		} elseif (empty($status)) {
+			throw new \RuntimeException(__('Unknown status found for action.', 'woocommerce'));
 		} else {
 			return $status;
 		}

plugins/woocommerce/packages/action-scheduler/classes/data-stores/ActionScheduler_DBLogger.php

Spacing +32 added lines, -32 removed lines

@@ -18,16 +18,16 @@ 
 	 *
 	 * @return int     The log entry ID.
 	 */
-	public function log( $action_id, $message, DateTime $date = null ) {
-		if ( empty( $date ) ) {
+	public function log($action_id, $message, DateTime $date = null) {
+		if (empty($date)) {
 			$date = as_get_datetime_object();
 		} else {
 			$date = clone $date;
 		}
 
-		$date_gmt = $date->format( 'Y-m-d H:i:s' );
-		ActionScheduler_TimezoneHelper::set_local_timezone( $date );
-		$date_local = $date->format( 'Y-m-d H:i:s' );
+		$date_gmt = $date->format('Y-m-d H:i:s');
+		ActionScheduler_TimezoneHelper::set_local_timezone($date);
+		$date_local = $date->format('Y-m-d H:i:s');
 
 		/** @var \wpdb $wpdb */ //phpcs:ignore Generic.Commenting.DocComment.MissingShort
 		global $wpdb;
@@ -39,7 +39,7 @@ 
 				'log_date_gmt'   => $date_gmt,
 				'log_date_local' => $date_local,
 			),
-			array( '%d', '%s', '%s', '%s' )
+			array('%d', '%s', '%s', '%s')
 		);
 
 		return $wpdb->insert_id;
@@ -52,12 +52,12 @@ 
 	 *
 	 * @return ActionScheduler_LogEntry
 	 */
-	public function get_entry( $entry_id ) {
+	public function get_entry($entry_id) {
 		/** @var \wpdb $wpdb */ //phpcs:ignore Generic.Commenting.DocComment.MissingShort
 		global $wpdb;
-		$entry = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_logs} WHERE log_id=%d", $entry_id ) );
+		$entry = $wpdb->get_row($wpdb->prepare("SELECT * FROM {$wpdb->actionscheduler_logs} WHERE log_id=%d", $entry_id));
 
-		return $this->create_entry_from_db_record( $entry );
+		return $this->create_entry_from_db_record($entry);
 	}
 
 	/**
@@ -67,18 +67,18 @@ 
 	 *
 	 * @return ActionScheduler_LogEntry
 	 */
-	private function create_entry_from_db_record( $record ) {
-		if ( empty( $record ) ) {
+	private function create_entry_from_db_record($record) {
+		if (empty($record)) {
 			return new ActionScheduler_NullLogEntry();
 		}
 
-		if ( is_null( $record->log_date_gmt ) ) {
-			$date = as_get_datetime_object( ActionScheduler_StoreSchema::DEFAULT_DATE );
+		if (is_null($record->log_date_gmt)) {
+			$date = as_get_datetime_object(ActionScheduler_StoreSchema::DEFAULT_DATE);
 		} else {
-			$date = as_get_datetime_object( $record->log_date_gmt );
+			$date = as_get_datetime_object($record->log_date_gmt);
 		}
 
-		return new ActionScheduler_LogEntry( $record->action_id, $record->message, $date );
+		return new ActionScheduler_LogEntry($record->action_id, $record->message, $date);
 	}
 
 	/**
@@ -88,13 +88,13 @@ 
 	 *
 	 * @return ActionScheduler_LogEntry[]
 	 */
-	public function get_logs( $action_id ) {
+	public function get_logs($action_id) {
 		/** @var \wpdb $wpdb */ //phpcs:ignore Generic.Commenting.DocComment.MissingShort
 		global $wpdb;
 
-		$records = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_logs} WHERE action_id=%d", $action_id ) );
+		$records = $wpdb->get_results($wpdb->prepare("SELECT * FROM {$wpdb->actionscheduler_logs} WHERE action_id=%d", $action_id));
 
-		return array_map( array( $this, 'create_entry_from_db_record' ), $records );
+		return array_map(array($this, 'create_entry_from_db_record'), $records);
 	}
 
 	/**
@@ -109,7 +109,7 @@ 
 
 		parent::init();
 
-		add_action( 'action_scheduler_deleted_action', array( $this, 'clear_deleted_action_logs' ), 10, 1 );
+		add_action('action_scheduler_deleted_action', array($this, 'clear_deleted_action_logs'), 10, 1);
 	}
 
 	/**
@@ -117,10 +117,10 @@ 
 	 *
 	 * @param int $action_id Action ID.
 	 */
-	public function clear_deleted_action_logs( $action_id ) {
+	public function clear_deleted_action_logs($action_id) {
 		/** @var \wpdb $wpdb */ //phpcs:ignore Generic.Commenting.DocComment.MissingShort
 		global $wpdb;
-		$wpdb->delete( $wpdb->actionscheduler_logs, array( 'action_id' => $action_id ), array( '%d' ) );
+		$wpdb->delete($wpdb->actionscheduler_logs, array('action_id' => $action_id), array('%d'));
 	}
 
 	/**
@@ -128,27 +128,27 @@ 
 	 *
 	 * @param array $action_ids List of action ID.
 	 */
-	public function bulk_log_cancel_actions( $action_ids ) {
-		if ( empty( $action_ids ) ) {
+	public function bulk_log_cancel_actions($action_ids) {
+		if (empty($action_ids)) {
 			return;
 		}
 
 		/** @var \wpdb $wpdb */ //phpcs:ignore Generic.Commenting.DocComment.MissingShort
 		global $wpdb;
 		$date     = as_get_datetime_object();
-		$date_gmt = $date->format( 'Y-m-d H:i:s' );
-		ActionScheduler_TimezoneHelper::set_local_timezone( $date );
-		$date_local = $date->format( 'Y-m-d H:i:s' );
-		$message    = __( 'action canceled', 'woocommerce' );
-		$format     = '(%d, ' . $wpdb->prepare( '%s, %s, %s', $message, $date_gmt, $date_local ) . ')';
+		$date_gmt = $date->format('Y-m-d H:i:s');
+		ActionScheduler_TimezoneHelper::set_local_timezone($date);
+		$date_local = $date->format('Y-m-d H:i:s');
+		$message    = __('action canceled', 'woocommerce');
+		$format     = '(%d, ' . $wpdb->prepare('%s, %s, %s', $message, $date_gmt, $date_local) . ')';
 		$sql_query  = "INSERT {$wpdb->actionscheduler_logs} (action_id, message, log_date_gmt, log_date_local) VALUES ";
 		$value_rows = array();
 
-		foreach ( $action_ids as $action_id ) {
-			$value_rows[] = $wpdb->prepare( $format, $action_id ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		foreach ($action_ids as $action_id) {
+			$value_rows[] = $wpdb->prepare($format, $action_id); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
 		}
-		$sql_query .= implode( ',', $value_rows );
+		$sql_query .= implode(',', $value_rows);
 
-		$wpdb->query( $sql_query ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+		$wpdb->query($sql_query); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
 	}
 }

plugins/woocommerce/packages/action-scheduler/classes/data-stores/ActionScheduler_wpPostStore_TaxonomyRegistrar.php

Spacing +3 added lines, -3 removed lines

@@ -6,12 +6,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', 'woocommerce' ),
+			'label' => __('Action Group', 'woocommerce'),
 			'public' => false,
 			'hierarchical' => false,
 			'show_admin_column' => true,
@@ -19,7 +19,7 @@ 
 			'rewrite' => false,
 		);
 
-		$args = apply_filters( 'action_scheduler_taxonomy_args', $args );
+		$args = apply_filters('action_scheduler_taxonomy_args', $args);
 		return $args;
 	}
 }

plugins/woocommerce/packages/action-scheduler/classes/data-stores/ActionScheduler_wpPostStore.php

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', 'woocommerce' ), $e->getMessage() ), 0 );
+			throw new RuntimeException(sprintf(__('Error saving action: %s', 'woocommerce'), $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.', 'woocommerce' ) );
+		if (is_wp_error($post_id) || empty($post_id)) {
+			throw new RuntimeException(__('Unable to save action.', 'woocommerce'));
 		}
 		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.', 'woocommerce' ) );
+		if (!in_array($select_or_count, array('select', 'count'), true)) {
+			throw new InvalidArgumentException(__('Invalid schedule. Cannot save action.', 'woocommerce'));
 		}
 
 		$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 ( ! is_null( $query['args'] ) ) {
+		if (!is_null($query['args'])) {
 			$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 ( ! is_null( $query['claimed'] ) ) {
+		} elseif (!is_null($query['claimed'])) {
 			$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', 'woocommerce' ), $action_id ) );
+			throw new InvalidArgumentException(sprintf(__('Unidentified action %s', 'woocommerce'), $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', 'woocommerce' ), $action_id ) );
+			throw new InvalidArgumentException(sprintf(__('Unidentified action %s', 'woocommerce'), $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', 'woocommerce' ), $action_id ) );
+			throw new InvalidArgumentException(sprintf(__('Unidentified action %s', 'woocommerce'), $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.', 'woocommerce' ) );
+		if (false === $rows_affected) {
+			throw new RuntimeException(__('Unable to claim actions. Database error.', 'woocommerce'));
 		}
 
 		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.', 'woocommerce' ), $group ) );
+			throw new InvalidArgumentException(sprintf(__('The group "%s" does not exist.', 'woocommerce'), $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.', 'woocommerce' ), $claim->get_id() ) );
+			throw new RuntimeException(sprintf(__('Unable to unlock claim %s. Database error.', 'woocommerce'), $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.', 'woocommerce' ), $action_id ) );
+			throw new RuntimeException(sprintf(__('Unable to unlock claim on action %s. Database error.', 'woocommerce'), $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.', 'woocommerce' ), $action_id ) );
+			throw new RuntimeException(sprintf(__('Unable to mark failure on action %s. Database error.', 'woocommerce'), $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.', 'woocommerce' ) );
+		if (null === $status) {
+			throw new InvalidArgumentException(__('Invalid action ID. No status found.', 'woocommerce'));
 		}
 
-		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', 'woocommerce' ), $action_id ) );
+			throw new InvalidArgumentException(sprintf(__('Unidentified action %s', 'woocommerce'), $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.', 'woocommerce' ), $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.', 'woocommerce'), $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();

plugins/woocommerce/packages/action-scheduler/classes/data-stores/ActionScheduler_wpPostStore_PostTypeRegistrar.php

Spacing +17 added lines, -17 removed lines

@@ -6,7 +6,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());
 	}
 
 	/**
@@ -16,30 +16,30 @@ 
 	 */
 	protected function post_type_args() {
 		$args = array(
-			'label' => __( 'Scheduled Actions', 'woocommerce' ),
-			'description' => __( 'Scheduled actions are hooks triggered on a cetain date and time.', 'woocommerce' ),
+			'label' => __('Scheduled Actions', 'woocommerce'),
+			'description' => __('Scheduled actions are hooks triggered on a cetain date and time.', 'woocommerce'),
 			'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', 'woocommerce' ),
-				'singular_name' => __( 'Scheduled Action', 'woocommerce' ),
-				'menu_name' => _x( 'Scheduled Actions', 'Admin menu name', 'woocommerce' ),
-				'add_new' => __( 'Add', 'woocommerce' ),
-				'add_new_item' => __( 'Add New Scheduled Action', 'woocommerce' ),
-				'edit' => __( 'Edit', 'woocommerce' ),
-				'edit_item' => __( 'Edit Scheduled Action', 'woocommerce' ),
-				'new_item' => __( 'New Scheduled Action', 'woocommerce' ),
-				'view' => __( 'View Action', 'woocommerce' ),
-				'view_item' => __( 'View Action', 'woocommerce' ),
-				'search_items' => __( 'Search Scheduled Actions', 'woocommerce' ),
-				'not_found' => __( 'No actions found', 'woocommerce' ),
-				'not_found_in_trash' => __( 'No actions found in trash', 'woocommerce' ),
+				'name' => __('Scheduled Actions', 'woocommerce'),
+				'singular_name' => __('Scheduled Action', 'woocommerce'),
+				'menu_name' => _x('Scheduled Actions', 'Admin menu name', 'woocommerce'),
+				'add_new' => __('Add', 'woocommerce'),
+				'add_new_item' => __('Add New Scheduled Action', 'woocommerce'),
+				'edit' => __('Edit', 'woocommerce'),
+				'edit_item' => __('Edit Scheduled Action', 'woocommerce'),
+				'new_item' => __('New Scheduled Action', 'woocommerce'),
+				'view' => __('View Action', 'woocommerce'),
+				'view_item' => __('View Action', 'woocommerce'),
+				'search_items' => __('Search Scheduled Actions', 'woocommerce'),
+				'not_found' => __('No actions found', 'woocommerce'),
+				'not_found_in_trash' => __('No actions found in trash', 'woocommerce'),
 			),
 		);
 

plugins/woocommerce/packages/action-scheduler/classes/ActionScheduler_QueueRunner.php

Spacing +34 added lines, -34 removed lines

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

plugins/woocommerce/packages/action-scheduler/classes/abstracts/ActionScheduler_Logger.php

Spacing +59 added lines, -59 removed lines

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

plugins/woocommerce/packages/action-scheduler/classes/abstracts/ActionScheduler_Abstract_Schedule.php

Spacing +7 added lines, -7 removed lines

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

plugins/woocommerce/packages/action-scheduler/classes/abstracts/ActionScheduler_Store.php

Spacing +65 added lines, -65 removed lines

@@ -26,14 +26,14 @@ 
 	 *
 	 * @return int The action ID
 	 */
-	abstract public function save_action( ActionScheduler_Action $action, DateTime $scheduled_date = NULL );
+	abstract public function save_action(ActionScheduler_Action $action, DateTime $scheduled_date = NULL);
 
 	/**
 	 * @param string $action_id
 	 *
 	 * @return ActionScheduler_Action
 	 */
-	abstract public function fetch_action( $action_id );
+	abstract public function fetch_action($action_id);
 
 	/**
 	 * Find an action.
@@ -45,7 +45,7 @@ 
 	 *
 	 * @return string|null ID of the next action matching the criteria or NULL if not found.
 	 */
-	public function find_action( $hook, $params = array() ) {
+	public function find_action($hook, $params = array()) {
 		$params = wp_parse_args(
 			$params,
 			array(
@@ -60,17 +60,17 @@ 
 		$params['orderby']  = 'date';
 		$params['per_page'] = 1;
 
-		if ( ! empty( $params['status'] ) ) {
-			if ( self::STATUS_PENDING === $params['status'] ) {
+		if (!empty($params['status'])) {
+			if (self::STATUS_PENDING === $params['status']) {
 				$params['order'] = 'ASC'; // Find the next action that matches.
 			} else {
 				$params['order'] = 'DESC'; // Find the most recent action that matches.
 			}
 		}
 
-		$results = $this->query_actions( $params );
+		$results = $this->query_actions($params);
 
-		return empty( $results ) ? null : $results[0];
+		return empty($results) ? null : $results[0];
 	}
 
 	/**
@@ -99,7 +99,7 @@ 
 	 *
 	 * @return string|array|null The IDs of actions matching the query. Null on failure.
 	 */
-	abstract public function query_actions( $query = array(), $query_type = 'select' );
+	abstract public function query_actions($query = array(), $query_type = 'select');
 
 	/**
 	 * Run query to get a single action ID.
@@ -112,12 +112,12 @@ 
 	 *
 	 * @return int|null
 	 */
-	public function query_action( $query ) {
+	public function query_action($query) {
 		$query['per_page'] = 1;
 		$query['offset']   = 0;
-		$results           = $this->query_actions( $query );
+		$results           = $this->query_actions($query);
 
-		if ( empty( $results ) ) {
+		if (empty($results)) {
 			return null;
 		} else {
 			return (int) $results[0];
@@ -134,19 +134,19 @@ 
 	/**
 	 * @param string $action_id
 	 */
-	abstract public function cancel_action( $action_id );
+	abstract public function cancel_action($action_id);
 
 	/**
 	 * @param string $action_id
 	 */
-	abstract public function delete_action( $action_id );
+	abstract public function delete_action($action_id);
 
 	/**
 	 * @param string $action_id
 	 *
 	 * @return DateTime The date the action is schedule to run, or the date that it ran.
 	 */
-	abstract public function get_date( $action_id );
+	abstract public function get_date($action_id);
 
 
 	/**
@@ -157,7 +157,7 @@ 
 	 *
 	 * @return ActionScheduler_ActionClaim
 	 */
-	abstract public function stake_claim( $max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '' );
+	abstract public function stake_claim($max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '');
 
 	/**
 	 * @return int
@@ -167,53 +167,53 @@ 
 	/**
 	 * @param ActionScheduler_ActionClaim $claim
 	 */
-	abstract public function release_claim( ActionScheduler_ActionClaim $claim );
+	abstract public function release_claim(ActionScheduler_ActionClaim $claim);
 
 	/**
 	 * @param string $action_id
 	 */
-	abstract public function unclaim_action( $action_id );
+	abstract public function unclaim_action($action_id);
 
 	/**
 	 * @param string $action_id
 	 */
-	abstract public function mark_failure( $action_id );
+	abstract public function mark_failure($action_id);
 
 	/**
 	 * @param string $action_id
 	 */
-	abstract public function log_execution( $action_id );
+	abstract public function log_execution($action_id);
 
 	/**
 	 * @param string $action_id
 	 */
-	abstract public function mark_complete( $action_id );
+	abstract public function mark_complete($action_id);
 
 	/**
 	 * @param string $action_id
 	 *
 	 * @return string
 	 */
-	abstract public function get_status( $action_id );
+	abstract public function get_status($action_id);
 
 	/**
 	 * @param string $action_id
 	 * @return mixed
 	 */
-	abstract public function get_claim_id( $action_id );
+	abstract public function get_claim_id($action_id);
 
 	/**
 	 * @param string $claim_id
 	 * @return array
 	 */
-	abstract public function find_actions_by_claim_id( $claim_id );
+	abstract public function find_actions_by_claim_id($claim_id);
 
 	/**
 	 * @param string $comparison_operator
 	 * @return string
 	 */
-	protected function validate_sql_comparator( $comparison_operator ) {
-		if ( in_array( $comparison_operator, array('!=', '>', '>=', '<', '<=', '=') ) ) {
+	protected function validate_sql_comparator($comparison_operator) {
+		if (in_array($comparison_operator, array('!=', '>', '>=', '<', '<=', '='))) {
 			return $comparison_operator;
 		}
 		return '=';
@@ -226,14 +226,14 @@ 
 	 * @param DateTime $scheduled_date (optional)
 	 * @return string
 	 */
-	protected function get_scheduled_date_string( ActionScheduler_Action $action, DateTime $scheduled_date = NULL ) {
+	protected function get_scheduled_date_string(ActionScheduler_Action $action, DateTime $scheduled_date = NULL) {
 		$next = null === $scheduled_date ? $action->get_schedule()->get_date() : $scheduled_date;
-		if ( ! $next ) {
+		if (!$next) {
 			return '0000-00-00 00:00:00';
 		}
-		$next->setTimezone( new DateTimeZone( 'UTC' ) );
+		$next->setTimezone(new DateTimeZone('UTC'));
 
-		return $next->format( 'Y-m-d H:i:s' );
+		return $next->format('Y-m-d H:i:s');
 	}
 
 	/**
@@ -243,14 +243,14 @@ 
 	 * @param DateTime $scheduled_date (optional)
 	 * @return string
 	 */
-	protected function get_scheduled_date_string_local( ActionScheduler_Action $action, DateTime $scheduled_date = NULL ) {
+	protected function get_scheduled_date_string_local(ActionScheduler_Action $action, DateTime $scheduled_date = NULL) {
 		$next = null === $scheduled_date ? $action->get_schedule()->get_date() : $scheduled_date;
-		if ( ! $next ) {
+		if (!$next) {
 			return '0000-00-00 00:00:00';
 		}
 
-		ActionScheduler_TimezoneHelper::set_local_timezone( $next );
-		return $next->format( 'Y-m-d H:i:s' );
+		ActionScheduler_TimezoneHelper::set_local_timezone($next);
+		return $next->format('Y-m-d H:i:s');
 	}
 
 	/**
@@ -261,15 +261,15 @@ 
 	 *
 	 * @throws ActionScheduler_InvalidActionException When the decoded arguments are invalid.
 	 */
-	protected function validate_args( $args, $action_id ) {
+	protected function validate_args($args, $action_id) {
 		// Ensure we have an array of args.
-		if ( ! is_array( $args ) ) {
-			throw ActionScheduler_InvalidActionException::from_decoding_args( $action_id );
+		if (!is_array($args)) {
+			throw ActionScheduler_InvalidActionException::from_decoding_args($action_id);
 		}
 
 		// Validate JSON decoding if possible.
-		if ( function_exists( 'json_last_error' ) && JSON_ERROR_NONE !== json_last_error() ) {
-			throw ActionScheduler_InvalidActionException::from_decoding_args( $action_id, $args );
+		if (function_exists('json_last_error') && JSON_ERROR_NONE !== json_last_error()) {
+			throw ActionScheduler_InvalidActionException::from_decoding_args($action_id, $args);
 		}
 	}
 
@@ -281,9 +281,9 @@ 
 	 *
 	 * @throws ActionScheduler_InvalidActionException When the schedule is invalid.
 	 */
-	protected function validate_schedule( $schedule, $action_id ) {
-		if ( empty( $schedule ) || ! is_a( $schedule, 'ActionScheduler_Schedule' ) ) {
-			throw ActionScheduler_InvalidActionException::from_schedule( $action_id, $schedule );
+	protected function validate_schedule($schedule, $action_id) {
+		if (empty($schedule) || !is_a($schedule, 'ActionScheduler_Schedule')) {
+			throw ActionScheduler_InvalidActionException::from_schedule($action_id, $schedule);
 		}
 	}
 
@@ -296,9 +296,9 @@ 
 	 * @param  ActionScheduler_Action $action Action to be validated.
 	 * @throws InvalidArgumentException When json encoded args is too long.
 	 */
-	protected function validate_action( ActionScheduler_Action $action ) {
-		if ( strlen( json_encode( $action->get_args() ) ) > static::$max_args_length ) {
-			throw new InvalidArgumentException( sprintf( __( 'ActionScheduler_Action::$args too long. To ensure the args column can be indexed, action args should not be more than %d characters when encoded as JSON.', 'woocommerce' ), static::$max_args_length ) );
+	protected function validate_action(ActionScheduler_Action $action) {
+		if (strlen(json_encode($action->get_args())) > static::$max_args_length) {
+			throw new InvalidArgumentException(sprintf(__('ActionScheduler_Action::$args too long. To ensure the args column can be indexed, action args should not be more than %d characters when encoded as JSON.', 'woocommerce'), static::$max_args_length));
 		}
 	}
 
@@ -311,9 +311,9 @@ 
 	 *
 	 * @return void
 	 */
-	public function cancel_actions_by_hook( $hook ) {
+	public function cancel_actions_by_hook($hook) {
 		$action_ids = true;
-		while ( ! empty( $action_ids ) ) {
+		while (!empty($action_ids)) {
 			$action_ids = $this->query_actions(
 				array(
 					'hook'     => $hook,
@@ -323,7 +323,7 @@ 
 				)
 			);
 
-			$this->bulk_cancel_actions( $action_ids );
+			$this->bulk_cancel_actions($action_ids);
 		}
 	}
 
@@ -336,9 +336,9 @@ 
 	 *
 	 * @return void
 	 */
-	public function cancel_actions_by_group( $group ) {
+	public function cancel_actions_by_group($group) {
 		$action_ids = true;
-		while ( ! empty( $action_ids ) ) {
+		while (!empty($action_ids)) {
 			$action_ids = $this->query_actions(
 				array(
 					'group'    => $group,
@@ -348,7 +348,7 @@ 
 				)
 			);
 
-			$this->bulk_cancel_actions( $action_ids );
+			$this->bulk_cancel_actions($action_ids);
 		}
 	}
 
@@ -361,12 +361,12 @@ 
 	 *
 	 * @return void
 	 */
-	private function bulk_cancel_actions( $action_ids ) {
-		foreach ( $action_ids as $action_id ) {
-			$this->cancel_action( $action_id );
+	private function bulk_cancel_actions($action_ids) {
+		foreach ($action_ids as $action_id) {
+			$this->cancel_action($action_id);
 		}
 
-		do_action( 'action_scheduler_bulk_cancel_actions', $action_ids );
+		do_action('action_scheduler_bulk_cancel_actions', $action_ids);
 	}
 
 	/**
@@ -374,11 +374,11 @@ 
 	 */
 	public function get_status_labels() {
 		return array(
-			self::STATUS_COMPLETE => __( 'Complete', 'woocommerce' ),
-			self::STATUS_PENDING  => __( 'Pending', 'woocommerce' ),
-			self::STATUS_RUNNING  => __( 'In-progress', 'woocommerce' ),
-			self::STATUS_FAILED   => __( 'Failed', 'woocommerce' ),
-			self::STATUS_CANCELED => __( 'Canceled', 'woocommerce' ),
+			self::STATUS_COMPLETE => __('Complete', 'woocommerce'),
+			self::STATUS_PENDING  => __('Pending', 'woocommerce'),
+			self::STATUS_RUNNING  => __('In-progress', 'woocommerce'),
+			self::STATUS_FAILED   => __('Failed', 'woocommerce'),
+			self::STATUS_CANCELED => __('Canceled', 'woocommerce'),
 		);
 	}
 
@@ -390,13 +390,13 @@ 
 	 * @return string
 	 */
 	public function has_pending_actions_due() {
-		$pending_actions = $this->query_actions( array(
+		$pending_actions = $this->query_actions(array(
 			'date'    => as_get_datetime_object(),
 			'status'  => ActionScheduler_Store::STATUS_PENDING,
 			'orderby' => 'none',
-		) );
+		));
 
-		return ! empty( $pending_actions );
+		return !empty($pending_actions);
 	}
 
 	/**
@@ -407,14 +407,14 @@ 
 	/**
 	 * Callable function to mark an action as migrated optionally overridden in derived classes.
 	 */
-	public function mark_migrated( $action_id ) {}
+	public function mark_migrated($action_id) {}
 
 	/**
 	 * @return ActionScheduler_Store
 	 */
 	public static function instance() {
-		if ( empty( self::$store ) ) {
-			$class = apply_filters( 'action_scheduler_store_class', self::DEFAULT_CLASS );
+		if (empty(self::$store)) {
+			$class = apply_filters('action_scheduler_store_class', self::DEFAULT_CLASS);
 			self::$store = new $class();
 		}
 		return self::$store;