pryley / site-reviews

plugin/Controllers/ReviewController.php

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

plugin/Handlers/EnqueuePublicAssets.php

Braces +9 added lines, -3 removed lines

@@ -55,7 +55,9 @@ 
 	 */
 	public function enqueuePolyfillService()
 	{
-		if( !apply_filters( 'site-reviews/assets/polyfill', true ))return;
+		if( !apply_filters( 'site-reviews/assets/polyfill', true )) {
+			return;
+		}
 		wp_enqueue_script( Application::ID.'/polyfill', add_query_arg([
 			'features' => 'CustomEvent,Element.prototype.closest,Element.prototype.dataset,Event',
 			'flags' => 'gated',
@@ -70,7 +72,9 @@ 
 		// wpforms-recaptcha
 		// google-recaptcha
 		// nf-google-recaptcha
-		if( !glsr( OptionManager::class )->isRecaptchaEnabled() )return;
+		if( !glsr( OptionManager::class )->isRecaptchaEnabled() ) {
+			return;
+		}
 		$language = apply_filters( 'site-reviews/recaptcha/language', get_locale() );
 		wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg([
 			'hl' => $language,
@@ -84,7 +88,9 @@ 
 	public function inlineStyles()
 	{
 		$inlineStylesheetPath = glsr()->path( 'assets/styles/inline-styles.css' );
-		if( !apply_filters( 'site-reviews/assets/css', true ))return;
+		if( !apply_filters( 'site-reviews/assets/css', true )) {
+			return;
+		}
 		if( !file_exists( $inlineStylesheetPath )) {
 			glsr_log()->error( 'Inline stylesheet is missing: '.$inlineStylesheetPath );
 			return;

plugin/Modules/Validator/ValidateReview.php

Braces +27 added lines, -9 removed lines

@@ -166,8 +166,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:' );
 		$this->error = __( 'Your review cannot be submitted at this time. Please try again later.', 'site-reviews' );
 	}
@@ -177,8 +181,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:' );
@@ -193,9 +201,13 @@ 
 	 */
 	protected function validateCustom()
 	{
-		if( !empty( $this->error ))return;
+		if( !empty( $this->error )) {
+			return;
+		}
 		$validated = apply_filters( 'site-reviews/validate/review/submission', true, $this->request );
-		if( $validated === true )return;
+		if( $validated === true ) {
+			return;
+		}
 		$this->setSessionValues( 'errors', [] );
 		$this->setSessionValues( 'values', $this->request );
 		$this->error = is_string( $validated )
@@ -208,8 +220,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' );
 	}
@@ -219,7 +235,9 @@ 
 	 */
 	protected function validateRecaptcha()
 	{
-		if( !empty( $this->error ))return;
+		if( !empty( $this->error )) {
+			return;
+		}
 		$isValid = $this->isRecaptchaResponseValid();
 		if( is_null( $isValid )) {
 			$this->setSessionValues( 'recaptcha', 'unset' );

plugin/Router.php

Braces +15 added lines, -5 removed lines

@@ -17,7 +17,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 );
 	}
@@ -39,10 +41,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 );
 	}
 
@@ -51,7 +59,9 @@ 
 	 */
 	protected function checkAjaxNonce( array $request )
 	{
-		if( !is_user_logged_in() )return;
+		if( !is_user_logged_in() ) {
+			return;
+		}
 		if( !isset( $request['nonce'] )) {
 			$this->sendAjaxError( 'request is missing a nonce', $request );
 		}

plugin/Modules/Slack.php

Braces +3 added lines, -1 removed lines

@@ -35,7 +35,9 @@
 	 */
 	public function compose( Review $review, array $notification )
 	{
-		if( empty( $this->endpoint ))return;
+		if( empty( $this->endpoint )) {
+			return;
+		}
 		$args = shortcode_atts( glsr( SlackDefaults::class )->defaults(), $notification );
 		$this->review = $review;
 		$notification = [

plugin/Modules/Html/Partials/SiteReviewsForm.php

Braces +15 added lines, -5 removed lines

@@ -85,7 +85,9 @@ 
 	 */
 	protected function buildRecaptcha()
 	{
-		if( !glsr( OptionManager::class )->isRecaptchaEnabled() )return;
+		if( !glsr( OptionManager::class )->isRecaptchaEnabled() ) {
+			return;
+		}
 		return glsr( Builder::class )->div([
 			'class' => 'glsr-recaptcha-holder',
 			'data-badge' => glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.position' ),
@@ -161,7 +163,9 @@ 
 	 */
 	protected function getRegisterText()
 	{
-		if( !get_option( 'users_can_register' ) || glsr( OptionManager::class )->get( 'settings.general.require.login' ) != 'yes' )return;
+		if( !get_option( 'users_can_register' ) || glsr( OptionManager::class )->get( 'settings.general.require.login' ) != 'yes' ) {
+			return;
+		}
 		$registerLink = glsr( Builder::class )->a([
 			'href' => wp_registration_url(),
 			'text' => __( 'register', 'site-reviews' ),
@@ -217,7 +221,9 @@ 
 	 */
 	protected function normalizeFieldErrors( Field &$field )
 	{
-		if( !array_key_exists( $field->field['path'], $this->errors ))return;
+		if( !array_key_exists( $field->field['path'], $this->errors )) {
+			return;
+		}
 		$field->field['errors'] = $this->errors[$field->field['path']];
 	}
 
@@ -226,7 +232,9 @@ 
 	 */
 	protected function normalizeFieldRequired( Field &$field )
 	{
-		if( !in_array( $field->field['path'], $this->required ))return;
+		if( !in_array( $field->field['path'], $this->required )) {
+			return;
+		}
 		$field->field['required'] = true;
 	}
 
@@ -249,7 +257,9 @@ 
 	 */
 	protected function normalizeFieldValue( Field &$field )
 	{
-		if( !array_key_exists( $field->field['path'], $this->values ))return;
+		if( !array_key_exists( $field->field['path'], $this->values )) {
+			return;
+		}
 		if( in_array( $field->field['type'], ['radio', 'checkbox'] )) {
 			$field->field['checked'] = $field->field['value'] == $this->values[$field->field['path']];
 		}

site-reviews.php

Braces +3 added lines, -1 removed lines

@@ -21,7 +21,9 @@
 if( !class_exists( 'GL_Plugin_Check_v3' )) {
 	require_once __DIR__.'/activate.php';
 }
-if( !(new GL_Plugin_Check_v3( __FILE__ ))->canProceed() )return;
+if( !(new GL_Plugin_Check_v3( __FILE__ ))->canProceed() ) {
+	return;
+}
 require_once __DIR__.'/autoload.php';
 require_once __DIR__.'/compatibility.php';
 require_once __DIR__.'/deprecated.php';

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