pryley / site-reviews

plugin/Controllers/ReviewController.php

Braces +18 added lines, -6 removed lines

@@ -26,7 +26,9 @@ 
 	{
 		sort( $termTaxonomyIds );
 		sort( $oldTermTaxonomyIds );
-		if( $termTaxonomyIds === $oldTermTaxonomyIds || !$this->isReviewPostId( $postId ))return;
+		if( $termTaxonomyIds === $oldTermTaxonomyIds || !$this->isReviewPostId( $postId )) {
+			return;
+		}
 		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
 		$ignoredIds = array_intersect( $oldTermTaxonomyIds, $termTaxonomyIds );
 		$decreasedIds = array_diff( $oldTermTaxonomyIds, $ignoredIds );
@@ -47,7 +49,9 @@ 
 	 */
 	public function onAfterChangeStatus( $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 );
@@ -63,7 +67,9 @@ 
 	 */
 	public function onAfterCreate( Review $review )
 	{
-		if( $review->status !== 'publish' )return;
+		if( $review->status !== 'publish' ) {
+			return;
+		}
 		glsr( CountsManager::class )->increase( $review );
 	}
 
@@ -74,7 +80,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 );
 	}
@@ -91,9 +99,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 );
 	}

plugin/Database/CountsManager.php

Braces +21 added lines, -7 removed lines

@@ -74,7 +74,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 );
 	}
@@ -85,7 +87,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 );
 		}
@@ -107,7 +111,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;
@@ -199,7 +205,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 )
@@ -249,7 +257,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 ));
@@ -270,7 +280,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, Rating::MAX_RATING ), 0 );
 			}
 			foreach( $reviews as $review ) {
@@ -323,7 +335,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.php

Braces +9 added lines, -3 removed lines

@@ -23,7 +23,9 @@ 
 		if( empty( $assignedTo )) {
 			$assignedTo = get_post_meta( $postId, 'assigned_to', true );
 		}
-		if( empty( $assignedTo ))return;
+		if( empty( $assignedTo )) {
+			return;
+		}
 		$assignedPost = get_post( $assignedTo );
 		if( $assignedPost instanceof WP_Post && $assignedPost->ID != $postId ) {
 			return $assignedPost;
@@ -80,7 +82,9 @@ 
 		$termIds = [];
 		foreach( $values as $value ) {
 			$term = get_term_by( $field, $value, Application::TAXONOMY );
-			if( !isset( $term->term_id ))continue;
+			if( !isset( $term->term_id )) {
+				continue;
+			}
 			$termIds[] = $term->term_id;
 		}
 		return $termIds;
@@ -127,7 +131,9 @@ 
 		add_filter( 'posts_search', [$queryBuilder, 'filterSearchByTitle'], 500, 2 );
 		$search = new WP_Query( $args );
 		remove_filter( 'posts_search', [$queryBuilder, 'filterSearchByTitle'], 500 );
-		if( !$search->have_posts() )return;
+		if( !$search->have_posts() ) {
+			return;
+		}
 		$results = '';
 		while( $search->have_posts() ) {
 			$search->the_post();

plugin/Shortcodes/SiteReviewsSummaryShortcode.php

Braces +2 added lines, -1 removed lines

@@ -6,7 +6,8 @@
 
 class SiteReviewsSummaryShortcode extends Shortcode
 {
-	protected function hideOptions() {
+	protected function hideOptions()
+	{
 		return [
 			'rating' => __( 'Hide the rating', 'site-reviews' ),
 			'stars' => __( 'Hide the stars', 'site-reviews' ),

plugin/Shortcodes/SiteReviewsShortcode.php

Braces +2 added lines, -1 removed lines

@@ -6,7 +6,8 @@
 
 class SiteReviewsShortcode extends Shortcode
 {
-	protected function hideOptions() {
+	protected function hideOptions()
+	{
 		return [
 			'title' => __( 'Hide the title', 'site-reviews' ),
 			'rating' => __( 'Hide the rating', 'site-reviews' ),

plugin/Shortcodes/SiteReviewsFormShortcode.php

Braces +2 added lines, -1 removed lines

@@ -6,7 +6,8 @@
 
 class SiteReviewsFormShortcode extends Shortcode
 {
-	protected function hideOptions() {
+	protected function hideOptions()
+	{
 		return [
 			'rating' => __( 'Hide the rating field', 'site-reviews' ),
 			'title' => __( 'Hide the title field', 'site-reviews' ),

plugin/Modules/Session.php

Braces +6 added lines, -2 removed lines

@@ -150,7 +150,9 @@ 
 		if( !empty( $sessions )) {
 			$now = time();
 			foreach( $sessions as $session ) {
-				if( $now <= $session->expiration )continue;
+				if( $now <= $session->expiration ) {
+					continue;
+				}
 				$expiredSessions[] = $session->name;
 				$expiredSessions[] = str_replace( '_expires_', '_', $session->name );
 			}
@@ -193,7 +195,9 @@ 
 	 */
 	protected function setCookie()
 	{
-		if( headers_sent() )return;
+		if( headers_sent() ) {
+			return;
+		}
 		$cookie = $this->sessionId.static::DELIMITER.$this->expiryTimestamp.static::DELIMITER.$this->expiryTimestampReset;
 		$cookiePath = preg_replace( '|https?://[^/]+|i', '', trailingslashit( (string)get_option( 'home' )));
 		setcookie( static::SESSION_COOKIE, $cookie, $this->expiryTimestamp, $cookiePath );

plugin/Modules/Notification.php

Braces +9 added lines, -3 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(),
@@ -146,7 +148,9 @@ 
 	protected function sendToEmail( array $args )
 	{
 		$email = $this->buildEmail( $args );
-		if( !$this->email )return;
+		if( !$this->email ) {
+			return;
+		}
 		if( empty( $email->to )) {
 			glsr_log()->error( 'Email notification was not sent (missing email address)' );
 			return;
@@ -161,7 +165,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/Validator/ValidateReview.php

Braces +30 added lines, -10 removed lines

@@ -175,8 +175,12 @@ 
 	 */
 	protected function validateAkismet()
 	{
-		if( !empty( $this->error ))return;
-		if( !glsr( Akismet::class )->isSpam( $this->request ))return;
+		if( !empty( $this->error )) {
+			return;
+		}
+		if( !glsr( Akismet::class )->isSpam( $this->request )) {
+			return;
+		}
 		$this->setSessionValues( 'errors', [], 'Akismet caught a spam submission (consider adding the IP address to the blacklist):' );
 		$this->error = __( 'This review has been flagged as possible spam and cannot be submitted.', 'site-reviews' );
 	}
@@ -186,8 +190,12 @@ 
 	 */
 	protected function validateBlacklist()
 	{
-		if( !empty( $this->error ))return;
-		if( !glsr( Blacklist::class )->isBlacklisted( $this->request ))return;
+		if( !empty( $this->error )) {
+			return;
+		}
+		if( !glsr( Blacklist::class )->isBlacklisted( $this->request )) {
+			return;
+		}
 		$blacklistAction = $this->getOption( 'settings.submissions.blacklist.action' );
 		if( $blacklistAction == 'reject' ) {
 			$this->setSessionValues( 'errors', [], 'Blacklisted submission detected:' );
@@ -202,9 +210,13 @@ 
 	 */
 	protected function validateCustom()
 	{
-		if( !empty( $this->error ))return;
+		if( !empty( $this->error )) {
+			return;
+		}
 		$validated = apply_filters( 'site-reviews/validate/custom', true, $this->request );
-		if( $validated === true )return;
+		if( $validated === true ) {
+			return;
+		}
 		$this->setSessionValues( 'errors', [] );
 		$this->setSessionValues( 'values', $this->request );
 		$this->error = is_string( $validated )
@@ -217,8 +229,12 @@ 
 	 */
 	protected function validateHoneyPot()
 	{
-		if( !empty( $this->error ))return;
-		if( empty( $this->request['gotcha'] ))return;
+		if( !empty( $this->error )) {
+			return;
+		}
+		if( empty( $this->request['gotcha'] )) {
+			return;
+		}
 		$this->setSessionValues( 'errors', [], 'The Honeypot caught a bad submission:' );
 		$this->error = __( 'The review submission failed. Please notify the site administrator.', 'site-reviews' );
 	}
@@ -228,9 +244,13 @@ 
 	 */
 	protected function validateRecaptcha()
 	{
-		if( !empty( $this->error ))return;
+		if( !empty( $this->error )) {
+			return;
+		}
 		$status = $this->getRecaptchaStatus();
-		if( in_array( $status, [static::RECAPTCHA_DISABLED, static::RECAPTCHA_VALID] ))return;
+		if( in_array( $status, [static::RECAPTCHA_DISABLED, static::RECAPTCHA_VALID] )) {
+			return;
+		}
 		if( $status == static::RECAPTCHA_EMPTY ) {
 			$this->setSessionValues( 'recaptcha', 'unset' );
 			$this->recaptchaIsUnset = true;