WordPoints / wordpoints

src/classes/hook/extension/miss/listeneri.php

Doc Comments +1 added lines

@@ -20,6 +20,7 @@
 	 * @since 2.1.0
 	 *
 	 * @param WordPoints_Hook_Fire $fire The fire that was a miss.
+	 * @return void
 	 */
 	public function after_miss( WordPoints_Hook_Fire $fire );
 }

src/classes/hook/settingsi.php

Doc Comments +1 added lines

@@ -38,6 +38,7 @@
 	 *
 	 * @param WordPoints_Hook_ReactionI $reaction The reaction object.
 	 * @param array                     $settings The settings for a hook reaction.
+	 * @return void
 	 */
 	public function update_settings(
 		WordPoints_Hook_ReactionI $reaction,

src/components/points/classes/logs/viewing/restrictioni.php

Doc Comments +1 added lines

@@ -18,6 +18,7 @@
 	 * @since 2.2.0
 	 *
 	 * @param object $log The object for the points log the restriction is for.
+	 * @return void
 	 */
 	public function __construct( $log );
 

src/classes/shortcode.php

Doc Comments +1 added lines, -1 removed lines

@@ -280,7 +280,7 @@
 	 *
 	 * @since 1.8.0
 	 *
-	 * @return string|WP_Error|null An error message or object on failure.
+	 * @return string|null An error message or object on failure.
 	 */
 	protected function verify_atts() {
 

Spacing +6 added lines, -6 removed lines

@@ -171,9 +171,9 @@ 
 	protected function verify_atts() {
 
 		// If this shortcode supports the user_id attribute, verify it.
-		if ( isset( $this->pairs['user_id'] ) ) {
+		if ( isset( $this->pairs[ 'user_id' ] ) ) {
 
-			if ( 'post_author' === $this->atts['user_id'] ) {
+			if ( 'post_author' === $this->atts[ 'user_id' ] ) {
 
 				$post = get_post();
 
@@ -186,13 +186,13 @@ 
 					);
 				}
 
-				$this->atts['user_id'] = $post->post_author;
+				$this->atts[ 'user_id' ] = $post->post_author;
 
-			} elseif ( ! $this->atts['user_id'] ) {
+			} elseif ( ! $this->atts[ 'user_id' ] ) {
 
-				$this->atts['user_id'] = get_current_user_id();
+				$this->atts[ 'user_id' ] = get_current_user_id();
 
-			} elseif ( ! wordpoints_posint( $this->atts['user_id'] ) ) {
+			} elseif ( ! wordpoints_posint( $this->atts[ 'user_id' ] ) ) {
 
 				return sprintf(
 					// translators: 1. Attribute name; 2. Shortcode name; 3. Expected value.

src/classes/un/installer/base.php

Doc Comments +1 added lines, -1 removed lines

@@ -697,7 +697,7 @@
 	 * @since 2.0.0
 	 *
 	 * @param string $option The name of the option to set.
-	 * @param mixed  $value  The value of the option.
+	 * @param string  $value  The value of the option.
 	 */
 	private function _set_option( $option, $value = true ) {
 

Spacing +55 added lines, -55 removed lines

@@ -922,7 +922,7 @@ 
 		$option_name = $this->_get_installed_site_ids_option_name();
 
 		$sites = wordpoints_get_array_option( $option_name, 'site' );
-		$sites[] = $id;
+		$sites[ ] = $id;
 
 		update_site_option( $option_name, $sites );
 	}
@@ -984,12 +984,12 @@ 
 
 		if ( 'wordpoints' === $this->slug ) {
 
-			if ( isset( $wordpoints_data['version'] ) ) {
-				return $wordpoints_data['version'];
+			if ( isset( $wordpoints_data[ 'version' ] ) ) {
+				return $wordpoints_data[ 'version' ];
 			}
 
-		} elseif ( isset( $wordpoints_data[ "{$this->type}s" ][ $this->slug ]['version'] ) ) {
-			return $wordpoints_data[ "{$this->type}s" ][ $this->slug ]['version'];
+		} elseif ( isset( $wordpoints_data[ "{$this->type}s" ][ $this->slug ][ 'version' ] ) ) {
+			return $wordpoints_data[ "{$this->type}s" ][ $this->slug ][ 'version' ];
 		}
 
 		return false;
@@ -1016,9 +1016,9 @@ 
 		}
 
 		if ( 'wordpoints' === $this->slug ) {
-			$wordpoints_data['version'] = $version;
+			$wordpoints_data[ 'version' ] = $version;
 		} else {
-			$wordpoints_data[ "{$this->type}s" ][ $this->slug ]['version'] = $version;
+			$wordpoints_data[ "{$this->type}s" ][ $this->slug ][ 'version' ] = $version;
 		}
 
 		if ( $this->network_wide ) {
@@ -1074,8 +1074,8 @@ 
 			'wordpoints_data'
 		);
 
-		if ( empty( $wordpoints_data['components'][ $component ]['version'] ) ) {
-			$wordpoints_data['components'][ $component ]['version'] = $version;
+		if ( empty( $wordpoints_data[ 'components' ][ $component ][ 'version' ] ) ) {
+			$wordpoints_data[ 'components' ][ $component ][ 'version' ] = $version;
 		}
 
 		wordpoints_update_maybe_network_option( 'wordpoints_data', $wordpoints_data );
@@ -1096,7 +1096,7 @@ 
 		$this->custom_caps_keys = array_keys( $this->custom_caps );
 
 		if ( 'uninstall' === $this->action ) {
-			$this->uninstall['local']['custom_caps'] = true;
+			$this->uninstall[ 'local' ][ 'custom_caps' ] = true;
 		}
 	}
 
@@ -1145,7 +1145,7 @@ 
 	 */
 	public function get_db_schema() {
 
-		if ( ! isset( $this->schema[ $this->context ]['tables'] ) ) {
+		if ( ! isset( $this->schema[ $this->context ][ 'tables' ] ) ) {
 			return '';
 		}
 
@@ -1161,7 +1161,7 @@ 
 			$prefix = $wpdb->base_prefix;
 		}
 
-		foreach ( $this->schema[ $this->context ]['tables'] as $table_name => $table_schema ) {
+		foreach ( $this->schema[ $this->context ][ 'tables' ] as $table_name => $table_schema ) {
 
 			$table_name   = str_replace( '`', '``', $table_name );
 			$table_schema = trim( $table_schema );
@@ -1191,18 +1191,18 @@ 
 		// Add any tables to uninstall based on the db schema.
 		foreach ( $this->schema as $context => $schema ) {
 
-			if ( ! isset( $schema['tables'] ) ) {
+			if ( ! isset( $schema[ 'tables' ] ) ) {
 				continue;
 			}
 
-			if ( ! isset( $this->uninstall[ $context ]['tables'] ) ) {
-				$this->uninstall[ $context ]['tables'] = array();
+			if ( ! isset( $this->uninstall[ $context ][ 'tables' ] ) ) {
+				$this->uninstall[ $context ][ 'tables' ] = array();
 			}
 
-			$this->uninstall[ $context ]['tables'] = array_unique(
+			$this->uninstall[ $context ][ 'tables' ] = array_unique(
 				array_merge(
-					$this->uninstall[ $context ]['tables']
-					, array_keys( $schema['tables'] )
+					$this->uninstall[ $context ][ 'tables' ]
+					, array_keys( $schema[ 'tables' ] )
 				)
 			);
 		}
@@ -1242,7 +1242,7 @@ 
 	 */
 	protected function prepare_uninstall_list_tables() {
 
-		if ( ! isset( $this->uninstall['list_tables'] ) ) {
+		if ( ! isset( $this->uninstall[ 'list_tables' ] ) ) {
 			return;
 		}
 
@@ -1253,14 +1253,14 @@ 
 		);
 
 		// Loop through all of the list table screens.
-		foreach ( $this->uninstall['list_tables'] as $screen_id => $args ) {
+		foreach ( $this->uninstall[ 'list_tables' ] as $screen_id => $args ) {
 
 			// Back-compat for pre-2.1.0.
-			if ( isset( $args['parent'] ) && '_page' === substr( $args['parent'], -5 /* _page */ ) ) {
-				$args['parent'] = substr( $args['parent'], 0, -5 );
+			if ( isset( $args[ 'parent' ] ) && '_page' === substr( $args[ 'parent' ], -5 /* _page */ ) ) {
+				$args[ 'parent' ] = substr( $args[ 'parent' ], 0, -5 );
 			}
 
-			$this->uninstall['universal']['list_tables'][ $screen_id ] = $args;
+			$this->uninstall[ 'universal' ][ 'list_tables' ][ $screen_id ] = $args;
 		}
 
 		$this->prepare_uninstall_non_per_site_items( 'list_tables' );
@@ -1283,20 +1283,20 @@ 
 	 */
 	protected function prepare_uninstall_non_per_site_items( $items_key ) {
 
-		if ( isset( $this->uninstall['universal'][ $items_key ] ) ) {
+		if ( isset( $this->uninstall[ 'universal' ][ $items_key ] ) ) {
 
-			$this->uninstall['global'][ $items_key ]
-				= $this->uninstall['universal'][ $items_key ];
+			$this->uninstall[ 'global' ][ $items_key ]
+				= $this->uninstall[ 'universal' ][ $items_key ];
 
-			unset( $this->uninstall['universal'][ $items_key ] );
+			unset( $this->uninstall[ 'universal' ][ $items_key ] );
 		}
 
-		if ( isset( $this->uninstall['site'][ $items_key ] ) ) {
+		if ( isset( $this->uninstall[ 'site' ][ $items_key ] ) ) {
 
-			$this->uninstall['network'][ $items_key ]
-				= $this->uninstall['site'][ $items_key ];
+			$this->uninstall[ 'network' ][ $items_key ]
+				= $this->uninstall[ 'site' ][ $items_key ];
 
-			unset( $this->uninstall['site'][ $items_key ] );
+			unset( $this->uninstall[ 'site' ][ $items_key ] );
 		}
 	}
 
@@ -1346,7 +1346,7 @@ 
 			}
 
 			foreach ( $this->uninstall[ $type ][ $shortcut ] as $slug ) {
-				$this->uninstall[ $type ][ $canonical ][] = $args['prefix'] . $slug;
+				$this->uninstall[ $type ][ $canonical ][ ] = $args[ 'prefix' ] . $slug;
 			}
 		}
 	}
@@ -1459,7 +1459,7 @@ 
 			$prefix = $wpdb->prefix;
 		}
 
-		foreach ( $this->schema[ $type ]['tables'] as $table_name => $schema ) {
+		foreach ( $this->schema[ $type ][ 'tables' ] as $table_name => $schema ) {
 			maybe_convert_table_to_utf8mb4( $prefix . $table_name );
 		}
 	}
@@ -1489,31 +1489,31 @@ 
 			, $this->uninstall[ $type ]
 		);
 
-		if ( ! empty( $uninstall['custom_caps'] ) ) {
+		if ( ! empty( $uninstall[ 'custom_caps' ] ) ) {
 			$this->uninstall_custom_caps( $this->custom_caps_keys );
 		}
 
-		foreach ( $uninstall['user_meta'] as $meta_key ) {
+		foreach ( $uninstall[ 'user_meta' ] as $meta_key ) {
 			$this->uninstall_metadata( 'user', $meta_key );
 		}
 
-		foreach ( $uninstall['options'] as $option ) {
+		foreach ( $uninstall[ 'options' ] as $option ) {
 			$this->uninstall_option( $option );
 		}
 
-		foreach ( $uninstall['tables'] as $table ) {
+		foreach ( $uninstall[ 'tables' ] as $table ) {
 			$this->uninstall_table( $table );
 		}
 
-		foreach ( $uninstall['comment_meta'] as $meta_key ) {
+		foreach ( $uninstall[ 'comment_meta' ] as $meta_key ) {
 			$this->uninstall_metadata( 'comment', $meta_key );
 		}
 
-		foreach ( $uninstall['meta_boxes'] as $screen_id => $args ) {
+		foreach ( $uninstall[ 'meta_boxes' ] as $screen_id => $args ) {
 			$this->uninstall_meta_boxes( $screen_id, $args );
 		}
 
-		foreach ( $uninstall['list_tables'] as $screen_id => $args ) {
+		foreach ( $uninstall[ 'list_tables' ] as $screen_id => $args ) {
 			$this->uninstall_list_table( $screen_id, $args );
 		}
 	}
@@ -1541,7 +1541,7 @@ 
 	protected function uninstall_metadata( $type, $key ) {
 
 		if ( 'user' === $type && 'site' === $this->context ) {
-			$key = $GLOBALS['wpdb']->get_blog_prefix() . $key;
+			$key = $GLOBALS[ 'wpdb' ]->get_blog_prefix() . $key;
 		}
 
 		if ( false !== strpos( $key, '%' ) ) {
@@ -1608,20 +1608,20 @@ 
 		);
 
 		$args = array_merge( $defaults, $args );
-		$args['options'] = array_merge( $defaults['options'], $args['options'] );
+		$args[ 'options' ] = array_merge( $defaults[ 'options' ], $args[ 'options' ] );
 
 		// Each user gets to set the options to their liking.
-		foreach ( $args['options'] as $option ) {
+		foreach ( $args[ 'options' ] as $option ) {
 
 			$this->uninstall_metadata(
 				'user'
-				, "{$option}_{$args['parent']}_page_{$screen_id}"
+				, "{$option}_{$args[ 'parent' ]}_page_{$screen_id}"
 			);
 
 			if ( 'network' === $this->context ) {
 				$this->uninstall_metadata(
 					'user'
-					, "{$option}_{$args['parent']}_page_{$screen_id}-network"
+					, "{$option}_{$args[ 'parent' ]}_page_{$screen_id}-network"
 				);
 			}
 		}
@@ -1668,7 +1668,7 @@ 
 
 		$args = array_merge( $defaults, $args );
 
-		$network_parent = $args['parent'];
+		$network_parent = $args[ 'parent' ];
 		$parent = $network_parent;
 
 		// The parent page is usually the same on a multisite site, but we need to
@@ -1680,20 +1680,20 @@ 
 		$meta_keys = array();
 
 		// Each user can hide specific columns of the table.
-		$meta_keys[] = "manage{$parent}_page_{$screen_id}columnshidden";
+		$meta_keys[ ] = "manage{$parent}_page_{$screen_id}columnshidden";
 
 		if ( 'network' === $this->context ) {
-			$meta_keys[] = "manage{$network_parent}_page_{$screen_id}-networkcolumnshidden";
+			$meta_keys[ ] = "manage{$network_parent}_page_{$screen_id}-networkcolumnshidden";
 		}
 
 		// Loop through each of the other options provided by this list table.
-		foreach ( $args['options'] as $option ) {
+		foreach ( $args[ 'options' ] as $option ) {
 
 			// Each user gets to set the options to their liking.
-			$meta_keys[]  = "{$parent}_page_{$screen_id}_{$option}";
+			$meta_keys[ ]  = "{$parent}_page_{$screen_id}_{$option}";
 
 			if ( 'network' === $this->context ) {
-				$meta_keys[] = "{$network_parent}_page_{$screen_id}_network_{$option}";
+				$meta_keys[ ] = "{$network_parent}_page_{$screen_id}_network_{$option}";
 			}
 		}
 
@@ -1857,7 +1857,7 @@ 
 	 */
 	protected function install_site() {
 
-		if ( isset( $this->schema['site'] ) ) {
+		if ( isset( $this->schema[ 'site' ] ) ) {
 			$this->install_db_schema();
 		}
 
@@ -1921,7 +1921,7 @@ 
 	 */
 	protected function uninstall_network() {
 
-		if ( ! empty( $this->uninstall['network'] ) ) {
+		if ( ! empty( $this->uninstall[ 'network' ] ) ) {
 			$this->uninstall_( 'network' );
 		}
 
@@ -1939,7 +1939,7 @@ 
 	 */
 	protected function uninstall_site() {
 
-		if ( ! empty( $this->uninstall['site'] ) ) {
+		if ( ! empty( $this->uninstall[ 'site' ] ) ) {
 			$this->uninstall_( 'site' );
 		}
 
@@ -1957,7 +1957,7 @@ 
 	 */
 	protected function uninstall_single() {
 
-		if ( ! empty( $this->uninstall['single'] ) ) {
+		if ( ! empty( $this->uninstall[ 'single' ] ) ) {
 			$this->uninstall_( 'single' );
 		}
 

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

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

Braces +2 added lines, -1 removed lines

@@ -28,7 +28,8 @@
 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/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.