WordPoints / wordpoints

src/classes/hook/reaction/store.php

Doc Comments +1 added lines, -1 removed lines

@@ -190,7 +190,7 @@
 	 *
 	 * @param string $event_slug The slug of the event this reaction is for.
 	 *
-	 * @return int|false The reaction ID, or false if not created.
+	 * @return integer The reaction ID, or false if not created.
 	 */
 	abstract protected function _create_reaction( $event_slug );
 }

Spacing +5 added lines, -5 removed lines

@@ -144,7 +144,7 @@ 
 
 		if ( $is_new ) {
 
-			$id = $this->_create_reaction( $settings['event'] );
+			$id = $this->_create_reaction( $settings[ 'event' ] );
 
 			if ( ! $id ) {
 				return false;
@@ -153,14 +153,14 @@ 
 
 		$reaction = $this->get_reaction( $id );
 
-		$reaction->update_event_slug( $settings['event'] );
+		$reaction->update_event_slug( $settings[ 'event' ] );
 
-		unset( $settings['event'] );
+		unset( $settings[ 'event' ] );
 
-		$reaction->update_meta( 'reactor', $settings['reactor'] );
+		$reaction->update_meta( 'reactor', $settings[ 'reactor' ] );
 
 		/** @var WordPoints_Hook_ReactorI $reactor */
-		$reactor = wordpoints_hooks()->get_sub_app( 'reactors' )->get( $settings['reactor'] );
+		$reactor = wordpoints_hooks()->get_sub_app( 'reactors' )->get( $settings[ 'reactor' ] );
 		$reactor->update_settings( $reaction, $settings );
 
 		/** @var WordPoints_Hook_ExtensionI $extension */

src/components/points/includes/class-wordpoints-points-hook.php

Doc Comments +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
 	 *
 	 * @param int|string $number The hook number or ID to get the points for.
 	 *
-	 * @return int|false The number of points for this instance, or false.
+	 * @return integer The number of points for this instance, or false.
 	 */
 	public function get_points( $number = null ) {
 

Spacing +17 added lines, -17 removed lines

@@ -115,8 +115,8 @@ 
 
 		$instances = $this->get_instances();
 
-		if ( isset( $instances[ $number ]['points'] ) ) {
-			return $instances[ $number ]['points'];
+		if ( isset( $instances[ $number ][ 'points' ] ) ) {
+			return $instances[ $number ][ 'points' ];
 		}
 
 		return false;
@@ -151,9 +151,9 @@ 
 
 		$new_instance = array_merge( array( 'points' => 0 ), $old_instance, $new_instance );
 
-		if ( false === wordpoints_posint( $new_instance['points'] ) ) {
-			if ( isset( $this->defaults['points'] ) ) {
-				$new_instance['points'] = $this->defaults['points'];
+		if ( false === wordpoints_posint( $new_instance[ 'points' ] ) ) {
+			if ( isset( $this->defaults[ 'points' ] ) ) {
+				$new_instance[ 'points' ] = $this->defaults[ 'points' ];
 			} else {
 				return false;
 			}
@@ -180,20 +180,20 @@ 
 	 */
 	protected function form( $instance ) {
 
-		if ( ! isset( $instance['points'] ) ) {
+		if ( ! isset( $instance[ 'points' ] ) ) {
 
-			if ( isset( $this->defaults['points'] ) ) {
-				$instance['points'] = $this->defaults['points'];
+			if ( isset( $this->defaults[ 'points' ] ) ) {
+				$instance[ 'points' ] = $this->defaults[ 'points' ];
 			} else {
-				$instance['points'] = 0;
+				$instance[ 'points' ] = 0;
 			}
 		}
 
 		?>
 
 		<p>
-			<label for="<?php $this->the_field_id( 'points' ); ?>"><?php echo esc_html( $this->options['points_label'] ); ?></label>
-			<input class="widefat" name="<?php $this->the_field_name( 'points' ); ?>" id="<?php $this->the_field_id( 'points' ); ?>" type="number" min="0" value="<?php echo wordpoints_posint( $instance['points'] ); ?>" />
+			<label for="<?php $this->the_field_id( 'points' ); ?>"><?php echo esc_html( $this->options[ 'points_label' ] ); ?></label>
+			<input class="widefat" name="<?php $this->the_field_name( 'points' ); ?>" id="<?php $this->the_field_id( 'points' ); ?>" type="number" min="0" value="<?php echo wordpoints_posint( $instance[ 'points' ] ); ?>" />
 		</p>
 
 		<?php
@@ -212,7 +212,7 @@ 
 	 */
 	protected function generate_description( $instance = array() ) {
 
-		return $this->options['description'];
+		return $this->options[ 'description' ];
 	}
 
 	//
@@ -442,7 +442,7 @@ 
 			break;
 		}
 
-		unset( $instances['__i__'] );
+		unset( $instances[ '__i__' ] );
 
 		return $instances;
 	}
@@ -673,8 +673,8 @@ 
 
 		$instances = $this->get_instances();
 
-		if ( 'generated' !== $type && ! empty( $instances[ $this->number ]['_description'] ) ) {
-			return $instances[ $this->number ]['_description'];
+		if ( 'generated' !== $type && ! empty( $instances[ $this->number ][ '_description' ] ) ) {
+			return $instances[ $this->number ][ '_description' ];
 		}
 
 		$instance = ( isset( $instances[ $this->number ] ) ) ? $instances[ $this->number ] : array();
@@ -775,7 +775,7 @@ 
 			, $options
 		);
 
-		$this->options['_classname'] = $this->option_name;
+		$this->options[ '_classname' ] = $this->option_name;
 	}
 
 	//
@@ -792,7 +792,7 @@ 
 	final private function save_instances( $instances ) {
 
 		// This needs to start at 1.
-		unset( $instances[0] );
+		unset( $instances[ 0 ] );
 
 		if ( WordPoints_Points_Hooks::get_network_mode() ) {
 			update_site_option( $this->option_name, $instances );

src/components/points/includes/hooks/abstracts/post-type.php

Doc Comments +1 added lines, -1 removed lines

@@ -421,7 +421,7 @@
 	 * @since 2.0.0
 	 *
 	 * @param string $key       The log meta key to match on.
-	 * @param mixed  $value     The log meta value to match on.
+	 * @param integer  $value     The log meta value to match on.
 	 * @param string $new_key   The key for the new meta value to add.
 	 * @param mixed  $new_value The new meta value to add. If falsey, new meta isn't added.
 	 */

Spacing +17 added lines, -17 removed lines

@@ -99,14 +99,14 @@ 
 
 			$instance = array_merge( $this->defaults, $instance );
 
-			if ( $post_type === $instance['post_type'] ) {
-				return ! empty( $instance['auto_reverse'] );
-			} elseif ( 'ALL' === $instance['post_type'] ) {
+			if ( $post_type === $instance[ 'post_type' ] ) {
+				return ! empty( $instance[ 'auto_reverse' ] );
+			} elseif ( 'ALL' === $instance[ 'post_type' ] ) {
 				$all_posts_instance = $instance;
 			}
 		}
 
-		return ! empty( $all_posts_instance['auto_reverse'] );
+		return ! empty( $all_posts_instance[ 'auto_reverse' ] );
 	}
 
 	/**
@@ -186,8 +186,8 @@ 
 
 		$post = false;
 
-		if ( '' === $reverse && isset( $meta['post_id'] ) ) {
-			$post = get_post( $meta['post_id'] );
+		if ( '' === $reverse && isset( $meta[ 'post_id' ] ) ) {
+			$post = get_post( $meta[ 'post_id' ] );
 		}
 
 		if ( $post ) {
@@ -202,22 +202,22 @@ 
 
 			if (
 				$this->get_option( 'log_text_post_type' . $reverse )
-				&& isset( $meta['post_type'] )
-				&& post_type_exists( $meta['post_type'] )
+				&& isset( $meta[ 'post_type' ] )
+				&& post_type_exists( $meta[ 'post_type' ] )
 			) {
 
 				// We do know the type of post though, so include that in the log.
 				$text = sprintf(
 					$this->get_option( 'log_text_post_type' . $reverse )
-					, get_post_type_object( $meta['post_type'] )->labels->singular_name
+					, get_post_type_object( $meta[ 'post_type' ] )->labels->singular_name
 				);
 
-			} elseif ( isset( $meta['post_title'] ) ) {
+			} elseif ( isset( $meta[ 'post_title' ] ) ) {
 
 				// If the title is saved as metadata, then we can use it.
 				$text = sprintf(
 					$this->get_option( 'log_text_post_title' . $reverse )
-					, $meta['post_title']
+					, $meta[ 'post_title' ]
 				);
 			}
 		}
@@ -246,7 +246,7 @@ 
 
 		$args = array();
 
-		$args[] = '<a href="' . esc_url( $url ) . '">'
+		$args[ ] = '<a href="' . esc_url( $url ) . '">'
 			. ( $post_title ? $post_title : _x( '(no title)', 'post title', 'wordpoints' ) )
 			. '</a>';
 
@@ -255,7 +255,7 @@ 
 		$post_type = get_post_field( 'post_type', $post_id );
 
 		if ( $text && $post_type && post_type_exists( $post_type ) ) {
-			$args[] = get_post_type_object( $post_type )->labels->singular_name;
+			$args[ ] = get_post_type_object( $post_type )->labels->singular_name;
 		} else {
 			$text = $this->get_option( 'log_text_post_title' . $reverse );
 		}
@@ -274,8 +274,8 @@ 
 	 */
 	protected function generate_description( $instance = array() ) {
 
-		if ( ! empty( $instance['post_type'] ) && 'ALL' !== $instance['post_type'] ) {
-			$post_type = get_post_type_object( $instance['post_type'] );
+		if ( ! empty( $instance[ 'post_type' ] ) && 'ALL' !== $instance[ 'post_type' ] ) {
+			$post_type = get_post_type_object( $instance[ 'post_type' ] );
 
 			if ( $post_type ) {
 				return sprintf( $this->get_option( 'post_type_description' ), $post_type->labels->singular_name );
@@ -306,7 +306,7 @@ 
 
 			wordpoints_list_post_types(
 				array(
-					'selected' => $instance['post_type'],
+					'selected' => $instance[ 'post_type' ],
 					'id'       => $this->get_field_id( 'post_type' ),
 					'name'     => $this->get_field_name( 'post_type' ),
 					'class'    => 'widefat wordpoints-append-to-hook-title',
@@ -329,7 +329,7 @@ 
 			?>
 
 			<p>
-				<input class="widefat" name="<?php $this->the_field_name( 'auto_reverse' ); ?>" id="<?php $this->the_field_id( 'auto_reverse' ); ?>" type="checkbox" value="1" <?php checked( '1', $instance['auto_reverse'] ); ?> />
+				<input class="widefat" name="<?php $this->the_field_name( 'auto_reverse' ); ?>" id="<?php $this->the_field_id( 'auto_reverse' ); ?>" type="checkbox" value="1" <?php checked( '1', $instance[ 'auto_reverse' ] ); ?> />
 				<label for="<?php $this->the_field_id( 'auto_reverse' ); ?>"><?php echo esc_html( $disable_label ); ?></label>
 			</p>
 

src/components/points/includes/logs.php

Doc Comments +11 added lines, -1 removed lines

@@ -294,7 +294,7 @@ 
  *        'default', which will return a list of the most recent points logs, with
  *        users excluded according to the general settings.
  *
- * @return WordPoints_Points_Logs_Query|false Logs query instance, or false.
+ * @return WordPoints_Points_Logs_Query Logs query instance, or false.
  */
 function wordpoints_get_points_logs_query( $points_type, $query_slug = 'default' ) {
 
@@ -466,6 +466,11 @@ 
  * @since 1.9.0
  *
  * @WordPress\action wordpoints_points_log-comment_disapprove
+ * @param string $text
+ * @param integer $points
+ * @param string $points_type
+ * @param integer $user_id
+ * @param string $log_type
  */
 function wordpoints_points_logs_comment_disapprove( $text, $points, $points_type, $user_id, $log_type, $meta ) {
 
@@ -492,6 +497,11 @@ 
  * @since 1.9.0
  *
  * @WordPress\action wordpoints_points_log-post_delete
+ * @param string $text
+ * @param integer $points
+ * @param string $points_type
+ * @param integer $user_id
+ * @param string $log_type
  */
 function wordpoints_points_logs_post_delete( $text, $points, $points_type, $user_id, $log_type, $meta ) {
 

Spacing +18 added lines, -18 removed lines

@@ -89,7 +89,7 @@ 
 
 	// The current user needs to be set dynamically, since it can change at times.
 	if ( 'current_user' === $query_slug ) {
-		$args['user_id'] = get_current_user_id();
+		$args[ 'user_id' ] = get_current_user_id();
 	}
 
 	/**
@@ -137,12 +137,12 @@ 
 
 	$query_data = WordPoints_Points_Log_Queries::get_query_data( $query_slug );
 
-	if ( $query_data['cache_queries'] ) {
+	if ( $query_data[ 'cache_queries' ] ) {
 
 		$query->prime_cache(
-			$query_data['cache_key']
+			$query_data[ 'cache_key' ]
 			, null
-			, $query_data['network_wide']
+			, $query_data[ 'network_wide' ]
 		);
 	}
 
@@ -196,7 +196,7 @@ 
 
 	$args = array_merge( $defaults, $args );
 
-	if ( ! $args['datatable'] ) {
+	if ( ! $args[ 'datatable' ] ) {
 
 		_deprecated_argument(
 			__FUNCTION__
@@ -204,13 +204,13 @@ 
 			, '$args["datatable"] is deprecated and should no longer be used. Use $args["paginate"] instead.'
 		);
 
-		$args['paginate'] = false;
+		$args[ 'paginate' ] = false;
 	}
 
 	$view = wordpoints_component( 'points' )
 		->get_sub_app( 'logs' )
 		->get_sub_app( 'views' )
-		->get( $args['view'], array( $logs_query, $args ) );
+		->get( $args[ 'view' ], array( $logs_query, $args ) );
 
 	if ( ! $view instanceof WordPoints_Points_Logs_View ) {
 		return;
@@ -284,10 +284,10 @@ 
  */
 function wordpoints_points_logs_profile_edit( $text, $points, $points_type, $user_id, $log_type, $meta ) {
 
-	$user_name = sanitize_user_field( 'display_name', get_userdata( $meta['user_id'] )->display_name, $meta['user_id'], 'display' );
+	$user_name = sanitize_user_field( 'display_name', get_userdata( $meta[ 'user_id' ] )->display_name, $meta[ 'user_id' ], 'display' );
 
 	// translators: 1. User name; 2. Reason given.
-	return sprintf( _x( 'Points adjusted by %1$s. Reason: %2$s', 'points log description', 'wordpoints' ), $user_name, esc_html( $meta['reason'] ) );
+	return sprintf( _x( 'Points adjusted by %1$s. Reason: %2$s', 'points log description', 'wordpoints' ), $user_name, esc_html( $meta[ 'reason' ] ) );
 }
 
 /**
@@ -299,7 +299,7 @@ 
  */
 function wordpoints_points_logs_comment_disapprove( $text, $points, $points_type, $user_id, $log_type, $meta ) {
 
-	switch ( $meta['status'] ) {
+	switch ( $meta[ 'status' ] ) {
 
 		case 'spam':
 			$text = _x( 'Comment marked as spam.', 'points log description', 'wordpoints' );
@@ -325,9 +325,9 @@ 
  */
 function wordpoints_points_logs_post_delete( $text, $points, $points_type, $user_id, $log_type, $meta ) {
 
-	if ( isset( $meta['post_type'] ) ) {
+	if ( isset( $meta[ 'post_type' ] ) ) {
 
-		$post_type = get_post_type_object( $meta['post_type'] );
+		$post_type = get_post_type_object( $meta[ 'post_type' ] );
 
 		if ( ! is_null( $post_type ) ) {
 
@@ -389,18 +389,18 @@ 
 		'%user_id%',
 	);
 
-	if ( empty( $args['points_type'] ) ) {
+	if ( empty( $args[ 'points_type' ] ) ) {
 		$points_types = array_keys( wordpoints_get_points_types() );
 	} else {
-		$points_types = (array) $args['points_type'];
+		$points_types = (array) $args[ 'points_type' ];
 	}
 
 	foreach ( $points_types as $points_type ) {
 		foreach ( WordPoints_Points_Log_Queries::get_queries() as $query ) {
 
-			if ( ! empty( $query['cache_key'] ) ) {
+			if ( ! empty( $query[ 'cache_key' ] ) ) {
 
-				if ( $query['network_wide'] ) {
+				if ( $query[ 'network_wide' ] ) {
 					$group = 'wordpoints_network_points_logs_query';
 				} else {
 					$group = 'wordpoints_points_logs_query';
@@ -408,11 +408,11 @@ 
 
 				$replace = array(
 					$points_type,
-					$args['user_id'],
+					$args[ 'user_id' ],
 				);
 
 				wp_cache_delete(
-					str_replace( $find, $replace, $query['cache_key'] )
+					str_replace( $find, $replace, $query[ 'cache_key' ] )
 					, $group
 				);
 			}

src/components/points/includes/points.php

Doc Comments +1 added lines, -1 removed lines

@@ -363,7 +363,7 @@
  *
  * @param string $type The slug for a points type.
  *
- * @return int|false The minimum for this type of points. False if $type is bad.
+ * @return integer The minimum for this type of points. False if $type is bad.
  */
 function wordpoints_get_points_minimum( $type ) {
 

Spacing +24 added lines, -24 removed lines

@@ -117,11 +117,11 @@ 
  */
 function wordpoints_add_points_type( $settings ) {
 
-	if ( ! is_array( $settings ) || ! isset( $settings['name'] ) ) {
+	if ( ! is_array( $settings ) || ! isset( $settings[ 'name' ] ) ) {
 		return false;
 	}
 
-	$slug = $settings['name'];
+	$slug = $settings[ 'name' ];
 	$slug = preg_replace( '/\s+/', '-', $slug );
 	$slug = preg_replace( '/-+/', '-', $slug );
 	$slug = trim( $slug, '-' );
@@ -165,7 +165,7 @@ 
 
 	$points_types = wordpoints_get_points_types();
 
-	if ( ! is_array( $settings ) || ! isset( $points_types[ $slug ], $settings['name'] ) ) {
+	if ( ! is_array( $settings ) || ! isset( $points_types[ $slug ], $settings[ 'name' ] ) ) {
 		return false;
 	}
 
@@ -1035,13 +1035,13 @@ 
 				, array( '%d' )
 			);
 
-			if ( ! isset( $flushed['points_types'][ $log->points_type ], $flushed['user_ids'][ $log->user_id ] ) ) {
+			if ( ! isset( $flushed[ 'points_types' ][ $log->points_type ], $flushed[ 'user_ids' ][ $log->user_id ] ) ) {
 				wordpoints_flush_points_logs_caches(
 					array( 'user_id' => $log->user_id, 'points_type' => $log->points_type )
 				);
 
-				$flushed['points_types'][ $log->points_type ] = true;
-				$flushed['user_ids'][ $log->user_id ]         = true;
+				$flushed[ 'points_types' ][ $log->points_type ] = true;
+				$flushed[ 'user_ids' ][ $log->user_id ]         = true;
 			}
 		}
 
@@ -1075,9 +1075,9 @@ 
 		$cache = array( 'is_max' => false, 'top_users' => array() );
 	}
 
-	$cached_users = count( $cache['top_users'] );
+	$cached_users = count( $cache[ 'top_users' ] );
 
-	if ( $num_users > $cached_users && ! $cache['is_max'] ) {
+	if ( $num_users > $cached_users && ! $cache[ 'is_max' ] ) {
 
 		global $wpdb;
 
@@ -1126,19 +1126,19 @@ 
 			return array();
 		}
 
-		$cache['top_users'] = array_merge( $cache['top_users'], $top_users );
+		$cache[ 'top_users' ] = array_merge( $cache[ 'top_users' ], $top_users );
 
-		if ( count( $cache['top_users'] ) < $num_users ) {
-			$cache['is_max'] = true;
+		if ( count( $cache[ 'top_users' ] ) < $num_users ) {
+			$cache[ 'is_max' ] = true;
 		}
 
-		$cache['top_users'] = array_map( 'intval', $cache['top_users'] );
+		$cache[ 'top_users' ] = array_map( 'intval', $cache[ 'top_users' ] );
 
 		wp_cache_set( $points_type, $cache, 'wordpoints_points_top_users' );
 
 	} // End if ( not cached ).
 
-	return array_slice( $cache['top_users'], 0, $num_users );
+	return array_slice( $cache[ 'top_users' ], 0, $num_users );
 }
 
 /**
@@ -1170,7 +1170,7 @@ 
 	$points_type_name = wordpoints_get_points_type_setting( $points_type, 'name' );
 
 	if ( ! empty( $points_type_name ) ) {
-		$column_headers['points'] = $points_type_name;
+		$column_headers[ 'points' ] = $points_type_name;
 	}
 
 	/**
@@ -1195,11 +1195,11 @@ 
 		<thead>
 			<tr>
 				<th scope="col">
-					<span aria-hidden="true"><?php echo esc_html( $column_headers['#'] ); ?></span>
-					<span class="screen-reader-text"><?php echo esc_html( $column_headers['position'] ); ?></span>
+					<span aria-hidden="true"><?php echo esc_html( $column_headers[ '#' ] ); ?></span>
+					<span class="screen-reader-text"><?php echo esc_html( $column_headers[ 'position' ] ); ?></span>
 				</th>
-				<th scope="col"><?php echo esc_html( $column_headers['user'] ); ?></th>
-				<th scope="col"><?php echo esc_html( $column_headers['points'] ); ?></th>
+				<th scope="col"><?php echo esc_html( $column_headers[ 'user' ] ); ?></th>
+				<th scope="col"><?php echo esc_html( $column_headers[ 'points' ] ); ?></th>
 			</tr>
 		</thead>
 		<tbody>
@@ -1267,11 +1267,11 @@ 
 		<tfoot>
 		<tr>
 			<th scope="col">
-				<span aria-hidden="true"><?php echo esc_html( $column_headers['#'] ); ?></span>
-				<span class="screen-reader-text"><?php echo esc_html( $column_headers['position'] ); ?></span>
+				<span aria-hidden="true"><?php echo esc_html( $column_headers[ '#' ] ); ?></span>
+				<span class="screen-reader-text"><?php echo esc_html( $column_headers[ 'position' ] ); ?></span>
 			</th>
-			<th scope="col"><?php echo esc_html( $column_headers['user'] ); ?></th>
-			<th scope="col"><?php echo esc_html( $column_headers['points'] ); ?></th>
+			<th scope="col"><?php echo esc_html( $column_headers[ 'user' ] ); ?></th>
+			<th scope="col"><?php echo esc_html( $column_headers[ 'points' ] ); ?></th>
 		</tr>
 		</tfoot>
 	</table>
@@ -1310,7 +1310,7 @@ 
 		$cache = wp_cache_get( $slug, 'wordpoints_points_top_users' );
 
 		// If there aren't fewer users than the cache holds, we don't need to clear it.
-		if ( ! is_array( $cache ) || ! $cache['is_max'] ) {
+		if ( ! is_array( $cache ) || ! $cache[ 'is_max' ] ) {
 			continue;
 		}
 
@@ -1335,7 +1335,7 @@ 
 		$cache = wp_cache_get( $slug, 'wordpoints_points_top_users' );
 
 		// If this user isn't in the cache, we don't need to clear it.
-		if ( ! is_array( $cache ) || ! in_array( (int) $user_id, array_map( 'intval', $cache['top_users'] ), true ) ) {
+		if ( ! is_array( $cache ) || ! in_array( (int) $user_id, array_map( 'intval', $cache[ 'top_users' ] ), true ) ) {
 			continue;
 		}
 

src/includes/functions.php

Doc Comments +2 added lines, -2 removed lines

@@ -173,7 +173,7 @@ 
  *
  * @param mixed $maybe_int The value to convert to an integer.
  *
- * @return int|false False if $maybe_int couldn't reliably be converted to an integer.
+ * @return integer False if $maybe_int couldn't reliably be converted to an integer.
  */
 function wordpoints_int( &$maybe_int ) {
 
@@ -969,7 +969,7 @@ 
  * @since 1.0.1
  * @since 1.8.0 The $message now supports WP_Error objects.
  *
- * @param string|WP_Error $message The error message.
+ * @param string $message The error message.
  *
  * @return string The error message.
  */

Braces +8 added lines, -4 removed lines

@@ -88,7 +88,8 @@ 
  */
 function wordpoints_maintenance_shutdown_print_rand_str() {
 
-	if ( ! isset( $_GET['wordpoints_module_check'] ) ) { // WPCS: CSRF OK.
+	if ( ! isset( $_GET['wordpoints_module_check'] ) ) {
+// WPCS: CSRF OK.
 		return;
 	}
 
@@ -98,7 +99,8 @@ 
 		$nonce = get_option( 'wordpoints_module_check_nonce' );
 	}
 
-	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET['wordpoints_module_check'] ) ) ) { // WPCS: CSRF OK.
+	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET['wordpoints_module_check'] ) ) ) {
+// WPCS: CSRF OK.
 		return;
 	}
 
@@ -128,7 +130,8 @@ 
  */
 function wordpoints_maintenance_filter_modules( $modules ) {
 
-	if ( ! isset( $_GET['check_module'], $_GET['wordpoints_module_check'] ) ) { // WPCS: CSRF OK.
+	if ( ! isset( $_GET['check_module'], $_GET['wordpoints_module_check'] ) ) {
+// WPCS: CSRF OK.
 		return $modules;
 	}
 
@@ -138,7 +141,8 @@ 
 		$nonce = get_option( 'wordpoints_module_check_nonce' );
 	}
 
-	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET['wordpoints_module_check'] ) ) ) { // WPCS: CSRF OK.
+	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET['wordpoints_module_check'] ) ) ) {
+// WPCS: CSRF OK.
 		return $modules;
 	}
 

Spacing +24 added lines, -24 removed lines

@@ -91,13 +91,13 @@ 
 
 	// Normally this should never happen, because the plugins_loaded action won't
 	// run until the next request after WordPoints is activated and installs itself.
-	if ( empty( $wordpoints_data['version'] ) ) {
+	if ( empty( $wordpoints_data[ 'version' ] ) ) {
 		return;
 	}
 
 	// The major version is determined by the first number, so we can just cast to
 	// an integer. IF the major versions are equal, we don't need to do anything.
-	if ( (int) WORDPOINTS_VERSION === (int) $wordpoints_data['version'] ) {
+	if ( (int) WORDPOINTS_VERSION === (int) $wordpoints_data[ 'version' ] ) {
 		return;
 	}
 
@@ -115,7 +115,7 @@ 
  */
 function wordpoints_maintenance_shutdown_print_rand_str() {
 
-	if ( ! isset( $_GET['wordpoints_module_check'] ) ) { // WPCS: CSRF OK.
+	if ( ! isset( $_GET[ 'wordpoints_module_check' ] ) ) { // WPCS: CSRF OK.
 		return;
 	}
 
@@ -125,7 +125,7 @@ 
 		$nonce = get_option( 'wordpoints_module_check_nonce' );
 	}
 
-	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET['wordpoints_module_check'] ) ) ) { // WPCS: CSRF OK.
+	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET[ 'wordpoints_module_check' ] ) ) ) { // WPCS: CSRF OK.
 		return;
 	}
 
@@ -155,7 +155,7 @@ 
  */
 function wordpoints_maintenance_filter_modules( $modules ) {
 
-	if ( ! isset( $_GET['check_module'], $_GET['wordpoints_module_check'] ) ) { // WPCS: CSRF OK.
+	if ( ! isset( $_GET[ 'check_module' ], $_GET[ 'wordpoints_module_check' ] ) ) { // WPCS: CSRF OK.
 		return $modules;
 	}
 
@@ -165,13 +165,13 @@ 
 		$nonce = get_option( 'wordpoints_module_check_nonce' );
 	}
 
-	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET['wordpoints_module_check'] ) ) ) { // WPCS: CSRF OK.
+	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET[ 'wordpoints_module_check' ] ) ) ) { // WPCS: CSRF OK.
 		return $modules;
 	}
 
 	$modules = explode(
 		','
-		, sanitize_text_field( wp_unslash( $_GET['check_module'] ) ) // WPCS: CSRF OK.
+		, sanitize_text_field( wp_unslash( $_GET[ 'check_module' ] ) ) // WPCS: CSRF OK.
 	);
 
 	if ( 'pre_site_option_wordpoints_sitewide_active_modules' === current_filter() ) {
@@ -353,7 +353,7 @@ 
 				return false;
 			}
 
-			$values[] = $request[ $value ];
+			$values[ ] = $request[ $value ];
 		}
 
 		$action_format = vsprintf( $action_format, $values );
@@ -395,10 +395,10 @@ 
 
 	$error_data = $error->error_data;
 
-	if ( isset( $error_data[ $code ]['title'] ) ) {
+	if ( isset( $error_data[ $code ][ 'title' ] ) ) {
 
-		$error_data[ $code ]['title'] = wp_kses(
-			$error->error_data[ $code ]['title']
+		$error_data[ $code ][ 'title' ] = wp_kses(
+			$error->error_data[ $code ][ 'title' ]
 			, 'wordpoints_sanitize_wp_error_title'
 		);
 
@@ -799,16 +799,16 @@ 
 
 	$options = array_merge( $defaults, $options );
 
-	echo '<select class="' . esc_attr( $options['class'] ) . '" name="' . esc_attr( $options['name'] ) . '" id="' . esc_attr( $options['id'] ) . '">';
-	echo '<option value="ALL"' . selected( $options['selected'], 'ALL', false ) . '>' . esc_html_x( 'Any', 'post type', 'wordpoints' ) . '</option>';
+	echo '<select class="' . esc_attr( $options[ 'class' ] ) . '" name="' . esc_attr( $options[ 'name' ] ) . '" id="' . esc_attr( $options[ 'id' ] ) . '">';
+	echo '<option value="ALL"' . selected( $options[ 'selected' ], 'ALL', false ) . '>' . esc_html_x( 'Any', 'post type', 'wordpoints' ) . '</option>';
 
 	foreach ( get_post_types( $args, 'objects' ) as $post_type ) {
 
-		if ( isset( $options['filter'] ) && ! call_user_func( $options['filter'], $post_type ) ) {
+		if ( isset( $options[ 'filter' ] ) && ! call_user_func( $options[ 'filter' ], $post_type ) ) {
 			continue;
 		}
 
-		echo '<option value="' . esc_attr( $post_type->name ) . '"' . selected( $options['selected'], $post_type->name, false ) . '>' . esc_html( $post_type->label ) . '</option>';
+		echo '<option value="' . esc_attr( $post_type->name ) . '"' . selected( $options[ 'selected' ], $post_type->name, false ) . '>' . esc_html( $post_type->label ) . '</option>';
 	}
 
 	echo '</select>';
@@ -1006,18 +1006,18 @@ 
 			, esc_html( "{$cap} is deprecated, use {$deprecated[ $cap ]} instead." )
 		);
 
-		$caps[] = $deprecated[ $cap ];
+		$caps[ ] = $deprecated[ $cap ];
 	}
 
 	switch ( $cap ) {
 		case 'install_wordpoints_extensions':
 		case 'update_wordpoints_extensions':
 			if ( defined( 'DISALLOW_FILE_MODS' ) && DISALLOW_FILE_MODS ) {
-				$caps[] = 'do_not_allow';
+				$caps[ ] = 'do_not_allow';
 			} elseif ( is_multisite() && ! is_super_admin( $user_id ) ) {
-				$caps[] = 'do_not_allow';
+				$caps[ ] = 'do_not_allow';
 			} else {
-				$caps[] = $cap;
+				$caps[ ] = $cap;
 			}
 		break;
 	}
@@ -1194,15 +1194,15 @@ 
 		case 0:
 			return new $class_name();
 		case 1:
-			return new $class_name( $args[0] );
+			return new $class_name( $args[ 0 ] );
 		case 2:
-			return new $class_name( $args[0], $args[1] );
+			return new $class_name( $args[ 0 ], $args[ 1 ] );
 		case 3:
-			return new $class_name( $args[0], $args[1], $args[2] );
+			return new $class_name( $args[ 0 ], $args[ 1 ], $args[ 2 ] );
 		case 4:
-			return new $class_name( $args[0], $args[1], $args[2], $args[3] );
+			return new $class_name( $args[ 0 ], $args[ 1 ], $args[ 2 ], $args[ 3 ] );
 		case 5:
-			return new $class_name( $args[0], $args[1], $args[2], $args[3], $args[4] );
+			return new $class_name( $args[ 0 ], $args[ 1 ], $args[ 2 ], $args[ 3 ], $args[ 4 ] );
 		default:
 			return false;
 	}

src/classes/class/registry/childreni.php

Doc Comments +2 added lines

@@ -92,6 +92,7 @@ 
 	 *
 	 * @param string $parent_slug The group slug.
 	 * @param string $slug        The slug of the class to deregister.
+	 * @return void
 	 */
 	public function deregister( $parent_slug, $slug );
 
@@ -101,6 +102,7 @@ 
 	 * @since 2.1.0
 	 *
 	 * @param string $parent_slug The group slug.
+	 * @return void
 	 */
 	public function deregister_children( $parent_slug );
 

src/classes/class/registry/deepi.php

Doc Comments +2 added lines

@@ -76,6 +76,7 @@ 
 	 * @param string   $slug         The slug of the class to deregister.
 	 * @param string[] $parent_slugs The slug(s) of the class's parent(s) in the
 	 *                               hierarchy.
+	 * @return void
 	 */
 	public function deregister( $slug, array $parent_slugs = array() );
 
@@ -85,6 +86,7 @@ 
 	 * @since 2.2.0
 	 *
 	 * @param string[] $parent_slugs The hierarchy of the parent.
+	 * @return void
 	 */
 	public function deregister_children( array $parent_slugs = array() );
 

src/classes/class/registryi.php

Doc Comments +1 added lines

@@ -71,6 +71,7 @@
 	 * @since 2.1.0
 	 *
 	 * @param string $slug The slug of the class to deregister.
+	 * @return void
 	 */
 	public function deregister( $slug );