@@ -24,7 +24,7 @@ discard block |
||
24 | 24 | * |
25 | 25 | * @param ActionScheduler_Store $source_store Source store object. |
26 | 26 | */ |
27 | - public function __construct( Store $source_store ) { |
|
27 | + public function __construct(Store $source_store) { |
|
28 | 28 | $this->store = $source_store; |
29 | 29 | } |
30 | 30 | |
@@ -35,10 +35,10 @@ discard block |
||
35 | 35 | * |
36 | 36 | * @return int[] A list of action IDs |
37 | 37 | */ |
38 | - public function fetch( $count = 10 ) { |
|
39 | - foreach ( $this->get_query_strategies( $count ) as $query ) { |
|
40 | - $action_ids = $this->store->query_actions( $query ); |
|
41 | - if ( ! empty( $action_ids ) ) { |
|
38 | + public function fetch($count = 10) { |
|
39 | + foreach ($this->get_query_strategies($count) as $query) { |
|
40 | + $action_ids = $this->store->query_actions($query); |
|
41 | + if (!empty($action_ids)) { |
|
42 | 42 | return $action_ids; |
43 | 43 | } |
44 | 44 | } |
@@ -53,7 +53,7 @@ discard block |
||
53 | 53 | * |
54 | 54 | * @return array |
55 | 55 | */ |
56 | - private function get_query_strategies( $count ) { |
|
56 | + private function get_query_strategies($count) { |
|
57 | 57 | $now = as_get_datetime_object(); |
58 | 58 | $args = [ |
59 | 59 | 'date' => $now, |
@@ -72,15 +72,15 @@ discard block |
||
72 | 72 | '', // any other unanticipated status |
73 | 73 | ]; |
74 | 74 | |
75 | - foreach ( $priorities as $status ) { |
|
76 | - yield wp_parse_args( [ |
|
75 | + foreach ($priorities as $status) { |
|
76 | + yield wp_parse_args([ |
|
77 | 77 | 'status' => $status, |
78 | 78 | 'date_compare' => '<=', |
79 | - ], $args ); |
|
80 | - yield wp_parse_args( [ |
|
79 | + ], $args); |
|
80 | + yield wp_parse_args([ |
|
81 | 81 | 'status' => $status, |
82 | 82 | 'date_compare' => '>=', |
83 | - ], $args ); |
|
83 | + ], $args); |
|
84 | 84 | } |
85 | 85 | } |
86 | 86 | } |
87 | 87 | \ No newline at end of file |
@@ -48,8 +48,8 @@ discard block |
||
48 | 48 | * @return ActionScheduler_Store |
49 | 49 | */ |
50 | 50 | public function get_source_store() { |
51 | - if ( empty( $this->source_store ) ) { |
|
52 | - throw new \RuntimeException( __( 'Source store must be configured before running a migration', 'action-scheduler' ) ); |
|
51 | + if (empty($this->source_store)) { |
|
52 | + throw new \RuntimeException(__('Source store must be configured before running a migration', 'action-scheduler')); |
|
53 | 53 | } |
54 | 54 | |
55 | 55 | return $this->source_store; |
@@ -60,7 +60,7 @@ discard block |
||
60 | 60 | * |
61 | 61 | * @param ActionScheduler_Store $store Source store object. |
62 | 62 | */ |
63 | - public function set_source_store( Store $store ) { |
|
63 | + public function set_source_store(Store $store) { |
|
64 | 64 | $this->source_store = $store; |
65 | 65 | } |
66 | 66 | |
@@ -70,8 +70,8 @@ discard block |
||
70 | 70 | * @return ActionScheduler_Logger |
71 | 71 | */ |
72 | 72 | public function get_source_logger() { |
73 | - if ( empty( $this->source_logger ) ) { |
|
74 | - throw new \RuntimeException( __( 'Source logger must be configured before running a migration', 'action-scheduler' ) ); |
|
73 | + if (empty($this->source_logger)) { |
|
74 | + throw new \RuntimeException(__('Source logger must be configured before running a migration', 'action-scheduler')); |
|
75 | 75 | } |
76 | 76 | |
77 | 77 | return $this->source_logger; |
@@ -82,7 +82,7 @@ discard block |
||
82 | 82 | * |
83 | 83 | * @param ActionScheduler_Logger $logger |
84 | 84 | */ |
85 | - public function set_source_logger( Logger $logger ) { |
|
85 | + public function set_source_logger(Logger $logger) { |
|
86 | 86 | $this->source_logger = $logger; |
87 | 87 | } |
88 | 88 | |
@@ -92,8 +92,8 @@ discard block |
||
92 | 92 | * @return ActionScheduler_Store |
93 | 93 | */ |
94 | 94 | public function get_destination_store() { |
95 | - if ( empty( $this->destination_store ) ) { |
|
96 | - throw new \RuntimeException( __( 'Destination store must be configured before running a migration', 'action-scheduler' ) ); |
|
95 | + if (empty($this->destination_store)) { |
|
96 | + throw new \RuntimeException(__('Destination store must be configured before running a migration', 'action-scheduler')); |
|
97 | 97 | } |
98 | 98 | |
99 | 99 | return $this->destination_store; |
@@ -104,7 +104,7 @@ discard block |
||
104 | 104 | * |
105 | 105 | * @param ActionScheduler_Store $store |
106 | 106 | */ |
107 | - public function set_destination_store( Store $store ) { |
|
107 | + public function set_destination_store(Store $store) { |
|
108 | 108 | $this->destination_store = $store; |
109 | 109 | } |
110 | 110 | |
@@ -114,8 +114,8 @@ discard block |
||
114 | 114 | * @return ActionScheduler_Logger |
115 | 115 | */ |
116 | 116 | public function get_destination_logger() { |
117 | - if ( empty( $this->destination_logger ) ) { |
|
118 | - throw new \RuntimeException( __( 'Destination logger must be configured before running a migration', 'action-scheduler' ) ); |
|
117 | + if (empty($this->destination_logger)) { |
|
118 | + throw new \RuntimeException(__('Destination logger must be configured before running a migration', 'action-scheduler')); |
|
119 | 119 | } |
120 | 120 | |
121 | 121 | return $this->destination_logger; |
@@ -126,7 +126,7 @@ discard block |
||
126 | 126 | * |
127 | 127 | * @param ActionScheduler_Logger $logger |
128 | 128 | */ |
129 | - public function set_destination_logger( Logger $logger ) { |
|
129 | + public function set_destination_logger(Logger $logger) { |
|
130 | 130 | $this->destination_logger = $logger; |
131 | 131 | } |
132 | 132 | |
@@ -144,7 +144,7 @@ discard block |
||
144 | 144 | * |
145 | 145 | * @param bool $dry_run |
146 | 146 | */ |
147 | - public function set_dry_run( $dry_run ) { |
|
147 | + public function set_dry_run($dry_run) { |
|
148 | 148 | $this->dry_run = (bool) $dry_run; |
149 | 149 | } |
150 | 150 | |
@@ -162,7 +162,7 @@ discard block |
||
162 | 162 | * |
163 | 163 | * @param ActionScheduler\WPCLI\ProgressBar $progress_bar |
164 | 164 | */ |
165 | - public function set_progress_bar( ProgressBar $progress_bar ) { |
|
165 | + public function set_progress_bar(ProgressBar $progress_bar) { |
|
166 | 166 | $this->progress_bar = $progress_bar; |
167 | 167 | } |
168 | 168 | } |
@@ -42,22 +42,22 @@ discard block |
||
42 | 42 | * |
43 | 43 | * @param Config $config Migration configuration object. |
44 | 44 | */ |
45 | - public function __construct( Config $config ) { |
|
45 | + public function __construct(Config $config) { |
|
46 | 46 | $this->source_store = $config->get_source_store(); |
47 | 47 | $this->destination_store = $config->get_destination_store(); |
48 | 48 | $this->source_logger = $config->get_source_logger(); |
49 | 49 | $this->destination_logger = $config->get_destination_logger(); |
50 | 50 | |
51 | - $this->batch_fetcher = new BatchFetcher( $this->source_store ); |
|
52 | - if ( $config->get_dry_run() ) { |
|
53 | - $this->log_migrator = new DryRun_LogMigrator( $this->source_logger, $this->destination_logger ); |
|
54 | - $this->action_migrator = new DryRun_ActionMigrator( $this->source_store, $this->destination_store, $this->log_migrator ); |
|
51 | + $this->batch_fetcher = new BatchFetcher($this->source_store); |
|
52 | + if ($config->get_dry_run()) { |
|
53 | + $this->log_migrator = new DryRun_LogMigrator($this->source_logger, $this->destination_logger); |
|
54 | + $this->action_migrator = new DryRun_ActionMigrator($this->source_store, $this->destination_store, $this->log_migrator); |
|
55 | 55 | } else { |
56 | - $this->log_migrator = new LogMigrator( $this->source_logger, $this->destination_logger ); |
|
57 | - $this->action_migrator = new ActionMigrator( $this->source_store, $this->destination_store, $this->log_migrator ); |
|
56 | + $this->log_migrator = new LogMigrator($this->source_logger, $this->destination_logger); |
|
57 | + $this->action_migrator = new ActionMigrator($this->source_store, $this->destination_store, $this->log_migrator); |
|
58 | 58 | } |
59 | 59 | |
60 | - if ( defined( 'WP_CLI' ) && WP_CLI ) { |
|
60 | + if (defined('WP_CLI') && WP_CLI) { |
|
61 | 61 | $this->progress_bar = $config->get_progress_bar(); |
62 | 62 | } |
63 | 63 | } |
@@ -69,21 +69,21 @@ discard block |
||
69 | 69 | * |
70 | 70 | * @return int Size of batch processed. |
71 | 71 | */ |
72 | - public function run( $batch_size = 10 ) { |
|
73 | - $batch = $this->batch_fetcher->fetch( $batch_size ); |
|
74 | - $batch_size = count( $batch ); |
|
72 | + public function run($batch_size = 10) { |
|
73 | + $batch = $this->batch_fetcher->fetch($batch_size); |
|
74 | + $batch_size = count($batch); |
|
75 | 75 | |
76 | - if ( ! $batch_size ) { |
|
76 | + if (!$batch_size) { |
|
77 | 77 | return 0; |
78 | 78 | } |
79 | 79 | |
80 | - if ( $this->progress_bar ) { |
|
80 | + if ($this->progress_bar) { |
|
81 | 81 | /* translators: %d: amount of actions */ |
82 | - $this->progress_bar->set_message( sprintf( _n( 'Migrating %d action', 'Migrating %d actions', $batch_size, 'action-scheduler' ), number_format_i18n( $batch_size ) ) ); |
|
83 | - $this->progress_bar->set_count( $batch_size ); |
|
82 | + $this->progress_bar->set_message(sprintf(_n('Migrating %d action', 'Migrating %d actions', $batch_size, 'action-scheduler'), number_format_i18n($batch_size))); |
|
83 | + $this->progress_bar->set_count($batch_size); |
|
84 | 84 | } |
85 | 85 | |
86 | - $this->migrate_actions( $batch ); |
|
86 | + $this->migrate_actions($batch); |
|
87 | 87 | |
88 | 88 | return $batch_size; |
89 | 89 | } |
@@ -93,37 +93,37 @@ discard block |
||
93 | 93 | * |
94 | 94 | * @param array $action_ids List of action IDs to migrate. |
95 | 95 | */ |
96 | - public function migrate_actions( array $action_ids ) { |
|
97 | - do_action( 'action_scheduler/migration_batch_starting', $action_ids ); |
|
96 | + public function migrate_actions(array $action_ids) { |
|
97 | + do_action('action_scheduler/migration_batch_starting', $action_ids); |
|
98 | 98 | |
99 | 99 | \ActionScheduler::logger()->unhook_stored_action(); |
100 | 100 | $this->destination_logger->unhook_stored_action(); |
101 | 101 | |
102 | - foreach ( $action_ids as $source_action_id ) { |
|
103 | - $destination_action_id = $this->action_migrator->migrate( $source_action_id ); |
|
104 | - if ( $destination_action_id ) { |
|
105 | - $this->destination_logger->log( $destination_action_id, sprintf( |
|
102 | + foreach ($action_ids as $source_action_id) { |
|
103 | + $destination_action_id = $this->action_migrator->migrate($source_action_id); |
|
104 | + if ($destination_action_id) { |
|
105 | + $this->destination_logger->log($destination_action_id, sprintf( |
|
106 | 106 | /* translators: 1: source action ID 2: source store class 3: destination action ID 4: destination store class */ |
107 | - __( 'Migrated action with ID %1$d in %2$s to ID %3$d in %4$s', 'action-scheduler' ), |
|
107 | + __('Migrated action with ID %1$d in %2$s to ID %3$d in %4$s', 'action-scheduler'), |
|
108 | 108 | $source_action_id, |
109 | - get_class( $this->source_store ), |
|
109 | + get_class($this->source_store), |
|
110 | 110 | $destination_action_id, |
111 | - get_class( $this->destination_store ) |
|
112 | - ) ); |
|
111 | + get_class($this->destination_store) |
|
112 | + )); |
|
113 | 113 | } |
114 | 114 | |
115 | - if ( $this->progress_bar ) { |
|
115 | + if ($this->progress_bar) { |
|
116 | 116 | $this->progress_bar->tick(); |
117 | 117 | } |
118 | 118 | } |
119 | 119 | |
120 | - if ( $this->progress_bar ) { |
|
120 | + if ($this->progress_bar) { |
|
121 | 121 | $this->progress_bar->finish(); |
122 | 122 | } |
123 | 123 | |
124 | 124 | \ActionScheduler::logger()->hook_stored_action(); |
125 | 125 | |
126 | - do_action( 'action_scheduler/migration_batch_complete', $action_ids ); |
|
126 | + do_action('action_scheduler/migration_batch_complete', $action_ids); |
|
127 | 127 | } |
128 | 128 | |
129 | 129 | /** |
@@ -27,7 +27,7 @@ discard block |
||
27 | 27 | * @param ActionScheduler_Logger $source_logger Source logger object. |
28 | 28 | * @param ActionScheduler_Logger $destination_Logger Destination logger object. |
29 | 29 | */ |
30 | - public function __construct( ActionScheduler_Logger $source_logger, ActionScheduler_Logger $destination_Logger ) { |
|
30 | + public function __construct(ActionScheduler_Logger $source_logger, ActionScheduler_Logger $destination_Logger) { |
|
31 | 31 | $this->source = $source_logger; |
32 | 32 | $this->destination = $destination_Logger; |
33 | 33 | } |
@@ -38,11 +38,11 @@ discard block |
||
38 | 38 | * @param int $source_action_id Source logger object. |
39 | 39 | * @param int $destination_action_id Destination logger object. |
40 | 40 | */ |
41 | - public function migrate( $source_action_id, $destination_action_id ) { |
|
42 | - $logs = $this->source->get_logs( $source_action_id ); |
|
43 | - foreach ( $logs as $log ) { |
|
44 | - if ( $log->get_action_id() == $source_action_id ) { |
|
45 | - $this->destination->log( $destination_action_id, $log->get_message(), $log->get_date() ); |
|
41 | + public function migrate($source_action_id, $destination_action_id) { |
|
42 | + $logs = $this->source->get_logs($source_action_id); |
|
43 | + foreach ($logs as $log) { |
|
44 | + if ($log->get_action_id() == $source_action_id) { |
|
45 | + $this->destination->log($destination_action_id, $log->get_message(), $log->get_date()); |
|
46 | 46 | } |
47 | 47 | } |
48 | 48 | } |
@@ -23,14 +23,14 @@ discard block |
||
23 | 23 | * Set up the callback for the scheduled job. |
24 | 24 | */ |
25 | 25 | public function hook() { |
26 | - add_action( self::HOOK, array( $this, 'run_migration' ), 10, 0 ); |
|
26 | + add_action(self::HOOK, array($this, 'run_migration'), 10, 0); |
|
27 | 27 | } |
28 | 28 | |
29 | 29 | /** |
30 | 30 | * Remove the callback for the scheduled job. |
31 | 31 | */ |
32 | 32 | public function unhook() { |
33 | - remove_action( self::HOOK, array( $this, 'run_migration' ), 10 ); |
|
33 | + remove_action(self::HOOK, array($this, 'run_migration'), 10); |
|
34 | 34 | } |
35 | 35 | |
36 | 36 | /** |
@@ -38,12 +38,12 @@ discard block |
||
38 | 38 | */ |
39 | 39 | public function run_migration() { |
40 | 40 | $migration_runner = $this->get_migration_runner(); |
41 | - $count = $migration_runner->run( $this->get_batch_size() ); |
|
41 | + $count = $migration_runner->run($this->get_batch_size()); |
|
42 | 42 | |
43 | - if ( $count === 0 ) { |
|
43 | + if ($count === 0) { |
|
44 | 44 | $this->mark_complete(); |
45 | 45 | } else { |
46 | - $this->schedule_migration( time() + $this->get_schedule_interval() ); |
|
46 | + $this->schedule_migration(time() + $this->get_schedule_interval()); |
|
47 | 47 | } |
48 | 48 | } |
49 | 49 | |
@@ -54,7 +54,7 @@ discard block |
||
54 | 54 | $this->unschedule_migration(); |
55 | 55 | |
56 | 56 | \ActionScheduler_DataController::mark_migration_complete(); |
57 | - do_action( 'action_scheduler/migration_complete' ); |
|
57 | + do_action('action_scheduler/migration_complete'); |
|
58 | 58 | } |
59 | 59 | |
60 | 60 | /** |
@@ -63,9 +63,9 @@ discard block |
||
63 | 63 | * @return bool Whether there is a pending action in the store to handle the migration |
64 | 64 | */ |
65 | 65 | public function is_migration_scheduled() { |
66 | - $next = as_next_scheduled_action( self::HOOK ); |
|
66 | + $next = as_next_scheduled_action(self::HOOK); |
|
67 | 67 | |
68 | - return ! empty( $next ); |
|
68 | + return !empty($next); |
|
69 | 69 | } |
70 | 70 | |
71 | 71 | /** |
@@ -75,25 +75,25 @@ discard block |
||
75 | 75 | * |
76 | 76 | * @return string The action ID |
77 | 77 | */ |
78 | - public function schedule_migration( $when = 0 ) { |
|
79 | - $next = as_next_scheduled_action( self::HOOK ); |
|
78 | + public function schedule_migration($when = 0) { |
|
79 | + $next = as_next_scheduled_action(self::HOOK); |
|
80 | 80 | |
81 | - if ( ! empty( $next ) ) { |
|
81 | + if (!empty($next)) { |
|
82 | 82 | return $next; |
83 | 83 | } |
84 | 84 | |
85 | - if ( empty( $when ) ) { |
|
85 | + if (empty($when)) { |
|
86 | 86 | $when = time(); |
87 | 87 | } |
88 | 88 | |
89 | - return as_schedule_single_action( $when, self::HOOK, array(), self::GROUP ); |
|
89 | + return as_schedule_single_action($when, self::HOOK, array(), self::GROUP); |
|
90 | 90 | } |
91 | 91 | |
92 | 92 | /** |
93 | 93 | * Remove the scheduled migration action. |
94 | 94 | */ |
95 | 95 | public function unschedule_migration() { |
96 | - as_unschedule_action( self::HOOK, null, self::GROUP ); |
|
96 | + as_unschedule_action(self::HOOK, null, self::GROUP); |
|
97 | 97 | } |
98 | 98 | |
99 | 99 | /** |
@@ -102,7 +102,7 @@ discard block |
||
102 | 102 | * @return int Seconds between migration runs. Defaults to 0 seconds to allow chaining migration via Async Runners. |
103 | 103 | */ |
104 | 104 | private function get_schedule_interval() { |
105 | - return (int) apply_filters( 'action_scheduler/migration_interval', 0 ); |
|
105 | + return (int) apply_filters('action_scheduler/migration_interval', 0); |
|
106 | 106 | } |
107 | 107 | |
108 | 108 | /** |
@@ -111,7 +111,7 @@ discard block |
||
111 | 111 | * @return int Number of actions to migrate in each batch. Defaults to 250. |
112 | 112 | */ |
113 | 113 | private function get_batch_size() { |
114 | - return (int) apply_filters( 'action_scheduler/migration_batch_size', 250 ); |
|
114 | + return (int) apply_filters('action_scheduler/migration_batch_size', 250); |
|
115 | 115 | } |
116 | 116 | |
117 | 117 | /** |
@@ -122,7 +122,7 @@ discard block |
||
122 | 122 | private function get_migration_runner() { |
123 | 123 | $config = Controller::instance()->get_migration_config_object(); |
124 | 124 | |
125 | - return new Runner( $config ); |
|
125 | + return new Runner($config); |
|
126 | 126 | } |
127 | 127 | |
128 | 128 | } |
@@ -20,8 +20,8 @@ |
||
20 | 20 | * |
21 | 21 | * @return int |
22 | 22 | */ |
23 | - public function migrate( $source_action_id ) { |
|
24 | - do_action( 'action_scheduler/migrate_action_dry_run', $source_action_id ); |
|
23 | + public function migrate($source_action_id) { |
|
24 | + do_action('action_scheduler/migrate_action_dry_run', $source_action_id); |
|
25 | 25 | |
26 | 26 | return 0; |
27 | 27 | } |
@@ -17,7 +17,7 @@ |
||
17 | 17 | * @param int $source_action_id Source logger object. |
18 | 18 | * @param int $destination_action_id Destination logger object. |
19 | 19 | */ |
20 | - public function migrate( $source_action_id, $destination_action_id ) { |
|
20 | + public function migrate($source_action_id, $destination_action_id) { |
|
21 | 21 | // no-op |
22 | 22 | } |
23 | 23 | } |
24 | 24 | \ No newline at end of file |
@@ -23,25 +23,25 @@ |
||
23 | 23 | * @return string The action ID |
24 | 24 | * @throws \RuntimeException When the action is not saved. |
25 | 25 | */ |
26 | - public function save_action( ActionScheduler_Action $action, \DateTime $scheduled_date = null, \DateTime $last_attempt_date = null ){ |
|
26 | + public function save_action(ActionScheduler_Action $action, \DateTime $scheduled_date = null, \DateTime $last_attempt_date = null) { |
|
27 | 27 | try { |
28 | 28 | /** @var \wpdb $wpdb */ |
29 | 29 | global $wpdb; |
30 | 30 | |
31 | - $action_id = parent::save_action( $action, $scheduled_date ); |
|
31 | + $action_id = parent::save_action($action, $scheduled_date); |
|
32 | 32 | |
33 | - if ( null !== $last_attempt_date ) { |
|
33 | + if (null !== $last_attempt_date) { |
|
34 | 34 | $data = [ |
35 | - 'last_attempt_gmt' => $this->get_scheduled_date_string( $action, $last_attempt_date ), |
|
36 | - 'last_attempt_local' => $this->get_scheduled_date_string_local( $action, $last_attempt_date ), |
|
35 | + 'last_attempt_gmt' => $this->get_scheduled_date_string($action, $last_attempt_date), |
|
36 | + 'last_attempt_local' => $this->get_scheduled_date_string_local($action, $last_attempt_date), |
|
37 | 37 | ]; |
38 | 38 | |
39 | - $wpdb->update( $wpdb->actionscheduler_actions, $data, array( 'action_id' => $action_id ), array( '%s', '%s' ), array( '%d' ) ); |
|
39 | + $wpdb->update($wpdb->actionscheduler_actions, $data, array('action_id' => $action_id), array('%s', '%s'), array('%d')); |
|
40 | 40 | } |
41 | 41 | |
42 | 42 | return $action_id; |
43 | - } catch ( \Exception $e ) { |
|
44 | - throw new \RuntimeException( sprintf( __( 'Error saving action: %s', 'action-scheduler' ), $e->getMessage() ), 0 ); |
|
43 | + } catch (\Exception $e) { |
|
44 | + throw new \RuntimeException(sprintf(__('Error saving action: %s', 'action-scheduler'), $e->getMessage()), 0); |
|
45 | 45 | } |
46 | 46 | } |
47 | 47 | } |
@@ -36,7 +36,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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; |