WordPoints / wordpoints

src/components/points/admin/includes/functions.php

Braces +9 added lines, -4 removed lines

@@ -143,7 +143,8 @@ 
  */
 function wordpoints_points_admin_screen_hooks() {
 
-	if ( isset( $_GET['edithook'] ) || isset( $_POST['savehook'] ) || isset( $_POST['removehook'] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['edithook'] ) || isset( $_POST['savehook'] ) || isset( $_POST['removehook'] ) ) {
+// WPCS: CSRF OK.
 
 		// - We're doing this without AJAX (JS).
 
@@ -206,7 +207,8 @@ 
  */
 function wordpoints_no_js_points_hooks_save() {
 
-	if ( ! isset( $_POST['savehook'] ) && ! isset( $_POST['removehook'] ) ) { // WPCS: CSRF OK.
+	if ( ! isset( $_POST['savehook'] ) && ! isset( $_POST['removehook'] ) ) {
+// WPCS: CSRF OK.
 		return;
 	}
 
@@ -290,9 +292,12 @@ 
 
 	<?php if ( $has_form ) : ?>
 		<hr />
-	<?php else : ?>
+	<?php else {
+	: ?>
 		<br />
-	<?php endif; ?>
+	<?php endif;
+}
+?>
 
 	<div class="hook-instance-description">
 		<label for="<?php $hook->the_field_id( '_description' ); ?>"><?php echo esc_html_x( 'Description (optional):', 'points hook', 'wordpoints' ); ?></label>

Spacing +6 added lines, -6 removed lines

@@ -438,17 +438,17 @@
 
 		if (
 			isset(
-				$_POST[ "wordpoints_points_set-{$slug}" ]
-				, $_POST[ "wordpoints_points-{$slug}" ]
-				, $_POST[ "wordpoints_points_old-{$slug}" ]
+				$_POST["wordpoints_points_set-{$slug}"]
+				, $_POST["wordpoints_points-{$slug}"]
+				, $_POST["wordpoints_points_old-{$slug}"]
 			)
-			&& false !== wordpoints_int( $_POST[ "wordpoints_points-{$slug}" ] )
-			&& false !== wordpoints_int( $_POST[ "wordpoints_points_old-{$slug}" ] )
+			&& false !== wordpoints_int( $_POST["wordpoints_points-{$slug}"] )
+			&& false !== wordpoints_int( $_POST["wordpoints_points_old-{$slug}"] )
 		) {
 
 			wordpoints_alter_points(
 				$user_id
-				, (int) $_POST[ "wordpoints_points-{$slug}" ] - (int) $_POST[ "wordpoints_points_old-{$slug}" ]
+				, (int) $_POST["wordpoints_points-{$slug}"] - (int) $_POST["wordpoints_points_old-{$slug}"]
 				, $slug
 				, 'profile_edit'
 				, array(

src/components/points/admin/screens/hooks.php

Braces +4 added lines, -2 removed lines

@@ -43,7 +43,8 @@ 
 		return;
 	}
 
-	if ( isset( $_GET['message'] ) && isset( $messages[ (int) $_GET['message'] ] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['message'] ) && isset( $messages[ (int) $_GET['message'] ] ) ) {
+// WPCS: CSRF OK.
 
 		wordpoints_show_admin_message(
 			esc_html( $messages[ (int) $_GET['message'] ] ) // WPCS: CSRF OK.
@@ -51,7 +52,8 @@ 
 			, array( 'dismissible' => true )
 		);
 
-	} elseif ( isset( $_GET['error'] ) && isset( $errors[ (int) $_GET['error'] ] ) ) { // WPCS: CSRF OK.
+	} elseif ( isset( $_GET['error'] ) && isset( $errors[ (int) $_GET['error'] ] ) ) {
+// WPCS: CSRF OK.
 
 		wordpoints_show_admin_error(
 			esc_html( $errors[ (int) $_GET['error'] ] ) // WPCS: CSRF OK.

Spacing +4 added lines, -4 removed lines

@@ -43,18 +43,18 @@
 		return;
 	}
 
-	if ( isset( $_GET['message'] ) && isset( $messages[ (int) $_GET['message'] ] ) ) { // WPCS: CSRF OK.
+	if ( isset( $_GET['message'] ) && isset( $messages[(int) $_GET['message']] ) ) { // WPCS: CSRF OK.
 
 		wordpoints_show_admin_message(
-			esc_html( $messages[ (int) $_GET['message'] ] ) // WPCS: CSRF OK.
+			esc_html( $messages[(int) $_GET['message']] ) // WPCS: CSRF OK.
 			, 'success'
 			, array( 'dismissible' => true )
 		);
 
-	} elseif ( isset( $_GET['error'] ) && isset( $errors[ (int) $_GET['error'] ] ) ) { // WPCS: CSRF OK.
+	} elseif ( isset( $_GET['error'] ) && isset( $errors[(int) $_GET['error']] ) ) { // WPCS: CSRF OK.
 
 		wordpoints_show_admin_error(
-			esc_html( $errors[ (int) $_GET['error'] ] ) // WPCS: CSRF OK.
+			esc_html( $errors[(int) $_GET['error']] ) // WPCS: CSRF OK.
 			, array( 'dismissible' => true )
 		);
 	}

src/components/points/classes/logs/view/table.php

Braces +2 added lines, -1 removed lines

@@ -222,7 +222,8 @@
 
 		$search_term = '';
 
-		if ( isset( $_POST['wordpoints_points_logs_search'] ) ) { // WPCS: CSRF OK
+		if ( isset( $_POST['wordpoints_points_logs_search'] ) ) {
+// WPCS: CSRF OK
 			$search_term = trim(
 				sanitize_text_field(
 					wp_unslash( $_POST['wordpoints_points_logs_search'] ) // WPCS: CSRF OK

Spacing +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
 		$extra_classes = array();
 
 		foreach ( array( 'searchable', 'paginate' ) as $arg ) {
-			if ( $this->args[ $arg ] ) {
+			if ( $this->args[$arg] ) {
 				$extra_classes[] = $arg;
 			}
 		}

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 +6 added lines, -6 removed lines

@@ -196,8 +196,8 @@ 
 		$context = $this->get_context();
 
 		return (
-			isset( $guid[ $this->get_slug() ] )
-			&& ( '' === $context || isset( $guid[ $context ] ) )
+			isset( $guid[$this->get_slug()] )
+			&& ( '' === $context || isset( $guid[$context] ) )
 		);
 	}
 
@@ -216,10 +216,10 @@ 
 	protected function split_guid( $guid ) {
 
 		$slug    = $this->get_slug();
-		$id      = $guid[ $slug ];
+		$id      = $guid[$slug];
 		$context = $guid;
 
-		unset( $context[ $slug ] );
+		unset( $context[$slug] );
 
 		return array( 'id' => $id, 'context' => $context );
 	}
@@ -237,8 +237,8 @@ 
 	protected function get_attr_value( $entity, $attr ) {
 
 		if ( is_array( $entity ) ) {
-			if ( isset( $entity[ $attr ] ) ) {
-				return $entity[ $attr ];
+			if ( isset( $entity[$attr] ) ) {
+				return $entity[$attr];
 			}
 		} else {
 			if ( isset( $entity->{$attr} ) ) {

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 +10 added lines, -10 removed lines

@@ -94,7 +94,7 @@ 
 			, 'WordPoints_Installables_App::register'
 		);
 
-		self::$registered[ $type ][ $slug ] = $data;
+		self::$registered[$type][$slug] = $data;
 
 		/** @var WordPoints_Installables_App $installables */
 		$installables = wordpoints_apps()->get_sub_app( 'installables' );
@@ -125,7 +125,7 @@ 
 
 		_deprecated_function( __METHOD__, '2.4.0' );
 
-		if ( ! isset( self::$registered[ $type ][ $slug ] ) ) {
+		if ( ! isset( self::$registered[$type][$slug] ) ) {
 			return false;
 		}
 
@@ -155,7 +155,7 @@ 
 
 		_deprecated_function( __METHOD__, '2.4.0' );
 
-		if ( ! isset( self::$registered[ $type ][ $slug ] ) ) {
+		if ( ! isset( self::$registered[$type][$slug] ) ) {
 			return false;
 		}
 
@@ -217,24 +217,24 @@ 
 
 		if ( ! isset( $un_installer ) ) {
 
-			if ( ! isset( self::$registered[ $type ][ $slug ] ) ) {
+			if ( ! isset( self::$registered[$type][$slug] ) ) {
 				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 ] ) ) {
+		if ( ! isset( self::$installers[$type][$slug] ) ) {
 
 			if ( ! file_exists( $un_installer ) ) {
 				return false;
 			}
 
-			self::$installers[ $type ][ $slug ] = require $un_installer;
+			self::$installers[$type][$slug] = require $un_installer;
 		}
 
-		return new self::$installers[ $type ][ $slug ]( $slug, $version );
+		return new self::$installers[$type][$slug]( $slug, $version );
 	}
 
 	/**
@@ -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 +20 added lines, -20 removed lines

@@ -115,19 +115,19 @@ 
 
 		$slug = self::get_slug( $module );
 
-		if ( ! isset( self::$registered[ $slug ] ) ) {
+		if ( ! isset( self::$registered[$slug] ) ) {
 			return false;
 		}
 
 		if ( $data ) {
-			if ( ! isset( self::$registered[ $slug ][ $data ] ) ) {
+			if ( ! isset( self::$registered[$slug][$data] ) ) {
 				return false;
 			}
 
-			return self::$registered[ $slug ][ $data ];
+			return self::$registered[$slug][$data];
 		}
 
-		return self::$registered[ $slug ];
+		return self::$registered[$slug];
 	}
 
 	/**
@@ -144,14 +144,14 @@ 
 
 		$slug = self::get_slug( $file );
 
-		if ( isset( self::$registered[ $slug ] ) ) {
+		if ( isset( self::$registered[$slug] ) ) {
 			return false;
 		}
 
-		self::$registered[ $slug ] = self::parse_headers( $data );
+		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 );
@@ -186,11 +186,11 @@ 
 
 			$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 );
 
@@ -290,11 +290,11 @@ 
 
 		$installable = false;
 
-		if ( ! isset( self::$registered[ $slug ] ) ) {
+		if ( ! isset( self::$registered[$slug] ) ) {
 			return $installable;
 		}
 
-		$module = self::$registered[ $slug ];
+		$module = self::$registered[$slug];
 
 		$uninstaller = dirname( $module['raw_file'] ) . '/includes/class-un-installer.php';
 

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