WordPoints / 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 )

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

Spacing +2 added lines, -2 removed lines

@@ -59,8 +59,8 @@
 
 		foreach ( $this->get_instances() as $number => $instance ) {
 
-			if ( isset( $instance['points'] ) ) {
-				wordpoints_add_points( $user_id, $instance['points'], $this->points_type( $number ), 'register' );
+			if ( isset( $instance[ 'points' ] ) ) {
+				wordpoints_add_points( $user_id, $instance[ 'points' ], $this->points_type( $number ), 'register' );
 			}
 		}
 	}

src/components/points/includes/hooks/abstracts/comment-approved.php

Spacing +4 added lines, -4 removed lines

@@ -86,14 +86,14 @@ 
 
 			if (
 				'approved' === $last_status
-				|| ! $this->is_matching_post_type( $post->post_type, $instance['post_type'] )
+				|| ! $this->is_matching_post_type( $post->post_type, $instance[ 'post_type' ] )
 			) {
 				continue;
 			}
 
 			wordpoints_add_points(
 				$this->select_user_to_award( $comment, $post )
-				, $instance['points']
+				, $instance[ 'points' ]
 				, $points_type
 				, $this->log_type
 				, array( 'comment_id' => $comment->comment_ID )
@@ -258,8 +258,8 @@ 
 
 		$comment = false;
 
-		if ( isset( $meta['comment_id'] ) ) {
-			$comment = get_comment( $meta['comment_id'] );
+		if ( isset( $meta[ 'comment_id' ] ) ) {
+			$comment = get_comment( $meta[ 'comment_id' ] );
 		}
 
 		if ( ! $comment ) {

src/components/points/includes/class-wordpoints-points-logs-query.php

Spacing +16 added lines, -16 removed lines

@@ -183,12 +183,12 @@ 
 
 		$this->table_name = $wpdb->wordpoints_points_logs;
 
-		$this->columns['points_type']['values'] = array_keys(
+		$this->columns[ 'points_type' ][ 'values' ] = array_keys(
 			wordpoints_get_points_types()
 		);
 
-		$this->defaults['order_by'] = 'date';
-		$this->defaults['text__compare'] = 'LIKE';
+		$this->defaults[ 'order_by' ] = 'date';
+		$this->defaults[ 'text__compare' ] = 'LIKE';
 
 		// Back-compat for pre-2.3.0, in case an object or string is passed.
 		$args = wp_parse_args( $args );
@@ -223,7 +223,7 @@ 
 	protected function convert_deprecated_arg_values( $args ) {
 
 		// Back-compat for pre-2.3.0, when the fields arg supported 'all'.
-		if ( isset( $args['fields'] ) && 'all' === $args['fields'] ) {
+		if ( isset( $args[ 'fields' ] ) && 'all' === $args[ 'fields' ] ) {
 
 			_deprecated_argument(
 				__METHOD__
@@ -231,11 +231,11 @@ 
 				, esc_html( "Passing 'fields' => 'all' is deprecated, just omit the 'fields' arg instead, since all fields returned by default." )
 			);
 
-			unset( $args['fields'] );
+			unset( $args[ 'fields' ] );
 		}
 
 		// Back-compat for pre-2.3.0, when the orderby arg supported 'none'.
-		if ( isset( $args['orderby'] ) && 'none' === $args['orderby'] ) {
+		if ( isset( $args[ 'orderby' ] ) && 'none' === $args[ 'orderby' ] ) {
 
 			_deprecated_argument(
 				__METHOD__
@@ -243,8 +243,8 @@ 
 				, esc_html( "Passing 'orderby' => 'none' is deprecated, pass 'order_by' => null instead." )
 			);
 
-			$args['order_by'] = null;
-			unset( $args['orderby'] );
+			$args[ 'order_by' ] = null;
+			unset( $args[ 'orderby' ] );
 
 			return $args;
 		}
@@ -367,7 +367,7 @@ 
 			if ( false !== $cache ) {
 				return array_slice(
 					$cache
-					, $start - $this->args['start']
+					, $start - $this->args[ 'start' ]
 					, $per_page
 				);
 			}
@@ -376,14 +376,14 @@ 
 		// Stash the args so we can restore them later.
 		$args = $this->args;
 
-		$this->args['start'] += $start;
+		$this->args[ 'start' ] += $start;
 
-		if ( ! empty( $this->args['limit'] ) ) {
-			$this->args['limit'] -= $start;
+		if ( ! empty( $this->args[ 'limit' ] ) ) {
+			$this->args[ 'limit' ] -= $start;
 		}
 
-		if ( empty( $this->args['limit'] ) || $this->args['limit'] > $per_page ) {
-			$this->args['limit'] = $per_page;
+		if ( empty( $this->args[ 'limit' ] ) || $this->args[ 'limit' ] > $per_page ) {
+			$this->args[ 'limit' ] = $per_page;
 		}
 
 		// Regenerate the query limit after changing the start and limit args.
@@ -466,8 +466,8 @@ 
 				'%user_id%',
 			)
 			, array(
-				isset( $this->args['points_type'] ) ? $this->args['points_type'] : '',
-				isset( $this->args['user_id'] ) ? $this->args['user_id'] : 0,
+				isset( $this->args[ 'points_type' ] ) ? $this->args[ 'points_type' ] : '',
+				isset( $this->args[ 'user_id' ] ) ? $this->args[ 'user_id' ] : 0,
 			)
 			, $key
 		);

src/components/points/includes/functions.php

Spacing +12 added lines, -12 removed lines

@@ -182,19 +182,19 @@ 
 
 	$points_type = wordpoints_get_points_type( $type );
 
-	if ( isset( $points_type['prefix'] ) ) {
+	if ( isset( $points_type[ 'prefix' ] ) ) {
 
 		if ( $points < 0 ) {
 
 			$points = abs( $points );
-			$points_type['prefix'] = '-' . $points_type['prefix'];
+			$points_type[ 'prefix' ] = '-' . $points_type[ 'prefix' ];
 		}
 
-		$formatted = esc_html( $points_type['prefix'] . $points );
+		$formatted = esc_html( $points_type[ 'prefix' ] . $points );
 	}
 
-	if ( isset( $points_type['suffix'] ) ) {
-		$formatted = $formatted . esc_html( $points_type['suffix'] );
+	if ( isset( $points_type[ 'suffix' ] ) ) {
+		$formatted = $formatted . esc_html( $points_type[ 'suffix' ] );
 	}
 
 	return $formatted;
@@ -217,11 +217,11 @@ 
 
 	foreach ( wordpoints_get_points_types() as $slug => $settings ) {
 
-		$points_types[ $slug ] = $settings['name'];
+		$points_types[ $slug ] = $settings[ 'name' ];
 	}
 
-	if ( isset( $args['options'] ) && is_array( $args['options'] ) ) {
-		$points_types = $args['options'] + $points_types;
+	if ( isset( $args[ 'options' ] ) && is_array( $args[ 'options' ] ) ) {
+		$points_types = $args[ 'options' ] + $points_types;
 	}
 
 	$dropdown = new WordPoints_Dropdown_Builder( $points_types, $args );
@@ -246,7 +246,7 @@ 
 
 	// If the user is being deleted from all blogs on multisite.
 	if ( is_multisite() && ! get_userdata( $user_id ) ) {
-		$query_args['blog_id'] = 0;
+		$query_args[ 'blog_id' ] = 0;
 	}
 
 	// Delete log meta.
@@ -258,8 +258,8 @@ 
 
 	$where = array( 'user_id' => $user_id );
 
-	if ( ! isset( $query_args['blog_id'] ) ) {
-		$where['blog_id'] = $wpdb->blogid;
+	if ( ! isset( $query_args[ 'blog_id' ] ) ) {
+		$where[ 'blog_id' ] = $wpdb->blogid;
 	}
 
 	// Now delete the logs.
@@ -370,7 +370,7 @@ 
 		);
 
 		if ( is_wordpoints_network_active() ) {
-			$groups[] = 'wordpoints_points_top_users';
+			$groups[ ] = 'wordpoints_points_top_users';
 		}
 
 		wp_cache_add_global_groups( $groups );

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

Spacing +21 added lines, -21 removed lines

@@ -38,7 +38,7 @@ 
 	}
 
 	// Save hooks order for all points types.
-	if ( ! isset( $_POST['points_types'] ) || ! is_array( $_POST['points_types'] ) ) {
+	if ( ! isset( $_POST[ 'points_types' ] ) || ! is_array( $_POST[ 'points_types' ] ) ) {
 		wp_die( -1, '', array( 'response' => 400 ) );
 	}
 
@@ -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();
 
@@ -106,7 +106,7 @@ 
 
 	$error = __( 'An error has occurred. Please reload the page and try again.', 'wordpoints' );
 
-	if ( isset( $_POST['points-slug'] ) ) {
+	if ( isset( $_POST[ 'points-slug' ] ) ) {
 
 		// - We are saving the settings for a points type.
 
@@ -123,19 +123,19 @@ 
 
 		$settings = array();
 
-		if ( isset( $_POST['points-name'] ) ) {
-			$settings['name'] = trim( sanitize_text_field( wp_unslash( $_POST['points-name'] ) ) );
+		if ( isset( $_POST[ 'points-name' ] ) ) {
+			$settings[ 'name' ] = trim( sanitize_text_field( wp_unslash( $_POST[ 'points-name' ] ) ) );
 		}
 
-		if ( isset( $_POST['points-prefix'] ) ) {
-			$settings['prefix'] = ltrim( sanitize_text_field( wp_unslash( $_POST['points-prefix'] ) ) );
+		if ( isset( $_POST[ 'points-prefix' ] ) ) {
+			$settings[ 'prefix' ] = ltrim( sanitize_text_field( wp_unslash( $_POST[ 'points-prefix' ] ) ) );
 		}
 
-		if ( isset( $_POST['points-suffix'] ) ) {
-			$settings['suffix'] = rtrim( sanitize_text_field( wp_unslash( $_POST['points-suffix'] ) ) );
+		if ( isset( $_POST[ 'points-suffix' ] ) ) {
+			$settings[ 'suffix' ] = rtrim( sanitize_text_field( wp_unslash( $_POST[ 'points-suffix' ] ) ) );
 		}
 
-		$points_type = sanitize_key( $_POST['points-slug'] );
+		$points_type = sanitize_key( $_POST[ 'points-slug' ] );
 
 		$old_settings = wordpoints_get_points_type( $points_type );
 
@@ -159,14 +159,14 @@ 
 
 		// - We are creating/updating/deleting an instance of a hook.
 
-		if ( ! isset( $_POST['id_base'], $_POST['hook-id'], $_POST['points_type'], $_POST['hook_number'] ) ) {
+		if ( ! isset( $_POST[ 'id_base' ], $_POST[ 'hook-id' ], $_POST[ 'points_type' ], $_POST[ 'hook_number' ] ) ) {
 			wp_die( -1, '', array( 'response' => 400 ) );
 		}
 
-		$id_base        = sanitize_key( $_POST['id_base'] );
-		$hook_id        = sanitize_key( $_POST['hook-id'] );
-		$points_type_id = sanitize_key( $_POST['points_type'] );
-		$number         = (int) $_POST['hook_number'];
+		$id_base        = sanitize_key( $_POST[ 'id_base' ] );
+		$hook_id        = sanitize_key( $_POST[ 'hook-id' ] );
+		$points_type_id = sanitize_key( $_POST[ 'points_type' ] );
+		$number         = (int) $_POST[ 'hook_number' ];
 
 		/*
 		 * Normally the hook ID will be in 'hook-id' when we are updating a hook.
@@ -177,11 +177,11 @@ 
 		if ( ! $number ) {
 
 			// This holds the ID number if the hook is brand new.
-			if ( ! isset( $_POST['multi_number'] ) || ! wordpoints_posint( $_POST['multi_number'] ) ) {
+			if ( ! isset( $_POST[ 'multi_number' ] ) || ! wordpoints_posint( $_POST[ 'multi_number' ] ) ) {
 				wp_die( '<p>' . esc_html( $error ) . '</p>', '', array( 'response' => 400 ) );
 			}
 
-			$number  = (int) $_POST['multi_number'];
+			$number  = (int) $_POST[ 'multi_number' ];
 			$hook_id = $id_base . '-' . $number;
 		}
 
@@ -198,7 +198,7 @@ 
 		// Get the hooks for this points type.
 		$points_type_hooks = ( isset( $points_types_hooks[ $points_type_id ] ) ) ? $points_types_hooks[ $points_type_id ] : array();
 
-		if ( ! empty( $_POST['delete_hook'] ) ) {
+		if ( ! empty( $_POST[ 'delete_hook' ] ) ) {
 
 			// - We are deleting a hook instance.
 
@@ -229,7 +229,7 @@ 
 			$new_instance = reset( $settings );
 
 			// Save the points types-hooks associations.
-			$points_type_hooks[] = $hook->get_id( $number );
+			$points_type_hooks[ ] = $hook->get_id( $number );
 			$points_types_hooks[ $points_type_id ] = $points_type_hooks;
 			WordPoints_Points_Hooks::save_points_types_hooks( $points_types_hooks );
 
@@ -247,7 +247,7 @@ 
 
 		$hook->update_callback( wp_unslash( $new_instance ), $number );
 
-		if ( empty( $_POST['add_new'] ) ) {
+		if ( empty( $_POST[ 'add_new' ] ) ) {
 			$hook->form_callback( $number );
 		}
 

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

Spacing +30 added lines, -30 removed lines

@@ -267,7 +267,7 @@ 
 				<input class="widefat" type="text"
 				       id="points-name-<?php echo esc_attr( $slug ); ?>"
 				       name="points-name"
-				       value="<?php echo esc_attr( $points_type['name'] ); ?>"/>
+				       value="<?php echo esc_attr( $points_type[ 'name' ] ); ?>"/>
 			</p>
 			<p>
 				<label
@@ -275,7 +275,7 @@ 
 				<input class="widefat" type="text"
 				       id="points-prefix-<?php echo esc_attr( $slug ); ?>"
 				       name="points-prefix"
-				       value="<?php echo esc_attr( $points_type['prefix'] ); ?>"/>
+				       value="<?php echo esc_attr( $points_type[ 'prefix' ] ); ?>"/>
 			</p>
 			<p>
 				<label
@@ -283,7 +283,7 @@ 
 				<input class="widefat" type="text"
 				       id="points-suffix-<?php echo esc_attr( $slug ); ?>"
 				       name="points-suffix"
-				       value="<?php echo esc_attr( $points_type['suffix'] ); ?>"/>
+				       value="<?php echo esc_attr( $points_type[ 'suffix' ] ); ?>"/>
 			</p>
 
 			<?php
@@ -428,13 +428,13 @@ 
 	 */
 	public function display_event_meta_box( $points_type, $meta_box ) {
 
-		$event_slug = $meta_box['args']['slug'];
+		$event_slug = $meta_box[ 'args' ][ 'slug' ];
 
 		$data = array();
 
 		foreach ( $this->reaction_store->get_reactions_to_event( $event_slug ) as $id => $reaction ) {
 			if ( $reaction->get_meta( 'points_type' ) === $this->current_points_type ) {
-				$data[] = WordPoints_Admin_Ajax_Hooks::prepare_hook_reaction(
+				$data[ ] = WordPoints_Admin_Ajax_Hooks::prepare_hook_reaction(
 					$reaction
 				);
 			}
@@ -444,13 +444,13 @@ 
 
 		foreach ( $this->event_args->get_children( $event_slug ) as $slug => $arg ) {
 
-			$event_data['args'][ $slug ] = array(
+			$event_data[ 'args' ][ $slug ] = array(
 				'slug' => $slug,
 			);
 
 			if ( $arg instanceof WordPoints_Hook_ArgI ) {
-				$event_data['args'][ $slug ]['title'] = $arg->get_title();
-				$event_data['args'][ $slug ]['is_stateful'] = $arg->is_stateful();
+				$event_data[ 'args' ][ $slug ][ 'title' ] = $arg->get_title();
+				$event_data[ 'args' ][ $slug ][ 'is_stateful' ] = $arg->is_stateful();
 			}
 		}
 
@@ -463,7 +463,7 @@ 
 
 		<div class="wordpoints-hook-reaction-group-container">
 			<p class="description wordpoints-hook-reaction-group-description">
-				<?php echo esc_html( $meta_box['args']['event']->get_description() ); ?>
+				<?php echo esc_html( $meta_box[ 'args' ][ 'event' ]->get_description() ); ?>
 			</p>
 
 			<div class="wordpoints-hook-reaction-group"
@@ -505,10 +505,10 @@ 
 		$tabs = array();
 
 		foreach ( $points_types as $slug => $settings ) {
-			$tabs[ $slug ] = $settings['name'];
+			$tabs[ $slug ] = $settings[ 'name' ];
 		}
 
-		$tabs['add-new'] = __( 'Add New', 'wordpoints' );
+		$tabs[ 'add-new' ] = __( 'Add New', 'wordpoints' );
 
 		$tab = wordpoints_admin_get_current_tab( $tabs );
 
@@ -532,15 +532,15 @@ 
 			return;
 		}
 
-		if ( isset( $_POST['save-points-type'] ) ) { // WPCS: CSRF OK
+		if ( isset( $_POST[ 'save-points-type' ] ) ) { // WPCS: CSRF OK
 
-			if ( ! empty( $_POST['add_new'] ) ) { // WPCS: CSRF OK
+			if ( ! empty( $_POST[ 'add_new' ] ) ) { // WPCS: CSRF OK
 				$this->add_points_type();
 			} else {
 				$this->update_points_type();
 			}
 
-		} elseif ( ! empty( $_POST['delete-points-type'] ) ) { // WPCS: CSRF OK
+		} elseif ( ! empty( $_POST[ 'delete-points-type' ] ) ) { // WPCS: CSRF OK
 
 			$this->delete_points_type();
 		}
@@ -557,21 +557,21 @@ 
 
 		$settings = array();
 
-		if ( isset( $_POST['points-name'] ) ) { // WPCS: CSRF OK
-			$settings['name'] = trim(
-				sanitize_text_field( wp_unslash( $_POST['points-name'] ) ) // WPCS: CSRF OK
+		if ( isset( $_POST[ 'points-name' ] ) ) { // WPCS: CSRF OK
+			$settings[ 'name' ] = trim(
+				sanitize_text_field( wp_unslash( $_POST[ 'points-name' ] ) ) // WPCS: CSRF OK
 			);
 		}
 
-		if ( isset( $_POST['points-prefix'] ) ) { // WPCS: CSRF OK
-			$settings['prefix'] = ltrim(
-				sanitize_text_field( wp_unslash( $_POST['points-prefix'] ) ) // WPCS: CSRF OK
+		if ( isset( $_POST[ 'points-prefix' ] ) ) { // WPCS: CSRF OK
+			$settings[ 'prefix' ] = ltrim(
+				sanitize_text_field( wp_unslash( $_POST[ 'points-prefix' ] ) ) // WPCS: CSRF OK
 			);
 		}
 
-		if ( isset( $_POST['points-suffix'] ) ) { // WPCS: CSRF OK
-			$settings['suffix'] = rtrim(
-				sanitize_text_field( wp_unslash( $_POST['points-suffix'] ) ) // WPCS: CSRF OK
+		if ( isset( $_POST[ 'points-suffix' ] ) ) { // WPCS: CSRF OK
+			$settings[ 'suffix' ] = rtrim(
+				sanitize_text_field( wp_unslash( $_POST[ 'points-suffix' ] ) ) // WPCS: CSRF OK
 			);
 		}
 
@@ -592,14 +592,14 @@ 
 				, array( 'points-slug' )
 				, 'post'
 			)
-			|| ! isset( $_POST['points-slug'] )
+			|| ! isset( $_POST[ 'points-slug' ] )
 		) {
 			return;
 		}
 
 		$settings = $this->get_points_type_settings();
 
-		if ( empty( $settings['name'] ) ) {
+		if ( empty( $settings[ 'name' ] ) ) {
 
 			add_settings_error(
 				'points-name'
@@ -610,7 +610,7 @@ 
 			return;
 		}
 
-		$points_type = sanitize_key( $_POST['points-slug'] );
+		$points_type = sanitize_key( $_POST[ 'points-slug' ] );
 
 		$old_settings = wordpoints_get_points_type( $points_type );
 
@@ -680,7 +680,7 @@ 
 
 		} else {
 
-			$_GET['tab'] = $slug;
+			$_GET[ 'tab' ] = $slug;
 
 			add_settings_error(
 				''
@@ -705,12 +705,12 @@ 
 				, array( 'points-slug' )
 				, 'post'
 			)
-			&& isset( $_POST['points-slug'] )
+			&& isset( $_POST[ 'points-slug' ] )
 		) {
 
 			if (
 				wordpoints_delete_points_type(
-					sanitize_key( $_POST['points-slug'] )
+					sanitize_key( $_POST[ 'points-slug' ] )
 				)
 			) {
 
@@ -754,7 +754,7 @@ 
 
 		if ( isset( $this->current_points_type ) ) {
 			$points_type = wordpoints_get_points_type( $this->current_points_type );
-			$points_type['slug'] = $this->current_points_type;
+			$points_type[ 'slug' ] = $this->current_points_type;
 		} else {
 			$points_type = false;
 		}

Braces +17 added lines, -8 removed lines

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

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

Spacing +9 added lines, -9 removed lines

@@ -11,11 +11,11 @@ 
 	return;
 }
 
-if ( ! isset( $_POST['hook-id'] ) ) {
+if ( ! isset( $_POST[ 'hook-id' ] ) ) {
 	return;
 }
 
-$hook_id = sanitize_key( $_POST['hook-id'] );
+$hook_id = sanitize_key( $_POST[ 'hook-id' ] );
 
 check_admin_referer( "save-delete-hook-{$hook_id}" );
 
@@ -25,12 +25,12 @@ 
 	WordPoints_Points_Hooks::set_network_mode( true );
 }
 
-if ( ! isset( $_POST['points_type'], $_POST['id_base'] ) ) {
+if ( ! isset( $_POST[ 'points_type' ], $_POST[ 'id_base' ] ) ) {
 	return;
 }
 
-$points_type_id = sanitize_key( $_POST['points_type'] );
-$id_base        = sanitize_key( $_POST['id_base'] );
+$points_type_id = sanitize_key( $_POST[ 'points_type' ] );
+$id_base        = sanitize_key( $_POST[ 'id_base' ] );
 
 // These are the hooks grouped by points type.
 $points_types_hooks = WordPoints_Points_Hooks::get_points_types_hooks();
@@ -47,7 +47,7 @@ 
 
 $hook = WordPoints_Points_Hooks::get_handler_by_id_base( $id_base );
 
-if ( ! empty( $_POST['removehook'] ) ) {
+if ( ! empty( $_POST[ 'removehook' ] ) ) {
 
 	// - We are deleting an instance of a hook.
 
@@ -63,11 +63,11 @@ 
 
 	$hook->delete_callback( $hook_id );
 
-} elseif ( ! empty( $_POST['savehook'] ) ) {
+} elseif ( ! empty( $_POST[ 'savehook' ] ) ) {
 
 	// - We are saving an instance of a hook.
 
-	$number = isset( $_POST['multi_number'] ) ? (int) $_POST['multi_number'] : '';
+	$number = isset( $_POST[ 'multi_number' ] ) ? (int) $_POST[ 'multi_number' ] : '';
 
 	if ( $number ) {
 
@@ -101,7 +101,7 @@ 
 	// Add hook it to this points type.
 	if ( ! in_array( $hook_id, $points_type_hooks, true ) ) {
 
-		$points_type_hooks[] = $hook_id;
+		$points_type_hooks[ ] = $hook_id;
 		$points_types_hooks[ $points_type_id ] = $points_type_hooks;
 	}
 

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

Spacing +2 added lines, -2 removed lines

@@ -41,11 +41,11 @@
 $accessibility_mode = get_user_setting( 'wordpoints_points_hooks_access' );
 
 if (
-	isset( $_GET['accessibility-mode'], $_GET['wordpoints-accessiblity-nonce'] )
+	isset( $_GET[ 'accessibility-mode' ], $_GET[ 'wordpoints-accessiblity-nonce' ] )
 	&& wordpoints_verify_nonce( 'wordpoints-accessiblity-nonce', 'wordpoints_points_hooks_accessiblity' )
 ) {
 
-	$accessibility_mode = ( 'on' === sanitize_key( $_GET['accessibility-mode'] ) ) ? 'on' : 'off';
+	$accessibility_mode = ( 'on' === sanitize_key( $_GET[ 'accessibility-mode' ] ) ) ? 'on' : 'off';
 	set_user_setting( 'wordpoints_points_hooks_access', $accessibility_mode );
 }