WordPoints / wordpoints

src/includes/maintenance.php

Braces +2 added lines, -1 removed lines

@@ -20,7 +20,8 @@
 }
 
 // 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/includes/functions.php

Braces +8 added lines, -4 removed lines

@@ -88,7 +88,8 @@ 
  */
 function wordpoints_maintenance_shutdown_print_rand_str() {
 
-	if ( ! isset( $_GET['wordpoints_module_check'] ) ) { // WPCS: CSRF OK.
+	if ( ! isset( $_GET['wordpoints_module_check'] ) ) {
+// WPCS: CSRF OK.
 		return;
 	}
 
@@ -98,7 +99,8 @@ 
 		$nonce = get_option( 'wordpoints_module_check_nonce' );
 	}
 
-	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET['wordpoints_module_check'] ) ) ) { // WPCS: CSRF OK.
+	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET['wordpoints_module_check'] ) ) ) {
+// WPCS: CSRF OK.
 		return;
 	}
 
@@ -128,7 +130,8 @@ 
  */
 function wordpoints_maintenance_filter_modules( $modules ) {
 
-	if ( ! isset( $_GET['check_module'], $_GET['wordpoints_module_check'] ) ) { // WPCS: CSRF OK.
+	if ( ! isset( $_GET['check_module'], $_GET['wordpoints_module_check'] ) ) {
+// WPCS: CSRF OK.
 		return $modules;
 	}
 
@@ -138,7 +141,8 @@ 
 		$nonce = get_option( 'wordpoints_module_check_nonce' );
 	}
 
-	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET['wordpoints_module_check'] ) ) ) { // WPCS: CSRF OK.
+	if ( ! $nonce || ! hash_equals( $nonce, sanitize_key( $_GET['wordpoints_module_check'] ) ) ) {
+// WPCS: CSRF OK.
 		return $modules;
 	}
 

Spacing +18 added lines, -18 removed lines

@@ -339,7 +339,7 @@ 
 		$request = $_GET; // WPCS: CSRF OK.
 	}
 
-	if ( ! isset( $request[ $nonce_key ] ) ) {
+	if ( ! isset( $request[$nonce_key] ) ) {
 		return false;
 	}
 
@@ -349,18 +349,18 @@ 
 
 		foreach ( (array) $format_values as $value ) {
 
-			if ( ! isset( $request[ $value ] ) ) {
+			if ( ! isset( $request[$value] ) ) {
 				return false;
 			}
 
-			$values[] = $request[ $value ];
+			$values[] = $request[$value];
 		}
 
 		$action_format = vsprintf( $action_format, $values );
 	}
 
 	$is_valid = wp_verify_nonce(
-		sanitize_key( $request[ $nonce_key ] )
+		sanitize_key( $request[$nonce_key] )
 		, strip_tags( $action_format )
 	);
 
@@ -395,10 +395,10 @@ 
 
 	$error_data = $error->error_data;
 
-	if ( isset( $error_data[ $code ]['title'] ) ) {
+	if ( isset( $error_data[$code]['title'] ) ) {
 
-		$error_data[ $code ]['title'] = wp_kses(
-			$error->error_data[ $code ]['title']
+		$error_data[$code]['title'] = wp_kses(
+			$error->error_data[$code]['title']
 			, 'wordpoints_sanitize_wp_error_title'
 		);
 
@@ -410,7 +410,7 @@ 
 	foreach ( $all_errors as $code => $errors ) {
 
 		foreach ( $errors as $key => $message ) {
-			$all_errors[ $code ][ $key ] = wp_kses(
+			$all_errors[$code][$key] = wp_kses(
 				$message
 				, 'wordpoints_sanitize_wp_error_message'
 			);
@@ -998,15 +998,15 @@ 
 		'delete_wordpoints_modules'         => 'delete_wordpoints_extensions',
 	);
 
-	if ( isset( $deprecated[ $cap ] ) ) {
+	if ( isset( $deprecated[$cap] ) ) {
 
 		_deprecated_argument(
 			'current_user_can'
 			, '2.4.0'
-			, esc_html( "{$cap} is deprecated, use {$deprecated[ $cap ]} instead." )
+			, esc_html( "{$cap} is deprecated, use {$deprecated[$cap]} instead." )
 		);
 
-		$caps[] = $deprecated[ $cap ];
+		$caps[] = $deprecated[$cap];
 	}
 
 	switch ( $cap ) {
@@ -1276,26 +1276,26 @@ 
 
 	foreach ( $map as $shortcut => $canonicals ) {
 
-		if ( ! isset( $array[ $shortcut ] ) ) {
+		if ( ! isset( $array[$shortcut] ) ) {
 			continue;
 		}
 
 		foreach ( $canonicals as $canonical ) {
 
-			if ( ! isset( $array[ $canonical ] ) ) {
+			if ( ! isset( $array[$canonical] ) ) {
 
-				$array[ $canonical ] = $array[ $shortcut ];
+				$array[$canonical] = $array[$shortcut];
 
 			} else {
 
-				$array[ $canonical ] = array_merge(
-					$array[ $canonical ]
-					, $array[ $shortcut ]
+				$array[$canonical] = array_merge(
+					$array[$canonical]
+					, $array[$shortcut]
 				);
 			}
 		}
 
-		unset( $array[ $shortcut ] );
+		unset( $array[$shortcut] );
 	}
 
 	return $array;

src/admin/classes/ajax/hooks.php

Braces +4 added lines, -2 removed lines

@@ -334,7 +334,8 @@ 
 	 */
 	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' );
 		}
 
@@ -361,7 +362,8 @@ 
 	 */
 	protected function get_reaction() {
 
-		if ( ! isset( $_POST['id'] ) ) { // WPCS: CSRF OK.
+		if ( ! isset( $_POST['id'] ) ) {
+// WPCS: CSRF OK.
 			$this->unexpected_error( 'id' );
 		}
 

src/admin/screens/modules-load.php

Braces +4 added lines, -2 removed lines

@@ -11,9 +11,11 @@
 
 if ( isset( $_POST['clear-recent-list'] ) ) {
 	$action = 'clear-recent-list';
-} elseif ( isset( $_REQUEST['action'] ) && -1 !== (int) $_REQUEST['action'] ) { // WPCS: CSRF OK.
+} elseif ( isset( $_REQUEST['action'] ) && -1 !== (int) $_REQUEST['action'] ) {
+// WPCS: CSRF OK.
 	$action = sanitize_key( $_REQUEST['action'] );
-} elseif ( isset( $_REQUEST['action2'] ) && -1 !== (int) $_REQUEST['action2'] ) { // WPCS: CSRF OK.
+} elseif ( isset( $_REQUEST['action2'] ) && -1 !== (int) $_REQUEST['action2'] ) {
+// WPCS: CSRF OK.
 	$action = sanitize_key( $_REQUEST['action2'] );
 } else {
 	$action = '';

Spacing +12 added lines, -12 removed lines

@@ -76,7 +76,7 @@ 
 		if ( ! is_network_admin() ) {
 
 			$recent = wordpoints_get_array_option( 'wordpoints_recently_activated_modules' );
-			unset( $recent[ $module ] );
+			unset( $recent[$module] );
 			update_option( 'wordpoints_recently_activated_modules', $recent );
 		}
 
@@ -100,7 +100,7 @@ 
 
 			foreach ( $modules as $i => $module ) {
 				if ( is_wordpoints_module_active_for_network( $module ) ) {
-					unset( $modules[ $i ] );
+					unset( $modules[$i] );
 				}
 			}
 
@@ -108,7 +108,7 @@ 
 
 			foreach ( $modules as $i => $module ) {
 				if ( is_wordpoints_module_active( $module ) || is_network_only_wordpoints_module( $module ) ) {
-					unset( $modules[ $i ] );
+					unset( $modules[$i] );
 				}
 			}
 		}
@@ -130,7 +130,7 @@ 
 			$recent = wordpoints_get_array_option( 'wordpoints_recently_activated_modules' );
 
 			foreach ( $modules as $module ) {
-				unset( $recent[ $module ] );
+				unset( $recent[$module] );
 			}
 
 			update_option( 'wordpoints_recently_activated_modules', $recent );
@@ -244,7 +244,7 @@ 
 			$deactivated = array();
 
 			foreach ( $modules as $module ) {
-				$deactivated[ $module ] = time();
+				$deactivated[$module] = time();
 			}
 
 			update_option( 'wordpoints_recently_activated_modules', $deactivated + wordpoints_get_array_option( 'wordpoints_recently_activated_modules' ) );
@@ -278,7 +278,7 @@ 
 				true !== wordpoints_validate_module( $module )
 				|| is_wordpoints_module_active( $module )
 			) {
-				unset( $modules[ $key ] );
+				unset( $modules[$key] );
 			}
 		}
 
@@ -316,10 +316,10 @@ 
 
 						if ( ! empty( $data ) ) {
 
-							$module_info[ $module ] = $data;
-							$module_info[ $module ]['is_uninstallable'] = is_uninstallable_wordpoints_module( $module );
+							$module_info[$module] = $data;
+							$module_info[$module]['is_uninstallable'] = is_uninstallable_wordpoints_module( $module );
 
-							if ( ! $module_info[ $module ]['network'] ) {
+							if ( ! $module_info[$module]['network'] ) {
 								$have_non_network_modules = true;
 							}
 						}
@@ -338,10 +338,10 @@ 
 
 						foreach ( $folder_modules as $module_file => $data ) {
 
-							$module_info[ $module_file ] = $data;
-							$module_info[ $module_file ]['is_uninstallable'] = is_uninstallable_wordpoints_module( $module );
+							$module_info[$module_file] = $data;
+							$module_info[$module_file]['is_uninstallable'] = is_uninstallable_wordpoints_module( $module );
 
-							if ( ! $module_info[ $module_file ]['network'] ) {
+							if ( ! $module_info[$module_file]['network'] ) {
 								$have_non_network_modules = true;
 							}
 						}

src/admin/screens/configure-components.php

Braces +2 added lines, -1 removed lines

@@ -17,7 +17,8 @@
 // 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'] );
 

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
 		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 )
 			);
@@ -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 )
 			);
 		}

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

Braces +9 added lines, -4 removed lines

@@ -143,7 +143,8 @@ 
  */
 function wordpoints_points_admin_screen_hooks() {
 
-	if ( isset( $_GET['edithook'] ) || isset( $_POST['savehook'] ) || isset( $_POST['removehook'] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['edithook'] ) || isset( $_POST['savehook'] ) || isset( $_POST['removehook'] ) ) {
+// WPCS: CSRF OK.
 
 		// - We're doing this without AJAX (JS).
 
@@ -206,7 +207,8 @@ 
  */
 function wordpoints_no_js_points_hooks_save() {
 
-	if ( ! isset( $_POST['savehook'] ) && ! isset( $_POST['removehook'] ) ) { // WPCS: CSRF OK.
+	if ( ! isset( $_POST['savehook'] ) && ! isset( $_POST['removehook'] ) ) {
+// WPCS: CSRF OK.
 		return;
 	}
 
@@ -290,9 +292,12 @@ 
 
 	<?php if ( $has_form ) : ?>
 		<hr />
-	<?php else : ?>
+	<?php else {
+	: ?>
 		<br />
-	<?php endif; ?>
+	<?php endif;
+}
+?>
 
 	<div class="hook-instance-description">
 		<label for="<?php $hook->the_field_id( '_description' ); ?>"><?php echo esc_html_x( 'Description (optional):', 'points hook', 'wordpoints' ); ?></label>

Spacing +6 added lines, -6 removed lines

@@ -438,17 +438,17 @@
 
 		if (
 			isset(
-				$_POST[ "wordpoints_points_set-{$slug}" ]
-				, $_POST[ "wordpoints_points-{$slug}" ]
-				, $_POST[ "wordpoints_points_old-{$slug}" ]
+				$_POST["wordpoints_points_set-{$slug}"]
+				, $_POST["wordpoints_points-{$slug}"]
+				, $_POST["wordpoints_points_old-{$slug}"]
 			)
-			&& false !== wordpoints_int( $_POST[ "wordpoints_points-{$slug}" ] )
-			&& false !== wordpoints_int( $_POST[ "wordpoints_points_old-{$slug}" ] )
+			&& false !== wordpoints_int( $_POST["wordpoints_points-{$slug}"] )
+			&& false !== wordpoints_int( $_POST["wordpoints_points_old-{$slug}"] )
 		) {
 
 			wordpoints_alter_points(
 				$user_id
-				, (int) $_POST[ "wordpoints_points-{$slug}" ] - (int) $_POST[ "wordpoints_points_old-{$slug}" ]
+				, (int) $_POST["wordpoints_points-{$slug}"] - (int) $_POST["wordpoints_points_old-{$slug}"]
 				, $slug
 				, 'profile_edit'
 				, array(

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

Braces +4 added lines, -2 removed lines

@@ -43,7 +43,8 @@ 
 		return;
 	}
 
-	if ( isset( $_GET['message'] ) && isset( $messages[ (int) $_GET['message'] ] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['message'] ) && isset( $messages[ (int) $_GET['message'] ] ) ) {
+// WPCS: CSRF OK.
 
 		wordpoints_show_admin_message(
 			esc_html( $messages[ (int) $_GET['message'] ] ) // WPCS: CSRF OK.
@@ -51,7 +52,8 @@ 
 			, array( 'dismissible' => true )
 		);
 
-	} elseif ( isset( $_GET['error'] ) && isset( $errors[ (int) $_GET['error'] ] ) ) { // WPCS: CSRF OK.
+	} elseif ( isset( $_GET['error'] ) && isset( $errors[ (int) $_GET['error'] ] ) ) {
+// WPCS: CSRF OK.
 
 		wordpoints_show_admin_error(
 			esc_html( $errors[ (int) $_GET['error'] ] ) // WPCS: CSRF OK.

Spacing +4 added lines, -4 removed lines

@@ -43,18 +43,18 @@
 		return;
 	}
 
-	if ( isset( $_GET['message'] ) && isset( $messages[ (int) $_GET['message'] ] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['message'] ) && isset( $messages[(int) $_GET['message']] ) ) { // WPCS: CSRF OK.
 
 		wordpoints_show_admin_message(
-			esc_html( $messages[ (int) $_GET['message'] ] ) // WPCS: CSRF OK.
+			esc_html( $messages[(int) $_GET['message']] ) // WPCS: CSRF OK.
 			, 'success'
 			, array( 'dismissible' => true )
 		);
 
-	} elseif ( isset( $_GET['error'] ) && isset( $errors[ (int) $_GET['error'] ] ) ) { // WPCS: CSRF OK.
+	} elseif ( isset( $_GET['error'] ) && isset( $errors[(int) $_GET['error']] ) ) { // WPCS: CSRF OK.
 
 		wordpoints_show_admin_error(
-			esc_html( $errors[ (int) $_GET['error'] ] ) // WPCS: CSRF OK.
+			esc_html( $errors[(int) $_GET['error']] ) // WPCS: CSRF OK.
 			, array( 'dismissible' => true )
 		);
 	}

src/components/points/classes/logs/view/table.php

Braces +2 added lines, -1 removed lines

@@ -222,7 +222,8 @@
 
 		$search_term = '';
 
-		if ( isset( $_POST['wordpoints_points_logs_search'] ) ) { // WPCS: CSRF OK
+		if ( isset( $_POST['wordpoints_points_logs_search'] ) ) {
+// WPCS: CSRF OK
 			$search_term = trim(
 				sanitize_text_field(
 					wp_unslash( $_POST['wordpoints_points_logs_search'] ) // WPCS: CSRF OK

Spacing +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
 		$extra_classes = array();
 
 		foreach ( array( 'searchable', 'paginate' ) as $arg ) {
-			if ( $this->args[ $arg ] ) {
+			if ( $this->args[$arg] ) {
 				$extra_classes[] = $arg;
 			}
 		}

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

Braces +17 added lines, -8 removed lines

@@ -245,8 +245,11 @@ 
 					<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 esc_html_e( 'The new points type will be global across all sites on this network.', 'wordpoints' ); ?>
+						<?php else {
+	: ?>
+							<?php esc_html_e( 'The new points type will be global across all sites on this network.', 'wordpoints' );
+}
+?>
 						<?php endif; ?>
 					</p>
 				</div>
@@ -655,15 +658,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();
 		}
@@ -680,19 +686,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
 			);

Spacing +7 added lines, -7 removed lines

@@ -517,7 +517,7 @@ 
 
 			if (
 				! array_intersect_key(
-					$event_action_types[ $slug ]
+					$event_action_types[$slug]
 					, $reactor_action_types
 				)
 			) {
@@ -570,18 +570,18 @@ 
 
 		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();
 			}
 		}
 
-		$this->events_data[ $event_slug ]    = $event_data;
-		$this->reactions_data[ $event_slug ] = $data;
+		$this->events_data[$event_slug]    = $event_data;
+		$this->reactions_data[$event_slug] = $data;
 
 		?>
 
@@ -629,7 +629,7 @@ 
 		$tabs = array();
 
 		foreach ( $points_types as $slug => $settings ) {
-			$tabs[ $slug ] = $settings['name'];
+			$tabs[$slug] = $settings['name'];
 		}
 
 		$tabs['add-new'] = __( 'Add New', 'wordpoints' );