pryley / site-reviews

plugin/Controllers/EditorController/Metaboxes.php

Spacing +6 added lines, -6 removed lines

@@ -13,10 +13,10 @@ 
 	 */
 	public function saveAssignedToMetabox( $postId )
 	{
-		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' ))return;
-		$assignedTo = strval( glsr( Helper::class )->filterInput( 'assigned_to' ));
+		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' ) )return;
+		$assignedTo = strval( glsr( Helper::class )->filterInput( 'assigned_to' ) );
 		if( get_post_meta( $postId, 'assigned_to', true ) != $assignedTo ) {
-			$review = glsr( ReviewManager::class )->single( get_post( $postId ));
+			$review = glsr( ReviewManager::class )->single( get_post( $postId ) );
 			glsr( CountsManager::class )->decreasePostCounts( $review );
 		}
 		update_post_meta( $postId, 'assigned_to', $assignedTo );
@@ -28,12 +28,12 @@ 
 	 */
 	public function saveResponseMetabox( $postId )
 	{
-		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' ))return;
-		$response = strval( glsr( Helper::class )->filterInput( 'response' ));
+		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' ) )return;
+		$response = strval( glsr( Helper::class )->filterInput( 'response' ) );
 		update_post_meta( $postId, 'response', trim( wp_kses( $response, [
 			'a' => ['href' => [], 'title' => []],
 			'em' => [],
 			'strong' => [],
-		])));
+		] ) ) );
 	}
 }

Braces +6 added lines, -2 removed lines

@@ -13,7 +13,9 @@ 
 	 */
 	public function saveAssignedToMetabox( $postId )
 	{
-		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' ))return;
+		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-assigned-to' ), 'assigned_to' )) {
+			return;
+		}
 		$assignedTo = strval( glsr( Helper::class )->filterInput( 'assigned_to' ));
 		if( get_post_meta( $postId, 'assigned_to', true ) != $assignedTo ) {
 			$review = glsr( ReviewManager::class )->single( get_post( $postId ));
@@ -28,7 +30,9 @@ 
 	 */
 	public function saveResponseMetabox( $postId )
 	{
-		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' ))return;
+		if( !wp_verify_nonce( glsr( Helper::class )->filterInput( '_nonce-response' ), 'response' )) {
+			return;
+		}
 		$response = strval( glsr( Helper::class )->filterInput( 'response' ));
 		update_post_meta( $postId, 'response', trim( wp_kses( $response, [
 			'a' => ['href' => [], 'title' => []],

plugin/Database/CountsManager.php

Spacing +33 added lines, -33 removed lines

@@ -65,7 +65,7 @@ 
 			$this->getCounts(),
 			$review->review_type,
 			$review->rating
-		));
+		) );
 	}
 
 	/**
@@ -73,7 +73,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 );
 	}
@@ -84,7 +84,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 );
 		}
@@ -96,17 +96,17 @@ 
 	public function flatten( array $reviewCounts, array $args = [] )
 	{
 		$counts = [];
-		array_walk_recursive( $reviewCounts, function( $num, $index ) use( &$counts ) {
-			$counts[$index] = isset( $counts[$index] )
+		array_walk_recursive( $reviewCounts, function( $num, $index ) use(&$counts) {
+			$counts[$index] = isset($counts[$index])
 				? $num + $counts[$index]
 				: $num;
 		});
 		$args = wp_parse_args( $args, [
 			'max' => Rating::MAX_RATING,
 			'min' => Rating::MIN_RATING,
-		]);
+		] );
 		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;
@@ -121,7 +121,7 @@ 
 			'post_ids' => [],
 			'term_ids' => [],
 			'type' => 'local',
-		]);
+		] );
 		$counts = [];
 		foreach( $args['post_ids'] as $postId ) {
 			$counts[] = $this->getPostCounts( $postId );
@@ -129,7 +129,7 @@ 
 		foreach( $args['term_ids'] as $termId ) {
 			$counts[] = $this->getTermCounts( $termId );
 		}
-		if( empty( $counts )) {
+		if( empty($counts) ) {
 			$counts[] = $this->getCounts();
 		}
 		return array_column( $counts, $args['type'] );
@@ -149,7 +149,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 ) );
 	}
 
 	/**
@@ -158,7 +158,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 ) );
 	}
 
 	/**
@@ -176,14 +176,14 @@ 
 	 */
 	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
-		));
+		) );
 	}
 
 	/**
@@ -191,9 +191,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 );
@@ -204,10 +204,10 @@ 
 	 */
 	public function increaseTermCounts( Review $review )
 	{
-		$termIds = glsr( ReviewManager::class )->normalizeTerms( implode( ',', $review->term_ids ));
+		$termIds = glsr( ReviewManager::class )->normalizeTerms( implode( ',', $review->term_ids ) );
 		foreach( $termIds as $termId ) {
 			$counts = $this->getTermCounts( $termId );
-			$counts = empty( $counts )
+			$counts = empty($counts)
 				? $this->buildTermCounts( $termId )
 				: $this->increaseRating( $counts, $review->review_type, $review->rating );
 			$this->setTermCounts( $termId, $counts );
@@ -230,8 +230,8 @@ 
 	{
 		$ratingCounts = glsr( CountsManager::class )->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 ) );
 	}
 
 	/**
@@ -240,11 +240,11 @@ 
 	 */
 	public function setTermCounts( $termId, array $reviewCounts )
 	{
-		if( !term_exists( $termId ))return;
+		if( !term_exists( $termId ) )return;
 		$ratingCounts = glsr( CountsManager::class )->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 ) );
 	}
 
 	/**
@@ -255,10 +255,10 @@ 
 	{
 		$counts = [];
 		$lastPostId = 0;
-		while( $reviews = $this->queryReviews( $args, $lastPostId, $limit )) {
-			$types = array_keys( array_flip( array_column( $reviews, 'type' )));
+		while( $reviews = $this->queryReviews( $args, $lastPostId, $limit ) ) {
+			$types = array_keys( array_flip( array_column( $reviews, 'type' ) ) );
 			foreach( $types as $type ) {
-				if( isset( $counts[$type] ))continue;
+				if( isset($counts[$type]) )continue;
 				$counts[$type] = array_fill_keys( range( 0, Rating::MAX_RATING ), 0 );
 			}
 			foreach( $reviews as $review ) {
@@ -276,7 +276,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;
@@ -289,10 +289,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 );
@@ -307,7 +307,7 @@ 
 	{
 		foreach( $reviewCounts as &$counts ) {
 			foreach( range( 0, Rating::MAX_RATING ) as $index ) {
-				if( isset( $counts[$index] ))continue;
+				if( isset($counts[$index]) )continue;
 				$counts[$index] = 0;
 			}
 			ksort( $counts );
@@ -322,14 +322,14 @@ 
 	 */
 	protected function queryReviews( array $args = [], $lastPostId, $limit )
 	{
-		$args = wp_parse_args( $args, array_fill_keys( ['post_id', 'term_id'], '' ));
-		if( empty( array_filter( $args ))) {
+		$args = wp_parse_args( $args, array_fill_keys( ['post_id', 'term_id'], '' ) );
+		if( empty(array_filter( $args )) ) {
 			return glsr( SqlQueries::class )->getReviewCounts( $lastPostId, $limit );
 		}
-		if( !empty( $args['post_id'] )) {
+		if( !empty($args['post_id']) ) {
 			return glsr( SqlQueries::class )->getReviewPostCounts( $args['post_id'], $lastPostId, $limit );
 		}
-		if( !empty( $args['term_id'] )) {
+		if( !empty($args['term_id']) ) {
 			return glsr( SqlQueries::class )->getReviewTermCounts( $args['term_id'], $lastPostId, $limit );
 		}
 	}

Braces +21 added lines, -7 removed lines

@@ -73,7 +73,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 );
 	}
@@ -84,7 +86,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 );
 		}
@@ -106,7 +110,9 @@ 
 			'min' => Rating::MIN_RATING,
 		]);
 		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;
@@ -191,7 +197,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 )
@@ -240,7 +248,9 @@ 
 	 */
 	public function setTermCounts( $termId, array $reviewCounts )
 	{
-		if( !term_exists( $termId ))return;
+		if( !term_exists( $termId )) {
+			return;
+		}
 		$ratingCounts = glsr( CountsManager::class )->flatten( $reviewCounts );
 		update_term_meta( $termId, static::META_COUNT, $reviewCounts );
 		update_term_meta( $termId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts ));
@@ -258,7 +268,9 @@ 
 		while( $reviews = $this->queryReviews( $args, $lastPostId, $limit )) {
 			$types = array_keys( array_flip( array_column( $reviews, 'type' )));
 			foreach( $types as $type ) {
-				if( isset( $counts[$type] ))continue;
+				if( isset( $counts[$type] )) {
+					continue;
+				}
 				$counts[$type] = array_fill_keys( range( 0, Rating::MAX_RATING ), 0 );
 			}
 			foreach( $reviews as $review ) {
@@ -307,7 +319,9 @@ 
 	{
 		foreach( $reviewCounts as &$counts ) {
 			foreach( range( 0, Rating::MAX_RATING ) as $index ) {
-				if( isset( $counts[$index] ))continue;
+				if( isset( $counts[$index] )) {
+					continue;
+				}
 				$counts[$index] = 0;
 			}
 			ksort( $counts );

plugin/Database/ReviewManager.php

Braces +12 added lines, -4 removed lines

@@ -111,7 +111,9 @@ 
 		$termIds = array_filter( array_map( 'trim', explode( ',', $commaSeparatedTermIds )));
 		foreach( $termIds as $termId ) {
 			$term = get_term( $termId, Application::TAXONOMY );
-			if( !isset( $term->term_id ))continue;
+			if( !isset( $term->term_id )) {
+				continue;
+			}
 			$terms[] = $term->term_id;
 		}
 		return $terms;
@@ -123,7 +125,9 @@ 
 	 */
 	public function revert( $postId )
 	{
-		if( get_post_field( 'post_type', $postId ) != Application::POST_TYPE )return;
+		if( get_post_field( 'post_type', $postId ) != Application::POST_TYPE ) {
+			return;
+		}
 		delete_post_meta( $postId, '_edit_last' );
 		$result = wp_update_post([
 			'ID' => $postId,
@@ -141,7 +145,9 @@ 
 	 */
 	public function single( WP_Post $post )
 	{
-		if( $post->post_type != Application::POST_TYPE )return;
+		if( $post->post_type != Application::POST_TYPE ) {
+			return;
+		}
 		$review = new Review( $post );
 		return apply_filters( 'site-reviews/get/review', $review, $post );
 	}
@@ -166,7 +172,9 @@ 
 	protected function setTerms( $postId, $termIds )
 	{
 		$terms = $this->normalizeTerms( $termIds );
-		if( empty( $terms ))return;
+		if( empty( $terms )) {
+			return;
+		}
 		$result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY );
 		if( is_wp_error( $result )) {
 			glsr_log()->error( $result->get_error_message() );

Spacing +13 added lines, -13 removed lines

@@ -35,13 +35,13 @@ 
 			'post_type' => Application::POST_TYPE,
 		];
 		$postId = wp_insert_post( $post, true );
-		if( is_wp_error( $postId )) {
+		if( is_wp_error( $postId ) ) {
 			glsr_log()->error( $postId->get_error_message() )->debug( $post );
 			return false;
 		}
 		$this->setTerms( $postId, $command->category );
 		do_action( 'site-reviews/create/review', $post, $review, $postId );
-		return $this->single( get_post( $postId ));
+		return $this->single( get_post( $postId ) );
 	}
 
 	/**
@@ -50,7 +50,7 @@ 
 	 */
 	public function delete( $metaReviewId )
 	{
-		if( $postId = $this->getPostId( $metaReviewId )) {
+		if( $postId = $this->getPostId( $metaReviewId ) ) {
 			wp_delete_post( $postId, true );
 		}
 	}
@@ -72,7 +72,7 @@ 
 		$paged = glsr( QueryBuilder::class )->getPaged(
 			wp_validate_boolean( $args['pagination'] )
 		);
-		$reviews = new WP_Query([
+		$reviews = new WP_Query( [
 			'meta_key' => 'pinned',
 			'meta_query' => $metaQuery,
 			'offset' => $args['offset'],
@@ -85,7 +85,7 @@ 
 			'post_type' => Application::POST_TYPE,
 			'posts_per_page' => $args['count'],
 			'tax_query' => $taxQuery,
-		]);
+		] );
 		return (object)[
 			'results' => array_map( [$this, 'single'], $reviews->posts ),
 			'max_num_pages' => $reviews->max_num_pages,
@@ -108,10 +108,10 @@ 
 	public function normalizeTerms( $commaSeparatedTermIds )
 	{
 		$terms = [];
-		$termIds = array_filter( array_map( 'trim', explode( ',', $commaSeparatedTermIds )));
+		$termIds = array_filter( array_map( 'trim', explode( ',', $commaSeparatedTermIds ) ) );
 		foreach( $termIds as $termId ) {
 			$term = get_term( $termId, Application::TAXONOMY );
-			if( !isset( $term->term_id ))continue;
+			if( !isset($term->term_id) )continue;
 			$terms[] = $term->term_id;
 		}
 		return $terms;
@@ -125,13 +125,13 @@ 
 	{
 		if( get_post_field( 'post_type', $postId ) != Application::POST_TYPE )return;
 		delete_post_meta( $postId, '_edit_last' );
-		$result = wp_update_post([
+		$result = wp_update_post( [
 			'ID' => $postId,
 			'post_content' => get_post_meta( $postId, 'content', true ),
 			'post_date' => get_post_meta( $postId, 'date', true ),
 			'post_title' => get_post_meta( $postId, 'title', true ),
-		]);
-		if( is_wp_error( $result )) {
+		] );
+		if( is_wp_error( $result ) ) {
 			glsr_log()->error( $result->get_error_message() );
 		}
 	}
@@ -153,7 +153,7 @@ 
 	protected function getNewPostStatus( array $review, $isBlacklisted )
 	{
 		$requireApprovalOption = glsr( OptionManager::class )->get( 'settings.general.require.approval' );
-		return $review['review_type'] == 'local' && ( $requireApprovalOption == 'yes' || $isBlacklisted )
+		return $review['review_type'] == 'local' && ($requireApprovalOption == 'yes' || $isBlacklisted)
 			? 'pending'
 			: 'publish';
 	}
@@ -166,9 +166,9 @@ 
 	protected function setTerms( $postId, $termIds )
 	{
 		$terms = $this->normalizeTerms( $termIds );
-		if( empty( $terms ))return;
+		if( empty($terms) )return;
 		$result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY );
-		if( is_wp_error( $result )) {
+		if( is_wp_error( $result ) ) {
 			glsr_log()->error( $result->get_error_message() );
 		}
 	}

plugin/Controllers/ReviewController.php

Spacing +11 added lines, -11 removed lines

@@ -25,13 +25,13 @@ 
 	{
 		sort( $termIds );
 		sort( $oldTermIds );
-		if( $termIds === $oldTermIds || !$this->isReviewPostId( $postId ))return;
-		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
+		if( $termIds === $oldTermIds || !$this->isReviewPostId( $postId ) )return;
+		$review = glsr( ReviewManager::class )->single( get_post( $postId ) );
 		$ignoredTerms = array_intersect( $oldTermIds, $termIds );
-		if( $review->term_ids = array_diff( $oldTermIds, $ignoredTerms )) {
+		if( $review->term_ids = array_diff( $oldTermIds, $ignoredTerms ) ) {
 			glsr( CountsManager::class )->decreaseTermCounts( $review );
 		}
-		if( $review->term_ids = array_diff( $termIds, $ignoredTerms )) {
+		if( $review->term_ids = array_diff( $termIds, $ignoredTerms ) ) {
 			glsr( CountsManager::class )->increaseTermCounts( $review );
 		}
 	}
@@ -45,8 +45,8 @@ 
 	 */
 	public function onAfterCreate( $postData, $meta, $postId )
 	{
-		if( !$this->isReviewPostId( $postId ))return;
-		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
+		if( !$this->isReviewPostId( $postId ) )return;
+		$review = glsr( ReviewManager::class )->single( get_post( $postId ) );
 		glsr( CountsManager::class )->increase( $review );
 	}
 
@@ -57,8 +57,8 @@ 
 	 */
 	public function onBeforeDelete( $postId )
 	{
-		if( !$this->isReviewPostId( $postId ))return;
-		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
+		if( !$this->isReviewPostId( $postId ) )return;
+		$review = glsr( ReviewManager::class )->single( get_post( $postId ) );
 		glsr( CountsManager::class )->decrease( $review );
 	}
 
@@ -75,7 +75,7 @@ 
 		if( !$this->isReviewPostId( $postId )
 			|| !in_array( $metaKey, ['assigned_to', 'rating', 'review_type'] )
 		)return;
-		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
+		$review = glsr( ReviewManager::class )->single( get_post( $postId ) );
 		if( $review->$metaKey == $metaValue )return;
 		$method = glsr( Helper::class )->buildMethodName( $metaKey, 'onBeforeChange' );
 		call_user_func( [$this, $method], $review, $metaValue );
@@ -122,8 +122,8 @@ 
 	 */
 	public function onChangeStatus( $newStatus, $oldStatus, WP_Post $post )
 	{
-		if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ))return;
-		$review = glsr( ReviewManager::class )->single( get_post( $post->ID ));
+		if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ) )return;
+		$review = glsr( ReviewManager::class )->single( get_post( $post->ID ) );
 		if( $post->post_status == 'publish' ) {
 			glsr( CountsManager::class )->increase( $review );
 		}

Braces +18 added lines, -6 removed lines

@@ -25,7 +25,9 @@ 
 	{
 		sort( $termIds );
 		sort( $oldTermIds );
-		if( $termIds === $oldTermIds || !$this->isReviewPostId( $postId ))return;
+		if( $termIds === $oldTermIds || !$this->isReviewPostId( $postId )) {
+			return;
+		}
 		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
 		$ignoredTerms = array_intersect( $oldTermIds, $termIds );
 		if( $review->term_ids = array_diff( $oldTermIds, $ignoredTerms )) {
@@ -45,7 +47,9 @@ 
 	 */
 	public function onAfterCreate( $postData, $meta, $postId )
 	{
-		if( !$this->isReviewPostId( $postId ))return;
+		if( !$this->isReviewPostId( $postId )) {
+			return;
+		}
 		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
 		glsr( CountsManager::class )->increase( $review );
 	}
@@ -57,7 +61,9 @@ 
 	 */
 	public function onBeforeDelete( $postId )
 	{
-		if( !$this->isReviewPostId( $postId ))return;
+		if( !$this->isReviewPostId( $postId )) {
+			return;
+		}
 		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
 		glsr( CountsManager::class )->decrease( $review );
 	}
@@ -74,9 +80,13 @@ 
 	{
 		if( !$this->isReviewPostId( $postId )
 			|| !in_array( $metaKey, ['assigned_to', 'rating', 'review_type'] )
-		)return;
+		) {
+			return;
+		}
 		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
-		if( $review->$metaKey == $metaValue )return;
+		if( $review->$metaKey == $metaValue ) {
+			return;
+		}
 		$method = glsr( Helper::class )->buildMethodName( $metaKey, 'onBeforeChange' );
 		call_user_func( [$this, $method], $review, $metaValue );
 	}
@@ -122,7 +132,9 @@ 
 	 */
 	public function onChangeStatus( $newStatus, $oldStatus, WP_Post $post )
 	{
-		if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ))return;
+		if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] )) {
+			return;
+		}
 		$review = glsr( ReviewManager::class )->single( get_post( $post->ID ));
 		if( $post->post_status == 'publish' ) {
 			glsr( CountsManager::class )->increase( $review );

plugin/Modules/Html/Partials/StarRating.php

Spacing +2 added lines, -2 removed lines

@@ -12,12 +12,12 @@
 	 */
 	public function build( array $args = [] )
 	{
-		require_once( ABSPATH.'wp-admin/includes/template.php' );
+		require_once(ABSPATH.'wp-admin/includes/template.php');
 		ob_start();
 		wp_star_rating( $args );
 		$stars = ob_get_clean();
 		return glsr( Builder::class )->div( $stars, [
 			'class' => 'glsr-stars',
-		]);
+		] );
 	}
 }

plugin/Modules/Html/Partials/SiteReviewsSummary.php

Spacing +21 added lines, -21 removed lines

@@ -33,14 +33,14 @@ 
 	public function build( array $args = [] )
 	{
 		$this->args = $args;
-		$counts = glsr( CountsManager::class )->get([
-			'post_ids' => array_filter( array_map( 'trim', explode( ',', $args['assigned_to'] ))),
-			'term_ids' => array_filter( array_map( 'trim', explode( ',', $args['category'] ))),
-		]);
+		$counts = glsr( CountsManager::class )->get( [
+			'post_ids' => array_filter( array_map( 'trim', explode( ',', $args['assigned_to'] ) ) ),
+			'term_ids' => array_filter( array_map( 'trim', explode( ',', $args['category'] ) ) ),
+		] );
 		$this->ratingCounts = glsr( CountsManager::class )->flatten( $counts, [
 			'min' => $args['rating'],
-		]);
-		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', [
@@ -52,7 +52,7 @@ 
 				'stars' => $this->buildStars(),
 				'text' => $this->buildText(),
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -60,16 +60,16 @@ 
 	 */
 	protected function buildPercentage()
 	{
-		if( $this->isHidden( 'bars' ))return;
+		if( $this->isHidden( 'bars' ) )return;
 		$range = range( Rating::MAX_RATING, 1 );
-		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts ));
-		$bars = array_reduce( $range, function( $carry, $level ) use( $percentages ) {
+		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts ) );
+		$bars = array_reduce( $range, function( $carry, $level ) use($percentages) {
 			$label = $this->buildPercentageLabel( $this->args['labels'][$level] );
 			$background = $this->buildPercentageBackground( $percentages[$level] );
 			$percent = $this->buildPercentagePercent( $percentages[$level] );
 			return $carry.glsr( Builder::class )->div( $label.$background.$percent, [
 				'class' => 'glsr-bar',
-			]);
+			] );
 		});
 		return $this->wrap( 'percentage', $bars );
 	}
@@ -80,10 +80,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>';
 	}
 
@@ -110,7 +110,7 @@ 
 	 */
 	protected function buildRating()
 	{
-		if( $this->isHidden( 'rating' ))return;
+		if( $this->isHidden( 'rating' ) )return;
 		return $this->wrap( 'rating', '<span>'.$this->averageRating.'</span>' );
 	}
 
@@ -119,7 +119,7 @@ 
 	 */
 	protected function buildStars()
 	{
-		if( $this->isHidden( 'stars' ))return;
+		if( $this->isHidden( 'stars' ) )return;
 		$stars = glsr( Partial::class )->build( 'star-rating', ['rating' => $this->averageRating] );
 		return $this->wrap( 'stars', $stars );
 	}
@@ -129,9 +129,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)',
@@ -142,7 +142,7 @@ 
 			);
 		}
 		$summary = str_replace(
-			['{rating}','{max}', '{num}'],
+			['{rating}', '{max}', '{num}'],
 			[$this->averageRating, Rating::MAX_RATING, $count],
 			$this->args['text']
 		);
@@ -154,7 +154,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 )
 		);
@@ -186,6 +186,6 @@ 
 	{
 		return glsr( Builder::class )->div( $value, [
 			'class' => 'glsr-summary-'.$key,
-		]);
+		] );
 	}
 }

Braces +18 added lines, -6 removed lines

@@ -40,7 +40,9 @@ 
 		$this->ratingCounts = glsr( CountsManager::class )->flatten( $counts, [
 			'min' => $args['rating'],
 		]);
-		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', [
@@ -60,7 +62,9 @@ 
 	 */
 	protected function buildPercentage()
 	{
-		if( $this->isHidden( 'bars' ))return;
+		if( $this->isHidden( 'bars' )) {
+			return;
+		}
 		$range = range( Rating::MAX_RATING, 1 );
 		$percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts ));
 		$bars = array_reduce( $range, function( $carry, $level ) use( $percentages ) {
@@ -110,7 +114,9 @@ 
 	 */
 	protected function buildRating()
 	{
-		if( $this->isHidden( 'rating' ))return;
+		if( $this->isHidden( 'rating' )) {
+			return;
+		}
 		return $this->wrap( 'rating', '<span>'.$this->averageRating.'</span>' );
 	}
 
@@ -119,7 +125,9 @@ 
 	 */
 	protected function buildStars()
 	{
-		if( $this->isHidden( 'stars' ))return;
+		if( $this->isHidden( 'stars' )) {
+			return;
+		}
 		$stars = glsr( Partial::class )->build( 'star-rating', ['rating' => $this->averageRating] );
 		return $this->wrap( 'stars', $stars );
 	}
@@ -129,7 +137,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
@@ -154,7 +164,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/Style.php

Braces +6 added lines, -2 removed lines

@@ -47,7 +47,9 @@ 
 		}
 		$views = $this->generatePossibleViews( $view );
 		foreach( $views as $possibleView ) {
-			if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' )))continue;
+			if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' ))) {
+				continue;
+			}
 			return $possibleView;
 		}
 		return $view;
@@ -84,7 +86,9 @@ 
 	 */
 	public function modifyField( Builder $instance )
 	{
-		if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->fields )))return;
+		if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->fields ))) {
+			return;
+		}
 		call_user_func_array( [$this, 'customize'], [&$instance] );
 	}
 

Spacing +6 added lines, -6 removed lines

@@ -44,12 +44,12 @@ 
 			'templates/form/submit-button',
 			'templates/reviews-form',
 		];
-		if( !preg_match( '('.implode( '|', $styledViews ).')', $view )) {
+		if( !preg_match( '('.implode( '|', $styledViews ).')', $view ) ) {
 			return $view;
 		}
 		$views = $this->generatePossibleViews( $view );
 		foreach( $views as $possibleView ) {
-			if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' )))continue;
+			if( !file_exists( glsr()->path( 'views/'.$possibleView.'.php' ) ) )continue;
 			return $possibleView;
 		}
 		return $view;
@@ -81,7 +81,7 @@ 
 	 */
 	public function modifyField( Builder $instance )
 	{
-		if( !$this->isPublicInstance( $instance ) || empty( array_filter( $this->fields )))return;
+		if( !$this->isPublicInstance( $instance ) || empty(array_filter( $this->fields )) )return;
 		call_user_func_array( [$this, 'customize'], [&$instance] );
 	}
 
@@ -90,9 +90,9 @@ 
 	 */
 	protected function customize( Builder $instance )
 	{
-		$args = wp_parse_args( $instance->args, array_fill_keys( ['class', 'type'], '' ));
+		$args = wp_parse_args( $instance->args, array_fill_keys( ['class', 'type'], '' ) );
 		$key = $instance->tag.'_'.$args['type'];
-		$classes = !isset( $this->fields[$key] )
+		$classes = !isset($this->fields[$key])
 			? $this->fields[$instance->tag]
 			: $this->fields[$key];
 		$instance->args['class'] = trim( $args['class'].' '.$classes );
@@ -126,7 +126,7 @@ 
 		$args = wp_parse_args( $instance->args, [
 			'is_public' => false,
 			'is_raw' => false,
-		]);
+		] );
 		if( is_admin() || !$args['is_public'] || $args['is_raw'] ) {
 			return false;
 		}

plugin/Modules/Validator.php

Braces +9 added lines, -3 removed lines

@@ -93,7 +93,9 @@ 
 		foreach( $this->rules as $attribute => $rules ) {
 			foreach( $rules as $rule ) {
 				$this->validateAttribute( $attribute, $rule );
-				if( $this->shouldStopValidating( $attribute ))break;
+				if( $this->shouldStopValidating( $attribute )) {
+					break;
+				}
 			}
 		}
 		return $this->errors;
@@ -109,7 +111,9 @@ 
 	public function validateAttribute( $attribute, $rule )
 	{
 		list( $rule, $parameters ) = $this->parseRule( $rule );
-		if( $rule == '' )return;
+		if( $rule == '' ) {
+			return;
+		}
 		$value = $this->getValue( $attribute );
 		if( !method_exists( $this, $method = 'validate'.$rule )) {
 			throw new BadMethodCallException( "Method [$method] does not exist." );
@@ -178,7 +182,9 @@ 
 	 */
 	protected function getRule( $attribute, $rules )
 	{
-		if( !array_key_exists( $attribute, $this->rules ))return;
+		if( !array_key_exists( $attribute, $this->rules )) {
+			return;
+		}
 		$rules = (array)$rules;
 		foreach( $this->rules[$attribute] as $rule ) {
 			list( $rule, $parameters ) = $this->parseRule( $rule );

Spacing +14 added lines, -14 removed lines

@@ -74,7 +74,7 @@ 
 		foreach( $this->rules as $attribute => $rules ) {
 			foreach( $rules as $rule ) {
 				$this->validateAttribute( $attribute, $rule );
-				if( $this->shouldStopValidating( $attribute ))break;
+				if( $this->shouldStopValidating( $attribute ) )break;
 			}
 		}
 		return $this->errors;
@@ -89,13 +89,13 @@ 
 	 */
 	public function validateAttribute( $attribute, $rule )
 	{
-		list( $rule, $parameters ) = $this->parseRule( $rule );
+		list($rule, $parameters) = $this->parseRule( $rule );
 		if( $rule == '' )return;
 		$value = $this->getValue( $attribute );
-		if( !method_exists( $this, $method = 'validate'.$rule )) {
+		if( !method_exists( $this, $method = 'validate'.$rule ) ) {
 			throw new BadMethodCallException( "Method [$method] does not exist." );
 		}
-		if( !$this->$method( $value, $attribute, $parameters )) {
+		if( !$this->$method( $value, $attribute, $parameters ) ) {
 			$this->addFailure( $attribute, $rule, $parameters );
 		}
 	}
@@ -144,7 +144,7 @@ 
 	 */
 	protected function getMessage( $attribute, $rule, array $parameters )
 	{
-		if( in_array( $rule, $this->sizeRules )) {
+		if( in_array( $rule, $this->sizeRules ) ) {
 			return $this->getSizeMessage( $attribute, $rule, $parameters );
 		}
 		$lowerRule = glsr( Helper::class )->snakeCase( $rule );
@@ -159,11 +159,11 @@ 
 	 */
 	protected function getRule( $attribute, $rules )
 	{
-		if( !array_key_exists( $attribute, $this->rules ))return;
+		if( !array_key_exists( $attribute, $this->rules ) )return;
 		$rules = (array)$rules;
 		foreach( $this->rules[$attribute] as $rule ) {
-			list( $rule, $parameters ) = $this->parseRule( $rule );
-			if( in_array( $rule, $rules )) {
+			list($rule, $parameters) = $this->parseRule( $rule );
+			if( in_array( $rule, $rules ) ) {
 				return [$rule, $parameters];
 			}
 		}
@@ -181,7 +181,7 @@ 
 		if( is_numeric( $value ) && $hasNumeric ) {
 			return $value;
 		}
-		elseif( is_array( $value )) {
+		elseif( is_array( $value ) ) {
 			return count( $value );
 		}
 		return mb_strlen( $value );
@@ -207,7 +207,7 @@ 
 	 */
 	protected function getValue( $attribute )
 	{
-		if( isset( $this->data[$attribute] )) {
+		if( isset($this->data[$attribute]) ) {
 			return $this->data[$attribute];
 		}
 	}
@@ -220,7 +220,7 @@ 
 	 */
 	protected function hasRule( $attribute, $rules )
 	{
-		return !is_null( $this->getRule( $attribute, $rules ));
+		return !is_null( $this->getRule( $attribute, $rules ) );
 	}
 
 	/**
@@ -258,7 +258,7 @@ 
 	{
 		$parameters = [];
 		if( strpos( $rule, ':' ) !== false ) {
-			list( $rule, $parameter ) = explode( ':', $rule, 2 );
+			list($rule, $parameter) = explode( ':', $rule, 2 );
 			$parameters = $this->parseParameters( $rule, $parameter );
 		}
 		$rule = glsr( Helper::class )->camelCase( $rule );
@@ -287,7 +287,7 @@ 
 	protected function shouldStopValidating( $attribute )
 	{
 		return $this->hasRule( $attribute, $this->implicitRules )
-			&& isset( $this->failedRules[$attribute] )
+			&& isset($this->failedRules[$attribute])
 			&& array_intersect( array_keys( $this->failedRules[$attribute] ), $this->implicitRules );
 	}
 
@@ -300,7 +300,7 @@ 
 	protected function translator( $key, array $parameters )
 	{
 		$strings = glsr( ValidationStrings::class )->defaults();
-		if( isset( $strings[$key] )) {
+		if( isset($strings[$key]) ) {
 			return $this->replace( $strings[$key], $parameters );
 		}
 	}

plugin/Modules/Validator/ValidationRules.php

Spacing +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
 		if( strpos( $message, '%s' ) === false ) {
 			return $message;
 		}
-		return preg_replace_callback( '/(%s)/', function() use( &$parameters ) {
+		return preg_replace_callback( '/(%s)/', function() use(&$parameters) {
 			foreach( $parameters as $key => $value ) {
 				return array_shift( $parameters );
 			}
@@ -113,8 +113,8 @@ 
 	public function validateRequired( $value )
 	{
 		return is_null( $value )
-			|| ( is_string( $value ) && trim( $value ) === '' )
-			|| ( is_array( $value ) && count( $value ) < 1 )
+			|| (is_string( $value ) && trim( $value ) === '')
+			|| (is_array( $value ) && count( $value ) < 1)
 			? false
 			: true;
 	}