pryley / site-reviews

plugin/Modules/Notification.php

Spacing +16 added lines, -16 removed lines

@@ -33,7 +33,7 @@ 
 	public function __construct()
 	{
 		$types = glsr( OptionManager::class )->get( 'settings.general.notifications', [] );
-		$this->email = count( array_intersect( ['admin', 'custom'], $types )) > 0;
+		$this->email = count( array_intersect( ['admin', 'custom'], $types ) ) > 0;
 		$this->slack = in_array( 'slack', $types );
 		$this->types = $types;
 	}
@@ -43,7 +43,7 @@ 
 	 */
 	public function send( Review $review )
 	{
-		if( empty( $this->types ))return;
+		if( empty($this->types) )return;
 		$this->review = $review;
 		$args = [
 			'link' => $this->getLink(),
@@ -58,7 +58,7 @@ 
 	 */
 	protected function buildEmail( array $args )
 	{
-		return glsr( Email::class )->compose([
+		return glsr( Email::class )->compose( [
 			'to' => $this->getEmailAddresses(),
 			'subject' => $args['title'],
 			'template' => 'email-notification',
@@ -71,7 +71,7 @@ 
 				'review_rating' => $this->review->rating,
 				'review_title' => $this->review->title,
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -83,7 +83,7 @@ 
 			'button_url' => $args['link'],
 			'fallback' => $this->buildEmail( $args )->read( 'plaintext' ),
 			'pretext' => $args['title'],
-		]);
+		] );
 	}
 
 	/**
@@ -92,23 +92,23 @@ 
 	protected function getEmailAddresses()
 	{
 		$emails = [];
-		if( in_array( 'admin', $this->types )) {
+		if( in_array( 'admin', $this->types ) ) {
 			$emails[] = get_option( 'admin_email' );
 		}
-		if( in_array( 'author', $this->types )) {
-			$assignedPost = get_post( intval( $this->review->assigned_to ));
+		if( in_array( 'author', $this->types ) ) {
+			$assignedPost = get_post( intval( $this->review->assigned_to ) );
 			if( $assignedPost instanceof WP_Post ) {
 				$this->email = true;
-				$emails[] = get_the_author_meta( 'user_email', intval( $assignedPost->post_author ));
+				$emails[] = get_the_author_meta( 'user_email', intval( $assignedPost->post_author ) );
 			}
 		}
-		if( in_array( 'custom', $this->types )) {
+		if( in_array( 'custom', $this->types ) ) {
 			$customEmails = glsr( OptionManager::class )->get( 'settings.general.notification_email' );
 			$customEmails = str_replace( [' ', ',', ';'], ',', $customEmails );
 			$customEmails = explode( ',', $customEmails );
 			$emails = array_merge( $emails, $customEmails );
 		}
-		$emails = array_filter( array_keys( array_flip( $emails )));
+		$emails = array_filter( array_keys( array_flip( $emails ) ) );
 		return apply_filters( 'site-reviews/notification/emails', $emails, $this->review );
 	}
 
@@ -125,16 +125,16 @@ 
 	 */
 	protected function getTitle()
 	{
-		$assignedTitle = get_the_title( intval( $this->review->assigned_to ));
+		$assignedTitle = get_the_title( intval( $this->review->assigned_to ) );
 		$title = _nx(
 			'New %s-star review',
 			'New %s-star review of: %s',
-			intval( empty( $assignedTitle )),
+			intval( empty($assignedTitle) ),
 			'This string differs depending on whether or not the review has been assigned to a post.',
 			'site-reviews'
 		);
 		$title = sprintf( '[%s] %s',
-			wp_specialchars_decode( strval( get_option( 'blogname' )), ENT_QUOTES ),
+			wp_specialchars_decode( strval( get_option( 'blogname' ) ), ENT_QUOTES ),
 			sprintf( $title, $this->review->rating, $assignedTitle )
 		);
 		return apply_filters( 'site-reviews/notification/title', $title, $this->review );
@@ -146,7 +146,7 @@ 
 	protected function sendToEmail( array $args )
 	{
 		$email = $this->buildEmail( $args );
-		if( empty( $email->to )) {
+		if( empty($email->to) ) {
 			glsr_log()->error( 'Email notification was not sent (missing email address)' );
 			return;
 		}
@@ -161,7 +161,7 @@ 
 		if( !$this->slack )return;
 		$notification = $this->buildSlackNotification( $args );
 		$result = $notification->send();
-		if( is_wp_error( $result )) {
+		if( is_wp_error( $result ) ) {
 			$notification->review = null;
 			glsr_log()->error( $result->get_error_message() )->debug( $notification );
 		}

Braces +6 added lines, -2 removed lines

@@ -43,7 +43,9 @@ 
 	 */
 	public function send( Review $review )
 	{
-		if( empty( $this->types ))return;
+		if( empty( $this->types )) {
+			return;
+		}
 		$this->review = $review;
 		$args = [
 			'link' => $this->getLink(),
@@ -158,7 +160,9 @@ 
 	 */
 	protected function sendToSlack( array $args )
 	{
-		if( !$this->slack )return;
+		if( !$this->slack ) {
+			return;
+		}
 		$notification = $this->buildSlackNotification( $args );
 		$result = $notification->send();
 		if( is_wp_error( $result )) {

plugin/Modules/Html/Settings.php

Spacing +19 added lines, -19 removed lines

@@ -54,7 +54,7 @@ 
 	 */
 	protected function getSettingFields( $path )
 	{
-		return array_filter( $this->settings, function( $key ) use( $path ) {
+		return array_filter( $this->settings, function( $key ) use($path) {
 			return glsr( Helper::class )->startsWith( $path, $key );
 		}, ARRAY_FILTER_USE_KEY );
 	}
@@ -69,8 +69,8 @@ 
 			$field = wp_parse_args( $field, [
 				'is_setting' => true,
 				'name' => $name,
-			]);
-			$rows.= new Field( $this->normalize( $field ));
+			] );
+			$rows .= new Field( $this->normalize( $field ) );
 		}
 		return $rows;
 	}
@@ -81,7 +81,7 @@ 
 	 */
 	protected function getTemplateData( $id )
 	{
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
 		return [
 			'context' => [
 				'rows' => $this->getSettingRows( $fields ),
@@ -95,12 +95,12 @@ 
 	 */
 	protected function getTemplateDataForAddons( $id )
 	{
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
 		$settings = glsr( Helper::class )->convertDotNotationArray( $fields );
 		$settingKeys = array_keys( $settings['settings']['addons'] );
 		$results = [];
 		foreach( $settingKeys as $key ) {
-			$addonFields = array_filter( $fields, function( $path ) use( $key ) {
+			$addonFields = array_filter( $fields, function( $path ) use($key) {
 				return glsr( Helper::class )->startsWith( 'settings.addons.'.$key, $path );
 			}, ARRAY_FILTER_USE_KEY );
 			$results[$key] = $this->getSettingRows( $addonFields );
@@ -117,7 +117,7 @@ 
 	 */
 	protected function getTemplateDataForLicenses( $id )
 	{
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
 		ksort( $fields );
 		return [
 			'context' => [
@@ -132,7 +132,7 @@ 
 	protected function getTemplateDataForTranslations()
 	{
 		$translations = glsr( Translation::class )->renderAll();
-		$class = empty( $translations )
+		$class = empty($translations)
 			? 'glsr-hidden'
 			: '';
 		return [
@@ -155,9 +155,9 @@ 
 			$path,
 			glsr( Helper::class )->dataGet( glsr()->defaults, $path )
 		);
-		if( is_array( $expectedValue )) {
+		if( is_array( $expectedValue ) ) {
 			return is_array( $optionValue )
-				? count( array_intersect( $optionValue, $expectedValue )) === 0
+				? count( array_intersect( $optionValue, $expectedValue ) ) === 0
 				: !in_array( $optionValue, $expectedValue );
 		}
 		return $optionValue != $expectedValue;
@@ -168,10 +168,10 @@ 
 	 */
 	protected function isMultiDependency( $path )
 	{
-		if( isset( $this->settings[$path] )) {
+		if( isset($this->settings[$path]) ) {
 			$field = $this->settings[$path];
-			return ( $field['type'] == 'checkbox' && !empty( $field['options'] ))
-				|| !empty( $field['multiple'] );
+			return ($field['type'] == 'checkbox' && !empty($field['options']))
+				|| !empty($field['multiple']);
 		}
 		return false;
 	}
@@ -192,7 +192,7 @@ 
 	 */
 	protected function normalizeDependsOn( array $field )
 	{
-		if( !empty( $field['depends_on'] ) && is_array( $field['depends_on'] )) {
+		if( !empty($field['depends_on']) && is_array( $field['depends_on'] ) ) {
 			$isFieldHidden = false;
 			$conditions = [];
 			foreach( $field['depends_on'] as $path => $value ) {
@@ -200,11 +200,11 @@ 
 					'name' => $this->getFieldNameForDependsOn( $path ),
 					'value' => $value,
 				];
-				if( $this->isFieldHidden( $path, $value )) {
+				if( $this->isFieldHidden( $path, $value ) ) {
 					$isFieldHidden = true;
 				}
 			}
-			$field['data-depends'] = json_encode( $conditions, JSON_HEX_APOS|JSON_HEX_QUOT );
+			$field['data-depends'] = json_encode( $conditions, JSON_HEX_APOS | JSON_HEX_QUOT );
 			$field['is_hidden'] = $isFieldHidden;
 		}
 		return $field;
@@ -215,9 +215,9 @@ 
 	 */
 	protected function normalizeLabelAndLegend( array $field )
 	{
-		if( !empty( $field['label'] )) {
+		if( !empty($field['label']) ) {
 			$field['legend'] = $field['label'];
-			unset( $field['label'] );
+			unset($field['label']);
 		}
 		else {
 			$field['is_valid'] = false;
@@ -231,7 +231,7 @@ 
 	 */
 	protected function normalizeValue( array $field )
 	{
-		if( !isset( $field['value'] )) {
+		if( !isset($field['value']) ) {
 			$field['value'] = glsr( OptionManager::class )->get(
 				$field['name'],
 				$this->getFieldDefault( $field )

plugin/Modules/Html/Partials/SiteReviewsSummary.php

Spacing +16 added lines, -16 removed lines

@@ -34,7 +34,7 @@ 
 	{
 		$this->args = $args;
 		$this->ratingCounts = glsr( ReviewManager::class )->getRatingCounts( $args );
-		if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' ))return;
+		if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' ) )return;
 		$this->averageRating = glsr( Rating::class )->getAverage( $this->ratingCounts );
 		$this->generateSchema();
 		return glsr( Template::class )->build( 'templates/reviews-summary', [
@@ -48,7 +48,7 @@ 
 				'stars' => $this->buildStars(),
 				'text' => $this->buildText(),
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -56,9 +56,9 @@ 
 	 */
 	protected function buildPercentage()
 	{
-		if( $this->isHidden( 'bars' ))return;
-		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts ));
-		$bars = array_reduce( range( glsr()->constant( 'MAX_RATING', Rating::class ), 1 ), function( $carry, $level ) use( $percentages ) {
+		if( $this->isHidden( 'bars' ) )return;
+		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts ) );
+		$bars = array_reduce( range( glsr()->constant( 'MAX_RATING', Rating::class ), 1 ), function( $carry, $level ) use($percentages) {
 			$label = $this->buildPercentageLabel( $this->args['labels'][$level] );
 			$background = $this->buildPercentageBackground( $percentages[$level] );
 			$count = apply_filters( 'site-reviews/summary/counts',
@@ -68,7 +68,7 @@ 
 			$percent = $this->buildPercentageCount( $count );
 			return $carry.glsr( Builder::class )->div( $label.$background.$percent, [
 				'class' => 'glsr-bar',
-			]);
+			] );
 		});
 		return $this->wrap( 'percentage', $bars );
 	}
@@ -79,10 +79,10 @@ 
 	 */
 	protected function buildPercentageBackground( $percent )
 	{
-		$backgroundPercent = glsr( Builder::class )->span([
+		$backgroundPercent = glsr( Builder::class )->span( [
 			'class' => 'glsr-bar-background-percent',
 			'style' => 'width:'.$percent,
-		]);
+		] );
 		return '<span class="glsr-bar-background">'.$backgroundPercent.'</span>';
 	}
 
@@ -109,7 +109,7 @@ 
 	 */
 	protected function buildRating()
 	{
-		if( $this->isHidden( 'rating' ))return;
+		if( $this->isHidden( 'rating' ) )return;
 		return $this->wrap( 'rating', '<span>'.$this->averageRating.'</span>' );
 	}
 
@@ -118,7 +118,7 @@ 
 	 */
 	protected function buildStars()
 	{
-		if( $this->isHidden( 'stars' ))return;
+		if( $this->isHidden( 'stars' ) )return;
 		$stars = glsr_star_rating( $this->averageRating );
 		return $this->wrap( 'stars', $stars );
 	}
@@ -128,9 +128,9 @@ 
 	 */
 	protected function buildText()
 	{
-		if( $this->isHidden( 'summary' ))return;
-		$count = intval( array_sum( $this->ratingCounts ));
-		if( empty( $this->args['text'] )) {
+		if( $this->isHidden( 'summary' ) )return;
+		$count = intval( array_sum( $this->ratingCounts ) );
+		if( empty($this->args['text']) ) {
 			// @todo document this change
 			 $this->args['text'] = _nx(
 				'{rating} out of {max} stars (based on {num} review)',
@@ -141,7 +141,7 @@ 
 			);
 		}
 		$summary = str_replace(
-			['{rating}','{max}', '{num}'],
+			['{rating}', '{max}', '{num}'],
 			[$this->averageRating, glsr()->constant( 'MAX_RATING', Rating::class ), $count],
 			$this->args['text']
 		);
@@ -153,7 +153,7 @@ 
 	 */
 	protected function generateSchema()
 	{
-		if( !wp_validate_boolean( $this->args['schema'] ))return;
+		if( !wp_validate_boolean( $this->args['schema'] ) )return;
 		glsr( Schema::class )->store(
 			glsr( Schema::class )->buildSummary( $this->args )
 		);
@@ -185,6 +185,6 @@ 
 	{
 		return glsr( Builder::class )->div( $value, [
 			'class' => 'glsr-summary-'.$key,
-		]);
+		] );
 	}
 }

Braces +18 added lines, -6 removed lines

@@ -34,7 +34,9 @@ 
 	{
 		$this->args = $args;
 		$this->ratingCounts = glsr( ReviewManager::class )->getRatingCounts( $args );
-		if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' ))return;
+		if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' )) {
+			return;
+		}
 		$this->averageRating = glsr( Rating::class )->getAverage( $this->ratingCounts );
 		$this->generateSchema();
 		return glsr( Template::class )->build( 'templates/reviews-summary', [
@@ -56,7 +58,9 @@ 
 	 */
 	protected function buildPercentage()
 	{
-		if( $this->isHidden( 'bars' ))return;
+		if( $this->isHidden( 'bars' )) {
+			return;
+		}
 		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts ));
 		$bars = array_reduce( range( glsr()->constant( 'MAX_RATING', Rating::class ), 1 ), function( $carry, $level ) use( $percentages ) {
 			$label = $this->buildPercentageLabel( $this->args['labels'][$level] );
@@ -109,7 +113,9 @@ 
 	 */
 	protected function buildRating()
 	{
-		if( $this->isHidden( 'rating' ))return;
+		if( $this->isHidden( 'rating' )) {
+			return;
+		}
 		return $this->wrap( 'rating', '<span>'.$this->averageRating.'</span>' );
 	}
 
@@ -118,7 +124,9 @@ 
 	 */
 	protected function buildStars()
 	{
-		if( $this->isHidden( 'stars' ))return;
+		if( $this->isHidden( 'stars' )) {
+			return;
+		}
 		$stars = glsr_star_rating( $this->averageRating );
 		return $this->wrap( 'stars', $stars );
 	}
@@ -128,7 +136,9 @@ 
 	 */
 	protected function buildText()
 	{
-		if( $this->isHidden( 'summary' ))return;
+		if( $this->isHidden( 'summary' )) {
+			return;
+		}
 		$count = intval( array_sum( $this->ratingCounts ));
 		if( empty( $this->args['text'] )) {
 			// @todo document this change
@@ -153,7 +163,9 @@ 
 	 */
 	protected function generateSchema()
 	{
-		if( !wp_validate_boolean( $this->args['schema'] ))return;
+		if( !wp_validate_boolean( $this->args['schema'] )) {
+			return;
+		}
 		glsr( Schema::class )->store(
 			glsr( Schema::class )->buildSummary( $this->args )
 		);

plugin/Modules/Schema.php

Spacing +33 added lines, -33 removed lines

@@ -38,10 +38,10 @@ 
 			if( $review->review_type != 'local' )continue;
 			$reviews[] = $this->buildReview( $review );
 		}
-		if( !empty( $reviews )) {
+		if( !empty($reviews) ) {
 			array_walk( $reviews, function( &$review ) {
-				unset( $review['@context'] );
-				unset( $review['itemReviewed'] );
+				unset($review['@context']);
+				unset($review['itemReviewed']);
 			});
 			$schema['review'] = $reviews;
 		}
@@ -54,7 +54,7 @@ 
 	 */
 	public function buildSummary( $args = null )
 	{
-		if( is_array( $args )) {
+		if( is_array( $args ) ) {
 			$this->args = $args;
 		}
 		$buildSummary = glsr( Helper::class )->buildMethodName( $this->getSchemaOptionValue( 'type' ), 'buildSummaryFor' );
@@ -62,13 +62,13 @@ 
 		$schema = method_exists( $this, $buildSummary )
 			? $this->$buildSummary()
 			: $this->buildSummaryForCustom();
-		if( !empty( $count )) {
+		if( !empty($count) ) {
 			$schema->aggregateRating(
 				$this->getSchemaType( 'AggregateRating' )
 					->ratingValue( $this->getRatingValue() )
 					->reviewCount( $count )
-					->bestRating( glsr()->constant( 'MAX_RATING', Rating::class ))
-					->worstRating( glsr()->constant( 'MIN_RATING', Rating::class ))
+					->bestRating( glsr()->constant( 'MAX_RATING', Rating::class ) )
+					->worstRating( glsr()->constant( 'MIN_RATING', Rating::class ) )
 			);
 		}
 		$schema = $schema->toArray();
@@ -80,11 +80,11 @@ 
 	 */
 	public function render()
 	{
-		if( empty( glsr()->schemas ))return;
+		if( empty(glsr()->schemas) )return;
 		printf( '<script type="application/ld+json">%s</script>', json_encode(
 			apply_filters( 'site-reviews/schema/all', glsr()->schemas ),
 			JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES
-		));
+		) );
 	}
 
 	/**
@@ -94,7 +94,7 @@ 
 	{
 		$schemas = glsr()->schemas;
 		$schemas[] = $schema;
-		glsr()->schemas = array_map( 'unserialize', array_unique( array_map( 'serialize', $schemas )));
+		glsr()->schemas = array_map( 'unserialize', array_unique( array_map( 'serialize', $schemas ) ) );
 	}
 
 	/**
@@ -104,21 +104,21 @@ 
 	protected function buildReview( $review )
 	{
 		$schema = $this->getSchemaType( 'Review' )
-			->doIf( !in_array( 'title', $this->args['hide'] ), function( $schema ) use( $review ) {
+			->doIf( !in_array( 'title', $this->args['hide'] ), function( $schema ) use($review) {
 				$schema->name( $review->title );
 			})
-			->doIf( !in_array( 'excerpt', $this->args['hide'] ), function( $schema ) use( $review ) {
+			->doIf( !in_array( 'excerpt', $this->args['hide'] ), function( $schema ) use($review) {
 				$schema->reviewBody( $review->content );
 			})
-			->datePublished(( new DateTime( $review->date )))
-			->author( $this->getSchemaType( 'Person' )->name( $review->author ))
-			->itemReviewed( $this->getSchemaType()->name( $this->getSchemaOptionValue( 'name' )));
-		if( !empty( $review->rating )) {
+			->datePublished( (new DateTime( $review->date )) )
+			->author( $this->getSchemaType( 'Person' )->name( $review->author ) )
+			->itemReviewed( $this->getSchemaType()->name( $this->getSchemaOptionValue( 'name' ) ) );
+		if( !empty($review->rating) ) {
 			$schema->reviewRating(
 				$this->getSchemaType( 'Rating' )
 					->ratingValue( $review->rating )
-					->bestRating( glsr()->constant( 'MAX_RATING', Rating::class ))
-					->worstRating( glsr()->constant( 'MIN_RATING', Rating::class ))
+					->bestRating( glsr()->constant( 'MAX_RATING', Rating::class ) )
+					->worstRating( glsr()->constant( 'MIN_RATING', Rating::class ) )
 			);
 		}
 		return apply_filters( 'site-reviews/schema/review', $schema->toArray(), $review, $this->args );
@@ -132,7 +132,7 @@ 
 	{
 		foreach( $values as $value ) {
 			$option = $this->getSchemaOptionValue( $value );
-			if( empty( $option ))continue;
+			if( empty($option) )continue;
 			$schema->$value( $option );
 		}
 		return $schema;
@@ -145,7 +145,7 @@ 
 	{
 		return $this->buildSchemaValues( $this->getSchemaType(), [
 			'description', 'image', 'name', 'url',
-		]);
+		] );
 	}
 
 	/**
@@ -155,7 +155,7 @@ 
 	{
 		return $this->buildSchemaValues( $this->buildSummaryForCustom(), [
 			'address', 'priceRange', 'telephone',
-		]);
+		] );
 	}
 
 	/**
@@ -166,9 +166,9 @@ 
 		$offerType = $this->getSchemaOption( 'offerType', 'AggregateOffer' );
 		$offers = $this->buildSchemaValues( $this->getSchemaType( $offerType ), [
 			'highPrice', 'lowPrice', 'price', 'priceCurrency',
-		]);
+		] );
 		return $this->buildSummaryForCustom()
-			->doIf( !empty( $offers->getProperties() ), function( $schema ) use( $offers ) {
+			->doIf( !empty($offers->getProperties()), function( $schema ) use($offers) {
 				$schema->offers( $offers );
 			})
 			->setProperty( '@id', $this->getSchemaOptionValue( 'url' ).'#product' );
@@ -179,7 +179,7 @@ 
 	 */
 	protected function getRatingCounts()
 	{
-		if( !isset( $this->ratingCounts )) {
+		if( !isset($this->ratingCounts) ) {
 			$this->ratingCounts = glsr( ReviewManager::class )->getRatingCounts( $this->args );
 		}
 		return $this->ratingCounts;
@@ -201,14 +201,14 @@ 
 	protected function getSchemaOption( $option, $fallback )
 	{
 		$option = strtolower( $option );
-		if( $schemaOption = trim( (string)get_post_meta( intval( get_the_ID() ), 'schema_'.$option, true ))) {
+		if( $schemaOption = trim( (string)get_post_meta( intval( get_the_ID() ), 'schema_'.$option, true ) ) ) {
 			return $schemaOption;
 		}
 		$setting = glsr( OptionManager::class )->get( 'settings.schema.'.$option );
-		if( is_array( $setting )) {
+		if( is_array( $setting ) ) {
 			return $this->getSchemaOptionDefault( $setting, $fallback );
 		}
-		return !empty( $setting )
+		return !empty($setting)
 			? $setting
 			: $fallback;
 	}
@@ -222,7 +222,7 @@ 
 		$setting = wp_parse_args( $setting, [
 			'custom' => '',
 			'default' => $fallback,
-		]);
+		] );
 		return $setting['default'] != 'custom'
 			? $setting['default']
 			: $setting['custom'];
@@ -241,7 +241,7 @@ 
 		}
 		if( !is_single() && !is_page() )return;
 		$method = glsr( Helper::class )->buildMethodName( $option, 'getThing' );
-		if( method_exists( $this, $method )) {
+		if( method_exists( $this, $method ) ) {
 			return $this->$method();
 		}
 	}
@@ -252,7 +252,7 @@ 
 	 */
 	protected function getSchemaType( $type = null )
 	{
-		if( !is_string( $type )) {
+		if( !is_string( $type ) ) {
 			$type = $this->getSchemaOption( 'type', 'LocalBusiness' );
 		}
 		$className = glsr( Helper::class )->buildClassName( $type, 'Modules\Schema' );
@@ -267,11 +267,11 @@ 
 	protected function getThingDescription()
 	{
 		$post = get_post();
-		if( !( $post instanceof WP_Post )) {
+		if( !($post instanceof WP_Post) ) {
 			return '';
 		}
-		$text = strip_shortcodes( wp_strip_all_tags( $post->post_excerpt ));
-		return wp_trim_words( $text, apply_filters( 'excerpt_length', 55 ));
+		$text = strip_shortcodes( wp_strip_all_tags( $post->post_excerpt ) );
+		return wp_trim_words( $text, apply_filters( 'excerpt_length', 55 ) );
 	}
 
 	/**

Braces +12 added lines, -4 removed lines

@@ -35,7 +35,9 @@ 
 		foreach( glsr( ReviewManager::class )->get( $this->args ) as $review ) {
 			// Only include critic reviews that have been directly produced by your site, not reviews from third-party sites or syndicated reviews.
 			// @see https://developers.google.com/search/docs/data-types/review
-			if( $review->review_type != 'local' )continue;
+			if( $review->review_type != 'local' ) {
+				continue;
+			}
 			$reviews[] = $this->buildReview( $review );
 		}
 		if( !empty( $reviews )) {
@@ -80,7 +82,9 @@ 
 	 */
 	public function render()
 	{
-		if( empty( glsr()->schemas ))return;
+		if( empty( glsr()->schemas )) {
+			return;
+		}
 		printf( '<script type="application/ld+json">%s</script>', json_encode(
 			apply_filters( 'site-reviews/schema/all', glsr()->schemas ),
 			JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES
@@ -132,7 +136,9 @@ 
 	{
 		foreach( $values as $value ) {
 			$option = $this->getSchemaOptionValue( $value );
-			if( empty( $option ))continue;
+			if( empty( $option )) {
+				continue;
+			}
 			$schema->$value( $option );
 		}
 		return $schema;
@@ -239,7 +245,9 @@ 
 		if( $value != $fallback ) {
 			return $value;
 		}
-		if( !is_single() && !is_page() )return;
+		if( !is_single() && !is_page() ) {
+			return;
+		}
 		$method = glsr( Helper::class )->buildMethodName( $option, 'getThing' );
 		if( method_exists( $this, $method )) {
 			return $this->$method();

plugin/Modules/System.php

Spacing +25 added lines, -25 removed lines

@@ -40,7 +40,7 @@ 
 			'setting' => 'Plugin Settings',
 			'reviews' => 'Review Counts',
 		];
-		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use( $details ) {
+		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use($details) {
 			$methodName = glsr( Helper::class )->buildMethodName( 'get-'.$key.'-details' );
 			if( method_exists( $this, $methodName ) && $systemDetails = $this->$methodName() ) {
 				return $carry.$this->implode(
@@ -60,8 +60,8 @@ 
 	{
 		$plugins = get_plugins();
 		$activePlugins = (array)get_option( 'active_plugins', [] );
-		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ));
-		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
+		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ) );
+		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
 	}
 
 	/**
@@ -95,7 +95,7 @@ 
 	public function getInactivePluginDetails()
 	{
 		$activePlugins = (array)get_option( 'active_plugins', [] );
-		$inactivePlugins = $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins )));
+		$inactivePlugins = $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins ) ) );
 		$multisitePlugins = $this->getMultisitePluginDetails();
 		return is_array( $multisitePlugins )
 			? array_diff( $inactivePlugins, $multisitePlugins )
@@ -107,7 +107,7 @@ 
 	 */
 	public function getMuPluginDetails()
 	{
-		if( empty( $plugins = get_mu_plugins() ))return;
+		if( empty($plugins = get_mu_plugins()) )return;
 		return $this->normalizePluginList( $plugins );
 	}
 
@@ -117,8 +117,8 @@ 
 	public function getMultisitePluginDetails()
 	{
 		$activePlugins = (array)get_site_option( 'active_sitewide_plugins', [] );
-		if( !is_multisite() || empty( $activePlugins ))return;
-		return $this->normalizePluginList( array_intersect_key( get_plugins(), $activePlugins ));
+		if( !is_multisite() || empty($activePlugins) )return;
+		return $this->normalizePluginList( array_intersect_key( get_plugins(), $activePlugins ) );
 	}
 
 	/**
@@ -145,11 +145,11 @@ 
 			'Memory Limit' => ini_get( 'memory_limit' ),
 			'Post Max Size' => ini_get( 'post_max_size' ),
 			'Sendmail Path' => ini_get( 'sendmail_path' ),
-			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' )),
-			'Session Name' => esc_html( ini_get( 'session.name' )),
-			'Session Save Path' => esc_html( ini_get( 'session.save_path' )),
-			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' )), true ),
-			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' )), true ),
+			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' ) ),
+			'Session Name' => esc_html( ini_get( 'session.name' ) ),
+			'Session Save Path' => esc_html( ini_get( 'session.save_path' ) ),
+			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' ) ), true ),
+			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' ) ), true ),
 			'Upload Max Filesize' => ini_get( 'upload_max_filesize' ),
 		];
 	}
@@ -161,8 +161,8 @@ 
 	{
 		$counts = glsr( CountsManager::class )->getCounts();
 		$counts = glsr( Helper::class )->flattenArray( $counts );
- 		array_walk( $counts, function( &$ratings ) use( $counts ) {
-			if( !is_array( $ratings )) {
+ 		array_walk( $counts, function( &$ratings ) use($counts) {
+			if( !is_array( $ratings ) ) {
 				glsr_log()
 					->error( '$ratings is not an array, possibly due to incorrectly imported reviews.' )
 					->debug( $ratings )
@@ -201,8 +201,8 @@ 
 		ksort( $settings );
 		$details = [];
 		foreach( $settings as $key => $value ) {
-			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue;
-			$value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' );
+			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ) )continue;
+			$value = htmlspecialchars( trim( preg_replace( '/\s\s+/', '\\n', $value ) ), ENT_QUOTES, 'UTF-8' );
 			$details[$key] = $value;
 		}
 		return $details;
@@ -216,7 +216,7 @@ 
 		return [
 			'Console level' => glsr( Console::class )->humanLevel(),
 			'Console size' => glsr( Console::class )->humanSize( '0' ),
-			'Last Rating Count' => date_i18n( 'Y-m-d H:i', glsr( OptionManager::class )->get( 'last_review_count' )),
+			'Last Rating Count' => date_i18n( 'Y-m-d H:i', glsr( OptionManager::class )->get( 'last_review_count' ) ),
 			'Version (current)' => glsr()->version,
 			'Version (previous)' => glsr( OptionManager::class )->get( 'version_upgraded_from' ),
 		];
@@ -277,10 +277,10 @@ 
 			'WPE_APIKEY' => 'WP Engine',
 		];
 		foreach( $checks as $key => $value ) {
-			if( !$this->isWebhostCheckValid( $key ))continue;
+			if( !$this->isWebhostCheckValid( $key ) )continue;
 			return $value;
 		}
-		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ));
+		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ) );
 	}
 
 	/**
@@ -301,8 +301,8 @@ 
 	{
 		$plugins = get_plugins();
 		$activePlugins = (array)get_option( 'active_plugins', [] );
-		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins )));
-		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
+		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins ) ) );
+		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
 		return $active + $inactive;
 	}
 
@@ -313,7 +313,7 @@ 
 	protected function implode( $title, array $details )
 	{
 		$strings = ['['.$title.']'];
-		$padding = max( array_map( 'strlen', array_keys( $details )) );
+		$padding = max( array_map( 'strlen', array_keys( $details ) ) );
 		$padding = max( [$padding, static::PAD] );
 		foreach( $details as $key => $value ) {
 			$strings[] = is_string( $key )
@@ -342,7 +342,7 @@ 
 	protected function normalizePluginList( array $plugins )
 	{
 		$plugins = array_map( function( $plugin ) {
-			return sprintf( '%s v%s', glsr_get( $plugin, 'Name' ), glsr_get( $plugin, 'Version' ));
+			return sprintf( '%s v%s', glsr_get( $plugin, 'Name' ), glsr_get( $plugin, 'Version' ) );
 		}, $plugins );
 		natcasesort( $plugins );
 		return array_flip( $plugins );
@@ -356,9 +356,9 @@ 
 		$keys = [
 			'licenses.', 'submissions.recaptcha.key', 'submissions.recaptcha.secret',
 		];
-		array_walk( $settings, function( &$value, $setting ) use( $keys ) {
+		array_walk( $settings, function( &$value, $setting ) use($keys) {
 			foreach( $keys as $key ) {
-				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value ))continue;
+				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty($value) )continue;
 				$value = str_repeat( '•', 13 );
 				return;
 			}

Braces +15 added lines, -5 removed lines

@@ -107,7 +107,9 @@ 
 	 */
 	public function getMuPluginDetails()
 	{
-		if( empty( $plugins = get_mu_plugins() ))return;
+		if( empty( $plugins = get_mu_plugins() )) {
+			return;
+		}
 		return $this->normalizePluginList( $plugins );
 	}
 
@@ -117,7 +119,9 @@ 
 	public function getMultisitePluginDetails()
 	{
 		$activePlugins = (array)get_site_option( 'active_sitewide_plugins', [] );
-		if( !is_multisite() || empty( $activePlugins ))return;
+		if( !is_multisite() || empty( $activePlugins )) {
+			return;
+		}
 		return $this->normalizePluginList( array_intersect_key( get_plugins(), $activePlugins ));
 	}
 
@@ -201,7 +205,9 @@ 
 		ksort( $settings );
 		$details = [];
 		foreach( $settings as $key => $value ) {
-			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue;
+			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key )) {
+				continue;
+			}
 			$value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' );
 			$details[$key] = $value;
 		}
@@ -277,7 +283,9 @@ 
 			'WPE_APIKEY' => 'WP Engine',
 		];
 		foreach( $checks as $key => $value ) {
-			if( !$this->isWebhostCheckValid( $key ))continue;
+			if( !$this->isWebhostCheckValid( $key )) {
+				continue;
+			}
 			return $value;
 		}
 		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ));
@@ -358,7 +366,9 @@ 
 		];
 		array_walk( $settings, function( &$value, $setting ) use( $keys ) {
 			foreach( $keys as $key ) {
-				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value ))continue;
+				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value )) {
+					continue;
+				}
 				$value = str_repeat( '•', 13 );
 				return;
 			}

plugin/Modules/Slack.php

Spacing +9 added lines, -9 removed lines

@@ -35,7 +35,7 @@ 
 	 */
 	public function compose( Review $review, array $notification )
 	{
-		if( empty( $this->endpoint )) {
+		if( empty($this->endpoint) ) {
 			return $this;
 		}
 		$args = shortcode_atts( glsr( SlackDefaults::class )->defaults(), $notification );
@@ -60,7 +60,7 @@ 
 	 */
 	public function send()
 	{
-		if( empty( $this->endpoint )) {
+		if( empty($this->endpoint) ) {
 			return new WP_Error( 'slack', 'Slack notification was not sent: missing endpoint' );
 		}
 		return wp_remote_post( $this->endpoint, [
@@ -72,7 +72,7 @@ 
 			'redirection' => 5,
 			'sslverify' => false,
 			'timeout' => 45,
-		]);
+		] );
 	}
 
 	/**
@@ -92,11 +92,11 @@ 
 	 */
 	protected function buildAuthorField()
 	{
-		$email = !empty( $this->review->email )
+		$email = !empty($this->review->email)
 			? '<'.$this->review->email.'>'
 			: '';
 		$author = trim( rtrim( $this->review->author ).' '.$email );
-		return ['value' => implode( ' - ', array_filter( [$author, $this->review->ip_address] ))];
+		return ['value' => implode( ' - ', array_filter( [$author, $this->review->ip_address] ) )];
 	}
 
 	/**
@@ -104,7 +104,7 @@ 
 	 */
 	protected function buildContentField()
 	{
-		return !empty( $this->review->content )
+		return !empty($this->review->content)
 			? ['value' => $this->review->content]
 			: [];
 	}
@@ -129,9 +129,9 @@ 
 	protected function buildStarsField()
 	{
 		$solidStars = str_repeat( '★', $this->review->rating );
-		$emptyStars = str_repeat( '☆', max( 0, glsr()->constant( 'MAX_RATING', Rating::class ) - $this->review->rating ));
+		$emptyStars = str_repeat( '☆', max( 0, glsr()->constant( 'MAX_RATING', Rating::class ) - $this->review->rating ) );
 		$stars = $solidStars.$emptyStars;
-		$stars = apply_filters( 'site-reviews/slack/stars', $stars, $this->review->rating, glsr()->constant( 'MAX_RATING', Rating::class ));
+		$stars = apply_filters( 'site-reviews/slack/stars', $stars, $this->review->rating, glsr()->constant( 'MAX_RATING', Rating::class ) );
 		return ['title' => $stars];
 	}
 
@@ -140,7 +140,7 @@ 
 	 */
 	protected function buildTitleField()
 	{
-		return !empty( $this->review->title )
+		return !empty($this->review->title)
 			? ['title' => $this->review->title]
 			: [];
 	}

plugin/Database/CountsManager.php

Spacing +40 added lines, -40 removed lines

@@ -30,12 +30,12 @@ 
 		$query = $this->queryReviews( $args );
 		while( $query ) {
 			// glsr_log($query);
-			$types = array_keys( array_flip( glsr_array_column( $query->reviews, 'type' )));
-			$types = array_unique( array_merge( ['local'], $types ));
+			$types = array_keys( array_flip( glsr_array_column( $query->reviews, 'type' ) ) );
+			$types = array_unique( array_merge( ['local'], $types ) );
 			foreach( $types as $type ) {
 				$type = $this->normalizeType( $type );
-				if( isset( $counts[$type] ))continue;
-				$counts[$type] = array_fill_keys( range( 0, glsr()->constant( 'MAX_RATING', Rating::class )), 0 );
+				if( isset($counts[$type]) )continue;
+				$counts[$type] = array_fill_keys( range( 0, glsr()->constant( 'MAX_RATING', Rating::class ) ), 0 );
 			}
 			foreach( $query->reviews as $review ) {
 				$type = $this->normalizeType( $review->type );
@@ -73,11 +73,11 @@ 
 	{
 		$terms = glsr( Database::class )->getTerms( ['fields' => 'all'] );
 		foreach( $terms as $term ) {
-			$this->setTermCounts( $term->term_id, $this->buildTermCounts( $term->term_taxonomy_id ));
+			$this->setTermCounts( $term->term_id, $this->buildTermCounts( $term->term_taxonomy_id ) );
 		}
 		$postIds = glsr( SqlQueries::class )->getReviewsMeta( 'assigned_to' );
 		foreach( $postIds as $postId ) {
-			$this->setPostCounts( $postId, $this->buildPostCounts( $postId ));
+			$this->setPostCounts( $postId, $this->buildPostCounts( $postId ) );
 		}
 		$this->setCounts( $this->buildCounts() );
 	}
@@ -101,7 +101,7 @@ 
 			$this->getCounts(),
 			$review->review_type,
 			$review->rating
-		));
+		) );
 	}
 
 	/**
@@ -109,7 +109,7 @@ 
 	 */
 	public function decreasePostCounts( Review $review )
 	{
-		if( empty( $counts = $this->getPostCounts( $review->assigned_to )))return;
+		if( empty($counts = $this->getPostCounts( $review->assigned_to )) )return;
 		$counts = $this->decreaseRating( $counts, $review->review_type, $review->rating );
 		$this->setPostCounts( $review->assigned_to, $counts );
 	}
@@ -120,7 +120,7 @@ 
 	public function decreaseTermCounts( Review $review )
 	{
 		foreach( $review->term_ids as $termId ) {
-			if( empty( $counts = $this->getTermCounts( $termId )))continue;
+			if( empty($counts = $this->getTermCounts( $termId )) )continue;
 			$counts = $this->decreaseRating( $counts, $review->review_type, $review->rating );
 			$this->setTermCounts( $termId, $counts );
 		}
@@ -132,15 +132,15 @@ 
 	public function flatten( array $reviewCounts, array $args = [] )
 	{
 		$counts = [];
-		array_walk_recursive( $reviewCounts, function( $num, $index ) use( &$counts ) {
-			$counts[$index] = $num + intval( glsr_get( $counts, $index, 0 ));
+		array_walk_recursive( $reviewCounts, function( $num, $index ) use(&$counts) {
+			$counts[$index] = $num + intval( glsr_get( $counts, $index, 0 ) );
 		});
 		$args = wp_parse_args( $args, [
 			'max' => glsr()->constant( 'MAX_RATING', Rating::class ),
 			'min' => glsr()->constant( 'MIN_RATING', Rating::class ),
-		]);
+		] );
 		foreach( $counts as $index => &$num ) {
-			if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] ))continue;
+			if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] ) )continue;
 			$num = 0;
 		}
 		return $counts;
@@ -153,7 +153,7 @@ 
 	{
 		$args = $this->normalizeArgs( $args );
 		$counts = [];
-		if( $this->isMixedCount( $args )) {
+		if( $this->isMixedCount( $args ) ) {
 			$counts = [$this->buildCounts( $args )]; // force query the database
 		}
 		else {
@@ -163,13 +163,13 @@ 
 			foreach( $args['term_ids'] as $termId ) {
 				$counts[] = $this->getTermCounts( $termId );
 			}
-			if( empty( $counts )) {
+			if( empty($counts) ) {
 				$counts[] = $this->getCounts();
 			}
 		}
 		return in_array( $args['type'], ['', 'all'] )
 			? $this->normalize( [$this->flatten( $counts )] )
-			: $this->normalize( glsr_array_column( $counts, $args['type'] ));
+			: $this->normalize( glsr_array_column( $counts, $args['type'] ) );
 	}
 
 	/**
@@ -178,7 +178,7 @@ 
 	public function getCounts()
 	{
 		$counts = glsr( OptionManager::class )->get( 'counts', [] );
-		if( !is_array( $counts )) {
+		if( !is_array( $counts ) ) {
 			glsr_log()->error( '$counts is not an array' )->debug( $counts );
 			return [];
 		}
@@ -191,7 +191,7 @@ 
 	 */
 	public function getPostCounts( $postId )
 	{
-		return array_filter( (array)get_post_meta( $postId, static::META_COUNT, true ));
+		return array_filter( (array)get_post_meta( $postId, static::META_COUNT, true ) );
 	}
 
 	/**
@@ -200,7 +200,7 @@ 
 	 */
 	public function getTermCounts( $termId )
 	{
-		return array_filter( (array)get_term_meta( $termId, static::META_COUNT, true ));
+		return array_filter( (array)get_term_meta( $termId, static::META_COUNT, true ) );
 	}
 
 	/**
@@ -218,10 +218,10 @@ 
 	 */
 	public function increaseCounts( Review $review )
 	{
-		if( empty( $counts = $this->getCounts() )) {
+		if( empty($counts = $this->getCounts()) ) {
 			$counts = $this->buildCounts();
 		}
-		$this->setCounts( $this->increaseRating( $counts, $review->review_type, $review->rating ));
+		$this->setCounts( $this->increaseRating( $counts, $review->review_type, $review->rating ) );
 	}
 
 	/**
@@ -229,9 +229,9 @@ 
 	 */
 	public function increasePostCounts( Review $review )
 	{
-		if( !( get_post( $review->assigned_to ) instanceof WP_Post ))return;
+		if( !(get_post( $review->assigned_to ) instanceof WP_Post) )return;
 		$counts = $this->getPostCounts( $review->assigned_to );
-		$counts = empty( $counts )
+		$counts = empty($counts)
 			? $this->buildPostCounts( $review->assigned_to )
 			: $this->increaseRating( $counts, $review->review_type, $review->rating );
 		$this->setPostCounts( $review->assigned_to, $counts );
@@ -242,10 +242,10 @@ 
 	 */
 	public function increaseTermCounts( Review $review )
 	{
-		$terms = glsr( ReviewManager::class )->normalizeTerms( implode( ',', $review->term_ids ));
+		$terms = glsr( ReviewManager::class )->normalizeTerms( implode( ',', $review->term_ids ) );
 		foreach( $terms as $term ) {
 			$counts = $this->getTermCounts( $term['term_id'] );
-			$counts = empty( $counts )
+			$counts = empty($counts)
 				? $this->buildTermCounts( $term['term_taxonomy_id'] )
 				: $this->increaseRating( $counts, $review->review_type, $review->rating );
 			$this->setTermCounts( $term['term_id'], $counts );
@@ -268,8 +268,8 @@ 
 	{
 		$ratingCounts = $this->flatten( $reviewCounts );
 		update_post_meta( $postId, static::META_COUNT, $reviewCounts );
-		update_post_meta( $postId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts ));
-		update_post_meta( $postId, static::META_RANKING, glsr( Rating::class )->getRanking( $ratingCounts ));
+		update_post_meta( $postId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts ) );
+		update_post_meta( $postId, static::META_RANKING, glsr( Rating::class )->getRanking( $ratingCounts ) );
 	}
 
 	/**
@@ -279,11 +279,11 @@ 
 	public function setTermCounts( $termId, array $reviewCounts )
 	{
 		$term = get_term( $termId, Application::TAXONOMY );
-		if( !isset( $term->term_id ))return;
+		if( !isset($term->term_id) )return;
 		$ratingCounts = $this->flatten( $reviewCounts );
 		update_term_meta( $termId, static::META_COUNT, $reviewCounts );
-		update_term_meta( $termId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts ));
-		update_term_meta( $termId, static::META_RANKING, glsr( Rating::class )->getRanking( $ratingCounts ));
+		update_term_meta( $termId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts ) );
+		update_term_meta( $termId, static::META_RANKING, glsr( Rating::class )->getRanking( $ratingCounts ) );
 	}
 
 	/**
@@ -293,7 +293,7 @@ 
 	 */
 	protected function decreaseRating( array $reviewCounts, $type, $rating )
 	{
-		if( isset( $reviewCounts[$type][$rating] )) {
+		if( isset($reviewCounts[$type][$rating]) ) {
 			$reviewCounts[$type][$rating] = max( 0, $reviewCounts[$type][$rating] - 1 );
 		}
 		return $reviewCounts;
@@ -306,10 +306,10 @@ 
 	 */
 	protected function increaseRating( array $reviewCounts, $type, $rating )
 	{
-		if( !array_key_exists( $type, glsr()->reviewTypes )) {
+		if( !array_key_exists( $type, glsr()->reviewTypes ) ) {
 			return $reviewCounts;
 		}
-		if( !array_key_exists( $type, $reviewCounts )) {
+		if( !array_key_exists( $type, $reviewCounts ) ) {
 			$reviewCounts[$type] = [];
 		}
 		$reviewCounts = $this->normalize( $reviewCounts );
@@ -322,7 +322,7 @@ 
 	 */
 	protected function isMixedCount( array $args )
 	{
-		return !empty( $args['post_ids'] ) && !empty( $args['term_ids'] );
+		return !empty($args['post_ids']) && !empty($args['term_ids']);
 	}
 
 	/**
@@ -330,12 +330,12 @@ 
 	 */
 	protected function normalize( array $reviewCounts )
 	{
-		if( empty( $reviewCounts )) {
+		if( empty($reviewCounts) ) {
 			$reviewCounts = [[]];
 		}
 		foreach( $reviewCounts as &$counts ) {
-			foreach( range( 0, glsr()->constant( 'MAX_RATING', Rating::class )) as $index ) {
-				if( isset( $counts[$index] ))continue;
+			foreach( range( 0, glsr()->constant( 'MAX_RATING', Rating::class ) ) as $index ) {
+				if( isset($counts[$index]) )continue;
 				$counts[$index] = 0;
 			}
 			ksort( $counts );
@@ -352,7 +352,7 @@ 
 			'post_ids' => [],
 			'term_ids' => [],
 			'type' => 'local',
-		]);
+		] );
 		$args['post_ids'] = glsr( Polylang::class )->getPostIds( $args['post_ids'] );
 		$args['type'] = $this->normalizeType( $args['type'] );
 		return $args;
@@ -364,7 +364,7 @@ 
 	 */
 	protected function normalizeType( $type )
 	{
-		return empty( $type ) || !is_string( $type )
+		return empty($type) || !is_string( $type )
 			? 'local'
 			: $type;
 	}
@@ -379,7 +379,7 @@ 
 		$hasMore = is_array( $reviews )
 			? count( $reviews ) == static::LIMIT
 			: false;
-		return (object) [
+		return (object)[
 			'has_more' => $hasMore,
 			'reviews' => $reviews,
 		];

Braces +21 added lines, -7 removed lines

@@ -34,7 +34,9 @@ 
 			$types = array_unique( array_merge( ['local'], $types ));
 			foreach( $types as $type ) {
 				$type = $this->normalizeType( $type );
-				if( isset( $counts[$type] ))continue;
+				if( isset( $counts[$type] )) {
+					continue;
+				}
 				$counts[$type] = array_fill_keys( range( 0, glsr()->constant( 'MAX_RATING', Rating::class )), 0 );
 			}
 			foreach( $query->reviews as $review ) {
@@ -109,7 +111,9 @@ 
 	 */
 	public function decreasePostCounts( Review $review )
 	{
-		if( empty( $counts = $this->getPostCounts( $review->assigned_to )))return;
+		if( empty( $counts = $this->getPostCounts( $review->assigned_to ))) {
+			return;
+		}
 		$counts = $this->decreaseRating( $counts, $review->review_type, $review->rating );
 		$this->setPostCounts( $review->assigned_to, $counts );
 	}
@@ -120,7 +124,9 @@ 
 	public function decreaseTermCounts( Review $review )
 	{
 		foreach( $review->term_ids as $termId ) {
-			if( empty( $counts = $this->getTermCounts( $termId )))continue;
+			if( empty( $counts = $this->getTermCounts( $termId ))) {
+				continue;
+			}
 			$counts = $this->decreaseRating( $counts, $review->review_type, $review->rating );
 			$this->setTermCounts( $termId, $counts );
 		}
@@ -140,7 +146,9 @@ 
 			'min' => glsr()->constant( 'MIN_RATING', Rating::class ),
 		]);
 		foreach( $counts as $index => &$num ) {
-			if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] ))continue;
+			if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] )) {
+				continue;
+			}
 			$num = 0;
 		}
 		return $counts;
@@ -229,7 +237,9 @@ 
 	 */
 	public function increasePostCounts( Review $review )
 	{
-		if( !( get_post( $review->assigned_to ) instanceof WP_Post ))return;
+		if( !( get_post( $review->assigned_to ) instanceof WP_Post )) {
+			return;
+		}
 		$counts = $this->getPostCounts( $review->assigned_to );
 		$counts = empty( $counts )
 			? $this->buildPostCounts( $review->assigned_to )
@@ -279,7 +289,9 @@ 
 	public function setTermCounts( $termId, array $reviewCounts )
 	{
 		$term = get_term( $termId, Application::TAXONOMY );
-		if( !isset( $term->term_id ))return;
+		if( !isset( $term->term_id )) {
+			return;
+		}
 		$ratingCounts = $this->flatten( $reviewCounts );
 		update_term_meta( $termId, static::META_COUNT, $reviewCounts );
 		update_term_meta( $termId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts ));
@@ -335,7 +347,9 @@ 
 		}
 		foreach( $reviewCounts as &$counts ) {
 			foreach( range( 0, glsr()->constant( 'MAX_RATING', Rating::class )) as $index ) {
-				if( isset( $counts[$index] ))continue;
+				if( isset( $counts[$index] )) {
+					continue;
+				}
 				$counts[$index] = 0;
 			}
 			ksort( $counts );

plugin/Database/SqlQueries.php

Spacing +30 added lines, -30 removed lines

@@ -22,11 +22,11 @@ 
 	 */
 	public function deleteAllSessions( $sessionCookiePrefix )
 	{
-		return $this->db->query("
+		return $this->db->query( "
 			DELETE
 			FROM {$this->db->options}
 			WHERE option_name LIKE '{$sessionCookiePrefix}_%'
-		");
+		" );
 	}
 
 	/**
@@ -35,11 +35,11 @@ 
 	 */
 	public function deleteExpiredSessions( $expiredSessions )
 	{
-		return $this->db->query("
+		return $this->db->query( "
 			DELETE
 			FROM {$this->db->options}
 			WHERE option_name IN ('{$expiredSessions}')
-		");
+		" );
 	}
 
 	/**
@@ -49,13 +49,13 @@ 
 	 */
 	public function getExpiredSessions( $sessionCookiePrefix, $limit )
 	{
-		return (array) $this->db->get_results("
+		return (array)$this->db->get_results( "
 			SELECT option_name AS name, option_value AS expiration
 			FROM {$this->db->options}
 			WHERE option_name LIKE '{$sessionCookiePrefix}_expires_%'
 			ORDER BY option_value ASC
 			LIMIT 0, {$limit}
-		");
+		" );
 	}
 
 	/**
@@ -64,14 +64,14 @@ 
 	 */
 	public function getPostIdFromReviewId( $metaReviewId )
 	{
-		$postId = $this->db->get_var("
+		$postId = $this->db->get_var( "
 			SELECT p.ID
 			FROM {$this->db->posts} AS p
 			INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
 			WHERE p.post_type = '{$this->postType}'
 			AND m.meta_key = 'review_id'
 			AND m.meta_value = '{$metaReviewId}'
-		");
+		" );
 		return intval( $postId );
 	}
 
@@ -82,7 +82,7 @@ 
 	 */
 	public function getReviewCounts( array $args, $lastPostId = 0, $limit = 500 )
 	{
-		return (array) $this->db->get_results("
+		return (array)$this->db->get_results( "
 			SELECT DISTINCT p.ID, m1.meta_value AS rating, m2.meta_value AS type
 			FROM {$this->db->posts} AS p
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -96,7 +96,7 @@ 
 			{$this->getAndForCounts( $args )}
 			ORDER By p.ID ASC
 			LIMIT {$limit}
-		");
+		" );
 	}
 
 	/**
@@ -106,14 +106,14 @@ 
 	 */
 	public function getReviewCountsFor( $metaKey )
 	{
-		return (array) $this->db->get_results("
+		return (array)$this->db->get_results( "
 			SELECT DISTINCT m.meta_value AS name, COUNT(*) num_posts
 			FROM {$this->db->posts} AS p
 			INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
 			WHERE p.post_type = '{$this->postType}'
 			AND m.meta_key = '{$metaKey}'
 			GROUP BY name
-		");
+		" );
 	}
 
 	/**
@@ -123,7 +123,7 @@ 
 	 */
 	public function getReviewIdsByType( $reviewType )
 	{
-		$results = $this->db->get_col("
+		$results = $this->db->get_col( "
 			SELECT DISTINCT m1.meta_value AS review_id
 			FROM {$this->db->posts} AS p
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -132,8 +132,8 @@ 
 			AND m1.meta_key = 'review_id'
 			AND m2.meta_key = 'review_type'
 			AND m2.meta_value = '{$reviewType}'
-		");
-		return array_keys( array_flip( $results ));
+		" );
+		return array_keys( array_flip( $results ) );
 	}
 
 	/**
@@ -144,9 +144,9 @@ 
 	public function getReviewRatingsFromIds( array $postIds, $greaterThanId = 0, $limit = 100 )
 	{
 		sort( $postIds );
-		$postIds = array_slice( $postIds, intval( array_search( $greaterThanId, $postIds )), $limit );
+		$postIds = array_slice( $postIds, intval( array_search( $greaterThanId, $postIds ) ), $limit );
 		$postIds = implode( ',', $postIds );
-		return (array) $this->db->get_results("
+		return (array)$this->db->get_results( "
 			SELECT p.ID, m.meta_value AS rating
 			FROM {$this->db->posts} AS p
 			INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
@@ -158,7 +158,7 @@ 
 			GROUP BY p.ID
 			ORDER By p.ID ASC
 			LIMIT {$limit}
-		");
+		" );
 	}
 
 	/**
@@ -168,7 +168,7 @@ 
 	 */
 	public function getReviewsMeta( $key, $status = 'publish' )
 	{
-		$values = $this->db->get_col("
+		$values = $this->db->get_col( "
 			SELECT DISTINCT m.meta_value
 			FROM {$this->db->postmeta} m
 			LEFT JOIN {$this->db->posts} p ON p.ID = m.post_id
@@ -178,7 +178,7 @@ 
 			AND p.post_status = '{$status}'
 			GROUP BY p.ID -- remove duplicate meta_value entries
 			ORDER BY m.meta_id ASC -- sort by oldest meta_value
-		");
+		" );
 		sort( $values );
 		return $values;
 	}
@@ -189,16 +189,16 @@ 
 	 */
 	protected function getAndForCounts( array $args, $and = '' )
 	{
-		$postIds = implode( ',', array_filter( glsr_get( $args, 'post_ids')));
-		$termIds = implode( ',', array_filter( glsr_get( $args, 'term_ids')));
-		if( !empty( $args['type'] )) {
-			$and.= "AND m2.meta_value = '{$args['type']}' ";
+		$postIds = implode( ',', array_filter( glsr_get( $args, 'post_ids' ) ) );
+		$termIds = implode( ',', array_filter( glsr_get( $args, 'term_ids' ) ) );
+		if( !empty($args['type']) ) {
+			$and .= "AND m2.meta_value = '{$args['type']}' ";
 		}
 		if( $postIds ) {
-			$and.= "AND m3.meta_key = 'assigned_to' AND m3.meta_value IN ({$postIds}) ";
+			$and .= "AND m3.meta_key = 'assigned_to' AND m3.meta_value IN ({$postIds}) ";
 		}
 		if( $termIds ) {
-			$and.= "AND tr.term_taxonomy_id IN ({$termIds}) ";
+			$and .= "AND tr.term_taxonomy_id IN ({$termIds}) ";
 		}
 		return apply_filters( 'site-reviews/query/and-for-counts', $and );
 	}
@@ -209,11 +209,11 @@ 
 	 */
 	protected function getInnerJoinForCounts( array $args, $innerJoin = '' )
 	{
-		if( !empty( glsr_get( $args, 'post_ids'))) {
-			$innerJoin.= "INNER JOIN {$this->db->postmeta} AS m3 ON p.ID = m3.post_id ";
+		if( !empty(glsr_get( $args, 'post_ids' )) ) {
+			$innerJoin .= "INNER JOIN {$this->db->postmeta} AS m3 ON p.ID = m3.post_id ";
 		}
-		if( !empty( glsr_get( $args, 'term_ids'))) {
-			$innerJoin.= "INNER JOIN {$this->db->term_relationships} AS tr ON p.ID = tr.object_id ";
+		if( !empty(glsr_get( $args, 'term_ids' )) ) {
+			$innerJoin .= "INNER JOIN {$this->db->term_relationships} AS tr ON p.ID = tr.object_id ";
 		}
 		return apply_filters( 'site-reviews/query/inner-join-for-counts', $innerJoin );
 	}

plugin/Router.php

Spacing +14 added lines, -14 removed lines

@@ -20,7 +20,7 @@ 
 	{
 		$this->unguardedActions = apply_filters( 'site-reviews/router/unguarded-actions', [
 			'fetch-paged-reviews',
-		]);
+		] );
 	}
 
 	/**
@@ -29,7 +29,7 @@ 
 	public function routeAdminPostRequest()
 	{
 		$request = $this->getRequest();
-		if( !$this->isValidPostRequest( $request ))return;
+		if( !$this->isValidPostRequest( $request ) )return;
 		check_admin_referer( $request['_action'] );
 		$this->routeRequest( 'admin', $request['_action'], $request );
 	}
@@ -53,8 +53,8 @@ 
 	{
 		if( is_admin() )return;
 		$request = $this->getRequest();
-		if( !$this->isValidPostRequest( $request ))return;
-		if( !$this->isValidPublicNonce( $request ))return;
+		if( !$this->isValidPostRequest( $request ) )return;
+		if( !$this->isValidPublicNonce( $request ) )return;
 		$this->routeRequest( 'public', $request['_action'], $request );
 	}
 
@@ -66,10 +66,10 @@ 
 		if( !is_user_logged_in()
 			|| in_array( glsr_get( $request, '_action' ), $this->unguardedActions )
 		)return;
-		if( !isset( $request['_nonce'] )) {
+		if( !isset($request['_nonce']) ) {
 			$this->sendAjaxError( 'request is missing a nonce', $request );
 		}
-		if( !wp_verify_nonce( $request['_nonce'], $request['_action'] )) {
+		if( !wp_verify_nonce( $request['_nonce'], $request['_action'] ) ) {
 			$this->sendAjaxError( 'request failed the nonce check', $request, 403 );
 		}
 	}
@@ -79,10 +79,10 @@ 
 	 */
 	protected function checkAjaxRequest( array $request )
 	{
-		if( !isset( $request['_action'] )) {
+		if( !isset($request['_action']) ) {
 			$this->sendAjaxError( 'request must include an action', $request );
 		}
-		if( empty( $request['_ajax_request'] )) {
+		if( empty($request['_ajax_request']) ) {
 			$this->sendAjaxError( 'request is invalid', $request );
 		}
 	}
@@ -109,7 +109,7 @@ 
 	 */
 	protected function isValidPostRequest( array $request = [] )
 	{
-		return !empty( $request['_action'] ) && empty( $request['_ajax_request'] );
+		return !empty($request['_action']) && empty($request['_ajax_request']);
 	}
 
 	/**
@@ -117,7 +117,7 @@ 
 	 */
 	protected function isValidPublicNonce( array $request )
 	{
-		if( is_user_logged_in() && !wp_verify_nonce( $request['_nonce'], $request['_action'] )) {
+		if( is_user_logged_in() && !wp_verify_nonce( $request['_nonce'], $request['_action'] ) ) {
 			glsr_log()->error( 'nonce check failed for public request' )->debug( $request );
 			return false;
 		}
@@ -132,11 +132,11 @@ 
 	protected function routeRequest( $type, $action, array $request = [] )
 	{
 		$actionHook = 'site-reviews/route/'.$type.'/request';
-		$controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' ));
+		$controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' ) );
 		$method = glsr( Helper::class )->buildMethodName( $action, 'router' );
 		$request = apply_filters( 'site-reviews/route/request', $request, $action, $type );
 		do_action( $actionHook, $action, $request );
-		if( is_callable( [$controller, $method] )) {
+		if( is_callable( [$controller, $method] ) ) {
 			call_user_func( [$controller, $method], $request );
 			return;
 		}
@@ -154,10 +154,10 @@ 
 	{
 		glsr_log()->error( $error )->debug( $request );
 		glsr( Notice::class )->addError( __( 'There was an error (try refreshing the page).', 'site-reviews' ).' <code>'.$error.'</code>' );
-		wp_send_json_error([
+		wp_send_json_error( [
 			'message' => __( 'The form could not be submitted. Please notify the site administrator.', 'site-reviews' ),
 			'notices' => glsr( Notice::class )->get(),
 			'error' => $error,
-		]);
+		] );
 	}
 }

Braces +15 added lines, -5 removed lines

@@ -29,7 +29,9 @@ 
 	public function routeAdminPostRequest()
 	{
 		$request = $this->getRequest();
-		if( !$this->isValidPostRequest( $request ))return;
+		if( !$this->isValidPostRequest( $request )) {
+			return;
+		}
 		check_admin_referer( $request['_action'] );
 		$this->routeRequest( 'admin', $request['_action'], $request );
 	}
@@ -51,10 +53,16 @@ 
 	 */
 	public function routePublicPostRequest()
 	{
-		if( is_admin() )return;
+		if( is_admin() ) {
+			return;
+		}
 		$request = $this->getRequest();
-		if( !$this->isValidPostRequest( $request ))return;
-		if( !$this->isValidPublicNonce( $request ))return;
+		if( !$this->isValidPostRequest( $request )) {
+			return;
+		}
+		if( !$this->isValidPublicNonce( $request )) {
+			return;
+		}
 		$this->routeRequest( 'public', $request['_action'], $request );
 	}
 
@@ -65,7 +73,9 @@ 
 	{
 		if( !is_user_logged_in()
 			|| in_array( glsr_get( $request, '_action' ), $this->unguardedActions )
-		)return;
+		) {
+			return;
+		}
 		if( !isset( $request['_nonce'] )) {
 			$this->sendAjaxError( 'request is missing a nonce', $request );
 		}