@@ -21,6 +21,7 @@ |
||
| 21 | 21 | * @param string[] $hierarchy The hierarchy of this entity, consisting of the |
| 22 | 22 | * entity slug followed by the slugs leading to the |
| 23 | 23 | * descendant in question, if not the entity itself. |
| 24 | + * @return void |
|
| 24 | 25 | */ |
| 25 | 26 | public function __construct( $entity_id, array $hierarchy ); |
| 26 | 27 | |
@@ -20,6 +20,7 @@ |
||
| 20 | 20 | * @since 2.1.0 |
| 21 | 21 | * |
| 22 | 22 | * @param WordPoints_Hook_Fire $fire The hook fire object. |
| 23 | + * @return void |
|
| 23 | 24 | */ |
| 24 | 25 | public function after_hit( WordPoints_Hook_Fire $fire ); |
| 25 | 26 | } |
@@ -20,6 +20,7 @@ |
||
| 20 | 20 | * @since 2.1.0 |
| 21 | 21 | * |
| 22 | 22 | * @param WordPoints_Hook_Fire $fire The fire that was a miss. |
| 23 | + * @return void |
|
| 23 | 24 | */ |
| 24 | 25 | public function after_miss( WordPoints_Hook_Fire $fire ); |
| 25 | 26 | } |
@@ -38,6 +38,7 @@ |
||
| 38 | 38 | * |
| 39 | 39 | * @param WordPoints_Hook_ReactionI $reaction The reaction object. |
| 40 | 40 | * @param array $settings The settings for a hook reaction. |
| 41 | + * @return void |
|
| 41 | 42 | */ |
| 42 | 43 | public function update_settings( |
| 43 | 44 | WordPoints_Hook_ReactionI $reaction, |
@@ -18,6 +18,7 @@ |
||
| 18 | 18 | * @since 2.2.0 |
| 19 | 19 | * |
| 20 | 20 | * @param object $log The object for the points log the restriction is for. |
| 21 | + * @return void |
|
| 21 | 22 | */ |
| 22 | 23 | public function __construct( $log ); |
| 23 | 24 | |
@@ -280,7 +280,7 @@ |
||
| 280 | 280 | * |
| 281 | 281 | * @since 1.8.0 |
| 282 | 282 | * |
| 283 | - * @return string|WP_Error|null An error message or object on failure. |
|
| 283 | + * @return string|null An error message or object on failure. |
|
| 284 | 284 | */ |
| 285 | 285 | protected function verify_atts() { |
| 286 | 286 | |
@@ -171,9 +171,9 @@ discard block |
||
| 171 | 171 | protected function verify_atts() { |
| 172 | 172 | |
| 173 | 173 | // If this shortcode supports the user_id attribute, verify it. |
| 174 | - if ( isset( $this->pairs['user_id'] ) ) { |
|
| 174 | + if ( isset( $this->pairs[ 'user_id' ] ) ) { |
|
| 175 | 175 | |
| 176 | - if ( 'post_author' === $this->atts['user_id'] ) { |
|
| 176 | + if ( 'post_author' === $this->atts[ 'user_id' ] ) { |
|
| 177 | 177 | |
| 178 | 178 | $post = get_post(); |
| 179 | 179 | |
@@ -186,13 +186,13 @@ discard block |
||
| 186 | 186 | ); |
| 187 | 187 | } |
| 188 | 188 | |
| 189 | - $this->atts['user_id'] = $post->post_author; |
|
| 189 | + $this->atts[ 'user_id' ] = $post->post_author; |
|
| 190 | 190 | |
| 191 | - } elseif ( ! $this->atts['user_id'] ) { |
|
| 191 | + } elseif ( ! $this->atts[ 'user_id' ] ) { |
|
| 192 | 192 | |
| 193 | - $this->atts['user_id'] = get_current_user_id(); |
|
| 193 | + $this->atts[ 'user_id' ] = get_current_user_id(); |
|
| 194 | 194 | |
| 195 | - } elseif ( ! wordpoints_posint( $this->atts['user_id'] ) ) { |
|
| 195 | + } elseif ( ! wordpoints_posint( $this->atts[ 'user_id' ] ) ) { |
|
| 196 | 196 | |
| 197 | 197 | return sprintf( |
| 198 | 198 | // translators: 1. Attribute name; 2. Shortcode name; 3. Expected value. |
@@ -20,7 +20,7 @@ |
||
| 20 | 20 | } |
| 21 | 21 | |
| 22 | 22 | // If we're not running a module check, let the maintenance message show. |
| 23 | -if ( ! isset( $_GET['wordpoints_module_check'], $_GET['check_module'] ) ) { // WPCS: CSRF OK. |
|
| 23 | +if ( ! isset( $_GET[ 'wordpoints_module_check' ], $_GET[ 'check_module' ] ) ) { // WPCS: CSRF OK. |
|
| 24 | 24 | return; |
| 25 | 25 | } |
| 26 | 26 | |
@@ -20,7 +20,8 @@ |
||
| 20 | 20 | } |
| 21 | 21 | |
| 22 | 22 | // If we're not running a module check, let the maintenance message show. |
| 23 | -if ( ! isset( $_GET['wordpoints_module_check'], $_GET['check_module'] ) ) { // WPCS: CSRF OK. |
|
| 23 | +if ( ! isset( $_GET['wordpoints_module_check'], $_GET['check_module'] ) ) { |
|
| 24 | +// WPCS: CSRF OK. |
|
| 24 | 25 | return; |
| 25 | 26 | } |
| 26 | 27 | |
@@ -316,7 +316,7 @@ discard block |
||
| 316 | 316 | private function verify_request( $action ) { |
| 317 | 317 | |
| 318 | 318 | if ( |
| 319 | - empty( $_POST['nonce'] ) |
|
| 319 | + empty( $_POST[ 'nonce' ] ) |
|
| 320 | 320 | || ! wordpoints_verify_nonce( 'nonce', $action, null, 'post' ) |
| 321 | 321 | ) { |
| 322 | 322 | wp_send_json_error( |
@@ -334,11 +334,11 @@ discard block |
||
| 334 | 334 | */ |
| 335 | 335 | protected function get_reaction_store() { |
| 336 | 336 | |
| 337 | - if ( ! isset( $_POST['reaction_store'] ) ) { // WPCS: CSRF OK. |
|
| 337 | + if ( ! isset( $_POST[ 'reaction_store' ] ) ) { // WPCS: CSRF OK. |
|
| 338 | 338 | $this->unexpected_error( 'reaction_store' ); |
| 339 | 339 | } |
| 340 | 340 | |
| 341 | - $reactor_slug = sanitize_key( $_POST['reaction_store'] ); // WPCS: CSRF OK. |
|
| 341 | + $reactor_slug = sanitize_key( $_POST[ 'reaction_store' ] ); // WPCS: CSRF OK. |
|
| 342 | 342 | |
| 343 | 343 | $reaction_store = wordpoints_hooks()->get_reaction_store( $reactor_slug ); |
| 344 | 344 | |
@@ -361,12 +361,12 @@ discard block |
||
| 361 | 361 | */ |
| 362 | 362 | protected function get_reaction() { |
| 363 | 363 | |
| 364 | - if ( ! isset( $_POST['id'] ) ) { // WPCS: CSRF OK. |
|
| 364 | + if ( ! isset( $_POST[ 'id' ] ) ) { // WPCS: CSRF OK. |
|
| 365 | 365 | $this->unexpected_error( 'id' ); |
| 366 | 366 | } |
| 367 | 367 | |
| 368 | 368 | $reaction = $this->reaction_store->get_reaction( |
| 369 | - wordpoints_int( $_POST['id'] ) // WPCS: CSRF OK. |
|
| 369 | + wordpoints_int( $_POST[ 'id' ] ) // WPCS: CSRF OK. |
|
| 370 | 370 | ); |
| 371 | 371 | |
| 372 | 372 | if ( ! $reaction ) { |
@@ -387,7 +387,7 @@ discard block |
||
| 387 | 387 | |
| 388 | 388 | $data = wp_unslash( $_POST ); // WPCS: CSRF OK. |
| 389 | 389 | |
| 390 | - unset( $data['id'], $data['action'], $data['nonce'], $data['reaction_store'] ); |
|
| 390 | + unset( $data[ 'id' ], $data[ 'action' ], $data[ 'nonce' ], $data[ 'reaction_store' ] ); |
|
| 391 | 391 | |
| 392 | 392 | return $data; |
| 393 | 393 | } |
@@ -334,7 +334,8 @@ discard block |
||
| 334 | 334 | */ |
| 335 | 335 | protected function get_reaction_store() { |
| 336 | 336 | |
| 337 | - if ( ! isset( $_POST['reaction_store'] ) ) { // WPCS: CSRF OK. |
|
| 337 | + if ( ! isset( $_POST['reaction_store'] ) ) { |
|
| 338 | +// WPCS: CSRF OK. |
|
| 338 | 339 | $this->unexpected_error( 'reaction_store' ); |
| 339 | 340 | } |
| 340 | 341 | |
@@ -361,7 +362,8 @@ discard block |
||
| 361 | 362 | */ |
| 362 | 363 | protected function get_reaction() { |
| 363 | 364 | |
| 364 | - if ( ! isset( $_POST['id'] ) ) { // WPCS: CSRF OK. |
|
| 365 | + if ( ! isset( $_POST['id'] ) ) { |
|
| 366 | +// WPCS: CSRF OK. |
|
| 365 | 367 | $this->unexpected_error( 'id' ); |
| 366 | 368 | } |
| 367 | 369 | |
@@ -13,15 +13,15 @@ |
||
| 13 | 13 | |
| 14 | 14 | if ( |
| 15 | 15 | 'components' !== wordpoints_admin_get_current_tab() |
| 16 | - || ! isset( $_POST['wordpoints_component'], $_POST['wordpoints_component_action'], $_POST['_wpnonce'] ) |
|
| 16 | + || ! isset( $_POST[ 'wordpoints_component' ], $_POST[ 'wordpoints_component_action' ], $_POST[ '_wpnonce' ] ) |
|
| 17 | 17 | ) { |
| 18 | 18 | return; |
| 19 | 19 | } |
| 20 | 20 | |
| 21 | 21 | $components = WordPoints_Components::instance(); |
| 22 | -$component = sanitize_key( $_POST['wordpoints_component'] ); |
|
| 22 | +$component = sanitize_key( $_POST[ 'wordpoints_component' ] ); |
|
| 23 | 23 | |
| 24 | -$action = sanitize_key( $_POST['wordpoints_component_action'] ); |
|
| 24 | +$action = sanitize_key( $_POST[ 'wordpoints_component_action' ] ); |
|
| 25 | 25 | |
| 26 | 26 | check_admin_referer( "wordpoints_{$action}_component-{$component}" ); |
| 27 | 27 | |