WordPoints / wordpoints

src/includes/maintenance.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
 }
 
 // If we're not running a module check, let the maintenance message show.
-if ( ! isset( $_GET['wordpoints_module_check'], $_GET['check_module'] ) ) { // WPCS: CSRF OK.
+if ( ! isset( $_GET[ 'wordpoints_module_check' ], $_GET[ 'check_module' ] ) ) { // WPCS: CSRF OK.
 	return;
 }
 

src/admin/classes/ajax/hooks.php

Spacing +6 added lines, -6 removed lines

@@ -316,7 +316,7 @@ 
 	private function verify_request( $action ) {
 
 		if (
-			empty( $_POST['nonce'] )
+			empty( $_POST[ 'nonce' ] )
 			|| ! wordpoints_verify_nonce( 'nonce', $action, null, 'post' )
 		) {
 			wp_send_json_error(
@@ -334,11 +334,11 @@ 
 	 */
 	protected function get_reaction_store() {
 
-		if ( ! isset( $_POST['reaction_store'] ) ) { // WPCS: CSRF OK.
+		if ( ! isset( $_POST[ 'reaction_store' ] ) ) { // WPCS: CSRF OK.
 			$this->unexpected_error( 'reaction_store' );
 		}
 
-		$reactor_slug = sanitize_key( $_POST['reaction_store'] ); // WPCS: CSRF OK.
+		$reactor_slug = sanitize_key( $_POST[ 'reaction_store' ] ); // WPCS: CSRF OK.
 
 		$reaction_store = wordpoints_hooks()->get_reaction_store( $reactor_slug );
 
@@ -361,12 +361,12 @@ 
 	 */
 	protected function get_reaction() {
 
-		if ( ! isset( $_POST['id'] ) ) { // WPCS: CSRF OK.
+		if ( ! isset( $_POST[ 'id' ] ) ) { // WPCS: CSRF OK.
 			$this->unexpected_error( 'id' );
 		}
 
 		$reaction = $this->reaction_store->get_reaction(
-			wordpoints_int( $_POST['id'] ) // WPCS: CSRF OK.
+			wordpoints_int( $_POST[ 'id' ] ) // WPCS: CSRF OK.
 		);
 
 		if ( ! $reaction ) {
@@ -387,7 +387,7 @@ 
 
 		$data = wp_unslash( $_POST ); // WPCS: CSRF OK.
 
-		unset( $data['id'], $data['action'], $data['nonce'], $data['reaction_store'] );
+		unset( $data[ 'id' ], $data[ 'action' ], $data[ 'nonce' ], $data[ 'reaction_store' ] );
 
 		return $data;
 	}

src/admin/screens/configure-components-load.php

Spacing +3 added lines, -3 removed lines

@@ -13,15 +13,15 @@
 
 if (
 	'components' !== wordpoints_admin_get_current_tab()
-	|| ! isset( $_POST['wordpoints_component'], $_POST['wordpoints_component_action'], $_POST['_wpnonce'] )
+	|| ! isset( $_POST[ 'wordpoints_component' ], $_POST[ 'wordpoints_component_action' ], $_POST[ '_wpnonce' ] )
 ) {
 	return;
 }
 
 $components = WordPoints_Components::instance();
-$component  = sanitize_key( $_POST['wordpoints_component'] );
+$component  = sanitize_key( $_POST[ 'wordpoints_component' ] );
 
-$action = sanitize_key( $_POST['wordpoints_component_action'] );
+$action = sanitize_key( $_POST[ 'wordpoints_component_action' ] );
 
 check_admin_referer( "wordpoints_{$action}_component-{$component}" );
 

src/admin/screens/configure-settings.php

Spacing +2 added lines, -2 removed lines

@@ -12,13 +12,13 @@
 }
 
 if (
-	isset( $_POST['wordpoints_settings_nonce'], $_POST['excluded_users'] )
+	isset( $_POST[ 'wordpoints_settings_nonce' ], $_POST[ 'excluded_users' ] )
 	&& wordpoints_verify_nonce( 'wordpoints_settings_nonce', 'wordpoints_settings_submit', null, 'post' )
 ) {
 
 	// - The form has been submitted.
 
-	$excluded_users = preg_replace( '/\s/', '', sanitize_text_field( wp_unslash( $_POST['excluded_users'] ) ) );
+	$excluded_users = preg_replace( '/\s/', '', sanitize_text_field( wp_unslash( $_POST[ 'excluded_users' ] ) ) );
 
 	if ( ! empty( $excluded_users ) ) {
 

src/admin/screens/configure-components.php

Spacing +24 added lines, -24 removed lines

@@ -17,13 +17,13 @@ 
 // Show messages and errors.
 //
 
-if ( isset( $_GET['wordpoints_component'], $_GET['_wpnonce'] ) && $wordpoints_components->is_registered( sanitize_key( $_GET['wordpoints_component'] ) ) ) { // WPCS: CSRF OK.
+if ( isset( $_GET[ 'wordpoints_component' ], $_GET[ '_wpnonce' ] ) && $wordpoints_components->is_registered( sanitize_key( $_GET[ 'wordpoints_component' ] ) ) ) { // WPCS: CSRF OK.
 
-	$component = sanitize_key( $_GET['wordpoints_component'] );
+	$component = sanitize_key( $_GET[ 'wordpoints_component' ] );
 
-	if ( isset( $_GET['message'] ) && wordpoints_verify_nonce( '_wpnonce', "wordpoints_component_message-{$component}" ) ) {
+	if ( isset( $_GET[ 'message' ] ) && wordpoints_verify_nonce( '_wpnonce', "wordpoints_component_message-{$component}" ) ) {
 
-		switch ( (int) $_GET['message'] ) {
+		switch ( (int) $_GET[ 'message' ] ) {
 
 			case 1:
 				if ( $wordpoints_components->is_active( $component ) ) {
@@ -43,15 +43,15 @@ 
 		if ( isset( $message ) ) {
 
 			wordpoints_show_admin_message(
-				esc_html( sprintf( $message, $components[ $component ]['name'] ) )
+				esc_html( sprintf( $message, $components[ $component ][ 'name' ] ) )
 				, 'success'
 				, array( 'dismissible' => true )
 			);
 		}
 
-	} elseif ( isset( $_GET['error'] ) && wordpoints_verify_nonce( '_wpnonce', "wordpoints_component_error-{$component}" ) ) {
+	} elseif ( isset( $_GET[ 'error' ] ) && wordpoints_verify_nonce( '_wpnonce', "wordpoints_component_error-{$component}" ) ) {
 
-		switch ( (int) $_GET['error'] ) {
+		switch ( (int) $_GET[ 'error' ] ) {
 
 			case 1:
 				if ( ! $wordpoints_components->is_active( $component ) ) {
@@ -71,7 +71,7 @@ 
 		if ( isset( $error ) ) {
 
 			wordpoints_show_admin_error(
-				esc_html( sprintf( $error, $components[ $component ]['name'] ) )
+				esc_html( sprintf( $error, $components[ $component ][ 'name' ] ) )
 				, array( 'dismissible' => true )
 			);
 		}
@@ -109,7 +109,7 @@ 
 
 		<?php
 
-		if ( $wordpoints_components->is_active( $component['slug'] ) ) {
+		if ( $wordpoints_components->is_active( $component[ 'slug' ] ) ) {
 
 			$action = 'deactivate';
 			$button = __( 'Deactivate', 'wordpoints' );
@@ -124,38 +124,38 @@ 
 
 		<tr>
 			<td>
-				<?php if ( '' !== $component['component_uri'] ) : ?>
-					<a href="<?php echo esc_url( $component['component_uri'] ); ?>">
+				<?php if ( '' !== $component[ 'component_uri' ] ) : ?>
+					<a href="<?php echo esc_url( $component[ 'component_uri' ] ); ?>">
 				<?php endif; ?>
-					<?php echo esc_html( $component['name'] ); ?>
-				<?php if ( '' !== $component['component_uri'] ) : ?>
+					<?php echo esc_html( $component[ 'name' ] ); ?>
+				<?php if ( '' !== $component[ 'component_uri' ] ) : ?>
 					</a>
 				<?php endif; ?>
 			</td>
 			<td>
-				<?php echo wp_kses( $component['description'], 'wordpoints_component_description' ); ?>
-				<?php if ( '' !== $component['author'] ) : ?>
+				<?php echo wp_kses( $component[ 'description' ], 'wordpoints_component_description' ); ?>
+				<?php if ( '' !== $component[ 'author' ] ) : ?>
 					&nbsp;|&nbsp;
 					<?php
 					// translators: Author name.
 					echo esc_html( sprintf( __( 'By %s', 'wordpoints' ), '' /* This space intentionally left blank */ ) );
 					?>
-					<?php if ( '' !== $component['author_uri'] ) : ?>
-						<a href="<?php echo esc_url( $component['author_uri'] ); ?>">
+					<?php if ( '' !== $component[ 'author_uri' ] ) : ?>
+						<a href="<?php echo esc_url( $component[ 'author_uri' ] ); ?>">
 					<?php endif; ?>
-						<?php echo esc_html( $component['author'] ); ?>
-					<?php if ( '' !== $component['author_uri'] ) : ?>
+						<?php echo esc_html( $component[ 'author' ] ); ?>
+					<?php if ( '' !== $component[ 'author_uri' ] ) : ?>
 						</a>
 					<?php endif; ?>
 				<?php endif; ?>
 			</td>
-			<td><?php echo esc_html( $component['version'] ); ?></td>
+			<td><?php echo esc_html( $component[ 'version' ] ); ?></td>
 			<td>
-				<form method="post" name="wordpoints_components_form_<?php echo esc_attr( $component['slug'] ); ?>">
+				<form method="post" name="wordpoints_components_form_<?php echo esc_attr( $component[ 'slug' ] ); ?>">
 					<input type="hidden" name="wordpoints_component_action" value="<?php echo esc_attr( $action ); ?>" />
-					<input type="hidden" name="wordpoints_component" value="<?php echo esc_attr( $component['slug'] ); ?>" />
-					<?php wp_nonce_field( "wordpoints_{$action}_component-{$component['slug']}" ); ?>
-					<?php submit_button( $button, "secondary wordpoints-component-{$action}", "wordpoints-component-{$action}_{$component['slug']}", false ); ?>
+					<input type="hidden" name="wordpoints_component" value="<?php echo esc_attr( $component[ 'slug' ] ); ?>" />
+					<?php wp_nonce_field( "wordpoints_{$action}_component-{$component[ 'slug' ]}" ); ?>
+					<?php submit_button( $button, "secondary wordpoints-component-{$action}", "wordpoints-component-{$action}_{$component[ 'slug' ]}", false ); ?>
 				</form>
 			</td>
 		</tr>

src/components/points/includes/logs.php

Spacing +18 added lines, -18 removed lines

@@ -89,7 +89,7 @@ 
 
 	// The current user needs to be set dynamically, since it can change at times.
 	if ( 'current_user' === $query_slug ) {
-		$args['user_id'] = get_current_user_id();
+		$args[ 'user_id' ] = get_current_user_id();
 	}
 
 	/**
@@ -137,12 +137,12 @@ 
 
 	$query_data = WordPoints_Points_Log_Queries::get_query_data( $query_slug );
 
-	if ( $query_data['cache_queries'] ) {
+	if ( $query_data[ 'cache_queries' ] ) {
 
 		$query->prime_cache(
-			$query_data['cache_key']
+			$query_data[ 'cache_key' ]
 			, null
-			, $query_data['network_wide']
+			, $query_data[ 'network_wide' ]
 		);
 	}
 
@@ -196,7 +196,7 @@ 
 
 	$args = array_merge( $defaults, $args );
 
-	if ( ! $args['datatable'] ) {
+	if ( ! $args[ 'datatable' ] ) {
 
 		_deprecated_argument(
 			__FUNCTION__
@@ -204,13 +204,13 @@ 
 			, '$args["datatable"] is deprecated and should no longer be used. Use $args["paginate"] instead.'
 		);
 
-		$args['paginate'] = false;
+		$args[ 'paginate' ] = false;
 	}
 
 	$view = wordpoints_component( 'points' )
 		->get_sub_app( 'logs' )
 		->get_sub_app( 'views' )
-		->get( $args['view'], array( $logs_query, $args ) );
+		->get( $args[ 'view' ], array( $logs_query, $args ) );
 
 	if ( ! $view instanceof WordPoints_Points_Logs_View ) {
 		return;
@@ -284,10 +284,10 @@ 
  */
 function wordpoints_points_logs_profile_edit( $text, $points, $points_type, $user_id, $log_type, $meta ) {
 
-	$user_name = sanitize_user_field( 'display_name', get_userdata( $meta['user_id'] )->display_name, $meta['user_id'], 'display' );
+	$user_name = sanitize_user_field( 'display_name', get_userdata( $meta[ 'user_id' ] )->display_name, $meta[ 'user_id' ], 'display' );
 
 	// translators: 1. User name; 2. Reason given.
-	return sprintf( _x( 'Points adjusted by %1$s. Reason: %2$s', 'points log description', 'wordpoints' ), $user_name, esc_html( $meta['reason'] ) );
+	return sprintf( _x( 'Points adjusted by %1$s. Reason: %2$s', 'points log description', 'wordpoints' ), $user_name, esc_html( $meta[ 'reason' ] ) );
 }
 
 /**
@@ -299,7 +299,7 @@ 
  */
 function wordpoints_points_logs_comment_disapprove( $text, $points, $points_type, $user_id, $log_type, $meta ) {
 
-	switch ( $meta['status'] ) {
+	switch ( $meta[ 'status' ] ) {
 
 		case 'spam':
 			$text = _x( 'Comment marked as spam.', 'points log description', 'wordpoints' );
@@ -325,9 +325,9 @@ 
  */
 function wordpoints_points_logs_post_delete( $text, $points, $points_type, $user_id, $log_type, $meta ) {
 
-	if ( isset( $meta['post_type'] ) ) {
+	if ( isset( $meta[ 'post_type' ] ) ) {
 
-		$post_type = get_post_type_object( $meta['post_type'] );
+		$post_type = get_post_type_object( $meta[ 'post_type' ] );
 
 		if ( ! is_null( $post_type ) ) {
 
@@ -389,18 +389,18 @@ 
 		'%user_id%',
 	);
 
-	if ( empty( $args['points_type'] ) ) {
+	if ( empty( $args[ 'points_type' ] ) ) {
 		$points_types = array_keys( wordpoints_get_points_types() );
 	} else {
-		$points_types = (array) $args['points_type'];
+		$points_types = (array) $args[ 'points_type' ];
 	}
 
 	foreach ( $points_types as $points_type ) {
 		foreach ( WordPoints_Points_Log_Queries::get_queries() as $query ) {
 
-			if ( ! empty( $query['cache_key'] ) ) {
+			if ( ! empty( $query[ 'cache_key' ] ) ) {
 
-				if ( $query['network_wide'] ) {
+				if ( $query[ 'network_wide' ] ) {
 					$group = 'wordpoints_network_points_logs_query';
 				} else {
 					$group = 'wordpoints_points_logs_query';
@@ -408,11 +408,11 @@ 
 
 				$replace = array(
 					$points_type,
-					$args['user_id'],
+					$args[ 'user_id' ],
 				);
 
 				wp_cache_delete(
-					str_replace( $find, $replace, $query['cache_key'] )
+					str_replace( $find, $replace, $query[ 'cache_key' ] )
 					, $group
 				);
 			}

src/components/points/includes/deprecated.php

Spacing +4 added lines, -4 removed lines

@@ -184,13 +184,13 @@ 
 
 				$instance = array_merge( $this->defaults, $instance );
 
-				if ( ! $this->is_matching_post_type( $post->post_type, $instance['post_type'] ) ) {
+				if ( ! $this->is_matching_post_type( $post->post_type, $instance[ 'post_type' ] ) ) {
 					continue;
 				}
 
 				$points_type = $this->points_type( $number );
 
-				wordpoints_subtract_points( $comment->user_id, $instance['points'], $points_type, 'comment_disapprove', array( 'status' => $new_status ) );
+				wordpoints_subtract_points( $comment->user_id, $instance[ 'points' ], $points_type, 'comment_disapprove', array( 'status' => $new_status ) );
 
 				update_comment_meta(
 					$comment->comment_ID
@@ -375,14 +375,14 @@ 
 			$instance = array_merge( $this->defaults, $instance );
 
 			if (
-				$this->is_matching_post_type( $post->post_type, $instance['post_type'] )
+				$this->is_matching_post_type( $post->post_type, $instance[ 'post_type' ] )
 				&& 'auto-draft' !== $post->post_status
 				&& __( 'Auto Draft', 'default' ) !== $post->post_title
 			) {
 
 				wordpoints_alter_points(
 					$post->post_author
-					, -$instance['points']
+					, -$instance[ 'points' ]
 					, $this->points_type( $number )
 					, 'post_delete'
 					, array( 'post_title' => $post->post_title, 'post_type' => $post->post_type )

src/components/points/includes/hooks/periodic.php

Spacing +12 added lines, -12 removed lines

@@ -85,15 +85,15 @@ 
 
 			if (
 				! isset( $last_visit[ $points_type ] )
-				|| (int) ( $last_visit[ $points_type ] / $instance['period'] ) < (int) ( $now / $instance['period'] )
+				|| (int) ( $last_visit[ $points_type ] / $instance[ 'period' ] ) < (int) ( $now / $instance[ 'period' ] )
 			) {
 
 				wordpoints_add_points(
 					$user_id
-					, $instance['points']
+					, $instance[ 'points' ]
 					, $points_type
 					, 'periodic'
-					, array( 'period' => $instance['period'] )
+					, array( 'period' => $instance[ 'period' ] )
 				);
 
 				$last_visit[ $points_type ] = $now;
@@ -133,7 +133,7 @@ 
 	 */
 	public function logs( $text, $points, $points_type, $user_id, $log_type, $meta ) {
 
-		switch ( $meta['period'] ) {
+		switch ( $meta[ 'period' ] ) {
 
 			case HOUR_IN_SECONDS:
 				$message = _x( 'Hourly points.', 'points log description', 'wordpoints' );
@@ -169,9 +169,9 @@ 
 	 */
 	protected function generate_description( $instance = array() ) {
 
-		if ( ! empty( $instance['period'] ) ) {
+		if ( ! empty( $instance[ 'period' ] ) ) {
 
-			switch ( $instance['period'] ) {
+			switch ( $instance[ 'period' ] ) {
 
 				case HOUR_IN_SECONDS:
 					return __( 'Visiting the site at least once in an hour.', 'wordpoints' );
@@ -207,8 +207,8 @@ 
 
 		$new_instance = array_merge( $this->defaults, $old_instance, $new_instance );
 
-		wordpoints_posint( $new_instance['points'] );
-		wordpoints_posint( $new_instance['period'] );
+		wordpoints_posint( $new_instance[ 'points' ] );
+		wordpoints_posint( $new_instance[ 'period' ] );
 
 		return $new_instance;
 	}
@@ -227,7 +227,7 @@ 
 		$instance = array_merge( $this->defaults, $instance );
 
 		$dropdown_args = array(
-			'selected' => $instance['period'],
+			'selected' => $instance[ 'period' ],
 			'id'       => $this->get_field_id( 'period' ),
 			'name'     => $this->get_field_name( 'period' ),
 			'class'    => 'widefat wordpoints-append-to-hook-title',
@@ -261,9 +261,9 @@ 
 	protected function get_periods() {
 
 		$periods = array(
-			HOUR_IN_SECONDS     => __( 'hourly',  'wordpoints' ),
-			DAY_IN_SECONDS      => __( 'daily',   'wordpoints' ),
-			WEEK_IN_SECONDS     => __( 'weekly',  'wordpoints' ),
+			HOUR_IN_SECONDS     => __( 'hourly', 'wordpoints' ),
+			DAY_IN_SECONDS      => __( 'daily', 'wordpoints' ),
+			WEEK_IN_SECONDS     => __( 'weekly', 'wordpoints' ),
 			30 * DAY_IN_SECONDS => __( 'monthly', 'wordpoints' ),
 		);
 

src/components/points/includes/hooks/post.php

Spacing +5 added lines, -5 removed lines

@@ -111,22 +111,22 @@
 			$points_type = $this->points_type( $number );
 
 			if (
-				$this->is_matching_post_type( $post->post_type, $instance['post_type'] )
+				$this->is_matching_post_type( $post->post_type, $instance[ 'post_type' ] )
 				&& ! $this->awarded_points_already( $post->ID, $points_type )
 			) {
 
-				if ( isset( $instance['publish'] ) ) {
+				if ( isset( $instance[ 'publish' ] ) ) {
 					_deprecated_argument( __METHOD__, '1.4.0', 'The "publish" hook setting is no longer used to hold the value for the points. Use "points" instead.' );
-					$instance['points'] = $instance['publish'];
+					$instance[ 'points' ] = $instance[ 'publish' ];
 				}
 
-				if ( ! isset( $instance['points'] ) ) {
+				if ( ! isset( $instance[ 'points' ] ) ) {
 					continue;
 				}
 
 				wordpoints_alter_points(
 					$post->post_author
-					, $instance['points']
+					, $instance[ 'points' ]
 					, $points_type
 					, 'post_publish'
 					, array( 'post_id' => $post->ID )