WordPoints / wordpoints

src/components/ranks/classes/user/ranks/maybe/change.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
 			$new_rank_users = array();
 
 			foreach ( $new_ranks as $user_id => $new_rank ) {
-				$new_rank_users[ $new_rank ][] = $user_id;
+				$new_rank_users[ $new_rank ][ ] = $user_id;
 			}
 
 			unset( $new_ranks );

src/components/ranks/classes/ranks/installable.php

Spacing +4 added lines, -4 removed lines

@@ -63,7 +63,7 @@ 
 		$factories = parent::get_update_routine_factories();
 
 		// v2.0.0.
-		$factories[] = new WordPoints_Updater_Factory(
+		$factories[ ] = new WordPoints_Updater_Factory(
 			'2.0.0'
 			, array(
 				'global' => array(
@@ -83,12 +83,12 @@ 
 		);
 
 		// v2.4.0-alpha-4.
-		$factories[] = new WordPoints_Updater_Factory(
+		$factories[ ] = new WordPoints_Updater_Factory(
 			'2.4.0-alpha-4'
 			, array( 'global' => array( 'WordPoints_Ranks_Updater_2_4_0_Tables' ) )
 		);
 
-		$factories[] = new WordPoints_Updater_Factory(
+		$factories[ ] = new WordPoints_Updater_Factory(
 			'2.4.0-alpha-4'
 			, array(
 				'local' => array( 'WordPoints_Ranks_Updater_2_4_0_User_Ranks' ),
@@ -107,7 +107,7 @@ 
 
 		$factories = parent::get_uninstall_routine_factories();
 
-		$factories[] = new WordPoints_Uninstaller_Factory_Options(
+		$factories[ ] = new WordPoints_Uninstaller_Factory_Options(
 			array(
 				'local' => array(
 					'wordpoints_filled_base_ranks',

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

Spacing +10 added lines, -10 removed lines

@@ -77,7 +77,7 @@ 
 	 */
 	public function __construct( array $args ) {
 
-		$this->slug = $args['slug'];
+		$this->slug = $args[ 'slug' ];
 	}
 
 	//
@@ -193,10 +193,10 @@ 
 
 			// If we aren't using placeholders, calculate the value. Hidden fields
 			// never use placeholders.
-			if ( ! $args['placeholders'] || 'hidden' === $field['type'] ) {
+			if ( ! $args[ 'placeholders' ] || 'hidden' === $field[ 'type' ] ) {
 
 				// Default to the default value.
-				$value = $field['default'];
+				$value = $field[ 'default' ];
 
 				// If the value is set use that instead.
 				if ( isset( $meta[ $name ] ) ) {
@@ -204,23 +204,23 @@ 
 				}
 			}
 
-			switch ( $field['type'] ) {
+			switch ( $field[ 'type' ] ) {
 
 				case 'hidden':
 				case 'number':
 				case 'text':
-					if ( isset( $field['label'] ) ) {
+					if ( isset( $field[ 'label' ] ) ) {
 						?>
 						<p class="description description-thin"><label>
 						<?php
-						echo esc_html( $field['label'] );
+						echo esc_html( $field[ 'label' ] );
 					}
 
 					?>
 					<input
-						type="<?php echo esc_attr( $field['type'] ); ?>"
+						type="<?php echo esc_attr( $field[ 'type' ] ); ?>"
 						name="<?php echo esc_attr( $name ); ?>"
-						<?php if ( $args['placeholders'] && 'hidden' !== $field['type'] ) : ?>
+						<?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 ); ?>"
@@ -229,7 +229,7 @@ 
 					/>
 					<?php
 
-					if ( isset( $field['label'] ) ) {
+					if ( isset( $field[ 'label' ] ) ) {
 						?>
 						</label></p>
 						<?php
@@ -241,7 +241,7 @@ 
 						__METHOD__
 						, sprintf(
 							'WordPoints Error: Unknown field type "%s".'
-							, esc_html( $field['type'] )
+							, esc_html( $field[ 'type' ] )
 						)
 						, '1.7.0'
 					);

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"
 					/>

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

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
 			return false;
 		}
 
-		$args['slug'] = $type;
+		$args[ 'slug' ] = $type;
 
 		self::$types[ $type ] = new $class( $args );
 

src/components/ranks/classes/rank/shortcode/rank/list.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 		$result .= $headings;
 		$result .= '</thead>';
 
-		$group = WordPoints_Rank_Groups::get_group( $this->atts['rank_group'] );
+		$group = WordPoints_Rank_Groups::get_group( $this->atts[ 'rank_group' ] );
 
 		foreach ( $group->get_ranks() as $rank_id ) {
 

src/components/ranks/includes/rank-types/points.php

Spacing +12 added lines, -12 removed lines

@@ -47,7 +47,7 @@ 
 
 		$this->name = _x( 'Points', 'rank type', 'wordpoints' );
 
-		if ( ! isset( $args['points_type'] ) ) {
+		if ( ! isset( $args[ 'points_type' ] ) ) {
 			_doing_it_wrong(
 				__METHOD__
 				, 'WordPoints Error: The "points_type" argument is required.'
@@ -56,8 +56,8 @@ 
 			return;
 		}
 
-		$this->meta_fields['points']['label']        = _x( 'Points', 'form label', 'wordpoints' );
-		$this->meta_fields['points_type']['default'] = $args['points_type'];
+		$this->meta_fields[ 'points' ][ 'label' ]        = _x( 'Points', 'form label', 'wordpoints' );
+		$this->meta_fields[ 'points_type' ][ 'default' ] = $args[ 'points_type' ];
 
 		add_action( 'wordpoints_points_altered', array( $this, 'hook' ), 10, 3 );
 	}
@@ -83,7 +83,7 @@ 
 	 */
 	public function hook( $user_id, $points, $points_type ) {
 
-		if ( $points_type !== $this->meta_fields['points_type']['default'] ) {
+		if ( $points_type !== $this->meta_fields[ 'points_type' ][ 'default' ] ) {
 			return;
 		}
 
@@ -95,7 +95,7 @@ 
 	 */
 	public function validate_rank_meta( array $meta ) {
 
-		if ( ! isset( $meta['points'] ) || false === wordpoints_int( $meta['points'] ) ) {
+		if ( ! isset( $meta[ 'points' ] ) || false === wordpoints_int( $meta[ 'points' ] ) ) {
 			return new WP_Error(
 				'wordpoints_points_rank_type_invalid_points'
 				, __( 'The amount of points is required, and must be a valid number.', 'wordpoints' )
@@ -104,22 +104,22 @@ 
 		}
 
 		if (
-			! isset( $meta['points_type'] )
-			|| ! wordpoints_is_points_type( $meta['points_type'] )
+			! isset( $meta[ 'points_type' ] )
+			|| ! wordpoints_is_points_type( $meta[ 'points_type' ] )
 		) {
 			return false;
 		}
 
-		$minimum = wordpoints_get_points_minimum( $meta['points_type'] );
+		$minimum = wordpoints_get_points_minimum( $meta[ 'points_type' ] );
 
-		if ( $meta['points'] < $minimum ) {
+		if ( $meta[ 'points' ] < $minimum ) {
 
 			return new WP_Error(
 				'wordpoints_points_rank_type_points_less_than_minimum'
 				, sprintf(
 					// translators: Minimum number of points.
 					__( 'The number of points must be more than the minimum (%s).', 'wordpoints' )
-					, wordpoints_format_points( $minimum, $meta['points_type'], 'points_rank_error' )
+					, wordpoints_format_points( $minimum, $meta[ 'points_type' ], 'points_rank_error' )
 				)
 				, array( 'field' => 'points' )
 			);
@@ -133,7 +133,7 @@ 
 	 */
 	public function can_transition_user_ranks( array $user_ids, WordPoints_Rank $rank, array $args ) {
 
-		if ( $rank->points_type !== $this->meta_fields['points_type']['default'] ) {
+		if ( $rank->points_type !== $this->meta_fields[ 'points_type' ][ 'default' ] ) {
 			return array();
 		}
 
@@ -185,7 +185,7 @@ 
 	 */
 	protected function can_transition_user_rank( $user_id, $rank, array $args ) {
 
-		if ( $rank->points_type !== $this->meta_fields['points_type']['default'] ) {
+		if ( $rank->points_type !== $this->meta_fields[ 'points_type' ][ 'default' ] ) {
 			return false;
 		}
 

src/components/ranks/includes/integration/points/functions.php

Spacing +8 added lines, -8 removed lines

@@ -20,7 +20,7 @@ 
 
 		$rank = wordpoints_get_formatted_user_rank(
 			get_current_user_id()
-			, "points_type-{$instance['points_type']}"
+			, "points_type-{$instance[ 'points_type' ]}"
 			, 'my-points-widget'
 			, array( 'widget_settings' => $instance )
 		);
@@ -89,18 +89,18 @@ 
  */
 function wordpoints_user_rank_shortcode_points_type_attr( $out, $pairs, $atts ) {
 
-	if ( empty( $out['rank_group'] ) ) {
+	if ( empty( $out[ 'rank_group' ] ) ) {
 
-		if ( isset( $atts['points_type'] ) ) {
+		if ( isset( $atts[ 'points_type' ] ) ) {
 
-			$out['rank_group'] = "points_type-{$atts['points_type']}";
+			$out[ 'rank_group' ] = "points_type-{$atts[ 'points_type' ]}";
 
 		} else {
 
 			$points_type = wordpoints_get_default_points_type();
 
 			if ( $points_type ) {
-				$out['rank_group'] = "points_type-{$points_type}";
+				$out[ 'rank_group' ] = "points_type-{$points_type}";
 			}
 		}
 	}
@@ -122,11 +122,11 @@ 
 		WordPoints_Rank_Groups::register_group(
 			"points_type-{$slug}"
 			, array(
-				'name'        => $points_type['name'],
+				'name'        => $points_type[ 'name' ],
 				'description' => sprintf(
 					// translators: Points type name.
 					__( 'This rank group is associated with the “%s” points type.', 'wordpoints' )
-					, $points_type['name']
+					, $points_type[ 'name' ]
 				),
 			)
 		);
@@ -183,7 +183,7 @@ 
 
 	?>
 
-	<a href="<?php echo esc_url( self_admin_url( 'admin.php?page=wordpoints_ranks&tab=points_type-' . $points_type['slug'] ) ); ?>">
+	<a href="<?php echo esc_url( self_admin_url( 'admin.php?page=wordpoints_ranks&tab=points_type-' . $points_type[ 'slug' ] ) ); ?>">
 		<?php esc_html_e( 'Go to the ranks for this points type.', 'wordpoints' ); ?>
 	</a>
 

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

Spacing +61 added lines, -61 removed lines

@@ -49,7 +49,7 @@ 
 		parent::__construct(
 			array(
 				'plural' => 'wordpoints-extensions',
-				'screen' => ( isset( $args['screen'] ) ) ? $args['screen'] : null,
+				'screen' => ( isset( $args[ 'screen' ] ) ) ? $args[ 'screen' ] : null,
 			)
 		);
 
@@ -76,12 +76,12 @@ 
 		 */
 		$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.
-			$status = sanitize_key( $_REQUEST['module_status'] ); // 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.
-			$_SERVER['REQUEST_URI'] = add_query_arg( 's', sanitize_text_field( wp_unslash( $_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.
 		}
 
 		$page = $this->get_pagenum();
@@ -96,7 +96,7 @@ 
 	 */
 	public function get_table_classes() {
 
-		return array( 'widefat', $this->_args['plural'], 'modules', 'plugins' );
+		return array( 'widefat', $this->_args[ 'plural' ], 'modules', 'plugins' );
 	}
 
 	/**
@@ -202,7 +202,7 @@ 
 
 		} // End if ( not network admin ).
 
-		foreach ( (array) $modules['all'] as $module_file => $module_data ) {
+		foreach ( (array) $modules[ 'all' ] as $module_file => $module_data ) {
 
 			// Filter into individual sections.
 			if (
@@ -212,9 +212,9 @@ 
 			) {
 
 				if ( $show_network_active ) {
-					$modules['inactive'][ $module_file ] = $module_data;
+					$modules[ 'inactive' ][ $module_file ] = $module_data;
 				} 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;
+					$modules[ 'active' ][ $module_file ] = $module_data;
 				} else {
-					unset( $modules['all'][ $module_file ] );
+					unset( $modules[ 'all' ][ $module_file ] );
 				}
 
 			} elseif (
@@ -238,16 +238,16 @@ 
 				)
 			) {
 
-				$modules['active'][ $module_file ] = $module_data;
+				$modules[ 'active' ][ $module_file ] = $module_data;
 
 			} else {
 
 				// Was the module recently activated?
 				if ( ! $this->screen->in_admin( 'network' ) && isset( $recently_activated[ $module_file ] ) ) {
-					$modules['recently_activated'][ $module_file ] = $module_data;
+					$modules[ 'recently_activated' ][ $module_file ] = $module_data;
 				}
 
-				$modules['inactive'][ $module_file ] = $module_data;
+				$modules[ 'inactive' ][ $module_file ] = $module_data;
 
 			} // 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 ];
+				$modules[ 'upgrade' ][ $module_file ] = $modules[ 'all' ][ $module_file ];
 			}
 
 		} // End foreach ( modules ).
@@ -264,7 +264,7 @@ 
 		if ( $s ) {
 
 			$status            = 'search';
-			$modules['search'] = array_filter( $modules['all'], array( $this, 'search_callback' ) );
+			$modules[ 'search' ] = array_filter( $modules[ 'all' ], array( $this, 'search_callback' ) );
 		}
 
 		/**
@@ -342,8 +342,8 @@ 
 
 		static $term;
 
-		if ( is_null( $term ) && isset( $_REQUEST['s'] ) ) { // WPCS: CSRF OK.
-			$term = sanitize_text_field( wp_unslash( $_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.
 		}
 
 		foreach ( $module_data as $value ) {
@@ -395,7 +395,7 @@ 
 
 		global $modules;
 
-		if ( ! empty( $modules['all'] ) ) {
+		if ( ! empty( $modules[ 'all' ] ) ) {
 			esc_html_e( 'No extensions found.', 'wordpoints' );
 		} else {
 			esc_html_e( 'There are not any extensions installed.', 'wordpoints' );
@@ -514,11 +514,11 @@ 
 		$actions = array();
 
 		if ( 'active' !== $status ) {
-			$actions['activate-selected'] = ( $this->screen->in_admin( 'network' ) ) ? esc_html__( 'Network Activate', 'wordpoints' ) : esc_html__( 'Activate', 'wordpoints' );
+			$actions[ 'activate-selected' ] = ( $this->screen->in_admin( 'network' ) ) ? esc_html__( 'Network Activate', 'wordpoints' ) : esc_html__( 'Activate', 'wordpoints' );
 		}
 
 		if ( 'inactive' !== $status && 'recent' !== $status ) {
-			$actions['deactivate-selected'] = ( $this->screen->in_admin( 'network' ) ) ? esc_html__( 'Network Deactivate', 'wordpoints' ) : esc_html__( 'Deactivate', 'wordpoints' );
+			$actions[ 'deactivate-selected' ] = ( $this->screen->in_admin( 'network' ) ) ? esc_html__( 'Network Deactivate', 'wordpoints' ) : esc_html__( 'Deactivate', 'wordpoints' );
 		}
 
 		if (
@@ -526,14 +526,14 @@ 
 			&& current_user_can( 'delete_wordpoints_extensions' )
 			&& 'active' !== $status
 		) {
-			$actions['delete-selected'] = esc_html__( 'Delete', 'wordpoints' );
+			$actions[ 'delete-selected' ] = esc_html__( 'Delete', 'wordpoints' );
 		}
 
 		if (
 			( ! is_multisite() || is_network_admin() )
 			&& current_user_can( 'update_wordpoints_extensions' )
 		) {
-			$actions['update-selected'] = esc_html__( 'Update', 'wordpoints' );
+			$actions[ 'update-selected' ] = esc_html__( 'Update', 'wordpoints' );
 		}
 
 		/**
@@ -555,11 +555,11 @@ 
 	protected function display_tablenav( $which ) {
 
 		// Back-compat for the old nonce name.
-		$this->_args['plural'] = 'modules';
+		$this->_args[ 'plural' ] = 'modules';
 
 		parent::display_tablenav( $which );
 
-		$this->_args['plural'] = 'wordpoints-extensions';
+		$this->_args[ 'plural' ] = 'wordpoints-extensions';
 	}
 
 	/**
@@ -643,7 +643,7 @@ 
 		$class = apply_filters( 'wordpoints_module_list_row_class', $class, $module_file, $module_data, $context );
 
 		?>
-		<tr id="<?php echo esc_attr( sanitize_title( $module_data['name'] ) ); ?>" class="<?php echo esc_attr( $class ); ?>">
+		<tr id="<?php echo esc_attr( sanitize_title( $module_data[ 'name' ] ) ); ?>" class="<?php echo esc_attr( $class ); ?>">
 			<?php $this->single_row_columns( $item, $class, $is_active ); ?>
 		</tr>
 		<?php
@@ -683,7 +683,7 @@ 
 
 		list( $module_file, $module_data ) = $item;
 
-		$module_data['extra']['module_file'] = $module_file;
+		$module_data[ 'extra' ][ 'module_file' ] = $module_file;
 
 		if ( $this->screen->in_admin( 'network' ) ) {
 
@@ -696,11 +696,11 @@ 
 			$restricted_network_only   = ( is_multisite() && is_network_only_wordpoints_module( $module_file ) && ! $is_active );
 		}
 
-		$module_data['extra']['is_active'] = $is_active;
-		$module_data['extra']['restricted_network_active'] = $restricted_network_active;
-		$module_data['extra']['restricted_network_only']   = $restricted_network_only;
+		$module_data[ 'extra' ][ 'is_active' ] = $is_active;
+		$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 ) {
 
@@ -737,11 +737,11 @@ 
 	 */
 	protected function column_cb( $module_data ) {
 
-		$checkbox_id = 'checkbox_' . sanitize_key( $module_data['extra']['module_file'] );
+		$checkbox_id = 'checkbox_' . sanitize_key( $module_data[ 'extra' ][ 'module_file' ] );
 
 		?>
 		<th scope="row" class="check-column">
-			<?php if ( ! $module_data['extra']['restricted_network_active'] && ! $module_data['extra']['restricted_network_only'] ) : ?>
+			<?php if ( ! $module_data[ 'extra' ][ 'restricted_network_active' ] && ! $module_data[ 'extra' ][ 'restricted_network_only' ] ) : ?>
 				<label class="screen-reader-text" for="<?php echo esc_attr( $checkbox_id ); ?>">
 					<?php
 
@@ -749,13 +749,13 @@ 
 						sprintf(
 							// translators: Extension name.
 							__( 'Select %s', 'wordpoints' )
-							, $module_data['name']
+							, $module_data[ 'name' ]
 						)
 					);
 
 					?>
 				</label>
-				<input type="checkbox" name="checked[]" value="<?php echo esc_attr( $module_data['extra']['module_file'] ); ?>" id="<?php echo esc_attr( $checkbox_id ); ?>" />
+				<input type="checkbox" name="checked[]" value="<?php echo esc_attr( $module_data[ 'extra' ][ 'module_file' ] ); ?>" id="<?php echo esc_attr( $checkbox_id ); ?>" />
 			<?php endif; ?>
 		</th>
 		<?php
@@ -774,7 +774,7 @@ 
 
 		?>
 		<td class="module-title<?php echo ( $is_hidden ) ? ' hidden' : ''; ?><?php echo ( $is_primary ) ? ' has-row-actions column-primary' : ''; ?>">
-			<strong><?php echo esc_html( $module_data['name'] ); ?></strong>
+			<strong><?php echo esc_html( $module_data[ 'name' ] ); ?></strong>
 			<?php if ( $is_primary ) : ?>
 				<?php echo $this->row_actions( $this->get_module_row_actions( $module_data ), true ); /* WPCS XSS OK. */ ?>
 			<?php endif; ?>
@@ -800,8 +800,8 @@ 
 		<td class="column-description desc<?php echo ( $is_hidden ) ? ' hidden' : ''; ?><?php echo ( $is_primary ) ? ' has-row-actions column-primary' : ''; ?>">
 			<div class="module-description">
 				<p>
-					<?php if ( ! empty( $module_data['description'] ) ) : ?>
-						<?php echo wp_kses( $module_data['description'] , 'wordpoints_module_description' ); ?>
+					<?php if ( ! empty( $module_data[ 'description' ] ) ) : ?>
+						<?php echo wp_kses( $module_data[ 'description' ], 'wordpoints_module_description' ); ?>
 					<?php else : ?>
 						&nbsp;
 					<?php endif; ?>
@@ -812,25 +812,25 @@ 
 
 				$module_meta = array();
 
-				if ( ! empty( $module_data['version'] ) ) {
+				if ( ! empty( $module_data[ 'version' ] ) ) {
 					// translators: Extension version.
-					$module_meta[] = sprintf( esc_html__( 'Version %s', 'wordpoints' ), $module_data['version'] );
+					$module_meta[ ] = sprintf( esc_html__( 'Version %s', 'wordpoints' ), $module_data[ 'version' ] );
 				}
 
-				if ( ! empty( $module_data['author'] ) ) {
+				if ( ! empty( $module_data[ 'author' ] ) ) {
 
-					$author = $module_data['author'];
+					$author = $module_data[ 'author' ];
 
-					if ( ! empty( $module_data['author_uri'] ) ) {
-						$author = '<a href="' . esc_url( $module_data['author_uri'] ) . '">' . esc_html( $module_data['author'] ) . '</a>';
+					if ( ! empty( $module_data[ 'author_uri' ] ) ) {
+						$author = '<a href="' . esc_url( $module_data[ 'author_uri' ] ) . '">' . esc_html( $module_data[ 'author' ] ) . '</a>';
 					}
 
 					// translators: Author name.
-					$module_meta[] = sprintf( __( 'By %s', 'wordpoints' ), $author );
+					$module_meta[ ] = sprintf( __( 'By %s', 'wordpoints' ), $author );
 				}
 
-				if ( ! empty( $module_data['uri'] ) ) {
-					$module_meta[] = '<a href="' . esc_url( $module_data['uri'] ) . '">' . esc_html__( 'Visit extension site', 'wordpoints' ) . '</a>';
+				if ( ! empty( $module_data[ 'uri' ] ) ) {
+					$module_meta[ ] = '<a href="' . esc_url( $module_data[ 'uri' ] ) . '">' . esc_html__( 'Visit extension site', 'wordpoints' ) . '</a>';
 				}
 
 				/**
@@ -846,7 +846,7 @@ 
 				 * @param array  $module_data The info about the module.
 				 * @param string $status      The module status being displayed.
 				 */
-				$module_meta = apply_filters( 'wordpoints_module_row_meta', $module_meta, $module_data['extra']['module_file'], $module_data, $status );
+				$module_meta = apply_filters( 'wordpoints_module_row_meta', $module_meta, $module_data[ 'extra' ][ 'module_file' ], $module_data, $status );
 
 				echo wp_kses( implode( ' | ', $module_meta ), 'data' );
 
@@ -883,7 +883,7 @@ 
 			 * @param string $module_file The main file of the current module.
 			 * @param array  $module_data The module's info.
 			 */
-			do_action( 'wordpoints_manage_modules_custom_column', $column_name, $module_data['module_file'], $module_data );
+			do_action( 'wordpoints_manage_modules_custom_column', $column_name, $module_data[ 'module_file' ], $module_data );
 			?>
 			<?php if ( $is_primary ) : ?>
 				<?php echo $this->row_actions( $this->get_module_row_actions( $module_data ), true ); /* WPCS XSS OK. */ ?>
@@ -912,8 +912,8 @@ 
 
 		$context = $status;
 
-		$module_file = $module_data['extra']['module_file'];
-		$args        = $module_data['extra'];
+		$module_file = $module_data[ 'extra' ][ 'module_file' ];
+		$args        = $module_data[ 'extra' ];
 
 		// Pre-order.
 		$actions = array(
@@ -926,49 +926,49 @@ 
 
 		if ( $this->screen->in_admin( 'network' ) ) {
 
-			if ( $args['is_active'] ) {
+			if ( $args[ 'is_active' ] ) {
 
 				if ( current_user_can( 'manage_network_wordpoints_extensions' ) ) {
 					// translators: Extension name.
-					$actions['deactivate'] = '<a href="' . wp_nonce_url( add_query_arg( 'action', 'deactivate', $url ), "deactivate-module_{$module_file}" ) . '" aria-label="' . esc_attr( sprintf( __( 'Network deactivate %s', 'wordpoints' ), $module_data['name'] ) ) . '">' . esc_html__( 'Network Deactivate', 'wordpoints' ) . '</a>';
+					$actions[ 'deactivate' ] = '<a href="' . wp_nonce_url( add_query_arg( 'action', 'deactivate', $url ), "deactivate-module_{$module_file}" ) . '" aria-label="' . esc_attr( sprintf( __( 'Network deactivate %s', 'wordpoints' ), $module_data[ 'name' ] ) ) . '">' . esc_html__( 'Network Deactivate', 'wordpoints' ) . '</a>';
 				}
 
 			} else {
 
 				if ( current_user_can( 'manage_network_wordpoints_extensions' ) ) {
 					// translators: Extension name.
-					$actions['activate'] = '<a href="' . wp_nonce_url( add_query_arg( 'action', 'activate', $url ), "activate-module_{$module_file}" ) . '" aria-label="' . esc_attr( sprintf( __( 'Network activate %s', 'wordpoints' ), $module_data['name'] ) ) . '" class="edit">' . esc_html__( 'Network Activate', 'wordpoints' ) . '</a>';
+					$actions[ 'activate' ] = '<a href="' . wp_nonce_url( add_query_arg( 'action', 'activate', $url ), "activate-module_{$module_file}" ) . '" aria-label="' . esc_attr( sprintf( __( 'Network activate %s', 'wordpoints' ), $module_data[ 'name' ] ) ) . '" class="edit">' . esc_html__( 'Network Activate', 'wordpoints' ) . '</a>';
 				}
 
 				if ( current_user_can( 'delete_wordpoints_extensions' ) && ! is_wordpoints_module_active( $module_file ) ) {
 					// translators: Extension name.
-					$actions['delete'] = '<a href="' . wp_nonce_url( self_admin_url( 'admin.php?page=wordpoints_extensions&action=delete-selected&amp;checked[]=' . $module_file . '&amp;module_status=' . $context . '&amp;paged=' . $page . '&amp;s=' . $s ), 'bulk-modules' ) . '" aria-label="' . esc_attr( sprintf( __( 'Delete %s', 'wordpoints' ), $module_data['name'] ) ) . '" class="delete">' . esc_html__( 'Delete', 'wordpoints' ) . '</a>';
+					$actions[ 'delete' ] = '<a href="' . wp_nonce_url( self_admin_url( 'admin.php?page=wordpoints_extensions&action=delete-selected&amp;checked[]=' . $module_file . '&amp;module_status=' . $context . '&amp;paged=' . $page . '&amp;s=' . $s ), 'bulk-modules' ) . '" aria-label="' . esc_attr( sprintf( __( 'Delete %s', 'wordpoints' ), $module_data[ 'name' ] ) ) . '" class="delete">' . esc_html__( 'Delete', 'wordpoints' ) . '</a>';
 				}
 			}
 
 		} else {
 
-			if ( $args['restricted_network_active'] ) {
+			if ( $args[ 'restricted_network_active' ] ) {
 
 				$actions = array( 'network_active' => __( 'Network Active', 'wordpoints' ) );
 
-			} elseif ( $args['restricted_network_only'] ) {
+			} elseif ( $args[ 'restricted_network_only' ] ) {
 
 				$actions = array( 'network_only' => __( 'Network Only', 'wordpoints' ) );
 
-			} elseif ( $args['is_active'] ) {
+			} elseif ( $args[ 'is_active' ] ) {
 
 				// translators: Extension name.
-				$actions['deactivate'] = '<a href="' . wp_nonce_url( add_query_arg( 'action', 'deactivate', $url ), "deactivate-module_{$module_file}" ) . '" aria-label="' . esc_attr( sprintf( __( 'Deactivate %s', 'wordpoints' ), $module_data['name'] ) ) . '">' . esc_html__( 'Deactivate', 'wordpoints' ) . '</a>';
+				$actions[ 'deactivate' ] = '<a href="' . wp_nonce_url( add_query_arg( 'action', 'deactivate', $url ), "deactivate-module_{$module_file}" ) . '" aria-label="' . esc_attr( sprintf( __( 'Deactivate %s', 'wordpoints' ), $module_data[ 'name' ] ) ) . '">' . esc_html__( 'Deactivate', 'wordpoints' ) . '</a>';
 
 			} else {
 
 				// translators: Extension name.
-				$actions['activate'] = '<a href="' . wp_nonce_url( add_query_arg( 'action', 'activate', $url ), "activate-module_{$module_file}" ) . '" aria-label="' . esc_attr( sprintf( __( 'Activate %s', 'wordpoints' ), $module_data['name'] ) ) . '" class="edit">' . esc_html__( 'Activate', 'wordpoints' ) . '</a>';
+				$actions[ 'activate' ] = '<a href="' . wp_nonce_url( add_query_arg( 'action', 'activate', $url ), "activate-module_{$module_file}" ) . '" aria-label="' . esc_attr( sprintf( __( 'Activate %s', 'wordpoints' ), $module_data[ 'name' ] ) ) . '" class="edit">' . esc_html__( 'Activate', 'wordpoints' ) . '</a>';
 
 				if ( ! is_multisite() && current_user_can( 'delete_wordpoints_extensions' ) ) {
 					// translators: Extension name.
-					$actions['delete'] = '<a href="' . wp_nonce_url( self_admin_url( 'admin.php?page=wordpoints_extensions&action=delete-selected&amp;checked[]=' . $module_file . '&amp;module_status=' . $context . '&amp;paged=' . $page . '&amp;s=' . $s ), 'bulk-modules' ) . '" aria-label="' . esc_attr( sprintf( __( 'Delete %s', 'wordpoints' ), $module_data['name'] ) ) . '" class="delete">' . esc_html__( 'Delete', 'wordpoints' ) . '</a>';
+					$actions[ 'delete' ] = '<a href="' . wp_nonce_url( self_admin_url( 'admin.php?page=wordpoints_extensions&action=delete-selected&amp;checked[]=' . $module_file . '&amp;module_status=' . $context . '&amp;paged=' . $page . '&amp;s=' . $s ), 'bulk-modules' ) . '" aria-label="' . esc_attr( sprintf( __( 'Delete %s', 'wordpoints' ), $module_data[ 'name' ] ) ) . '" class="delete">' . esc_html__( 'Delete', 'wordpoints' ) . '</a>';
 				}
 			}
 

Braces +11 added lines, -5 removed lines

@@ -68,11 +68,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.
 		}
 
@@ -311,7 +313,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.
 		}
 
@@ -737,9 +740,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">

src/admin/screens/modules.php

Spacing +15 added lines, -15 removed lines

@@ -25,22 +25,22 @@ 
 	}
 }
 
-if ( isset( $_GET['error'] ) ) {
+if ( isset( $_GET[ 'error' ] ) ) {
 
-	if ( isset( $_GET['main'] ) ) {
+	if ( isset( $_GET[ 'main' ] ) ) {
 
 		wordpoints_show_admin_error(
 			esc_html__( 'You cannot delete an extension while it is active on the main site.', 'wordpoints' )
 			, array( 'dismissible' => true )
 		);
 
-	} elseif ( isset( $_GET['charsout'] ) ) {
+	} elseif ( isset( $_GET[ 'charsout' ] ) ) {
 
 		wordpoints_show_admin_message(
 			sprintf(
 				// translators: Number of characters.
 				__( 'The extension generated %d characters of <strong>unexpected output</strong> during activation. If you notice &#8220;headers already sent&#8221; messages, problems with syndication feeds or other issues, try deactivating or removing this extension.', 'wordpoints' )
-				, (int) $_GET['charsout'] // WPCS: CSRF OK.
+				, (int) $_GET[ 'charsout' ] // WPCS: CSRF OK.
 			)
 			, 'warning'
 			, array( 'dismissible' => true )
@@ -51,14 +51,14 @@ 
 		$error_message = __( 'Extension could not be activated because it triggered a <strong>fatal error</strong>.', 'wordpoints' );
 
 		if (
-			isset( $_GET['_error_nonce'], $_GET['module'] )
+			isset( $_GET[ '_error_nonce' ], $_GET[ 'module' ] )
 			&& wordpoints_verify_nonce( '_error_nonce', 'module-activation-error_%s', array( 'module' ) )
 		) {
 
 			$url = self_admin_url(
 				'admin.php?page=wordpoints_extensions&action=error_scrape&amp;module='
-					. sanitize_text_field( wp_unslash( $_GET['module'] ) )
-					. '&amp;_wpnonce=' . sanitize_key( $_GET['_error_nonce'] )
+					. sanitize_text_field( wp_unslash( $_GET[ 'module' ] ) )
+					. '&amp;_wpnonce=' . sanitize_key( $_GET[ '_error_nonce' ] )
 			);
 
 			?>
@@ -82,7 +82,7 @@ 
 
 	} // End if ( main error ) elseif ( unexpected output error ) else.
 
-} elseif ( isset( $_GET['deleted'] ) ) {
+} elseif ( isset( $_GET[ 'deleted' ] ) ) {
 
 	$user_id       = get_current_user_id();
 	$delete_result = get_transient( 'wordpoints_modules_delete_result_' . $user_id );
@@ -110,7 +110,7 @@ 
 		);
 	}
 
-} elseif ( isset( $_GET['activate'] ) ) {
+} elseif ( isset( $_GET[ 'activate' ] ) ) {
 
 	wordpoints_show_admin_message(
 		__( 'Extension <strong>activated</strong>.', 'wordpoints' )
@@ -118,7 +118,7 @@ 
 		, array( 'dismissible' => true )
 	);
 
-} elseif ( isset( $_GET['activate-multi'] ) ) {
+} elseif ( isset( $_GET[ 'activate-multi' ] ) ) {
 
 	wordpoints_show_admin_message(
 		__( 'Selected extensions <strong>activated</strong>.', 'wordpoints' )
@@ -126,7 +126,7 @@ 
 		, array( 'dismissible' => true )
 	);
 
-} elseif ( isset( $_GET['deactivate'] ) ) {
+} elseif ( isset( $_GET[ 'deactivate' ] ) ) {
 
 	wordpoints_show_admin_message(
 		__( 'Extension <strong>deactivated</strong>.', 'wordpoints' )
@@ -134,7 +134,7 @@ 
 		, array( 'dismissible' => true )
 	);
 
-} elseif ( isset( $_GET['deactivate-multi'] ) ) {
+} elseif ( isset( $_GET[ 'deactivate-multi' ] ) ) {
 
 	wordpoints_show_admin_message(
 		__( 'Selected extensions <strong>deactivated</strong>.', 'wordpoints' )
@@ -142,7 +142,7 @@ 
 		, array( 'dismissible' => true )
 	);
 
-} elseif ( isset( $_REQUEST['action'] ) && 'update-selected' === sanitize_key( $_REQUEST['action'] ) ) {
+} elseif ( isset( $_REQUEST[ 'action' ] ) && 'update-selected' === sanitize_key( $_REQUEST[ 'action' ] ) ) {
 
 	wordpoints_show_admin_message(
 		esc_html__( 'No out of date extensions were selected.', 'wordpoints' )
@@ -162,7 +162,7 @@ 
 			<a href="<?php echo esc_url( self_admin_url( 'admin.php?page=wordpoints_install_extensions' ) ); ?>" class="page-title-action"><?php echo esc_html_x( 'Add New', 'extension', 'wordpoints' ); ?></a>
 		<?php endif; ?>
 
-		<?php if ( ! empty( $_REQUEST['s'] ) ) : ?>
+		<?php if ( ! empty( $_REQUEST[ 's' ] ) ) : ?>
 			<span class="subtitle">
 				<?php
 
@@ -170,7 +170,7 @@ 
 					sprintf(
 						// translators: Search term.
 						__( 'Search results for &#8220;%s&#8221;', 'wordpoints' )
-						, sanitize_text_field( wp_unslash( $_REQUEST['s'] ) )
+						, sanitize_text_field( wp_unslash( $_REQUEST[ 's' ] ) )
 					)
 				);