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;
 	}
 

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 = '';

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'] );
 

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>

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.

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

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
 			);