pryley / site-reviews

plugin/Widgets/SiteReviewsWidget.php

Spacing +11 added lines, -11 removed lines

@@ -20,14 +20,14 @@ 
 			'class' => 'widefat',
 			'label' => __( 'Title', 'site-reviews' ),
 			'name' => 'title',
-		]);
+		] );
 		$this->renderField( 'number', [
 			'class' => 'small-text',
 			'default' => 5,
 			'label' => __( 'How many reviews would you like to display?', 'site-reviews' ),
 			'max' => 100,
 			'name' => 'count',
-		]);
+		] );
 		$this->renderField( 'select', [
 			'label' => __( 'What is the minimum rating to display?', 'site-reviews' ),
 			'name' => 'rating',
@@ -38,22 +38,22 @@ 
 				'2' => sprintf( _n( '%s star', '%s stars', 2, 'site-reviews' ), 2 ),
 				'1' => sprintf( _n( '%s star', '%s stars', 1, 'site-reviews' ), 1 ),
 			],
-		]);
+		] );
 		if( count( glsr()->reviewTypes ) > 1 ) {
 			$this->renderField( 'select', [
 				'class' => 'widefat',
 				'label' => __( 'Which type of review would you like to display?', 'site-reviews' ),
 				'name' => 'type',
 				'options' => ['' => __( 'All Reviews', 'site-reviews' )] + glsr()->reviewTypes,
-			]);
+			] );
 		}
-		if( !empty( $terms )) {
+		if( !empty($terms) ) {
 			$this->renderField( 'select', [
 				'class' => 'widefat',
 				'label' => __( 'Limit reviews to this category', 'site-reviews' ),
 				'name' => 'category',
 				'options' => ['' => __( 'All Categories', 'site-reviews' )] + $terms,
-			]);
+			] );
 		}
 		$this->renderField( 'text', [
 			'class' => 'widefat',
@@ -61,16 +61,16 @@ 
 			'description' => sprintf( __( "Separate multiple ID's with a comma. You may also enter %s to automatically represent the current page/post ID.", 'site-reviews' ), '<code>post_id</code>' ),
 			'label' => __( 'Limit reviews to those assigned to this page/post ID', 'site-reviews' ),
 			'name' => 'assigned_to',
-		]);
+		] );
 		$this->renderField( 'text', [
 			'class' => 'widefat',
 			'label' => __( 'Enter any custom CSS classes here', 'site-reviews' ),
 			'name' => 'class',
-		]);
+		] );
 		$this->renderField( 'checkbox', [
 			'name' => 'hide',
 			'options' => glsr( SiteReviewsShortcode::class )->getHideOptions(),
-		]);
+		] );
 	}
 
 	/**
@@ -80,10 +80,10 @@ 
 	 */
 	public function update( $newInstance, $oldInstance )
 	{
-		if( !is_numeric( $newInstance['count'] )) {
+		if( !is_numeric( $newInstance['count'] ) ) {
 			$newInstance['count'] = 10;
 		}
-		$newInstance['count'] = min( 50, max( 0, intval( $newInstance['count'] )));
+		$newInstance['count'] = min( 50, max( 0, intval( $newInstance['count'] ) ) );
 		return parent::update( $newInstance, $oldInstance );
 	}
 

plugin/Widgets/SiteReviewsFormWidget.php

Spacing +6 added lines, -6 removed lines

@@ -20,34 +20,34 @@
 			'class' => 'widefat',
 			'label' => __( 'Title', 'site-reviews' ),
 			'name' => 'title',
-		]);
+		] );
 		$this->renderField( 'textarea', [
 			'class' => 'widefat',
 			'label' => __( 'Description', 'site-reviews' ),
 			'name' => 'description',
-		]);
+		] );
 		$this->renderField( 'select', [
 			'class' => 'widefat',
 			'label' => __( 'Automatically assign a category', 'site-reviews' ),
 			'name' => 'category',
 			'options' => ['' => __( 'Do not assign a category', 'site-reviews' )] + $terms,
-		]);
+		] );
 		$this->renderField( 'text', [
 			'class' => 'widefat',
 			'default' => '',
 			'description' => sprintf( __( 'You may also enter %s to assign to the current post.', 'site-reviews' ), '<code>post_id</code>' ),
 			'label' => __( 'Assign reviews to a custom page/post ID', 'site-reviews' ),
 			'name' => 'assign_to',
-		]);
+		] );
 		$this->renderField( 'text', [
 			'class' => 'widefat',
 			'label' => __( 'Enter any custom CSS classes here', 'site-reviews' ),
 			'name' => 'class',
-		]);
+		] );
 		$this->renderField( 'checkbox', [
 			'name' => 'hide',
 			'options' => glsr( SiteReviewsFormShortcode::class )->getHideOptions(),
-		]);
+		] );
 	}
 
 	/**

plugin/Widgets/SiteReviewsSummaryWidget.php

Spacing +7 added lines, -7 removed lines

@@ -20,22 +20,22 @@ 
 			'class' => 'widefat',
 			'label' => __( 'Title', 'site-reviews' ),
 			'name' => 'title',
-		]);
+		] );
 		if( count( glsr()->reviewTypes ) > 1 ) {
 			$this->renderField( 'select', [
 				'class' => 'widefat',
 				'label' => __( 'Which type of review would you like to use?', 'site-reviews' ),
 				'name' => 'type',
 				'options' => ['' => __( 'All review types', 'site-reviews' )] + glsr()->reviewTypes,
-			]);
+			] );
 		}
-		if( !empty( $terms )) {
+		if( !empty($terms) ) {
 			$this->renderField( 'select', [
 				'class' => 'widefat',
 				'label' => __( 'Limit summary to this category', 'site-reviews' ),
 				'name' => 'category',
 				'options' => ['' => __( 'All Categories', 'site-reviews' )] + $terms,
-			]);
+			] );
 		}
 		$this->renderField( 'text', [
 			'class' => 'widefat',
@@ -43,16 +43,16 @@ 
 			'description' => sprintf( __( "Separate multiple ID's with a comma. You may also enter %s to automatically represent the current page/post ID.", 'site-reviews' ), '<code>post_id</code>' ),
 			'label' => __( 'Limit summary to reviews assigned to a page/post ID', 'site-reviews' ),
 			'name' => 'assigned_to',
-		]);
+		] );
 		$this->renderField( 'text', [
 			'class' => 'widefat',
 			'label' => __( 'Enter any custom CSS classes here', 'site-reviews' ),
 			'name' => 'class',
-		]);
+		] );
 		$this->renderField( 'checkbox', [
 			'name' => 'hide',
 			'options' => glsr( SiteReviewsSummaryShortcode::class )->getHideOptions(),
-		]);
+		] );
 	}
 
 	/**

plugin/Modules/Slack.php

Spacing +8 added lines, -8 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,7 +129,7 @@ 
 	protected function buildStarsField()
 	{
 		$solidStars = str_repeat( '★', $this->review->rating );
-		$emptyStars = str_repeat( '☆', max( 0, Rating::MAX_RATING - $this->review->rating ));
+		$emptyStars = str_repeat( '☆', max( 0, Rating::MAX_RATING - $this->review->rating ) );
 		$stars = $solidStars.$emptyStars;
 		$stars = apply_filters( 'site-reviews/slack/stars', $stars, $this->review->rating, Rating::MAX_RATING );
 		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/Modules/Style.php

Spacing +6 added lines, -6 removed lines

@@ -50,12 +50,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()->file( $possibleView )))continue;
+			if( !file_exists( glsr()->file( $possibleView ) ) )continue;
 			return glsr( Helper::class )->removePrefix( 'views/', $possibleView );
 		}
 		return $view;
@@ -88,7 +88,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] );
 	}
 
@@ -105,9 +105,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 );
@@ -141,7 +141,7 @@ 
 		$args = wp_parse_args( $instance->args, [
 			'is_public' => false,
 			'is_raw' => false,
-		]);
+		] );
 		if( is_admin() || !$args['is_public'] || $args['is_raw'] ) {
 			return false;
 		}

Braces +6 added lines, -2 removed lines

@@ -55,7 +55,9 @@ 
 		}
 		$views = $this->generatePossibleViews( $view );
 		foreach( $views as $possibleView ) {
-			if( !file_exists( glsr()->file( $possibleView )))continue;
+			if( !file_exists( glsr()->file( $possibleView ))) {
+				continue;
+			}
 			return glsr( Helper::class )->removePrefix( 'views/', $possibleView );
 		}
 		return $view;
@@ -88,7 +90,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] );
 	}
 

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

Spacing +12 added lines, -12 removed lines

@@ -38,7 +38,7 @@ 
 	{
 		$cookieId = filter_input( INPUT_COOKIE, static::SESSION_COOKIE );
 		if( strpos( $cookieId, static::DELIMITER ) !== false ) {
-			$cookie = explode( static::DELIMITER, stripslashes( $cookieId ));
+			$cookie = explode( static::DELIMITER, stripslashes( $cookieId ) );
 			$this->sessionId = preg_replace( '/[^A-Za-z0-9_]/', '', $cookie[0] );
 			$this->expiryTimestamp = absint( $cookie[1] );
 			$this->expiryTimestampReset = absint( $cookie[2] );
@@ -77,7 +77,7 @@ 
 	 */
 	public function deleteExpiredSessions( $limit = 1000 )
 	{
-		if( $expiredSessions = implode( "','", $this->getExpiredSessions( $limit ))) {
+		if( $expiredSessions = implode( "','", $this->getExpiredSessions( $limit ) ) ) {
 			glsr( SqlQueries::class )->deleteExpiredSessions( $expiredSessions );
 		}
 	}
@@ -91,11 +91,11 @@ 
 	public function get( $key, $fallback = '', $unset = false )
 	{
 		$key = sanitize_key( $key );
-		$value = isset( $this->sessionData[$key] )
+		$value = isset($this->sessionData[$key])
 			? maybe_unserialize( $this->sessionData[$key] )
 			: $fallback;
-		if( isset( $this->sessionData[$key] ) && $unset ) {
-			unset( $this->sessionData[$key] );
+		if( isset($this->sessionData[$key]) && $unset ) {
+			unset($this->sessionData[$key]);
 			$this->updateSession();
 		}
 		return $value;
@@ -129,7 +129,7 @@ 
 	protected function deleteSession()
 	{
 		delete_option( $this->getSessionId() );
-		delete_option( $this->getSessionId( 'expires' ));
+		delete_option( $this->getSessionId( 'expires' ) );
 	}
 
 	/**
@@ -137,7 +137,7 @@ 
 	 */
 	protected function generateSessionId()
 	{
-		return md5(( new PasswordHash( 8, false ))->get_random_bytes( 32 ));
+		return md5( (new PasswordHash( 8, false ))->get_random_bytes( 32 ) );
 	}
 
 	/**
@@ -147,8 +147,8 @@ 
 	protected function getExpiredSessions( $limit )
 	{
 		$expiredSessions = [];
-		$sessions = glsr( SqlQueries::class )->getExpiredSessions( static::SESSION_COOKIE, absint( $limit ));
-		if( !empty( $sessions )) {
+		$sessions = glsr( SqlQueries::class )->getExpiredSessions( static::SESSION_COOKIE, absint( $limit ) );
+		if( !empty($sessions) ) {
 			$now = time();
 			foreach( $sessions as $session ) {
 				if( $now <= $session->expiration )continue;
@@ -165,7 +165,7 @@ 
 	 */
 	protected function getSessionId( $separator = '' )
 	{
-		return implode( '_', array_filter( [static::SESSION_COOKIE, $separator, $this->sessionId] ));
+		return implode( '_', array_filter( [static::SESSION_COOKIE, $separator, $this->sessionId] ) );
 	}
 
 	/**
@@ -196,7 +196,7 @@ 
 	{
 		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' )));
+		$cookiePath = preg_replace( '|https?://[^/]+|i', '', trailingslashit( (string)get_option( 'home' ) ) );
 		setcookie( static::SESSION_COOKIE, $cookie, $this->expiryTimestamp, $cookiePath );
 	}
 
@@ -214,7 +214,7 @@ 
 	 */
 	protected function updateSession()
 	{
-		if( false === get_option( $this->getSessionId() )) {
+		if( false === get_option( $this->getSessionId() ) ) {
 			return $this->createSession();
 		}
 		update_option( $this->getSessionId(), $this->sessionData, false );

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 );
@@ -147,7 +147,7 @@ 
 	{
 		$email = $this->buildEmail( $args );
 		if( !$this->email )return;
-		if( empty( $email->to )) {
+		if( empty($email->to) ) {
 			glsr_log()->error( 'Email notification was not sent (missing email address)' );
 			return;
 		}
@@ -164,7 +164,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 +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;

Spacing +22 added lines, -22 removed lines

@@ -67,7 +67,7 @@ 
 		$this->validateBlacklist();
 		$this->validateAkismet();
 		$this->validateRecaptcha();
-		if( !empty( $this->error )) {
+		if( !empty($this->error) ) {
 			$this->setSessionValues( 'message', $this->error );
 		}
 		return $this;
@@ -91,8 +91,8 @@ 
 		if( !glsr( OptionManager::class )->isRecaptchaEnabled() ) {
 			return static::RECAPTCHA_DISABLED;
 		}
-		if( empty( $this->request['_recaptcha-token'] )) {
-			return $this->request['_counter'] < intval( apply_filters( 'site-reviews/recaptcha/timeout', 5 ))
+		if( empty($this->request['_recaptcha-token']) ) {
+			return $this->request['_counter'] < intval( apply_filters( 'site-reviews/recaptcha/timeout', 5 ) )
 				? static::RECAPTCHA_EMPTY
 				: static::RECAPTCHA_FAILED;
 		}
@@ -104,17 +104,17 @@ 
 	 */
 	protected function getRecaptchaTokenStatus()
 	{
-		$endpoint = add_query_arg([
+		$endpoint = add_query_arg( [
 			'remoteip' => glsr( Helper::class )->getIpAddress(),
 			'response' => $this->request['_recaptcha-token'],
 			'secret' => $this->getOption( 'settings.submissions.recaptcha.secret' ),
 		], static::RECAPTCHA_ENDPOINT );
-		if( is_wp_error( $response = wp_remote_get( $endpoint ))) {
+		if( is_wp_error( $response = wp_remote_get( $endpoint ) ) ) {
 			glsr_log()->error( $response->get_error_message() );
 			return static::RECAPTCHA_FAILED;
 		}
-		$response = json_decode( wp_remote_retrieve_body( $response ));
-		if( !empty( $response->success )) {
+		$response = json_decode( wp_remote_retrieve_body( $response ) );
+		if( !empty($response->success) ) {
 			return boolval( $response->success )
 				? static::RECAPTCHA_VALID
 				: static::RECAPTCHA_INVALID;
@@ -132,12 +132,12 @@ 
 	{
 		$rules = array_intersect_key(
 			apply_filters( 'site-reviews/validation/rules', static::VALIDATION_RULES, $request ),
-			array_flip( $this->getOption( 'settings.submissions.required', [] ))
+			array_flip( $this->getOption( 'settings.submissions.required', [] ) )
 		);
-		$excluded = isset( $request['excluded'] )
+		$excluded = isset($request['excluded'])
 			? explode( ',', $request['excluded'] )
 			: [];
-		return array_diff_key( $rules, array_flip( $excluded ));
+		return array_diff_key( $rules, array_flip( $excluded ) );
 	}
 
 	/**
@@ -147,7 +147,7 @@ 
 	{
 		$rules = $this->getValidationRules( $request );
 		$errors = glsr( Validator::class )->validate( $request, $rules );
-		if( empty( $errors )) {
+		if( empty($errors) ) {
 			return true;
 		}
 		$this->setSessionValues( 'errors', $errors );
@@ -164,7 +164,7 @@ 
 	protected function setSessionValues( $type, $value, $loggedMessage = '' )
 	{
 		glsr( Session::class )->set( $this->form_id.$type, $value );
-		if( !empty( $loggedMessage )) {
+		if( !empty($loggedMessage) ) {
 			glsr_log()->warning( $loggedMessage );
 			glsr_log()->warning( $this->request );
 		}
@@ -175,8 +175,8 @@ 
 	 */
 	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 +186,8 @@ 
 	 */
 	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,7 +202,7 @@ 
 	 */
 	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;
 		$this->setSessionValues( 'errors', [] );
@@ -217,8 +217,8 @@ 
 	 */
 	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 +228,9 @@ 
 	 */
 	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;
@@ -250,7 +250,7 @@ 
 	 */
 	protected function validateRequest( array $request )
 	{
-		if( !$this->isRequestValid( $request )) {
+		if( !$this->isRequestValid( $request ) ) {
 			$this->error = __( 'Please fix the submission errors.', 'site-reviews' );
 			return $request;
 		}

plugin/Controllers/SettingsController.php

Spacing +12 added lines, -12 removed lines

@@ -17,7 +17,7 @@ 
 	 */
 	public function callbackRegisterSettings( $input )
 	{
-		if( !is_array( $input )) {
+		if( !is_array( $input ) ) {
 			$input = ['settings' => []];
 		}
 		if( key( $input ) == 'settings' ) {
@@ -26,7 +26,7 @@ 
 			$options = $this->sanitizeSubmissions( $input, $options );
 			$options = $this->sanitizeTranslations( $input, $options );
 			if( filter_input( INPUT_POST, 'option_page' ) == Application::ID.'-settings' ) {
-				glsr( Notice::class )->addSuccess( __( 'Settings updated.', 'site-reviews' ));
+				glsr( Notice::class )->addSuccess( __( 'Settings updated.', 'site-reviews' ) );
 			}
 			return $options;
 		}
@@ -41,7 +41,7 @@ 
 	{
 		register_setting( Application::ID.'-settings', OptionManager::databaseKey(), [
 			'sanitize_callback' => [$this, 'callbackRegisterSettings'],
-		]);
+		] );
 	}
 
 	/**
@@ -56,7 +56,7 @@ 
 		if( trim( $inputForm['notification_message'] ) == '' ) {
 			$options['settings']['general']['notification_message'] = glsr()->defaults['settings']['general']['notification_message'];
 		}
-		$options['settings']['general']['notifications'] = isset( $inputForm['notifications'] )
+		$options['settings']['general']['notifications'] = isset($inputForm['notifications'])
 			? $inputForm['notifications']
 			: [];
 		return $options;
@@ -68,7 +68,7 @@ 
 	protected function sanitizeSubmissions( array $input, array $options )
 	{
 		$inputForm = $input['settings']['submissions'];
-		$options['settings']['submissions']['required'] = isset( $inputForm['required'] )
+		$options['settings']['submissions']['required'] = isset($inputForm['required'])
 			? $inputForm['required']
 			: [];
 		return $options;
@@ -79,17 +79,17 @@ 
 	 */
 	protected function sanitizeTranslations( array $input, array $options )
 	{
-		if( isset( $input['settings']['strings'] )) {
-			$options['settings']['strings'] = array_values( array_filter( $input['settings']['strings'] ));
+		if( isset($input['settings']['strings']) ) {
+			$options['settings']['strings'] = array_values( array_filter( $input['settings']['strings'] ) );
 			$allowedTags = [
 				'a' => ['class' => [], 'href' => [], 'target' => []],
 				'span' => ['class' => []],
 			];
-			array_walk( $options['settings']['strings'], function( &$string ) use( $allowedTags ) {
-				if( isset( $string['s2'] )) {
+			array_walk( $options['settings']['strings'], function( &$string ) use($allowedTags) {
+				if( isset($string['s2']) ) {
 					$string['s2'] = wp_kses( $string['s2'], $allowedTags );
 				}
-				if( isset( $string['p2'] )) {
+				if( isset($string['p2']) ) {
 					$string['p2'] = wp_kses( $string['p2'], $allowedTags );
 				}
 			});
@@ -103,11 +103,11 @@ 
 	protected function isPolylangActiveAndSupported()
 	{
 		if( !glsr( Polylang::class )->isActive() ) {
-			glsr( Notice::class )->addError( __( 'Please install/activate the Polylang plugin to enable integration.', 'site-reviews' ));
+			glsr( Notice::class )->addError( __( 'Please install/activate the Polylang plugin to enable integration.', 'site-reviews' ) );
 			return false;
 		}
 		else if( !glsr( Polylang::class )->isSupported() ) {
-			glsr( Notice::class )->addError( __( 'Please update the Polylang plugin to v2.3.0 or greater to enable integration.', 'site-reviews' ));
+			glsr( Notice::class )->addError( __( 'Please update the Polylang plugin to v2.3.0 or greater to enable integration.', 'site-reviews' ) );
 			return false;
 		}
 		return true;