WordPoints / wordpoints

src/components/points/classes/updater/2/1/4/logs.php

Doc Comments +1 added lines

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

Spacing +13 added lines, -13 removed lines

@@ -50,8 +50,8 @@ 
 
 			$post_publish_log_id         = min( array_keys( $original_log_ids ) );
 			$post_update_log_id          = max( array_keys( $original_log_ids ) );
-			$post_publish_reverse_log_id = $original_log_ids[ $post_publish_log_id ];
-			$post_update_reverse_log_id  = $original_log_ids[ $post_update_log_id ];
+			$post_publish_reverse_log_id = $original_log_ids[$post_publish_log_id];
+			$post_update_reverse_log_id  = $original_log_ids[$post_update_log_id];
 
 			$query = new WordPoints_Points_Logs_Query(
 				array( 'id__in' => array( $post_publish_reverse_log_id ) )
@@ -105,18 +105,18 @@ 
 
 		foreach ( $post_ids as $post_id ) {
 
-			if ( ! isset( $legacy_logs[ $post_id ] ) ) {
+			if ( ! isset( $legacy_logs[$post_id] ) ) {
 				continue;
 			}
 
-			array_map( array( $this, 'revert_log' ), $legacy_logs[ $post_id ] );
+			array_map( array( $this, 'revert_log' ), $legacy_logs[$post_id] );
 
 			$logs_to_delete = array_merge(
 				$logs_to_delete
-				, wp_list_pluck( $legacy_logs[ $post_id ], 'id' )
+				, wp_list_pluck( $legacy_logs[$post_id], 'id' )
 			);
 
-			unset( $legacy_logs[ $post_id ] );
+			unset( $legacy_logs[$post_id] );
 		}
 
 		foreach ( $legacy_logs as $logs ) {
@@ -178,7 +178,7 @@ 
 		$event_slugs = array();
 
 		foreach ( $post_types as $slug ) {
-			$event_slugs[ "reverse-post_publish\\{$slug}" ] = true;
+			$event_slugs["reverse-post_publish\\{$slug}"] = true;
 		}
 
 		return $event_slugs;
@@ -234,7 +234,7 @@ 
 				, true
 			);
 
-			$original_log_ids[ $original_log_id ] = $log_id;
+			$original_log_ids[$original_log_id] = $log_id;
 		}
 
 		return $original_log_ids;
@@ -372,7 +372,7 @@ 
 				, true
 			);
 
-			$reversal_logs[ $original_log_id ] = $log;
+			$reversal_logs[$original_log_id] = $log;
 		}
 
 		foreach ( $other_logs as $index => $log ) {
@@ -381,11 +381,11 @@ 
 			// of its occurrence, it is almost certainly the result of another
 			// update, and it is just cluttering things up.
 			if (
-				isset( $reversal_logs[ $log->id ] )
-				&& strtotime( $reversal_logs[ $log->id ]->date ) - strtotime( $log->date ) < 2
+				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[] = $reversal_logs[$log->id]->id;
 			}
 		}
 
@@ -466,7 +466,7 @@ 
 				, true
 			);
 
-			$legacy_logs[ $post_id ][] = $legacy_log;
+			$legacy_logs[$post_id][] = $legacy_log;
 		}
 
 		return $legacy_logs;

src/components/ranks/classes/rank/group.php

Doc Comments +2 added lines, -2 removed lines

@@ -368,7 +368,7 @@ 
 	 *
 	 * @since 1.7.0
 	 *
-	 * @return int|false The ID of the rank, or false if not found.
+	 * @return integer The ID of the rank, or false if not found.
 	 */
 	public function get_base_rank() {
 
@@ -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 +16 added lines, -16 removed lines

@@ -83,8 +83,8 @@ 
 
 		if ( isset( $this->$key ) ) {
 			return $this->$key;
-		} elseif ( isset( $this->data[ $key ] ) ) {
-			return $this->data[ $key ];
+		} elseif ( isset( $this->data[$key] ) ) {
+			return $this->data[$key];
 		} else {
 			return null;
 		}
@@ -128,7 +128,7 @@ 
 			return false;
 		}
 
-		return isset( $this->types[ $type ] );
+		return isset( $this->types[$type] );
 	}
 
 	/**
@@ -146,11 +146,11 @@ 
 			return false;
 		}
 
-		if ( isset( $this->types[ $type ] ) ) {
+		if ( isset( $this->types[$type] ) ) {
 			return false;
 		}
 
-		$this->types[ $type ] = $type;
+		$this->types[$type] = $type;
 
 		return true;
 	}
@@ -176,11 +176,11 @@ 
 			return false;
 		}
 
-		if ( ! isset( $this->types[ $type ] ) ) {
+		if ( ! isset( $this->types[$type] ) ) {
 			return false;
 		}
 
-		unset( $this->types[ $type ] );
+		unset( $this->types[$type] );
 
 		return true;
 	}
@@ -250,7 +250,7 @@ 
 
 		$ranks = $this->get_ranks();
 
-		unset( $ranks[ $current_position ] );
+		unset( $ranks[$current_position] );
 
 		ksort( $ranks );
 
@@ -295,16 +295,16 @@ 
 
 		$ranks = $this->get_ranks();
 
-		unset( $ranks[ $position ] );
+		unset( $ranks[$position] );
 
 		if ( ! $this->save_ranks( $ranks ) ) {
 			return false;
 		}
 
 		// Assign the previous rank to users who have this rank.
-		if ( isset( $ranks[ $position - 1 ] ) ) {
+		if ( isset( $ranks[$position - 1] ) ) {
 
-			$this->move_users_from_rank_to_rank( $rank_id, $ranks[ $position - 1 ] );
+			$this->move_users_from_rank_to_rank( $rank_id, $ranks[$position - 1] );
 
 		} else {
 
@@ -316,7 +316,7 @@ 
 			);
 
 			$group_ranks = wp_cache_get( $this->slug, 'wordpoints_user_ranks' );
-			unset( $group_ranks[ $rank_id ] );
+			unset( $group_ranks[$rank_id] );
 			wp_cache_set( $this->slug, $group_ranks, 'wordpoints_user_ranks' );
 
 			unset( $group_ranks );
@@ -388,11 +388,11 @@ 
 
 		$ranks = $this->get_ranks();
 
-		if ( ! isset( $ranks[ $position ] ) ) {
+		if ( ! isset( $ranks[$position] ) ) {
 			return false;
 		}
 
-		return $ranks[ $position ];
+		return $ranks[$position];
 	}
 
 	/**
@@ -434,7 +434,7 @@ 
 
 		if ( $count === $position ) {
 
-			$ranks[ $position ] = $rank_id;
+			$ranks[$position] = $rank_id;
 
 		} elseif ( $count < $position ) {
 
@@ -473,7 +473,7 @@ 
 
 		$group_ranks = wp_cache_get( $this->slug, 'wordpoints_user_ranks' );
 
-		unset( $group_ranks[ $rank_from_id ], $group_ranks[ $rank_to_id ] );
+		unset( $group_ranks[$rank_from_id], $group_ranks[$rank_to_id] );
 
 		wp_cache_set( $this->slug, $group_ranks, 'wordpoints_user_ranks' );
 

src/components/points/admin/classes/screen/points/types.php

Braces +17 added lines, -8 removed lines

@@ -245,8 +245,11 @@ 
 					<p>
 						<?php if ( $slug ) : ?>
 							<?php esc_html_e( 'Changes to this points type&#8217;s settings will affect all sites on this network.', 'wordpoints' ); ?>
-						<?php else : ?>
-							<?php esc_html_e( 'The new points type will be global across all sites on this network.', 'wordpoints' ); ?>
+						<?php else {
+	: ?>
+							<?php esc_html_e( 'The new points type will be global across all sites on this network.', 'wordpoints' );
+}
+?>
 						<?php endif; ?>
 					</p>
 				</div>
@@ -655,15 +658,18 @@ 
 			return;
 		}
 
-		if ( isset( $_POST['save-points-type'] ) ) { // WPCS: CSRF OK
+		if ( isset( $_POST['save-points-type'] ) ) {
+// WPCS: CSRF OK
 
-			if ( ! empty( $_POST['add_new'] ) ) { // WPCS: CSRF OK
+			if ( ! empty( $_POST['add_new'] ) ) {
+// WPCS: CSRF OK
 				$this->add_points_type();
 			} else {
 				$this->update_points_type();
 			}
 
-		} elseif ( ! empty( $_POST['delete-points-type'] ) ) { // WPCS: CSRF OK
+		} elseif ( ! empty( $_POST['delete-points-type'] ) ) {
+// WPCS: CSRF OK
 
 			$this->delete_points_type();
 		}
@@ -680,19 +686,22 @@ 
 
 		$settings = array();
 
-		if ( isset( $_POST['points-name'] ) ) { // WPCS: CSRF OK
+		if ( isset( $_POST['points-name'] ) ) {
+// WPCS: CSRF OK
 			$settings['name'] = trim(
 				sanitize_text_field( wp_unslash( $_POST['points-name'] ) ) // WPCS: CSRF OK
 			);
 		}
 
-		if ( isset( $_POST['points-prefix'] ) ) { // WPCS: CSRF OK
+		if ( isset( $_POST['points-prefix'] ) ) {
+// WPCS: CSRF OK
 			$settings['prefix'] = ltrim(
 				sanitize_text_field( wp_unslash( $_POST['points-prefix'] ) ) // WPCS: CSRF OK
 			);
 		}
 
-		if ( isset( $_POST['points-suffix'] ) ) { // WPCS: CSRF OK
+		if ( isset( $_POST['points-suffix'] ) ) {
+// WPCS: CSRF OK
 			$settings['suffix'] = rtrim(
 				sanitize_text_field( wp_unslash( $_POST['points-suffix'] ) ) // WPCS: CSRF OK
 			);

Spacing +5 added lines, -5 removed lines

@@ -514,7 +514,7 @@ 
 
 			if (
 				! array_intersect_key(
-					$event_action_types[ $slug ]
+					$event_action_types[$slug]
 					, $reactor_action_types
 				)
 			) {
@@ -567,13 +567,13 @@ 
 
 		foreach ( $this->event_args->get_children( $event_slug ) as $slug => $arg ) {
 
-			$event_data['args'][ $slug ] = array(
+			$event_data['args'][$slug] = array(
 				'slug' => $slug,
 			);
 
 			if ( $arg instanceof WordPoints_Hook_ArgI ) {
-				$event_data['args'][ $slug ]['title']       = $arg->get_title();
-				$event_data['args'][ $slug ]['is_stateful'] = $arg->is_stateful();
+				$event_data['args'][$slug]['title']       = $arg->get_title();
+				$event_data['args'][$slug]['is_stateful'] = $arg->is_stateful();
 			}
 		}
 
@@ -628,7 +628,7 @@ 
 		$tabs = array();
 
 		foreach ( $points_types as $slug => $settings ) {
-			$tabs[ $slug ] = $settings['name'];
+			$tabs[$slug] = $settings['name'];
 		}
 
 		$tabs['add-new'] = __( 'Add New', 'wordpoints' );

src/components/points/admin/screens/hooks-no-js.php

Braces +11 added lines, -5 removed lines

@@ -7,7 +7,8 @@ 
  * @since 1.0.0
  */
 
-if ( ! isset( $_GET['edithook'] ) ) { // WPCS: CSRF OK.
+if ( ! isset( $_GET['edithook'] ) ) {
+// WPCS: CSRF OK.
 	return;
 }
 
@@ -15,7 +16,8 @@ 
 
 $points_types = wordpoints_get_points_types();
 
-if ( isset( $_GET['addnew'] ) ) { // WPCS: CSRF OK.
+if ( isset( $_GET['addnew'] ) ) {
+// WPCS: CSRF OK.
 
 	// - We are adding a new points hook.
 
@@ -33,7 +35,8 @@ 
 		return;
 	}
 
-	if ( isset( $_GET['base'], $_GET['num'] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['base'], $_GET['num'] ) ) {
+// WPCS: CSRF OK.
 
 		// Copy minimal info from an existing instance of this hook to a new instance.
 		$hook = WordPoints_Points_Hooks::get_handler_by_id_base(
@@ -132,8 +135,11 @@ 
 			<div class="hook-control-actions">
 				<?php if ( isset( $_GET['addnew'] ) ) : /* WPCS: CSRF OK. */ ?>
 					<a href="<?php echo esc_url( self_admin_url( 'admin.php?page=wordpoints_points_hooks' ) ); ?>" class="button alignleft"><?php esc_html_e( 'Cancel', 'wordpoints' ); ?></a>
-				<?php else : ?>
-					<?php submit_button( _x( 'Delete', 'points hook', 'wordpoints' ), 'button alignleft', 'removehook', false ); ?>
+				<?php else {
+	: ?>
+					<?php submit_button( _x( 'Delete', 'points hook', 'wordpoints' ), 'button alignleft', 'removehook', false );
+}
+?>
 				<?php endif; ?>
 
 				<?php submit_button( __( 'Save Hook', 'wordpoints' ), 'button-primary alignright', 'savehook', false ); ?>

src/components/ranks/admin/includes/ajax.php

Braces +8 added lines, -4 removed lines

@@ -305,7 +305,8 @@ 
 	 */
 	private function get_group() {
 
-		if ( ! isset( $_POST['group'] ) ) { // WPCS: CSRF OK.
+		if ( ! isset( $_POST['group'] ) ) {
+// WPCS: CSRF OK.
 			$this->unexpected_error( 'group' );
 		}
 
@@ -327,7 +328,8 @@ 
 	 */
 	private function get_rank() {
 
-		if ( ! isset( $_POST['id'] ) ) { // WPCS: CSRF OK.
+		if ( ! isset( $_POST['id'] ) ) {
+// WPCS: CSRF OK.
 			$this->unexpected_error( 'id' );
 		}
 
@@ -351,7 +353,8 @@ 
 
 		$name = '';
 
-		if ( ! empty( $_POST['name'] ) ) { // WPCS: CSRF OK.
+		if ( ! empty( $_POST['name'] ) ) {
+// WPCS: CSRF OK.
 			$name = sanitize_text_field( wp_unslash( $_POST['name'] ) ); // WPCS: CSRF OK.
 		}
 
@@ -376,7 +379,8 @@ 
 	 */
 	private function get_rank_type() {
 
-		if ( empty( $_POST['type'] ) ) { // WPCS: CSRF OK.
+		if ( empty( $_POST['type'] ) ) {
+// WPCS: CSRF OK.
 			$this->unexpected_error( 'type' );
 		}
 

Spacing +2 added lines, -2 removed lines

@@ -67,7 +67,7 @@ 
 		$ranks       = array();
 
 		foreach ( $rank_groups as $group ) {
-			$ranks[ $group->slug ] = self::prepare_group_ranks( $group );
+			$ranks[$group->slug] = self::prepare_group_ranks( $group );
 		}
 
 		return $ranks;
@@ -503,7 +503,7 @@ 
 		$rank_type = WordPoints_Rank_Types::get_type( $rank->type );
 
 		foreach ( $rank_type->get_meta_fields() as $field => $data ) {
-			$prepared_rank[ $field ] = $rank->{$field};
+			$prepared_rank[$field] = $rank->{$field};
 		}
 
 		return $prepared_rank;

src/components/ranks/classes/rank/type.php

Braces +5 added lines, -2 removed lines

@@ -222,8 +222,11 @@
 						name="<?php echo esc_attr( $name ); ?>"
 						<?php if ( $args['placeholders'] && 'hidden' !== $field['type'] ) : ?>
 						value="<% if ( typeof <?php echo preg_replace( '/[^a-z0-9_]/i', '', $name ); /* WPCS XSS OK. */ ?> !== "undefined" ) { print( <?php echo preg_replace( '/[^a-z0-9_]/i', '', $name ); /* WPCS XSS OK. */ ?> ); } %>"
-						<?php else : ?>
-						value="<?php echo esc_attr( $value ); ?>"
+						<?php else {
+	: ?>
+						value="<?php echo esc_attr( $value );
+}
+?>"
 						<?php endif; ?>
 						class="widefat"
 					/>

Spacing +4 added lines, -4 removed lines

@@ -199,8 +199,8 @@ 
 				$value = $field['default'];
 
 				// If the value is set use that instead.
-				if ( isset( $meta[ $name ] ) ) {
-					$value = $meta[ $name ];
+				if ( isset( $meta[$name] ) ) {
+					$value = $meta[$name];
 				}
 			}
 
@@ -332,7 +332,7 @@ 
 					continue;
 				}
 
-				$user_ranks[ $user_id ] = $new_rank->ID;
+				$user_ranks[$user_id] = $new_rank->ID;
 			}
 		}
 
@@ -425,7 +425,7 @@ 
 					continue;
 				}
 
-				$user_ranks[ $user_id ] = $new_rank->ID;
+				$user_ranks[$user_id] = $new_rank->ID;
 			}
 		}
 

src/admin/includes/functions.php

Braces +22 added lines, -10 removed lines

@@ -925,7 +925,8 @@ 
 
 	$tab = '';
 
-	if ( isset( $_GET['tab'] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['tab'] ) ) {
+// WPCS: CSRF OK.
 
 		$tab = sanitize_key( $_GET['tab'] ); // WPCS: CSRF OK.
 	}
@@ -964,7 +965,8 @@ 
 
 	$page = '';
 
-	if ( isset( $_GET['page'] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['page'] ) ) {
+// WPCS: CSRF OK.
 		$page = sanitize_key( $_GET['page'] ); // WPCS: CSRF OK.
 	}
 
@@ -1360,9 +1362,12 @@ 
 							>
 								<?php esc_html_e( 'Update now', 'wordpoints' ); ?>
 							</a>
-						<?php else : ?>
+						<?php else {
+	: ?>
 							<em>
-								<?php esc_html_e( 'Automatic update is unavailable for this extension.', 'wordpoints' ); ?>
+								<?php esc_html_e( 'Automatic update is unavailable for this extension.', 'wordpoints' );
+}
+?>
 							</em>
 						<?php endif; ?>
 					<?php endif; ?>
@@ -1711,8 +1716,11 @@ 
 									?>
 									<a href="<?php echo esc_url( $license->get_renewal_url() ); ?>" aria-label="<?php echo esc_attr( sprintf( $aria_label, $extension['name'] ) ); ?>"><?php esc_html_e( 'Renew License', 'wordpoints' ); ?></a>
 								<?php endif; ?>
-							<?php else : ?>
-								<?php esc_html_e( 'This license key is expired.', 'wordpoints' ); ?>
+							<?php else {
+	: ?>
+								<?php esc_html_e( 'This license key is expired.', 'wordpoints' );
+}
+?>
 							<?php endif; ?>
 						<?php endif; ?>
 					<?php endif; ?>
@@ -1738,7 +1746,8 @@ 
 		wp_die( esc_html__( 'Sorry, you are not allowed to update WordPoints extensions for this site.', 'wordpoints' ), 403 );
 	}
 
-	if ( empty( $_GET['extension'] ) ) { // WPCS: CSRF OK.
+	if ( empty( $_GET['extension'] ) ) {
+// WPCS: CSRF OK.
 		wp_die( esc_html__( 'No extension supplied.', 'wordpoints' ), 200 );
 	}
 
@@ -2265,7 +2274,8 @@ 
  */
 function wordpoints_admin_ajax_breaking_module_check() {
 
-	if ( ! isset( $_GET['wordpoints_module_check'] ) ) { // WPCS: CSRF OK.
+	if ( ! isset( $_GET['wordpoints_module_check'] ) ) {
+// WPCS: CSRF OK.
 		wp_die( '', 400 );
 	}
 
@@ -2275,7 +2285,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.
 		wp_die( '', 403 );
 	}
 
@@ -2571,7 +2582,8 @@ 
 
 	// Don't show them on the extensions screen, because they would be shown before
 	// license activation notices, etc.
-	if ( isset( $_GET['page'] ) && 'wordpoints_extensions' === $_GET['page'] ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['page'] ) && 'wordpoints_extensions' === $_GET['page'] ) {
+// WPCS: CSRF OK.
 		return;
 	}
 

Spacing +34 added lines, -34 removed lines

@@ -641,7 +641,7 @@ 
 	foreach ( $objects as $slug => $object ) {
 
 		if ( $object instanceof WordPoints_Hook_UI_Script_Data_ProviderI ) {
-			$data[ $slug ] = $object->get_ui_script_data();
+			$data[$slug] = $object->get_ui_script_data();
 		}
 
 		if ( wp_script_is( "wordpoints-hooks-{$type}-{$slug}", 'registered' ) ) {
@@ -676,21 +676,21 @@ 
 		/** @var WordPoints_EntityishI $child */
 		foreach ( $entity_children->get_children( $slug ) as $child_slug => $child ) {
 
-			$child_data[ $child_slug ] = array(
+			$child_data[$child_slug] = array(
 				'slug'  => $child_slug,
 				'title' => $child->get_title(),
 			);
 
 			if ( $child instanceof WordPoints_Entity_Attr ) {
 
-				$child_data[ $child_slug ]['_type']     = 'attr';
-				$child_data[ $child_slug ]['data_type'] = $child->get_data_type();
+				$child_data[$child_slug]['_type']     = 'attr';
+				$child_data[$child_slug]['data_type'] = $child->get_data_type();
 
 			} elseif ( $child instanceof WordPoints_Entity_Relationship ) {
 
-				$child_data[ $child_slug ]['_type']     = 'relationship';
-				$child_data[ $child_slug ]['primary']   = $child->get_primary_entity_slug();
-				$child_data[ $child_slug ]['secondary'] = $child->get_related_entity_slug();
+				$child_data[$child_slug]['_type']     = 'relationship';
+				$child_data[$child_slug]['primary']   = $child->get_primary_entity_slug();
+				$child_data[$child_slug]['secondary'] = $child->get_related_entity_slug();
 			}
 
 			/**
@@ -701,14 +701,14 @@ 
 			 * @param array                $data  The data for the entity child.
 			 * @param WordPoints_Entityish $child The child's object.
 			 */
-			$child_data[ $child_slug ] = apply_filters(
+			$child_data[$child_slug] = apply_filters(
 				'wordpoints_hooks_ui_data_entity_child'
-				, $child_data[ $child_slug ]
+				, $child_data[$child_slug]
 				, $child
 			);
 		}
 
-		$entities_data[ $slug ] = array(
+		$entities_data[$slug] = array(
 			'slug'     => $slug,
 			'title'    => $entity->get_title(),
 			'children' => $child_data,
@@ -729,7 +729,7 @@ 
 				}
 			}
 
-			$entities_data[ $slug ]['values'] = $values;
+			$entities_data[$slug]['values'] = $values;
 		}
 
 		/**
@@ -738,9 +738,9 @@ 
 		 * @param array             $data   The data for the entity.
 		 * @param WordPoints_Entity $entity The entity object.
 		 */
-		$entities_data[ $slug ] = apply_filters(
+		$entities_data[$slug] = apply_filters(
 			'wordpoints_hooks_ui_data_entity'
-			, $entities_data[ $slug ]
+			, $entities_data[$slug]
 			, $entity
 		);
 
@@ -773,7 +773,7 @@ 
 	// this, so that we have the action types indexed by event slug.
 	foreach ( $event_index as $action_type => $events ) {
 		foreach ( $events as $event => $unused ) {
-			$event_action_types[ $event ][ $action_type ] = true;
+			$event_action_types[$event][$action_type] = true;
 		}
 	}
 
@@ -930,7 +930,7 @@ 
 		$tab = sanitize_key( $_GET['tab'] ); // WPCS: CSRF OK.
 	}
 
-	if ( isset( $tabs ) && ! isset( $tabs[ $tab ] ) ) {
+	if ( isset( $tabs ) && ! isset( $tabs[$tab] ) ) {
 
 		reset( $tabs );
 		$tab = key( $tabs );
@@ -957,7 +957,7 @@ 
 	if ( $show_heading ) {
 
 		// translators: Current tab name.
-		echo '<h1>', esc_html( sprintf( __( 'WordPoints — %s', 'wordpoints' ), $tabs[ $current ] ) ), '</h1>';
+		echo '<h1>', esc_html( sprintf( __( 'WordPoints — %s', 'wordpoints' ), $tabs[$current] ) ), '</h1>';
 	}
 
 	echo '<h2 class="nav-tab-wrapper">';
@@ -1431,20 +1431,20 @@ 
 
 		$url = sanitize_title_with_dashes( $server->get_slug() );
 
-		if ( ! isset( $_POST[ "license_key-{$url}-{$extension['ID']}" ] ) ) {
+		if ( ! isset( $_POST["license_key-{$url}-{$extension['ID']}"] ) ) {
 			continue;
 		}
 
 		$license_key = sanitize_key(
-			$_POST[ "license_key-{$url}-{$extension['ID']}" ]
+			$_POST["license_key-{$url}-{$extension['ID']}"]
 		);
 
 		$license = $api->get_extension_license_object( $extension_data, $license_key );
 
 		if (
 			isset(
-				$_POST[ "activate-license-{$extension['ID']}" ]
-				, $_POST[ "wordpoints_activate_license_key-{$extension['ID']}" ]
+				$_POST["activate-license-{$extension['ID']}"]
+				, $_POST["wordpoints_activate_license_key-{$extension['ID']}"]
 			)
 			&& wordpoints_verify_nonce(
 				"wordpoints_activate_license_key-{$extension['ID']}"
@@ -1489,8 +1489,8 @@ 
 
 		} elseif (
 			isset(
-				$_POST[ "deactivate-license-{$extension['ID']}" ]
-				, $_POST[ "wordpoints_deactivate_license_key-{$extension['ID']}" ]
+				$_POST["deactivate-license-{$extension['ID']}"]
+				, $_POST["wordpoints_deactivate_license_key-{$extension['ID']}"]
 			)
 			&& wordpoints_verify_nonce(
 				"wordpoints_deactivate_license_key-{$extension['ID']}"
@@ -1746,11 +1746,11 @@ 
 
 	$extensions = wordpoints_get_modules();
 
-	if ( ! isset( $extensions[ $extension_file ] ) ) {
+	if ( ! isset( $extensions[$extension_file] ) ) {
 		wp_die( esc_html__( 'That extension does not exist.', 'wordpoints' ), 200 );
 	}
 
-	$server = wordpoints_get_server_for_extension( $extensions[ $extension_file ] );
+	$server = wordpoints_get_server_for_extension( $extensions[$extension_file] );
 
 	if ( ! $server ) {
 		wp_die( esc_html__( 'There is no server specified for this extension.', 'wordpoints' ), 200 );
@@ -1763,7 +1763,7 @@ 
 	}
 
 	$extension_data = new WordPoints_Extension_Server_API_Extension_Data(
-		$extensions[ $extension_file ]['ID']
+		$extensions[$extension_file]['ID']
 		, $server
 	);
 
@@ -2140,7 +2140,7 @@ 
 
 			foreach ( $merged_extensions as $i => $extension ) {
 				if ( true !== wordpoints_validate_module( $extension ) ) {
-					unset( $merged_extensions[ $i ] );
+					unset( $merged_extensions[$i] );
 				}
 			}
 
@@ -2318,11 +2318,11 @@ 
 	// this transient.
 	$updates = get_site_transient( 'update_plugins' );
 
-	if ( ! isset( $updates->response[ $plugin_basename ] ) ) {
+	if ( ! isset( $updates->response[$plugin_basename] ) ) {
 		return false;
 	}
 
-	if ( ! isset( $updates->response[ $plugin_basename ]->wordpoints_required_php ) ) {
+	if ( ! isset( $updates->response[$plugin_basename]->wordpoints_required_php ) ) {
 
 		/**
 		 * The plugin install functions.
@@ -2357,13 +2357,13 @@ 
 			$version = $matches[1];
 		}
 
-		$updates->response[ $plugin_basename ]->wordpoints_required_php = $version;
+		$updates->response[$plugin_basename]->wordpoints_required_php = $version;
 
 		set_site_transient( 'update_plugins', $updates );
 
 	} // End if ( PHP requirements not in cache ).
 
-	return $updates->response[ $plugin_basename ]->wordpoints_required_php;
+	return $updates->response[$plugin_basename]->wordpoints_required_php;
 }
 
 /**
@@ -2445,11 +2445,11 @@ 
 	// First check that there is actually an update available.
 	$updates = get_site_transient( 'update_plugins' );
 
-	if ( ! isset( $updates->response[ $file ] ) ) {
+	if ( ! isset( $updates->response[$file] ) ) {
 		return;
 	}
 
-	$response = $updates->response[ $file ];
+	$response = $updates->response[$file];
 
 	$wp_list_table = _get_list_table( 'WP_Plugins_List_Table' );
 
@@ -2553,8 +2553,8 @@ 
 
 	$plugin_basename = plugin_basename( WORDPOINTS_DIR . '/wordpoints.php' );
 
-	if ( isset( $data->response[ $plugin_basename ] ) ) {
-		unset( $data->response[ $plugin_basename ] );
+	if ( isset( $data->response[$plugin_basename] ) ) {
+		unset( $data->response[$plugin_basename] );
 	}
 
 	return $data;

src/classes/db/query.php

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -487,7 +487,7 @@ 
 
 		$fields = implode( ', ', array_map( 'wordpoints_escape_mysql_identifier', $fields ) );
 
-		$this->select = "SELECT {$fields}";
+		$this->select = "select {$fields}";
 	}
 
 	/**
@@ -777,7 +777,7 @@ 
 		}
 
 		if ( $this->args['limit'] > 0 && $this->args['offset'] >= 0 ) {
-			$this->limit = "LIMIT {$this->args['offset']}, {$this->args['limit']}";
+			$this->limit = "limit {$this->args['offset']}, {$this->args['limit']}";
 		}
 	}
 

Spacing +31 added lines, -31 removed lines

@@ -243,7 +243,7 @@ 
 		}
 
 		foreach ( $this->deprecated_args as $arg => $data ) {
-			if ( isset( $args[ $arg ] ) ) {
+			if ( isset( $args[$arg] ) ) {
 
 				_deprecated_argument(
 					esc_html( "{$data['class']}::__construct" )
@@ -251,9 +251,9 @@ 
 					, esc_html( "{$arg} is deprecated, use {$data['replacement']} instead" )
 				);
 
-				$args[ $data['replacement'] ] = $args[ $arg ];
+				$args[$data['replacement']] = $args[$arg];
 
-				unset( $args[ $arg ] );
+				unset( $args[$arg] );
 			}
 		}
 
@@ -271,19 +271,19 @@ 
 	 */
 	public function get_arg( $arg ) {
 
-		if ( isset( $this->deprecated_args[ $arg ] ) ) {
+		if ( isset( $this->deprecated_args[$arg] ) ) {
 
 			_deprecated_argument(
-				esc_html( "{$this->deprecated_args[ $arg ]['class']}::get_arg" )
-				, esc_html( $this->deprecated_args[ $arg ]['version'] )
-				, esc_html( "{$arg} is deprecated, use {$this->deprecated_args[ $arg ]['replacement']} instead" )
+				esc_html( "{$this->deprecated_args[$arg]['class']}::get_arg" )
+				, esc_html( $this->deprecated_args[$arg]['version'] )
+				, esc_html( "{$arg} is deprecated, use {$this->deprecated_args[$arg]['replacement']} instead" )
 			);
 
-			$arg = $this->deprecated_args[ $arg ]['replacement'];
+			$arg = $this->deprecated_args[$arg]['replacement'];
 		}
 
-		if ( isset( $this->args[ $arg ] ) ) {
-			return $this->args[ $arg ];
+		if ( isset( $this->args[$arg] ) ) {
+			return $this->args[$arg];
 		} else {
 			return null;
 		}
@@ -304,7 +304,7 @@ 
 	public function set_args( array $args ) {
 
 		foreach ( $this->deprecated_args as $arg => $data ) {
-			if ( isset( $args[ $arg ] ) ) {
+			if ( isset( $args[$arg] ) ) {
 
 				_deprecated_argument(
 					esc_html( "{$data['class']}::set_args" )
@@ -312,9 +312,9 @@ 
 					, esc_html( "{$arg} is deprecated, use {$data['replacement']} instead" )
 				);
 
-				$args[ $data['replacement'] ] = $args[ $arg ];
+				$args[$data['replacement']] = $args[$arg];
 
-				unset( $args[ $arg ] );
+				unset( $args[$arg] );
 			}
 		}
 
@@ -531,7 +531,7 @@ 
 			$value = $this->validate_value( $value, $validators );
 
 			if ( false === $value ) {
-				unset( $values[ $index ] );
+				unset( $values[$index] );
 			}
 		}
 
@@ -595,11 +595,11 @@ 
 	protected function prepare_column_where( $column, $data ) {
 
 		// If a single value has been supplied for the column, it takes precedence.
-		if ( isset( $this->args[ $column ] ) ) {
+		if ( isset( $this->args[$column] ) ) {
 			$this->prepare_column( $column, $data );
-		} elseif ( isset( $this->args[ "{$column}__in" ] ) ) {
+		} elseif ( isset( $this->args["{$column}__in"] ) ) {
 			$this->prepare_column__in( $column, $data );
-		} elseif ( isset( $this->args[ "{$column}__not_in" ] ) ) {
+		} elseif ( isset( $this->args["{$column}__not_in"] ) ) {
 			$this->prepare_column__in( $column, $data, 'NOT IN' );
 		}
 	}
@@ -618,13 +618,13 @@ 
 
 		if (
 			isset( $data['values'] )
-			&& ! in_array( $this->args[ $column ], $data['values'], true )
+			&& ! in_array( $this->args[$column], $data['values'], true )
 		) {
 			return;
 		}
 
 		$value = $this->validate_value(
-			$this->args[ $column ]
+			$this->args[$column]
 			, $this->get_validators_for_column( $data )
 		);
 
@@ -665,10 +665,10 @@ 
 		$comparator = '=';
 
 		if (
-			isset( $this->args[ "{$column}__compare" ] )
-			&& in_array( $this->args[ "{$column}__compare" ], $comparisons, true )
+			isset( $this->args["{$column}__compare"] )
+			&& in_array( $this->args["{$column}__compare"], $comparisons, true )
 		) {
-			$comparator = $this->args[ "{$column}__compare" ];
+			$comparator = $this->args["{$column}__compare"];
 		}
 
 		return $comparator;
@@ -687,11 +687,11 @@ 
 
 		$key = "{$column}__" . strtolower( str_replace( ' ', '_', $type ) );
 
-		if ( empty( $this->args[ $key ] ) || ! is_array( $this->args[ $key ] ) ) {
+		if ( empty( $this->args[$key] ) || ! is_array( $this->args[$key] ) ) {
 			return;
 		}
 
-		$values = $this->args[ $key ];
+		$values = $this->args[$key];
 
 		if ( isset( $data['values'] ) ) {
 			$values = array_intersect( $values, $data['values'] );
@@ -758,9 +758,9 @@ 
 		foreach ( array( 'limit', 'offset' ) as $key ) {
 
 			// Save a backup of the arg value since wordpoints_int() is by reference.
-			$arg = $this->args[ $key ];
+			$arg = $this->args[$key];
 
-			if ( false === wordpoints_int( $this->args[ $key ] ) ) {
+			if ( false === wordpoints_int( $this->args[$key] ) ) {
 
 				_doing_it_wrong(
 					__METHOD__
@@ -772,7 +772,7 @@ 
 					, '1.0.0'
 				);
 
-				$this->args[ $key ] = 0;
+				$this->args[$key] = 0;
 			}
 		}
 
@@ -820,7 +820,7 @@ 
 				$order_by = "{$meta_table_name}.meta_value";
 			}
 
-		} elseif ( isset( $this->columns[ $order_by ] ) ) {
+		} elseif ( isset( $this->columns[$order_by] ) ) {
 
 			$order_by = wordpoints_escape_mysql_identifier( $order_by );
 
@@ -843,15 +843,15 @@ 
 	protected function prepare_date_where( $column ) {
 
 		if (
-			empty( $this->args[ "{$column}_query" ] )
-			|| ! is_array( $this->args[ "{$column}_query" ] )
+			empty( $this->args["{$column}_query"] )
+			|| ! is_array( $this->args["{$column}_query"] )
 		) {
 			return;
 		}
 
 		add_filter( 'date_query_valid_columns', array( $this, 'date_query_valid_columns_filter' ) );
 
-		$date_query = new WP_Date_Query( $this->args[ "{$column}_query" ], $column );
+		$date_query = new WP_Date_Query( $this->args["{$column}_query"], $column );
 		$date_query = $date_query->get_sql();
 
 		if ( ! empty( $date_query ) ) {

src/includes/filters.php

Braces +2 added lines, -1 removed lines

@@ -39,7 +39,8 @@
 add_action( 'wordpoints_extensions_loaded', 'wordpoints_installables_maybe_update', 5 );
 add_action( 'wpmu_new_blog', 'wordpoints_installables_install_on_new_site' );
 
-if ( isset( $_GET['wordpoints_module_check'], $_GET['check_module'] ) ) { // WPCS: CSRF OK.
+if ( isset( $_GET['wordpoints_module_check'], $_GET['check_module'] ) ) {
+// WPCS: CSRF OK.
 
 	add_action( 'shutdown', 'wordpoints_maintenance_shutdown_print_rand_str' );