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-un-installer.php

Doc Comments +1 added lines

@@ -1073,6 +1073,7 @@
 	 *
 	 * @since 2.1.4
 	 *
+	 * @param string[] $post_types
 	 * @return array The slugs of the events.
 	 */
 	protected function _2_1_4_get_reversal_log_types( $post_types ) {

Spacing +26 added lines, -26 removed lines

@@ -150,12 +150,12 @@ 
 		if ( 1 === version_compare( '1.5.0', $this->updating_from ) ) {
 
 			if ( ! $this->network_wide ) {
-				unset( $this->updates['1_5_0'] );
+				unset( $this->updates[ '1_5_0' ] );
 			}
 		}
 
 		if ( $this->network_wide ) {
-			unset( $this->updates['1_8_0'] );
+			unset( $this->updates[ '1_8_0' ] );
 		}
 
 		if ( 1 === version_compare( '1.9.0', $this->updating_from ) ) {
@@ -495,8 +495,8 @@ 
 				continue;
 			}
 
-			if ( ! isset( $settings['post_type'] ) ) {
-				$settings['post_type'] = 'ALL';
+			if ( ! isset( $settings[ 'post_type' ] ) ) {
+				$settings[ 'post_type' ] = 'ALL';
 			}
 
 			// If the trash points are set, create a post delete points hook instead.
@@ -505,7 +505,7 @@ 
 				$new_hook->update_callback(
 					array(
 						'points'    => $settings[ $split_key ],
-						'post_type' => $settings['post_type'],
+						'post_type' => $settings[ 'post_type' ],
 					)
 					, $new_hook->next_hook_id_number()
 				);
@@ -514,13 +514,13 @@ 
 				$points_type = $hook->points_type( $network_ . $number );
 
 				// Add this instance to the points-types-hooks list.
-				$points_types_hooks[ $points_type ][] = $new_hook->get_id( $number );
+				$points_types_hooks[ $points_type ][ ] = $new_hook->get_id( $number );
 			}
 
 			// If the publish points are set, update the settings of the hook.
 			if ( isset( $settings[ $key ] ) && wordpoints_posint( $settings[ $key ] ) ) {
 
-				$settings['points'] = $settings[ $key ];
+				$settings[ 'points' ] = $settings[ $key ];
 
 				$hook->update_callback( $settings, $number );
 
@@ -554,9 +554,9 @@ 
 	public function _1_4_0_clean_hook_settings( $instance, $new_instance, $old_instance, $hook ) {
 
 		if ( $hook instanceof WordPoints_Post_Points_Hook ) {
-			unset( $instance['trash'], $instance['publish'] );
+			unset( $instance[ 'trash' ], $instance[ 'publish' ] );
 		} elseif ( $hook instanceof WordPoints_Comment_Points_Hook ) {
-			unset( $instance['approve'], $instance['disapprove'] );
+			unset( $instance[ 'approve' ], $instance[ 'disapprove' ] );
 		}
 
 		return $instance;
@@ -729,9 +729,9 @@ 
 
 			$hook_instances_indexed
 				[ $hook->points_type( $network_ . $number ) ]
-				[ $instance['post_type'] ]
-				[ $instance['points'] ]
-				[] = $number;
+				[ $instance[ 'post_type' ] ]
+				[ $instance[ 'points' ] ]
+				[ ] = $number;
 		}
 
 		foreach ( $hook_reverse_instances as $number => $instance ) {
@@ -742,12 +742,12 @@ 
 
 			// We use empty() instead of isset() because array_pop() below may leave
 			// us with an empty array as the value.
-			if ( empty( $hook_instances_indexed[ $points_type ][ $instance['post_type'] ][ $instance['points'] ] ) ) {
+			if ( empty( $hook_instances_indexed[ $points_type ][ $instance[ 'post_type' ] ][ $instance[ 'points' ] ] ) ) {
 				continue;
 			}
 
 			$comment_instance_number = array_pop(
-				$hook_instances_indexed[ $points_type ][ $instance['post_type'] ][ $instance['points'] ]
+				$hook_instances_indexed[ $points_type ][ $instance[ 'post_type' ] ][ $instance[ 'points' ] ]
 			);
 
 			// We need to unset this instance from the list of hook instances. It
@@ -768,7 +768,7 @@ 
 		if ( ! empty( $hook_instances ) ) {
 
 			foreach ( $hook_instances as $number => $instance ) {
-				$instance['auto_reverse'] = 0;
+				$instance[ 'auto_reverse' ] = 0;
 				$hook->update_callback( $instance, $number );
 			}
 
@@ -826,7 +826,7 @@ 
 		);
 
 		if ( $network_wide ) {
-			$query_args['blog_id'] = false;
+			$query_args[ 'blog_id' ] = false;
 		}
 
 		$query = new WordPoints_Points_Logs_Query( $query_args );
@@ -983,9 +983,9 @@ 
 				continue;
 			}
 
-			$logs_to_delete[] = $post_publish_reverse_log_id;
-			$logs_to_delete[] = $post_update_reverse_log_id;
-			$logs_to_delete[] = $post_update_log_id;
+			$logs_to_delete[ ] = $post_publish_reverse_log_id;
+			$logs_to_delete[ ] = $post_update_reverse_log_id;
+			$logs_to_delete[ ] = $post_update_log_id;
 
 			// Give the user their points back, as they were removed in error.
 			$this->_2_1_4_revert_log( $post_publish_reverse_log );
@@ -1025,7 +1025,7 @@ 
 			if ( count( $logs ) > 1 ) {
 
 				// The first one is the original, so keep it.
-				unset( $logs[0] );
+				unset( $logs[ 0 ] );
 
 				array_map( array( $this, '_2_1_4_revert_log' ), $logs );
 
@@ -1166,7 +1166,7 @@ 
 			);
 
 			if ( $hit_id ) {
-				$hits_to_delete[] = $hit_id;
+				$hits_to_delete[ ] = $hit_id;
 			}
 
 			wordpoints_points_log_delete_all_metadata( $log_id );
@@ -1201,7 +1201,7 @@ 
 
 		foreach ( $hit_ids as $hit_id ) {
 
-			$hit_ids[] = get_metadata(
+			$hit_ids[ ] = get_metadata(
 				'wordpoints_hook_hit'
 				, $hit_id
 				, 'hook_hit_id'
@@ -1286,8 +1286,8 @@ 
 				isset( $reversal_logs[ $log->id ] )
 				&& strtotime( $reversal_logs[ $log->id ]->date ) - strtotime( $log->date ) < 2
 			) {
-				$logs_to_delete[] = $log->id;
-				$logs_to_delete[] = $reversal_logs[ $log->id ]->id;
+				$logs_to_delete[ ] = $log->id;
+				$logs_to_delete[ ] = $reversal_logs[ $log->id ]->id;
 			}
 		}
 
@@ -1345,7 +1345,7 @@ 
 		$legacy_log_types = array();
 
 		foreach ( $post_types as $post_type ) {
-			$legacy_log_types[] = "points_legacy_post_publish\\{$post_type}";
+			$legacy_log_types[ ] = "points_legacy_post_publish\\{$post_type}";
 		}
 
 		$query = new WordPoints_Points_Logs_Query(
@@ -1368,7 +1368,7 @@ 
 				, true
 			);
 
-			$legacy_logs[ $post_id ][] = $legacy_log;
+			$legacy_logs[ $post_id ][ ] = $legacy_log;
 		}
 
 		return $legacy_logs;

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 +21 added lines, -21 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;
 	}
 
@@ -1094,13 +1094,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;
 			}
 		}
 
@@ -1134,9 +1134,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;
 
@@ -1185,17 +1185,17 @@ 
 			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;
 		}
 
 		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 );
 }
 
 /**
@@ -1226,7 +1226,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;
 	}
 
 	/**
@@ -1250,9 +1250,9 @@ 
 	<table class="wordpoints-points-top-users <?php echo esc_attr( implode( ' ', $extra_classes ) ); ?>">
 		<thead>
 			<tr>
-				<th scope="col"><?php echo esc_html( $column_headers['position'] ); ?></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[ 'position' ] ); ?></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>
@@ -1310,9 +1310,9 @@ 
 		</tbody>
 		<tfoot>
 		<tr>
-			<th scope="col"><?php echo esc_html( $column_headers['position'] ); ?></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[ 'position' ] ); ?></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>
@@ -1351,7 +1351,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;
 		}
 
@@ -1376,7 +1376,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/components/ranks/includes/class-wordpoints-rank-group.php

Doc Comments +1 added lines, -1 removed lines

@@ -402,7 +402,7 @@
 	 *
 	 * @param int $rank_id The ID of the rank to get the position of.
 	 *
-	 * @return int|false The rank's position, or false.
+	 * @return integer The rank's position, or false.
 	 */
 	public function get_rank_position( $rank_id ) {
 

Spacing +1 added lines, -1 removed lines

@@ -440,7 +440,7 @@
 
 		} elseif ( $count < $position ) {
 
-			$ranks[] = $rank_id;
+			$ranks[ ] = $rank_id;
 
 		} else {
 

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.
  */

Spacing +22 added lines, -22 removed lines

@@ -64,13 +64,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;
 	}
 
@@ -88,7 +88,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;
 	}
 
@@ -98,7 +98,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;
 	}
 
@@ -128,7 +128,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;
 	}
 
@@ -138,13 +138,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() ) {
@@ -304,7 +304,7 @@ 
 				return false;
 			}
 
-			$values[] = $request[ $value ];
+			$values[ ] = $request[ $value ];
 		}
 
 		$action_format = vsprintf( $action_format, $values );
@@ -346,10 +346,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'
 		);
 
@@ -750,16 +750,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>';
@@ -936,11 +936,11 @@ 
 	switch ( $cap ) {
 		case 'install_wordpoints_modules':
 			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;
 	}
@@ -1116,13 +1116,13 @@ 
 		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 ] );
 		default:
 			return false;
 	}

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;
 	}
 

src/components/points/admin/includes/functions.php

Indentation +3 added lines, -3 removed lines

@@ -246,9 +246,9 @@
 			}
 
 			$screen_options = '<p><a id="access-on" href="' . esc_url( wp_nonce_url( $url, 'wordpoints_points_hooks_accessiblity', 'wordpoints-accessiblity-nonce' ) ) . '&amp;accessibility-mode=on">'
-			                  . esc_html__( 'Enable accessibility mode', 'wordpoints' )
-			                  . '</a><a id="access-off" href="' . esc_url( wp_nonce_url( $url, 'wordpoints_points_hooks_accessiblity', 'wordpoints-accessiblity-nonce' ) ) . '&amp;accessibility-mode=off">'
-			                  . esc_html__( 'Disable accessibility mode', 'wordpoints' ) . "</a></p>\n";
+							  . esc_html__( 'Enable accessibility mode', 'wordpoints' )
+							  . '</a><a id="access-off" href="' . esc_url( wp_nonce_url( $url, 'wordpoints_points_hooks_accessiblity', 'wordpoints-accessiblity-nonce' ) ) . '&amp;accessibility-mode=off">'
+							  . esc_html__( 'Disable accessibility mode', 'wordpoints' ) . "</a></p>\n";
 			break;
 	}
 

Spacing +10 added lines, -10 removed lines

@@ -143,7 +143,7 @@ 
  */
 function wordpoints_points_admin_screen_hooks() {
 
-	if ( isset( $_GET['edithook'] ) || isset( $_POST['savehook'] ) || isset( $_POST['removehook'] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET[ 'edithook' ] ) || isset( $_POST[ 'savehook' ] ) || isset( $_POST[ 'removehook' ] ) ) { // WPCS: CSRF OK.
 
 		// - We're doing this without AJAX (JS).
 
@@ -206,7 +206,7 @@ 
  */
 function wordpoints_no_js_points_hooks_save() {
 
-	if ( ! isset( $_POST['savehook'] ) && ! isset( $_POST['removehook'] ) ) { // WPCS: CSRF OK.
+	if ( ! isset( $_POST[ 'savehook' ] ) && ! isset( $_POST[ 'removehook' ] ) ) { // WPCS: CSRF OK.
 		return;
 	}
 
@@ -284,7 +284,7 @@ 
  */
 function wordpoints_points_hook_description_form( $has_form, $instance, $hook ) {
 
-	$description = ( isset( $instance['_description'] ) ) ? $instance['_description'] : '';
+	$description = ( isset( $instance[ '_description' ] ) ) ? $instance[ '_description' ] : '';
 
 	?>
 
@@ -350,7 +350,7 @@ 
 			?>
 
 			<tr>
-				<th scope="row"><?php echo esc_html( $type['name'] ); ?></th>
+				<th scope="row"><?php echo esc_html( $type[ 'name' ] ); ?></th>
 				<td>
 					<input type="hidden" name="<?php echo esc_attr( "wordpoints_points_old-{$slug}" ); ?>" value="<?php echo esc_attr( $points ); ?>" />
 					<input type="number" name="<?php echo esc_attr( "wordpoints_points-{$slug}" ); ?>" value="<?php echo esc_attr( $points ); ?>" autocomplete="off" />
@@ -389,7 +389,7 @@ 
 		<tbody>
 		<?php foreach ( wordpoints_get_points_types() as $slug => $type ) : ?>
 			<tr>
-				<th scope="row" style="text-align: left;"><?php echo esc_html( $type['name'] ); ?></th>
+				<th scope="row" style="text-align: left;"><?php echo esc_html( $type[ 'name' ] ); ?></th>
 				<td style="text-align: right;"><?php wordpoints_display_points( $user->ID, $slug, 'profile_page' ); ?></td>
 			</tr>
 		<?php endforeach; ?>
@@ -419,7 +419,7 @@ 
 	}
 
 	if (
-		! isset( $_POST['wordpoints_points_set_nonce'], $_POST['wordpoints_set_reason'] )
+		! isset( $_POST[ 'wordpoints_points_set_nonce' ], $_POST[ 'wordpoints_set_reason' ] )
 		|| ! wordpoints_verify_nonce( 'wordpoints_points_set_nonce', 'wordpoints_points_set_profile', null, 'post' )
 	) {
 		return;
@@ -444,7 +444,7 @@ 
 				, 'profile_edit'
 				, array(
 					'user_id' => get_current_user_id(),
-					'reason' => sanitize_text_field( wp_unslash( $_POST['wordpoints_set_reason'] ) ),
+					'reason' => sanitize_text_field( wp_unslash( $_POST[ 'wordpoints_set_reason' ] ) ),
 				)
 			);
 		}
@@ -500,11 +500,11 @@ 
 function wordpoints_points_admin_settings_save() {
 
 	if (
-		isset( $_POST['default_points_type'] )
+		isset( $_POST[ 'default_points_type' ] )
 		&& wordpoints_verify_nonce( 'wordpoints_default_points_type_nonce', 'wordpoints_default_points_type', null, 'post' )
 	) {
 
-		$points_type = sanitize_key( $_POST['default_points_type'] );
+		$points_type = sanitize_key( $_POST[ 'default_points_type' ] );
 
 		if ( '-1' === $points_type ) {
 
@@ -527,7 +527,7 @@ 
 function wordpoints_points_admin_notices() {
 
 	if (
-		( ! isset( $_GET['page'] ) || 'wordpoints_points_types' !== $_GET['page'] ) // WPCS: CSRF OK.
+		( ! isset( $_GET[ 'page' ] ) || 'wordpoints_points_types' !== $_GET[ 'page' ] ) // WPCS: CSRF OK.
 		&& current_user_can( 'manage_wordpoints_points_types' )
 		&& ! wordpoints_get_points_types()
 	) {

Braces +9 added lines, -4 removed lines

@@ -143,7 +143,8 @@ 
  */
 function wordpoints_points_admin_screen_hooks() {
 
-	if ( isset( $_GET['edithook'] ) || isset( $_POST['savehook'] ) || isset( $_POST['removehook'] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['edithook'] ) || isset( $_POST['savehook'] ) || isset( $_POST['removehook'] ) ) {
+// WPCS: CSRF OK.
 
 		// - We're doing this without AJAX (JS).
 
@@ -206,7 +207,8 @@ 
  */
 function wordpoints_no_js_points_hooks_save() {
 
-	if ( ! isset( $_POST['savehook'] ) && ! isset( $_POST['removehook'] ) ) { // WPCS: CSRF OK.
+	if ( ! isset( $_POST['savehook'] ) && ! isset( $_POST['removehook'] ) ) {
+// WPCS: CSRF OK.
 		return;
 	}
 
@@ -290,9 +292,12 @@ 
 
 	<?php if ( $has_form ) : ?>
 		<hr />
-	<?php else : ?>
+	<?php else {
+	: ?>
 		<br />
-	<?php endif; ?>
+	<?php endif;
+}
+?>
 
 	<div class="hook-instance-description">
 		<label for="<?php $hook->the_field_id( '_description' ); ?>"><?php echo esc_html_x( 'Description (optional):', 'points hook', 'wordpoints' ); ?></label>