@@ -25,29 +25,29 @@ |
||
| 25 | 25 | * |
| 26 | 26 | */ |
| 27 | 27 | |
| 28 | -if ( ! function_exists( 'action_scheduler_register_3_dot_1_dot_4' ) ) { |
|
| 28 | +if (!function_exists('action_scheduler_register_3_dot_1_dot_4')) { |
|
| 29 | 29 | |
| 30 | - if ( ! class_exists( 'ActionScheduler_Versions' ) ) { |
|
| 31 | - require_once( 'classes/ActionScheduler_Versions.php' ); |
|
| 32 | - add_action( 'plugins_loaded', array( 'ActionScheduler_Versions', 'initialize_latest_version' ), 1, 0 ); |
|
| 30 | + if (!class_exists('ActionScheduler_Versions')) { |
|
| 31 | + require_once('classes/ActionScheduler_Versions.php'); |
|
| 32 | + add_action('plugins_loaded', array('ActionScheduler_Versions', 'initialize_latest_version'), 1, 0); |
|
| 33 | 33 | } |
| 34 | 34 | |
| 35 | - add_action( 'plugins_loaded', 'action_scheduler_register_3_dot_1_dot_4', 0, 0 ); |
|
| 35 | + add_action('plugins_loaded', 'action_scheduler_register_3_dot_1_dot_4', 0, 0); |
|
| 36 | 36 | |
| 37 | 37 | function action_scheduler_register_3_dot_1_dot_4() { |
| 38 | 38 | $versions = ActionScheduler_Versions::instance(); |
| 39 | - $versions->register( '3.1.4', 'action_scheduler_initialize_3_dot_1_dot_4' ); |
|
| 39 | + $versions->register('3.1.4', 'action_scheduler_initialize_3_dot_1_dot_4'); |
|
| 40 | 40 | } |
| 41 | 41 | |
| 42 | 42 | function action_scheduler_initialize_3_dot_1_dot_4() { |
| 43 | - require_once( 'classes/abstracts/ActionScheduler.php' ); |
|
| 44 | - ActionScheduler::init( __FILE__ ); |
|
| 43 | + require_once('classes/abstracts/ActionScheduler.php'); |
|
| 44 | + ActionScheduler::init(__FILE__); |
|
| 45 | 45 | } |
| 46 | 46 | |
| 47 | 47 | // Support usage in themes - load this version if no plugin has loaded a version yet. |
| 48 | - if ( did_action( 'plugins_loaded' ) && ! class_exists( 'ActionScheduler' ) ) { |
|
| 48 | + if (did_action('plugins_loaded') && !class_exists('ActionScheduler')) { |
|
| 49 | 49 | action_scheduler_initialize_3_dot_1_dot_4(); |
| 50 | - do_action( 'action_scheduler_pre_theme_init' ); |
|
| 50 | + do_action('action_scheduler_pre_theme_init'); |
|
| 51 | 51 | ActionScheduler_Versions::initialize_latest_version(); |
| 52 | 52 | } |
| 53 | 53 | } |
@@ -10,30 +10,30 @@ discard block |
||
| 10 | 10 | private $store = NULL; |
| 11 | 11 | private $action_id = 0; |
| 12 | 12 | |
| 13 | - public function __construct( ActionScheduler_Store $store ) { |
|
| 13 | + public function __construct(ActionScheduler_Store $store) { |
|
| 14 | 14 | $this->store = $store; |
| 15 | 15 | } |
| 16 | 16 | |
| 17 | - public function attach( ActionScheduler_ActionClaim $claim ) { |
|
| 17 | + public function attach(ActionScheduler_ActionClaim $claim) { |
|
| 18 | 18 | $this->claim = $claim; |
| 19 | - add_action( 'shutdown', array( $this, 'handle_unexpected_shutdown' ) ); |
|
| 20 | - add_action( 'action_scheduler_before_execute', array( $this, 'track_current_action' ), 0, 1 ); |
|
| 21 | - add_action( 'action_scheduler_after_execute', array( $this, 'untrack_action' ), 0, 0 ); |
|
| 22 | - add_action( 'action_scheduler_execution_ignored', array( $this, 'untrack_action' ), 0, 0 ); |
|
| 23 | - add_action( 'action_scheduler_failed_execution', array( $this, 'untrack_action' ), 0, 0 ); |
|
| 19 | + add_action('shutdown', array($this, 'handle_unexpected_shutdown')); |
|
| 20 | + add_action('action_scheduler_before_execute', array($this, 'track_current_action'), 0, 1); |
|
| 21 | + add_action('action_scheduler_after_execute', array($this, 'untrack_action'), 0, 0); |
|
| 22 | + add_action('action_scheduler_execution_ignored', array($this, 'untrack_action'), 0, 0); |
|
| 23 | + add_action('action_scheduler_failed_execution', array($this, 'untrack_action'), 0, 0); |
|
| 24 | 24 | } |
| 25 | 25 | |
| 26 | 26 | public function detach() { |
| 27 | 27 | $this->claim = NULL; |
| 28 | 28 | $this->untrack_action(); |
| 29 | - remove_action( 'shutdown', array( $this, 'handle_unexpected_shutdown' ) ); |
|
| 30 | - remove_action( 'action_scheduler_before_execute', array( $this, 'track_current_action' ), 0 ); |
|
| 31 | - remove_action( 'action_scheduler_after_execute', array( $this, 'untrack_action' ), 0 ); |
|
| 32 | - remove_action( 'action_scheduler_execution_ignored', array( $this, 'untrack_action' ), 0 ); |
|
| 33 | - remove_action( 'action_scheduler_failed_execution', array( $this, 'untrack_action' ), 0 ); |
|
| 29 | + remove_action('shutdown', array($this, 'handle_unexpected_shutdown')); |
|
| 30 | + remove_action('action_scheduler_before_execute', array($this, 'track_current_action'), 0); |
|
| 31 | + remove_action('action_scheduler_after_execute', array($this, 'untrack_action'), 0); |
|
| 32 | + remove_action('action_scheduler_execution_ignored', array($this, 'untrack_action'), 0); |
|
| 33 | + remove_action('action_scheduler_failed_execution', array($this, 'untrack_action'), 0); |
|
| 34 | 34 | } |
| 35 | 35 | |
| 36 | - public function track_current_action( $action_id ) { |
|
| 36 | + public function track_current_action($action_id) { |
|
| 37 | 37 | $this->action_id = $action_id; |
| 38 | 38 | } |
| 39 | 39 | |
@@ -42,14 +42,14 @@ discard block |
||
| 42 | 42 | } |
| 43 | 43 | |
| 44 | 44 | public function handle_unexpected_shutdown() { |
| 45 | - if ( $error = error_get_last() ) { |
|
| 46 | - if ( in_array( $error['type'], array( E_ERROR, E_PARSE, E_COMPILE_ERROR, E_USER_ERROR, E_RECOVERABLE_ERROR ) ) ) { |
|
| 47 | - if ( !empty($this->action_id) ) { |
|
| 48 | - $this->store->mark_failure( $this->action_id ); |
|
| 49 | - do_action( 'action_scheduler_unexpected_shutdown', $this->action_id, $error ); |
|
| 45 | + if ($error = error_get_last()) { |
|
| 46 | + if (in_array($error['type'], array(E_ERROR, E_PARSE, E_COMPILE_ERROR, E_USER_ERROR, E_RECOVERABLE_ERROR))) { |
|
| 47 | + if (!empty($this->action_id)) { |
|
| 48 | + $this->store->mark_failure($this->action_id); |
|
| 49 | + do_action('action_scheduler_unexpected_shutdown', $this->action_id, $error); |
|
| 50 | 50 | } |
| 51 | 51 | } |
| 52 | - $this->store->release_claim( $this->claim ); |
|
| 52 | + $this->store->release_claim($this->claim); |
|
| 53 | 53 | } |
| 54 | 54 | } |
| 55 | 55 | } |
@@ -12,7 +12,7 @@ discard block |
||
| 12 | 12 | */ |
| 13 | 13 | protected $store; |
| 14 | 14 | |
| 15 | - function __construct( $store ) { |
|
| 15 | + function __construct($store) { |
|
| 16 | 16 | $this->store = $store; |
| 17 | 17 | } |
| 18 | 18 | |
@@ -24,9 +24,9 @@ discard block |
||
| 24 | 24 | public function render() { |
| 25 | 25 | $action_counts = $this->store->action_counts(); |
| 26 | 26 | $status_labels = $this->store->get_status_labels(); |
| 27 | - $oldest_and_newest = $this->get_oldest_and_newest( array_keys( $status_labels ) ); |
|
| 27 | + $oldest_and_newest = $this->get_oldest_and_newest(array_keys($status_labels)); |
|
| 28 | 28 | |
| 29 | - $this->get_template( $status_labels, $action_counts, $oldest_and_newest ); |
|
| 29 | + $this->get_template($status_labels, $action_counts, $oldest_and_newest); |
|
| 30 | 30 | } |
| 31 | 31 | |
| 32 | 32 | /** |
@@ -35,22 +35,22 @@ discard block |
||
| 35 | 35 | * @param array $status_keys Set of statuses to find oldest & newest action for. |
| 36 | 36 | * @return array |
| 37 | 37 | */ |
| 38 | - protected function get_oldest_and_newest( $status_keys ) { |
|
| 38 | + protected function get_oldest_and_newest($status_keys) { |
|
| 39 | 39 | |
| 40 | 40 | $oldest_and_newest = array(); |
| 41 | 41 | |
| 42 | - foreach ( $status_keys as $status ) { |
|
| 43 | - $oldest_and_newest[ $status ] = array( |
|
| 42 | + foreach ($status_keys as $status) { |
|
| 43 | + $oldest_and_newest[$status] = array( |
|
| 44 | 44 | 'oldest' => '–', |
| 45 | 45 | 'newest' => '–', |
| 46 | 46 | ); |
| 47 | 47 | |
| 48 | - if ( 'in-progress' === $status ) { |
|
| 48 | + if ('in-progress' === $status) { |
|
| 49 | 49 | continue; |
| 50 | 50 | } |
| 51 | 51 | |
| 52 | - $oldest_and_newest[ $status ]['oldest'] = $this->get_action_status_date( $status, 'oldest' ); |
|
| 53 | - $oldest_and_newest[ $status ]['newest'] = $this->get_action_status_date( $status, 'newest' ); |
|
| 52 | + $oldest_and_newest[$status]['oldest'] = $this->get_action_status_date($status, 'oldest'); |
|
| 53 | + $oldest_and_newest[$status]['newest'] = $this->get_action_status_date($status, 'newest'); |
|
| 54 | 54 | } |
| 55 | 55 | |
| 56 | 56 | return $oldest_and_newest; |
@@ -63,20 +63,20 @@ discard block |
||
| 63 | 63 | * @param string $date_type Oldest or Newest. |
| 64 | 64 | * @return DateTime |
| 65 | 65 | */ |
| 66 | - protected function get_action_status_date( $status, $date_type = 'oldest' ) { |
|
| 66 | + protected function get_action_status_date($status, $date_type = 'oldest') { |
|
| 67 | 67 | |
| 68 | 68 | $order = 'oldest' === $date_type ? 'ASC' : 'DESC'; |
| 69 | 69 | |
| 70 | - $action = $this->store->query_actions( array( |
|
| 70 | + $action = $this->store->query_actions(array( |
|
| 71 | 71 | 'claimed' => false, |
| 72 | 72 | 'status' => $status, |
| 73 | 73 | 'per_page' => 1, |
| 74 | 74 | 'order' => $order, |
| 75 | - ) ); |
|
| 75 | + )); |
|
| 76 | 76 | |
| 77 | - if ( ! empty( $action ) ) { |
|
| 78 | - $date_object = $this->store->get_date( $action[0] ); |
|
| 79 | - $action_date = $date_object->format( 'Y-m-d H:i:s O' ); |
|
| 77 | + if (!empty($action)) { |
|
| 78 | + $date_object = $this->store->get_date($action[0]); |
|
| 79 | + $action_date = $date_object->format('Y-m-d H:i:s O'); |
|
| 80 | 80 | } else { |
| 81 | 81 | $action_date = '–'; |
| 82 | 82 | } |
@@ -91,37 +91,37 @@ discard block |
||
| 91 | 91 | * @param array $action_counts Number of actions grouped by status. |
| 92 | 92 | * @param array $oldest_and_newest Date of the oldest and newest action with each status. |
| 93 | 93 | */ |
| 94 | - protected function get_template( $status_labels, $action_counts, $oldest_and_newest ) { |
|
| 94 | + protected function get_template($status_labels, $action_counts, $oldest_and_newest) { |
|
| 95 | 95 | $as_version = ActionScheduler_Versions::instance()->latest_version(); |
| 96 | 96 | ?> |
| 97 | 97 | |
| 98 | 98 | <table class="wc_status_table widefat" cellspacing="0"> |
| 99 | 99 | <thead> |
| 100 | 100 | <tr> |
| 101 | - <th colspan="5" data-export-label="Action Scheduler"><h2><?php esc_html_e( 'Action Scheduler', 'action-scheduler' ); ?><?php echo wc_help_tip( esc_html__( 'This section shows scheduled action counts.', 'action-scheduler' ) ); ?></h2></th> |
|
| 101 | + <th colspan="5" data-export-label="Action Scheduler"><h2><?php esc_html_e('Action Scheduler', 'action-scheduler'); ?><?php echo wc_help_tip(esc_html__('This section shows scheduled action counts.', 'action-scheduler')); ?></h2></th> |
|
| 102 | 102 | </tr> |
| 103 | 103 | <tr> |
| 104 | - <td colspan="2" data-export-label="Version"><?php esc_html_e( 'Version:', 'action-scheduler' ); ?></td> |
|
| 105 | - <td colspan="3"><?php echo esc_html( $as_version ); ?></td> |
|
| 104 | + <td colspan="2" data-export-label="Version"><?php esc_html_e('Version:', 'action-scheduler'); ?></td> |
|
| 105 | + <td colspan="3"><?php echo esc_html($as_version); ?></td> |
|
| 106 | 106 | </tr> |
| 107 | 107 | <tr> |
| 108 | - <td><strong><?php esc_html_e( 'Action Status', 'action-scheduler' ); ?></strong></td> |
|
| 108 | + <td><strong><?php esc_html_e('Action Status', 'action-scheduler'); ?></strong></td> |
|
| 109 | 109 | <td class="help"> </td> |
| 110 | - <td><strong><?php esc_html_e( 'Count', 'action-scheduler' ); ?></strong></td> |
|
| 111 | - <td><strong><?php esc_html_e( 'Oldest Scheduled Date', 'action-scheduler' ); ?></strong></td> |
|
| 112 | - <td><strong><?php esc_html_e( 'Newest Scheduled Date', 'action-scheduler' ); ?></strong></td> |
|
| 110 | + <td><strong><?php esc_html_e('Count', 'action-scheduler'); ?></strong></td> |
|
| 111 | + <td><strong><?php esc_html_e('Oldest Scheduled Date', 'action-scheduler'); ?></strong></td> |
|
| 112 | + <td><strong><?php esc_html_e('Newest Scheduled Date', 'action-scheduler'); ?></strong></td> |
|
| 113 | 113 | </tr> |
| 114 | 114 | </thead> |
| 115 | 115 | <tbody> |
| 116 | 116 | <?php |
| 117 | - foreach ( $action_counts as $status => $count ) { |
|
| 117 | + foreach ($action_counts as $status => $count) { |
|
| 118 | 118 | // WC uses the 3rd column for export, so we need to display more data in that (hidden when viewed as part of the table) and add an empty 2nd column. |
| 119 | 119 | printf( |
| 120 | 120 | '<tr><td>%1$s</td><td> </td><td>%2$s<span style="display: none;">, Oldest: %3$s, Newest: %4$s</span></td><td>%3$s</td><td>%4$s</td></tr>', |
| 121 | - esc_html( $status_labels[ $status ] ), |
|
| 122 | - number_format_i18n( $count ), |
|
| 123 | - $oldest_and_newest[ $status ]['oldest'], |
|
| 124 | - $oldest_and_newest[ $status ]['newest'] |
|
| 121 | + esc_html($status_labels[$status]), |
|
| 122 | + number_format_i18n($count), |
|
| 123 | + $oldest_and_newest[$status]['oldest'], |
|
| 124 | + $oldest_and_newest[$status]['newest'] |
|
| 125 | 125 | ); |
| 126 | 126 | } |
| 127 | 127 | ?> |
@@ -140,11 +140,11 @@ discard block |
||
| 140 | 140 | * @return mixed |
| 141 | 141 | * @link https://php.net/manual/en/language.oop5.overloading.php#language.oop5.overloading.methods |
| 142 | 142 | */ |
| 143 | - public function __call( $name, $arguments ) { |
|
| 144 | - switch ( $name ) { |
|
| 143 | + public function __call($name, $arguments) { |
|
| 144 | + switch ($name) { |
|
| 145 | 145 | case 'print': |
| 146 | - _deprecated_function( __CLASS__ . '::print()', '2.2.4', __CLASS__ . '::render()' ); |
|
| 147 | - return call_user_func_array( array( $this, 'render' ), $arguments ); |
|
| 146 | + _deprecated_function(__CLASS__ . '::print()', '2.2.4', __CLASS__ . '::render()'); |
|
| 147 | + return call_user_func_array(array($this, 'render'), $arguments); |
|
| 148 | 148 | } |
| 149 | 149 | |
| 150 | 150 | return null; |
@@ -6,8 +6,8 @@ discard block |
||
| 6 | 6 | */ |
| 7 | 7 | class ActionScheduler_wpPostStore_PostStatusRegistrar { |
| 8 | 8 | public function register() { |
| 9 | - register_post_status( ActionScheduler_Store::STATUS_RUNNING, array_merge( $this->post_status_args(), $this->post_status_running_labels() ) ); |
|
| 10 | - register_post_status( ActionScheduler_Store::STATUS_FAILED, array_merge( $this->post_status_args(), $this->post_status_failed_labels() ) ); |
|
| 9 | + register_post_status(ActionScheduler_Store::STATUS_RUNNING, array_merge($this->post_status_args(), $this->post_status_running_labels())); |
|
| 10 | + register_post_status(ActionScheduler_Store::STATUS_FAILED, array_merge($this->post_status_args(), $this->post_status_failed_labels())); |
|
| 11 | 11 | } |
| 12 | 12 | |
| 13 | 13 | /** |
@@ -23,7 +23,7 @@ discard block |
||
| 23 | 23 | 'show_in_admin_status_list' => true, |
| 24 | 24 | ); |
| 25 | 25 | |
| 26 | - return apply_filters( 'action_scheduler_post_status_args', $args ); |
|
| 26 | + return apply_filters('action_scheduler_post_status_args', $args); |
|
| 27 | 27 | } |
| 28 | 28 | |
| 29 | 29 | /** |
@@ -33,12 +33,12 @@ discard block |
||
| 33 | 33 | */ |
| 34 | 34 | protected function post_status_failed_labels() { |
| 35 | 35 | $labels = array( |
| 36 | - 'label' => _x( 'Failed', 'post', 'action-scheduler' ), |
|
| 36 | + 'label' => _x('Failed', 'post', 'action-scheduler'), |
|
| 37 | 37 | /* translators: %s: count */ |
| 38 | - 'label_count' => _n_noop( 'Failed <span class="count">(%s)</span>', 'Failed <span class="count">(%s)</span>', 'action-scheduler' ), |
|
| 38 | + 'label_count' => _n_noop('Failed <span class="count">(%s)</span>', 'Failed <span class="count">(%s)</span>', 'action-scheduler'), |
|
| 39 | 39 | ); |
| 40 | 40 | |
| 41 | - return apply_filters( 'action_scheduler_post_status_failed_labels', $labels ); |
|
| 41 | + return apply_filters('action_scheduler_post_status_failed_labels', $labels); |
|
| 42 | 42 | } |
| 43 | 43 | |
| 44 | 44 | /** |
@@ -48,11 +48,11 @@ discard block |
||
| 48 | 48 | */ |
| 49 | 49 | protected function post_status_running_labels() { |
| 50 | 50 | $labels = array( |
| 51 | - 'label' => _x( 'In-Progress', 'post', 'action-scheduler' ), |
|
| 51 | + 'label' => _x('In-Progress', 'post', 'action-scheduler'), |
|
| 52 | 52 | /* translators: %s: count */ |
| 53 | - 'label_count' => _n_noop( 'In-Progress <span class="count">(%s)</span>', 'In-Progress <span class="count">(%s)</span>', 'action-scheduler' ), |
|
| 53 | + 'label_count' => _n_noop('In-Progress <span class="count">(%s)</span>', 'In-Progress <span class="count">(%s)</span>', 'action-scheduler'), |
|
| 54 | 54 | ); |
| 55 | 55 | |
| 56 | - return apply_filters( 'action_scheduler_post_status_running_labels', $labels ); |
|
| 56 | + return apply_filters('action_scheduler_post_status_running_labels', $labels); |
|
| 57 | 57 | } |
| 58 | 58 | } |
@@ -37,14 +37,14 @@ discard block |
||
| 37 | 37 | * |
| 38 | 38 | * @param Config $config Migration config object. |
| 39 | 39 | */ |
| 40 | - public function __construct( Config $config = null ) { |
|
| 41 | - $this->demarkation_id = (int) get_option( self::DEMARKATION_OPTION, 0 ); |
|
| 42 | - if ( empty( $config ) ) { |
|
| 40 | + public function __construct(Config $config = null) { |
|
| 41 | + $this->demarkation_id = (int) get_option(self::DEMARKATION_OPTION, 0); |
|
| 42 | + if (empty($config)) { |
|
| 43 | 43 | $config = Controller::instance()->get_migration_config_object(); |
| 44 | 44 | } |
| 45 | 45 | $this->primary_store = $config->get_destination_store(); |
| 46 | 46 | $this->secondary_store = $config->get_source_store(); |
| 47 | - $this->migration_runner = new Runner( $config ); |
|
| 47 | + $this->migration_runner = new Runner($config); |
|
| 48 | 48 | } |
| 49 | 49 | |
| 50 | 50 | /** |
@@ -53,10 +53,10 @@ discard block |
||
| 53 | 53 | * @codeCoverageIgnore |
| 54 | 54 | */ |
| 55 | 55 | public function init() { |
| 56 | - add_action( 'action_scheduler/created_table', [ $this, 'set_autoincrement' ], 10, 2 ); |
|
| 56 | + add_action('action_scheduler/created_table', [$this, 'set_autoincrement'], 10, 2); |
|
| 57 | 57 | $this->primary_store->init(); |
| 58 | 58 | $this->secondary_store->init(); |
| 59 | - remove_action( 'action_scheduler/created_table', [ $this, 'set_autoincrement' ], 10 ); |
|
| 59 | + remove_action('action_scheduler/created_table', [$this, 'set_autoincrement'], 10); |
|
| 60 | 60 | } |
| 61 | 61 | |
| 62 | 62 | /** |
@@ -70,9 +70,9 @@ discard block |
||
| 70 | 70 | * @return void |
| 71 | 71 | * @codeCoverageIgnore |
| 72 | 72 | */ |
| 73 | - public function set_autoincrement( $table_name, $table_suffix ) { |
|
| 74 | - if ( ActionScheduler_StoreSchema::ACTIONS_TABLE === $table_suffix ) { |
|
| 75 | - if ( empty( $this->demarkation_id ) ) { |
|
| 73 | + public function set_autoincrement($table_name, $table_suffix) { |
|
| 74 | + if (ActionScheduler_StoreSchema::ACTIONS_TABLE === $table_suffix) { |
|
| 75 | + if (empty($this->demarkation_id)) { |
|
| 76 | 76 | $this->demarkation_id = $this->set_demarkation_id(); |
| 77 | 77 | } |
| 78 | 78 | /** @var \wpdb $wpdb */ |
@@ -87,7 +87,7 @@ discard block |
||
| 87 | 87 | ); |
| 88 | 88 | $wpdb->delete( |
| 89 | 89 | $wpdb->{ActionScheduler_StoreSchema::ACTIONS_TABLE}, |
| 90 | - [ 'action_id' => $this->demarkation_id ] |
|
| 90 | + ['action_id' => $this->demarkation_id] |
|
| 91 | 91 | ); |
| 92 | 92 | } |
| 93 | 93 | } |
@@ -102,14 +102,14 @@ discard block |
||
| 102 | 102 | * |
| 103 | 103 | * @codeCoverageIgnore |
| 104 | 104 | */ |
| 105 | - private function set_demarkation_id( $id = null ) { |
|
| 106 | - if ( empty( $id ) ) { |
|
| 105 | + private function set_demarkation_id($id = null) { |
|
| 106 | + if (empty($id)) { |
|
| 107 | 107 | /** @var \wpdb $wpdb */ |
| 108 | 108 | global $wpdb; |
| 109 | - $id = (int) $wpdb->get_var( "SELECT MAX(ID) FROM $wpdb->posts" ); |
|
| 110 | - $id ++; |
|
| 109 | + $id = (int) $wpdb->get_var("SELECT MAX(ID) FROM $wpdb->posts"); |
|
| 110 | + $id++; |
|
| 111 | 111 | } |
| 112 | - update_option( self::DEMARKATION_OPTION, $id ); |
|
| 112 | + update_option(self::DEMARKATION_OPTION, $id); |
|
| 113 | 113 | |
| 114 | 114 | return $id; |
| 115 | 115 | } |
@@ -125,13 +125,13 @@ discard block |
||
| 125 | 125 | * |
| 126 | 126 | * @return string |
| 127 | 127 | */ |
| 128 | - public function find_action( $hook, $params = [] ) { |
|
| 129 | - $found_unmigrated_action = $this->secondary_store->find_action( $hook, $params ); |
|
| 130 | - if ( ! empty( $found_unmigrated_action ) ) { |
|
| 131 | - $this->migrate( [ $found_unmigrated_action ] ); |
|
| 128 | + public function find_action($hook, $params = []) { |
|
| 129 | + $found_unmigrated_action = $this->secondary_store->find_action($hook, $params); |
|
| 130 | + if (!empty($found_unmigrated_action)) { |
|
| 131 | + $this->migrate([$found_unmigrated_action]); |
|
| 132 | 132 | } |
| 133 | 133 | |
| 134 | - return $this->primary_store->find_action( $hook, $params ); |
|
| 134 | + return $this->primary_store->find_action($hook, $params); |
|
| 135 | 135 | } |
| 136 | 136 | |
| 137 | 137 | /** |
@@ -144,13 +144,13 @@ discard block |
||
| 144 | 144 | * |
| 145 | 145 | * @return int[] |
| 146 | 146 | */ |
| 147 | - public function query_actions( $query = [], $query_type = 'select' ) { |
|
| 148 | - $found_unmigrated_actions = $this->secondary_store->query_actions( $query, 'select' ); |
|
| 149 | - if ( ! empty( $found_unmigrated_actions ) ) { |
|
| 150 | - $this->migrate( $found_unmigrated_actions ); |
|
| 147 | + public function query_actions($query = [], $query_type = 'select') { |
|
| 148 | + $found_unmigrated_actions = $this->secondary_store->query_actions($query, 'select'); |
|
| 149 | + if (!empty($found_unmigrated_actions)) { |
|
| 150 | + $this->migrate($found_unmigrated_actions); |
|
| 151 | 151 | } |
| 152 | 152 | |
| 153 | - return $this->primary_store->query_actions( $query, $query_type ); |
|
| 153 | + return $this->primary_store->query_actions($query, $query_type); |
|
| 154 | 154 | } |
| 155 | 155 | |
| 156 | 156 | /** |
@@ -163,22 +163,22 @@ discard block |
||
| 163 | 163 | $migrated_actions_count = $this->primary_store->action_counts(); |
| 164 | 164 | $actions_count_by_status = array(); |
| 165 | 165 | |
| 166 | - foreach ( $this->get_status_labels() as $status_key => $status_label ) { |
|
| 166 | + foreach ($this->get_status_labels() as $status_key => $status_label) { |
|
| 167 | 167 | |
| 168 | 168 | $count = 0; |
| 169 | 169 | |
| 170 | - if ( isset( $unmigrated_actions_count[ $status_key ] ) ) { |
|
| 171 | - $count += $unmigrated_actions_count[ $status_key ]; |
|
| 170 | + if (isset($unmigrated_actions_count[$status_key])) { |
|
| 171 | + $count += $unmigrated_actions_count[$status_key]; |
|
| 172 | 172 | } |
| 173 | 173 | |
| 174 | - if ( isset( $migrated_actions_count[ $status_key ] ) ) { |
|
| 175 | - $count += $migrated_actions_count[ $status_key ]; |
|
| 174 | + if (isset($migrated_actions_count[$status_key])) { |
|
| 175 | + $count += $migrated_actions_count[$status_key]; |
|
| 176 | 176 | } |
| 177 | 177 | |
| 178 | - $actions_count_by_status[ $status_key ] = $count; |
|
| 178 | + $actions_count_by_status[$status_key] = $count; |
|
| 179 | 179 | } |
| 180 | 180 | |
| 181 | - $actions_count_by_status = array_filter( $actions_count_by_status ); |
|
| 181 | + $actions_count_by_status = array_filter($actions_count_by_status); |
|
| 182 | 182 | |
| 183 | 183 | return $actions_count_by_status; |
| 184 | 184 | } |
@@ -193,17 +193,17 @@ discard block |
||
| 193 | 193 | * |
| 194 | 194 | * @return ActionScheduler_ActionClaim |
| 195 | 195 | */ |
| 196 | - public function stake_claim( $max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '' ) { |
|
| 197 | - $claim = $this->secondary_store->stake_claim( $max_actions, $before_date, $hooks, $group ); |
|
| 196 | + public function stake_claim($max_actions = 10, DateTime $before_date = null, $hooks = array(), $group = '') { |
|
| 197 | + $claim = $this->secondary_store->stake_claim($max_actions, $before_date, $hooks, $group); |
|
| 198 | 198 | |
| 199 | 199 | $claimed_actions = $claim->get_actions(); |
| 200 | - if ( ! empty( $claimed_actions ) ) { |
|
| 201 | - $this->migrate( $claimed_actions ); |
|
| 200 | + if (!empty($claimed_actions)) { |
|
| 201 | + $this->migrate($claimed_actions); |
|
| 202 | 202 | } |
| 203 | 203 | |
| 204 | - $this->secondary_store->release_claim( $claim ); |
|
| 204 | + $this->secondary_store->release_claim($claim); |
|
| 205 | 205 | |
| 206 | - return $this->primary_store->stake_claim( $max_actions, $before_date, $hooks, $group ); |
|
| 206 | + return $this->primary_store->stake_claim($max_actions, $before_date, $hooks, $group); |
|
| 207 | 207 | } |
| 208 | 208 | |
| 209 | 209 | /** |
@@ -211,8 +211,8 @@ discard block |
||
| 211 | 211 | * |
| 212 | 212 | * @param array $action_ids List of action IDs. |
| 213 | 213 | */ |
| 214 | - private function migrate( $action_ids ) { |
|
| 215 | - $this->migration_runner->migrate_actions( $action_ids ); |
|
| 214 | + private function migrate($action_ids) { |
|
| 215 | + $this->migration_runner->migrate_actions($action_ids); |
|
| 216 | 216 | } |
| 217 | 217 | |
| 218 | 218 | /** |
@@ -221,8 +221,8 @@ discard block |
||
| 221 | 221 | * @param ActionScheduler_Action $action Action object to be saved. |
| 222 | 222 | * @param DateTime $date Optional. Schedule date. Default null. |
| 223 | 223 | */ |
| 224 | - public function save_action( ActionScheduler_Action $action, DateTime $date = null ) { |
|
| 225 | - return $this->primary_store->save_action( $action, $date ); |
|
| 224 | + public function save_action(ActionScheduler_Action $action, DateTime $date = null) { |
|
| 225 | + return $this->primary_store->save_action($action, $date); |
|
| 226 | 226 | } |
| 227 | 227 | |
| 228 | 228 | /** |
@@ -230,11 +230,11 @@ discard block |
||
| 230 | 230 | * |
| 231 | 231 | * @param int $action_id Action ID. |
| 232 | 232 | */ |
| 233 | - public function fetch_action( $action_id ) { |
|
| 234 | - if ( $action_id < $this->demarkation_id ) { |
|
| 235 | - return $this->secondary_store->fetch_action( $action_id ); |
|
| 233 | + public function fetch_action($action_id) { |
|
| 234 | + if ($action_id < $this->demarkation_id) { |
|
| 235 | + return $this->secondary_store->fetch_action($action_id); |
|
| 236 | 236 | } else { |
| 237 | - return $this->primary_store->fetch_action( $action_id ); |
|
| 237 | + return $this->primary_store->fetch_action($action_id); |
|
| 238 | 238 | } |
| 239 | 239 | } |
| 240 | 240 | |
@@ -243,11 +243,11 @@ discard block |
||
| 243 | 243 | * |
| 244 | 244 | * @param int $action_id Action ID. |
| 245 | 245 | */ |
| 246 | - public function cancel_action( $action_id ) { |
|
| 247 | - if ( $action_id < $this->demarkation_id ) { |
|
| 248 | - $this->secondary_store->cancel_action( $action_id ); |
|
| 246 | + public function cancel_action($action_id) { |
|
| 247 | + if ($action_id < $this->demarkation_id) { |
|
| 248 | + $this->secondary_store->cancel_action($action_id); |
|
| 249 | 249 | } else { |
| 250 | - $this->primary_store->cancel_action( $action_id ); |
|
| 250 | + $this->primary_store->cancel_action($action_id); |
|
| 251 | 251 | } |
| 252 | 252 | } |
| 253 | 253 | |
@@ -256,11 +256,11 @@ discard block |
||
| 256 | 256 | * |
| 257 | 257 | * @param int $action_id Action ID. |
| 258 | 258 | */ |
| 259 | - public function delete_action( $action_id ) { |
|
| 260 | - if ( $action_id < $this->demarkation_id ) { |
|
| 261 | - $this->secondary_store->delete_action( $action_id ); |
|
| 259 | + public function delete_action($action_id) { |
|
| 260 | + if ($action_id < $this->demarkation_id) { |
|
| 261 | + $this->secondary_store->delete_action($action_id); |
|
| 262 | 262 | } else { |
| 263 | - $this->primary_store->delete_action( $action_id ); |
|
| 263 | + $this->primary_store->delete_action($action_id); |
|
| 264 | 264 | } |
| 265 | 265 | } |
| 266 | 266 | |
@@ -269,11 +269,11 @@ discard block |
||
| 269 | 269 | * |
| 270 | 270 | * @param int $action_id Action ID. |
| 271 | 271 | */ |
| 272 | - public function get_date( $action_id ) { |
|
| 273 | - if ( $action_id < $this->demarkation_id ) { |
|
| 274 | - return $this->secondary_store->get_date( $action_id ); |
|
| 272 | + public function get_date($action_id) { |
|
| 273 | + if ($action_id < $this->demarkation_id) { |
|
| 274 | + return $this->secondary_store->get_date($action_id); |
|
| 275 | 275 | } else { |
| 276 | - return $this->primary_store->get_date( $action_id ); |
|
| 276 | + return $this->primary_store->get_date($action_id); |
|
| 277 | 277 | } |
| 278 | 278 | } |
| 279 | 279 | |
@@ -282,11 +282,11 @@ discard block |
||
| 282 | 282 | * |
| 283 | 283 | * @param int $action_id Action ID. |
| 284 | 284 | */ |
| 285 | - public function mark_failure( $action_id ) { |
|
| 286 | - if ( $action_id < $this->demarkation_id ) { |
|
| 287 | - $this->secondary_store->mark_failure( $action_id ); |
|
| 285 | + public function mark_failure($action_id) { |
|
| 286 | + if ($action_id < $this->demarkation_id) { |
|
| 287 | + $this->secondary_store->mark_failure($action_id); |
|
| 288 | 288 | } else { |
| 289 | - $this->primary_store->mark_failure( $action_id ); |
|
| 289 | + $this->primary_store->mark_failure($action_id); |
|
| 290 | 290 | } |
| 291 | 291 | } |
| 292 | 292 | |
@@ -295,11 +295,11 @@ discard block |
||
| 295 | 295 | * |
| 296 | 296 | * @param int $action_id Action ID. |
| 297 | 297 | */ |
| 298 | - public function log_execution( $action_id ) { |
|
| 299 | - if ( $action_id < $this->demarkation_id ) { |
|
| 300 | - $this->secondary_store->log_execution( $action_id ); |
|
| 298 | + public function log_execution($action_id) { |
|
| 299 | + if ($action_id < $this->demarkation_id) { |
|
| 300 | + $this->secondary_store->log_execution($action_id); |
|
| 301 | 301 | } else { |
| 302 | - $this->primary_store->log_execution( $action_id ); |
|
| 302 | + $this->primary_store->log_execution($action_id); |
|
| 303 | 303 | } |
| 304 | 304 | } |
| 305 | 305 | |
@@ -308,11 +308,11 @@ discard block |
||
| 308 | 308 | * |
| 309 | 309 | * @param int $action_id Action ID. |
| 310 | 310 | */ |
| 311 | - public function mark_complete( $action_id ) { |
|
| 312 | - if ( $action_id < $this->demarkation_id ) { |
|
| 313 | - $this->secondary_store->mark_complete( $action_id ); |
|
| 311 | + public function mark_complete($action_id) { |
|
| 312 | + if ($action_id < $this->demarkation_id) { |
|
| 313 | + $this->secondary_store->mark_complete($action_id); |
|
| 314 | 314 | } else { |
| 315 | - $this->primary_store->mark_complete( $action_id ); |
|
| 315 | + $this->primary_store->mark_complete($action_id); |
|
| 316 | 316 | } |
| 317 | 317 | } |
| 318 | 318 | |
@@ -321,11 +321,11 @@ discard block |
||
| 321 | 321 | * |
| 322 | 322 | * @param int $action_id Action ID. |
| 323 | 323 | */ |
| 324 | - public function get_status( $action_id ) { |
|
| 325 | - if ( $action_id < $this->demarkation_id ) { |
|
| 326 | - return $this->secondary_store->get_status( $action_id ); |
|
| 324 | + public function get_status($action_id) { |
|
| 325 | + if ($action_id < $this->demarkation_id) { |
|
| 326 | + return $this->secondary_store->get_status($action_id); |
|
| 327 | 327 | } else { |
| 328 | - return $this->primary_store->get_status( $action_id ); |
|
| 328 | + return $this->primary_store->get_status($action_id); |
|
| 329 | 329 | } |
| 330 | 330 | } |
| 331 | 331 | |
@@ -347,8 +347,8 @@ discard block |
||
| 347 | 347 | * |
| 348 | 348 | * @param int $action_id Action ID. |
| 349 | 349 | */ |
| 350 | - public function get_claim_id( $action_id ) { |
|
| 351 | - return $this->primary_store->get_claim_id( $action_id ); |
|
| 350 | + public function get_claim_id($action_id) { |
|
| 351 | + return $this->primary_store->get_claim_id($action_id); |
|
| 352 | 352 | } |
| 353 | 353 | |
| 354 | 354 | /** |
@@ -356,8 +356,8 @@ discard block |
||
| 356 | 356 | * |
| 357 | 357 | * @param ActionScheduler_ActionClaim $claim Claim object. |
| 358 | 358 | */ |
| 359 | - public function release_claim( ActionScheduler_ActionClaim $claim ) { |
|
| 360 | - $this->primary_store->release_claim( $claim ); |
|
| 359 | + public function release_claim(ActionScheduler_ActionClaim $claim) { |
|
| 360 | + $this->primary_store->release_claim($claim); |
|
| 361 | 361 | } |
| 362 | 362 | |
| 363 | 363 | /** |
@@ -365,8 +365,8 @@ discard block |
||
| 365 | 365 | * |
| 366 | 366 | * @param int $action_id Action ID. |
| 367 | 367 | */ |
| 368 | - public function unclaim_action( $action_id ) { |
|
| 369 | - $this->primary_store->unclaim_action( $action_id ); |
|
| 368 | + public function unclaim_action($action_id) { |
|
| 369 | + $this->primary_store->unclaim_action($action_id); |
|
| 370 | 370 | } |
| 371 | 371 | |
| 372 | 372 | /** |
@@ -374,7 +374,7 @@ discard block |
||
| 374 | 374 | * |
| 375 | 375 | * @param int $claim_id Claim ID. |
| 376 | 376 | */ |
| 377 | - public function find_actions_by_claim_id( $claim_id ) { |
|
| 378 | - return $this->primary_store->find_actions_by_claim_id( $claim_id ); |
|
| 377 | + public function find_actions_by_claim_id($claim_id) { |
|
| 378 | + return $this->primary_store->find_actions_by_claim_id($claim_id); |
|
| 379 | 379 | } |
| 380 | 380 | } |
@@ -18,25 +18,25 @@ discard block |
||
| 18 | 18 | * |
| 19 | 19 | * @return int The log entry ID. |
| 20 | 20 | */ |
| 21 | - public function log( $action_id, $message, DateTime $date = null ) { |
|
| 22 | - if ( empty( $date ) ) { |
|
| 21 | + public function log($action_id, $message, DateTime $date = null) { |
|
| 22 | + if (empty($date)) { |
|
| 23 | 23 | $date = as_get_datetime_object(); |
| 24 | 24 | } else { |
| 25 | 25 | $date = clone $date; |
| 26 | 26 | } |
| 27 | 27 | |
| 28 | - $date_gmt = $date->format( 'Y-m-d H:i:s' ); |
|
| 29 | - ActionScheduler_TimezoneHelper::set_local_timezone( $date ); |
|
| 30 | - $date_local = $date->format( 'Y-m-d H:i:s' ); |
|
| 28 | + $date_gmt = $date->format('Y-m-d H:i:s'); |
|
| 29 | + ActionScheduler_TimezoneHelper::set_local_timezone($date); |
|
| 30 | + $date_local = $date->format('Y-m-d H:i:s'); |
|
| 31 | 31 | |
| 32 | 32 | /** @var \wpdb $wpdb */ |
| 33 | 33 | global $wpdb; |
| 34 | - $wpdb->insert( $wpdb->actionscheduler_logs, [ |
|
| 34 | + $wpdb->insert($wpdb->actionscheduler_logs, [ |
|
| 35 | 35 | 'action_id' => $action_id, |
| 36 | 36 | 'message' => $message, |
| 37 | 37 | 'log_date_gmt' => $date_gmt, |
| 38 | 38 | 'log_date_local' => $date_local, |
| 39 | - ], [ '%d', '%s', '%s', '%s' ] ); |
|
| 39 | + ], ['%d', '%s', '%s', '%s']); |
|
| 40 | 40 | |
| 41 | 41 | return $wpdb->insert_id; |
| 42 | 42 | } |
@@ -48,12 +48,12 @@ discard block |
||
| 48 | 48 | * |
| 49 | 49 | * @return ActionScheduler_LogEntry |
| 50 | 50 | */ |
| 51 | - public function get_entry( $entry_id ) { |
|
| 51 | + public function get_entry($entry_id) { |
|
| 52 | 52 | /** @var \wpdb $wpdb */ |
| 53 | 53 | global $wpdb; |
| 54 | - $entry = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_logs} WHERE log_id=%d", $entry_id ) ); |
|
| 54 | + $entry = $wpdb->get_row($wpdb->prepare("SELECT * FROM {$wpdb->actionscheduler_logs} WHERE log_id=%d", $entry_id)); |
|
| 55 | 55 | |
| 56 | - return $this->create_entry_from_db_record( $entry ); |
|
| 56 | + return $this->create_entry_from_db_record($entry); |
|
| 57 | 57 | } |
| 58 | 58 | |
| 59 | 59 | /** |
@@ -63,14 +63,14 @@ discard block |
||
| 63 | 63 | * |
| 64 | 64 | * @return ActionScheduler_LogEntry |
| 65 | 65 | */ |
| 66 | - private function create_entry_from_db_record( $record ) { |
|
| 67 | - if ( empty( $record ) ) { |
|
| 66 | + private function create_entry_from_db_record($record) { |
|
| 67 | + if (empty($record)) { |
|
| 68 | 68 | return new ActionScheduler_NullLogEntry(); |
| 69 | 69 | } |
| 70 | 70 | |
| 71 | - $date = as_get_datetime_object( $record->log_date_gmt ); |
|
| 71 | + $date = as_get_datetime_object($record->log_date_gmt); |
|
| 72 | 72 | |
| 73 | - return new ActionScheduler_LogEntry( $record->action_id, $record->message, $date ); |
|
| 73 | + return new ActionScheduler_LogEntry($record->action_id, $record->message, $date); |
|
| 74 | 74 | } |
| 75 | 75 | |
| 76 | 76 | /** |
@@ -80,13 +80,13 @@ discard block |
||
| 80 | 80 | * |
| 81 | 81 | * @return ActionScheduler_LogEntry[] |
| 82 | 82 | */ |
| 83 | - public function get_logs( $action_id ) { |
|
| 83 | + public function get_logs($action_id) { |
|
| 84 | 84 | /** @var \wpdb $wpdb */ |
| 85 | 85 | global $wpdb; |
| 86 | 86 | |
| 87 | - $records = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_logs} WHERE action_id=%d", $action_id ) ); |
|
| 87 | + $records = $wpdb->get_results($wpdb->prepare("SELECT * FROM {$wpdb->actionscheduler_logs} WHERE action_id=%d", $action_id)); |
|
| 88 | 88 | |
| 89 | - return array_map( [ $this, 'create_entry_from_db_record' ], $records ); |
|
| 89 | + return array_map([$this, 'create_entry_from_db_record'], $records); |
|
| 90 | 90 | } |
| 91 | 91 | |
| 92 | 92 | /** |
@@ -101,7 +101,7 @@ discard block |
||
| 101 | 101 | |
| 102 | 102 | parent::init(); |
| 103 | 103 | |
| 104 | - add_action( 'action_scheduler_deleted_action', [ $this, 'clear_deleted_action_logs' ], 10, 1 ); |
|
| 104 | + add_action('action_scheduler_deleted_action', [$this, 'clear_deleted_action_logs'], 10, 1); |
|
| 105 | 105 | } |
| 106 | 106 | |
| 107 | 107 | /** |
@@ -109,10 +109,10 @@ discard block |
||
| 109 | 109 | * |
| 110 | 110 | * @param int $action_id Action ID. |
| 111 | 111 | */ |
| 112 | - public function clear_deleted_action_logs( $action_id ) { |
|
| 112 | + public function clear_deleted_action_logs($action_id) { |
|
| 113 | 113 | /** @var \wpdb $wpdb */ |
| 114 | 114 | global $wpdb; |
| 115 | - $wpdb->delete( $wpdb->actionscheduler_logs, [ 'action_id' => $action_id, ], [ '%d' ] ); |
|
| 115 | + $wpdb->delete($wpdb->actionscheduler_logs, ['action_id' => $action_id, ], ['%d']); |
|
| 116 | 116 | } |
| 117 | 117 | |
| 118 | 118 | /** |
@@ -120,27 +120,27 @@ discard block |
||
| 120 | 120 | * |
| 121 | 121 | * @param array $action_ids List of action ID. |
| 122 | 122 | */ |
| 123 | - public function bulk_log_cancel_actions( $action_ids ) { |
|
| 124 | - if ( empty( $action_ids ) ) { |
|
| 123 | + public function bulk_log_cancel_actions($action_ids) { |
|
| 124 | + if (empty($action_ids)) { |
|
| 125 | 125 | return; |
| 126 | 126 | } |
| 127 | 127 | |
| 128 | 128 | /** @var \wpdb $wpdb */ |
| 129 | 129 | global $wpdb; |
| 130 | 130 | $date = as_get_datetime_object(); |
| 131 | - $date_gmt = $date->format( 'Y-m-d H:i:s' ); |
|
| 132 | - ActionScheduler_TimezoneHelper::set_local_timezone( $date ); |
|
| 133 | - $date_local = $date->format( 'Y-m-d H:i:s' ); |
|
| 134 | - $message = __( 'action canceled', 'action-scheduler' ); |
|
| 135 | - $format = '(%d, ' . $wpdb->prepare( '%s, %s, %s', $message, $date_gmt, $date_local ) . ')'; |
|
| 131 | + $date_gmt = $date->format('Y-m-d H:i:s'); |
|
| 132 | + ActionScheduler_TimezoneHelper::set_local_timezone($date); |
|
| 133 | + $date_local = $date->format('Y-m-d H:i:s'); |
|
| 134 | + $message = __('action canceled', 'action-scheduler'); |
|
| 135 | + $format = '(%d, ' . $wpdb->prepare('%s, %s, %s', $message, $date_gmt, $date_local) . ')'; |
|
| 136 | 136 | $sql_query = "INSERT {$wpdb->actionscheduler_logs} (action_id, message, log_date_gmt, log_date_local) VALUES "; |
| 137 | 137 | $value_rows = []; |
| 138 | 138 | |
| 139 | - foreach ( $action_ids as $action_id ) { |
|
| 140 | - $value_rows[] = $wpdb->prepare( $format, $action_id ); |
|
| 139 | + foreach ($action_ids as $action_id) { |
|
| 140 | + $value_rows[] = $wpdb->prepare($format, $action_id); |
|
| 141 | 141 | } |
| 142 | - $sql_query .= implode( ',', $value_rows ); |
|
| 142 | + $sql_query .= implode(',', $value_rows); |
|
| 143 | 143 | |
| 144 | - $wpdb->query( $sql_query ); |
|
| 144 | + $wpdb->query($sql_query); |
|
| 145 | 145 | } |
| 146 | 146 | } |
@@ -6,12 +6,12 @@ discard block |
||
| 6 | 6 | */ |
| 7 | 7 | class ActionScheduler_wpPostStore_TaxonomyRegistrar { |
| 8 | 8 | public function register() { |
| 9 | - register_taxonomy( ActionScheduler_wpPostStore::GROUP_TAXONOMY, ActionScheduler_wpPostStore::POST_TYPE, $this->taxonomy_args() ); |
|
| 9 | + register_taxonomy(ActionScheduler_wpPostStore::GROUP_TAXONOMY, ActionScheduler_wpPostStore::POST_TYPE, $this->taxonomy_args()); |
|
| 10 | 10 | } |
| 11 | 11 | |
| 12 | 12 | protected function taxonomy_args() { |
| 13 | 13 | $args = array( |
| 14 | - 'label' => __( 'Action Group', 'action-scheduler' ), |
|
| 14 | + 'label' => __('Action Group', 'action-scheduler'), |
|
| 15 | 15 | 'public' => false, |
| 16 | 16 | 'hierarchical' => false, |
| 17 | 17 | 'show_admin_column' => true, |
@@ -19,7 +19,7 @@ discard block |
||
| 19 | 19 | 'rewrite' => false, |
| 20 | 20 | ); |
| 21 | 21 | |
| 22 | - $args = apply_filters( 'action_scheduler_taxonomy_args', $args ); |
|
| 22 | + $args = apply_filters('action_scheduler_taxonomy_args', $args); |
|
| 23 | 23 | return $args; |
| 24 | 24 | } |
| 25 | 25 | } |
@@ -6,7 +6,7 @@ discard block |
||
| 6 | 6 | */ |
| 7 | 7 | class ActionScheduler_wpPostStore_PostTypeRegistrar { |
| 8 | 8 | public function register() { |
| 9 | - register_post_type( ActionScheduler_wpPostStore::POST_TYPE, $this->post_type_args() ); |
|
| 9 | + register_post_type(ActionScheduler_wpPostStore::POST_TYPE, $this->post_type_args()); |
|
| 10 | 10 | } |
| 11 | 11 | |
| 12 | 12 | /** |
@@ -16,30 +16,30 @@ discard block |
||
| 16 | 16 | */ |
| 17 | 17 | protected function post_type_args() { |
| 18 | 18 | $args = array( |
| 19 | - 'label' => __( 'Scheduled Actions', 'action-scheduler' ), |
|
| 20 | - 'description' => __( 'Scheduled actions are hooks triggered on a cetain date and time.', 'action-scheduler' ), |
|
| 19 | + 'label' => __('Scheduled Actions', 'action-scheduler'), |
|
| 20 | + 'description' => __('Scheduled actions are hooks triggered on a cetain date and time.', 'action-scheduler'), |
|
| 21 | 21 | 'public' => false, |
| 22 | 22 | 'map_meta_cap' => true, |
| 23 | 23 | 'hierarchical' => false, |
| 24 | - 'supports' => array('title', 'editor','comments'), |
|
| 24 | + 'supports' => array('title', 'editor', 'comments'), |
|
| 25 | 25 | 'rewrite' => false, |
| 26 | 26 | 'query_var' => false, |
| 27 | 27 | 'can_export' => true, |
| 28 | 28 | 'ep_mask' => EP_NONE, |
| 29 | 29 | 'labels' => array( |
| 30 | - 'name' => __( 'Scheduled Actions', 'action-scheduler' ), |
|
| 31 | - 'singular_name' => __( 'Scheduled Action', 'action-scheduler' ), |
|
| 32 | - 'menu_name' => _x( 'Scheduled Actions', 'Admin menu name', 'action-scheduler' ), |
|
| 33 | - 'add_new' => __( 'Add', 'action-scheduler' ), |
|
| 34 | - 'add_new_item' => __( 'Add New Scheduled Action', 'action-scheduler' ), |
|
| 35 | - 'edit' => __( 'Edit', 'action-scheduler' ), |
|
| 36 | - 'edit_item' => __( 'Edit Scheduled Action', 'action-scheduler' ), |
|
| 37 | - 'new_item' => __( 'New Scheduled Action', 'action-scheduler' ), |
|
| 38 | - 'view' => __( 'View Action', 'action-scheduler' ), |
|
| 39 | - 'view_item' => __( 'View Action', 'action-scheduler' ), |
|
| 40 | - 'search_items' => __( 'Search Scheduled Actions', 'action-scheduler' ), |
|
| 41 | - 'not_found' => __( 'No actions found', 'action-scheduler' ), |
|
| 42 | - 'not_found_in_trash' => __( 'No actions found in trash', 'action-scheduler' ), |
|
| 30 | + 'name' => __('Scheduled Actions', 'action-scheduler'), |
|
| 31 | + 'singular_name' => __('Scheduled Action', 'action-scheduler'), |
|
| 32 | + 'menu_name' => _x('Scheduled Actions', 'Admin menu name', 'action-scheduler'), |
|
| 33 | + 'add_new' => __('Add', 'action-scheduler'), |
|
| 34 | + 'add_new_item' => __('Add New Scheduled Action', 'action-scheduler'), |
|
| 35 | + 'edit' => __('Edit', 'action-scheduler'), |
|
| 36 | + 'edit_item' => __('Edit Scheduled Action', 'action-scheduler'), |
|
| 37 | + 'new_item' => __('New Scheduled Action', 'action-scheduler'), |
|
| 38 | + 'view' => __('View Action', 'action-scheduler'), |
|
| 39 | + 'view_item' => __('View Action', 'action-scheduler'), |
|
| 40 | + 'search_items' => __('Search Scheduled Actions', 'action-scheduler'), |
|
| 41 | + 'not_found' => __('No actions found', 'action-scheduler'), |
|
| 42 | + 'not_found_in_trash' => __('No actions found in trash', 'action-scheduler'), |
|
| 43 | 43 | ), |
| 44 | 44 | ); |
| 45 | 45 | |
@@ -33,46 +33,46 @@ discard block |
||
| 33 | 33 | * |
| 34 | 34 | * @return int Action ID. |
| 35 | 35 | */ |
| 36 | - public function save_action( ActionScheduler_Action $action, \DateTime $date = null ) { |
|
| 36 | + public function save_action(ActionScheduler_Action $action, \DateTime $date = null) { |
|
| 37 | 37 | try { |
| 38 | 38 | |
| 39 | - $this->validate_action( $action ); |
|
| 39 | + $this->validate_action($action); |
|
| 40 | 40 | |
| 41 | 41 | /** @var \wpdb $wpdb */ |
| 42 | 42 | global $wpdb; |
| 43 | 43 | $data = [ |
| 44 | 44 | 'hook' => $action->get_hook(), |
| 45 | - 'status' => ( $action->is_finished() ? self::STATUS_COMPLETE : self::STATUS_PENDING ), |
|
| 46 | - 'scheduled_date_gmt' => $this->get_scheduled_date_string( $action, $date ), |
|
| 47 | - 'scheduled_date_local' => $this->get_scheduled_date_string_local( $action, $date ), |
|
| 48 | - 'schedule' => serialize( $action->get_schedule() ), |
|
| 49 | - 'group_id' => $this->get_group_id( $action->get_group() ), |
|
| 45 | + 'status' => ($action->is_finished() ? self::STATUS_COMPLETE : self::STATUS_PENDING), |
|
| 46 | + 'scheduled_date_gmt' => $this->get_scheduled_date_string($action, $date), |
|
| 47 | + 'scheduled_date_local' => $this->get_scheduled_date_string_local($action, $date), |
|
| 48 | + 'schedule' => serialize($action->get_schedule()), |
|
| 49 | + 'group_id' => $this->get_group_id($action->get_group()), |
|
| 50 | 50 | ]; |
| 51 | - $args = wp_json_encode( $action->get_args() ); |
|
| 52 | - if ( strlen( $args ) <= static::$max_index_length ) { |
|
| 51 | + $args = wp_json_encode($action->get_args()); |
|
| 52 | + if (strlen($args) <= static::$max_index_length) { |
|
| 53 | 53 | $data['args'] = $args; |
| 54 | 54 | } else { |
| 55 | - $data['args'] = $this->hash_args( $args ); |
|
| 55 | + $data['args'] = $this->hash_args($args); |
|
| 56 | 56 | $data['extended_args'] = $args; |
| 57 | 57 | } |
| 58 | 58 | |
| 59 | - $table_name = ! empty( $wpdb->actionscheduler_actions ) ? $wpdb->actionscheduler_actions : $wpdb->prefix . 'actionscheduler_actions'; |
|
| 60 | - $wpdb->insert( $table_name, $data ); |
|
| 59 | + $table_name = !empty($wpdb->actionscheduler_actions) ? $wpdb->actionscheduler_actions : $wpdb->prefix . 'actionscheduler_actions'; |
|
| 60 | + $wpdb->insert($table_name, $data); |
|
| 61 | 61 | $action_id = $wpdb->insert_id; |
| 62 | 62 | |
| 63 | - if ( is_wp_error( $action_id ) ) { |
|
| 64 | - throw new RuntimeException( $action_id->get_error_message() ); |
|
| 63 | + if (is_wp_error($action_id)) { |
|
| 64 | + throw new RuntimeException($action_id->get_error_message()); |
|
| 65 | 65 | } |
| 66 | - elseif ( empty( $action_id ) ) { |
|
| 67 | - throw new RuntimeException( $wpdb->last_error ? $wpdb->last_error : __( 'Database error.', 'action-scheduler' ) ); |
|
| 66 | + elseif (empty($action_id)) { |
|
| 67 | + throw new RuntimeException($wpdb->last_error ? $wpdb->last_error : __('Database error.', 'action-scheduler')); |
|
| 68 | 68 | } |
| 69 | 69 | |
| 70 | - do_action( 'action_scheduler_stored_action', $action_id ); |
|
| 70 | + do_action('action_scheduler_stored_action', $action_id); |
|
| 71 | 71 | |
| 72 | 72 | return $action_id; |
| 73 | - } catch ( \Exception $e ) { |
|
| 73 | + } catch (\Exception $e) { |
|
| 74 | 74 | /* translators: %s: error message */ |
| 75 | - throw new \RuntimeException( sprintf( __( 'Error saving action: %s', 'action-scheduler' ), $e->getMessage() ), 0 ); |
|
| 75 | + throw new \RuntimeException(sprintf(__('Error saving action: %s', 'action-scheduler'), $e->getMessage()), 0); |
|
| 76 | 76 | } |
| 77 | 77 | } |
| 78 | 78 | |
@@ -82,8 +82,8 @@ discard block |
||
| 82 | 82 | * @param string $args JSON encoded action args. |
| 83 | 83 | * @return string |
| 84 | 84 | */ |
| 85 | - protected function hash_args( $args ) { |
|
| 86 | - return md5( $args ); |
|
| 85 | + protected function hash_args($args) { |
|
| 86 | + return md5($args); |
|
| 87 | 87 | } |
| 88 | 88 | |
| 89 | 89 | /** |
@@ -92,12 +92,12 @@ discard block |
||
| 92 | 92 | * @param array $args Action args. |
| 93 | 93 | * @return string |
| 94 | 94 | */ |
| 95 | - protected function get_args_for_query( $args ) { |
|
| 96 | - $encoded = wp_json_encode( $args ); |
|
| 97 | - if ( strlen( $encoded ) <= static::$max_index_length ) { |
|
| 95 | + protected function get_args_for_query($args) { |
|
| 96 | + $encoded = wp_json_encode($args); |
|
| 97 | + if (strlen($encoded) <= static::$max_index_length) { |
|
| 98 | 98 | return $encoded; |
| 99 | 99 | } |
| 100 | - return $this->hash_args( $encoded ); |
|
| 100 | + return $this->hash_args($encoded); |
|
| 101 | 101 | } |
| 102 | 102 | /** |
| 103 | 103 | * Get a group's ID based on its name/slug. |
@@ -107,15 +107,15 @@ discard block |
||
| 107 | 107 | * |
| 108 | 108 | * @return int The group's ID, if it exists or is created, or 0 if it does not exist and is not created. |
| 109 | 109 | */ |
| 110 | - protected function get_group_id( $slug, $create_if_not_exists = true ) { |
|
| 111 | - if ( empty( $slug ) ) { |
|
| 110 | + protected function get_group_id($slug, $create_if_not_exists = true) { |
|
| 111 | + if (empty($slug)) { |
|
| 112 | 112 | return 0; |
| 113 | 113 | } |
| 114 | 114 | /** @var \wpdb $wpdb */ |
| 115 | 115 | global $wpdb; |
| 116 | - $group_id = (int) $wpdb->get_var( $wpdb->prepare( "SELECT group_id FROM {$wpdb->actionscheduler_groups} WHERE slug=%s", $slug ) ); |
|
| 117 | - if ( empty( $group_id ) && $create_if_not_exists ) { |
|
| 118 | - $group_id = $this->create_group( $slug ); |
|
| 116 | + $group_id = (int) $wpdb->get_var($wpdb->prepare("SELECT group_id FROM {$wpdb->actionscheduler_groups} WHERE slug=%s", $slug)); |
|
| 117 | + if (empty($group_id) && $create_if_not_exists) { |
|
| 118 | + $group_id = $this->create_group($slug); |
|
| 119 | 119 | } |
| 120 | 120 | |
| 121 | 121 | return $group_id; |
@@ -128,10 +128,10 @@ discard block |
||
| 128 | 128 | * |
| 129 | 129 | * @return int Group ID. |
| 130 | 130 | */ |
| 131 | - protected function create_group( $slug ) { |
|
| 131 | + protected function create_group($slug) { |
|
| 132 | 132 | /** @var \wpdb $wpdb */ |
| 133 | 133 | global $wpdb; |
| 134 | - $wpdb->insert( $wpdb->actionscheduler_groups, [ 'slug' => $slug ] ); |
|
| 134 | + $wpdb->insert($wpdb->actionscheduler_groups, ['slug' => $slug]); |
|
| 135 | 135 | |
| 136 | 136 | return (int) $wpdb->insert_id; |
| 137 | 137 | } |
@@ -143,27 +143,27 @@ discard block |
||
| 143 | 143 | * |
| 144 | 144 | * @return ActionScheduler_Action |
| 145 | 145 | */ |
| 146 | - public function fetch_action( $action_id ) { |
|
| 146 | + public function fetch_action($action_id) { |
|
| 147 | 147 | /** @var \wpdb $wpdb */ |
| 148 | 148 | global $wpdb; |
| 149 | - $data = $wpdb->get_row( $wpdb->prepare( |
|
| 149 | + $data = $wpdb->get_row($wpdb->prepare( |
|
| 150 | 150 | "SELECT a.*, g.slug AS `group` FROM {$wpdb->actionscheduler_actions} a LEFT JOIN {$wpdb->actionscheduler_groups} g ON a.group_id=g.group_id WHERE a.action_id=%d", |
| 151 | 151 | $action_id |
| 152 | - ) ); |
|
| 152 | + )); |
|
| 153 | 153 | |
| 154 | - if ( empty( $data ) ) { |
|
| 154 | + if (empty($data)) { |
|
| 155 | 155 | return $this->get_null_action(); |
| 156 | 156 | } |
| 157 | 157 | |
| 158 | - if ( ! empty( $data->extended_args ) ) { |
|
| 158 | + if (!empty($data->extended_args)) { |
|
| 159 | 159 | $data->args = $data->extended_args; |
| 160 | - unset( $data->extended_args ); |
|
| 160 | + unset($data->extended_args); |
|
| 161 | 161 | } |
| 162 | 162 | |
| 163 | 163 | try { |
| 164 | - $action = $this->make_action_from_db_record( $data ); |
|
| 165 | - } catch ( ActionScheduler_InvalidActionException $exception ) { |
|
| 166 | - do_action( 'action_scheduler_failed_fetch_action', $action_id, $exception ); |
|
| 164 | + $action = $this->make_action_from_db_record($data); |
|
| 165 | + } catch (ActionScheduler_InvalidActionException $exception) { |
|
| 166 | + do_action('action_scheduler_failed_fetch_action', $action_id, $exception); |
|
| 167 | 167 | return $this->get_null_action(); |
| 168 | 168 | } |
| 169 | 169 | |
@@ -186,21 +186,21 @@ discard block |
||
| 186 | 186 | * |
| 187 | 187 | * @return ActionScheduler_Action|ActionScheduler_CanceledAction|ActionScheduler_FinishedAction |
| 188 | 188 | */ |
| 189 | - protected function make_action_from_db_record( $data ) { |
|
| 189 | + protected function make_action_from_db_record($data) { |
|
| 190 | 190 | |
| 191 | 191 | $hook = $data->hook; |
| 192 | - $args = json_decode( $data->args, true ); |
|
| 193 | - $schedule = unserialize( $data->schedule ); |
|
| 192 | + $args = json_decode($data->args, true); |
|
| 193 | + $schedule = unserialize($data->schedule); |
|
| 194 | 194 | |
| 195 | - $this->validate_args( $args, $data->action_id ); |
|
| 196 | - $this->validate_schedule( $schedule, $data->action_id ); |
|
| 195 | + $this->validate_args($args, $data->action_id); |
|
| 196 | + $this->validate_schedule($schedule, $data->action_id); |
|
| 197 | 197 | |
| 198 | - if ( empty( $schedule ) ) { |
|
| 198 | + if (empty($schedule)) { |
|
| 199 | 199 | $schedule = new ActionScheduler_NullSchedule(); |
| 200 | 200 | } |
| 201 | 201 | $group = $data->group ? $data->group : ''; |
| 202 | 202 | |
| 203 | - return ActionScheduler::factory()->get_stored_action( $data->status, $data->hook, $args, $schedule, $group ); |
|
| 203 | + return ActionScheduler::factory()->get_stored_action($data->status, $data->hook, $args, $schedule, $group); |
|
| 204 | 204 | } |
| 205 | 205 | |
| 206 | 206 | /** |
@@ -211,34 +211,34 @@ discard block |
||
| 211 | 211 | * |
| 212 | 212 | * @return string|null ID of the next action matching the criteria or NULL if not found. |
| 213 | 213 | */ |
| 214 | - public function find_action( $hook, $params = [] ) { |
|
| 215 | - $params = wp_parse_args( $params, [ |
|
| 214 | + public function find_action($hook, $params = []) { |
|
| 215 | + $params = wp_parse_args($params, [ |
|
| 216 | 216 | 'args' => null, |
| 217 | 217 | 'status' => self::STATUS_PENDING, |
| 218 | 218 | 'group' => '', |
| 219 | - ] ); |
|
| 219 | + ]); |
|
| 220 | 220 | |
| 221 | 221 | /** @var wpdb $wpdb */ |
| 222 | 222 | global $wpdb; |
| 223 | 223 | $query = "SELECT a.action_id FROM {$wpdb->actionscheduler_actions} a"; |
| 224 | 224 | $args = []; |
| 225 | - if ( ! empty( $params[ 'group' ] ) ) { |
|
| 226 | - $query .= " INNER JOIN {$wpdb->actionscheduler_groups} g ON g.group_id=a.group_id AND g.slug=%s"; |
|
| 227 | - $args[] = $params[ 'group' ]; |
|
| 225 | + if (!empty($params['group'])) { |
|
| 226 | + $query .= " INNER JOIN {$wpdb->actionscheduler_groups} g ON g.group_id=a.group_id AND g.slug=%s"; |
|
| 227 | + $args[] = $params['group']; |
|
| 228 | 228 | } |
| 229 | 229 | $query .= " WHERE a.hook=%s"; |
| 230 | 230 | $args[] = $hook; |
| 231 | - if ( ! is_null( $params[ 'args' ] ) ) { |
|
| 232 | - $query .= " AND a.args=%s"; |
|
| 233 | - $args[] = $this->get_args_for_query( $params[ 'args' ] ); |
|
| 231 | + if (!is_null($params['args'])) { |
|
| 232 | + $query .= " AND a.args=%s"; |
|
| 233 | + $args[] = $this->get_args_for_query($params['args']); |
|
| 234 | 234 | } |
| 235 | 235 | |
| 236 | 236 | $order = 'ASC'; |
| 237 | - if ( ! empty( $params[ 'status' ] ) ) { |
|
| 238 | - $query .= " AND a.status=%s"; |
|
| 239 | - $args[] = $params[ 'status' ]; |
|
| 237 | + if (!empty($params['status'])) { |
|
| 238 | + $query .= " AND a.status=%s"; |
|
| 239 | + $args[] = $params['status']; |
|
| 240 | 240 | |
| 241 | - if ( self::STATUS_PENDING == $params[ 'status' ] ) { |
|
| 241 | + if (self::STATUS_PENDING == $params['status']) { |
|
| 242 | 242 | $order = 'ASC'; // Find the next action that matches. |
| 243 | 243 | } else { |
| 244 | 244 | $order = 'DESC'; // Find the most recent action that matches. |
@@ -247,9 +247,9 @@ discard block |
||
| 247 | 247 | |
| 248 | 248 | $query .= " ORDER BY scheduled_date_gmt $order LIMIT 1"; |
| 249 | 249 | |
| 250 | - $query = $wpdb->prepare( $query, $args ); |
|
| 250 | + $query = $wpdb->prepare($query, $args); |
|
| 251 | 251 | |
| 252 | - $id = $wpdb->get_var( $query ); |
|
| 252 | + $id = $wpdb->get_var($query); |
|
| 253 | 253 | |
| 254 | 254 | return $id; |
| 255 | 255 | } |
@@ -262,13 +262,13 @@ discard block |
||
| 262 | 262 | * |
| 263 | 263 | * @return string SQL statement already properly escaped. |
| 264 | 264 | */ |
| 265 | - protected function get_query_actions_sql( array $query, $select_or_count = 'select' ) { |
|
| 265 | + protected function get_query_actions_sql(array $query, $select_or_count = 'select') { |
|
| 266 | 266 | |
| 267 | - if ( ! in_array( $select_or_count, array( 'select', 'count' ) ) ) { |
|
| 268 | - throw new InvalidArgumentException( __( 'Invalid value for select or count parameter. Cannot query actions.', 'action-scheduler' ) ); |
|
| 267 | + if (!in_array($select_or_count, array('select', 'count'))) { |
|
| 268 | + throw new InvalidArgumentException(__('Invalid value for select or count parameter. Cannot query actions.', 'action-scheduler')); |
|
| 269 | 269 | } |
| 270 | 270 | |
| 271 | - $query = wp_parse_args( $query, [ |
|
| 271 | + $query = wp_parse_args($query, [ |
|
| 272 | 272 | 'hook' => '', |
| 273 | 273 | 'args' => null, |
| 274 | 274 | 'date' => null, |
@@ -282,74 +282,74 @@ discard block |
||
| 282 | 282 | 'offset' => 0, |
| 283 | 283 | 'orderby' => 'date', |
| 284 | 284 | 'order' => 'ASC', |
| 285 | - ] ); |
|
| 285 | + ]); |
|
| 286 | 286 | |
| 287 | 287 | /** @var \wpdb $wpdb */ |
| 288 | 288 | global $wpdb; |
| 289 | - $sql = ( 'count' === $select_or_count ) ? 'SELECT count(a.action_id)' : 'SELECT a.action_id'; |
|
| 289 | + $sql = ('count' === $select_or_count) ? 'SELECT count(a.action_id)' : 'SELECT a.action_id'; |
|
| 290 | 290 | $sql .= " FROM {$wpdb->actionscheduler_actions} a"; |
| 291 | 291 | $sql_params = []; |
| 292 | 292 | |
| 293 | - if ( ! empty( $query[ 'group' ] ) || 'group' === $query[ 'orderby' ] ) { |
|
| 293 | + if (!empty($query['group']) || 'group' === $query['orderby']) { |
|
| 294 | 294 | $sql .= " LEFT JOIN {$wpdb->actionscheduler_groups} g ON g.group_id=a.group_id"; |
| 295 | 295 | } |
| 296 | 296 | |
| 297 | 297 | $sql .= " WHERE 1=1"; |
| 298 | 298 | |
| 299 | - if ( ! empty( $query[ 'group' ] ) ) { |
|
| 300 | - $sql .= " AND g.slug=%s"; |
|
| 301 | - $sql_params[] = $query[ 'group' ]; |
|
| 299 | + if (!empty($query['group'])) { |
|
| 300 | + $sql .= " AND g.slug=%s"; |
|
| 301 | + $sql_params[] = $query['group']; |
|
| 302 | 302 | } |
| 303 | 303 | |
| 304 | - if ( $query[ 'hook' ] ) { |
|
| 305 | - $sql .= " AND a.hook=%s"; |
|
| 306 | - $sql_params[] = $query[ 'hook' ]; |
|
| 304 | + if ($query['hook']) { |
|
| 305 | + $sql .= " AND a.hook=%s"; |
|
| 306 | + $sql_params[] = $query['hook']; |
|
| 307 | 307 | } |
| 308 | - if ( ! is_null( $query[ 'args' ] ) ) { |
|
| 309 | - $sql .= " AND a.args=%s"; |
|
| 310 | - $sql_params[] = $this->get_args_for_query( $query[ 'args' ] ); |
|
| 308 | + if (!is_null($query['args'])) { |
|
| 309 | + $sql .= " AND a.args=%s"; |
|
| 310 | + $sql_params[] = $this->get_args_for_query($query['args']); |
|
| 311 | 311 | } |
| 312 | 312 | |
| 313 | - if ( $query[ 'status' ] ) { |
|
| 314 | - $sql .= " AND a.status=%s"; |
|
| 315 | - $sql_params[] = $query[ 'status' ]; |
|
| 313 | + if ($query['status']) { |
|
| 314 | + $sql .= " AND a.status=%s"; |
|
| 315 | + $sql_params[] = $query['status']; |
|
| 316 | 316 | } |
| 317 | 317 | |
| 318 | - if ( $query[ 'date' ] instanceof \DateTime ) { |
|
| 319 | - $date = clone $query[ 'date' ]; |
|
| 320 | - $date->setTimezone( new \DateTimeZone( 'UTC' ) ); |
|
| 321 | - $date_string = $date->format( 'Y-m-d H:i:s' ); |
|
| 322 | - $comparator = $this->validate_sql_comparator( $query[ 'date_compare' ] ); |
|
| 323 | - $sql .= " AND a.scheduled_date_gmt $comparator %s"; |
|
| 318 | + if ($query['date'] instanceof \DateTime) { |
|
| 319 | + $date = clone $query['date']; |
|
| 320 | + $date->setTimezone(new \DateTimeZone('UTC')); |
|
| 321 | + $date_string = $date->format('Y-m-d H:i:s'); |
|
| 322 | + $comparator = $this->validate_sql_comparator($query['date_compare']); |
|
| 323 | + $sql .= " AND a.scheduled_date_gmt $comparator %s"; |
|
| 324 | 324 | $sql_params[] = $date_string; |
| 325 | 325 | } |
| 326 | 326 | |
| 327 | - if ( $query[ 'modified' ] instanceof \DateTime ) { |
|
| 328 | - $modified = clone $query[ 'modified' ]; |
|
| 329 | - $modified->setTimezone( new \DateTimeZone( 'UTC' ) ); |
|
| 330 | - $date_string = $modified->format( 'Y-m-d H:i:s' ); |
|
| 331 | - $comparator = $this->validate_sql_comparator( $query[ 'modified_compare' ] ); |
|
| 332 | - $sql .= " AND a.last_attempt_gmt $comparator %s"; |
|
| 327 | + if ($query['modified'] instanceof \DateTime) { |
|
| 328 | + $modified = clone $query['modified']; |
|
| 329 | + $modified->setTimezone(new \DateTimeZone('UTC')); |
|
| 330 | + $date_string = $modified->format('Y-m-d H:i:s'); |
|
| 331 | + $comparator = $this->validate_sql_comparator($query['modified_compare']); |
|
| 332 | + $sql .= " AND a.last_attempt_gmt $comparator %s"; |
|
| 333 | 333 | $sql_params[] = $date_string; |
| 334 | 334 | } |
| 335 | 335 | |
| 336 | - if ( $query[ 'claimed' ] === true ) { |
|
| 336 | + if ($query['claimed'] === true) { |
|
| 337 | 337 | $sql .= " AND a.claim_id != 0"; |
| 338 | - } elseif ( $query[ 'claimed' ] === false ) { |
|
| 338 | + } elseif ($query['claimed'] === false) { |
|
| 339 | 339 | $sql .= " AND a.claim_id = 0"; |
| 340 | - } elseif ( ! is_null( $query[ 'claimed' ] ) ) { |
|
| 341 | - $sql .= " AND a.claim_id = %d"; |
|
| 342 | - $sql_params[] = $query[ 'claimed' ]; |
|
| 340 | + } elseif (!is_null($query['claimed'])) { |
|
| 341 | + $sql .= " AND a.claim_id = %d"; |
|
| 342 | + $sql_params[] = $query['claimed']; |
|
| 343 | 343 | } |
| 344 | 344 | |
| 345 | - if ( ! empty( $query['search'] ) ) { |
|
| 345 | + if (!empty($query['search'])) { |
|
| 346 | 346 | $sql .= " AND (a.hook LIKE %s OR (a.extended_args IS NULL AND a.args LIKE %s) OR a.extended_args LIKE %s"; |
| 347 | - for( $i = 0; $i < 3; $i++ ) { |
|
| 348 | - $sql_params[] = sprintf( '%%%s%%', $query['search'] ); |
|
| 347 | + for ($i = 0; $i < 3; $i++) { |
|
| 348 | + $sql_params[] = sprintf('%%%s%%', $query['search']); |
|
| 349 | 349 | } |
| 350 | 350 | |
| 351 | 351 | $search_claim_id = (int) $query['search']; |
| 352 | - if ( $search_claim_id ) { |
|
| 352 | + if ($search_claim_id) { |
|
| 353 | 353 | $sql .= ' OR a.claim_id = %d'; |
| 354 | 354 | $sql_params[] = $search_claim_id; |
| 355 | 355 | } |
@@ -357,8 +357,8 @@ discard block |
||
| 357 | 357 | $sql .= ')'; |
| 358 | 358 | } |
| 359 | 359 | |
| 360 | - if ( 'select' === $select_or_count ) { |
|
| 361 | - switch ( $query['orderby'] ) { |
|
| 360 | + if ('select' === $select_or_count) { |
|
| 361 | + switch ($query['orderby']) { |
|
| 362 | 362 | case 'hook': |
| 363 | 363 | $orderby = 'a.hook'; |
| 364 | 364 | break; |
@@ -373,21 +373,21 @@ discard block |
||
| 373 | 373 | $orderby = 'a.scheduled_date_gmt'; |
| 374 | 374 | break; |
| 375 | 375 | } |
| 376 | - if ( strtoupper( $query[ 'order' ] ) == 'ASC' ) { |
|
| 376 | + if (strtoupper($query['order']) == 'ASC') { |
|
| 377 | 377 | $order = 'ASC'; |
| 378 | 378 | } else { |
| 379 | 379 | $order = 'DESC'; |
| 380 | 380 | } |
| 381 | 381 | $sql .= " ORDER BY $orderby $order"; |
| 382 | - if ( $query[ 'per_page' ] > 0 ) { |
|
| 383 | - $sql .= " LIMIT %d, %d"; |
|
| 384 | - $sql_params[] = $query[ 'offset' ]; |
|
| 385 | - $sql_params[] = $query[ 'per_page' ]; |
|
| 382 | + if ($query['per_page'] > 0) { |
|
| 383 | + $sql .= " LIMIT %d, %d"; |
|
| 384 | + $sql_params[] = $query['offset']; |
|
| 385 | + $sql_params[] = $query['per_page']; |
|
| 386 | 386 | } |
| 387 | 387 | } |
| 388 | 388 | |
| 389 | - if ( ! empty( $sql_params ) ) { |
|
| 390 | - $sql = $wpdb->prepare( $sql, $sql_params ); |
|
| 389 | + if (!empty($sql_params)) { |
|
| 390 | + $sql = $wpdb->prepare($sql, $sql_params); |
|
| 391 | 391 | } |
| 392 | 392 | |
| 393 | 393 | return $sql; |
@@ -401,13 +401,13 @@ discard block |
||
| 401 | 401 | * |
| 402 | 402 | * @return null|string|array The IDs of actions matching the query |
| 403 | 403 | */ |
| 404 | - public function query_actions( $query = [], $query_type = 'select' ) { |
|
| 404 | + public function query_actions($query = [], $query_type = 'select') { |
|
| 405 | 405 | /** @var wpdb $wpdb */ |
| 406 | 406 | global $wpdb; |
| 407 | 407 | |
| 408 | - $sql = $this->get_query_actions_sql( $query, $query_type ); |
|
| 408 | + $sql = $this->get_query_actions_sql($query, $query_type); |
|
| 409 | 409 | |
| 410 | - return ( 'count' === $query_type ) ? $wpdb->get_var( $sql ) : $wpdb->get_col( $sql ); |
|
| 410 | + return ('count' === $query_type) ? $wpdb->get_var($sql) : $wpdb->get_col($sql); |
|
| 411 | 411 | } |
| 412 | 412 | |
| 413 | 413 | /** |
@@ -425,10 +425,10 @@ discard block |
||
| 425 | 425 | $actions_count_by_status = array(); |
| 426 | 426 | $action_stati_and_labels = $this->get_status_labels(); |
| 427 | 427 | |
| 428 | - foreach ( $wpdb->get_results( $sql ) as $action_data ) { |
|
| 428 | + foreach ($wpdb->get_results($sql) as $action_data) { |
|
| 429 | 429 | // Ignore any actions with invalid status |
| 430 | - if ( array_key_exists( $action_data->status, $action_stati_and_labels ) ) { |
|
| 431 | - $actions_count_by_status[ $action_data->status ] = $action_data->count; |
|
| 430 | + if (array_key_exists($action_data->status, $action_stati_and_labels)) { |
|
| 431 | + $actions_count_by_status[$action_data->status] = $action_data->count; |
|
| 432 | 432 | } |
| 433 | 433 | } |
| 434 | 434 | |
@@ -442,22 +442,22 @@ discard block |
||
| 442 | 442 | * |
| 443 | 443 | * @return void |
| 444 | 444 | */ |
| 445 | - public function cancel_action( $action_id ) { |
|
| 445 | + public function cancel_action($action_id) { |
|
| 446 | 446 | /** @var \wpdb $wpdb */ |
| 447 | 447 | global $wpdb; |
| 448 | 448 | |
| 449 | 449 | $updated = $wpdb->update( |
| 450 | 450 | $wpdb->actionscheduler_actions, |
| 451 | - [ 'status' => self::STATUS_CANCELED ], |
|
| 452 | - [ 'action_id' => $action_id ], |
|
| 453 | - [ '%s' ], |
|
| 454 | - [ '%d' ] |
|
| 451 | + ['status' => self::STATUS_CANCELED], |
|
| 452 | + ['action_id' => $action_id], |
|
| 453 | + ['%s'], |
|
| 454 | + ['%d'] |
|
| 455 | 455 | ); |
| 456 | - if ( empty( $updated ) ) { |
|
| 456 | + if (empty($updated)) { |
|
| 457 | 457 | /* translators: %s: action ID */ |
| 458 | - throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) ); |
|
| 458 | + throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'action-scheduler'), $action_id)); |
|
| 459 | 459 | } |
| 460 | - do_action( 'action_scheduler_canceled_action', $action_id ); |
|
| 460 | + do_action('action_scheduler_canceled_action', $action_id); |
|
| 461 | 461 | } |
| 462 | 462 | |
| 463 | 463 | /** |
@@ -469,8 +469,8 @@ discard block |
||
| 469 | 469 | * |
| 470 | 470 | * @return void |
| 471 | 471 | */ |
| 472 | - public function cancel_actions_by_hook( $hook ) { |
|
| 473 | - $this->bulk_cancel_actions( [ 'hook' => $hook ] ); |
|
| 472 | + public function cancel_actions_by_hook($hook) { |
|
| 473 | + $this->bulk_cancel_actions(['hook' => $hook]); |
|
| 474 | 474 | } |
| 475 | 475 | |
| 476 | 476 | /** |
@@ -480,8 +480,8 @@ discard block |
||
| 480 | 480 | * |
| 481 | 481 | * @return void |
| 482 | 482 | */ |
| 483 | - public function cancel_actions_by_group( $group ) { |
|
| 484 | - $this->bulk_cancel_actions( [ 'group' => $group ] ); |
|
| 483 | + public function cancel_actions_by_group($group) { |
|
| 484 | + $this->bulk_cancel_actions(['group' => $group]); |
|
| 485 | 485 | } |
| 486 | 486 | |
| 487 | 487 | /** |
@@ -491,16 +491,16 @@ discard block |
||
| 491 | 491 | * |
| 492 | 492 | * @param array $query_args Query parameters. |
| 493 | 493 | */ |
| 494 | - protected function bulk_cancel_actions( $query_args ) { |
|
| 494 | + protected function bulk_cancel_actions($query_args) { |
|
| 495 | 495 | /** @var \wpdb $wpdb */ |
| 496 | 496 | global $wpdb; |
| 497 | 497 | |
| 498 | - if ( ! is_array( $query_args ) ) { |
|
| 498 | + if (!is_array($query_args)) { |
|
| 499 | 499 | return; |
| 500 | 500 | } |
| 501 | 501 | |
| 502 | 502 | // Don't cancel actions that are already canceled. |
| 503 | - if ( isset( $query_args['status'] ) && $query_args['status'] == self::STATUS_CANCELED ) { |
|
| 503 | + if (isset($query_args['status']) && $query_args['status'] == self::STATUS_CANCELED) { |
|
| 504 | 504 | return; |
| 505 | 505 | } |
| 506 | 506 | |
@@ -513,16 +513,16 @@ discard block |
||
| 513 | 513 | ] |
| 514 | 514 | ); |
| 515 | 515 | |
| 516 | - while ( $action_ids ) { |
|
| 517 | - $action_ids = $this->query_actions( $query_args ); |
|
| 518 | - if ( empty( $action_ids ) ) { |
|
| 516 | + while ($action_ids) { |
|
| 517 | + $action_ids = $this->query_actions($query_args); |
|
| 518 | + if (empty($action_ids)) { |
|
| 519 | 519 | break; |
| 520 | 520 | } |
| 521 | 521 | |
| 522 | - $format = array_fill( 0, count( $action_ids ), '%d' ); |
|
| 523 | - $query_in = '(' . implode( ',', $format ) . ')'; |
|
| 522 | + $format = array_fill(0, count($action_ids), '%d'); |
|
| 523 | + $query_in = '(' . implode(',', $format) . ')'; |
|
| 524 | 524 | $parameters = $action_ids; |
| 525 | - array_unshift( $parameters, self::STATUS_CANCELED ); |
|
| 525 | + array_unshift($parameters, self::STATUS_CANCELED); |
|
| 526 | 526 | |
| 527 | 527 | $wpdb->query( |
| 528 | 528 | $wpdb->prepare( // wpcs: PreparedSQLPlaceholders replacement count ok. |
@@ -531,7 +531,7 @@ discard block |
||
| 531 | 531 | ) |
| 532 | 532 | ); |
| 533 | 533 | |
| 534 | - do_action( 'action_scheduler_bulk_cancel_actions', $action_ids ); |
|
| 534 | + do_action('action_scheduler_bulk_cancel_actions', $action_ids); |
|
| 535 | 535 | } |
| 536 | 536 | } |
| 537 | 537 | |
@@ -540,14 +540,14 @@ discard block |
||
| 540 | 540 | * |
| 541 | 541 | * @param int $action_id Action ID. |
| 542 | 542 | */ |
| 543 | - public function delete_action( $action_id ) { |
|
| 543 | + public function delete_action($action_id) { |
|
| 544 | 544 | /** @var \wpdb $wpdb */ |
| 545 | 545 | global $wpdb; |
| 546 | - $deleted = $wpdb->delete( $wpdb->actionscheduler_actions, [ 'action_id' => $action_id ], [ '%d' ] ); |
|
| 547 | - if ( empty( $deleted ) ) { |
|
| 548 | - throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) ); |
|
| 546 | + $deleted = $wpdb->delete($wpdb->actionscheduler_actions, ['action_id' => $action_id], ['%d']); |
|
| 547 | + if (empty($deleted)) { |
|
| 548 | + throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'action-scheduler'), $action_id)); |
|
| 549 | 549 | } |
| 550 | - do_action( 'action_scheduler_deleted_action', $action_id ); |
|
| 550 | + do_action('action_scheduler_deleted_action', $action_id); |
|
| 551 | 551 | } |
| 552 | 552 | |
| 553 | 553 | /** |
@@ -558,9 +558,9 @@ discard block |
||
| 558 | 558 | * @throws \InvalidArgumentException |
| 559 | 559 | * @return \DateTime The local date the action is scheduled to run, or the date that it ran. |
| 560 | 560 | */ |
| 561 | - public function get_date( $action_id ) { |
|
| 562 | - $date = $this->get_date_gmt( $action_id ); |
|
| 563 | - ActionScheduler_TimezoneHelper::set_local_timezone( $date ); |
|
| 561 | + public function get_date($action_id) { |
|
| 562 | + $date = $this->get_date_gmt($action_id); |
|
| 563 | + ActionScheduler_TimezoneHelper::set_local_timezone($date); |
|
| 564 | 564 | return $date; |
| 565 | 565 | } |
| 566 | 566 | |
@@ -572,17 +572,17 @@ discard block |
||
| 572 | 572 | * @throws \InvalidArgumentException |
| 573 | 573 | * @return \DateTime The GMT date the action is scheduled to run, or the date that it ran. |
| 574 | 574 | */ |
| 575 | - protected function get_date_gmt( $action_id ) { |
|
| 575 | + protected function get_date_gmt($action_id) { |
|
| 576 | 576 | /** @var \wpdb $wpdb */ |
| 577 | 577 | global $wpdb; |
| 578 | - $record = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d", $action_id ) ); |
|
| 579 | - if ( empty( $record ) ) { |
|
| 580 | - throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) ); |
|
| 578 | + $record = $wpdb->get_row($wpdb->prepare("SELECT * FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d", $action_id)); |
|
| 579 | + if (empty($record)) { |
|
| 580 | + throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'action-scheduler'), $action_id)); |
|
| 581 | 581 | } |
| 582 | - if ( $record->status == self::STATUS_PENDING ) { |
|
| 583 | - return as_get_datetime_object( $record->scheduled_date_gmt ); |
|
| 582 | + if ($record->status == self::STATUS_PENDING) { |
|
| 583 | + return as_get_datetime_object($record->scheduled_date_gmt); |
|
| 584 | 584 | } else { |
| 585 | - return as_get_datetime_object( $record->last_attempt_gmt ); |
|
| 585 | + return as_get_datetime_object($record->last_attempt_gmt); |
|
| 586 | 586 | } |
| 587 | 587 | } |
| 588 | 588 | |
@@ -594,12 +594,12 @@ discard block |
||
| 594 | 594 | * |
| 595 | 595 | * @return ActionScheduler_ActionClaim |
| 596 | 596 | */ |
| 597 | - public function stake_claim( $max_actions = 10, \DateTime $before_date = null, $hooks = array(), $group = '' ) { |
|
| 597 | + public function stake_claim($max_actions = 10, \DateTime $before_date = null, $hooks = array(), $group = '') { |
|
| 598 | 598 | $claim_id = $this->generate_claim_id(); |
| 599 | - $this->claim_actions( $claim_id, $max_actions, $before_date, $hooks, $group ); |
|
| 600 | - $action_ids = $this->find_actions_by_claim_id( $claim_id ); |
|
| 599 | + $this->claim_actions($claim_id, $max_actions, $before_date, $hooks, $group); |
|
| 600 | + $action_ids = $this->find_actions_by_claim_id($claim_id); |
|
| 601 | 601 | |
| 602 | - return new ActionScheduler_ActionClaim( $claim_id, $action_ids ); |
|
| 602 | + return new ActionScheduler_ActionClaim($claim_id, $action_ids); |
|
| 603 | 603 | } |
| 604 | 604 | |
| 605 | 605 | /** |
@@ -611,7 +611,7 @@ discard block |
||
| 611 | 611 | /** @var \wpdb $wpdb */ |
| 612 | 612 | global $wpdb; |
| 613 | 613 | $now = as_get_datetime_object(); |
| 614 | - $wpdb->insert( $wpdb->actionscheduler_claims, [ 'date_created_gmt' => $now->format( 'Y-m-d H:i:s' ) ] ); |
|
| 614 | + $wpdb->insert($wpdb->actionscheduler_claims, ['date_created_gmt' => $now->format('Y-m-d H:i:s')]); |
|
| 615 | 615 | |
| 616 | 616 | return $wpdb->insert_id; |
| 617 | 617 | } |
@@ -626,39 +626,39 @@ discard block |
||
| 626 | 626 | * @return int The number of actions that were claimed. |
| 627 | 627 | * @throws \RuntimeException |
| 628 | 628 | */ |
| 629 | - protected function claim_actions( $claim_id, $limit, \DateTime $before_date = null, $hooks = array(), $group = '' ) { |
|
| 629 | + protected function claim_actions($claim_id, $limit, \DateTime $before_date = null, $hooks = array(), $group = '') { |
|
| 630 | 630 | /** @var \wpdb $wpdb */ |
| 631 | 631 | global $wpdb; |
| 632 | 632 | |
| 633 | 633 | $now = as_get_datetime_object(); |
| 634 | - $date = is_null( $before_date ) ? $now : clone $before_date; |
|
| 634 | + $date = is_null($before_date) ? $now : clone $before_date; |
|
| 635 | 635 | |
| 636 | 636 | // can't use $wpdb->update() because of the <= condition |
| 637 | 637 | $update = "UPDATE {$wpdb->actionscheduler_actions} SET claim_id=%d, last_attempt_gmt=%s, last_attempt_local=%s"; |
| 638 | 638 | $params = array( |
| 639 | 639 | $claim_id, |
| 640 | - $now->format( 'Y-m-d H:i:s' ), |
|
| 641 | - current_time( 'mysql' ), |
|
| 640 | + $now->format('Y-m-d H:i:s'), |
|
| 641 | + current_time('mysql'), |
|
| 642 | 642 | ); |
| 643 | 643 | |
| 644 | 644 | $where = "WHERE claim_id = 0 AND scheduled_date_gmt <= %s AND status=%s"; |
| 645 | - $params[] = $date->format( 'Y-m-d H:i:s' ); |
|
| 645 | + $params[] = $date->format('Y-m-d H:i:s'); |
|
| 646 | 646 | $params[] = self::STATUS_PENDING; |
| 647 | 647 | |
| 648 | - if ( ! empty( $hooks ) ) { |
|
| 649 | - $placeholders = array_fill( 0, count( $hooks ), '%s' ); |
|
| 650 | - $where .= ' AND hook IN (' . join( ', ', $placeholders ) . ')'; |
|
| 651 | - $params = array_merge( $params, array_values( $hooks ) ); |
|
| 648 | + if (!empty($hooks)) { |
|
| 649 | + $placeholders = array_fill(0, count($hooks), '%s'); |
|
| 650 | + $where .= ' AND hook IN (' . join(', ', $placeholders) . ')'; |
|
| 651 | + $params = array_merge($params, array_values($hooks)); |
|
| 652 | 652 | } |
| 653 | 653 | |
| 654 | - if ( ! empty( $group ) ) { |
|
| 654 | + if (!empty($group)) { |
|
| 655 | 655 | |
| 656 | - $group_id = $this->get_group_id( $group, false ); |
|
| 656 | + $group_id = $this->get_group_id($group, false); |
|
| 657 | 657 | |
| 658 | 658 | // throw exception if no matching group found, this matches ActionScheduler_wpPostStore's behaviour |
| 659 | - if ( empty( $group_id ) ) { |
|
| 659 | + if (empty($group_id)) { |
|
| 660 | 660 | /* translators: %s: group name */ |
| 661 | - throw new InvalidArgumentException( sprintf( __( 'The group "%s" does not exist.', 'action-scheduler' ), $group ) ); |
|
| 661 | + throw new InvalidArgumentException(sprintf(__('The group "%s" does not exist.', 'action-scheduler'), $group)); |
|
| 662 | 662 | } |
| 663 | 663 | |
| 664 | 664 | $where .= ' AND group_id = %d'; |
@@ -668,11 +668,11 @@ discard block |
||
| 668 | 668 | $order = "ORDER BY attempts ASC, scheduled_date_gmt ASC, action_id ASC LIMIT %d"; |
| 669 | 669 | $params[] = $limit; |
| 670 | 670 | |
| 671 | - $sql = $wpdb->prepare( "{$update} {$where} {$order}", $params ); |
|
| 671 | + $sql = $wpdb->prepare("{$update} {$where} {$order}", $params); |
|
| 672 | 672 | |
| 673 | - $rows_affected = $wpdb->query( $sql ); |
|
| 674 | - if ( $rows_affected === false ) { |
|
| 675 | - throw new \RuntimeException( __( 'Unable to claim actions. Database error.', 'action-scheduler' ) ); |
|
| 673 | + $rows_affected = $wpdb->query($sql); |
|
| 674 | + if ($rows_affected === false) { |
|
| 675 | + throw new \RuntimeException(__('Unable to claim actions. Database error.', 'action-scheduler')); |
|
| 676 | 676 | } |
| 677 | 677 | |
| 678 | 678 | return (int) $rows_affected; |
@@ -687,9 +687,9 @@ discard block |
||
| 687 | 687 | global $wpdb; |
| 688 | 688 | |
| 689 | 689 | $sql = "SELECT COUNT(DISTINCT claim_id) FROM {$wpdb->actionscheduler_actions} WHERE claim_id != 0 AND status IN ( %s, %s)"; |
| 690 | - $sql = $wpdb->prepare( $sql, [ self::STATUS_PENDING, self::STATUS_RUNNING ] ); |
|
| 690 | + $sql = $wpdb->prepare($sql, [self::STATUS_PENDING, self::STATUS_RUNNING]); |
|
| 691 | 691 | |
| 692 | - return (int) $wpdb->get_var( $sql ); |
|
| 692 | + return (int) $wpdb->get_var($sql); |
|
| 693 | 693 | } |
| 694 | 694 | |
| 695 | 695 | /** |
@@ -698,14 +698,14 @@ discard block |
||
| 698 | 698 | * @param string $action_id Action ID. |
| 699 | 699 | * @return mixed |
| 700 | 700 | */ |
| 701 | - public function get_claim_id( $action_id ) { |
|
| 701 | + public function get_claim_id($action_id) { |
|
| 702 | 702 | /** @var \wpdb $wpdb */ |
| 703 | 703 | global $wpdb; |
| 704 | 704 | |
| 705 | 705 | $sql = "SELECT claim_id FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d"; |
| 706 | - $sql = $wpdb->prepare( $sql, $action_id ); |
|
| 706 | + $sql = $wpdb->prepare($sql, $action_id); |
|
| 707 | 707 | |
| 708 | - return (int) $wpdb->get_var( $sql ); |
|
| 708 | + return (int) $wpdb->get_var($sql); |
|
| 709 | 709 | } |
| 710 | 710 | |
| 711 | 711 | /** |
@@ -715,16 +715,16 @@ discard block |
||
| 715 | 715 | * |
| 716 | 716 | * @return int[] |
| 717 | 717 | */ |
| 718 | - public function find_actions_by_claim_id( $claim_id ) { |
|
| 718 | + public function find_actions_by_claim_id($claim_id) { |
|
| 719 | 719 | /** @var \wpdb $wpdb */ |
| 720 | 720 | global $wpdb; |
| 721 | 721 | |
| 722 | 722 | $sql = "SELECT action_id FROM {$wpdb->actionscheduler_actions} WHERE claim_id=%d"; |
| 723 | - $sql = $wpdb->prepare( $sql, $claim_id ); |
|
| 723 | + $sql = $wpdb->prepare($sql, $claim_id); |
|
| 724 | 724 | |
| 725 | - $action_ids = $wpdb->get_col( $sql ); |
|
| 725 | + $action_ids = $wpdb->get_col($sql); |
|
| 726 | 726 | |
| 727 | - return array_map( 'intval', $action_ids ); |
|
| 727 | + return array_map('intval', $action_ids); |
|
| 728 | 728 | } |
| 729 | 729 | |
| 730 | 730 | /** |
@@ -732,11 +732,11 @@ discard block |
||
| 732 | 732 | * |
| 733 | 733 | * @param ActionScheduler_ActionClaim $claim Claim object. |
| 734 | 734 | */ |
| 735 | - public function release_claim( ActionScheduler_ActionClaim $claim ) { |
|
| 735 | + public function release_claim(ActionScheduler_ActionClaim $claim) { |
|
| 736 | 736 | /** @var \wpdb $wpdb */ |
| 737 | 737 | global $wpdb; |
| 738 | - $wpdb->update( $wpdb->actionscheduler_actions, [ 'claim_id' => 0 ], [ 'claim_id' => $claim->get_id() ], [ '%d' ], [ '%d' ] ); |
|
| 739 | - $wpdb->delete( $wpdb->actionscheduler_claims, [ 'claim_id' => $claim->get_id() ], [ '%d' ] ); |
|
| 738 | + $wpdb->update($wpdb->actionscheduler_actions, ['claim_id' => 0], ['claim_id' => $claim->get_id()], ['%d'], ['%d']); |
|
| 739 | + $wpdb->delete($wpdb->actionscheduler_claims, ['claim_id' => $claim->get_id()], ['%d']); |
|
| 740 | 740 | } |
| 741 | 741 | |
| 742 | 742 | /** |
@@ -746,15 +746,15 @@ discard block |
||
| 746 | 746 | * |
| 747 | 747 | * @return void |
| 748 | 748 | */ |
| 749 | - public function unclaim_action( $action_id ) { |
|
| 749 | + public function unclaim_action($action_id) { |
|
| 750 | 750 | /** @var \wpdb $wpdb */ |
| 751 | 751 | global $wpdb; |
| 752 | 752 | $wpdb->update( |
| 753 | 753 | $wpdb->actionscheduler_actions, |
| 754 | - [ 'claim_id' => 0 ], |
|
| 755 | - [ 'action_id' => $action_id ], |
|
| 756 | - [ '%s' ], |
|
| 757 | - [ '%d' ] |
|
| 754 | + ['claim_id' => 0], |
|
| 755 | + ['action_id' => $action_id], |
|
| 756 | + ['%s'], |
|
| 757 | + ['%d'] |
|
| 758 | 758 | ); |
| 759 | 759 | } |
| 760 | 760 | |
@@ -763,18 +763,18 @@ discard block |
||
| 763 | 763 | * |
| 764 | 764 | * @param int $action_id Action ID. |
| 765 | 765 | */ |
| 766 | - public function mark_failure( $action_id ) { |
|
| 766 | + public function mark_failure($action_id) { |
|
| 767 | 767 | /** @var \wpdb $wpdb */ |
| 768 | 768 | global $wpdb; |
| 769 | 769 | $updated = $wpdb->update( |
| 770 | 770 | $wpdb->actionscheduler_actions, |
| 771 | - [ 'status' => self::STATUS_FAILED ], |
|
| 772 | - [ 'action_id' => $action_id ], |
|
| 773 | - [ '%s' ], |
|
| 774 | - [ '%d' ] |
|
| 771 | + ['status' => self::STATUS_FAILED], |
|
| 772 | + ['action_id' => $action_id], |
|
| 773 | + ['%s'], |
|
| 774 | + ['%d'] |
|
| 775 | 775 | ); |
| 776 | - if ( empty( $updated ) ) { |
|
| 777 | - throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) ); |
|
| 776 | + if (empty($updated)) { |
|
| 777 | + throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'action-scheduler'), $action_id)); |
|
| 778 | 778 | } |
| 779 | 779 | } |
| 780 | 780 | |
@@ -785,13 +785,13 @@ discard block |
||
| 785 | 785 | * |
| 786 | 786 | * @return void |
| 787 | 787 | */ |
| 788 | - public function log_execution( $action_id ) { |
|
| 788 | + public function log_execution($action_id) { |
|
| 789 | 789 | /** @var \wpdb $wpdb */ |
| 790 | 790 | global $wpdb; |
| 791 | 791 | |
| 792 | 792 | $sql = "UPDATE {$wpdb->actionscheduler_actions} SET attempts = attempts+1, status=%s, last_attempt_gmt = %s, last_attempt_local = %s WHERE action_id = %d"; |
| 793 | - $sql = $wpdb->prepare( $sql, self::STATUS_RUNNING, current_time( 'mysql', true ), current_time( 'mysql' ), $action_id ); |
|
| 794 | - $wpdb->query( $sql ); |
|
| 793 | + $sql = $wpdb->prepare($sql, self::STATUS_RUNNING, current_time('mysql', true), current_time('mysql'), $action_id); |
|
| 794 | + $wpdb->query($sql); |
|
| 795 | 795 | } |
| 796 | 796 | |
| 797 | 797 | /** |
@@ -801,22 +801,22 @@ discard block |
||
| 801 | 801 | * |
| 802 | 802 | * @return void |
| 803 | 803 | */ |
| 804 | - public function mark_complete( $action_id ) { |
|
| 804 | + public function mark_complete($action_id) { |
|
| 805 | 805 | /** @var \wpdb $wpdb */ |
| 806 | 806 | global $wpdb; |
| 807 | 807 | $updated = $wpdb->update( |
| 808 | 808 | $wpdb->actionscheduler_actions, |
| 809 | 809 | [ |
| 810 | 810 | 'status' => self::STATUS_COMPLETE, |
| 811 | - 'last_attempt_gmt' => current_time( 'mysql', true ), |
|
| 812 | - 'last_attempt_local' => current_time( 'mysql' ), |
|
| 811 | + 'last_attempt_gmt' => current_time('mysql', true), |
|
| 812 | + 'last_attempt_local' => current_time('mysql'), |
|
| 813 | 813 | ], |
| 814 | - [ 'action_id' => $action_id ], |
|
| 815 | - [ '%s' ], |
|
| 816 | - [ '%d' ] |
|
| 814 | + ['action_id' => $action_id], |
|
| 815 | + ['%s'], |
|
| 816 | + ['%d'] |
|
| 817 | 817 | ); |
| 818 | - if ( empty( $updated ) ) { |
|
| 819 | - throw new \InvalidArgumentException( sprintf( __( 'Unidentified action %s', 'action-scheduler' ), $action_id ) ); |
|
| 818 | + if (empty($updated)) { |
|
| 819 | + throw new \InvalidArgumentException(sprintf(__('Unidentified action %s', 'action-scheduler'), $action_id)); |
|
| 820 | 820 | } |
| 821 | 821 | } |
| 822 | 822 | |
@@ -827,17 +827,17 @@ discard block |
||
| 827 | 827 | * |
| 828 | 828 | * @return string |
| 829 | 829 | */ |
| 830 | - public function get_status( $action_id ) { |
|
| 830 | + public function get_status($action_id) { |
|
| 831 | 831 | /** @var \wpdb $wpdb */ |
| 832 | 832 | global $wpdb; |
| 833 | 833 | $sql = "SELECT status FROM {$wpdb->actionscheduler_actions} WHERE action_id=%d"; |
| 834 | - $sql = $wpdb->prepare( $sql, $action_id ); |
|
| 835 | - $status = $wpdb->get_var( $sql ); |
|
| 834 | + $sql = $wpdb->prepare($sql, $action_id); |
|
| 835 | + $status = $wpdb->get_var($sql); |
|
| 836 | 836 | |
| 837 | - if ( $status === null ) { |
|
| 838 | - throw new \InvalidArgumentException( __( 'Invalid action ID. No status found.', 'action-scheduler' ) ); |
|
| 839 | - } elseif ( empty( $status ) ) { |
|
| 840 | - throw new \RuntimeException( __( 'Unknown status found for action.', 'action-scheduler' ) ); |
|
| 837 | + if ($status === null) { |
|
| 838 | + throw new \InvalidArgumentException(__('Invalid action ID. No status found.', 'action-scheduler')); |
|
| 839 | + } elseif (empty($status)) { |
|
| 840 | + throw new \RuntimeException(__('Unknown status found for action.', 'action-scheduler')); |
|
| 841 | 841 | } else { |
| 842 | 842 | return $status; |
| 843 | 843 | } |