pryley / site-reviews

plugin/Modules/Validator.php

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/Database/OptionManager.php

Spacing +9 added lines, -9 removed lines

@@ -28,7 +28,7 @@ 
 	 */
 	public function all()
 	{
-		if( empty( $this->options )) {
+		if( empty($this->options) ) {
 			$this->reset();
 		}
 		return $this->options;
@@ -45,10 +45,10 @@ 
 		$options = $this->all();
 		$pointer = &$options;
 		foreach( $keys as $key ) {
-			if( !isset( $pointer[$key] ) || !is_array( $pointer[$key] ))continue;
+			if( !isset($pointer[$key]) || !is_array( $pointer[$key] ) )continue;
 			$pointer = &$pointer[$key];
 		}
-		unset( $pointer[$last] );
+		unset($pointer[$last]);
 		return $this->set( $options );
 	}
 
@@ -80,8 +80,8 @@ 
 			glsr( DefaultsManager::class )->defaults()
 		);
 		array_walk( $options, function( &$value ) {
-			if( !is_string( $value ))return;
-			$value = wp_kses( $value, wp_kses_allowed_html( 'post' ));
+			if( !is_string( $value ) )return;
+			$value = wp_kses( $value, wp_kses_allowed_html( 'post' ) );
 		});
 		return glsr( Helper::class )->convertDotNotationArray( $options );
 	}
@@ -92,7 +92,7 @@ 
 	public function isRecaptchaEnabled()
 	{
 		$integration = $this->get( 'settings.submissions.recaptcha.integration' );
-		return $integration == 'all' || ( $integration == 'guest' && !is_user_logged_in() );
+		return $integration == 'all' || ($integration == 'guest' && !is_user_logged_in());
 	}
 
 	/**
@@ -101,7 +101,7 @@ 
 	public function reset()
 	{
 		$options = get_option( static::databaseKey(), [] );
-		if( !is_array( $options ) || empty( $options )) {
+		if( !is_array( $options ) || empty($options) ) {
 			delete_option( static::databaseKey() );
 			$options = wp_parse_args( glsr()->defaults, ['settings' => []] );
 		}
@@ -115,10 +115,10 @@ 
 	 */
 	public function set( $pathOrOptions, $value = '' )
 	{
-		if( is_string( $pathOrOptions )) {
+		if( is_string( $pathOrOptions ) ) {
 			$pathOrOptions = glsr( Helper::class )->setPathValue( $pathOrOptions, $value, $this->all() );
 		}
-		if( $result = update_option( static::databaseKey(), (array)$pathOrOptions )) {
+		if( $result = update_option( static::databaseKey(), (array)$pathOrOptions ) ) {
 			$this->reset();
 		}
 		return $result;

plugin/Modules/Html/Builder.php

Spacing +36 added lines, -36 removed lines

@@ -66,7 +66,7 @@ 
 	{
 		$instance = new static;
 		$instance->setTagFromMethod( $method );
-		call_user_func_array( [$instance, 'normalize'], $args += ['',''] );
+		call_user_func_array( [$instance, 'normalize'], $args += ['', ''] );
 		$tags = array_merge( static::TAGS_FORM, static::TAGS_SINGLE, static::TAGS_STRUCTURE, static::TAGS_TEXT );
 		do_action_ref_array( 'site-reviews/builder', [&$instance] );
 		$generatedTag = in_array( $instance->tag, $tags )
@@ -90,8 +90,8 @@ 
 			'render' => 'is_bool',
 			'tag' => 'is_string',
 		];
-		if( !isset( $properties[$property] )
-			|| empty( array_filter( [$value], $properties[$property] ))
+		if( !isset($properties[$property])
+			|| empty(array_filter( [$value], $properties[$property] ))
 		)return;
 		$this->$property = $value;
 	}
@@ -101,7 +101,7 @@ 
 	 */
 	public function getClosingTag()
 	{
-		if( empty( $this->tag ))return;
+		if( empty($this->tag) )return;
 		return '</'.$this->tag.'>';
 	}
 
@@ -110,8 +110,8 @@ 
 	 */
 	public function getOpeningTag()
 	{
-		if( empty( $this->tag ))return;
-		$attributes = glsr( Attributes::class )->{$this->tag}( $this->args )->toString();
+		if( empty($this->tag) )return;
+		$attributes = glsr( Attributes::class )->{$this->tag}($this->args)->toString();
 		return '<'.trim( $this->tag.' '.$attributes ).'>';
 	}
 
@@ -120,8 +120,8 @@ 
 	 */
 	public function raw( array $field )
 	{
-		unset( $field['label'] );
-		return $this->{$field['type']}( $field );
+		unset($field['label']);
+		return $this->{$field['type']}($field);
 	}
 
 	/**
@@ -130,7 +130,7 @@ 
 	protected function buildCustomField()
 	{
 		$className = $this->getCustomFieldClassName();
-		if( class_exists( $className )) {
+		if( class_exists( $className ) ) {
 			return (new $className( $this ))->build();
 		}
 		glsr_log()->error( 'Field missing: '.$className );
@@ -141,7 +141,7 @@ 
 	 */
 	protected function buildDefaultTag( $text = '' )
 	{
-		if( empty( $text )) {
+		if( empty($text) ) {
 			$text = $this->args['text'];
 		}
 		return $this->getOpeningTag().$text.$this->getClosingTag();
@@ -152,7 +152,7 @@ 
 	 */
 	protected function buildFieldDescription()
 	{
-		if( empty( $this->args['description'] ))return;
+		if( empty($this->args['description']) )return;
 		if( $this->args['is_widget'] ) {
 			return $this->small( $this->args['description'] );
 		}
@@ -164,13 +164,13 @@ 
 	 */
 	protected function buildFormInput()
 	{
-		if( !in_array( $this->args['type'], ['checkbox', 'radio'] )) {
-			if( isset( $this->args['multiple'] )) {
-				$this->args['name'].= '[]';
+		if( !in_array( $this->args['type'], ['checkbox', 'radio'] ) ) {
+			if( isset($this->args['multiple']) ) {
+				$this->args['name'] .= '[]';
 			}
 			return $this->buildFormLabel().$this->getOpeningTag();
 		}
-		return empty( $this->args['options'] )
+		return empty($this->args['options'])
 			? $this->buildFormInputChoice()
 			: $this->buildFormInputMultiChoice();
 	}
@@ -180,19 +180,19 @@ 
 	 */
 	protected function buildFormInputChoice()
 	{
-		if( !empty( $this->args['text'] )) {
+		if( !empty($this->args['text']) ) {
 			$this->args['label'] = $this->args['text'];
 		}
 		if( !$this->args['is_public'] ) {
-			return $this->buildFormLabel([
+			return $this->buildFormLabel( [
 				'class' => 'glsr-'.$this->args['type'].'-label',
 				'text' => $this->getOpeningTag().' '.$this->args['label'].'<span></span>',
-			]);
+			] );
 		}
-		return $this->getOpeningTag().$this->buildFormLabel([
+		return $this->getOpeningTag().$this->buildFormLabel( [
 			'class' => 'glsr-'.$this->args['type'].'-label',
 			'text' => $this->args['label'].'<span></span>',
-		]);
+		] );
 	}
 
 	/**
@@ -201,22 +201,22 @@ 
 	protected function buildFormInputMultiChoice()
 	{
 		if( $this->args['type'] == 'checkbox' ) {
-			$this->args['name'].= '[]';
+			$this->args['name'] .= '[]';
 		}
 		$index = 0;
-		$options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) use( &$index ) {
+		$options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) use(&$index) {
 			return $carry.$this->li( $this->{$this->args['type']}([
 				'checked' => in_array( $key, (array)$this->args['value'] ),
 				'id' => $this->args['id'].'-'.$index++,
 				'name' => $this->args['name'],
 				'text' => $this->args['options'][$key],
 				'value' => $key,
-			]));
+			]) );
 		});
 		return $this->ul( $options, [
 			'class' => $this->args['class'],
 			'id' => $this->args['id'],
-		]);
+		] );
 	}
 
 	/**
@@ -224,13 +224,13 @@ 
 	 */
 	protected function buildFormLabel( array $customArgs = [] )
 	{
-		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return;
+		if( empty($this->args['label']) || $this->args['type'] == 'hidden' )return;
 		return $this->label( wp_parse_args( $customArgs, [
 			'for' => $this->args['id'],
 			'is_public' => $this->args['is_public'],
 			'text' => $this->args['label'],
 			'type' => $this->args['type'],
-		]));
+		] ) );
 	}
 
 	/**
@@ -247,11 +247,11 @@ 
 	protected function buildFormSelectOptions()
 	{
 		return array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) {
-			return $carry.$this->option([
+			return $carry.$this->option( [
 				'selected' => $this->args['value'] == $key,
 				'text' => $this->args['options'][$key],
 				'value' => $key,
-			]);
+			] );
 		});
 	}
 
@@ -269,10 +269,10 @@ 
 	protected function buildTag()
 	{
 		$this->mergeArgsWithRequiredDefaults();
-		if( in_array( $this->tag, static::TAGS_SINGLE )) {
+		if( in_array( $this->tag, static::TAGS_SINGLE ) ) {
 			return $this->getOpeningTag();
 		}
-		if( !in_array( $this->tag, static::TAGS_FORM )) {
+		if( !in_array( $this->tag, static::TAGS_FORM ) ) {
 			return $this->buildDefaultTag();
 		}
 		return call_user_func( [$this, 'buildForm'.ucfirst( $this->tag )] ).$this->buildFieldDescription();
@@ -292,7 +292,7 @@ 
 	protected function mergeArgsWithRequiredDefaults()
 	{
 		$className = $this->getCustomFieldClassName();
-		if( class_exists( $className )) {
+		if( class_exists( $className ) ) {
 			$this->args = array_merge(
 				wp_parse_args( $this->args, $className::defaults() ),
 				$className::required()
@@ -307,16 +307,16 @@ 
 	 */
 	protected function normalize( ...$params )
 	{
-		if( is_string( $params[0] ) || is_numeric( $params[0] )) {
+		if( is_string( $params[0] ) || is_numeric( $params[0] ) ) {
 			$this->setNameOrTextAttributeForTag( $params[0] );
 		}
-		if( is_array( $params[0] )) {
+		if( is_array( $params[0] ) ) {
 			$this->args += $params[0];
 		}
-		else if( is_array( $params[1] )) {
+		else if( is_array( $params[1] ) ) {
 			$this->args += $params[1];
 		}
-		if( !isset( $this->args['is_public'] )) {
+		if( !isset($this->args['is_public']) ) {
 			$this->args['is_public'] = false;
 		}
 	}
@@ -340,7 +340,7 @@ 
 	protected function setTagFromMethod( $method )
 	{
 		$this->tag = strtolower( $method );
-		if( in_array( $this->tag, static::INPUT_TYPES )) {
+		if( in_array( $this->tag, static::INPUT_TYPES ) ) {
 			$this->args['type'] = $this->tag;
 			$this->tag = 'input';
 		}

plugin/Modules/Html/Settings.php

Spacing +18 added lines, -18 removed lines

@@ -29,7 +29,7 @@ 
 	 */
 	protected function getFieldDefault( array $field )
 	{
-		return isset( $field['default'] )
+		return isset($field['default'])
 			? $field['default']
 			: '';
 	}
@@ -40,7 +40,7 @@ 
 	protected function getSettingFields( $path )
 	{
 		$settings = glsr( DefaultsManager::class )->settings();
-		return array_filter( $settings, function( $key ) use( $path ) {
+		return array_filter( $settings, function( $key ) use($path) {
 			return glsr( Helper::class )->startsWith( $path, $key );
 		}, ARRAY_FILTER_USE_KEY );
 	}
@@ -55,8 +55,8 @@ 
 			$field = wp_parse_args( $field, [
 				'is_setting' => true,
 				'name' => $name,
-			]);
-			$rows.= new Field( $this->normalize( $field ));
+			] );
+			$rows .= new Field( $this->normalize( $field ) );
 		}
 		return $rows;
 	}
@@ -67,7 +67,7 @@ 
 	 */
 	protected function getTemplateData( $id )
 	{
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
 		return [
 			'context' => [
 				'rows' => $this->getSettingRows( $fields ),
@@ -81,12 +81,12 @@ 
 	 */
 	protected function getTemplateDataForAddons( $id )
 	{
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
 		$settings = glsr( Helper::class )->convertDotNotationArray( $fields );
 		$settingKeys = array_keys( $settings['settings']['addons'] );
 		$results = [];
 		foreach( $settingKeys as $key ) {
-			$addonFields = array_filter( $fields, function( $path ) use( $key ) {
+			$addonFields = array_filter( $fields, function( $path ) use($key) {
 				return glsr( Helper::class )->startsWith( 'settings.addons.'.$key, $path );
 			}, ARRAY_FILTER_USE_KEY );
 			$results[$key] = $this->getSettingRows( $addonFields );
@@ -102,7 +102,7 @@ 
 	protected function getTemplateDataForTranslations()
 	{
 		$translations = glsr( Translator::class )->renderAll();
-		$class = empty( $translations )
+		$class = empty($translations)
 			? 'glsr-hidden'
 			: '';
 		return [
@@ -125,9 +125,9 @@ 
 			$path,
 			glsr( Helper::class )->getPathValue( $path, glsr()->defaults )
 		);
-		if( is_array( $expectedValue )) {
+		if( is_array( $expectedValue ) ) {
 			return is_array( $optionValue )
-				? count( array_intersect( $optionValue, $expectedValue )) === 0
+				? count( array_intersect( $optionValue, $expectedValue ) ) === 0
 				: !in_array( $optionValue, $expectedValue );
 		}
 		return $optionValue != $expectedValue;
@@ -149,17 +149,17 @@ 
 	 */
 	protected function normalizeDependsOn( array $field )
 	{
-		if( !empty( $field['depends_on'] ) && is_array( $field['depends_on'] )) {
+		if( !empty($field['depends_on']) && is_array( $field['depends_on'] ) ) {
 			$path = key( $field['depends_on'] );
 			$expectedValue = $field['depends_on'][$path];
 			$fieldName = glsr( Helper::class )->convertPathToName( $path, OptionManager::databaseKey() );
-			if( is_array( $expectedValue )) {
-				$fieldName.= '[]';
+			if( is_array( $expectedValue ) ) {
+				$fieldName .= '[]';
 			}
-			$field['data-depends'] = json_encode([
+			$field['data-depends'] = json_encode( [
 				'name' => $fieldName,
 				'value' => $expectedValue,
-			], JSON_HEX_APOS|JSON_HEX_QUOT );
+			], JSON_HEX_APOS | JSON_HEX_QUOT );
 			$field['is_hidden'] = $this->isFieldHidden( $path, $expectedValue );
 		}
 		return $field;
@@ -170,9 +170,9 @@ 
 	 */
 	protected function normalizeLabelAndLegend( array $field )
 	{
-		if( !empty( $field['label'] )) {
+		if( !empty($field['label']) ) {
 			$field['legend'] = $field['label'];
-			unset( $field['label'] );
+			unset($field['label']);
 		}
 		else {
 			$field['is_valid'] = false;
@@ -186,7 +186,7 @@ 
 	 */
 	protected function normalizeValue( array $field )
 	{
-		if( !isset( $field['value'] )) {
+		if( !isset($field['value']) ) {
 			$field['value'] = glsr( OptionManager::class )->get(
 				$field['name'],
 				$this->getFieldDefault( $field )

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 ))return;
+		if( empty($this->endpoint) )return;
 		$args = shortcode_atts( glsr( SlackDefaults::class )->defaults(), $notification );
 		$this->review = $review;
 		$notification = [
@@ -58,7 +58,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, [
@@ -70,7 +70,7 @@ 
 			'redirection' => 5,
 			'sslverify' => false,
 			'timeout' => 45,
-		]);
+		] );
 	}
 
 	/**
@@ -90,11 +90,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] ) )];
 	}
 
 	/**
@@ -102,7 +102,7 @@ 
 	 */
 	protected function buildContentField()
 	{
-		return !empty( $this->review->content )
+		return !empty($this->review->content)
 			? ['value' => $this->review->content]
 			: [];
 	}
@@ -127,7 +127,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];
@@ -138,7 +138,7 @@ 
 	 */
 	protected function buildTitleField()
 	{
-		return !empty( $this->review->title )
+		return !empty($this->review->title)
 			? ['title' => $this->review->title]
 			: [];
 	}

plugin/Router.php

Spacing +13 added lines, -13 removed lines

@@ -17,7 +17,7 @@ 
 	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 );
 	}
@@ -41,8 +41,8 @@ 
 	{
 		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 );
 	}
 
@@ -52,10 +52,10 @@ 
 	protected function checkAjaxNonce( array $request )
 	{
 		if( !is_user_logged_in() )return;
-		if( !isset( $request['nonce'] )) {
+		if( !isset($request['nonce']) ) {
 			$this->sendAjaxError( 'request is missing a nonce', $request );
 		}
-		if( !wp_verify_nonce( $request['nonce'], $request['action'] )) {
+		if( !wp_verify_nonce( $request['nonce'], $request['action'] ) ) {
 			$this->sendAjaxError( 'request failed the nonce check', $request, 403 );
 		}
 	}
@@ -65,10 +65,10 @@ 
 	 */
 	protected function checkAjaxRequest( array $request )
 	{
-		if( !isset( $request['action'] )) {
+		if( !isset($request['action']) ) {
 			$this->sendAjaxError( 'request must include an action', $request );
 		}
-		if( empty( $request['ajax_request'] )) {
+		if( empty($request['ajax_request']) ) {
 			$this->sendAjaxError( 'request is invalid', $request );
 		}
 	}
@@ -95,7 +95,7 @@ 
 	 */
 	protected function isValidPostRequest( array $request = [] )
 	{
-		return !empty( $request['action'] ) && empty( $request['ajax_request'] );
+		return !empty($request['action']) && empty($request['ajax_request']);
 	}
 
 	/**
@@ -103,7 +103,7 @@ 
 	 */
 	protected function isValidPublicNonce( array $request )
 	{
-		if( is_user_logged_in() && !wp_verify_nonce( $request['nonce'], $request['action'] )) {
+		if( is_user_logged_in() && !wp_verify_nonce( $request['nonce'], $request['action'] ) ) {
 			glsr_log()->error( 'nonce check failed for public request' )->debug( $request );
 			return false;
 		}
@@ -118,11 +118,11 @@ 
 	protected function routeRequest( $type, $action, array $request = [] )
 	{
 		$actionHook = 'site-reviews/route/'.$type.'/request';
-		$controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' ));
+		$controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' ) );
 		$method = glsr( Helper::class )->buildMethodName( $action, 'router' );
 		$request = apply_filters( 'site-reviews/route/request', $request, $action, $type );
 		do_action( $actionHook, $action, $request );
-		if( is_callable( [$controller, $method] )) {
+		if( is_callable( [$controller, $method] ) ) {
 			call_user_func( [$controller, $method], $request );
 			return;
 		}
@@ -141,10 +141,10 @@ 
 		glsr_log()->error( $error )->debug( $request );
 		glsr_log( $_POST );
 		glsr( Notice::class )->addError( __( 'There was an error (try refreshing the page).', 'site-reviews' ).' <code>'.$error.'</code>' );
-		wp_send_json_error([
+		wp_send_json_error( [
 			'message' => __( 'The form could not be submitted. Please notify the site administrator.', 'site-reviews' ),
 			'notices' => glsr( Notice::class )->get(),
 			'error' => $error,
-		]);
+		] );
 	}
 }

plugin/Database/ReviewManager.php

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/Shortcodes/SiteReviewsButton.php

Spacing +23 added lines, -23 removed lines

@@ -13,15 +13,15 @@ 
 	public function fields()
 	{
 		return [[
-			'html' => sprintf( '<p class="strong">%s</p>', esc_html__( 'All settings are optional.', 'site-reviews' )),
+			'html' => sprintf( '<p class="strong">%s</p>', esc_html__( 'All settings are optional.', 'site-reviews' ) ),
 			'minWidth' => 320,
 			'type' => 'container',
-		],[
+		], [
 			'label' => esc_html__( 'Title', 'site-reviews' ),
 			'name' => 'title',
 			'tooltip' => esc_attr__( 'Enter a custom shortcode heading.', 'site-reviews' ),
 			'type' => 'textbox',
-		],[
+		], [
 			'label' => esc_html__( 'Count', 'site-reviews' ),
 			'maxLength' => 5,
 			'name' => 'count',
@@ -29,20 +29,20 @@ 
 			'text' => '10',
 			'tooltip' => esc_attr__( 'How many reviews would you like to display (default: 10)?', 'site-reviews' ),
 			'type' => 'textbox',
-		],[
+		], [
 			'label' => esc_html__( 'Rating', 'site-reviews' ),
 			'name' => 'rating',
 			'options' => [
-				'5' => esc_html( sprintf( _n( '%s star', '%s stars', 5, 'site-reviews' ), 5 )),
-				'4' => esc_html( sprintf( _n( '%s star', '%s stars', 4, 'site-reviews' ), 4 )),
-				'3' => esc_html( sprintf( _n( '%s star', '%s stars', 3, 'site-reviews' ), 3 )),
-				'2' => esc_html( sprintf( _n( '%s star', '%s stars', 2, 'site-reviews' ), 2 )),
-				'1' => esc_html( sprintf( _n( '%s star', '%s stars', 1, 'site-reviews' ), 1 )),
-				'0' => esc_html( __( 'Unrated', 'site-reviews' )),
+				'5' => esc_html( sprintf( _n( '%s star', '%s stars', 5, 'site-reviews' ), 5 ) ),
+				'4' => esc_html( sprintf( _n( '%s star', '%s stars', 4, 'site-reviews' ), 4 ) ),
+				'3' => esc_html( sprintf( _n( '%s star', '%s stars', 3, 'site-reviews' ), 3 ) ),
+				'2' => esc_html( sprintf( _n( '%s star', '%s stars', 2, 'site-reviews' ), 2 ) ),
+				'1' => esc_html( sprintf( _n( '%s star', '%s stars', 1, 'site-reviews' ), 1 ) ),
+				'0' => esc_html( __( 'Unrated', 'site-reviews' ) ),
 			],
 			'tooltip' => esc_attr__( 'What is the minimum rating to display (default: 1 star)?', 'site-reviews' ),
 			'type' => 'listbox',
-		],[
+		], [
 			'label' => esc_html__( 'Pagination', 'site-reviews' ),
 			'name' => 'pagination',
 			'options' => [
@@ -60,7 +60,7 @@ 
 			'name' => 'assigned_to',
 			'tooltip' => esc_attr__( 'Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page.', 'site-reviews' ),
 			'type' => 'textbox',
-		],[
+		], [
 			'label' => esc_html__( 'Schema', 'site-reviews' ),
 			'name' => 'schema',
 			'options' => [
@@ -69,49 +69,49 @@ 
 			],
 			'tooltip' => esc_attr__( 'Rich snippets are disabled by default.', 'site-reviews' ),
 			'type' => 'listbox',
-		],[
+		], [
 			'label' => esc_html__( 'Classes', 'site-reviews' ),
 			'name' => 'class',
 			'tooltip' => esc_attr__( 'Add custom CSS classes to the shortcode.', 'site-reviews' ),
 			'type' => 'textbox',
-		],[
+		], [
 			'columns' => 2,
 			'items' => [[
 				'name' => 'hide_assigned_to',
 				'text' => esc_html__( 'Assigned To', 'site-reviews' ),
 				'tooltip' => esc_attr__( 'Hide the assigned to link?', 'site-reviews' ),
 				'type' => 'checkbox',
-			],[
+			], [
 				'name' => 'hide_author',
 				'text' => esc_html__( 'Author', 'site-reviews' ),
 				'tooltip' => esc_attr__( 'Hide the review author?', 'site-reviews' ),
 				'type' => 'checkbox',
-			],[
+			], [
 				'name' => 'hide_avatar',
 				'text' => esc_html__( 'Avatar', 'site-reviews' ),
 				'tooltip' => esc_attr__( 'Hide the reviewer avatar if shown?', 'site-reviews' ),
 				'type' => 'checkbox',
-			],[
+			], [
 				'name' => 'hide_content',
 				'text' => esc_html__( 'Content', 'site-reviews' ),
 				'tooltip' => esc_attr__( 'Hide the review content?', 'site-reviews' ),
 				'type' => 'checkbox',
-			],[
+			], [
 				'name' => 'hide_date',
 				'text' => esc_html__( 'Date', 'site-reviews' ),
 				'tooltip' => esc_attr__( 'Hide the review date?', 'site-reviews' ),
 				'type' => 'checkbox',
-			],[
+			], [
 				'name' => 'hide_rating',
 				'text' => esc_html__( 'Rating', 'site-reviews' ),
 				'tooltip' => esc_attr__( 'Hide the review rating?', 'site-reviews' ),
 				'type' => 'checkbox',
-			],[
+			], [
 				'name' => 'hide_response',
 				'text' => esc_html__( 'Response', 'site-reviews' ),
 				'tooltip' => esc_attr__( 'Hide the review response?', 'site-reviews' ),
 				'type' => 'checkbox',
-			],[
+			], [
 				'name' => 'hide_title',
 				'text' => esc_html__( 'Title', 'site-reviews' ),
 				'tooltip' => esc_attr__( 'Hide the review title?', 'site-reviews' ),
@@ -121,7 +121,7 @@ 
 			'label' => esc_html__( 'Hide', 'site-reviews' ),
 			'spacing' => 5,
 			'type' => 'container',
-		],[
+		], [
 			'hidden' => true,
 			'name' => 'id',
 			'type' => 'textbox',
@@ -134,7 +134,7 @@ 
 	public function getTerms()
 	{
 		$terms = glsr( Database::class )->getTerms();
-		if( empty( $terms )) {
+		if( empty($terms) ) {
 			return [];
 		}
 		return [

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