WordPoints / wordpoints

src/includes/class-un-installer.php

Spacing +2 added lines, -2 removed lines

@@ -140,7 +140,7 @@ 
 		parent::before_update();
 
 		if ( $this->network_wide ) {
-			unset( $this->updates['1_8_0'] );
+			unset( $this->updates[ '1_8_0' ] );
 		}
 	}
 
@@ -168,7 +168,7 @@ 
 			// Make sure the version is set properly even if the data is already
 			// there, in case the plugin is being reactivated and things had been
 			// corrupted somehow.
-			$data['version'] = WORDPOINTS_VERSION;
+			$data[ 'version' ] = WORDPOINTS_VERSION;
 
 			wordpoints_update_maybe_network_option( 'wordpoints_data', $data );
 		}

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

Spacing +34 added lines, -34 removed lines

@@ -69,7 +69,7 @@ 
 
 	$module_data = wordpoints_get_module_data( wordpoints_modules_dir() . '/' . $module );
 
-	return $module_data['network'];
+	return $module_data[ 'network' ];
 }
 
 /**
@@ -257,17 +257,17 @@ 
 
 	$module_data = WordPoints_Modules::get_data( $module_file );
 
-	if ( $module_data && wp_normalize_path( $module_file ) === $module_data['raw_file'] ) {
-		unset( $module_data['raw'], $module_data['raw_file'] );
+	if ( $module_data && wp_normalize_path( $module_file ) === $module_data[ 'raw_file' ] ) {
+		unset( $module_data[ 'raw' ], $module_data[ 'raw_file' ] );
 	} else {
 		$module_data = get_file_data( $module_file, $default_headers, 'wordpoints_module' );
 	}
 
-	if ( ! empty( $module_data['update_api'] ) ) {
+	if ( ! empty( $module_data[ 'update_api' ] ) ) {
 		_deprecated_argument( __FUNCTION__, '1.10.0', 'The "Update API" module header has been deprecated in favor of "Channel".' );
 	}
 
-	$module_data['network'] = ( 'true' === strtolower( $module_data['network'] ) );
+	$module_data[ 'network' ] = ( 'true' === strtolower( $module_data[ 'network' ] ) );
 
 	if ( $markup || $translate ) {
 
@@ -277,7 +277,7 @@ 
 		// Translate fields
 		if ( $translate ) {
 
-			$textdomain = $module_data['text_domain'];
+			$textdomain = $module_data[ 'text_domain' ];
 
 			if ( $textdomain ) {
 
@@ -285,8 +285,8 @@ 
 
 					$domain_path = dirname( $module_file );
 
-					if ( $module_data['domain_path'] ) {
-						$domain_path .= $module_data['domain_path'];
+					if ( $module_data[ 'domain_path' ] ) {
+						$domain_path .= $module_data[ 'domain_path' ];
 					}
 
 					wordpoints_load_module_textdomain( $textdomain, $domain_path );
@@ -309,48 +309,48 @@ 
 		);
 
 		$allowed_tags = $allowed_tags_in_links;
-		$allowed_tags['a'] = array( 'href' => true, 'title' => true );
+		$allowed_tags[ 'a' ] = array( 'href' => true, 'title' => true );
 
 		// Name and author ar marked up inside <a> tags. Don't allow these.
-		$module_data['name']   = wp_kses( $module_data['name'],   $allowed_tags_in_links );
-		$module_data['author'] = wp_kses( $module_data['author'], $allowed_tags_in_links );
+		$module_data[ 'name' ]   = wp_kses( $module_data[ 'name' ], $allowed_tags_in_links );
+		$module_data[ 'author' ] = wp_kses( $module_data[ 'author' ], $allowed_tags_in_links );
 
-		$module_data['description'] = wp_kses( $module_data['description'], $allowed_tags );
-		$module_data['version']     = wp_kses( $module_data['version'],     $allowed_tags );
+		$module_data[ 'description' ] = wp_kses( $module_data[ 'description' ], $allowed_tags );
+		$module_data[ 'version' ]     = wp_kses( $module_data[ 'version' ], $allowed_tags );
 
-		$module_data['module_uri'] = esc_url( $module_data['module_uri'] );
-		$module_data['author_uri'] = esc_url( $module_data['author_uri'] );
+		$module_data[ 'module_uri' ] = esc_url( $module_data[ 'module_uri' ] );
+		$module_data[ 'author_uri' ] = esc_url( $module_data[ 'author_uri' ] );
 
-		$module_data['title']       = $module_data['name'];
-		$module_data['author_name'] = $module_data['author'];
+		$module_data[ 'title' ]       = $module_data[ 'name' ];
+		$module_data[ 'author_name' ] = $module_data[ 'author' ];
 
 		// Apply markup.
 		if ( $markup ) {
 
-			if ( $module_data['module_uri'] && $module_data['name'] ) {
-				$module_data['title'] = '<a href="' . $module_data['module_uri']
-					. '">' . $module_data['name'] . '</a>';
+			if ( $module_data[ 'module_uri' ] && $module_data[ 'name' ] ) {
+				$module_data[ 'title' ] = '<a href="' . $module_data[ 'module_uri' ]
+					. '">' . $module_data[ 'name' ] . '</a>';
 			}
 
-			if ( $module_data['author_uri'] && $module_data['author'] ) {
-				$module_data['author'] = '<a href="' . $module_data['author_uri']
-					. '">' . $module_data['author'] . '</a>';
+			if ( $module_data[ 'author_uri' ] && $module_data[ 'author' ] ) {
+				$module_data[ 'author' ] = '<a href="' . $module_data[ 'author_uri' ]
+					. '">' . $module_data[ 'author' ] . '</a>';
 			}
 
-			$module_data['description'] = wptexturize( $module_data['description'] );
+			$module_data[ 'description' ] = wptexturize( $module_data[ 'description' ] );
 
-			if ( $module_data['author'] ) {
-				$module_data['description'] .= ' <cite>'
+			if ( $module_data[ 'author' ] ) {
+				$module_data[ 'description' ] .= ' <cite>'
 					// translators: Author name.
-					. sprintf( __( 'By %s.', 'wordpoints' ), $module_data['author'] )
+					. sprintf( __( 'By %s.', 'wordpoints' ), $module_data[ 'author' ] )
 					. '</cite>';
 			}
 		}
 
 	} else {
 
-		$module_data['title']       = $module_data['name'];
-		$module_data['author_name'] = $module_data['author'];
+		$module_data[ 'title' ]       = $module_data[ 'name' ];
+		$module_data[ 'author_name' ] = $module_data[ 'author' ];
 
 	} // End if ( $markup || $translate ) else.
 
@@ -474,7 +474,7 @@ 
 
 		$module_data = wordpoints_get_module_data( $module_file, $markup, $translate );
 
-		if ( empty( $module_data['name'] ) ) {
+		if ( empty( $module_data[ 'name' ] ) ) {
 			continue;
 		}
 
@@ -573,7 +573,7 @@ 
  */
 function _wordpoints_sort_uname_callback( $a, $b ) {
 
-	return strnatcasecmp( $a['name'], $b['name'] );
+	return strnatcasecmp( $a[ 'name' ], $b[ 'name' ] );
 }
 
 /**
@@ -691,7 +691,7 @@ 
 
 	} else {
 
-		$current[] = $module;
+		$current[ ] = $module;
 		sort( $current );
 		update_option( 'wordpoints_active_modules', $current );
 	}
@@ -857,7 +857,7 @@ 
 	$checked = array();
 
 	foreach ( $modules as $module ) {
-		$checked[] = 'checked[]=' . $module;
+		$checked[ ] = 'checked[]=' . $module;
 	}
 
 	ob_start();
@@ -934,7 +934,7 @@ 
 		}
 
 		if ( ! $deleted ) {
-			$errors[] = $module_file;
+			$errors[ ] = $module_file;
 		}
 	}
 

src/includes/filters.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 add_action( 'admin_notices', 'WordPoints_Installables::admin_notices' );
 add_action( 'wpmu_new_blog', 'WordPoints_Installables::wpmu_new_blog' );
 
-if ( isset( $_GET['wordpoints_module_check'], $_GET['check_module'] ) ) { // WPCS: CSRF OK.
+if ( isset( $_GET[ 'wordpoints_module_check' ], $_GET[ 'check_module' ] ) ) { // WPCS: CSRF OK.
 
 	add_action( 'shutdown', 'wordpoints_maintenance_shutdown_print_rand_str' );
 

src/includes/functions.php

Spacing +22 added lines, -22 removed lines

@@ -64,13 +64,13 @@ 
 
 	// Normally this should never happen, because the plugins_loaded action won't
 	// run until the next request after WordPoints is activated and installs itself.
-	if ( empty( $wordpoints_data['version'] ) ) {
+	if ( empty( $wordpoints_data[ 'version' ] ) ) {
 		return;
 	}
 
 	// The major version is determined by the first number, so we can just cast to
 	// an integer. IF the major versions are equal, we don't need to do anything.
-	if ( (int) WORDPOINTS_VERSION === (int) $wordpoints_data['version'] ) {
+	if ( (int) WORDPOINTS_VERSION === (int) $wordpoints_data[ 'version' ] ) {
 		return;
 	}
 
@@ -88,7 +88,7 @@ 
  */
 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 +98,7 @@ 
 		$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 +128,7 @@ 
  */
 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,13 +138,13 @@ 
 		$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;
 	}
 
 	$modules = explode(
 		','
-		, sanitize_text_field( wp_unslash( $_GET['check_module'] ) ) // WPCS: CSRF OK.
+		, sanitize_text_field( wp_unslash( $_GET[ 'check_module' ] ) ) // WPCS: CSRF OK.
 	);
 
 	if ( 'pre_site_option_wordpoints_sitewide_active_modules' === current_filter() ) {
@@ -304,7 +304,7 @@ 
 				return false;
 			}
 
-			$values[] = $request[ $value ];
+			$values[ ] = $request[ $value ];
 		}
 
 		$action_format = vsprintf( $action_format, $values );
@@ -346,10 +346,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'
 		);
 
@@ -750,16 +750,16 @@ 
 
 	$options = array_merge( $defaults, $options );
 
-	echo '<select class="' . esc_attr( $options['class'] ) . '" name="' . esc_attr( $options['name'] ) . '" id="' . esc_attr( $options['id'] ) . '">';
-	echo '<option value="ALL"' . selected( $options['selected'], 'ALL', false ) . '>' . esc_html_x( 'Any', 'post type', 'wordpoints' ) . '</option>';
+	echo '<select class="' . esc_attr( $options[ 'class' ] ) . '" name="' . esc_attr( $options[ 'name' ] ) . '" id="' . esc_attr( $options[ 'id' ] ) . '">';
+	echo '<option value="ALL"' . selected( $options[ 'selected' ], 'ALL', false ) . '>' . esc_html_x( 'Any', 'post type', 'wordpoints' ) . '</option>';
 
 	foreach ( get_post_types( $args, 'objects' ) as $post_type ) {
 
-		if ( isset( $options['filter'] ) && ! call_user_func( $options['filter'], $post_type ) ) {
+		if ( isset( $options[ 'filter' ] ) && ! call_user_func( $options[ 'filter' ], $post_type ) ) {
 			continue;
 		}
 
-		echo '<option value="' . esc_attr( $post_type->name ) . '"' . selected( $options['selected'], $post_type->name, false ) . '>' . esc_html( $post_type->label ) . '</option>';
+		echo '<option value="' . esc_attr( $post_type->name ) . '"' . selected( $options[ 'selected' ], $post_type->name, false ) . '>' . esc_html( $post_type->label ) . '</option>';
 	}
 
 	echo '</select>';
@@ -936,11 +936,11 @@ 
 	switch ( $cap ) {
 		case 'install_wordpoints_modules':
 			if ( defined( 'DISALLOW_FILE_MODS' ) && DISALLOW_FILE_MODS ) {
-				$caps[] = 'do_not_allow';
+				$caps[ ] = 'do_not_allow';
 			} elseif ( is_multisite() && ! is_super_admin( $user_id ) ) {
-				$caps[] = 'do_not_allow';
+				$caps[ ] = 'do_not_allow';
 			} else {
-				$caps[] = $cap;
+				$caps[ ] = $cap;
 			}
 		break;
 	}
@@ -1116,13 +1116,13 @@ 
 		case 0:
 			return new $class_name();
 		case 1:
-			return new $class_name( $args[0] );
+			return new $class_name( $args[ 0 ] );
 		case 2:
-			return new $class_name( $args[0], $args[1] );
+			return new $class_name( $args[ 0 ], $args[ 1 ] );
 		case 3:
-			return new $class_name( $args[0], $args[1], $args[2] );
+			return new $class_name( $args[ 0 ], $args[ 1 ], $args[ 2 ] );
 		case 4:
-			return new $class_name( $args[0], $args[1], $args[2], $args[3] );
+			return new $class_name( $args[ 0 ], $args[ 1 ], $args[ 2 ], $args[ 3 ] );
 		default:
 			return false;
 	}

src/admin/includes/class-wordpoints-module-installer.php

Spacing +7 added lines, -7 removed lines

@@ -89,12 +89,12 @@ 
 			if ( $wp_filesystem->mkdir( $modules_dir, FS_CHMOD_DIR ) ) {
 				$wp_filesystem->put_contents( $modules_dir . '/index.php', '<?php // Gold is silent.' );
 			} else {
-				return new WP_Error( 'mkdir_failed_modules', $this->strings['mkdir_failed_modules'], $modules_dir );
+				return new WP_Error( 'mkdir_failed_modules', $this->strings[ 'mkdir_failed_modules' ], $modules_dir );
 			}
 		}
 
 		$module_dir = wordpoints_modules_dir();
-		$wp_theme_directories[] = $module_dir;
+		$wp_theme_directories[ ] = $module_dir;
 
 		$result = parent::install_package( $args );
 
@@ -188,14 +188,14 @@ 
 
 			$module_data = wordpoints_get_module_data( $file, false, false );
 
-			if ( ! empty( $module_data['name'] ) ) {
+			if ( ! empty( $module_data[ 'name' ] ) ) {
 				$modules_found = true;
 				break;
 			}
 		}
 
 		if ( ! $modules_found ) {
-			return new WP_Error( 'incompatible_archive_no_modules', $this->strings['incompatible_archive'], esc_html__( 'No valid modules were found.', 'wordpoints' ) );
+			return new WP_Error( 'incompatible_archive_no_modules', $this->strings[ 'incompatible_archive' ], esc_html__( 'No valid modules were found.', 'wordpoints' ) );
 		}
 
 		return $source;
@@ -212,11 +212,11 @@ 
 	 */
 	public function module_info() {
 
-		if ( ! is_array( $this->result ) || empty( $this->result['destination_name'] ) ) {
+		if ( ! is_array( $this->result ) || empty( $this->result[ 'destination_name' ] ) ) {
 			return false;
 		}
 
-		$module = wordpoints_get_modules( '/' . $this->result['destination_name'] );
+		$module = wordpoints_get_modules( '/' . $this->result[ 'destination_name' ] );
 
 		if ( empty( $module ) ) {
 			return false;
@@ -225,7 +225,7 @@ 
 		// Assume the requested module is the first in the list.
 		$module_files = array_keys( $module );
 
-		return $this->result['destination_name'] . '/' . $module_files[0];
+		return $this->result[ 'destination_name' ] . '/' . $module_files[ 0 ];
 	}
 
 } // class WordPoints_Module_Installer

src/admin/includes/class-wordpoints-module-installer-skin.php

Spacing +5 added lines, -5 removed lines

@@ -61,7 +61,7 @@ 
 
 		$args = array_merge( $defaults, $args );
 
-		$this->type = $args['type'];
+		$this->type = $args[ 'type' ];
 
 		parent::__construct( $args );
 	}
@@ -85,16 +85,16 @@ 
 
 			if ( is_multisite() && current_user_can( 'manage_network_wordpoints_modules' ) ) {
 
-				$install_actions['network_activate'] = '<a class="button button-primary" href="' . wp_nonce_url( self_admin_url( 'admin.php?page=wordpoints_modules&action=activate&amp;networkwide=1&amp;module=' . rawurlencode( $module_file ) ), "activate-module_{$module_file}" ) . '" target="_parent">' . esc_html__( 'Network Activate', 'wordpoints' ) . '</a>';
+				$install_actions[ 'network_activate' ] = '<a class="button button-primary" href="' . wp_nonce_url( self_admin_url( 'admin.php?page=wordpoints_modules&action=activate&amp;networkwide=1&amp;module=' . rawurlencode( $module_file ) ), "activate-module_{$module_file}" ) . '" target="_parent">' . esc_html__( 'Network Activate', 'wordpoints' ) . '</a>';
 
 			} elseif ( current_user_can( 'activate_wordpoints_modules' ) ) {
 
-				$install_actions['activate_module'] = '<a class="button button-primary" href="' . wp_nonce_url( self_admin_url( 'admin.php?page=wordpoints_modules&action=activate&amp;module=' . rawurlencode( $module_file ) ), "activate-module_{$module_file}" ) . '" target="_parent">' . esc_html__( 'Activate Module', 'wordpoints' ) . '</a>';
+				$install_actions[ 'activate_module' ] = '<a class="button button-primary" href="' . wp_nonce_url( self_admin_url( 'admin.php?page=wordpoints_modules&action=activate&amp;module=' . rawurlencode( $module_file ) ), "activate-module_{$module_file}" ) . '" target="_parent">' . esc_html__( 'Activate Module', 'wordpoints' ) . '</a>';
 			}
 		}
 
-		$install_actions['modules_page'] = '<a href="' . esc_url( self_admin_url( 'admin.php?page=wordpoints_modules' ) ) . '" target="_parent">' . esc_html__( 'Return to Modules page', 'wordpoints' ) . '</a>';
-		$install_actions['install_page'] = '<a href="' . esc_url( self_admin_url( 'admin.php?page=wordpoints_install_modules' ) ) . '" target="_parent">' . esc_html__( 'Return to Module Installer', 'wordpoints' ) . '</a>';
+		$install_actions[ 'modules_page' ] = '<a href="' . esc_url( self_admin_url( 'admin.php?page=wordpoints_modules' ) ) . '" target="_parent">' . esc_html__( 'Return to Modules page', 'wordpoints' ) . '</a>';
+		$install_actions[ 'install_page' ] = '<a href="' . esc_url( self_admin_url( 'admin.php?page=wordpoints_install_modules' ) ) . '" target="_parent">' . esc_html__( 'Return to Module Installer', 'wordpoints' ) . '</a>';
 
 		/**
 		 * The install module action links.

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