WordPoints / wordpoints

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 +35 added lines, -35 removed lines

@@ -173,7 +173,7 @@ 
 		'wordpoints_modules'            => true,
 	);
 
-	if ( $plugin_page && isset( $hidden_submenus[ $plugin_page ] ) ) {
+	if ( $plugin_page && isset( $hidden_submenus[$plugin_page] ) ) {
 		$submenu_file = 'wordpoints_extensions';
 	}
 
@@ -680,7 +680,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' ) ) {
@@ -715,21 +715,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();
 			}
 
 			/**
@@ -740,14 +740,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,
@@ -768,7 +768,7 @@ 
 				}
 			}
 
-			$entities_data[ $slug ]['values'] = $values;
+			$entities_data[$slug]['values'] = $values;
 		}
 
 		/**
@@ -777,9 +777,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
 		);
 
@@ -812,7 +812,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;
 		}
 	}
 
@@ -969,7 +969,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 );
@@ -996,7 +996,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">';
@@ -1470,20 +1470,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']}"
@@ -1529,8 +1529,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']}"
@@ -1786,11 +1786,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 );
@@ -1803,7 +1803,7 @@ 
 	}
 
 	$extension_data = new WordPoints_Extension_Server_API_Extension_Data(
-		$extensions[ $extension_file ]['ID']
+		$extensions[$extension_file]['ID']
 		, $server
 	);
 
@@ -2180,7 +2180,7 @@ 
 
 			foreach ( $merged_extensions as $i => $extension ) {
 				if ( true !== wordpoints_validate_module( $extension ) ) {
-					unset( $merged_extensions[ $i ] );
+					unset( $merged_extensions[$i] );
 				}
 			}
 
@@ -2358,11 +2358,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.
@@ -2397,13 +2397,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;
 }
 
 /**
@@ -2485,11 +2485,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' );
 
@@ -2618,8 +2618,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' );
 

src/admin/classes/list/table/extensions.php

Braces +11 added lines, -5 removed lines

@@ -76,11 +76,13 @@ 
 		 */
 		$module_statuses = apply_filters( 'wordpoints_module_statuses', $module_statuses );
 
-		if ( isset( $_REQUEST['module_status'] ) && in_array( wp_unslash( $_REQUEST['module_status'] ), $module_statuses, true ) ) { // WPCS: CSRF OK.
+		if ( isset( $_REQUEST['module_status'] ) && in_array( wp_unslash( $_REQUEST['module_status'] ), $module_statuses, true ) ) {
+// WPCS: CSRF OK.
 			$status = sanitize_key( $_REQUEST['module_status'] ); // WPCS: CSRF OK.
 		}
 
-		if ( isset( $_REQUEST['s'] ) ) { // WPCS: CSRF OK.
+		if ( isset( $_REQUEST['s'] ) ) {
+// WPCS: CSRF OK.
 			$_SERVER['REQUEST_URI'] = add_query_arg( 's', sanitize_text_field( wp_unslash( $_REQUEST['s'] ) ) ); // WPCS: CSRF OK.
 		}
 
@@ -342,7 +344,8 @@ 
 
 		static $term;
 
-		if ( is_null( $term ) && isset( $_REQUEST['s'] ) ) { // WPCS: CSRF OK.
+		if ( is_null( $term ) && isset( $_REQUEST['s'] ) ) {
+// WPCS: CSRF OK.
 			$term = sanitize_text_field( wp_unslash( $_REQUEST['s'] ) ); // WPCS: CSRF OK.
 		}
 
@@ -802,9 +805,12 @@ 
 				<p>
 					<?php if ( ! empty( $module_data['description'] ) ) : ?>
 						<?php echo wp_kses( $module_data['description'], 'wordpoints_module_description' ); ?>
-					<?php else : ?>
+					<?php else {
+	: ?>
 						&nbsp;
-					<?php endif; ?>
+					<?php endif;
+}
+?>
 				</p>
 			</div>
 			<div class="<?php echo esc_attr( $class ); ?> second module-version-author-uri">

Spacing +19 added lines, -19 removed lines

@@ -173,7 +173,7 @@ 
 			foreach ( $recently_activated as $key => $time ) {
 
 				if ( $time + WEEK_IN_SECONDS < time() ) {
-					unset( $recently_activated[ $key ] );
+					unset( $recently_activated[$key] );
 				}
 			}
 
@@ -212,9 +212,9 @@ 
 			) {
 
 				if ( $show_network_active ) {
-					$modules['inactive'][ $module_file ] = $module_data; // WPCS: prefix OK.
+					$modules['inactive'][$module_file] = $module_data; // WPCS: prefix OK.
 				} else {
-					unset( $modules['all'][ $module_file ] );
+					unset( $modules['all'][$module_file] );
 				}
 
 			} elseif (
@@ -223,9 +223,9 @@ 
 			) {
 
 				if ( $show_network_active ) {
-					$modules['active'][ $module_file ] = $module_data; // WPCS: prefix OK.
+					$modules['active'][$module_file] = $module_data; // WPCS: prefix OK.
 				} else {
-					unset( $modules['all'][ $module_file ] );
+					unset( $modules['all'][$module_file] );
 				}
 
 			} elseif (
@@ -238,16 +238,16 @@ 
 				)
 			) {
 
-				$modules['active'][ $module_file ] = $module_data; // WPCS: prefix OK.
+				$modules['active'][$module_file] = $module_data; // WPCS: prefix OK.
 
 			} else {
 
 				// Was the module recently activated?
-				if ( ! $this->screen->in_admin( 'network' ) && isset( $recently_activated[ $module_file ] ) ) {
-					$modules['recently_activated'][ $module_file ] = $module_data; // WPCS: prefix OK.
+				if ( ! $this->screen->in_admin( 'network' ) && isset( $recently_activated[$module_file] ) ) {
+					$modules['recently_activated'][$module_file] = $module_data; // WPCS: prefix OK.
 				}
 
-				$modules['inactive'][ $module_file ] = $module_data; // WPCS: prefix OK.
+				$modules['inactive'][$module_file] = $module_data; // WPCS: prefix OK.
 
 			} // End if ().
 
@@ -256,7 +256,7 @@ 
 			 * for modules that have an available update.
 			 */
 			if ( $user_can_update && $updates->has_update( $module_file ) ) {
-				$modules['upgrade'][ $module_file ] = $modules['all'][ $module_file ]; // WPCS: prefix OK.
+				$modules['upgrade'][$module_file] = $modules['all'][$module_file]; // WPCS: prefix OK.
 			}
 
 		} // End foreach ( modules ).
@@ -289,21 +289,21 @@ 
 		$totals = array();
 
 		foreach ( $modules as $type => $list ) {
-			$totals[ $type ] = count( $list );
+			$totals[$type] = count( $list );
 		}
 
-		if ( empty( $modules[ $status ] ) && ! in_array( $status, array( 'all', 'search' ), true ) ) {
+		if ( empty( $modules[$status] ) && ! in_array( $status, array( 'all', 'search' ), true ) ) {
 			$status = 'all';
 		}
 
 		$this->items = array();
 
-		foreach ( $modules[ $status ] as $module_file => $module_data ) {
+		foreach ( $modules[$status] as $module_file => $module_data ) {
 
-			$this->items[ $module_file ] = wordpoints_get_module_data( wordpoints_extensions_dir() . '/' . $module_file, false );
+			$this->items[$module_file] = wordpoints_get_module_data( wordpoints_extensions_dir() . '/' . $module_file, false );
 		}
 
-		$total_this_page = $totals[ $status ];
+		$total_this_page = $totals[$status];
 
 		if ( ! $orderby ) {
 			$orderby = 'name';
@@ -372,8 +372,8 @@ 
 
 		global $orderby, $order;
 
-		$a = $module_a[ $orderby ];
-		$b = $module_b[ $orderby ];
+		$a = $module_a[$orderby];
+		$b = $module_b[$orderby];
 
 		if ( $a === $b ) {
 			return 0;
@@ -480,7 +480,7 @@ 
 				 */
 				$text = apply_filters( "wordpoints_modules_status_link_text-{$type}", $text, $count );
 
-				$status_links[ $type ] = sprintf(
+				$status_links[$type] = sprintf(
 					"<a href='%s' %s>%s</a>"
 					, esc_url( add_query_arg( 'module_status', $type, self_admin_url( 'admin.php?page=wordpoints_extensions' ) ) )
 					, ( $type === $status ) ? ' class="current"' : ''
@@ -700,7 +700,7 @@ 
 		$module_data['extra']['restricted_network_active'] = $restricted_network_active;
 		$module_data['extra']['restricted_network_only']   = $restricted_network_only;
 
-		list( $columns, $hidden, , $primary ) = $this->get_column_info();
+		list( $columns, $hidden,, $primary ) = $this->get_column_info();
 
 		foreach ( $columns as $column_name => $column_display_name ) {
 

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

Spacing +8 added lines, -8 removed lines

@@ -39,8 +39,8 @@ 
 	$points_types_hooks = WordPoints_Points_Hooks::get_defaults();
 }
 
-if ( isset( $points_types_hooks[ $points_type_id ] ) ) {
-	$points_type_hooks = $points_types_hooks[ $points_type_id ];
+if ( isset( $points_types_hooks[$points_type_id] ) ) {
+	$points_type_hooks = $points_types_hooks[$points_type_id];
 } else {
 	$points_type_hooks = array();
 }
@@ -59,7 +59,7 @@ 
 	}
 
 	// Remove the hook from this points type.
-	$points_types_hooks[ $points_type_id ] = array_diff( $points_type_hooks, array( $hook_id ) );
+	$points_types_hooks[$points_type_id] = array_diff( $points_type_hooks, array( $hook_id ) );
 
 	$hook->delete_callback( $hook_id );
 
@@ -83,8 +83,8 @@ 
 
 	} else {
 
-		if ( isset( $_POST[ 'hook-' . $id_base ] ) && is_array( $_POST[ 'hook-' . $id_base ] ) ) {
-			$new_instance = wp_unslash( reset( $_POST[ 'hook-' . $id_base ] ) ); // WPCS sanitization OK.
+		if ( isset( $_POST['hook-' . $id_base] ) && is_array( $_POST['hook-' . $id_base] ) ) {
+			$new_instance = wp_unslash( reset( $_POST['hook-' . $id_base] ) ); // WPCS sanitization OK.
 		}
 
 		$number = $hook->get_number_by_id( $hook_id );
@@ -102,15 +102,15 @@ 
 	if ( ! in_array( $hook_id, $points_type_hooks, true ) ) {
 
 		$points_type_hooks[] = $hook_id;
-		$points_types_hooks[ $points_type_id ] = $points_type_hooks;
+		$points_types_hooks[$points_type_id] = $points_type_hooks;
 	}
 
 	// Remove from old points type if it has changed.
 	$old_points_type = WordPoints_Points_Hooks::get_points_type( $hook_id );
 
-	if ( $old_points_type && $old_points_type !== $points_type_id && is_array( $points_types_hooks[ $old_points_type ] ) ) {
+	if ( $old_points_type && $old_points_type !== $points_type_id && is_array( $points_types_hooks[$old_points_type] ) ) {
 
-		$points_types_hooks[ $old_points_type ] = array_diff( $points_types_hooks[ $old_points_type ], array( $hook_id ) );
+		$points_types_hooks[$old_points_type] = array_diff( $points_types_hooks[$old_points_type], array( $hook_id ) );
 	}
 
 } else {

src/components/points/admin/screens/logs.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 
 		foreach ( $points_types as $slug => $settings ) {
 
-			$tabs[ $slug ] = $settings['name'];
+			$tabs[$slug] = $settings['name'];
 		}
 
 		wordpoints_admin_show_tabs( $tabs, false );

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

Spacing +9 added lines, -9 removed lines

@@ -46,11 +46,11 @@ 
 
 	foreach ( array_keys( wordpoints_get_points_types() ) as $points_type ) {
 
-		if ( empty( $_POST['points_types'][ $points_type ] ) ) {
+		if ( empty( $_POST['points_types'][$points_type] ) ) {
 			continue;
 		}
 
-		$hooks = sanitize_text_field( wp_unslash( $_POST['points_types'][ $points_type ] ) );
+		$hooks = sanitize_text_field( wp_unslash( $_POST['points_types'][$points_type] ) );
 
 		$points_type_hooks = array();
 
@@ -62,10 +62,10 @@ 
 				continue;
 			}
 
-			$points_type_hooks[ $order ] = substr( $hook_id, strpos( $hook_id, '_' ) + 1 );
+			$points_type_hooks[$order] = substr( $hook_id, strpos( $hook_id, '_' ) + 1 );
 		}
 
-		$points_types_hooks[ $points_type ] = $points_type_hooks;
+		$points_types_hooks[$points_type] = $points_type_hooks;
 	}
 
 	WordPoints_Points_Hooks::save_points_types_hooks( $points_types_hooks );
@@ -189,14 +189,14 @@ 
 
 		$settings = false;
 
-		if ( isset( $_POST[ 'hook-' . $id_base ] ) && is_array( $_POST[ 'hook-' . $id_base ] ) ) {
-			$settings = wp_unslash( $_POST[ 'hook-' . $id_base ] ); // WPCS: sanitization OK.
+		if ( isset( $_POST['hook-' . $id_base] ) && is_array( $_POST['hook-' . $id_base] ) ) {
+			$settings = wp_unslash( $_POST['hook-' . $id_base] ); // WPCS: sanitization OK.
 		}
 
 		$points_types_hooks = WordPoints_Points_Hooks::get_points_types_hooks();
 
 		// Get the hooks for this points type.
-		$points_type_hooks = ( isset( $points_types_hooks[ $points_type_id ] ) ) ? $points_types_hooks[ $points_type_id ] : array();
+		$points_type_hooks = ( isset( $points_types_hooks[$points_type_id] ) ) ? $points_types_hooks[$points_type_id] : array();
 
 		if ( ! empty( $_POST['delete_hook'] ) ) {
 
@@ -209,7 +209,7 @@ 
 			$hook->delete_callback( $number );
 
 			// Remove this instance of the hook, and reset the positions (keys).
-			$points_types_hooks[ $points_type_id ] = array_diff( $points_type_hooks, array( $hook_id ) );
+			$points_types_hooks[$points_type_id] = array_diff( $points_type_hooks, array( $hook_id ) );
 
 			WordPoints_Points_Hooks::save_points_types_hooks( $points_types_hooks );
 
@@ -230,7 +230,7 @@ 
 
 			// Save the points types-hooks associations.
 			$points_type_hooks[] = $hook->get_id( $number );
-			$points_types_hooks[ $points_type_id ] = $points_type_hooks;
+			$points_types_hooks[$points_type_id] = $points_type_hooks;
 			WordPoints_Points_Hooks::save_points_types_hooks( $points_types_hooks );
 
 		} else {

src/components/points/classes/hook/extension/legacy/reversals.php

Spacing +4 added lines, -4 removed lines

@@ -65,8 +65,8 @@ 
 	 */
 	protected function get_points_logs_to_be_reversed( WordPoints_Hook_Fire $fire ) {
 
-		if ( isset( $fire->data[ $this->slug ]['points_logs'] ) ) {
-			return $fire->data[ $this->slug ]['points_logs'];
+		if ( isset( $fire->data[$this->slug]['points_logs'] ) ) {
+			return $fire->data[$this->slug]['points_logs'];
 		}
 
 		$meta_queries = array(
@@ -85,7 +85,7 @@ 
 			$entities = $fire->event_args->get_signature_args();
 
 			if ( ! $entities ) {
-				$fire->data[ $this->slug ]['points_logs'] = array();
+				$fire->data[$this->slug]['points_logs'] = array();
 				return array();
 			}
 
@@ -117,7 +117,7 @@ 
 			$logs = array();
 		}
 
-		$fire->data[ $this->slug ]['points_logs'] = $logs;
+		$fire->data[$this->slug]['points_logs'] = $logs;
 
 		return $logs;
 	}