WordPoints / wordpoints

src/admin/includes/functions.php

Spacing +27 added lines, -27 removed lines

@@ -611,14 +611,14 @@ 
 
 			if ( $child instanceof WordPoints_Entity_Attr ) {
 
-				$child_data[ $child_slug ]['_type']     = 'attr';
-				$child_data[ $child_slug ]['data_type'] = $child->get_data_type();
+				$child_data[ $child_slug ][ '_type' ]     = 'attr';
+				$child_data[ $child_slug ][ 'data_type' ] = $child->get_data_type();
 
 			} elseif ( $child instanceof WordPoints_Entity_Relationship ) {
 
-				$child_data[ $child_slug ]['_type']     = 'relationship';
-				$child_data[ $child_slug ]['primary']   = $child->get_primary_entity_slug();
-				$child_data[ $child_slug ]['secondary'] = $child->get_related_entity_slug();
+				$child_data[ $child_slug ][ '_type' ]     = 'relationship';
+				$child_data[ $child_slug ][ 'primary' ]   = $child->get_primary_entity_slug();
+				$child_data[ $child_slug ][ 'secondary' ] = $child->get_related_entity_slug();
 			}
 
 			/**
@@ -650,14 +650,14 @@ 
 
 			foreach ( $entity->get_enumerated_values() as $value ) {
 				if ( $entity->set_the_value( $value ) ) {
-					$values[] = array(
+					$values[ ] = array(
 						'value' => $entity->get_the_id(),
 						'label' => $entity->get_the_human_id(),
 					);
 				}
 			}
 
-			$entities_data[ $slug ]['values'] = $values;
+			$entities_data[ $slug ][ 'values' ] = $values;
 		}
 
 		/**
@@ -789,9 +789,9 @@ 
 
 	$args = array_merge( $defaults, $args );
 
-	if ( isset( $args['dismissable'] ) ) {
+	if ( isset( $args[ 'dismissable' ] ) ) {
 
-		$args['dismissible'] = $args['dismissable'];
+		$args[ 'dismissible' ] = $args[ 'dismissable' ];
 
 		_deprecated_argument(
 			__FUNCTION__
@@ -811,26 +811,26 @@ 
 		);
 	}
 
-	if ( $args['dismissible'] && $args['option'] ) {
+	if ( $args[ 'dismissible' ] && $args[ 'option' ] ) {
 		wp_enqueue_script( 'wordpoints-admin-dismiss-notice' );
 	}
 
 	?>
 
 	<div
-		class="notice notice-<?php echo sanitize_html_class( $type, 'success' ); ?><?php echo ( $args['dismissible'] ) ? ' is-dismissible' : ''; ?>"
-		<?php if ( $args['dismissible'] && $args['option'] ) : ?>
-			data-nonce="<?php echo esc_attr( wp_create_nonce( "wordpoints_dismiss_notice-{$args['option']}" ) ); ?>"
-			data-option="<?php echo esc_attr( $args['option'] ); ?>"
+		class="notice notice-<?php echo sanitize_html_class( $type, 'success' ); ?><?php echo ( $args[ 'dismissible' ] ) ? ' is-dismissible' : ''; ?>"
+		<?php if ( $args[ 'dismissible' ] && $args[ 'option' ] ) : ?>
+			data-nonce="<?php echo esc_attr( wp_create_nonce( "wordpoints_dismiss_notice-{$args[ 'option' ]}" ) ); ?>"
+			data-option="<?php echo esc_attr( $args[ 'option' ] ); ?>"
 		<?php endif; ?>
 		>
 		<p>
 			<?php echo wp_kses( $message, 'wordpoints_admin_message' ); ?>
 		</p>
-		<?php if ( $args['dismissible'] && $args['option'] ) : ?>
+		<?php if ( $args[ 'dismissible' ] && $args[ 'option' ] ) : ?>
 			<form method="post" class="wordpoints-notice-dismiss-form" style="padding-bottom: 5px;">
-				<input type="hidden" name="wordpoints_notice" value="<?php echo esc_html( $args['option'] ); ?>" />
-				<?php wp_nonce_field( "wordpoints_dismiss_notice-{$args['option']}" ); ?>
+				<input type="hidden" name="wordpoints_notice" value="<?php echo esc_html( $args[ 'option' ] ); ?>" />
+				<?php wp_nonce_field( "wordpoints_dismiss_notice-{$args[ 'option' ]}" ); ?>
 				<?php submit_button( __( 'Hide This Notice', 'wordpoints' ), 'secondary', 'wordpoints_dismiss_notice', false ); ?>
 			</form>
 		<?php endif; ?>
@@ -853,9 +853,9 @@ 
 
 	$tab = '';
 
-	if ( isset( $_GET['tab'] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET[ 'tab' ] ) ) { // WPCS: CSRF OK.
 
-		$tab = sanitize_key( $_GET['tab'] ); // WPCS: CSRF OK.
+		$tab = sanitize_key( $_GET[ 'tab' ] ); // WPCS: CSRF OK.
 	}
 
 	if ( isset( $tabs ) && ! isset( $tabs[ $tab ] ) ) {
@@ -892,8 +892,8 @@ 
 
 	$page = '';
 
-	if ( isset( $_GET['page'] ) ) { // WPCS: CSRF OK.
-		$page = sanitize_key( $_GET['page'] ); // WPCS: CSRF OK.
+	if ( isset( $_GET[ 'page' ] ) ) { // WPCS: CSRF OK.
+		$page = sanitize_key( $_GET[ 'page' ] ); // WPCS: CSRF OK.
 	}
 
 	foreach ( $tabs as $tab => $name ) {
@@ -1034,7 +1034,7 @@ 
 
 						$info = wordpoints_get_module_data( $file, false, false );
 
-						if ( ! empty( $info['name'] ) ) {
+						if ( ! empty( $info[ 'name' ] ) ) {
 							$source = new WP_Error(
 								'wordpoints_module_archive_not_plugin'
 								, $source->get_error_message()
@@ -1174,9 +1174,9 @@ 
 		, 'post'
 	);
 
-	if ( $is_notice_dismissed && isset( $_POST['wordpoints_notice'] ) ) {
+	if ( $is_notice_dismissed && isset( $_POST[ 'wordpoints_notice' ] ) ) {
 
-		$option = sanitize_key( $_POST['wordpoints_notice'] );
+		$option = sanitize_key( $_POST[ 'wordpoints_notice' ] );
 
 		if ( ! is_network_admin() && 'wordpoints_incompatible_modules' === $option ) {
 			delete_option( $option );
@@ -1231,7 +1231,7 @@ 
  */
 function wordpoints_admin_ajax_breaking_module_check() {
 
-	if ( ! isset( $_GET['wordpoints_module_check'] ) ) { // WPCS: CSRF OK.
+	if ( ! isset( $_GET[ 'wordpoints_module_check' ] ) ) { // WPCS: CSRF OK.
 		wp_die( '', 400 );
 	}
 
@@ -1241,12 +1241,12 @@ 
 		$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.
 		wp_die( '', 403 );
 	}
 
 	// The list table constructor calls WP_Screen::get(), which expects this.
-	$GLOBALS['hook_suffix'] = null;
+	$GLOBALS[ 'hook_suffix' ] = null;
 
 	wordpoints_admin_screen_modules();
 

Braces +8 added lines, -4 removed lines

@@ -853,7 +853,8 @@ 
 
 	$tab = '';
 
-	if ( isset( $_GET['tab'] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['tab'] ) ) {
+// WPCS: CSRF OK.
 
 		$tab = sanitize_key( $_GET['tab'] ); // WPCS: CSRF OK.
 	}
@@ -892,7 +893,8 @@ 
 
 	$page = '';
 
-	if ( isset( $_GET['page'] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['page'] ) ) {
+// WPCS: CSRF OK.
 		$page = sanitize_key( $_GET['page'] ); // WPCS: CSRF OK.
 	}
 
@@ -1231,7 +1233,8 @@ 
  */
 function wordpoints_admin_ajax_breaking_module_check() {
 
-	if ( ! isset( $_GET['wordpoints_module_check'] ) ) { // WPCS: CSRF OK.
+	if ( ! isset( $_GET['wordpoints_module_check'] ) ) {
+// WPCS: CSRF OK.
 		wp_die( '', 400 );
 	}
 
@@ -1241,7 +1244,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.
 		wp_die( '', 403 );
 	}
 

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/modules.php

Spacing +15 added lines, -15 removed lines

@@ -25,22 +25,22 @@ 
 	}
 }
 
-if ( isset( $_GET['error'] ) ) {
+if ( isset( $_GET[ 'error' ] ) ) {
 
-	if ( isset( $_GET['main'] ) ) {
+	if ( isset( $_GET[ 'main' ] ) ) {
 
 		wordpoints_show_admin_error(
 			esc_html__( 'You cannot delete a module while it is active on the main site.', 'wordpoints' )
 			, array( 'dismissible' => true )
 		);
 
-	} elseif ( isset( $_GET['charsout'] ) ) {
+	} elseif ( isset( $_GET[ 'charsout' ] ) ) {
 
 		wordpoints_show_admin_message(
 			sprintf(
 				// translators: Number of characters.
 				__( 'The module generated %d characters of <strong>unexpected output</strong> during activation. If you notice &#8220;headers already sent&#8221; messages, problems with syndication feeds or other issues, try deactivating or removing this module.', 'wordpoints' )
-				, (int) $_GET['charsout'] // WPCS: CSRF OK.
+				, (int) $_GET[ 'charsout' ] // WPCS: CSRF OK.
 			)
 			, 'warning'
 			, array( 'dismissible' => true )
@@ -51,14 +51,14 @@ 
 		$error_message = __( 'Module could not be activated because it triggered a <strong>fatal error</strong>.', 'wordpoints' );
 
 		if (
-			isset( $_GET['_error_nonce'], $_GET['module'] )
+			isset( $_GET[ '_error_nonce' ], $_GET[ 'module' ] )
 			&& wordpoints_verify_nonce( '_error_nonce', 'module-activation-error_%s', array( 'module' ) )
 		) {
 
 			$url = self_admin_url(
 				'admin.php?page=wordpoints_modules&action=error_scrape&amp;module='
-					. sanitize_text_field( wp_unslash( $_GET['module'] ) )
-					. '&amp;_wpnonce=' . sanitize_key( $_GET['_error_nonce'] )
+					. sanitize_text_field( wp_unslash( $_GET[ 'module' ] ) )
+					. '&amp;_wpnonce=' . sanitize_key( $_GET[ '_error_nonce' ] )
 			);
 
 			?>
@@ -82,7 +82,7 @@ 
 
 	} // End if ( main error ) elseif ( unexpected output error ) else.
 
-} elseif ( isset( $_GET['deleted'] ) ) {
+} elseif ( isset( $_GET[ 'deleted' ] ) ) {
 
 	$user_id       = get_current_user_id();
 	$delete_result = get_transient( 'wordpoints_modules_delete_result_' . $user_id );
@@ -110,7 +110,7 @@ 
 		);
 	}
 
-} elseif ( isset( $_GET['activate'] ) ) {
+} elseif ( isset( $_GET[ 'activate' ] ) ) {
 
 	wordpoints_show_admin_message(
 		__( 'Module <strong>activated</strong>.', 'wordpoints' )
@@ -118,7 +118,7 @@ 
 		, array( 'dismissible' => true )
 	);
 
-} elseif ( isset( $_GET['activate-multi'] ) ) {
+} elseif ( isset( $_GET[ 'activate-multi' ] ) ) {
 
 	wordpoints_show_admin_message(
 		__( 'Selected modules <strong>activated</strong>.', 'wordpoints' )
@@ -126,7 +126,7 @@ 
 		, array( 'dismissible' => true )
 	);
 
-} elseif ( isset( $_GET['deactivate'] ) ) {
+} elseif ( isset( $_GET[ 'deactivate' ] ) ) {
 
 	wordpoints_show_admin_message(
 		__( 'Module <strong>deactivated</strong>.', 'wordpoints' )
@@ -134,7 +134,7 @@ 
 		, array( 'dismissible' => true )
 	);
 
-} elseif ( isset( $_GET['deactivate-multi'] ) ) {
+} elseif ( isset( $_GET[ 'deactivate-multi' ] ) ) {
 
 	wordpoints_show_admin_message(
 		__( 'Selected modules <strong>deactivated</strong>.', 'wordpoints' )
@@ -142,7 +142,7 @@ 
 		, array( 'dismissible' => true )
 	);
 
-} elseif ( isset( $_REQUEST['action'] ) && 'update-selected' === sanitize_key( $_REQUEST['action'] ) ) {
+} elseif ( isset( $_REQUEST[ 'action' ] ) && 'update-selected' === sanitize_key( $_REQUEST[ 'action' ] ) ) {
 
 	wordpoints_show_admin_message(
 		esc_html__( 'No out of date modules were selected.', 'wordpoints' )
@@ -162,7 +162,7 @@ 
 			<a href="<?php echo esc_url( self_admin_url( 'admin.php?page=wordpoints_install_modules' ) ); ?>" class="page-title-action"><?php echo esc_html_x( 'Add New', 'module', 'wordpoints' ); ?></a>
 		<?php endif; ?>
 
-		<?php if ( ! empty( $_REQUEST['s'] ) ) : ?>
+		<?php if ( ! empty( $_REQUEST[ 's' ] ) ) : ?>
 			<span class="subtitle">
 				<?php
 
@@ -170,7 +170,7 @@ 
 					sprintf(
 						// translators: Search term.
 						__( 'Search results for &#8220;%s&#8221;', 'wordpoints' )
-						, sanitize_text_field( wp_unslash( $_REQUEST['s'] ) )
+						, sanitize_text_field( wp_unslash( $_REQUEST[ 's' ] ) )
 					)
 				);
 

src/admin/screens/modules-load.php

Spacing +24 added lines, -24 removed lines

@@ -9,22 +9,22 @@ 
 
 global $status, $wp_version;
 
-if ( isset( $_POST['clear-recent-list'] ) ) {
+if ( isset( $_POST[ 'clear-recent-list' ] ) ) {
 	$action = 'clear-recent-list';
-} 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.
-	$action = sanitize_key( $_REQUEST['action2'] );
+} 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.
+	$action = sanitize_key( $_REQUEST[ 'action2' ] );
 } else {
 	$action = '';
 }
 
-$page   = ( isset( $_REQUEST['paged'] ) ) ? max( 1, absint( $_REQUEST['paged'] ) ) : 1; // WPCS: CSRF OK.
-$module = ( isset( $_REQUEST['module'] ) ) ? sanitize_text_field( wp_unslash( $_REQUEST['module'] ) ) : ''; // WPCS: CSRF OK.
-$s      = ( isset( $_REQUEST['s'] ) ) ? sanitize_text_field( wp_unslash( $_REQUEST['s'] ) ) : ''; // WPCS: CSRF OK.
+$page   = ( isset( $_REQUEST[ 'paged' ] ) ) ? max( 1, absint( $_REQUEST[ 'paged' ] ) ) : 1; // WPCS: CSRF OK.
+$module = ( isset( $_REQUEST[ 'module' ] ) ) ? sanitize_text_field( wp_unslash( $_REQUEST[ 'module' ] ) ) : ''; // WPCS: CSRF OK.
+$s      = ( isset( $_REQUEST[ 's' ] ) ) ? sanitize_text_field( wp_unslash( $_REQUEST[ 's' ] ) ) : ''; // WPCS: CSRF OK.
 
 // Clean up request URI from temporary args for screen options/paging URI's to work as expected.
-$_SERVER['REQUEST_URI'] = remove_query_arg( array( 'error', 'deleted', 'activate', 'activate-multi', 'deactivate', 'deactivate-multi', '_error_nonce' ) );
+$_SERVER[ 'REQUEST_URI' ] = remove_query_arg( array( 'error', 'deleted', 'activate', 'activate-multi', 'deactivate', 'deactivate-multi', '_error_nonce' ) );
 
 $redirect_url = self_admin_url( "admin.php?page=wordpoints_modules&module_status={$status}&paged={$page}&s={$s}" );
 
@@ -90,8 +90,8 @@ 
 
 		check_admin_referer( 'bulk-modules' );
 
-		$modules = isset( $_POST['checked'] )
-			? array_map( 'sanitize_text_field', (array) wp_unslash( $_POST['checked'] ) )
+		$modules = isset( $_POST[ 'checked' ] )
+			? array_map( 'sanitize_text_field', (array) wp_unslash( $_POST[ 'checked' ] ) )
 			: array();
 
 		// Only activate modules which are not already active.
@@ -218,8 +218,8 @@ 
 
 		check_admin_referer( 'bulk-modules' );
 
-		$modules = isset( $_POST['checked'] )
-			? array_map( 'sanitize_text_field', (array) wp_unslash( $_POST['checked'] ) )
+		$modules = isset( $_POST[ 'checked' ] )
+			? array_map( 'sanitize_text_field', (array) wp_unslash( $_POST[ 'checked' ] ) )
 			: array();
 
 		$network_modules = array_filter( $modules, 'is_wordpoints_module_active_for_network' );
@@ -261,8 +261,8 @@ 
 		check_admin_referer( 'bulk-modules' );
 
 		// $_POST = from the module form; $_GET = from the FTP details screen.
-		$modules = isset( $_REQUEST['checked'] )
-			? array_map( 'sanitize_text_field', (array) wp_unslash( $_REQUEST['checked'] ) )
+		$modules = isset( $_REQUEST[ 'checked' ] )
+			? array_map( 'sanitize_text_field', (array) wp_unslash( $_REQUEST[ 'checked' ] ) )
 			: array();
 
 		if ( empty( $modules ) ) {
@@ -287,7 +287,7 @@ 
 
 		$parent_file = 'admin.php';
 
-		if ( ! isset( $_REQUEST['verify-delete'] ) ) {
+		if ( ! isset( $_REQUEST[ 'verify-delete' ] ) ) {
 
 			wp_enqueue_script( 'jquery' );
 			require_once ABSPATH . 'wp-admin/admin-header.php';
@@ -307,15 +307,15 @@ 
 
 					if ( '.' === dirname( $module ) ) {
 
-						$files_to_delete[] = $module_dir . '/' . $module;
+						$files_to_delete[ ] = $module_dir . '/' . $module;
 						$data = wordpoints_get_module_data( $module_dir . '/' . $module );
 
 						if ( ! empty( $data ) ) {
 
 							$module_info[ $module ] = $data;
-							$module_info[ $module ]['is_uninstallable'] = is_uninstallable_wordpoints_module( $module );
+							$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;
 							}
 						}
@@ -335,9 +335,9 @@ 
 						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 ][ 'is_uninstallable' ] = is_uninstallable_wordpoints_module( $module );
 
-							if ( ! $module_info[ $module_file ]['network'] ) {
+							if ( ! $module_info[ $module_file ][ 'network' ] ) {
 								$have_non_network_modules = true;
 							}
 						}
@@ -376,16 +376,16 @@ 
 
 						foreach ( $module_info as $module ) {
 
-							if ( $module['is_uninstallable'] ) {
+							if ( $module[ 'is_uninstallable' ] ) {
 
 								// translators: 1. Module name; 2. Module author.
-								echo '<li>', wp_kses( sprintf( __( '<strong>%1$s</strong> by <em>%2$s</em> (will also <strong>delete its data</strong>)', 'wordpoints' ), esc_html( $module['name'] ), esc_html( $module['author_name'] ) ), array( 'strong' => array(), 'em' => array() ) ), '</li>';
+								echo '<li>', wp_kses( sprintf( __( '<strong>%1$s</strong> by <em>%2$s</em> (will also <strong>delete its data</strong>)', 'wordpoints' ), esc_html( $module[ 'name' ] ), esc_html( $module[ 'author_name' ] ) ), array( 'strong' => array(), 'em' => array() ) ), '</li>';
 								$data_to_delete = true;
 
 							} else {
 
 								// translators: 1. Module name; 2. Module author.
-								echo '<li>', wp_kses( sprintf( __( '<strong>%1$s</strong> by <em>%2$s</em>', 'wordpoints' ), esc_html( $module['name'] ), esc_html( $module['author_name'] ) ), array( 'strong' => array(), 'em' => array() ) ), '</li>';
+								echo '<li>', wp_kses( sprintf( __( '<strong>%1$s</strong> by <em>%2$s</em>', 'wordpoints' ), esc_html( $module[ 'name' ] ), esc_html( $module[ 'author_name' ] ) ), array( 'strong' => array(), 'em' => array() ) ), '</li>';
 							}
 						}
 

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

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>

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/includes/class-wordpoints-points-hooks.php

Spacing +23 added lines, -23 removed lines

@@ -110,7 +110,7 @@ 
 	 */
 	public static function register( $class_name ) {
 
-		self::$classes[] = $class_name;
+		self::$classes[ ] = $class_name;
 	}
 
 	/**
@@ -286,10 +286,10 @@ 
 
 			$args = $hook_type->get_options();
 
-			$args['_add']       = 'multi';
-			$args['_display']   = 'template';
-			$args['_multi_num'] = $hook_type->next_hook_id_number();
-			$args['_id_slug']   = $i;
+			$args[ '_add' ]       = 'multi';
+			$args[ '_display' ]   = 'template';
+			$args[ '_multi_num' ] = $hook_type->next_hook_id_number();
+			$args[ '_id_slug' ]   = $i;
 
 			$hook_type->set_options( $args );
 
@@ -338,11 +338,11 @@ 
 
 			$options = $hook_type->get_options();
 
-			$options['_display'] = 'instance';
+			$options[ '_display' ] = 'instance';
 
-			unset( $options['_add'] );
+			unset( $options[ '_add' ] );
 
-			$options['_id_slug'] = $slug;
+			$options[ '_id_slug' ] = $slug;
 
 			$hook_type->set_options( $options );
 
@@ -596,15 +596,15 @@ 
 
 						<p>
 							<label for="points-name-<?php echo esc_attr( $slug ); ?>"><?php echo esc_html_x( 'Name:', 'points type', 'wordpoints' ); ?></label>
-							<input class="widefat" type="text" id="points-name-<?php echo esc_attr( $slug ); ?>" name="points-name" value="<?php echo esc_attr( $points_type['name'] ); ?>" />
+							<input class="widefat" type="text" id="points-name-<?php echo esc_attr( $slug ); ?>" name="points-name" value="<?php echo esc_attr( $points_type[ 'name' ] ); ?>" />
 						</p>
 						<p>
 							<label for="points-prefix-<?php echo esc_attr( $slug ); ?>"><?php echo esc_html( $prefix ); ?></label>
-							<input class="widefat" type="text" id="points-prefix-<?php echo esc_attr( $slug ); ?>" name="points-prefix" value="<?php echo esc_attr( $points_type['prefix'] ); ?>" />
+							<input class="widefat" type="text" id="points-prefix-<?php echo esc_attr( $slug ); ?>" name="points-prefix" value="<?php echo esc_attr( $points_type[ 'prefix' ] ); ?>" />
 						</p>
 						<p>
 							<label for="points-suffix-<?php echo esc_attr( $slug ); ?>"><?php echo esc_html( $suffix ); ?></label>
-							<input class="widefat" type="text" id="points-suffix-<?php echo esc_attr( $slug ); ?>" name="points-suffix" value="<?php echo esc_attr( $points_type['suffix'] ); ?>" />
+							<input class="widefat" type="text" id="points-suffix-<?php echo esc_attr( $slug ); ?>" name="points-suffix" value="<?php echo esc_attr( $points_type[ 'suffix' ] ); ?>" />
 						</p>
 
 						<?php
@@ -677,28 +677,28 @@ 
 
 		$id_format = $hook_id;
 
-		$multi_number = ( isset( $options['_multi_num'] ) ) ? $options['_multi_num'] : '';
-		$add_new      = ( isset( $options['_add'] ) )       ? $options['_add']       : '';
+		$multi_number = ( isset( $options[ '_multi_num' ] ) ) ? $options[ '_multi_num' ] : '';
+		$add_new      = ( isset( $options[ '_add' ] ) ) ? $options[ '_add' ] : '';
 
 		// Prepare the URL query string.
 		$query_arg = array( 'edithook' => $id_format );
 
 		if ( $add_new ) {
 
-			$query_arg['addnew'] = 1;
+			$query_arg[ 'addnew' ] = 1;
 
 			if ( $multi_number ) {
 
-				$query_arg['num']  = $multi_number;
-				$query_arg['base'] = $id_base;
+				$query_arg[ 'num' ]  = $multi_number;
+				$query_arg[ 'base' ] = $id_base;
 			}
 
 		} else {
 
-			$query_arg['points_type'] = $points_type;
+			$query_arg[ 'points_type' ] = $points_type;
 		}
 
-		if ( isset( $options['_display'] ) && 'template' === $options['_display'] ) {
+		if ( isset( $options[ '_display' ] ) && 'template' === $options[ '_display' ] ) {
 
 			/*
 			 * We aren't outputting the form for a hook, but a template form for this
@@ -714,7 +714,7 @@ 
 
 		?>
 
-		<div id="hook-<?php echo esc_html( $options['_id_slug'] ); ?>_<?php echo esc_attr( $id_format ); ?>" class="hook <?php echo esc_attr( $options['_classname'] ); ?>">
+		<div id="hook-<?php echo esc_html( $options[ '_id_slug' ] ); ?>_<?php echo esc_attr( $id_format ); ?>" class="hook <?php echo esc_attr( $options[ '_classname' ] ); ?>">
 		<div class="hook-top">
 			<div class="hook-title-action">
 				<a class="hook-action hide-if-no-js" href="#available-hooks"></a>
@@ -739,8 +739,8 @@ 
 
 				<input type="hidden" name="hook-id" class="hook-id" value="<?php echo esc_attr( $id_format ); ?>" />
 				<input type="hidden" name="id_base" class="id_base" value="<?php echo esc_attr( $id_base ); ?>" />
-				<input type="hidden" name="hook-width" class="hook-width" value="<?php echo isset( $options['width'] ) ? esc_attr( $options['width'] ) : ''; ?>" />
-				<input type="hidden" name="hook-height" class="hook-height" value="<?php echo isset( $options['height'] ) ? esc_attr( $options['height'] ) : ''; ?>" />
+				<input type="hidden" name="hook-width" class="hook-width" value="<?php echo isset( $options[ 'width' ] ) ? esc_attr( $options[ 'width' ] ) : ''; ?>" />
+				<input type="hidden" name="hook-height" class="hook-height" value="<?php echo isset( $options[ 'height' ] ) ? esc_attr( $options[ 'height' ] ) : ''; ?>" />
 				<input type="hidden" name="hook_number" class="hook_number" value="<?php echo esc_attr( $number ); ?>" />
 				<input type="hidden" name="multi_number" class="multi_number" value="<?php echo esc_attr( $multi_number ); ?>" />
 				<input type="hidden" name="add_new" class="add_new" value="<?php echo esc_attr( $add_new ); ?>" />
@@ -760,8 +760,8 @@ 
 		</div>
 
 		<div class="hook-description">
-			<?php if ( ! empty( $options['description'] ) ) : ?>
-				<?php echo esc_html( $options['description'] ); ?>
+			<?php if ( ! empty( $options[ 'description' ] ) ) : ?>
+				<?php echo esc_html( $options[ 'description' ] ); ?>
 			<?php endif; ?>
 		</div>
 		</div>

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