WordPoints / wordpoints

src/classes/hook/condition.php

Spacing +7 added lines, -7 removed lines

@@ -23,7 +23,7 @@ 
 		WordPoints_Hook_Reaction_Validator $validator
 	) {
 
-		if ( ! isset( $settings['value'] ) || '' === $settings['value'] ) {
+		if ( ! isset( $settings[ 'value' ] ) || '' === $settings[ 'value' ] ) {
 
 			$settings_fields = $this->get_settings_fields();
 
@@ -31,7 +31,7 @@ 
 				sprintf(
 					// translators: Form field name.
 					__( '%s is required.', 'wordpoints' )
-					, $settings_fields['value']['label']
+					, $settings_fields[ 'value' ][ 'label' ]
 				)
 				, 'value'
 			);
@@ -48,7 +48,7 @@ 
 				return $settings;
 			}
 
-			$validated_value = $data_type->validate_value( $settings['value'] );
+			$validated_value = $data_type->validate_value( $settings[ 'value' ] );
 
 			if ( is_wp_error( $validated_value ) ) {
 
@@ -57,7 +57,7 @@ 
 				$validator->add_error(
 					sprintf(
 						$validated_value->get_error_message()
-						, $settings_fields['value']['label']
+						, $settings_fields[ 'value' ][ 'label' ]
 					)
 					, 'value'
 				);
@@ -65,17 +65,17 @@ 
 				return $settings;
 			}
 
-			$settings['value'] = $validated_value;
+			$settings[ 'value' ] = $validated_value;
 
 		} elseif ( $arg instanceof WordPoints_Entity ) {
 
-			if ( ! $arg->exists( $settings['value'] ) ) {
+			if ( ! $arg->exists( $settings[ 'value' ] ) ) {
 				$validator->add_error(
 					sprintf(
 						// translators: 1. Singular item type name; 2. Item ID/slug.
 						__( '%1$s “%2$s” not found.', 'wordpoints' )
 						, $arg->get_title()
-						, $settings['value']
+						, $settings[ 'value' ]
 					)
 					, 'value'
 				);

src/classes/hook/condition/string/contains.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 
 		return false !== strpos(
 			$args->get_current()->get_the_value()
-			, $settings['value']
+			, $settings[ 'value' ]
 		);
 	}
 }

src/classes/hook/condition/entity/array/contains.php

Spacing +13 added lines, -13 removed lines

@@ -92,7 +92,7 @@ 
 
 		$this->validate_count();
 
-		if ( isset( $settings['conditions'] ) ) {
+		if ( isset( $settings[ 'conditions' ] ) ) {
 			$this->validate_conditions( $arg );
 		}
 
@@ -107,8 +107,8 @@ 
 	protected function validate_count() {
 
 		if (
-			! empty( $this->settings['max'] )
-			&& ! wordpoints_posint( $this->settings['max'] )
+			! empty( $this->settings[ 'max' ] )
+			&& ! wordpoints_posint( $this->settings[ 'max' ] )
 		) {
 			$this->validator->add_error(
 				__( 'The maximum must be a positive integer.', 'wordpoints' )
@@ -116,9 +116,9 @@ 
 			);
 		}
 
-		if ( ! empty( $this->settings['min'] ) ) {
+		if ( ! empty( $this->settings[ 'min' ] ) ) {
 
-			if ( ! wordpoints_posint( $this->settings['min'] ) ) {
+			if ( ! wordpoints_posint( $this->settings[ 'min' ] ) ) {
 
 				$this->validator->add_error(
 					__( 'The minimum must be a positive integer.', 'wordpoints' )
@@ -126,8 +126,8 @@ 
 				);
 
 			} elseif (
-				! empty( $this->settings['max'] )
-				&& $this->settings['max'] < $this->settings['min']
+				! empty( $this->settings[ 'max' ] )
+				&& $this->settings[ 'max' ] < $this->settings[ 'min' ]
 			) {
 
 				$this->validator->add_error(
@@ -164,8 +164,8 @@ 
 
 		$this->validator->push_field( 'conditions' );
 
-		$this->settings['conditions'] = $this->conditions_extension->validate_conditions(
-			$this->settings['conditions']
+		$this->settings[ 'conditions' ] = $this->conditions_extension->validate_conditions(
+			$this->settings[ 'conditions' ]
 			, $args
 		);
 
@@ -187,7 +187,7 @@ 
 
 			$entities = $arg->get_the_entities();
 
-			if ( isset( $this->settings['conditions'] ) ) {
+			if ( isset( $this->settings[ 'conditions' ] ) ) {
 				$entities = $this->filter_entities( $entities );
 			}
 		}
@@ -212,7 +212,7 @@ 
 			$event_args->add_entity( $entity );
 
 			$matches = $this->conditions_extension->conditions_are_met(
-				$this->settings['conditions']
+				$this->settings[ 'conditions' ]
 				, $event_args
 			);
 
@@ -235,11 +235,11 @@ 
 	 */
 	protected function check_count( $count ) {
 
-		if ( isset( $this->settings['max'] ) && $count > $this->settings['max'] ) {
+		if ( isset( $this->settings[ 'max' ] ) && $count > $this->settings[ 'max' ] ) {
 			return false;
 		}
 
-		if ( isset( $this->settings['min'] ) && $count < $this->settings['min'] ) {
+		if ( isset( $this->settings[ 'min' ] ) && $count < $this->settings[ 'min' ] ) {
 			return false;
 		}
 

src/classes/hook/condition/equals.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 	 */
 	public function is_met( array $settings, WordPoints_Hook_Event_Args $args ) {
 
-		return $settings['value'] === $args->get_current()->get_the_value();
+		return $settings[ 'value' ] === $args->get_current()->get_the_value();
 	}
 }
 

src/classes/dropdown/builder.php

Spacing +7 added lines, -7 removed lines

@@ -100,7 +100,7 @@ 
 	 */
 	public function start() {
 
-		echo '<select id="', esc_attr( $this->args['id'] ), '" name="', esc_attr( $this->args['name'] ), '" class="', esc_attr( $this->args['class'] ), '">';
+		echo '<select id="', esc_attr( $this->args[ 'id' ] ), '" name="', esc_attr( $this->args[ 'name' ] ), '" class="', esc_attr( $this->args[ 'class' ] ), '">';
 	}
 
 	/**
@@ -110,20 +110,20 @@ 
 	 */
 	public function options() {
 
-		if ( ! empty( $this->args['show_option_none'] ) ) {
+		if ( ! empty( $this->args[ 'show_option_none' ] ) ) {
 
-			echo '<option value="-1"', selected( '-1', $this->args['selected'], false ), '>', esc_html( $this->args['show_option_none'] ), '</option>';
+			echo '<option value="-1"', selected( '-1', $this->args[ 'selected' ], false ), '>', esc_html( $this->args[ 'show_option_none' ] ), '</option>';
 		}
 
 		foreach ( $this->options as $value => $option ) {
 
-			if ( isset( $this->args['values_key'], $this->args['options_key'] ) ) {
+			if ( isset( $this->args[ 'values_key' ], $this->args[ 'options_key' ] ) ) {
 				$option = (array) $option;
-				$value = $option[ $this->args['values_key'] ];
-				$option = $option[ $this->args['options_key'] ];
+				$value = $option[ $this->args[ 'values_key' ] ];
+				$option = $option[ $this->args[ 'options_key' ] ];
 			}
 
-			echo '<option value="', esc_attr( $value ), '"', selected( $value, $this->args['selected'], false ), '>', esc_html( $option ), '</option>';
+			echo '<option value="', esc_attr( $value ), '"', selected( $value, $this->args[ 'selected' ], false ), '>', esc_html( $option ), '</option>';
 		}
 	}
 

src/classes/module/paths.php

Spacing +2 added lines, -2 removed lines

@@ -119,8 +119,8 @@
 		}
 
 		foreach ( self::$paths as $path ) {
-			if ( 0 === strpos( $file, $path['module_realpath'] ) ) {
-				$file = $path['module_path'] . substr( $file, $path['realpath_length'] );
+			if ( 0 === strpos( $file, $path[ 'module_realpath' ] ) ) {
+				$file = $path[ 'module_path' ] . substr( $file, $path[ 'realpath_length' ] );
 			}
 		}
 

src/classes/widget.php

Spacing +11 added lines, -11 removed lines

@@ -62,7 +62,7 @@ 
 			$message = $message->get_error_message();
 		}
 
-		echo $args['before_widget']; // XSS OK here, WPCS.
+		echo $args[ 'before_widget' ]; // XSS OK here, WPCS.
 
 		?>
 
@@ -86,7 +86,7 @@ 
 
 		<?php
 
-		echo $args['after_widget']; // XSs OK here too, WPCS.
+		echo $args[ 'after_widget' ]; // XSs OK here too, WPCS.
 	}
 
 	/**
@@ -106,8 +106,8 @@ 
 	 */
 	protected function verify_settings( $instance ) {
 
-		if ( ! isset( $instance['title'] ) ) {
-			$instance['title'] = '';
+		if ( ! isset( $instance[ 'title' ] ) ) {
+			$instance[ 'title' ] = '';
 		}
 
 		return $instance;
@@ -130,7 +130,7 @@ 
 			return;
 		}
 
-		echo $args['before_widget']; // XSS OK here, WPCS.
+		echo $args[ 'before_widget' ]; // XSS OK here, WPCS.
 
 		/**
 		 * The widget's title.
@@ -139,14 +139,14 @@ 
 		 *
 		 * @param string $title The widget title.
 		 */
-		$title = apply_filters( 'widget_title', $instance['title'] );
+		$title = apply_filters( 'widget_title', $instance[ 'title' ] );
 
 		if ( ! empty( $title ) ) {
 
-			echo $args['before_title'] . $title . $args['after_title']; // XSS OK, WPCS.
+			echo $args[ 'before_title' ] . $title . $args[ 'after_title' ]; // XSS OK, WPCS.
 		}
 
-		$widget_slug = $this->widget_options['wordpoints_hook_slug'];
+		$widget_slug = $this->widget_options[ 'wordpoints_hook_slug' ];
 
 		/**
 		 * Before a WordPoints widget.
@@ -168,7 +168,7 @@ 
 		 */
 		do_action( "wordpoints_{$widget_slug}_widget_after", $instance );
 
-		echo $args['after_widget']; // XSS OK here too, WPCS.
+		echo $args[ 'after_widget' ]; // XSS OK here too, WPCS.
 	}
 
 	/**
@@ -206,7 +206,7 @@ 
 	 * @since 2.0.0
 	 */
 	public function update_title() {
-		$this->instance['title'] = strip_tags( $this->instance['title'] );
+		$this->instance[ 'title' ] = strip_tags( $this->instance[ 'title' ] );
 	}
 
 	/**
@@ -232,7 +232,7 @@ 
 
 		<p>
 			<label for="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>"><?php echo esc_html_x( 'Title', 'form label', 'wordpoints' ); ?></label>
-			<input type="text" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'title' ) ); ?>" value="<?php echo esc_attr( $this->instance['title'] ); ?>" />
+			<input type="text" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'title' ) ); ?>" value="<?php echo esc_attr( $this->instance[ 'title' ] ); ?>" />
 		</p>
 
 		<?php

src/classes/multisite/switched/state.php

Spacing +14 added lines, -14 removed lines

@@ -38,17 +38,17 @@ 
 	 */
 	public function backup() {
 
-		$this->backup['original_blog_id'] = get_current_blog_id();
+		$this->backup[ 'original_blog_id' ] = get_current_blog_id();
 
-		if ( isset( $GLOBALS['_wp_switched_stack'] ) ) {
-			$this->backup['switched_stack'] = $GLOBALS['_wp_switched_stack'];
+		if ( isset( $GLOBALS[ '_wp_switched_stack' ] ) ) {
+			$this->backup[ 'switched_stack' ] = $GLOBALS[ '_wp_switched_stack' ];
 		}
 
-		if ( isset( $GLOBALS['switched'] ) ) {
-			$this->backup['switched'] = $GLOBALS['switched'];
+		if ( isset( $GLOBALS[ 'switched' ] ) ) {
+			$this->backup[ 'switched' ] = $GLOBALS[ 'switched' ];
 		}
 
-		return $this->backup['original_blog_id'];
+		return $this->backup[ 'original_blog_id' ];
 	}
 
 	/**
@@ -62,21 +62,21 @@ 
 	 */
 	public function restore() {
 
-		switch_to_blog( $this->backup['original_blog_id'] );
+		switch_to_blog( $this->backup[ 'original_blog_id' ] );
 
-		if ( isset( $this->backup['switched_stack'] ) ) {
-			$GLOBALS['_wp_switched_stack'] = $this->backup['switched_stack'];
+		if ( isset( $this->backup[ 'switched_stack' ] ) ) {
+			$GLOBALS[ '_wp_switched_stack' ] = $this->backup[ 'switched_stack' ];
 		} else {
-			unset( $GLOBALS['_wp_switched_stack'] );
+			unset( $GLOBALS[ '_wp_switched_stack' ] );
 		}
 
-		if ( isset( $this->backup['switched'] ) ) {
-			$GLOBALS['switched'] = $this->backup['switched'];
+		if ( isset( $this->backup[ 'switched' ] ) ) {
+			$GLOBALS[ 'switched' ] = $this->backup[ 'switched' ];
 		} else {
-			unset( $GLOBALS['switched'] );
+			unset( $GLOBALS[ 'switched' ] );
 		}
 
-		return $this->backup['original_blog_id'];
+		return $this->backup[ 'original_blog_id' ];
 	}
 }
 

src/admin/includes/class-wordpoints-modules-list-table.php

Spacing +57 added lines, -57 removed lines

@@ -48,7 +48,7 @@ 
 		parent::__construct(
 			array(
 				'plural' => 'modules',
-				'screen' => ( isset( $args['screen'] ) ) ? $args['screen'] : null,
+				'screen' => ( isset( $args[ 'screen' ] ) ) ? $args[ 'screen' ] : null,
 			)
 		);
 
@@ -68,12 +68,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();
@@ -88,7 +88,7 @@ 
 	 */
 	public function get_table_classes() {
 
-		return array( 'widefat', $this->_args['plural'], 'plugins' );
+		return array( 'widefat', $this->_args[ 'plural' ], 'plugins' );
 	}
 
 	/**
@@ -180,7 +180,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 (
@@ -190,9 +190,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 (
@@ -201,9 +201,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 (
@@ -216,16 +216,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 ().
 
@@ -234,7 +234,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' ) );
 		}
 
 		/**
@@ -311,8 +311,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 ) {
@@ -364,7 +364,7 @@ 
 
 		global $modules;
 
-		if ( ! empty( $modules['all'] ) ) {
+		if ( ! empty( $modules[ 'all' ] ) ) {
 			esc_html_e( 'No modules found.', 'wordpoints' );
 		} else {
 			esc_html_e( 'There are not any modules installed.', 'wordpoints' );
@@ -478,11 +478,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 (
@@ -490,7 +490,7 @@ 
 			&& current_user_can( 'delete_wordpoints_modules' )
 			&& 'active' !== $status
 		) {
-			$actions['delete-selected'] = esc_html__( 'Delete', 'wordpoints' );
+			$actions[ 'delete-selected' ] = esc_html__( 'Delete', 'wordpoints' );
 		}
 
 		/**
@@ -578,7 +578,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
@@ -618,7 +618,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' ) ) {
 
@@ -631,11 +631,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 ) {
 
@@ -672,11 +672,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
 
@@ -684,13 +684,13 @@ 
 						sprintf(
 							// translators: Module 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
@@ -709,7 +709,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; ?>
@@ -735,8 +735,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; ?>
@@ -747,25 +747,25 @@ 
 
 				$module_meta = array();
 
-				if ( ! empty( $module_data['version'] ) ) {
+				if ( ! empty( $module_data[ 'version' ] ) ) {
 					// translators: Module 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['module_uri'] ) ) {
-					$module_meta[] = '<a href="' . esc_url( $module_data['module_uri'] ) . '">' . esc_html__( 'Visit module site', 'wordpoints' ) . '</a>';
+				if ( ! empty( $module_data[ 'module_uri' ] ) ) {
+					$module_meta[ ] = '<a href="' . esc_url( $module_data[ 'module_uri' ] ) . '">' . esc_html__( 'Visit module site', 'wordpoints' ) . '</a>';
 				}
 
 				/**
@@ -781,7 +781,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' );
 
@@ -818,7 +818,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. ?>
@@ -847,8 +847,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(
@@ -861,49 +861,49 @@ 
 
 		if ( $this->screen->in_admin( 'network' ) ) {
 
-			if ( $args['is_active'] ) {
+			if ( $args[ 'is_active' ] ) {
 
 				if ( current_user_can( 'manage_network_wordpoints_modules' ) ) {
 					// translators: Module 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_modules' ) ) {
 					// translators: Module 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_modules' ) && ! is_wordpoints_module_active( $module_file ) ) {
 					// translators: Module name.
-					$actions['delete'] = '<a href="' . wp_nonce_url( self_admin_url( 'admin.php?page=wordpoints_modules&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_modules&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: Module 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: Module 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_modules' ) ) {
 					// translators: Module name.
-					$actions['delete'] = '<a href="' . wp_nonce_url( self_admin_url( 'admin.php?page=wordpoints_modules&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_modules&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">