WordPoints / wordpoints

src/classes/hook/condition/number/greater/than.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
 			return false;
 		}
 
-		return $value > $settings['value'];
+		return $value > $settings[ 'value' ];
 	}
 }
 

src/classes/hook/condition/equals.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
 			}
 		}
 
-		return $settings['value'] === $value;
+		return $settings[ 'value' ] === $value;
 	}
 }
 

src/classes/entity.php

Doc Comments +1 added lines, -1 removed lines

@@ -453,7 +453,7 @@
 	 *
 	 * @since 2.1.0
 	 *
-	 * @return mixed The ID of the entity.
+	 * @return integer The ID of the entity.
 	 */
 	public function get_the_id() {
 

Spacing +3 added lines, -3 removed lines

@@ -408,10 +408,10 @@
 		} elseif ( $this->is_guid( $value ) ) {
 
 			$guid    = $this->split_guid( $value );
-			$context = $guid['context'];
-			$value   = $guid['id'];
+			$context = $guid[ 'context' ];
+			$value   = $guid[ 'id' ];
 
-			$entity = $this->get_entity_from_context( $guid['id'], $context );
+			$entity = $this->get_entity_from_context( $guid[ 'id' ], $context );
 
 		} else {
 

src/classes/extension/updatesi.php

Doc Comments +4 added lines

@@ -29,6 +29,7 @@ 
 	 * @since 2.4.0
 	 *
 	 * @param int $time The timestamp that the extension update check was performed.
+	 * @return void
 	 */
 	public function set_time_checked( $time );
 
@@ -48,6 +49,7 @@ 
 	 *
 	 * @param string[] $versions The extension versions that were checked, indexed by
 	 *                           extension basename.
+	 * @return void
 	 */
 	public function set_versions_checked( array $versions );
 
@@ -67,6 +69,7 @@ 
 	 *
 	 * @param string[] $versions The new extension versions that are available to
 	 *                           update to, indexed by extension basename.
+	 * @return void
 	 */
 	public function set_new_versions( array $versions );
 
@@ -99,6 +102,7 @@ 
 	 *
 	 * @param string $extension The extension basename.
 	 * @param string $version   The new version available.
+	 * @return void
 	 */
 	public function set_new_version( $extension, $version );
 }

src/classes/installablei.php

Doc Comments +10 added lines

@@ -57,6 +57,7 @@ 
 	 * @param string $version The version to set. Defaults to the code version.
 	 * @param bool   $network Whether to set the network version of the entity, or
 	 *                        just for the current site.
+	 * @return void
 	 */
 	public function set_db_version( $version = null, $network = false );
 
@@ -67,6 +68,7 @@ 
 	 *
 	 * @param bool $network Whether to delete the network version of the entity, or
 	 *                      just for the current site.
+	 * @return void
 	 */
 	public function unset_db_version( $network = false );
 
@@ -88,6 +90,7 @@ 
 	 * Sets this entity's status as network-installed.
 	 *
 	 * @since 2.4.0
+	 * @return void
 	 */
 	public function set_network_installed();
 
@@ -95,6 +98,7 @@ 
 	 * Deletes this entity's status as network-installed.
 	 *
 	 * @since 2.4.0
+	 * @return void
 	 */
 	public function unset_network_installed();
 
@@ -102,6 +106,7 @@ 
 	 * Sets that the per-site network installation has been skipped.
 	 *
 	 * @since 2.4.0
+	 * @return void
 	 */
 	public function set_network_install_skipped();
 
@@ -109,6 +114,7 @@ 
 	 * Deletes the network-install skipped flag for this entity.
 	 *
 	 * @since 2.4.0
+	 * @return void
 	 */
 	public function unset_network_install_skipped();
 
@@ -119,6 +125,7 @@ 
 	 *
 	 * @param string $updating_from The version that was being updated from. Defaults
 	 *                              to the current database version.
+	 * @return void
 	 */
 	public function set_network_update_skipped( $updating_from = null );
 
@@ -126,6 +133,7 @@ 
 	 * Deletes the network-update skipped flag for this entity.
 	 *
 	 * @since 2.4.0
+	 * @return void
 	 */
 	public function unset_network_update_skipped();
 
@@ -154,6 +162,7 @@ 
 	 * @since 2.4.0
 	 *
 	 * @param int $id The ID of the site to add. Defaults to the current site's ID.
+	 * @return void
 	 */
 	public function add_installed_site_id( $id = null );
 
@@ -161,6 +170,7 @@ 
 	 * Deletes the list of sites where this installable is installed.
 	 *
 	 * @since 2.4.0
+	 * @return void
 	 */
 	public function delete_installed_site_ids();
 

src/classes/installables.php

Doc Comments +1 added lines, -1 removed lines

@@ -211,7 +211,7 @@
 	 * @param string $version      The version of the installable. Only used if $un_installer is passed.
 	 * @param string $un_installer The path to the un/installer file.
 	 *
-	 * @return WordPoints_Un_Installer_Base|false The installer.
+	 * @return WordPoints_Un_Installer_Base The installer.
 	 */
 	public static function get_installer( $type, $slug, $version = null, $un_installer = null ) {
 

Spacing +5 added lines, -5 removed lines

@@ -102,8 +102,8 @@ 
 			'module' === $type ? 'extension' : $type
 			, $slug
 			, 'WordPoints_Installables::installer_loader'
-			, $data['version']
-			, ! empty( $data['network_wide'] )
+			, $data[ 'version' ]
+			, ! empty( $data[ 'network_wide' ] )
 		);
 
 		return true;
@@ -221,8 +221,8 @@ 
 				return false;
 			}
 
-			$version      = self::$registered[ $type ][ $slug ]['version'];
-			$un_installer = self::$registered[ $type ][ $slug ]['un_installer'];
+			$version      = self::$registered[ $type ][ $slug ][ 'version' ];
+			$un_installer = self::$registered[ $type ][ $slug ][ 'un_installer' ];
 		}
 
 		if ( ! isset( self::$installers[ $type ][ $slug ] ) ) {
@@ -271,7 +271,7 @@ 
 		return new WordPoints_Installable_Legacy(
 			$type
 			, $slug
-			, self::$registered[ $type ][ $slug ]['version']
+			, self::$registered[ $type ][ $slug ][ 'version' ]
 		);
 	}
 }

src/classes/modules.php

Doc Comments +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
 	 * @param string $module The full path to one of the module's files.
 	 * @param string $data   The slug of the piece of data to get.
 	 *
-	 * @return string[]|string|false The data, or false if it isn't registered.
+	 * @return string The data, or false if it isn't registered.
 	 */
 	public static function get_data( $module, $data = null ) {
 

Spacing +18 added lines, -18 removed lines

@@ -150,8 +150,8 @@ 
 
 		self::$registered[ $slug ] = self::parse_headers( $data );
 
-		self::$registered[ $slug ]['raw']      = $data;
-		self::$registered[ $slug ]['raw_file'] = wp_normalize_path( $file );
+		self::$registered[ $slug ][ 'raw' ]      = $data;
+		self::$registered[ $slug ][ 'raw_file' ] = wp_normalize_path( $file );
 
 		self::register_installable( $slug, $file );
 		self::maybe_load_textdomain( $slug, $file );
@@ -184,13 +184,13 @@ 
 
 			$parts = explode( ':', $line, 2 );
 
-			$data_slug = trim( $parts[0] );
+			$data_slug = trim( $parts[ 0 ] );
 
-			if ( isset( self::$default_headers[ $parts[0] ] ) ) {
-				$data_slug = self::$default_headers[ $parts[0] ];
+			if ( isset( self::$default_headers[ $parts[ 0 ] ] ) ) {
+				$data_slug = self::$default_headers[ $parts[ 0 ] ];
 			}
 
-			$parsed[ $data_slug ] = trim( $parts[1] );
+			$parsed[ $data_slug ] = trim( $parts[ 1 ] );
 		}
 
 		return $parsed + array_fill_keys( self::$default_headers, '' );
@@ -210,7 +210,7 @@ 
 			'extension'
 			, $slug
 			, 'WordPoints_Modules::get_installable'
-			, self::$registered[ $slug ]['version']
+			, self::$registered[ $slug ][ 'version' ]
 			, is_wordpoints_module_active_for_network( $file )
 		);
 	}
@@ -225,18 +225,18 @@ 
 	 */
 	private static function maybe_load_textdomain( $slug, $file ) {
 
-		if ( empty( self::$registered[ $slug ]['text_domain'] ) ) {
+		if ( empty( self::$registered[ $slug ][ 'text_domain' ] ) ) {
 			return;
 		}
 
 		$path = false;
 
-		if ( ! empty( self::$registered[ $slug ]['domain_path'] ) ) {
-			$path = wordpoints_module_basename( $file ) . self::$registered[ $slug ]['domain_path'];
+		if ( ! empty( self::$registered[ $slug ][ 'domain_path' ] ) ) {
+			$path = wordpoints_module_basename( $file ) . self::$registered[ $slug ][ 'domain_path' ];
 		}
 
 		wordpoints_load_module_textdomain(
-			self::$registered[ $slug ]['text_domain']
+			self::$registered[ $slug ][ 'text_domain' ]
 			, $path
 		);
 	}
@@ -264,8 +264,8 @@ 
 			WordPoints_Installables::get_installer(
 				'module'
 				, $slug
-				, self::$registered[ $slug ]['version']
-				, dirname( self::$registered[ $slug ]['raw_file'] ) . '/includes/class-un-installer.php'
+				, self::$registered[ $slug ][ 'version' ]
+				, dirname( self::$registered[ $slug ][ 'raw_file' ] ) . '/includes/class-un-installer.php'
 			)
 				->install( $network_wide );
 
@@ -296,14 +296,14 @@ 
 
 		$module = self::$registered[ $slug ];
 
-		$uninstaller = dirname( $module['raw_file'] ) . '/includes/class-un-installer.php';
+		$uninstaller = dirname( $module[ 'raw_file' ] ) . '/includes/class-un-installer.php';
 
 		if (
-			$module['namespace']
-			&& class_exists( "WordPoints_{$module['namespace']}_Installable" )
+			$module[ 'namespace' ]
+			&& class_exists( "WordPoints_{$module[ 'namespace' ]}_Installable" )
 		) {
 
-			$class       = "WordPoints_{$module['namespace']}_Installable";
+			$class       = "WordPoints_{$module[ 'namespace' ]}_Installable";
 			$installable = new $class( $slug );
 
 		} elseif ( file_exists( $uninstaller ) ) {
@@ -311,7 +311,7 @@ 
 			$installable = new WordPoints_Installable_Legacy(
 				'module'
 				, $slug
-				, $module['version']
+				, $module[ 'version' ]
 			);
 		}
 

src/classes/routinei.php

Doc Comments +1 added lines

@@ -18,6 +18,7 @@
 	 * Runs the routine.
 	 *
 	 * @since 2.4.0
+	 * @return null|boolean
 	 */
 	public function run();
 }

src/classes/un/installer/base.php

Doc Comments +1 added lines, -1 removed lines

@@ -701,7 +701,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 +42 added lines, -42 removed lines

@@ -893,7 +893,7 @@ 
 		$option_name = "{$this->option_prefix}installed_sites";
 
 		$sites   = wordpoints_get_array_option( $option_name, 'site' );
-		$sites[] = $id;
+		$sites[ ] = $id;
 
 		update_site_option( $option_name, $sites );
 	}
@@ -997,8 +997,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 );
@@ -1019,7 +1019,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;
 		}
 	}
 
@@ -1068,7 +1068,7 @@ 
 	 */
 	public function get_db_schema() {
 
-		if ( ! isset( $this->schema[ $this->context ]['tables'] ) ) {
+		if ( ! isset( $this->schema[ $this->context ][ 'tables' ] ) ) {
 			return '';
 		}
 
@@ -1084,7 +1084,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 );
@@ -1114,18 +1114,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' ] )
 				)
 			);
 		}
@@ -1165,7 +1165,7 @@ 
 	 */
 	protected function prepare_uninstall_list_tables() {
 
-		if ( ! isset( $this->uninstall['list_tables'] ) ) {
+		if ( ! isset( $this->uninstall[ 'list_tables' ] ) ) {
 			return;
 		}
 
@@ -1176,14 +1176,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' );
@@ -1279,7 +1279,7 @@ 
 			}
 
 			foreach ( $this->uninstall[ $type ][ $shortcut ] as $slug ) {
-				$this->uninstall[ $type ][ $canonical ][] = $args['prefix'] . $slug;
+				$this->uninstall[ $type ][ $canonical ][ ] = $args[ 'prefix' ] . $slug;
 			}
 		}
 	}
@@ -1392,7 +1392,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 );
 		}
 	}
@@ -1423,35 +1423,35 @@ 
 			, $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['transients'] as $transient ) {
+		foreach ( $uninstall[ 'transients' ] as $transient ) {
 			$this->uninstall_transient( $transient );
 		}
 
-		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 );
 		}
 	}
@@ -1479,7 +1479,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, '%' ) ) {
@@ -1546,20 +1546,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"
 				);
 			}
 		}
@@ -1606,7 +1606,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
@@ -1624,20 +1624,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}";
 			}
 		}
 
@@ -1817,7 +1817,7 @@ 
 	 */
 	protected function install_site() {
 
-		if ( isset( $this->schema['site'] ) ) {
+		if ( isset( $this->schema[ 'site' ] ) ) {
 			$this->install_db_schema();
 		}
 
@@ -1881,7 +1881,7 @@ 
 	 */
 	protected function uninstall_network() {
 
-		if ( ! empty( $this->uninstall['network'] ) ) {
+		if ( ! empty( $this->uninstall[ 'network' ] ) ) {
 			$this->uninstall_( 'network' );
 		}
 
@@ -1899,7 +1899,7 @@ 
 	 */
 	protected function uninstall_site() {
 
-		if ( ! empty( $this->uninstall['site'] ) ) {
+		if ( ! empty( $this->uninstall[ 'site' ] ) ) {
 			$this->uninstall_( 'site' );
 		}
 
@@ -1917,7 +1917,7 @@ 
 	 */
 	protected function uninstall_single() {
 
-		if ( ! empty( $this->uninstall['single'] ) ) {
+		if ( ! empty( $this->uninstall[ 'single' ] ) ) {
 			$this->uninstall_( 'single' );
 		}