pryley / site-reviews

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

plugin/Handlers/EnqueuePublicAssets.php

Spacing +11 added lines, -11 removed lines

@@ -27,7 +27,7 @@ 
 	 */
 	public function enqueueAssets()
 	{
-		if( apply_filters( 'site-reviews/assets/css', true )) {
+		if( apply_filters( 'site-reviews/assets/css', true ) ) {
 			wp_enqueue_style(
 				Application::ID,
 				$this->getStylesheet(),
@@ -35,7 +35,7 @@ 
 				glsr()->version
 			);
 		}
-		if( apply_filters( 'site-reviews/assets/js', true )) {
+		if( apply_filters( 'site-reviews/assets/js', true ) ) {
 			$dependencies = apply_filters( 'site-reviews/assets/polyfill', true )
 				? [Application::ID.'/polyfill']
 				: [];
@@ -55,11 +55,11 @@ 
 	 */
 	public function enqueuePolyfillService()
 	{
-		if( !apply_filters( 'site-reviews/assets/polyfill', true ))return;
-		wp_enqueue_script( Application::ID.'/polyfill', add_query_arg([
+		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',
-		], 'https://cdn.polyfill.io/v2/polyfill.js' ));
+		], 'https://cdn.polyfill.io/v2/polyfill.js' ) );
 	}
 
 	/**
@@ -72,10 +72,10 @@ 
 		// nf-google-recaptcha
 		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([
+		wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg( [
 			'hl' => $language,
 			'render' => 'explicit',
-		], 'https://www.google.com/recaptcha/api.js' ));
+		], 'https://www.google.com/recaptcha/api.js' ) );
 	}
 
 	/**
@@ -84,8 +84,8 @@ 
 	public function inlineStyles()
 	{
 		$inlineStylesheetPath = glsr()->path( 'assets/styles/inline-styles.css' );
-		if( !apply_filters( 'site-reviews/assets/css', true ))return;
-		if( !file_exists( $inlineStylesheetPath )) {
+		if( !apply_filters( 'site-reviews/assets/css', true ) )return;
+		if( !file_exists( $inlineStylesheetPath ) ) {
 			glsr_log()->error( 'Inline stylesheet is missing: '.$inlineStylesheetPath );
 			return;
 		}
@@ -120,7 +120,7 @@ 
 	 */
 	protected function getFixedSelectorsForPagination()
 	{
-		$selectors = ['#wpadminbar','.site-navigation-fixed'];
+		$selectors = ['#wpadminbar', '.site-navigation-fixed'];
 		return apply_filters( 'site-reviews/localize/pagination/selectors', $selectors );
 	}
 
@@ -130,7 +130,7 @@ 
 	protected function getStylesheet()
 	{
 		$currentStyle = glsr( Style::class )->style;
-		return file_exists( glsr()->path( 'assets/styles/custom/'.$currentStyle.'.css' ))
+		return file_exists( glsr()->path( 'assets/styles/custom/'.$currentStyle.'.css' ) )
 			? glsr()->url( 'assets/styles/custom/'.$currentStyle.'.css' )
 			: glsr()->url( 'assets/styles/'.Application::ID.'.css' );
 	}

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/Handlers/CreateReview.php

Spacing +13 added lines, -13 removed lines

@@ -28,14 +28,14 @@ 
 		$postId = glsr( ReviewManager::class )->create( $command );
 		if( $postId < 1 ) {
 			glsr( Session::class )->set( $command->form_id.'errors', [] );
-			glsr( Session::class )->set( $command->form_id.'message', __( 'Your review could not be submitted and the error has been logged. Please notify the site admin.', 'site-reviews' ));
+			glsr( Session::class )->set( $command->form_id.'message', __( 'Your review could not be submitted and the error has been logged. Please notify the site admin.', 'site-reviews' ) );
 			return;
 		}
 		$this->sendNotification( $postId );
 		do_action( 'site-reviews/local/review/submitted', $postId, $command );
-		glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ));
+		glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ) );
 		if( $command->ajax_request )return;
-		if( empty( $command->referer )) {
+		if( empty($command->referer) ) {
 			glsr_log()->error( 'The form referer ($_SERVER[REQUEST_URI]) is empty.' );
 			$command->referer = home_url();
 		}
@@ -84,8 +84,8 @@ 
 		if( $this->command->author ) {
 			$fields[] = ['value' => trim( $this->command->author.$this->command->email.' - '.$this->command->ip_address )];
 		}
-		$fields[] = ['value' => sprintf( '<%s|%s>', $args['notification_link'], __( 'View Review', 'site-reviews' ))];
-		return json_encode([
+		$fields[] = ['value' => sprintf( '<%s|%s>', $args['notification_link'], __( 'View Review', 'site-reviews' ) )];
+		return json_encode( [
 			'icon_url' => glsr()->url( 'assets/img/icon.png' ),
 			'username' => glsr()->name,
 			'attachments' => [[
@@ -94,7 +94,7 @@ 
 				'fallback' => $this->createEmailNotification( $args )->read( 'plaintext' ),
 				'fields' => $fields,
 			]],
-		]);
+		] );
 	}
 
 	/**
@@ -104,12 +104,12 @@ 
 	protected function sendNotification( $postId )
 	{
 		$notificationType = glsr( OptionManager::class )->get( 'settings.general.notification' );
-		if( !in_array( $notificationType, ['default','custom','webhook'] ))return;
+		if( !in_array( $notificationType, ['default', 'custom', 'webhook'] ) )return;
 		$assignedToTitle = get_the_title( (int)$this->command->assigned_to );
 		$notificationSubject = _nx(
 			'New %s-star review',
 			'New %s-star review of: %s',
-			intval( empty( $assignedToTitle )),
+			intval( empty($assignedToTitle) ),
 			'The text is different depending on whether or not the review has been assigned to a post.',
 			'site-reviews'
 		);
@@ -118,7 +118,7 @@ 
 			sprintf( $notificationSubject, $this->command->rating, $assignedToTitle )
 		);
 		$args = [
-			'notification_link' => esc_url( admin_url( sprintf( 'post.php?post=%s&action=edit', $postId ))),
+			'notification_link' => esc_url( admin_url( sprintf( 'post.php?post=%s&action=edit', $postId ) ) ),
 			'notification_title' => $notificationTitle,
 			'notification_type' => $notificationType,
 		];
@@ -136,7 +136,7 @@ 
 		$args['recipient'] = $args['notification_type'] !== 'default'
 			? glsr( OptionManager::class )->get( 'settings.general.notification_email' )
 			: get_option( 'admin_email' );
-		if( empty( $args['recipient'] )) {
+		if( empty($args['recipient']) ) {
 			glsr_log()->error( 'Email notification was not sent: missing email, subject, or message.' );
 		}
 		else {
@@ -152,7 +152,7 @@ 
 	 */
 	protected function sendWebhookNotification( array $args )
 	{
-		if( !( $endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' )))return;
+		if( !($endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' )) )return;
 		$notification = $this->createWebhookNotification( $args );
 		$result = wp_remote_post( $endpoint, [
 			'blocking' => false,
@@ -163,8 +163,8 @@ 
 			'redirection' => 5,
 			'sslverify' => false,
 			'timeout' => 45,
-		]);
-		if( is_wp_error( $result )) {
+		] );
+		if( is_wp_error( $result ) ) {
 			glsr_log()->error( $result->get_error_message() );
 		}
 	}

Braces +9 added lines, -3 removed lines

@@ -34,7 +34,9 @@ 
 		$this->sendNotification( $postId );
 		do_action( 'site-reviews/local/review/submitted', $postId, $command );
 		glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ));
-		if( $command->ajax_request )return;
+		if( $command->ajax_request ) {
+			return;
+		}
 		if( empty( $command->referer )) {
 			glsr_log()->error( 'The form referer ($_SERVER[REQUEST_URI]) is empty.' );
 			$command->referer = home_url();
@@ -104,7 +106,9 @@ 
 	protected function sendNotification( $postId )
 	{
 		$notificationType = glsr( OptionManager::class )->get( 'settings.general.notification' );
-		if( !in_array( $notificationType, ['default','custom','webhook'] ))return;
+		if( !in_array( $notificationType, ['default','custom','webhook'] )) {
+			return;
+		}
 		$assignedToTitle = get_the_title( (int)$this->command->assigned_to );
 		$notificationSubject = _nx(
 			'New %s-star review',
@@ -152,7 +156,9 @@ 
 	 */
 	protected function sendWebhookNotification( array $args )
 	{
-		if( !( $endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' )))return;
+		if( !( $endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' ))) {
+			return;
+		}
 		$notification = $this->createWebhookNotification( $args );
 		$result = wp_remote_post( $endpoint, [
 			'blocking' => false,

plugin/Defaults/DefaultsAbstract.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 	 */
 	public function __call( $name, array $args = [] )
 	{
-		if( !method_exists( $this, $name ))return;
+		if( !method_exists( $this, $name ) )return;
 		$defaults = call_user_func_array( [$this, $name], $args );
 		$className = (new ReflectionClass( $this ))->getShortName();
 		$className = str_replace( 'Defaults', '', $className );

Braces +3 added lines, -1 removed lines

@@ -13,7 +13,9 @@
 	 */
 	public function __call( $name, array $args = [] )
 	{
-		if( !method_exists( $this, $name ))return;
+		if( !method_exists( $this, $name )) {
+			return;
+		}
 		$defaults = call_user_func_array( [$this, $name], $args );
 		$className = (new ReflectionClass( $this ))->getShortName();
 		$className = str_replace( 'Defaults', '', $className );

plugin/Controllers/AjaxController.php

Spacing +18 added lines, -18 removed lines

@@ -22,7 +22,7 @@ 
 	 */
 	public function routerChangeStatus( array $request )
 	{
-		wp_send_json_success( $this->execute( new ChangeStatus( $request )));
+		wp_send_json_success( $this->execute( new ChangeStatus( $request ) ) );
 	}
 
 	/**
@@ -31,10 +31,10 @@ 
 	public function routerClearConsole()
 	{
 		glsr( AdminController::class )->routerClearConsole();
-		wp_send_json_success([
+		wp_send_json_success( [
 			'console' => glsr( Console::class )->get(),
 			'notices' => glsr( Notice::class )->get(),
-		]);
+		] );
 	}
 
 	/**
@@ -43,9 +43,9 @@ 
 	public function routerCountReviews()
 	{
 		glsr( AdminController::class )->routerCountReviews();
-		wp_send_json_success([
+		wp_send_json_success( [
 			'notices' => glsr( Notice::class )->get(),
-		]);
+		] );
 	}
 
 	/**
@@ -55,9 +55,9 @@ 
 	{
 		$shortcode = $request['shortcode'];
 		$response = false;
-		if( array_key_exists( $shortcode, glsr()->mceShortcodes )) {
+		if( array_key_exists( $shortcode, glsr()->mceShortcodes ) ) {
 			$data = glsr()->mceShortcodes[$shortcode];
-			if( !empty( $data['errors'] )) {
+			if( !empty($data['errors']) ) {
 				$data['btn_okay'] = [esc_html__( 'Okay', 'site-reviews' )];
 			}
 			$response = [
@@ -77,10 +77,10 @@ 
 	public function routerFetchConsole()
 	{
 		glsr( AdminController::class )->routerFetchConsole();
-		wp_send_json_success([
+		wp_send_json_success( [
 			'console' => glsr( Console::class )->get(),
 			'notices' => glsr( Notice::class )->get(),
-		]);
+		] );
 	}
 
 	/**
@@ -89,10 +89,10 @@ 
 	public function routerSearchPosts( array $request )
 	{
 		$results = glsr( Database::class )->searchPosts( $request['search'] );
-		wp_send_json_success([
+		wp_send_json_success( [
 			'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>',
 			'items' => $results,
-		]);
+		] );
 	}
 
 	/**
@@ -100,7 +100,7 @@ 
 	 */
 	public function routerSearchTranslations( array $request )
 	{
-		if( empty( $request['exclude'] )) {
+		if( empty($request['exclude']) ) {
 			$request['exclude'] = [];
 		}
 		$results = glsr( Translator::class )
@@ -108,10 +108,10 @@ 
 			->exclude()
 			->exclude( $request['exclude'] )
 			->renderResults();
-		wp_send_json_success([
+		wp_send_json_success( [
 			'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>',
 			'items' => $results,
-		]);
+		] );
 	}
 
 	/**
@@ -125,7 +125,7 @@ 
 			'message' => glsr( Session::class )->get( $request['form_id'].'message', '', true ),
 			'recaptcha' => glsr( Session::class )->get( $request['form_id'].'recaptcha', false, true ),
 		];
-		if( !wp_validate_boolean( $data['errors'] )) {
+		if( !wp_validate_boolean( $data['errors'] ) ) {
 			glsr( Session::class )->clear();
 			wp_send_json_success( $data );
 		}
@@ -137,10 +137,10 @@ 
 	 */
 	public function routerTogglePinned( array $request )
 	{
-		$isPinned = $this->execute( new TogglePinned( $request ));
-		wp_send_json_success([
+		$isPinned = $this->execute( new TogglePinned( $request ) );
+		wp_send_json_success( [
 			'notices' => glsr( Notice::class )->get(),
 			'pinned' => $isPinned,
-		]);
+		] );
 	}
 }

plugin/Modules/Validator/ValidateReview.php

Spacing +22 added lines, -22 removed lines

@@ -61,7 +61,7 @@ 
 		$this->validateBlacklist();
 		$this->validateAkismet();
 		$this->validateRecaptcha();
-		if( !empty( $this->error )) {
+		if( !empty($this->error) ) {
 			$this->setSessionValues( 'message', $this->error );
 		}
 		return $this;
@@ -84,12 +84,12 @@ 
 	{
 		$rules = array_intersect_key(
 			apply_filters( 'site-reviews/validation/rules', static::VALIDATION_RULES ),
-			array_flip( $this->getOption( 'settings.submissions.required', [] ))
+			array_flip( $this->getOption( 'settings.submissions.required', [] ) )
 		);
-		$excluded = isset( $request['excluded'] )
+		$excluded = isset($request['excluded'])
 			? (array)json_decode( $request['excluded'] )
 			: [];
-		return array_diff_key( $rules, array_flip( $excluded ));
+		return array_diff_key( $rules, array_flip( $excluded ) );
 	}
 
 	/**
@@ -100,7 +100,7 @@ 
 		if( !glsr( OptionManager::class )->isRecaptchaEnabled() ) {
 			return true;
 		}
-		if( empty( $this->request['recaptcha-token'] )) {
+		if( empty($this->request['recaptcha-token']) ) {
 			return null; // @see $this->validateRecaptcha()
 		}
 		return $this->isRecaptchaValid( $this->request['recaptcha-token'] );
@@ -112,17 +112,17 @@ 
 	 */
 	protected function isRecaptchaValid( $recaptchaToken )
 	{
-		$endpoint = add_query_arg([
+		$endpoint = add_query_arg( [
 			'remoteip' => glsr( Helper::class )->getIpAddress(),
 			'response' => $recaptchaToken,
 			'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 false;
 		}
-		$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 );
 		}
 		foreach( $response->{'error-codes'} as $error ) {
@@ -138,7 +138,7 @@ 
 	{
 		$rules = $this->getValidationRules( $request );
 		$errors = glsr( Validator::class )->validate( $request, $rules );
-		if( empty( $errors )) {
+		if( empty($errors) ) {
 			return true;
 		}
 		$this->setSessionValues( 'errors', $errors );
@@ -155,7 +155,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 );
 		}
@@ -166,8 +166,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:' );
 		$this->error = __( 'Your review cannot be submitted at this time. Please try again later.', 'site-reviews' );
 	}
@@ -177,8 +177,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:' );
@@ -193,7 +193,7 @@ 
 	 */
 	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;
 		$this->setSessionValues( 'errors', [] );
@@ -208,8 +208,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' );
 	}
@@ -219,9 +219,9 @@ 
 	 */
 	protected function validateRecaptcha()
 	{
-		if( !empty( $this->error ))return;
+		if( !empty($this->error) )return;
 		$isValid = $this->isRecaptchaResponseValid();
-		if( is_null( $isValid )) {
+		if( is_null( $isValid ) ) {
 			$this->setSessionValues( 'recaptcha', 'unset' );
 			$this->recaptchaIsUnset = true;
 		}
@@ -237,11 +237,11 @@ 
 	 */
 	protected function validateRequest( array $request )
 	{
-		if( !$this->isRequestValid( $request )) {
+		if( !$this->isRequestValid( $request ) ) {
 			$this->error = __( 'Please fix the submission errors.', 'site-reviews' );
 			return $request;
 		}
-		if( empty( $request['title'] )) {
+		if( empty($request['title']) ) {
 			$request['title'] = __( 'No Title', 'site-reviews' );
 		}
 		return array_merge( glsr( ValidateReviewDefaults::class )->defaults(), $request );

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/Modules/Html/Partials/SiteReviewsForm.php

Spacing +23 added lines, -23 removed lines

@@ -62,7 +62,7 @@ 
 				'response' => $this->buildResponse(),
 				'submit_button' => $this->buildSubmitButton().$this->buildRecaptcha(),
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -71,12 +71,12 @@ 
 	protected function buildRecaptcha()
 	{
 		if( !glsr( OptionManager::class )->isRecaptchaEnabled() )return;
-		return glsr( Builder::class )->div([
+		return glsr( Builder::class )->div( [
 			'class' => 'glsr-recaptcha-holder',
 			'data-badge' => glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.position' ),
-			'data-sitekey' => sanitize_text_field( glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.key' )),
+			'data-sitekey' => sanitize_text_field( glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.key' ) ),
 			'data-size' => 'invisible',
-		]);
+		] );
 	}
 
 	/**
@@ -84,7 +84,7 @@ 
 	 */
 	protected function buildResponse()
 	{
-		$classes = !empty( $this->errors )
+		$classes = !empty($this->errors)
 			? glsr( StyleValidationDefaults::class )->defaults()['message_error_class']
 			: '';
 		return glsr( Template::class )->build( 'templates/form/response', [
@@ -92,8 +92,8 @@ 
 				'class' => $classes,
 				'message' => wpautop( $this->message ),
 			],
-			'has_errors' => !empty( $this->errors ),
-		]);
+			'has_errors' => !empty($this->errors),
+		] );
 	}
 
 	/**
@@ -105,7 +105,7 @@ 
 			'context' => [
 				'text' => __( 'Submit your review', 'site-reviews' ),
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -124,7 +124,7 @@ 
 		$fields = array_merge(
 			$this->getHiddenFields(),
 			[$this->getHoneypotField()],
-			$this->normalizeFields( glsr( Form::class )->getFields( 'submission-form' ))
+			$this->normalizeFields( glsr( Form::class )->getFields( 'submission-form' ) )
 		);
 		return $fields;
 	}
@@ -137,27 +137,27 @@ 
 		$fields = [[
 			'name' => 'action',
 			'value' => 'submit-review',
-		],[
+		], [
 			'name' => 'assign_to',
 			'value' => $this->args['assign_to'],
-		],[
+		], [
 			'name' => 'category',
 			'value' => $this->args['category'],
-		],[
+		], [
 			'name' => 'excluded',
 			'value' => $this->args['excluded'], // @todo should default to "[]"
-		],[
+		], [
 			'name' => 'form_id',
 			'value' => $this->args['id'],
-		],[
+		], [
 			'name' => 'nonce',
 			'value' => wp_create_nonce( 'submit-review' ),
-		],[
+		], [
 			'name' => 'referer',
-			'value' => wp_unslash( filter_input( INPUT_SERVER, 'REQUEST_URI' )),
+			'value' => wp_unslash( filter_input( INPUT_SERVER, 'REQUEST_URI' ) ),
 		]];
 		return array_map( function( $field ) {
-			return new Field( wp_parse_args( $field, ['type' => 'hidden'] ));
+			return new Field( wp_parse_args( $field, ['type' => 'hidden'] ) );
 		}, $fields );
 	}
 
@@ -166,10 +166,10 @@ 
 	 */
 	protected function getHoneypotField()
 	{
-		return new Field([
+		return new Field( [
 			'name' => 'gotcha',
 			'type' => 'honeypot',
-		]);
+		] );
 	}
 
 	/**
@@ -177,7 +177,7 @@ 
 	 */
 	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']];
 	}
 
@@ -186,7 +186,7 @@ 
 	 */
 	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;
 	}
 
@@ -209,8 +209,8 @@ 
 	 */
 	protected function normalizeFieldValue( Field &$field )
 	{
-		if( !array_key_exists( $field->field['path'], $this->values ))return;
-		if( in_array( $field->field['type'], ['radio', 'checkbox'] )) {
+		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']];
 		}
 		else {

Braces +12 added lines, -4 removed lines

@@ -70,7 +70,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' ),
@@ -177,7 +179,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']];
 	}
 
@@ -186,7 +190,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;
 	}
 
@@ -209,7 +215,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']];
 		}

plugin/Commands/CreateReview.php

Spacing +13 added lines, -13 removed lines

@@ -27,21 +27,21 @@ 
 	public function __construct( $input )
 	{
 		$this->request = $input;
-		$this->ajax_request = isset( $input['ajax_request'] );
+		$this->ajax_request = isset($input['ajax_request']);
 		$this->assigned_to = $this->getNumeric( 'assign_to' );
-		$this->author = sanitize_text_field( $this->get( 'name' ));
-		$this->avatar = get_avatar_url( $this->get( 'email' ));
-		$this->blacklisted = isset( $input['blacklisted'] );
-		$this->category = sanitize_key( $this->get( 'category' ));
-		$this->content = sanitize_textarea_field( $this->get( 'content' ));
+		$this->author = sanitize_text_field( $this->get( 'name' ) );
+		$this->avatar = get_avatar_url( $this->get( 'email' ) );
+		$this->blacklisted = isset($input['blacklisted']);
+		$this->category = sanitize_key( $this->get( 'category' ) );
+		$this->content = sanitize_textarea_field( $this->get( 'content' ) );
 		$this->custom = $this->getCustom();
-		$this->email = sanitize_email( $this->get( 'email' ));
-		$this->form_id = sanitize_key( $this->get( 'form_id' ));
+		$this->email = sanitize_email( $this->get( 'email' ) );
+		$this->form_id = sanitize_key( $this->get( 'form_id' ) );
 		$this->ip_address = $this->get( 'ip_address' );
-		$this->rating = intval( $this->get( 'rating' ));
+		$this->rating = intval( $this->get( 'rating' ) );
 		$this->referer = $this->get( 'referer' );
-		$this->terms = isset( $input['terms'] );
-		$this->title = sanitize_text_field( $this->get( 'title' ));
+		$this->terms = isset($input['terms']);
+		$this->title = sanitize_text_field( $this->get( 'title' ) );
 	}
 
 	/**
@@ -50,7 +50,7 @@ 
 	 */
 	protected function get( $key )
 	{
-		return isset( $this->request[$key] )
+		return isset($this->request[$key])
 			? (string)$this->request[$key]
 			: '';
 	}
@@ -67,7 +67,7 @@ 
 		];
 		$custom = $this->request;
 		foreach( $unset as $value ) {
-			unset( $custom[$value] );
+			unset($custom[$value]);
 		}
 		return $custom;
 	}