@@ -32,11 +32,11 @@ discard block |
||
| 32 | 32 | use Wordlift\Post\Post_Adapter; |
| 33 | 33 | // If this file is called directly, abort. |
| 34 | 34 | if ( ! defined( 'WPINC' ) ) { |
| 35 | - die; |
|
| 35 | + die; |
|
| 36 | 36 | } |
| 37 | 37 | define( |
| 38 | - 'WORDLIFT_PLUGIN_FILE', |
|
| 39 | - __FILE__ |
|
| 38 | + 'WORDLIFT_PLUGIN_FILE', |
|
| 39 | + __FILE__ |
|
| 40 | 40 | ); |
| 41 | 41 | define( 'WORDLIFT_VERSION', '3.41.0-dev' ); |
| 42 | 42 | |
@@ -50,7 +50,7 @@ discard block |
||
| 50 | 50 | * @since 3.33.6 |
| 51 | 51 | */ |
| 52 | 52 | if ( ! apply_filters( 'wl_is_enabled', true ) ) { |
| 53 | - return; |
|
| 53 | + return; |
|
| 54 | 54 | } |
| 55 | 55 | |
| 56 | 56 | require_once plugin_dir_path( __FILE__ ) . 'vendor/autoload.php'; |
@@ -79,33 +79,33 @@ discard block |
||
| 79 | 79 | */ |
| 80 | 80 | function activate_wordlift() { |
| 81 | 81 | |
| 82 | - $log = Wordlift_Log_Service::get_logger( 'activate_wordlift' ); |
|
| 82 | + $log = Wordlift_Log_Service::get_logger( 'activate_wordlift' ); |
|
| 83 | 83 | |
| 84 | - $log->info( 'Activating WordLift...' ); |
|
| 84 | + $log->info( 'Activating WordLift...' ); |
|
| 85 | 85 | |
| 86 | - require_once plugin_dir_path( __FILE__ ) . 'includes/class-wordlift-activator.php'; |
|
| 87 | - Wordlift_Activator::activate(); |
|
| 86 | + require_once plugin_dir_path( __FILE__ ) . 'includes/class-wordlift-activator.php'; |
|
| 87 | + Wordlift_Activator::activate(); |
|
| 88 | 88 | |
| 89 | - /** |
|
| 90 | - * Tell the {@link Wordlift_Http_Api} class that we're activating, to let it run activation tasks. |
|
| 91 | - * |
|
| 92 | - * @see https://github.com/insideout10/wordlift-plugin/issues/820 related issue. |
|
| 93 | - * @since 3.19.2 |
|
| 94 | - */ |
|
| 95 | - Wordlift_Http_Api::activate(); |
|
| 96 | - |
|
| 97 | - // Ensure the post type is registered before flushing the rewrite rules. |
|
| 98 | - Wordlift_Entity_Post_Type_Service::get_instance()->register(); |
|
| 99 | - flush_rewrite_rules(); |
|
| 100 | - /** |
|
| 101 | - * @since 3.27.7 |
|
| 102 | - * @see https://github.com/insideout10/wordlift-plugin/issues/1214 |
|
| 103 | - */ |
|
| 104 | - Top_Entities::activate(); |
|
| 89 | + /** |
|
| 90 | + * Tell the {@link Wordlift_Http_Api} class that we're activating, to let it run activation tasks. |
|
| 91 | + * |
|
| 92 | + * @see https://github.com/insideout10/wordlift-plugin/issues/820 related issue. |
|
| 93 | + * @since 3.19.2 |
|
| 94 | + */ |
|
| 95 | + Wordlift_Http_Api::activate(); |
|
| 96 | + |
|
| 97 | + // Ensure the post type is registered before flushing the rewrite rules. |
|
| 98 | + Wordlift_Entity_Post_Type_Service::get_instance()->register(); |
|
| 99 | + flush_rewrite_rules(); |
|
| 100 | + /** |
|
| 101 | + * @since 3.27.7 |
|
| 102 | + * @see https://github.com/insideout10/wordlift-plugin/issues/1214 |
|
| 103 | + */ |
|
| 104 | + Top_Entities::activate(); |
|
| 105 | 105 | |
| 106 | - if ( ! wp_next_scheduled( 'wl_daily_cron' ) ) { |
|
| 107 | - wp_schedule_event( time(), 'daily', 'wl_daily_cron' ); |
|
| 108 | - } |
|
| 106 | + if ( ! wp_next_scheduled( 'wl_daily_cron' ) ) { |
|
| 107 | + wp_schedule_event( time(), 'daily', 'wl_daily_cron' ); |
|
| 108 | + } |
|
| 109 | 109 | |
| 110 | 110 | } |
| 111 | 111 | |
@@ -115,23 +115,23 @@ discard block |
||
| 115 | 115 | */ |
| 116 | 116 | function deactivate_wordlift() { |
| 117 | 117 | |
| 118 | - require_once plugin_dir_path( __FILE__ ) . 'includes/class-wordlift-deactivator.php'; |
|
| 119 | - Wordlift_Deactivator::deactivate(); |
|
| 120 | - Wordlift_Http_Api::deactivate(); |
|
| 121 | - Ttl_Cache_Cleaner::deactivate(); |
|
| 122 | - /** |
|
| 123 | - * @since 3.27.7 |
|
| 124 | - * @see https://github.com/insideout10/wordlift-plugin/issues/1214 |
|
| 125 | - */ |
|
| 126 | - Top_Entities::deactivate(); |
|
| 127 | - /** |
|
| 128 | - * @since 3.27.8 |
|
| 129 | - * Remove notification flag on deactivation. |
|
| 130 | - */ |
|
| 131 | - Key_Validation_Notice::remove_notification_flag(); |
|
| 132 | - flush_rewrite_rules(); |
|
| 133 | - |
|
| 134 | - wp_clear_scheduled_hook( 'wl_daily_cron' ); |
|
| 118 | + require_once plugin_dir_path( __FILE__ ) . 'includes/class-wordlift-deactivator.php'; |
|
| 119 | + Wordlift_Deactivator::deactivate(); |
|
| 120 | + Wordlift_Http_Api::deactivate(); |
|
| 121 | + Ttl_Cache_Cleaner::deactivate(); |
|
| 122 | + /** |
|
| 123 | + * @since 3.27.7 |
|
| 124 | + * @see https://github.com/insideout10/wordlift-plugin/issues/1214 |
|
| 125 | + */ |
|
| 126 | + Top_Entities::deactivate(); |
|
| 127 | + /** |
|
| 128 | + * @since 3.27.8 |
|
| 129 | + * Remove notification flag on deactivation. |
|
| 130 | + */ |
|
| 131 | + Key_Validation_Notice::remove_notification_flag(); |
|
| 132 | + flush_rewrite_rules(); |
|
| 133 | + |
|
| 134 | + wp_clear_scheduled_hook( 'wl_daily_cron' ); |
|
| 135 | 135 | |
| 136 | 136 | } |
| 137 | 137 | |
@@ -154,84 +154,84 @@ discard block |
||
| 154 | 154 | * @since 1.0.0 |
| 155 | 155 | */ |
| 156 | 156 | function run_wordlift() { |
| 157 | - /** |
|
| 158 | - * Filter: wl_feature__enable__widgets. |
|
| 159 | - * |
|
| 160 | - * @param bool whether the widgets needed to be registered, defaults to true. |
|
| 161 | - * |
|
| 162 | - * @return bool |
|
| 163 | - * @since 3.27.6 |
|
| 164 | - */ |
|
| 165 | - if ( apply_filters( 'wl_feature__enable__widgets', true ) ) { |
|
| 166 | - add_action( 'widgets_init', 'wl_register_chord_widget' ); |
|
| 167 | - add_action( 'widgets_init', 'wl_register_geo_widget' ); |
|
| 168 | - add_action( 'widgets_init', 'wl_register_timeline_widget' ); |
|
| 169 | - } |
|
| 170 | - add_filter( 'widget_text', 'do_shortcode' ); |
|
| 171 | - |
|
| 172 | - /** |
|
| 173 | - * Filter: wl_feature__enable__analysis |
|
| 174 | - * |
|
| 175 | - * @param bool Whether to send api request to analysis or not |
|
| 176 | - * |
|
| 177 | - * @return bool |
|
| 178 | - * @since 3.27.6 |
|
| 179 | - */ |
|
| 180 | - if ( apply_filters( 'wl_feature__enable__analysis', true ) ) { |
|
| 181 | - add_action( 'wp_ajax_wl_analyze', 'wl_ajax_analyze_action' ); |
|
| 182 | - } else { |
|
| 183 | - add_action( 'wp_ajax_wl_analyze', 'wl_ajax_analyze_disabled_action' ); |
|
| 184 | - } |
|
| 185 | - |
|
| 186 | - $plugin = new Wordlift(); |
|
| 187 | - $plugin->run(); |
|
| 188 | - |
|
| 189 | - // Initialize the TTL Cache Cleaner. |
|
| 190 | - new Ttl_Cache_Cleaner(); |
|
| 191 | - |
|
| 192 | - // Load the new Post Adapter. |
|
| 193 | - new Post_Adapter(); |
|
| 194 | - |
|
| 195 | - // Load the API Data Hooks. |
|
| 196 | - new Api_Data_Hooks(); |
|
| 197 | - |
|
| 198 | - add_action( |
|
| 199 | - 'plugins_loaded', |
|
| 200 | - function () { |
|
| 201 | - // All features from registry should be initialized here. |
|
| 202 | - $features_registry = Features_Registry::get_instance(); |
|
| 203 | - $features_registry->initialize_all_features(); |
|
| 204 | - }, |
|
| 205 | - 5 |
|
| 206 | - ); |
|
| 207 | - |
|
| 208 | - add_action( |
|
| 209 | - 'plugins_loaded', |
|
| 210 | - // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable |
|
| 211 | - function () use ( $plugin ) { |
|
| 212 | - |
|
| 213 | - new Wordlift_Products_Navigator_Shortcode_REST(); |
|
| 214 | - |
|
| 215 | - // Register the Dataset module, requires `$api_service`. |
|
| 216 | - require_once plugin_dir_path( __FILE__ ) . 'wordlift/dataset/index.php'; |
|
| 217 | - require_once plugin_dir_path( __FILE__ ) . 'wordlift/shipping-data/index.php'; |
|
| 218 | - |
|
| 219 | - /* |
|
| 157 | + /** |
|
| 158 | + * Filter: wl_feature__enable__widgets. |
|
| 159 | + * |
|
| 160 | + * @param bool whether the widgets needed to be registered, defaults to true. |
|
| 161 | + * |
|
| 162 | + * @return bool |
|
| 163 | + * @since 3.27.6 |
|
| 164 | + */ |
|
| 165 | + if ( apply_filters( 'wl_feature__enable__widgets', true ) ) { |
|
| 166 | + add_action( 'widgets_init', 'wl_register_chord_widget' ); |
|
| 167 | + add_action( 'widgets_init', 'wl_register_geo_widget' ); |
|
| 168 | + add_action( 'widgets_init', 'wl_register_timeline_widget' ); |
|
| 169 | + } |
|
| 170 | + add_filter( 'widget_text', 'do_shortcode' ); |
|
| 171 | + |
|
| 172 | + /** |
|
| 173 | + * Filter: wl_feature__enable__analysis |
|
| 174 | + * |
|
| 175 | + * @param bool Whether to send api request to analysis or not |
|
| 176 | + * |
|
| 177 | + * @return bool |
|
| 178 | + * @since 3.27.6 |
|
| 179 | + */ |
|
| 180 | + if ( apply_filters( 'wl_feature__enable__analysis', true ) ) { |
|
| 181 | + add_action( 'wp_ajax_wl_analyze', 'wl_ajax_analyze_action' ); |
|
| 182 | + } else { |
|
| 183 | + add_action( 'wp_ajax_wl_analyze', 'wl_ajax_analyze_disabled_action' ); |
|
| 184 | + } |
|
| 185 | + |
|
| 186 | + $plugin = new Wordlift(); |
|
| 187 | + $plugin->run(); |
|
| 188 | + |
|
| 189 | + // Initialize the TTL Cache Cleaner. |
|
| 190 | + new Ttl_Cache_Cleaner(); |
|
| 191 | + |
|
| 192 | + // Load the new Post Adapter. |
|
| 193 | + new Post_Adapter(); |
|
| 194 | + |
|
| 195 | + // Load the API Data Hooks. |
|
| 196 | + new Api_Data_Hooks(); |
|
| 197 | + |
|
| 198 | + add_action( |
|
| 199 | + 'plugins_loaded', |
|
| 200 | + function () { |
|
| 201 | + // All features from registry should be initialized here. |
|
| 202 | + $features_registry = Features_Registry::get_instance(); |
|
| 203 | + $features_registry->initialize_all_features(); |
|
| 204 | + }, |
|
| 205 | + 5 |
|
| 206 | + ); |
|
| 207 | + |
|
| 208 | + add_action( |
|
| 209 | + 'plugins_loaded', |
|
| 210 | + // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable |
|
| 211 | + function () use ( $plugin ) { |
|
| 212 | + |
|
| 213 | + new Wordlift_Products_Navigator_Shortcode_REST(); |
|
| 214 | + |
|
| 215 | + // Register the Dataset module, requires `$api_service`. |
|
| 216 | + require_once plugin_dir_path( __FILE__ ) . 'wordlift/dataset/index.php'; |
|
| 217 | + require_once plugin_dir_path( __FILE__ ) . 'wordlift/shipping-data/index.php'; |
|
| 218 | + |
|
| 219 | + /* |
|
| 220 | 220 | * Require the Entity annotation cleanup module. |
| 221 | 221 | * |
| 222 | 222 | * @since 3.34.6 |
| 223 | 223 | */ |
| 224 | - require_once plugin_dir_path( __FILE__ ) . 'wordlift/cleanup/index.php'; |
|
| 224 | + require_once plugin_dir_path( __FILE__ ) . 'wordlift/cleanup/index.php'; |
|
| 225 | 225 | |
| 226 | - /* |
|
| 226 | + /* |
|
| 227 | 227 | * Import LOD entities. |
| 228 | 228 | * |
| 229 | 229 | * @since 3.35.0 |
| 230 | 230 | */ |
| 231 | - require_once plugin_dir_path( __FILE__ ) . 'wordlift/lod-import/index.php'; |
|
| 231 | + require_once plugin_dir_path( __FILE__ ) . 'wordlift/lod-import/index.php'; |
|
| 232 | 232 | |
| 233 | - } |
|
| 234 | - ); |
|
| 233 | + } |
|
| 234 | + ); |
|
| 235 | 235 | |
| 236 | 236 | } |
| 237 | 237 | |
@@ -245,45 +245,45 @@ discard block |
||
| 245 | 245 | */ |
| 246 | 246 | function wordlift_plugin_autoload_register() { |
| 247 | 247 | |
| 248 | - spl_autoload_register( |
|
| 249 | - function ( $class_name ) { |
|
| 248 | + spl_autoload_register( |
|
| 249 | + function ( $class_name ) { |
|
| 250 | 250 | |
| 251 | - // Bail out if these are not our classes. |
|
| 252 | - if ( 0 !== strpos( $class_name, 'Wordlift\\' ) ) { |
|
| 253 | - return false; |
|
| 254 | - } |
|
| 251 | + // Bail out if these are not our classes. |
|
| 252 | + if ( 0 !== strpos( $class_name, 'Wordlift\\' ) ) { |
|
| 253 | + return false; |
|
| 254 | + } |
|
| 255 | 255 | |
| 256 | - $class_name_lc = strtolower( str_replace( '_', '-', $class_name ) ); |
|
| 256 | + $class_name_lc = strtolower( str_replace( '_', '-', $class_name ) ); |
|
| 257 | 257 | |
| 258 | - preg_match( '|^(?:(.*)\\\\)?(.+?)$|', $class_name_lc, $matches ); |
|
| 258 | + preg_match( '|^(?:(.*)\\\\)?(.+?)$|', $class_name_lc, $matches ); |
|
| 259 | 259 | |
| 260 | - $path = str_replace( '\\', DIRECTORY_SEPARATOR, $matches[1] ); |
|
| 261 | - $file = 'class-' . $matches[2] . '.php'; |
|
| 260 | + $path = str_replace( '\\', DIRECTORY_SEPARATOR, $matches[1] ); |
|
| 261 | + $file = 'class-' . $matches[2] . '.php'; |
|
| 262 | 262 | |
| 263 | - $full_path = plugin_dir_path( __FILE__ ) . $path . DIRECTORY_SEPARATOR . $file; |
|
| 263 | + $full_path = plugin_dir_path( __FILE__ ) . $path . DIRECTORY_SEPARATOR . $file; |
|
| 264 | 264 | |
| 265 | - if ( ! file_exists( $full_path ) ) { |
|
| 266 | - return false; |
|
| 267 | - } |
|
| 265 | + if ( ! file_exists( $full_path ) ) { |
|
| 266 | + return false; |
|
| 267 | + } |
|
| 268 | 268 | |
| 269 | - require_once $full_path; |
|
| 269 | + require_once $full_path; |
|
| 270 | 270 | |
| 271 | - return true; |
|
| 272 | - } |
|
| 273 | - ); |
|
| 271 | + return true; |
|
| 272 | + } |
|
| 273 | + ); |
|
| 274 | 274 | |
| 275 | 275 | } |
| 276 | 276 | |
| 277 | 277 | function wl_block_categories( $categories ) { |
| 278 | - return array_merge( |
|
| 279 | - $categories, |
|
| 280 | - array( |
|
| 281 | - array( |
|
| 282 | - 'slug' => 'wordlift', |
|
| 283 | - 'title' => __( 'WordLift', 'wordlift' ), |
|
| 284 | - ), |
|
| 285 | - ) |
|
| 286 | - ); |
|
| 278 | + return array_merge( |
|
| 279 | + $categories, |
|
| 280 | + array( |
|
| 281 | + array( |
|
| 282 | + 'slug' => 'wordlift', |
|
| 283 | + 'title' => __( 'WordLift', 'wordlift' ), |
|
| 284 | + ), |
|
| 285 | + ) |
|
| 286 | + ); |
|
| 287 | 287 | } |
| 288 | 288 | |
| 289 | 289 | /** |
@@ -291,19 +291,19 @@ discard block |
||
| 291 | 291 | * this has to be removed when removing the legacy fields from the ui. |
| 292 | 292 | */ |
| 293 | 293 | function wl_enqueue_leaflet( $in_footer = false ) { |
| 294 | - // Leaflet. |
|
| 295 | - wp_enqueue_style( 'wl-leaflet', plugin_dir_url( __FILE__ ) . 'js/leaflet/leaflet.css', array(), '1.6.0' ); |
|
| 296 | - wp_enqueue_script( 'wl-leaflet', plugin_dir_url( __FILE__ ) . 'js/leaflet/leaflet.js', array(), '1.6.0', $in_footer ); |
|
| 294 | + // Leaflet. |
|
| 295 | + wp_enqueue_style( 'wl-leaflet', plugin_dir_url( __FILE__ ) . 'js/leaflet/leaflet.css', array(), '1.6.0' ); |
|
| 296 | + wp_enqueue_script( 'wl-leaflet', plugin_dir_url( __FILE__ ) . 'js/leaflet/leaflet.js', array(), '1.6.0', $in_footer ); |
|
| 297 | 297 | } |
| 298 | 298 | |
| 299 | 299 | add_filter( 'block_categories', 'wl_block_categories', 10 ); |
| 300 | 300 | |
| 301 | 301 | // Temporary fix for a typo in WooCommerce Extension. |
| 302 | 302 | add_filter( |
| 303 | - 'wl_feature__enable__dataset', |
|
| 304 | - function ( $value ) { |
|
| 305 | - return apply_filters( 'wl_features__enable__dataset', $value ); |
|
| 306 | - } |
|
| 303 | + 'wl_feature__enable__dataset', |
|
| 304 | + function ( $value ) { |
|
| 305 | + return apply_filters( 'wl_features__enable__dataset', $value ); |
|
| 306 | + } |
|
| 307 | 307 | ); |
| 308 | 308 | |
| 309 | 309 | require_once __DIR__ . '/modules/food-kg/load.php'; |
@@ -312,26 +312,26 @@ discard block |
||
| 312 | 312 | require_once __DIR__ . '/modules/include-exclude-push-config/load.php'; |
| 313 | 313 | |
| 314 | 314 | add_action( |
| 315 | - 'update_plugins_adthrive.wordlift.io', |
|
| 316 | - function ( $update, $plugin_data, $plugin_file ) { |
|
| 317 | - // Bail out if it's not our plugin. |
|
| 318 | - $update_uri = $plugin_data['UpdateURI']; |
|
| 319 | - if ( 'wordlift/wordlift.php' !== $plugin_file || ! isset( $update_uri ) ) { |
|
| 320 | - return $update; |
|
| 321 | - } |
|
| 322 | - |
|
| 323 | - $response = wp_remote_get( "$update_uri?nocache=" . time() ); |
|
| 324 | - |
|
| 325 | - if ( is_wp_error( $response ) ) { |
|
| 326 | - return $update; |
|
| 327 | - } |
|
| 328 | - |
|
| 329 | - try { |
|
| 330 | - return json_decode( wp_remote_retrieve_body( $response ) ); |
|
| 331 | - } catch ( Exception $e ) { |
|
| 332 | - return $update; |
|
| 333 | - } |
|
| 334 | - }, |
|
| 335 | - 10, |
|
| 336 | - 3 |
|
| 315 | + 'update_plugins_adthrive.wordlift.io', |
|
| 316 | + function ( $update, $plugin_data, $plugin_file ) { |
|
| 317 | + // Bail out if it's not our plugin. |
|
| 318 | + $update_uri = $plugin_data['UpdateURI']; |
|
| 319 | + if ( 'wordlift/wordlift.php' !== $plugin_file || ! isset( $update_uri ) ) { |
|
| 320 | + return $update; |
|
| 321 | + } |
|
| 322 | + |
|
| 323 | + $response = wp_remote_get( "$update_uri?nocache=" . time() ); |
|
| 324 | + |
|
| 325 | + if ( is_wp_error( $response ) ) { |
|
| 326 | + return $update; |
|
| 327 | + } |
|
| 328 | + |
|
| 329 | + try { |
|
| 330 | + return json_decode( wp_remote_retrieve_body( $response ) ); |
|
| 331 | + } catch ( Exception $e ) { |
|
| 332 | + return $update; |
|
| 333 | + } |
|
| 334 | + }, |
|
| 335 | + 10, |
|
| 336 | + 3 |
|
| 337 | 337 | ); |
@@ -5,41 +5,41 @@ |
||
| 5 | 5 | * @package test_cases\lock |
| 6 | 6 | */ |
| 7 | 7 | class ActionScheduler_OptionLock_Test extends ActionScheduler_UnitTestCase { |
| 8 | - public function test_instance() { |
|
| 9 | - $lock = ActionScheduler::lock(); |
|
| 10 | - $this->assertInstanceOf( 'ActionScheduler_Lock', $lock ); |
|
| 11 | - $this->assertInstanceOf( 'ActionScheduler_OptionLock', $lock ); |
|
| 12 | - } |
|
| 8 | + public function test_instance() { |
|
| 9 | + $lock = ActionScheduler::lock(); |
|
| 10 | + $this->assertInstanceOf( 'ActionScheduler_Lock', $lock ); |
|
| 11 | + $this->assertInstanceOf( 'ActionScheduler_OptionLock', $lock ); |
|
| 12 | + } |
|
| 13 | 13 | |
| 14 | - public function test_is_locked() { |
|
| 15 | - $lock = ActionScheduler::lock(); |
|
| 16 | - $lock_type = md5( rand() ); |
|
| 14 | + public function test_is_locked() { |
|
| 15 | + $lock = ActionScheduler::lock(); |
|
| 16 | + $lock_type = md5( rand() ); |
|
| 17 | 17 | |
| 18 | - $this->assertFalse( $lock->is_locked( $lock_type ) ); |
|
| 18 | + $this->assertFalse( $lock->is_locked( $lock_type ) ); |
|
| 19 | 19 | |
| 20 | - $lock->set( $lock_type ); |
|
| 21 | - $this->assertTrue( $lock->is_locked( $lock_type ) ); |
|
| 22 | - } |
|
| 20 | + $lock->set( $lock_type ); |
|
| 21 | + $this->assertTrue( $lock->is_locked( $lock_type ) ); |
|
| 22 | + } |
|
| 23 | 23 | |
| 24 | - public function test_set() { |
|
| 25 | - $lock = ActionScheduler::lock(); |
|
| 26 | - $lock_type = md5( rand() ); |
|
| 24 | + public function test_set() { |
|
| 25 | + $lock = ActionScheduler::lock(); |
|
| 26 | + $lock_type = md5( rand() ); |
|
| 27 | 27 | |
| 28 | - $lock->set( $lock_type ); |
|
| 29 | - $this->assertTrue( $lock->is_locked( $lock_type ) ); |
|
| 30 | - } |
|
| 28 | + $lock->set( $lock_type ); |
|
| 29 | + $this->assertTrue( $lock->is_locked( $lock_type ) ); |
|
| 30 | + } |
|
| 31 | 31 | |
| 32 | - public function test_get_expiration() { |
|
| 33 | - $lock = ActionScheduler::lock(); |
|
| 34 | - $lock_type = md5( rand() ); |
|
| 32 | + public function test_get_expiration() { |
|
| 33 | + $lock = ActionScheduler::lock(); |
|
| 34 | + $lock_type = md5( rand() ); |
|
| 35 | 35 | |
| 36 | - $lock->set( $lock_type ); |
|
| 36 | + $lock->set( $lock_type ); |
|
| 37 | 37 | |
| 38 | - $expiration = $lock->get_expiration( $lock_type ); |
|
| 39 | - $current_time = time(); |
|
| 38 | + $expiration = $lock->get_expiration( $lock_type ); |
|
| 39 | + $current_time = time(); |
|
| 40 | 40 | |
| 41 | - $this->assertGreaterThanOrEqual( 0, $expiration ); |
|
| 42 | - $this->assertGreaterThan( $current_time, $expiration ); |
|
| 43 | - $this->assertLessThan( $current_time + MINUTE_IN_SECONDS + 1, $expiration ); |
|
| 44 | - } |
|
| 41 | + $this->assertGreaterThanOrEqual( 0, $expiration ); |
|
| 42 | + $this->assertGreaterThan( $current_time, $expiration ); |
|
| 43 | + $this->assertLessThan( $current_time + MINUTE_IN_SECONDS + 1, $expiration ); |
|
| 44 | + } |
|
| 45 | 45 | } |
@@ -15,62 +15,62 @@ |
||
| 15 | 15 | * @group migration |
| 16 | 16 | */ |
| 17 | 17 | class Controller_Test extends ActionScheduler_UnitTestCase { |
| 18 | - /** |
|
| 19 | - * Test to ensure the Migration Controller will schedule the migration. |
|
| 20 | - */ |
|
| 21 | - public function test_schedules_migration() { |
|
| 22 | - as_unschedule_action( Scheduler::HOOK ); |
|
| 23 | - Controller::instance()->schedule_migration(); |
|
| 18 | + /** |
|
| 19 | + * Test to ensure the Migration Controller will schedule the migration. |
|
| 20 | + */ |
|
| 21 | + public function test_schedules_migration() { |
|
| 22 | + as_unschedule_action( Scheduler::HOOK ); |
|
| 23 | + Controller::instance()->schedule_migration(); |
|
| 24 | 24 | |
| 25 | - $this->assertTrue( |
|
| 26 | - as_next_scheduled_action( Scheduler::HOOK ) > 0, |
|
| 27 | - 'Confirm that the Migration Controller scheduled the migration.' |
|
| 28 | - ); |
|
| 25 | + $this->assertTrue( |
|
| 26 | + as_next_scheduled_action( Scheduler::HOOK ) > 0, |
|
| 27 | + 'Confirm that the Migration Controller scheduled the migration.' |
|
| 28 | + ); |
|
| 29 | 29 | |
| 30 | - as_unschedule_action( Scheduler::HOOK ); |
|
| 31 | - } |
|
| 30 | + as_unschedule_action( Scheduler::HOOK ); |
|
| 31 | + } |
|
| 32 | 32 | |
| 33 | - /** |
|
| 34 | - * Test to ensure that if an essential table is missing, the Migration |
|
| 35 | - * Controller will not schedule a migration. |
|
| 36 | - * |
|
| 37 | - * @see https://github.com/woocommerce/action-scheduler/issues/653 |
|
| 38 | - */ |
|
| 39 | - public function test_migration_not_scheduled_if_tables_are_missing() { |
|
| 40 | - as_unschedule_action( Scheduler::HOOK ); |
|
| 41 | - $this->rename_claims_table(); |
|
| 42 | - Controller::instance()->schedule_migration(); |
|
| 33 | + /** |
|
| 34 | + * Test to ensure that if an essential table is missing, the Migration |
|
| 35 | + * Controller will not schedule a migration. |
|
| 36 | + * |
|
| 37 | + * @see https://github.com/woocommerce/action-scheduler/issues/653 |
|
| 38 | + */ |
|
| 39 | + public function test_migration_not_scheduled_if_tables_are_missing() { |
|
| 40 | + as_unschedule_action( Scheduler::HOOK ); |
|
| 41 | + $this->rename_claims_table(); |
|
| 42 | + Controller::instance()->schedule_migration(); |
|
| 43 | 43 | |
| 44 | - $this->assertFalse( |
|
| 45 | - as_next_scheduled_action( Scheduler::HOOK ), |
|
| 46 | - 'When required tables are missing, the migration will not be scheduled.' |
|
| 47 | - ); |
|
| 44 | + $this->assertFalse( |
|
| 45 | + as_next_scheduled_action( Scheduler::HOOK ), |
|
| 46 | + 'When required tables are missing, the migration will not be scheduled.' |
|
| 47 | + ); |
|
| 48 | 48 | |
| 49 | - $this->restore_claims_table_name(); |
|
| 50 | - } |
|
| 49 | + $this->restore_claims_table_name(); |
|
| 50 | + } |
|
| 51 | 51 | |
| 52 | - /** |
|
| 53 | - * Rename the claims table, so that it cannot be used by the library. |
|
| 54 | - */ |
|
| 55 | - private function rename_claims_table() { |
|
| 56 | - global $wpdb; |
|
| 57 | - $normal_table_name = $wpdb->prefix . Schema::CLAIMS_TABLE; |
|
| 58 | - $modified_table_name = $normal_table_name . 'x'; |
|
| 52 | + /** |
|
| 53 | + * Rename the claims table, so that it cannot be used by the library. |
|
| 54 | + */ |
|
| 55 | + private function rename_claims_table() { |
|
| 56 | + global $wpdb; |
|
| 57 | + $normal_table_name = $wpdb->prefix . Schema::CLAIMS_TABLE; |
|
| 58 | + $modified_table_name = $normal_table_name . 'x'; |
|
| 59 | 59 | |
| 60 | - // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared |
|
| 61 | - $wpdb->query( "RENAME TABLE {$normal_table_name} TO {$modified_table_name}" ); |
|
| 62 | - } |
|
| 60 | + // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared |
|
| 61 | + $wpdb->query( "RENAME TABLE {$normal_table_name} TO {$modified_table_name}" ); |
|
| 62 | + } |
|
| 63 | 63 | |
| 64 | - /** |
|
| 65 | - * Restore the expected name of the claims table, so that it can be used by the library |
|
| 66 | - * and any further tests. |
|
| 67 | - */ |
|
| 68 | - private function restore_claims_table_name() { |
|
| 69 | - global $wpdb; |
|
| 70 | - $normal_table_name = $wpdb->prefix . Schema::CLAIMS_TABLE; |
|
| 71 | - $modified_table_name = $normal_table_name . 'x'; |
|
| 64 | + /** |
|
| 65 | + * Restore the expected name of the claims table, so that it can be used by the library |
|
| 66 | + * and any further tests. |
|
| 67 | + */ |
|
| 68 | + private function restore_claims_table_name() { |
|
| 69 | + global $wpdb; |
|
| 70 | + $normal_table_name = $wpdb->prefix . Schema::CLAIMS_TABLE; |
|
| 71 | + $modified_table_name = $normal_table_name . 'x'; |
|
| 72 | 72 | |
| 73 | - // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared |
|
| 74 | - $wpdb->query( "RENAME TABLE {$modified_table_name} TO {$normal_table_name}" ); |
|
| 75 | - } |
|
| 73 | + // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared |
|
| 74 | + $wpdb->query( "RENAME TABLE {$modified_table_name} TO {$normal_table_name}" ); |
|
| 75 | + } |
|
| 76 | 76 | } |
@@ -6,44 +6,44 @@ |
||
| 6 | 6 | */ |
| 7 | 7 | abstract class ActionScheduler_Store_Deprecated { |
| 8 | 8 | |
| 9 | - /** |
|
| 10 | - * Mark an action that failed to fetch correctly as failed. |
|
| 11 | - * |
|
| 12 | - * @since 2.2.6 |
|
| 13 | - * |
|
| 14 | - * @param int $action_id The ID of the action. |
|
| 15 | - */ |
|
| 16 | - public function mark_failed_fetch_action( $action_id ) { |
|
| 17 | - _deprecated_function( __METHOD__, '3.0.0', 'ActionScheduler_Store::mark_failure()' ); |
|
| 18 | - self::$store->mark_failure( $action_id ); |
|
| 19 | - } |
|
| 9 | + /** |
|
| 10 | + * Mark an action that failed to fetch correctly as failed. |
|
| 11 | + * |
|
| 12 | + * @since 2.2.6 |
|
| 13 | + * |
|
| 14 | + * @param int $action_id The ID of the action. |
|
| 15 | + */ |
|
| 16 | + public function mark_failed_fetch_action( $action_id ) { |
|
| 17 | + _deprecated_function( __METHOD__, '3.0.0', 'ActionScheduler_Store::mark_failure()' ); |
|
| 18 | + self::$store->mark_failure( $action_id ); |
|
| 19 | + } |
|
| 20 | 20 | |
| 21 | - /** |
|
| 22 | - * Add base hooks |
|
| 23 | - * |
|
| 24 | - * @since 2.2.6 |
|
| 25 | - */ |
|
| 26 | - protected static function hook() { |
|
| 27 | - _deprecated_function( __METHOD__, '3.0.0' ); |
|
| 28 | - } |
|
| 21 | + /** |
|
| 22 | + * Add base hooks |
|
| 23 | + * |
|
| 24 | + * @since 2.2.6 |
|
| 25 | + */ |
|
| 26 | + protected static function hook() { |
|
| 27 | + _deprecated_function( __METHOD__, '3.0.0' ); |
|
| 28 | + } |
|
| 29 | 29 | |
| 30 | - /** |
|
| 31 | - * Remove base hooks |
|
| 32 | - * |
|
| 33 | - * @since 2.2.6 |
|
| 34 | - */ |
|
| 35 | - protected static function unhook() { |
|
| 36 | - _deprecated_function( __METHOD__, '3.0.0' ); |
|
| 37 | - } |
|
| 30 | + /** |
|
| 31 | + * Remove base hooks |
|
| 32 | + * |
|
| 33 | + * @since 2.2.6 |
|
| 34 | + */ |
|
| 35 | + protected static function unhook() { |
|
| 36 | + _deprecated_function( __METHOD__, '3.0.0' ); |
|
| 37 | + } |
|
| 38 | 38 | |
| 39 | - /** |
|
| 40 | - * Get the site's local time. |
|
| 41 | - * |
|
| 42 | - * @deprecated 2.1.0 |
|
| 43 | - * @return DateTimeZone |
|
| 44 | - */ |
|
| 45 | - protected function get_local_timezone() { |
|
| 46 | - _deprecated_function( __FUNCTION__, '2.1.0', 'ActionScheduler_TimezoneHelper::set_local_timezone()' ); |
|
| 47 | - return ActionScheduler_TimezoneHelper::get_local_timezone(); |
|
| 48 | - } |
|
| 39 | + /** |
|
| 40 | + * Get the site's local time. |
|
| 41 | + * |
|
| 42 | + * @deprecated 2.1.0 |
|
| 43 | + * @return DateTimeZone |
|
| 44 | + */ |
|
| 45 | + protected function get_local_timezone() { |
|
| 46 | + _deprecated_function( __FUNCTION__, '2.1.0', 'ActionScheduler_TimezoneHelper::set_local_timezone()' ); |
|
| 47 | + return ActionScheduler_TimezoneHelper::get_local_timezone(); |
|
| 48 | + } |
|
| 49 | 49 | } |
@@ -9,41 +9,41 @@ |
||
| 9 | 9 | */ |
| 10 | 10 | class ActionScheduler_OptionLock extends ActionScheduler_Lock { |
| 11 | 11 | |
| 12 | - /** |
|
| 13 | - * Set a lock using options for a given amount of time (60 seconds by default). |
|
| 14 | - * |
|
| 15 | - * Using an autoloaded option avoids running database queries or other resource intensive tasks |
|
| 16 | - * on frequently triggered hooks, like 'init' or 'shutdown'. |
|
| 17 | - * |
|
| 18 | - * For example, ActionScheduler_QueueRunner->maybe_dispatch_async_request() uses a lock to avoid |
|
| 19 | - * calling ActionScheduler_QueueRunner->has_maximum_concurrent_batches() every time the 'shutdown', |
|
| 20 | - * hook is triggered, because that method calls ActionScheduler_QueueRunner->store->get_claim_count() |
|
| 21 | - * to find the current number of claims in the database. |
|
| 22 | - * |
|
| 23 | - * @param string $lock_type A string to identify different lock types. |
|
| 24 | - * @bool True if lock value has changed, false if not or if set failed. |
|
| 25 | - */ |
|
| 26 | - public function set( $lock_type ) { |
|
| 27 | - return update_option( $this->get_key( $lock_type ), time() + $this->get_duration( $lock_type ) ); |
|
| 28 | - } |
|
| 12 | + /** |
|
| 13 | + * Set a lock using options for a given amount of time (60 seconds by default). |
|
| 14 | + * |
|
| 15 | + * Using an autoloaded option avoids running database queries or other resource intensive tasks |
|
| 16 | + * on frequently triggered hooks, like 'init' or 'shutdown'. |
|
| 17 | + * |
|
| 18 | + * For example, ActionScheduler_QueueRunner->maybe_dispatch_async_request() uses a lock to avoid |
|
| 19 | + * calling ActionScheduler_QueueRunner->has_maximum_concurrent_batches() every time the 'shutdown', |
|
| 20 | + * hook is triggered, because that method calls ActionScheduler_QueueRunner->store->get_claim_count() |
|
| 21 | + * to find the current number of claims in the database. |
|
| 22 | + * |
|
| 23 | + * @param string $lock_type A string to identify different lock types. |
|
| 24 | + * @bool True if lock value has changed, false if not or if set failed. |
|
| 25 | + */ |
|
| 26 | + public function set( $lock_type ) { |
|
| 27 | + return update_option( $this->get_key( $lock_type ), time() + $this->get_duration( $lock_type ) ); |
|
| 28 | + } |
|
| 29 | 29 | |
| 30 | - /** |
|
| 31 | - * If a lock is set, return the timestamp it was set to expiry. |
|
| 32 | - * |
|
| 33 | - * @param string $lock_type A string to identify different lock types. |
|
| 34 | - * @return bool|int False if no lock is set, otherwise the timestamp for when the lock is set to expire. |
|
| 35 | - */ |
|
| 36 | - public function get_expiration( $lock_type ) { |
|
| 37 | - return get_option( $this->get_key( $lock_type ) ); |
|
| 38 | - } |
|
| 30 | + /** |
|
| 31 | + * If a lock is set, return the timestamp it was set to expiry. |
|
| 32 | + * |
|
| 33 | + * @param string $lock_type A string to identify different lock types. |
|
| 34 | + * @return bool|int False if no lock is set, otherwise the timestamp for when the lock is set to expire. |
|
| 35 | + */ |
|
| 36 | + public function get_expiration( $lock_type ) { |
|
| 37 | + return get_option( $this->get_key( $lock_type ) ); |
|
| 38 | + } |
|
| 39 | 39 | |
| 40 | - /** |
|
| 41 | - * Get the key to use for storing the lock in the transient |
|
| 42 | - * |
|
| 43 | - * @param string $lock_type A string to identify different lock types. |
|
| 44 | - * @return string |
|
| 45 | - */ |
|
| 46 | - protected function get_key( $lock_type ) { |
|
| 47 | - return sprintf( 'action_scheduler_lock_%s', $lock_type ); |
|
| 48 | - } |
|
| 40 | + /** |
|
| 41 | + * Get the key to use for storing the lock in the transient |
|
| 42 | + * |
|
| 43 | + * @param string $lock_type A string to identify different lock types. |
|
| 44 | + * @return string |
|
| 45 | + */ |
|
| 46 | + protected function get_key( $lock_type ) { |
|
| 47 | + return sprintf( 'action_scheduler_lock_%s', $lock_type ); |
|
| 48 | + } |
|
| 49 | 49 | } |
@@ -15,35 +15,35 @@ |
||
| 15 | 15 | * @codeCoverageIgnore |
| 16 | 16 | */ |
| 17 | 17 | class LogMigrator { |
| 18 | - /** @var ActionScheduler_Logger */ |
|
| 19 | - private $source; |
|
| 18 | + /** @var ActionScheduler_Logger */ |
|
| 19 | + private $source; |
|
| 20 | 20 | |
| 21 | - /** @var ActionScheduler_Logger */ |
|
| 22 | - private $destination; |
|
| 21 | + /** @var ActionScheduler_Logger */ |
|
| 22 | + private $destination; |
|
| 23 | 23 | |
| 24 | - /** |
|
| 25 | - * ActionMigrator constructor. |
|
| 26 | - * |
|
| 27 | - * @param ActionScheduler_Logger $source_logger Source logger object. |
|
| 28 | - * @param ActionScheduler_Logger $destination_Logger Destination logger object. |
|
| 29 | - */ |
|
| 30 | - public function __construct( ActionScheduler_Logger $source_logger, ActionScheduler_Logger $destination_Logger ) { |
|
| 31 | - $this->source = $source_logger; |
|
| 32 | - $this->destination = $destination_Logger; |
|
| 33 | - } |
|
| 24 | + /** |
|
| 25 | + * ActionMigrator constructor. |
|
| 26 | + * |
|
| 27 | + * @param ActionScheduler_Logger $source_logger Source logger object. |
|
| 28 | + * @param ActionScheduler_Logger $destination_Logger Destination logger object. |
|
| 29 | + */ |
|
| 30 | + public function __construct( ActionScheduler_Logger $source_logger, ActionScheduler_Logger $destination_Logger ) { |
|
| 31 | + $this->source = $source_logger; |
|
| 32 | + $this->destination = $destination_Logger; |
|
| 33 | + } |
|
| 34 | 34 | |
| 35 | - /** |
|
| 36 | - * Migrate an action log. |
|
| 37 | - * |
|
| 38 | - * @param int $source_action_id Source logger object. |
|
| 39 | - * @param int $destination_action_id Destination logger object. |
|
| 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() ); |
|
| 46 | - } |
|
| 47 | - } |
|
| 48 | - } |
|
| 35 | + /** |
|
| 36 | + * Migrate an action log. |
|
| 37 | + * |
|
| 38 | + * @param int $source_action_id Source logger object. |
|
| 39 | + * @param int $destination_action_id Destination logger object. |
|
| 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() ); |
|
| 46 | + } |
|
| 47 | + } |
|
| 48 | + } |
|
| 49 | 49 | } |
@@ -17,152 +17,152 @@ |
||
| 17 | 17 | * A config builder for the ActionScheduler\Migration\Runner class |
| 18 | 18 | */ |
| 19 | 19 | class Config { |
| 20 | - /** @var ActionScheduler_Store */ |
|
| 21 | - private $source_store; |
|
| 22 | - |
|
| 23 | - /** @var ActionScheduler_Logger */ |
|
| 24 | - private $source_logger; |
|
| 25 | - |
|
| 26 | - /** @var ActionScheduler_Store */ |
|
| 27 | - private $destination_store; |
|
| 28 | - |
|
| 29 | - /** @var ActionScheduler_Logger */ |
|
| 30 | - private $destination_logger; |
|
| 31 | - |
|
| 32 | - /** @var Progress bar */ |
|
| 33 | - private $progress_bar; |
|
| 34 | - |
|
| 35 | - /** @var bool */ |
|
| 36 | - private $dry_run = false; |
|
| 37 | - |
|
| 38 | - /** |
|
| 39 | - * Config constructor. |
|
| 40 | - */ |
|
| 41 | - public function __construct() { |
|
| 42 | - |
|
| 43 | - } |
|
| 44 | - |
|
| 45 | - /** |
|
| 46 | - * Get the configured source store. |
|
| 47 | - * |
|
| 48 | - * @return ActionScheduler_Store |
|
| 49 | - */ |
|
| 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' ) ); |
|
| 53 | - } |
|
| 54 | - |
|
| 55 | - return $this->source_store; |
|
| 56 | - } |
|
| 57 | - |
|
| 58 | - /** |
|
| 59 | - * Set the configured source store. |
|
| 60 | - * |
|
| 61 | - * @param ActionScheduler_Store $store Source store object. |
|
| 62 | - */ |
|
| 63 | - public function set_source_store( Store $store ) { |
|
| 64 | - $this->source_store = $store; |
|
| 65 | - } |
|
| 66 | - |
|
| 67 | - /** |
|
| 68 | - * Get the configured source loger. |
|
| 69 | - * |
|
| 70 | - * @return ActionScheduler_Logger |
|
| 71 | - */ |
|
| 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' ) ); |
|
| 75 | - } |
|
| 76 | - |
|
| 77 | - return $this->source_logger; |
|
| 78 | - } |
|
| 79 | - |
|
| 80 | - /** |
|
| 81 | - * Set the configured source logger. |
|
| 82 | - * |
|
| 83 | - * @param ActionScheduler_Logger $logger |
|
| 84 | - */ |
|
| 85 | - public function set_source_logger( Logger $logger ) { |
|
| 86 | - $this->source_logger = $logger; |
|
| 87 | - } |
|
| 88 | - |
|
| 89 | - /** |
|
| 90 | - * Get the configured destination store. |
|
| 91 | - * |
|
| 92 | - * @return ActionScheduler_Store |
|
| 93 | - */ |
|
| 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' ) ); |
|
| 97 | - } |
|
| 98 | - |
|
| 99 | - return $this->destination_store; |
|
| 100 | - } |
|
| 101 | - |
|
| 102 | - /** |
|
| 103 | - * Set the configured destination store. |
|
| 104 | - * |
|
| 105 | - * @param ActionScheduler_Store $store |
|
| 106 | - */ |
|
| 107 | - public function set_destination_store( Store $store ) { |
|
| 108 | - $this->destination_store = $store; |
|
| 109 | - } |
|
| 110 | - |
|
| 111 | - /** |
|
| 112 | - * Get the configured destination logger. |
|
| 113 | - * |
|
| 114 | - * @return ActionScheduler_Logger |
|
| 115 | - */ |
|
| 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' ) ); |
|
| 119 | - } |
|
| 120 | - |
|
| 121 | - return $this->destination_logger; |
|
| 122 | - } |
|
| 123 | - |
|
| 124 | - /** |
|
| 125 | - * Set the configured destination logger. |
|
| 126 | - * |
|
| 127 | - * @param ActionScheduler_Logger $logger |
|
| 128 | - */ |
|
| 129 | - public function set_destination_logger( Logger $logger ) { |
|
| 130 | - $this->destination_logger = $logger; |
|
| 131 | - } |
|
| 132 | - |
|
| 133 | - /** |
|
| 134 | - * Get flag indicating whether it's a dry run. |
|
| 135 | - * |
|
| 136 | - * @return bool |
|
| 137 | - */ |
|
| 138 | - public function get_dry_run() { |
|
| 139 | - return $this->dry_run; |
|
| 140 | - } |
|
| 141 | - |
|
| 142 | - /** |
|
| 143 | - * Set flag indicating whether it's a dry run. |
|
| 144 | - * |
|
| 145 | - * @param bool $dry_run |
|
| 146 | - */ |
|
| 147 | - public function set_dry_run( $dry_run ) { |
|
| 148 | - $this->dry_run = (bool) $dry_run; |
|
| 149 | - } |
|
| 150 | - |
|
| 151 | - /** |
|
| 152 | - * Get progress bar object. |
|
| 153 | - * |
|
| 154 | - * @return ActionScheduler\WPCLI\ProgressBar |
|
| 155 | - */ |
|
| 156 | - public function get_progress_bar() { |
|
| 157 | - return $this->progress_bar; |
|
| 158 | - } |
|
| 159 | - |
|
| 160 | - /** |
|
| 161 | - * Set progress bar object. |
|
| 162 | - * |
|
| 163 | - * @param ActionScheduler\WPCLI\ProgressBar $progress_bar |
|
| 164 | - */ |
|
| 165 | - public function set_progress_bar( ProgressBar $progress_bar ) { |
|
| 166 | - $this->progress_bar = $progress_bar; |
|
| 167 | - } |
|
| 20 | + /** @var ActionScheduler_Store */ |
|
| 21 | + private $source_store; |
|
| 22 | + |
|
| 23 | + /** @var ActionScheduler_Logger */ |
|
| 24 | + private $source_logger; |
|
| 25 | + |
|
| 26 | + /** @var ActionScheduler_Store */ |
|
| 27 | + private $destination_store; |
|
| 28 | + |
|
| 29 | + /** @var ActionScheduler_Logger */ |
|
| 30 | + private $destination_logger; |
|
| 31 | + |
|
| 32 | + /** @var Progress bar */ |
|
| 33 | + private $progress_bar; |
|
| 34 | + |
|
| 35 | + /** @var bool */ |
|
| 36 | + private $dry_run = false; |
|
| 37 | + |
|
| 38 | + /** |
|
| 39 | + * Config constructor. |
|
| 40 | + */ |
|
| 41 | + public function __construct() { |
|
| 42 | + |
|
| 43 | + } |
|
| 44 | + |
|
| 45 | + /** |
|
| 46 | + * Get the configured source store. |
|
| 47 | + * |
|
| 48 | + * @return ActionScheduler_Store |
|
| 49 | + */ |
|
| 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' ) ); |
|
| 53 | + } |
|
| 54 | + |
|
| 55 | + return $this->source_store; |
|
| 56 | + } |
|
| 57 | + |
|
| 58 | + /** |
|
| 59 | + * Set the configured source store. |
|
| 60 | + * |
|
| 61 | + * @param ActionScheduler_Store $store Source store object. |
|
| 62 | + */ |
|
| 63 | + public function set_source_store( Store $store ) { |
|
| 64 | + $this->source_store = $store; |
|
| 65 | + } |
|
| 66 | + |
|
| 67 | + /** |
|
| 68 | + * Get the configured source loger. |
|
| 69 | + * |
|
| 70 | + * @return ActionScheduler_Logger |
|
| 71 | + */ |
|
| 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' ) ); |
|
| 75 | + } |
|
| 76 | + |
|
| 77 | + return $this->source_logger; |
|
| 78 | + } |
|
| 79 | + |
|
| 80 | + /** |
|
| 81 | + * Set the configured source logger. |
|
| 82 | + * |
|
| 83 | + * @param ActionScheduler_Logger $logger |
|
| 84 | + */ |
|
| 85 | + public function set_source_logger( Logger $logger ) { |
|
| 86 | + $this->source_logger = $logger; |
|
| 87 | + } |
|
| 88 | + |
|
| 89 | + /** |
|
| 90 | + * Get the configured destination store. |
|
| 91 | + * |
|
| 92 | + * @return ActionScheduler_Store |
|
| 93 | + */ |
|
| 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' ) ); |
|
| 97 | + } |
|
| 98 | + |
|
| 99 | + return $this->destination_store; |
|
| 100 | + } |
|
| 101 | + |
|
| 102 | + /** |
|
| 103 | + * Set the configured destination store. |
|
| 104 | + * |
|
| 105 | + * @param ActionScheduler_Store $store |
|
| 106 | + */ |
|
| 107 | + public function set_destination_store( Store $store ) { |
|
| 108 | + $this->destination_store = $store; |
|
| 109 | + } |
|
| 110 | + |
|
| 111 | + /** |
|
| 112 | + * Get the configured destination logger. |
|
| 113 | + * |
|
| 114 | + * @return ActionScheduler_Logger |
|
| 115 | + */ |
|
| 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' ) ); |
|
| 119 | + } |
|
| 120 | + |
|
| 121 | + return $this->destination_logger; |
|
| 122 | + } |
|
| 123 | + |
|
| 124 | + /** |
|
| 125 | + * Set the configured destination logger. |
|
| 126 | + * |
|
| 127 | + * @param ActionScheduler_Logger $logger |
|
| 128 | + */ |
|
| 129 | + public function set_destination_logger( Logger $logger ) { |
|
| 130 | + $this->destination_logger = $logger; |
|
| 131 | + } |
|
| 132 | + |
|
| 133 | + /** |
|
| 134 | + * Get flag indicating whether it's a dry run. |
|
| 135 | + * |
|
| 136 | + * @return bool |
|
| 137 | + */ |
|
| 138 | + public function get_dry_run() { |
|
| 139 | + return $this->dry_run; |
|
| 140 | + } |
|
| 141 | + |
|
| 142 | + /** |
|
| 143 | + * Set flag indicating whether it's a dry run. |
|
| 144 | + * |
|
| 145 | + * @param bool $dry_run |
|
| 146 | + */ |
|
| 147 | + public function set_dry_run( $dry_run ) { |
|
| 148 | + $this->dry_run = (bool) $dry_run; |
|
| 149 | + } |
|
| 150 | + |
|
| 151 | + /** |
|
| 152 | + * Get progress bar object. |
|
| 153 | + * |
|
| 154 | + * @return ActionScheduler\WPCLI\ProgressBar |
|
| 155 | + */ |
|
| 156 | + public function get_progress_bar() { |
|
| 157 | + return $this->progress_bar; |
|
| 158 | + } |
|
| 159 | + |
|
| 160 | + /** |
|
| 161 | + * Set progress bar object. |
|
| 162 | + * |
|
| 163 | + * @param ActionScheduler\WPCLI\ProgressBar $progress_bar |
|
| 164 | + */ |
|
| 165 | + public function set_progress_bar( ProgressBar $progress_bar ) { |
|
| 166 | + $this->progress_bar = $progress_bar; |
|
| 167 | + } |
|
| 168 | 168 | } |
@@ -13,16 +13,16 @@ |
||
| 13 | 13 | * @codeCoverageIgnore |
| 14 | 14 | */ |
| 15 | 15 | class DryRun_ActionMigrator extends ActionMigrator { |
| 16 | - /** |
|
| 17 | - * Simulate migrating an action. |
|
| 18 | - * |
|
| 19 | - * @param int $source_action_id Action ID. |
|
| 20 | - * |
|
| 21 | - * @return int |
|
| 22 | - */ |
|
| 23 | - public function migrate( $source_action_id ) { |
|
| 24 | - do_action( 'action_scheduler/migrate_action_dry_run', $source_action_id ); |
|
| 16 | + /** |
|
| 17 | + * Simulate migrating an action. |
|
| 18 | + * |
|
| 19 | + * @param int $source_action_id Action ID. |
|
| 20 | + * |
|
| 21 | + * @return int |
|
| 22 | + */ |
|
| 23 | + public function migrate( $source_action_id ) { |
|
| 24 | + do_action( 'action_scheduler/migrate_action_dry_run', $source_action_id ); |
|
| 25 | 25 | |
| 26 | - return 0; |
|
| 27 | - } |
|
| 26 | + return 0; |
|
| 27 | + } |
|
| 28 | 28 | } |
@@ -19,85 +19,85 @@ discard block |
||
| 19 | 19 | * @codeCoverageIgnore |
| 20 | 20 | */ |
| 21 | 21 | class Controller { |
| 22 | - private static $instance; |
|
| 23 | - |
|
| 24 | - /** @var Action_Scheduler\Migration\Scheduler */ |
|
| 25 | - private $migration_scheduler; |
|
| 26 | - |
|
| 27 | - /** @var string */ |
|
| 28 | - private $store_classname; |
|
| 29 | - |
|
| 30 | - /** @var string */ |
|
| 31 | - private $logger_classname; |
|
| 32 | - |
|
| 33 | - /** @var bool */ |
|
| 34 | - private $migrate_custom_store; |
|
| 35 | - |
|
| 36 | - /** |
|
| 37 | - * Controller constructor. |
|
| 38 | - * |
|
| 39 | - * @param Scheduler $migration_scheduler Migration scheduler object. |
|
| 40 | - */ |
|
| 41 | - protected function __construct( Scheduler $migration_scheduler ) { |
|
| 42 | - $this->migration_scheduler = $migration_scheduler; |
|
| 43 | - $this->store_classname = ''; |
|
| 44 | - } |
|
| 45 | - |
|
| 46 | - /** |
|
| 47 | - * Set the action store class name. |
|
| 48 | - * |
|
| 49 | - * @param string $class Classname of the store class. |
|
| 50 | - * |
|
| 51 | - * @return string |
|
| 52 | - */ |
|
| 53 | - public function get_store_class( $class ) { |
|
| 54 | - if ( \ActionScheduler_DataController::is_migration_complete() ) { |
|
| 55 | - return \ActionScheduler_DataController::DATASTORE_CLASS; |
|
| 56 | - } elseif ( \ActionScheduler_Store::DEFAULT_CLASS !== $class ) { |
|
| 57 | - $this->store_classname = $class; |
|
| 58 | - return $class; |
|
| 59 | - } else { |
|
| 60 | - return 'ActionScheduler_HybridStore'; |
|
| 61 | - } |
|
| 62 | - } |
|
| 63 | - |
|
| 64 | - /** |
|
| 65 | - * Set the action logger class name. |
|
| 66 | - * |
|
| 67 | - * @param string $class Classname of the logger class. |
|
| 68 | - * |
|
| 69 | - * @return string |
|
| 70 | - */ |
|
| 71 | - public function get_logger_class( $class ) { |
|
| 72 | - \ActionScheduler_Store::instance(); |
|
| 73 | - |
|
| 74 | - if ( $this->has_custom_datastore() ) { |
|
| 75 | - $this->logger_classname = $class; |
|
| 76 | - return $class; |
|
| 77 | - } else { |
|
| 78 | - return \ActionScheduler_DataController::LOGGER_CLASS; |
|
| 79 | - } |
|
| 80 | - } |
|
| 81 | - |
|
| 82 | - /** |
|
| 83 | - * Get flag indicating whether a custom datastore is in use. |
|
| 84 | - * |
|
| 85 | - * @return bool |
|
| 86 | - */ |
|
| 87 | - public function has_custom_datastore() { |
|
| 88 | - return (bool) $this->store_classname; |
|
| 89 | - } |
|
| 90 | - |
|
| 91 | - /** |
|
| 92 | - * Set up the background migration process. |
|
| 93 | - * |
|
| 94 | - * @return void |
|
| 95 | - */ |
|
| 96 | - public function schedule_migration() { |
|
| 97 | - $logging_tables = new ActionScheduler_LoggerSchema(); |
|
| 98 | - $store_tables = new ActionScheduler_StoreSchema(); |
|
| 99 | - |
|
| 100 | - /* |
|
| 22 | + private static $instance; |
|
| 23 | + |
|
| 24 | + /** @var Action_Scheduler\Migration\Scheduler */ |
|
| 25 | + private $migration_scheduler; |
|
| 26 | + |
|
| 27 | + /** @var string */ |
|
| 28 | + private $store_classname; |
|
| 29 | + |
|
| 30 | + /** @var string */ |
|
| 31 | + private $logger_classname; |
|
| 32 | + |
|
| 33 | + /** @var bool */ |
|
| 34 | + private $migrate_custom_store; |
|
| 35 | + |
|
| 36 | + /** |
|
| 37 | + * Controller constructor. |
|
| 38 | + * |
|
| 39 | + * @param Scheduler $migration_scheduler Migration scheduler object. |
|
| 40 | + */ |
|
| 41 | + protected function __construct( Scheduler $migration_scheduler ) { |
|
| 42 | + $this->migration_scheduler = $migration_scheduler; |
|
| 43 | + $this->store_classname = ''; |
|
| 44 | + } |
|
| 45 | + |
|
| 46 | + /** |
|
| 47 | + * Set the action store class name. |
|
| 48 | + * |
|
| 49 | + * @param string $class Classname of the store class. |
|
| 50 | + * |
|
| 51 | + * @return string |
|
| 52 | + */ |
|
| 53 | + public function get_store_class( $class ) { |
|
| 54 | + if ( \ActionScheduler_DataController::is_migration_complete() ) { |
|
| 55 | + return \ActionScheduler_DataController::DATASTORE_CLASS; |
|
| 56 | + } elseif ( \ActionScheduler_Store::DEFAULT_CLASS !== $class ) { |
|
| 57 | + $this->store_classname = $class; |
|
| 58 | + return $class; |
|
| 59 | + } else { |
|
| 60 | + return 'ActionScheduler_HybridStore'; |
|
| 61 | + } |
|
| 62 | + } |
|
| 63 | + |
|
| 64 | + /** |
|
| 65 | + * Set the action logger class name. |
|
| 66 | + * |
|
| 67 | + * @param string $class Classname of the logger class. |
|
| 68 | + * |
|
| 69 | + * @return string |
|
| 70 | + */ |
|
| 71 | + public function get_logger_class( $class ) { |
|
| 72 | + \ActionScheduler_Store::instance(); |
|
| 73 | + |
|
| 74 | + if ( $this->has_custom_datastore() ) { |
|
| 75 | + $this->logger_classname = $class; |
|
| 76 | + return $class; |
|
| 77 | + } else { |
|
| 78 | + return \ActionScheduler_DataController::LOGGER_CLASS; |
|
| 79 | + } |
|
| 80 | + } |
|
| 81 | + |
|
| 82 | + /** |
|
| 83 | + * Get flag indicating whether a custom datastore is in use. |
|
| 84 | + * |
|
| 85 | + * @return bool |
|
| 86 | + */ |
|
| 87 | + public function has_custom_datastore() { |
|
| 88 | + return (bool) $this->store_classname; |
|
| 89 | + } |
|
| 90 | + |
|
| 91 | + /** |
|
| 92 | + * Set up the background migration process. |
|
| 93 | + * |
|
| 94 | + * @return void |
|
| 95 | + */ |
|
| 96 | + public function schedule_migration() { |
|
| 97 | + $logging_tables = new ActionScheduler_LoggerSchema(); |
|
| 98 | + $store_tables = new ActionScheduler_StoreSchema(); |
|
| 99 | + |
|
| 100 | + /* |
|
| 101 | 101 | * In some unusual cases, the expected tables may not have been created. In such cases |
| 102 | 102 | * we do not schedule a migration as doing so will lead to fatal error conditions. |
| 103 | 103 | * |
@@ -107,120 +107,120 @@ discard block |
||
| 107 | 107 | * |
| 108 | 108 | * @see https://github.com/woocommerce/action-scheduler/issues/653 |
| 109 | 109 | */ |
| 110 | - if ( |
|
| 111 | - ActionScheduler_DataController::is_migration_complete() |
|
| 112 | - || $this->migration_scheduler->is_migration_scheduled() |
|
| 113 | - || ! $store_tables->tables_exist() |
|
| 114 | - || ! $logging_tables->tables_exist() |
|
| 115 | - ) { |
|
| 116 | - return; |
|
| 117 | - } |
|
| 118 | - |
|
| 119 | - $this->migration_scheduler->schedule_migration(); |
|
| 120 | - } |
|
| 121 | - |
|
| 122 | - /** |
|
| 123 | - * Get the default migration config object |
|
| 124 | - * |
|
| 125 | - * @return ActionScheduler\Migration\Config |
|
| 126 | - */ |
|
| 127 | - public function get_migration_config_object() { |
|
| 128 | - static $config = null; |
|
| 129 | - |
|
| 130 | - if ( ! $config ) { |
|
| 131 | - $source_store = $this->store_classname ? new $this->store_classname() : new \ActionScheduler_wpPostStore(); |
|
| 132 | - $source_logger = $this->logger_classname ? new $this->logger_classname() : new \ActionScheduler_wpCommentLogger(); |
|
| 133 | - |
|
| 134 | - $config = new Config(); |
|
| 135 | - $config->set_source_store( $source_store ); |
|
| 136 | - $config->set_source_logger( $source_logger ); |
|
| 137 | - $config->set_destination_store( new \ActionScheduler_DBStoreMigrator() ); |
|
| 138 | - $config->set_destination_logger( new \ActionScheduler_DBLogger() ); |
|
| 139 | - |
|
| 140 | - if ( defined( 'WP_CLI' ) && WP_CLI ) { |
|
| 141 | - $config->set_progress_bar( new ProgressBar( '', 0 ) ); |
|
| 142 | - } |
|
| 143 | - } |
|
| 144 | - |
|
| 145 | - return apply_filters( 'action_scheduler/migration_config', $config ); |
|
| 146 | - } |
|
| 147 | - |
|
| 148 | - /** |
|
| 149 | - * Hook dashboard migration notice. |
|
| 150 | - */ |
|
| 151 | - public function hook_admin_notices() { |
|
| 152 | - if ( ! $this->allow_migration() || \ActionScheduler_DataController::is_migration_complete() ) { |
|
| 153 | - return; |
|
| 154 | - } |
|
| 155 | - add_action( 'admin_notices', array( $this, 'display_migration_notice' ), 10, 0 ); |
|
| 156 | - } |
|
| 157 | - |
|
| 158 | - /** |
|
| 159 | - * Show a dashboard notice that migration is in progress. |
|
| 160 | - */ |
|
| 161 | - public function display_migration_notice() { |
|
| 162 | - printf( '<div class="notice notice-warning"><p>%s</p></div>', esc_html__( 'Action Scheduler migration in progress. The list of scheduled actions may be incomplete.', 'action-scheduler' ) ); |
|
| 163 | - } |
|
| 164 | - |
|
| 165 | - /** |
|
| 166 | - * Add store classes. Hook migration. |
|
| 167 | - */ |
|
| 168 | - private function hook() { |
|
| 169 | - add_filter( 'action_scheduler_store_class', array( $this, 'get_store_class' ), 100, 1 ); |
|
| 170 | - add_filter( 'action_scheduler_logger_class', array( $this, 'get_logger_class' ), 100, 1 ); |
|
| 171 | - add_action( 'init', array( $this, 'maybe_hook_migration' ) ); |
|
| 172 | - add_action( 'wp_loaded', array( $this, 'schedule_migration' ) ); |
|
| 173 | - |
|
| 174 | - // Action Scheduler may be displayed as a Tools screen or WooCommerce > Status administration screen |
|
| 175 | - add_action( 'load-tools_page_action-scheduler', array( $this, 'hook_admin_notices' ), 10, 0 ); |
|
| 176 | - add_action( 'load-woocommerce_page_wc-status', array( $this, 'hook_admin_notices' ), 10, 0 ); |
|
| 177 | - } |
|
| 178 | - |
|
| 179 | - /** |
|
| 180 | - * Possibly hook the migration scheduler action. |
|
| 181 | - * |
|
| 182 | - * @author Jeremy Pry |
|
| 183 | - */ |
|
| 184 | - public function maybe_hook_migration() { |
|
| 185 | - if ( ! $this->allow_migration() || \ActionScheduler_DataController::is_migration_complete() ) { |
|
| 186 | - return; |
|
| 187 | - } |
|
| 188 | - |
|
| 189 | - $this->migration_scheduler->hook(); |
|
| 190 | - } |
|
| 191 | - |
|
| 192 | - /** |
|
| 193 | - * Allow datastores to enable migration to AS tables. |
|
| 194 | - */ |
|
| 195 | - public function allow_migration() { |
|
| 196 | - if ( ! \ActionScheduler_DataController::dependencies_met() ) { |
|
| 197 | - return false; |
|
| 198 | - } |
|
| 199 | - |
|
| 200 | - if ( null === $this->migrate_custom_store ) { |
|
| 201 | - $this->migrate_custom_store = apply_filters( 'action_scheduler_migrate_data_store', false ); |
|
| 202 | - } |
|
| 203 | - |
|
| 204 | - return ( ! $this->has_custom_datastore() ) || $this->migrate_custom_store; |
|
| 205 | - } |
|
| 206 | - |
|
| 207 | - /** |
|
| 208 | - * Proceed with the migration if the dependencies have been met. |
|
| 209 | - */ |
|
| 210 | - public static function init() { |
|
| 211 | - if ( \ActionScheduler_DataController::dependencies_met() ) { |
|
| 212 | - self::instance()->hook(); |
|
| 213 | - } |
|
| 214 | - } |
|
| 215 | - |
|
| 216 | - /** |
|
| 217 | - * Singleton factory. |
|
| 218 | - */ |
|
| 219 | - public static function instance() { |
|
| 220 | - if ( ! isset( self::$instance ) ) { |
|
| 221 | - self::$instance = new static( new Scheduler() ); |
|
| 222 | - } |
|
| 223 | - |
|
| 224 | - return self::$instance; |
|
| 225 | - } |
|
| 110 | + if ( |
|
| 111 | + ActionScheduler_DataController::is_migration_complete() |
|
| 112 | + || $this->migration_scheduler->is_migration_scheduled() |
|
| 113 | + || ! $store_tables->tables_exist() |
|
| 114 | + || ! $logging_tables->tables_exist() |
|
| 115 | + ) { |
|
| 116 | + return; |
|
| 117 | + } |
|
| 118 | + |
|
| 119 | + $this->migration_scheduler->schedule_migration(); |
|
| 120 | + } |
|
| 121 | + |
|
| 122 | + /** |
|
| 123 | + * Get the default migration config object |
|
| 124 | + * |
|
| 125 | + * @return ActionScheduler\Migration\Config |
|
| 126 | + */ |
|
| 127 | + public function get_migration_config_object() { |
|
| 128 | + static $config = null; |
|
| 129 | + |
|
| 130 | + if ( ! $config ) { |
|
| 131 | + $source_store = $this->store_classname ? new $this->store_classname() : new \ActionScheduler_wpPostStore(); |
|
| 132 | + $source_logger = $this->logger_classname ? new $this->logger_classname() : new \ActionScheduler_wpCommentLogger(); |
|
| 133 | + |
|
| 134 | + $config = new Config(); |
|
| 135 | + $config->set_source_store( $source_store ); |
|
| 136 | + $config->set_source_logger( $source_logger ); |
|
| 137 | + $config->set_destination_store( new \ActionScheduler_DBStoreMigrator() ); |
|
| 138 | + $config->set_destination_logger( new \ActionScheduler_DBLogger() ); |
|
| 139 | + |
|
| 140 | + if ( defined( 'WP_CLI' ) && WP_CLI ) { |
|
| 141 | + $config->set_progress_bar( new ProgressBar( '', 0 ) ); |
|
| 142 | + } |
|
| 143 | + } |
|
| 144 | + |
|
| 145 | + return apply_filters( 'action_scheduler/migration_config', $config ); |
|
| 146 | + } |
|
| 147 | + |
|
| 148 | + /** |
|
| 149 | + * Hook dashboard migration notice. |
|
| 150 | + */ |
|
| 151 | + public function hook_admin_notices() { |
|
| 152 | + if ( ! $this->allow_migration() || \ActionScheduler_DataController::is_migration_complete() ) { |
|
| 153 | + return; |
|
| 154 | + } |
|
| 155 | + add_action( 'admin_notices', array( $this, 'display_migration_notice' ), 10, 0 ); |
|
| 156 | + } |
|
| 157 | + |
|
| 158 | + /** |
|
| 159 | + * Show a dashboard notice that migration is in progress. |
|
| 160 | + */ |
|
| 161 | + public function display_migration_notice() { |
|
| 162 | + printf( '<div class="notice notice-warning"><p>%s</p></div>', esc_html__( 'Action Scheduler migration in progress. The list of scheduled actions may be incomplete.', 'action-scheduler' ) ); |
|
| 163 | + } |
|
| 164 | + |
|
| 165 | + /** |
|
| 166 | + * Add store classes. Hook migration. |
|
| 167 | + */ |
|
| 168 | + private function hook() { |
|
| 169 | + add_filter( 'action_scheduler_store_class', array( $this, 'get_store_class' ), 100, 1 ); |
|
| 170 | + add_filter( 'action_scheduler_logger_class', array( $this, 'get_logger_class' ), 100, 1 ); |
|
| 171 | + add_action( 'init', array( $this, 'maybe_hook_migration' ) ); |
|
| 172 | + add_action( 'wp_loaded', array( $this, 'schedule_migration' ) ); |
|
| 173 | + |
|
| 174 | + // Action Scheduler may be displayed as a Tools screen or WooCommerce > Status administration screen |
|
| 175 | + add_action( 'load-tools_page_action-scheduler', array( $this, 'hook_admin_notices' ), 10, 0 ); |
|
| 176 | + add_action( 'load-woocommerce_page_wc-status', array( $this, 'hook_admin_notices' ), 10, 0 ); |
|
| 177 | + } |
|
| 178 | + |
|
| 179 | + /** |
|
| 180 | + * Possibly hook the migration scheduler action. |
|
| 181 | + * |
|
| 182 | + * @author Jeremy Pry |
|
| 183 | + */ |
|
| 184 | + public function maybe_hook_migration() { |
|
| 185 | + if ( ! $this->allow_migration() || \ActionScheduler_DataController::is_migration_complete() ) { |
|
| 186 | + return; |
|
| 187 | + } |
|
| 188 | + |
|
| 189 | + $this->migration_scheduler->hook(); |
|
| 190 | + } |
|
| 191 | + |
|
| 192 | + /** |
|
| 193 | + * Allow datastores to enable migration to AS tables. |
|
| 194 | + */ |
|
| 195 | + public function allow_migration() { |
|
| 196 | + if ( ! \ActionScheduler_DataController::dependencies_met() ) { |
|
| 197 | + return false; |
|
| 198 | + } |
|
| 199 | + |
|
| 200 | + if ( null === $this->migrate_custom_store ) { |
|
| 201 | + $this->migrate_custom_store = apply_filters( 'action_scheduler_migrate_data_store', false ); |
|
| 202 | + } |
|
| 203 | + |
|
| 204 | + return ( ! $this->has_custom_datastore() ) || $this->migrate_custom_store; |
|
| 205 | + } |
|
| 206 | + |
|
| 207 | + /** |
|
| 208 | + * Proceed with the migration if the dependencies have been met. |
|
| 209 | + */ |
|
| 210 | + public static function init() { |
|
| 211 | + if ( \ActionScheduler_DataController::dependencies_met() ) { |
|
| 212 | + self::instance()->hook(); |
|
| 213 | + } |
|
| 214 | + } |
|
| 215 | + |
|
| 216 | + /** |
|
| 217 | + * Singleton factory. |
|
| 218 | + */ |
|
| 219 | + public static function instance() { |
|
| 220 | + if ( ! isset( self::$instance ) ) { |
|
| 221 | + self::$instance = new static( new Scheduler() ); |
|
| 222 | + } |
|
| 223 | + |
|
| 224 | + return self::$instance; |
|
| 225 | + } |
|
| 226 | 226 | } |