Passed
Push — master ( 495efb...ace576 )
by Brian
06:14 queued 01:46
created
includes/libraries/action-scheduler/classes/migration/Controller.php 1 patch
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
 	 *
37 37
 	 * @param Scheduler $migration_scheduler Migration scheduler object.
38 38
 	 */
39
-	protected function __construct( Scheduler $migration_scheduler ) {
39
+	protected function __construct(Scheduler $migration_scheduler) {
40 40
 		$this->migration_scheduler = $migration_scheduler;
41 41
 		$this->store_classname     = '';
42 42
 	}
@@ -48,10 +48,10 @@  discard block
 block discarded – undo
48 48
 	 *
49 49
 	 * @return string
50 50
 	 */
51
-	public function get_store_class( $class ) {
52
-		if ( \ActionScheduler_DataController::is_migration_complete() ) {
51
+	public function get_store_class($class) {
52
+		if (\ActionScheduler_DataController::is_migration_complete()) {
53 53
 			return \ActionScheduler_DataController::DATASTORE_CLASS;
54
-		} elseif ( \ActionScheduler_Store::DEFAULT_CLASS !== $class ) {
54
+		} elseif (\ActionScheduler_Store::DEFAULT_CLASS !== $class) {
55 55
 			$this->store_classname = $class;
56 56
 			return $class;
57 57
 		} else {
@@ -66,10 +66,10 @@  discard block
 block discarded – undo
66 66
 	 *
67 67
 	 * @return string
68 68
 	 */
69
-	public function get_logger_class( $class ) {
69
+	public function get_logger_class($class) {
70 70
 		\ActionScheduler_Store::instance();
71 71
 
72
-		if ( $this->has_custom_datastore() ) {
72
+		if ($this->has_custom_datastore()) {
73 73
 			$this->logger_classname = $class;
74 74
 			return $class;
75 75
 		} else {
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 	 * @return void
93 93
 	 */
94 94
 	public function schedule_migration() {
95
-		if ( \ActionScheduler_DataController::is_migration_complete() || $this->migration_scheduler->is_migration_scheduled() ) {
95
+		if (\ActionScheduler_DataController::is_migration_complete() || $this->migration_scheduler->is_migration_scheduled()) {
96 96
 			return;
97 97
 		}
98 98
 
@@ -107,53 +107,53 @@  discard block
 block discarded – undo
107 107
 	public function get_migration_config_object() {
108 108
 		static $config = null;
109 109
 
110
-		if ( ! $config ) {
110
+		if (!$config) {
111 111
 			$source_store  = $this->store_classname ? new $this->store_classname() : new \ActionScheduler_wpPostStore();
112 112
 			$source_logger = $this->logger_classname ? new $this->logger_classname() : new \ActionScheduler_wpCommentLogger();
113 113
 
114 114
 			$config = new Config();
115
-			$config->set_source_store( $source_store );
116
-			$config->set_source_logger( $source_logger );
117
-			$config->set_destination_store( new \ActionScheduler_DBStoreMigrator() );
118
-			$config->set_destination_logger( new \ActionScheduler_DBLogger() );
115
+			$config->set_source_store($source_store);
116
+			$config->set_source_logger($source_logger);
117
+			$config->set_destination_store(new \ActionScheduler_DBStoreMigrator());
118
+			$config->set_destination_logger(new \ActionScheduler_DBLogger());
119 119
 
120
-			if ( defined( 'WP_CLI' ) && WP_CLI ) {
121
-				$config->set_progress_bar( new ProgressBar( '', 0 ) );
120
+			if (defined('WP_CLI') && WP_CLI) {
121
+				$config->set_progress_bar(new ProgressBar('', 0));
122 122
 			}
123 123
 		}
124 124
 
125
-		return apply_filters( 'action_scheduler/migration_config', $config );
125
+		return apply_filters('action_scheduler/migration_config', $config);
126 126
 	}
127 127
 
128 128
 	/**
129 129
 	 * Hook dashboard migration notice.
130 130
 	 */
131 131
 	public function hook_admin_notices() {
132
-		if ( ! $this->allow_migration() || \ActionScheduler_DataController::is_migration_complete() ) {
132
+		if (!$this->allow_migration() || \ActionScheduler_DataController::is_migration_complete()) {
133 133
 			return;
134 134
 		}
135
-		add_action( 'admin_notices', array( $this, 'display_migration_notice' ), 10, 0 );
135
+		add_action('admin_notices', array($this, 'display_migration_notice'), 10, 0);
136 136
 	}
137 137
 
138 138
 	/**
139 139
 	 * Show a dashboard notice that migration is in progress.
140 140
 	 */
141 141
 	public function display_migration_notice() {
142
-		printf( '<div class="notice notice-warning"><p>%s</p></div>', __( 'Action Scheduler migration in progress. The list of scheduled actions may be incomplete.', 'action-scheduler' ) );
142
+		printf('<div class="notice notice-warning"><p>%s</p></div>', __('Action Scheduler migration in progress. The list of scheduled actions may be incomplete.', 'action-scheduler'));
143 143
 	}
144 144
 
145 145
 	/**
146 146
 	 * Add store classes. Hook migration.
147 147
 	 */
148 148
 	private function hook() {
149
-		add_filter( 'action_scheduler_store_class', array( $this, 'get_store_class' ), 100, 1 );
150
-		add_filter( 'action_scheduler_logger_class', array( $this, 'get_logger_class' ), 100, 1 );
151
-		add_action( 'init', array( $this, 'maybe_hook_migration' ) );
152
-		add_action( 'shutdown', array( $this, 'schedule_migration' ), 0, 0 );
149
+		add_filter('action_scheduler_store_class', array($this, 'get_store_class'), 100, 1);
150
+		add_filter('action_scheduler_logger_class', array($this, 'get_logger_class'), 100, 1);
151
+		add_action('init', array($this, 'maybe_hook_migration'));
152
+		add_action('shutdown', array($this, 'schedule_migration'), 0, 0);
153 153
 
154 154
 		// Action Scheduler may be displayed as a Tools screen or WooCommerce > Status administration screen
155
-		add_action( 'load-tools_page_action-scheduler', array( $this, 'hook_admin_notices' ), 10, 0 );
156
-		add_action( 'load-woocommerce_page_wc-status', array( $this, 'hook_admin_notices' ), 10, 0 );
155
+		add_action('load-tools_page_action-scheduler', array($this, 'hook_admin_notices'), 10, 0);
156
+		add_action('load-woocommerce_page_wc-status', array($this, 'hook_admin_notices'), 10, 0);
157 157
 	}
158 158
 
159 159
 	/**
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
 	 * @author Jeremy Pry
163 163
 	 */
164 164
 	public function maybe_hook_migration() {
165
-		if ( ! $this->allow_migration() || \ActionScheduler_DataController::is_migration_complete() ) {
165
+		if (!$this->allow_migration() || \ActionScheduler_DataController::is_migration_complete()) {
166 166
 			return;
167 167
 		}
168 168
 
@@ -173,22 +173,22 @@  discard block
 block discarded – undo
173 173
 	 * Allow datastores to enable migration to AS tables.
174 174
 	 */
175 175
 	public function allow_migration() {
176
-		if ( ! \ActionScheduler_DataController::dependencies_met() ) {
176
+		if (!\ActionScheduler_DataController::dependencies_met()) {
177 177
 			return false;
178 178
 		}
179 179
 
180
-		if ( null === $this->migrate_custom_store ) {
181
-			$this->migrate_custom_store = apply_filters( 'action_scheduler_migrate_data_store', false );
180
+		if (null === $this->migrate_custom_store) {
181
+			$this->migrate_custom_store = apply_filters('action_scheduler_migrate_data_store', false);
182 182
 		}
183 183
 
184
-		return ( ! $this->has_custom_datastore() ) || $this->migrate_custom_store;
184
+		return (!$this->has_custom_datastore()) || $this->migrate_custom_store;
185 185
 	}
186 186
 
187 187
 	/**
188 188
 	 * Proceed with the migration if the dependencies have been met.
189 189
 	 */
190 190
 	public static function init() {
191
-		if ( \ActionScheduler_DataController::dependencies_met() ) {
191
+		if (\ActionScheduler_DataController::dependencies_met()) {
192 192
 			self::instance()->hook();
193 193
 		}
194 194
 	}
@@ -197,8 +197,8 @@  discard block
 block discarded – undo
197 197
 	 * Singleton factory.
198 198
 	 */
199 199
 	public static function instance() {
200
-		if ( ! isset( self::$instance ) ) {
201
-			self::$instance = new static( new Scheduler() );
200
+		if (!isset(self::$instance)) {
201
+			self::$instance = new static(new Scheduler());
202 202
 		}
203 203
 
204 204
 		return self::$instance;
Please login to merge, or discard this patch.
includes/libraries/action-scheduler/classes/migration/ActionMigrator.php 1 patch
Spacing   +26 added lines, -26 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 	 * @param ActionScheduler_Store $destination_store Destination store object.
30 30
 	 * @param LogMigrator           $log_migrator Log migrator object.
31 31
 	 */
32
-	public function __construct( \ActionScheduler_Store $source_store, \ActionScheduler_Store $destination_store, LogMigrator $log_migrator ) {
32
+	public function __construct(\ActionScheduler_Store $source_store, \ActionScheduler_Store $destination_store, LogMigrator $log_migrator) {
33 33
 		$this->source       = $source_store;
34 34
 		$this->destination  = $destination_store;
35 35
 		$this->log_migrator = $log_migrator;
@@ -42,25 +42,25 @@  discard block
 block discarded – undo
42 42
 	 *
43 43
 	 * @return int 0|new action ID
44 44
 	 */
45
-	public function migrate( $source_action_id ) {
45
+	public function migrate($source_action_id) {
46 46
 		try {
47
-			$action = $this->source->fetch_action( $source_action_id );
48
-			$status = $this->source->get_status( $source_action_id );
49
-		} catch ( \Exception $e ) {
47
+			$action = $this->source->fetch_action($source_action_id);
48
+			$status = $this->source->get_status($source_action_id);
49
+		} catch (\Exception $e) {
50 50
 			$action = null;
51 51
 			$status = '';
52 52
 		}
53 53
 
54
-		if ( is_null( $action ) || empty( $status ) || ! $action->get_schedule()->get_date() ) {
54
+		if (is_null($action) || empty($status) || !$action->get_schedule()->get_date()) {
55 55
 			// null action or empty status means the fetch operation failed or the action didn't exist
56 56
 			// null schedule means it's missing vital data
57 57
 			// delete it and move on
58 58
 			try {
59
-				$this->source->delete_action( $source_action_id );
60
-			} catch ( \Exception $e ) {
59
+				$this->source->delete_action($source_action_id);
60
+			} catch (\Exception $e) {
61 61
 				// nothing to do, it didn't exist in the first place
62 62
 			}
63
-			do_action( 'action_scheduler/no_action_to_migrate', $source_action_id, $this->source, $this->destination );
63
+			do_action('action_scheduler/no_action_to_migrate', $source_action_id, $this->source, $this->destination);
64 64
 
65 65
 			return 0;
66 66
 		}
@@ -68,40 +68,40 @@  discard block
 block discarded – undo
68 68
 		try {
69 69
 
70 70
 			// Make sure the last attempt date is set correctly for completed and failed actions
71
-			$last_attempt_date = ( $status !== \ActionScheduler_Store::STATUS_PENDING ) ? $this->source->get_date( $source_action_id ) : null;
71
+			$last_attempt_date = ($status !== \ActionScheduler_Store::STATUS_PENDING) ? $this->source->get_date($source_action_id) : null;
72 72
 
73
-			$destination_action_id = $this->destination->save_action( $action, null, $last_attempt_date );
74
-		} catch ( \Exception $e ) {
75
-			do_action( 'action_scheduler/migrate_action_failed', $source_action_id, $this->source, $this->destination );
73
+			$destination_action_id = $this->destination->save_action($action, null, $last_attempt_date);
74
+		} catch (\Exception $e) {
75
+			do_action('action_scheduler/migrate_action_failed', $source_action_id, $this->source, $this->destination);
76 76
 
77 77
 			return 0; // could not save the action in the new store
78 78
 		}
79 79
 
80 80
 		try {
81
-			switch ( $status ) {
81
+			switch ($status) {
82 82
 				case \ActionScheduler_Store::STATUS_FAILED :
83
-					$this->destination->mark_failure( $destination_action_id );
83
+					$this->destination->mark_failure($destination_action_id);
84 84
 					break;
85 85
 				case \ActionScheduler_Store::STATUS_CANCELED :
86
-					$this->destination->cancel_action( $destination_action_id );
86
+					$this->destination->cancel_action($destination_action_id);
87 87
 					break;
88 88
 			}
89 89
 
90
-			$this->log_migrator->migrate( $source_action_id, $destination_action_id );
91
-			$this->source->delete_action( $source_action_id );
90
+			$this->log_migrator->migrate($source_action_id, $destination_action_id);
91
+			$this->source->delete_action($source_action_id);
92 92
 
93
-			$test_action = $this->source->fetch_action( $source_action_id );
94
-			if ( ! is_a( $test_action, 'ActionScheduler_NullAction' ) ) {
95
-				throw new \RuntimeException( sprintf( __( 'Unable to remove source migrated action %s', 'action-scheduler' ), $source_action_id ) );
93
+			$test_action = $this->source->fetch_action($source_action_id);
94
+			if (!is_a($test_action, 'ActionScheduler_NullAction')) {
95
+				throw new \RuntimeException(sprintf(__('Unable to remove source migrated action %s', 'action-scheduler'), $source_action_id));
96 96
 			}
97
-			do_action( 'action_scheduler/migrated_action', $source_action_id, $destination_action_id, $this->source, $this->destination );
97
+			do_action('action_scheduler/migrated_action', $source_action_id, $destination_action_id, $this->source, $this->destination);
98 98
 
99 99
 			return $destination_action_id;
100
-		} catch ( \Exception $e ) {
100
+		} catch (\Exception $e) {
101 101
 			// could not delete from the old store
102
-			$this->source->mark_migrated( $source_action_id );
103
-			do_action( 'action_scheduler/migrate_action_incomplete', $source_action_id, $destination_action_id, $this->source, $this->destination );
104
-			do_action( 'action_scheduler/migrated_action', $source_action_id, $destination_action_id, $this->source, $this->destination );
102
+			$this->source->mark_migrated($source_action_id);
103
+			do_action('action_scheduler/migrate_action_incomplete', $source_action_id, $destination_action_id, $this->source, $this->destination);
104
+			do_action('action_scheduler/migrated_action', $source_action_id, $destination_action_id, $this->source, $this->destination);
105 105
 
106 106
 			return $destination_action_id;
107 107
 		}
Please login to merge, or discard this patch.
action-scheduler/classes/schedules/ActionScheduler_IntervalSchedule.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -22,8 +22,8 @@  discard block
 block discarded – undo
22 22
 	 * @param DateTime $after
23 23
 	 * @return DateTime
24 24
 	 */
25
-	protected function calculate_next( DateTime $after ) {
26
-		$after->modify( '+' . (int) $this->get_recurrence() . ' seconds' );
25
+	protected function calculate_next(DateTime $after) {
26
+		$after->modify('+' . (int) $this->get_recurrence() . ' seconds');
27 27
 		return $after;
28 28
 	}
29 29
 
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 	 * @return int
32 32
 	 */
33 33
 	public function interval_in_seconds() {
34
-		_deprecated_function( __METHOD__, '3.0.0', '(int)ActionScheduler_Abstract_RecurringSchedule::get_recurrence()' );
34
+		_deprecated_function(__METHOD__, '3.0.0', '(int)ActionScheduler_Abstract_RecurringSchedule::get_recurrence()');
35 35
 		return (int) $this->get_recurrence();
36 36
 	}
37 37
 
@@ -55,10 +55,10 @@  discard block
 block discarded – undo
55 55
 		$this->start_timestamp     = $this->scheduled_timestamp;
56 56
 		$this->interval_in_seconds = $this->recurrence;
57 57
 
58
-		return array_merge( $sleep_params, array(
58
+		return array_merge($sleep_params, array(
59 59
 			'start_timestamp',
60 60
 			'interval_in_seconds'
61
-		) );
61
+		));
62 62
 	}
63 63
 
64 64
 	/**
@@ -67,14 +67,14 @@  discard block
 block discarded – undo
67 67
 	 * For more background, @see ActionScheduler_Abstract_RecurringSchedule::__wakeup().
68 68
 	 */
69 69
 	public function __wakeup() {
70
-		if ( is_null( $this->scheduled_timestamp ) && ! is_null( $this->start_timestamp ) ) {
70
+		if (is_null($this->scheduled_timestamp) && !is_null($this->start_timestamp)) {
71 71
 			$this->scheduled_timestamp = $this->start_timestamp;
72
-			unset( $this->start_timestamp );
72
+			unset($this->start_timestamp);
73 73
 		}
74 74
 
75
-		if ( is_null( $this->recurrence ) && ! is_null( $this->interval_in_seconds ) ) {
75
+		if (is_null($this->recurrence) && !is_null($this->interval_in_seconds)) {
76 76
 			$this->recurrence = $this->interval_in_seconds;
77
-			unset( $this->interval_in_seconds );
77
+			unset($this->interval_in_seconds);
78 78
 		}
79 79
 		parent::__wakeup();
80 80
 	}
Please login to merge, or discard this patch.
action-scheduler/classes/schedules/ActionScheduler_SimpleSchedule.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@  discard block
 block discarded – undo
15 15
 	 *
16 16
 	 * @return DateTime|null
17 17
 	 */
18
-	public function calculate_next( DateTime $after ) {
18
+	public function calculate_next(DateTime $after) {
19 19
 		return null;
20 20
 	}
21 21
 
@@ -45,9 +45,9 @@  discard block
 block discarded – undo
45 45
 
46 46
 		$this->timestamp = $this->scheduled_timestamp;
47 47
 
48
-		return array_merge( $sleep_params, array(
48
+		return array_merge($sleep_params, array(
49 49
 			'timestamp',
50
-		) );
50
+		));
51 51
 	}
52 52
 
53 53
 	/**
@@ -62,9 +62,9 @@  discard block
 block discarded – undo
62 62
 	 */
63 63
 	public function __wakeup() {
64 64
 
65
-		if ( is_null( $this->scheduled_timestamp ) && ! is_null( $this->timestamp ) ) {
65
+		if (is_null($this->scheduled_timestamp) && !is_null($this->timestamp)) {
66 66
 			$this->scheduled_timestamp = $this->timestamp;
67
-			unset( $this->timestamp );
67
+			unset($this->timestamp);
68 68
 		}
69 69
 		parent::__wakeup();
70 70
 	}
Please login to merge, or discard this patch.
libraries/action-scheduler/classes/schedules/ActionScheduler_Schedule.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@
 block discarded – undo
8 8
 	 * @param DateTime $after
9 9
 	 * @return DateTime|null
10 10
 	 */
11
-	public function next( DateTime $after = NULL );
11
+	public function next(DateTime $after = NULL);
12 12
 
13 13
 	/**
14 14
 	 * @return bool
Please login to merge, or discard this patch.
action-scheduler/classes/schedules/ActionScheduler_NullSchedule.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@
 block discarded – undo
10 10
 	 *
11 11
 	 * @param null $date The date & time to run the action.
12 12
 	 */
13
-	public function __construct( DateTime $date = null ) {
13
+	public function __construct(DateTime $date = null) {
14 14
 		$this->scheduled_date = null;
15 15
 	}
16 16
 
Please login to merge, or discard this patch.
action-scheduler/classes/schedules/ActionScheduler_CanceledSchedule.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@  discard block
 block discarded – undo
15 15
 	 *
16 16
 	 * @return DateTime|null
17 17
 	 */
18
-	public function calculate_next( DateTime $after ) {
18
+	public function calculate_next(DateTime $after) {
19 19
 		return null;
20 20
 	}
21 21
 
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
 	 * @param DateTime $after
27 27
 	 * @return DateTime|null
28 28
 	 */
29
-	public function get_next( DateTime $after ) {
29
+	public function get_next(DateTime $after) {
30 30
 		return null;
31 31
 	}
32 32
 
@@ -48,9 +48,9 @@  discard block
 block discarded – undo
48 48
 	 * map the old property names with matching visibility.
49 49
 	 */
50 50
 	public function __wakeup() {
51
-		if ( ! is_null( $this->timestamp ) ) {
51
+		if (!is_null($this->timestamp)) {
52 52
 			$this->scheduled_timestamp = $this->timestamp;
53
-			unset( $this->timestamp );
53
+			unset($this->timestamp);
54 54
 		}
55 55
 		parent::__wakeup();
56 56
 	}
Please login to merge, or discard this patch.
action-scheduler/classes/schedules/ActionScheduler_CronSchedule.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -23,18 +23,18 @@  discard block
 block discarded – undo
23 23
 	 * @param CronExpression|string $recurrence The CronExpression used to calculate the schedule's next instance.
24 24
 	 * @param DateTime|null $first (Optional) The date & time the first instance of this interval schedule ran. Default null, meaning this is the first instance.
25 25
 	 */
26
-	public function __construct( DateTime $start, $recurrence, DateTime $first = null ) {
27
-		if ( ! is_a( $recurrence, 'CronExpression' ) ) {
28
-			$recurrence = CronExpression::factory( $recurrence );
26
+	public function __construct(DateTime $start, $recurrence, DateTime $first = null) {
27
+		if (!is_a($recurrence, 'CronExpression')) {
28
+			$recurrence = CronExpression::factory($recurrence);
29 29
 		}
30 30
 
31 31
 		// For backward compatibility, we need to make sure the date is set to the first matching cron date, not whatever date is passed in. Importantly, by passing true as the 3rd param, if $start matches the cron expression, then it will be used. This was previously handled in the now deprecated next() method.
32
-		$date = $recurrence->getNextRunDate( $start, 0, true );
32
+		$date = $recurrence->getNextRunDate($start, 0, true);
33 33
 
34 34
 		// parent::__construct() will set this to $date by default, but that may be different to $start now.
35
-		$first = empty( $first ) ? $start : $first;
35
+		$first = empty($first) ? $start : $first;
36 36
 
37
-		parent::__construct( $date, $recurrence, $first );
37
+		parent::__construct($date, $recurrence, $first);
38 38
 	}
39 39
 
40 40
 	/**
@@ -44,15 +44,15 @@  discard block
 block discarded – undo
44 44
 	 * @param DateTime $after
45 45
 	 * @return DateTime
46 46
 	 */
47
-	protected function calculate_next( DateTime $after ) {
48
-		return $this->recurrence->getNextRunDate( $after, 0, false );
47
+	protected function calculate_next(DateTime $after) {
48
+		return $this->recurrence->getNextRunDate($after, 0, false);
49 49
 	}
50 50
 
51 51
 	/**
52 52
 	 * @return string
53 53
 	 */
54 54
 	public function get_recurrence() {
55
-		return strval( $this->recurrence );
55
+		return strval($this->recurrence);
56 56
 	}
57 57
 
58 58
 	/**
@@ -75,10 +75,10 @@  discard block
 block discarded – undo
75 75
 		$this->start_timestamp = $this->scheduled_timestamp;
76 76
 		$this->cron            = $this->recurrence;
77 77
 
78
-		return array_merge( $sleep_params, array(
78
+		return array_merge($sleep_params, array(
79 79
 			'start_timestamp',
80 80
 			'cron'
81
-		) );
81
+		));
82 82
 	}
83 83
 
84 84
 	/**
@@ -87,14 +87,14 @@  discard block
 block discarded – undo
87 87
 	 * For more background, @see ActionScheduler_Abstract_RecurringSchedule::__wakeup().
88 88
 	 */
89 89
 	public function __wakeup() {
90
-		if ( is_null( $this->scheduled_timestamp ) && ! is_null( $this->start_timestamp ) ) {
90
+		if (is_null($this->scheduled_timestamp) && !is_null($this->start_timestamp)) {
91 91
 			$this->scheduled_timestamp = $this->start_timestamp;
92
-			unset( $this->start_timestamp );
92
+			unset($this->start_timestamp);
93 93
 		}
94 94
 
95
-		if ( is_null( $this->recurrence ) && ! is_null( $this->cron ) ) {
95
+		if (is_null($this->recurrence) && !is_null($this->cron)) {
96 96
 			$this->recurrence = $this->cron;
97
-			unset( $this->cron );
97
+			unset($this->cron);
98 98
 		}
99 99
 		parent::__wakeup();
100 100
 	}
Please login to merge, or discard this patch.
includes/libraries/action-scheduler/classes/ActionScheduler_ListTable.php 1 patch
Spacing   +120 added lines, -120 removed lines patch added patch discarded remove patch
@@ -80,26 +80,26 @@  discard block
 block discarded – undo
80 80
 	 * @param ActionScheduler_Logger $logger
81 81
 	 * @param ActionScheduler_QueueRunner $runner
82 82
 	 */
83
-	public function __construct( ActionScheduler_Store $store, ActionScheduler_Logger $logger, ActionScheduler_QueueRunner $runner ) {
83
+	public function __construct(ActionScheduler_Store $store, ActionScheduler_Logger $logger, ActionScheduler_QueueRunner $runner) {
84 84
 
85 85
 		$this->store  = $store;
86 86
 		$this->logger = $logger;
87 87
 		$this->runner = $runner;
88 88
 
89
-		$this->table_header = __( 'Scheduled Actions', 'action-scheduler' );
89
+		$this->table_header = __('Scheduled Actions', 'action-scheduler');
90 90
 
91 91
 		$this->bulk_actions = array(
92
-			'delete' => __( 'Delete', 'action-scheduler' ),
92
+			'delete' => __('Delete', 'action-scheduler'),
93 93
 		);
94 94
 
95 95
 		$this->columns = array(
96
-			'hook'        => __( 'Hook', 'action-scheduler' ),
97
-			'status'      => __( 'Status', 'action-scheduler' ),
98
-			'args'        => __( 'Arguments', 'action-scheduler' ),
99
-			'group'       => __( 'Group', 'action-scheduler' ),
100
-			'recurrence'  => __( 'Recurrence', 'action-scheduler' ),
101
-			'schedule'    => __( 'Scheduled Date', 'action-scheduler' ),
102
-			'log_entries' => __( 'Log', 'action-scheduler' ),
96
+			'hook'        => __('Hook', 'action-scheduler'),
97
+			'status'      => __('Status', 'action-scheduler'),
98
+			'args'        => __('Arguments', 'action-scheduler'),
99
+			'group'       => __('Group', 'action-scheduler'),
100
+			'recurrence'  => __('Recurrence', 'action-scheduler'),
101
+			'schedule'    => __('Scheduled Date', 'action-scheduler'),
102
+			'log_entries' => __('Log', 'action-scheduler'),
103 103
 		);
104 104
 
105 105
 		$this->sort_by = array(
@@ -116,22 +116,22 @@  discard block
 block discarded – undo
116 116
 
117 117
 		$request_status = $this->get_request_status();
118 118
 
119
-		if ( empty( $request_status ) ) {
119
+		if (empty($request_status)) {
120 120
 			$this->sort_by[] = 'status';
121
-		} elseif ( in_array( $request_status, array( 'in-progress', 'failed' ) ) ) {
122
-			$this->columns  += array( 'claim_id' => __( 'Claim ID', 'action-scheduler' ) );
121
+		} elseif (in_array($request_status, array('in-progress', 'failed'))) {
122
+			$this->columns  += array('claim_id' => __('Claim ID', 'action-scheduler'));
123 123
 			$this->sort_by[] = 'claim_id';
124 124
 		}
125 125
 
126 126
 		$this->row_actions = array(
127 127
 			'hook' => array(
128 128
 				'run' => array(
129
-					'name'  => __( 'Run', 'action-scheduler' ),
130
-					'desc'  => __( 'Process the action now as if it were run as part of a queue', 'action-scheduler' ),
129
+					'name'  => __('Run', 'action-scheduler'),
130
+					'desc'  => __('Process the action now as if it were run as part of a queue', 'action-scheduler'),
131 131
 				),
132 132
 				'cancel' => array(
133
-					'name'  => __( 'Cancel', 'action-scheduler' ),
134
-					'desc'  => __( 'Cancel the action now to avoid it being run in future', 'action-scheduler' ),
133
+					'name'  => __('Cancel', 'action-scheduler'),
134
+					'desc'  => __('Cancel the action now to avoid it being run in future', 'action-scheduler'),
135 135
 					'class' => 'cancel trash',
136 136
 				),
137 137
 			),
@@ -141,45 +141,45 @@  discard block
 block discarded – undo
141 141
 			array(
142 142
 				'seconds' => YEAR_IN_SECONDS,
143 143
 				/* translators: %s: amount of time */
144
-				'names'   => _n_noop( '%s year', '%s years', 'action-scheduler' ),
144
+				'names'   => _n_noop('%s year', '%s years', 'action-scheduler'),
145 145
 			),
146 146
 			array(
147 147
 				'seconds' => MONTH_IN_SECONDS,
148 148
 				/* translators: %s: amount of time */
149
-				'names'   => _n_noop( '%s month', '%s months', 'action-scheduler' ),
149
+				'names'   => _n_noop('%s month', '%s months', 'action-scheduler'),
150 150
 			),
151 151
 			array(
152 152
 				'seconds' => WEEK_IN_SECONDS,
153 153
 				/* translators: %s: amount of time */
154
-				'names'   => _n_noop( '%s week', '%s weeks', 'action-scheduler' ),
154
+				'names'   => _n_noop('%s week', '%s weeks', 'action-scheduler'),
155 155
 			),
156 156
 			array(
157 157
 				'seconds' => DAY_IN_SECONDS,
158 158
 				/* translators: %s: amount of time */
159
-				'names'   => _n_noop( '%s day', '%s days', 'action-scheduler' ),
159
+				'names'   => _n_noop('%s day', '%s days', 'action-scheduler'),
160 160
 			),
161 161
 			array(
162 162
 				'seconds' => HOUR_IN_SECONDS,
163 163
 				/* translators: %s: amount of time */
164
-				'names'   => _n_noop( '%s hour', '%s hours', 'action-scheduler' ),
164
+				'names'   => _n_noop('%s hour', '%s hours', 'action-scheduler'),
165 165
 			),
166 166
 			array(
167 167
 				'seconds' => MINUTE_IN_SECONDS,
168 168
 				/* translators: %s: amount of time */
169
-				'names'   => _n_noop( '%s minute', '%s minutes', 'action-scheduler' ),
169
+				'names'   => _n_noop('%s minute', '%s minutes', 'action-scheduler'),
170 170
 			),
171 171
 			array(
172 172
 				'seconds' => 1,
173 173
 				/* translators: %s: amount of time */
174
-				'names'   => _n_noop( '%s second', '%s seconds', 'action-scheduler' ),
174
+				'names'   => _n_noop('%s second', '%s seconds', 'action-scheduler'),
175 175
 			),
176 176
 		);
177 177
 
178
-		parent::__construct( array(
178
+		parent::__construct(array(
179 179
 			'singular' => 'action-scheduler',
180 180
 			'plural'   => 'action-scheduler',
181 181
 			'ajax'     => false,
182
-		) );
182
+		));
183 183
 	}
184 184
 
185 185
 	/**
@@ -195,24 +195,24 @@  discard block
 block discarded – undo
195 195
 	 * @param int $periods_to_include Depth of time periods to include, e.g. for an interval of 70, and $periods_to_include of 2, both minutes and seconds would be included. With a value of 1, only minutes would be included.
196 196
 	 * @return string A human friendly string representation of the interval.
197 197
 	 */
198
-	private static function human_interval( $interval, $periods_to_include = 2 ) {
198
+	private static function human_interval($interval, $periods_to_include = 2) {
199 199
 
200
-		if ( $interval <= 0 ) {
201
-			return __( 'Now!', 'action-scheduler' );
200
+		if ($interval <= 0) {
201
+			return __('Now!', 'action-scheduler');
202 202
 		}
203 203
 
204 204
 		$output = '';
205 205
 
206
-		for ( $time_period_index = 0, $periods_included = 0, $seconds_remaining = $interval; $time_period_index < count( self::$time_periods ) && $seconds_remaining > 0 && $periods_included < $periods_to_include; $time_period_index++ ) {
206
+		for ($time_period_index = 0, $periods_included = 0, $seconds_remaining = $interval; $time_period_index < count(self::$time_periods) && $seconds_remaining > 0 && $periods_included < $periods_to_include; $time_period_index++) {
207 207
 
208
-			$periods_in_interval = floor( $seconds_remaining / self::$time_periods[ $time_period_index ]['seconds'] );
208
+			$periods_in_interval = floor($seconds_remaining / self::$time_periods[$time_period_index]['seconds']);
209 209
 
210
-			if ( $periods_in_interval > 0 ) {
211
-				if ( ! empty( $output ) ) {
210
+			if ($periods_in_interval > 0) {
211
+				if (!empty($output)) {
212 212
 					$output .= ' ';
213 213
 				}
214
-				$output .= sprintf( _n( self::$time_periods[ $time_period_index ]['names'][0], self::$time_periods[ $time_period_index ]['names'][1], $periods_in_interval, 'action-scheduler' ), $periods_in_interval );
215
-				$seconds_remaining -= $periods_in_interval * self::$time_periods[ $time_period_index ]['seconds'];
214
+				$output .= sprintf(_n(self::$time_periods[$time_period_index]['names'][0], self::$time_periods[$time_period_index]['names'][1], $periods_in_interval, 'action-scheduler'), $periods_in_interval);
215
+				$seconds_remaining -= $periods_in_interval * self::$time_periods[$time_period_index]['seconds'];
216 216
 				$periods_included++;
217 217
 			}
218 218
 		}
@@ -227,20 +227,20 @@  discard block
 block discarded – undo
227 227
 	 *
228 228
 	 * @return string
229 229
 	 */
230
-	protected function get_recurrence( $action ) {
230
+	protected function get_recurrence($action) {
231 231
 		$schedule = $action->get_schedule();
232
-		if ( $schedule->is_recurring() ) {
232
+		if ($schedule->is_recurring()) {
233 233
 			$recurrence = $schedule->get_recurrence();
234 234
 
235
-			if ( is_numeric( $recurrence ) ) {
235
+			if (is_numeric($recurrence)) {
236 236
 				/* translators: %s: time interval */
237
-				return sprintf( __( 'Every %s', 'action-scheduler' ), self::human_interval( $recurrence ) );
237
+				return sprintf(__('Every %s', 'action-scheduler'), self::human_interval($recurrence));
238 238
 			} else {
239 239
 				return $recurrence;
240 240
 			}
241 241
 		}
242 242
 
243
-		return __( 'Non-repeating', 'action-scheduler' );
243
+		return __('Non-repeating', 'action-scheduler');
244 244
 	}
245 245
 
246 246
 	/**
@@ -250,18 +250,18 @@  discard block
 block discarded – undo
250 250
 	 *
251 251
 	 * @return string
252 252
 	 */
253
-	public function column_args( array $row ) {
254
-		if ( empty( $row['args'] ) ) {
255
-			return apply_filters( 'action_scheduler_list_table_column_args', '', $row );
253
+	public function column_args(array $row) {
254
+		if (empty($row['args'])) {
255
+			return apply_filters('action_scheduler_list_table_column_args', '', $row);
256 256
 		}
257 257
 
258 258
 		$row_html = '<ul>';
259
-		foreach ( $row['args'] as $key => $value ) {
260
-			$row_html .= sprintf( '<li><code>%s => %s</code></li>', esc_html( var_export( $key, true ) ), esc_html( var_export( $value, true ) ) );
259
+		foreach ($row['args'] as $key => $value) {
260
+			$row_html .= sprintf('<li><code>%s => %s</code></li>', esc_html(var_export($key, true)), esc_html(var_export($value, true)));
261 261
 		}
262 262
 		$row_html .= '</ul>';
263 263
 
264
-		return apply_filters( 'action_scheduler_list_table_column_args', $row_html, $row );
264
+		return apply_filters('action_scheduler_list_table_column_args', $row_html, $row);
265 265
 	}
266 266
 
267 267
 	/**
@@ -270,14 +270,14 @@  discard block
 block discarded – undo
270 270
 	 * @param array $row Action array.
271 271
 	 * @return string
272 272
 	 */
273
-	public function column_log_entries( array $row ) {
273
+	public function column_log_entries(array $row) {
274 274
 
275 275
 		$log_entries_html = '<ol>';
276 276
 
277
-		$timezone = new DateTimezone( 'UTC' );
277
+		$timezone = new DateTimezone('UTC');
278 278
 
279
-		foreach ( $row['log_entries'] as $log_entry ) {
280
-			$log_entries_html .= $this->get_log_entry_html( $log_entry, $timezone );
279
+		foreach ($row['log_entries'] as $log_entry) {
280
+			$log_entries_html .= $this->get_log_entry_html($log_entry, $timezone);
281 281
 		}
282 282
 
283 283
 		$log_entries_html .= '</ol>';
@@ -292,10 +292,10 @@  discard block
 block discarded – undo
292 292
 	 * @param DateTimezone $timezone
293 293
 	 * @return string
294 294
 	 */
295
-	protected function get_log_entry_html( ActionScheduler_LogEntry $log_entry, DateTimezone $timezone ) {
295
+	protected function get_log_entry_html(ActionScheduler_LogEntry $log_entry, DateTimezone $timezone) {
296 296
 		$date = $log_entry->get_date();
297
-		$date->setTimezone( $timezone );
298
-		return sprintf( '<li><strong>%s</strong><br/>%s</li>', esc_html( $date->format( 'Y-m-d H:i:s O' ) ), esc_html( $log_entry->get_message() ) );
297
+		$date->setTimezone($timezone);
298
+		return sprintf('<li><strong>%s</strong><br/>%s</li>', esc_html($date->format('Y-m-d H:i:s O')), esc_html($log_entry->get_message()));
299 299
 	}
300 300
 
301 301
 	/**
@@ -306,9 +306,9 @@  discard block
 block discarded – undo
306 306
 	 *
307 307
 	 * @return string
308 308
 	 */
309
-	protected function maybe_render_actions( $row, $column_name ) {
310
-		if ( 'pending' === strtolower( $row['status'] ) ) {
311
-			return parent::maybe_render_actions( $row, $column_name );
309
+	protected function maybe_render_actions($row, $column_name) {
310
+		if ('pending' === strtolower($row['status'])) {
311
+			return parent::maybe_render_actions($row, $column_name);
312 312
 		}
313 313
 
314 314
 		return '';
@@ -325,7 +325,7 @@  discard block
 block discarded – undo
325 325
 	public function display_admin_notices() {
326 326
 		global $wpdb;
327 327
 
328
-		if ( ( is_a( $this->store, 'ActionScheduler_HybridStore' ) || is_a( $this->store, 'ActionScheduler_DBStore' ) ) && apply_filters( 'action_scheduler_enable_recreate_data_store', true ) ) {
328
+		if ((is_a($this->store, 'ActionScheduler_HybridStore') || is_a($this->store, 'ActionScheduler_DBStore')) && apply_filters('action_scheduler_enable_recreate_data_store', true)) {
329 329
 			$table_list = array(
330 330
 				'actionscheduler_actions',
331 331
 				'actionscheduler_logs',
@@ -333,12 +333,12 @@  discard block
 block discarded – undo
333 333
 				'actionscheduler_claims',
334 334
 			);
335 335
 
336
-			$found_tables = $wpdb->get_col( "SHOW TABLES LIKE '{$wpdb->prefix}actionscheduler%'" ); // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
337
-			foreach ( $table_list as $table_name ) {
338
-				if ( ! in_array( $wpdb->prefix . $table_name, $found_tables ) ) {
336
+			$found_tables = $wpdb->get_col("SHOW TABLES LIKE '{$wpdb->prefix}actionscheduler%'"); // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
337
+			foreach ($table_list as $table_name) {
338
+				if (!in_array($wpdb->prefix . $table_name, $found_tables)) {
339 339
 					$this->admin_notices[] = array(
340 340
 						'class'   => 'error',
341
-						'message' => __( 'It appears one or more database tables were missing. Attempting to re-create the missing table(s).' , 'action-scheduler' ),
341
+						'message' => __('It appears one or more database tables were missing. Attempting to re-create the missing table(s).', 'action-scheduler'),
342 342
 					);
343 343
 					$this->recreate_tables();
344 344
 					parent::display_admin_notices();
@@ -348,7 +348,7 @@  discard block
 block discarded – undo
348 348
 			}
349 349
 		}
350 350
 
351
-		if ( $this->runner->has_maximum_concurrent_batches() ) {
351
+		if ($this->runner->has_maximum_concurrent_batches()) {
352 352
 			$claim_count           = $this->store->get_claim_count();
353 353
 			$this->admin_notices[] = array(
354 354
 				'class'   => 'updated',
@@ -363,18 +363,18 @@  discard block
 block discarded – undo
363 363
 					$claim_count
364 364
 				),
365 365
 			);
366
-		} elseif ( $this->store->has_pending_actions_due() ) {
366
+		} elseif ($this->store->has_pending_actions_due()) {
367 367
 
368
-			$async_request_lock_expiration = ActionScheduler::lock()->get_expiration( 'async-request-runner' );
368
+			$async_request_lock_expiration = ActionScheduler::lock()->get_expiration('async-request-runner');
369 369
 
370 370
 			// No lock set or lock expired
371
-			if ( false === $async_request_lock_expiration || $async_request_lock_expiration < time() ) {
372
-				$in_progress_url       = add_query_arg( 'status', 'in-progress', remove_query_arg( 'status' ) );
371
+			if (false === $async_request_lock_expiration || $async_request_lock_expiration < time()) {
372
+				$in_progress_url       = add_query_arg('status', 'in-progress', remove_query_arg('status'));
373 373
 				/* translators: %s: process URL */
374
-				$async_request_message = sprintf( __( 'A new queue has begun processing. <a href="%s">View actions in-progress &raquo;</a>', 'action-scheduler' ), esc_url( $in_progress_url ) );
374
+				$async_request_message = sprintf(__('A new queue has begun processing. <a href="%s">View actions in-progress &raquo;</a>', 'action-scheduler'), esc_url($in_progress_url));
375 375
 			} else {
376 376
 				/* translators: %d: seconds */
377
-				$async_request_message = sprintf( __( 'The next queue will begin processing in approximately %d seconds.', 'action-scheduler' ), $async_request_lock_expiration - time() );
377
+				$async_request_message = sprintf(__('The next queue will begin processing in approximately %d seconds.', 'action-scheduler'), $async_request_lock_expiration - time());
378 378
 			}
379 379
 
380 380
 			$this->admin_notices[] = array(
@@ -383,36 +383,36 @@  discard block
 block discarded – undo
383 383
 			);
384 384
 		}
385 385
 
386
-		$notification = get_transient( 'action_scheduler_admin_notice' );
386
+		$notification = get_transient('action_scheduler_admin_notice');
387 387
 
388
-		if ( is_array( $notification ) ) {
389
-			delete_transient( 'action_scheduler_admin_notice' );
388
+		if (is_array($notification)) {
389
+			delete_transient('action_scheduler_admin_notice');
390 390
 
391
-			$action = $this->store->fetch_action( $notification['action_id'] );
391
+			$action = $this->store->fetch_action($notification['action_id']);
392 392
 			$action_hook_html = '<strong><code>' . $action->get_hook() . '</code></strong>';
393
-			if ( 1 == $notification['success'] ) {
393
+			if (1 == $notification['success']) {
394 394
 				$class = 'updated';
395
-				switch ( $notification['row_action_type'] ) {
395
+				switch ($notification['row_action_type']) {
396 396
 					case 'run' :
397 397
 						/* translators: %s: action HTML */
398
-						$action_message_html = sprintf( __( 'Successfully executed action: %s', 'action-scheduler' ), $action_hook_html );
398
+						$action_message_html = sprintf(__('Successfully executed action: %s', 'action-scheduler'), $action_hook_html);
399 399
 						break;
400 400
 					case 'cancel' :
401 401
 						/* translators: %s: action HTML */
402
-						$action_message_html = sprintf( __( 'Successfully canceled action: %s', 'action-scheduler' ), $action_hook_html );
402
+						$action_message_html = sprintf(__('Successfully canceled action: %s', 'action-scheduler'), $action_hook_html);
403 403
 						break;
404 404
 					default :
405 405
 						/* translators: %s: action HTML */
406
-						$action_message_html = sprintf( __( 'Successfully processed change for action: %s', 'action-scheduler' ), $action_hook_html );
406
+						$action_message_html = sprintf(__('Successfully processed change for action: %s', 'action-scheduler'), $action_hook_html);
407 407
 						break;
408 408
 				}
409 409
 			} else {
410 410
 				$class = 'error';
411 411
 				/* translators: 1: action HTML 2: action ID 3: error message */
412
-				$action_message_html = sprintf( __( 'Could not process change for action: "%1$s" (ID: %2$d). Error: %3$s', 'action-scheduler' ), $action_hook_html, esc_html( $notification['action_id'] ), esc_html( $notification['error_message'] ) );
412
+				$action_message_html = sprintf(__('Could not process change for action: "%1$s" (ID: %2$d). Error: %3$s', 'action-scheduler'), $action_hook_html, esc_html($notification['action_id']), esc_html($notification['error_message']));
413 413
 			}
414 414
 
415
-			$action_message_html = apply_filters( 'action_scheduler_admin_notice_html', $action_message_html, $action, $notification );
415
+			$action_message_html = apply_filters('action_scheduler_admin_notice_html', $action_message_html, $action, $notification);
416 416
 
417 417
 			$this->admin_notices[] = array(
418 418
 				'class'   => $class,
@@ -430,8 +430,8 @@  discard block
 block discarded – undo
430 430
 	 *
431 431
 	 * @return string
432 432
 	 */
433
-	public function column_schedule( $row ) {
434
-		return $this->get_schedule_display_string( $row['schedule'] );
433
+	public function column_schedule($row) {
434
+		return $this->get_schedule_display_string($row['schedule']);
435 435
 	}
436 436
 
437 437
 	/**
@@ -440,25 +440,25 @@  discard block
 block discarded – undo
440 440
 	 * @param ActionScheduler_Schedule $schedule
441 441
 	 * @return string
442 442
 	 */
443
-	protected function get_schedule_display_string( ActionScheduler_Schedule $schedule ) {
443
+	protected function get_schedule_display_string(ActionScheduler_Schedule $schedule) {
444 444
 
445 445
 		$schedule_display_string = '';
446 446
 
447
-		if ( ! $schedule->get_date() ) {
447
+		if (!$schedule->get_date()) {
448 448
 			return '0000-00-00 00:00:00';
449 449
 		}
450 450
 
451 451
 		$next_timestamp = $schedule->get_date()->getTimestamp();
452 452
 
453
-		$schedule_display_string .= $schedule->get_date()->format( 'Y-m-d H:i:s O' );
453
+		$schedule_display_string .= $schedule->get_date()->format('Y-m-d H:i:s O');
454 454
 		$schedule_display_string .= '<br/>';
455 455
 
456
-		if ( gmdate( 'U' ) > $next_timestamp ) {
456
+		if (gmdate('U') > $next_timestamp) {
457 457
 			/* translators: %s: date interval */
458
-			$schedule_display_string .= sprintf( __( ' (%s ago)', 'action-scheduler' ), self::human_interval( gmdate( 'U' ) - $next_timestamp ) );
458
+			$schedule_display_string .= sprintf(__(' (%s ago)', 'action-scheduler'), self::human_interval(gmdate('U') - $next_timestamp));
459 459
 		} else {
460 460
 			/* translators: %s: date interval */
461
-			$schedule_display_string .= sprintf( __( ' (%s)', 'action-scheduler' ), self::human_interval( $next_timestamp - gmdate( 'U' ) ) );
461
+			$schedule_display_string .= sprintf(__(' (%s)', 'action-scheduler'), self::human_interval($next_timestamp - gmdate('U')));
462 462
 		}
463 463
 
464 464
 		return $schedule_display_string;
@@ -473,9 +473,9 @@  discard block
 block discarded – undo
473 473
 	 * @param array $ids
474 474
 	 * @param string $ids_sql Inherited and unused
475 475
 	 */
476
-	protected function bulk_delete( array $ids, $ids_sql ) {
477
-		foreach ( $ids as $id ) {
478
-			$this->store->delete_action( $id );
476
+	protected function bulk_delete(array $ids, $ids_sql) {
477
+		foreach ($ids as $id) {
478
+			$this->store->delete_action($id);
479 479
 		}
480 480
 	}
481 481
 
@@ -485,8 +485,8 @@  discard block
 block discarded – undo
485 485
 	 *
486 486
 	 * @param int $action_id
487 487
 	 */
488
-	protected function row_action_cancel( $action_id ) {
489
-		$this->process_row_action( $action_id, 'cancel' );
488
+	protected function row_action_cancel($action_id) {
489
+		$this->process_row_action($action_id, 'cancel');
490 490
 	}
491 491
 
492 492
 	/**
@@ -495,27 +495,27 @@  discard block
 block discarded – undo
495 495
 	 *
496 496
 	 * @param int $action_id
497 497
 	 */
498
-	protected function row_action_run( $action_id ) {
499
-		$this->process_row_action( $action_id, 'run' );
498
+	protected function row_action_run($action_id) {
499
+		$this->process_row_action($action_id, 'run');
500 500
 	}
501 501
 
502 502
 	/**
503 503
 	 * Force the data store schema updates.
504 504
 	 */
505 505
 	protected function recreate_tables() {
506
-		if ( is_a( $this->store, 'ActionScheduler_HybridStore' ) ) {
506
+		if (is_a($this->store, 'ActionScheduler_HybridStore')) {
507 507
 			$store = $this->store;
508 508
 		} else {
509 509
 			$store = new ActionScheduler_HybridStore();
510 510
 		}
511
-		add_action( 'action_scheduler/created_table', array( $store, 'set_autoincrement' ), 10, 2 );
511
+		add_action('action_scheduler/created_table', array($store, 'set_autoincrement'), 10, 2);
512 512
 
513 513
 		$store_schema  = new ActionScheduler_StoreSchema();
514 514
 		$logger_schema = new ActionScheduler_LoggerSchema();
515
-		$store_schema->register_tables( true );
516
-		$logger_schema->register_tables( true );
515
+		$store_schema->register_tables(true);
516
+		$logger_schema->register_tables(true);
517 517
 
518
-		remove_action( 'action_scheduler/created_table', array( $store, 'set_autoincrement' ), 10 );
518
+		remove_action('action_scheduler/created_table', array($store, 'set_autoincrement'), 10);
519 519
 	}
520 520
 	/**
521 521
 	 * Implements the logic behind processing an action once an action link is clicked on the list table.
@@ -523,24 +523,24 @@  discard block
 block discarded – undo
523 523
 	 * @param int $action_id
524 524
 	 * @param string $row_action_type The type of action to perform on the action.
525 525
 	 */
526
-	protected function process_row_action( $action_id, $row_action_type ) {
526
+	protected function process_row_action($action_id, $row_action_type) {
527 527
 		try {
528
-			switch ( $row_action_type ) {
528
+			switch ($row_action_type) {
529 529
 				case 'run' :
530
-					$this->runner->process_action( $action_id, 'Admin List Table' );
530
+					$this->runner->process_action($action_id, 'Admin List Table');
531 531
 					break;
532 532
 				case 'cancel' :
533
-					$this->store->cancel_action( $action_id );
533
+					$this->store->cancel_action($action_id);
534 534
 					break;
535 535
 			}
536 536
 			$success = 1;
537 537
 			$error_message = '';
538
-		} catch ( Exception $e ) {
538
+		} catch (Exception $e) {
539 539
 			$success = 0;
540 540
 			$error_message = $e->getMessage();
541 541
 		}
542 542
 
543
-		set_transient( 'action_scheduler_admin_notice', compact( 'action_id', 'success', 'error_message', 'row_action_type' ), 30 );
543
+		set_transient('action_scheduler_admin_notice', compact('action_id', 'success', 'error_message', 'row_action_type'), 30);
544 544
 	}
545 545
 
546 546
 	/**
@@ -549,7 +549,7 @@  discard block
 block discarded – undo
549 549
 	public function prepare_items() {
550 550
 		$this->prepare_column_headers();
551 551
 
552
-		$per_page = $this->get_items_per_page( $this->package . '_items_per_page', $this->items_per_page );
552
+		$per_page = $this->get_items_per_page($this->package . '_items_per_page', $this->items_per_page);
553 553
 		$query = array(
554 554
 			'per_page' => $per_page,
555 555
 			'offset'   => $this->get_items_offset(),
@@ -561,37 +561,37 @@  discard block
 block discarded – undo
561 561
 
562 562
 		$this->items = array();
563 563
 
564
-		$total_items = $this->store->query_actions( $query, 'count' );
564
+		$total_items = $this->store->query_actions($query, 'count');
565 565
 
566 566
 		$status_labels = $this->store->get_status_labels();
567 567
 
568
-		foreach ( $this->store->query_actions( $query ) as $action_id ) {
568
+		foreach ($this->store->query_actions($query) as $action_id) {
569 569
 			try {
570
-				$action = $this->store->fetch_action( $action_id );
571
-			} catch ( Exception $e ) {
570
+				$action = $this->store->fetch_action($action_id);
571
+			} catch (Exception $e) {
572 572
 				continue;
573 573
 			}
574
-			if ( is_a( $action, 'ActionScheduler_NullAction' ) ) {
574
+			if (is_a($action, 'ActionScheduler_NullAction')) {
575 575
 				continue;
576 576
 			}
577
-			$this->items[ $action_id ] = array(
577
+			$this->items[$action_id] = array(
578 578
 				'ID'          => $action_id,
579 579
 				'hook'        => $action->get_hook(),
580
-				'status'      => $status_labels[ $this->store->get_status( $action_id ) ],
580
+				'status'      => $status_labels[$this->store->get_status($action_id)],
581 581
 				'args'        => $action->get_args(),
582 582
 				'group'       => $action->get_group(),
583
-				'log_entries' => $this->logger->get_logs( $action_id ),
584
-				'claim_id'    => $this->store->get_claim_id( $action_id ),
585
-				'recurrence'  => $this->get_recurrence( $action ),
583
+				'log_entries' => $this->logger->get_logs($action_id),
584
+				'claim_id'    => $this->store->get_claim_id($action_id),
585
+				'recurrence'  => $this->get_recurrence($action),
586 586
 				'schedule'    => $action->get_schedule(),
587 587
 			);
588 588
 		}
589 589
 
590
-		$this->set_pagination_args( array(
590
+		$this->set_pagination_args(array(
591 591
 			'total_items' => $total_items,
592 592
 			'per_page'    => $per_page,
593
-			'total_pages' => ceil( $total_items / $per_page ),
594
-		) );
593
+			'total_pages' => ceil($total_items / $per_page),
594
+		));
595 595
 	}
596 596
 
597 597
 	/**
@@ -606,6 +606,6 @@  discard block
 block discarded – undo
606 606
 	 * Get the text to display in the search box on the list table.
607 607
 	 */
608 608
 	protected function get_search_box_button_text() {
609
-		return __( 'Search hook, args and claim ID', 'action-scheduler' );
609
+		return __('Search hook, args and claim ID', 'action-scheduler');
610 610
 	}
611 611
 }
Please login to merge, or discard this patch.