pryley / site-reviews

plugin/Modules/Blacklist.php

Spacing +9 added lines, -9 removed lines

@@ -11,14 +11,14 @@ 
 	 */
 	public function isBlacklisted( array $review )
 	{
-		$target = implode( "\n", array_filter([
+		$target = implode( "\n", array_filter( [
 			$review['name'],
 			$review['content'],
 			$review['email'],
 			$review['ip_address'],
 			$review['title'],
-		]));
-		return (bool) apply_filters( 'site-reviews/blacklist/is-blacklisted',
+		] ) );
+		return (bool)apply_filters( 'site-reviews/blacklist/is-blacklisted',
 			$this->check( $target ),
 			$review
 		);
@@ -30,16 +30,16 @@ 
 	 */
 	protected function check( $target )
 	{
-		$blacklist = trim( glsr( OptionManager::class )->get( 'settings.submissions.blacklist.entries' ));
-		if( empty( $blacklist )) {
+		$blacklist = trim( glsr( OptionManager::class )->get( 'settings.submissions.blacklist.entries' ) );
+		if( empty($blacklist) ) {
 			return false;
 		}
 		$lines = explode( "\n", $blacklist );
-		foreach( (array) $lines as $line ) {
+		foreach( (array)$lines as $line ) {
 			$line = trim( $line );
-			if( empty( $line ) || 256 < strlen( $line ))continue;
-			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ));
-			if( preg_match( $pattern, $target )) {
+			if( empty($line) || 256 < strlen( $line ) )continue;
+			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ) );
+			if( preg_match( $pattern, $target ) ) {
 				return true;
 			}
 		}

plugin/Modules/Html/Partials/Pagination.php

Spacing +10 added lines, -10 removed lines

@@ -22,11 +22,11 @@ 
 		$this->args = $this->normalize( $args );
 		if( $this->args['total'] < 2 )return;
 		$links = $this->buildLinksForDeprecatedThemes();
-		if( empty( $links )) {
+		if( empty($links) ) {
 			$links = $this->buildLinks();
 		}
 		$links = apply_filters( 'site-reviews/reviews/navigation_links', $links, $this->args );
-		if( empty( $links ))return;
+		if( empty($links) )return;
 		return $this->buildTemplate( $links );
 	}
 
@@ -45,7 +45,7 @@ 
 			'total' => $this->args['total'],
 		];
 		if( is_front_page() ) {
-			unset( $paginateArgs['format'] );
+			unset($paginateArgs['format']);
 		}
 		return paginate_links( $paginateArgs );
 	}
@@ -56,17 +56,17 @@ 
 	protected function buildLinksForDeprecatedThemes()
 	{
 		$theme = wp_get_theme()->get( 'TextDomain' );
-		if( !in_array( $theme, ['twentyten','twentyeleven','twentytwelve','twentythirteen'] ))return;
+		if( !in_array( $theme, ['twentyten', 'twentyeleven', 'twentytwelve', 'twentythirteen'] ) )return;
 		$links = '';
 		if( $this->args['paged'] > 1 ) {
-			$links.= sprintf( '<div class="nav-previous"><a href="%s"><span class="meta-nav">&larr;</span> %s</a></div>',
+			$links .= sprintf( '<div class="nav-previous"><a href="%s"><span class="meta-nav">&larr;</span> %s</a></div>',
 				$this->buildUrlForDeprecatedThemes(-1),
 				__( 'Previous', 'site-reviews' )
 			);
 		}
 		if( $this->args['paged'] < $this->args['total'] ) {
-			$links.= sprintf( '<div class="nav-next"><a href="%s">%s <span class="meta-nav">&rarr;</span></a></div>',
-				$this->buildUrlForDeprecatedThemes(1),
+			$links .= sprintf( '<div class="nav-next"><a href="%s">%s <span class="meta-nav">&rarr;</span></a></div>',
+				$this->buildUrlForDeprecatedThemes( 1 ),
 				__( 'Next', 'site-reviews' )
 			);
 		}
@@ -84,7 +84,7 @@ 
 		$screenReaderTemplate = '<h2 class="screen-reader-text">%2$s</h2>';
 		$screenReaderText = __( 'Site Reviews navigation', 'site-reviews' );
 		$innerTemplate = $screenReaderTemplate.'<div class="nav-links">%3$s</div>';
-		if( in_array( $theme, ['twentyten', 'twentyeleven', 'twentytwelve'] )) {
+		if( in_array( $theme, ['twentyten', 'twentyeleven', 'twentytwelve'] ) ) {
 			$innerTemplate = '%3$s';
 		}
 		else if( $theme == 'twentyfourteen' ) {
@@ -96,7 +96,7 @@ 
 		$template = sprintf( $template, $class, $screenReaderText, $links );
 		return glsr( Builder::class )->div( $template.'<div class="glsr-loader"></div>', [
 			'class' => 'glsr-navigation',
-		])
+		] )
 	}
 
 	/**
@@ -119,6 +119,6 @@ 
 		return wp_parse_args( $args, [
 			'paged' => glsr( QueryBuilder::class )->getPaged(),
 			'total' => 1,
-		]);
+		] );
 	}
 }

plugin/Modules/Html/Partials/FormResults.php

Spacing +3 added lines, -3 removed lines

@@ -19,10 +19,10 @@ 
 	public function build( array $args = [] )
 	{
 		$this->errors = $args['errors'];
-		if( empty( $args['message'] ))return;
+		if( empty($args['message']) )return;
 		return glsr( Builder::class )->div( wpautop( $args['message'] ), [
 			'class' => $this->getClass(),
-		]);
+		] );
 	}
 
 	/**
@@ -30,7 +30,7 @@ 
 	 */
 	protected function getClass()
 	{
-		$errorClass = !empty( $this->errors )
+		$errorClass = !empty($this->errors)
 			? 'glsr-has-errors'
 			: '';
 		return trim( 'glsr-form-messages '.$errorClass );

Braces +3 added lines, -1 removed lines

@@ -19,7 +19,9 @@
 	public function build( array $args = [] )
 	{
 		$this->errors = $args['errors'];
-		if( empty( $args['message'] ))return;
+		if( empty( $args['message'] )) {
+			return;
+		}
 		return glsr( Builder::class )->div( wpautop( $args['message'] ), [
 			'class' => $this->getClass(),
 		]);

plugin/Modules/Html/Partials/SiteReviewsForm.php

Spacing +18 added lines, -18 removed lines

@@ -57,7 +57,7 @@ 
 				'submit_button' => $this->buildSubmitButton(),
 			],
 			'fields' => $this->getFields(),
-		]);
+		] );
 	}
 
 	/**
@@ -68,7 +68,7 @@ 
 		return glsr( Partial::class )->build( 'form-results', [
 			'errors' => $this->errors,
 			'message' => $this->message,
-		]);
+		] );
 	}
 
 	/**
@@ -78,7 +78,7 @@ 
 	{
 		return glsr( Builder::class )->button( '<span></span>'.__( 'Submit your review', 'site-reviews' ), [
 			'type' => 'submit',
-		]);
+		] );
 	}
 
 	/**
@@ -98,7 +98,7 @@ 
 		$fields = array_merge(
 			$this->getHiddenFields(),
 			[$this->getHoneypotField()],
-			$this->normalizeFields( glsr( Form::class )->getFields( 'submission-form' ))
+			$this->normalizeFields( glsr( Form::class )->getFields( 'submission-form' ) )
 		);
 		// glsr_debug( $fields );
 		return $fields;
@@ -112,27 +112,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' => '_wp_http_referer',
-			'value' => wp_unslash( filter_input( INPUT_SERVER, 'REQUEST_URI' )), // @todo this doesn't work, maybe need to get this on submit
-		],[
+			'value' => wp_unslash( filter_input( INPUT_SERVER, 'REQUEST_URI' ) ), // @todo this doesn't work, maybe need to get this on submit
+		], [
 			'name' => '_wpnonce',
 			'value' => wp_create_nonce( 'submit-review' ),
 		]];
 		return array_map( function( $field ) {
-			return new Field( wp_parse_args( $field, ['type' => 'hidden'] ));
+			return new Field( wp_parse_args( $field, ['type' => 'hidden'] ) );
 		}, $fields );
 	}
 
@@ -141,10 +141,10 @@ 
 	 */
 	protected function getHoneypotField()
 	{
-		return new Field([
+		return new Field( [
 			'name' => 'gotcha',
 			'type' => 'honeypot',
-		]);
+		] );
 	}
 
 	/**
@@ -152,7 +152,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']];
 	}
 
@@ -161,7 +161,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;
 	}
 
@@ -183,8 +183,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 +9 added lines, -3 removed lines

@@ -152,7 +152,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']];
 	}
 
@@ -161,7 +163,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;
 	}
 
@@ -183,7 +187,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/Modules/Html/Builder.php

Spacing +26 added lines, -26 removed lines

@@ -55,7 +55,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 );
 		$generatedTag = in_array( $instance->tag, $tags )
 			? $instance->buildTag()
@@ -78,8 +78,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;
 	}
@@ -97,7 +97,7 @@ 
 	 */
 	public function getOpeningTag()
 	{
-		$attributes = glsr( Attributes::class )->{$this->tag}( $this->args )->toString();
+		$attributes = glsr( Attributes::class )->{$this->tag}($this->args)->toString();
 		return '<'.trim( $this->tag.' '.$attributes ).'>';
 	}
 
@@ -107,7 +107,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 );
@@ -118,7 +118,7 @@ 
 	 */
 	protected function buildDefaultTag( $text = '' )
 	{
-		if( empty( $text )) {
+		if( empty($text) ) {
 			$text = $this->args['text'];
 		}
 		return $this->getOpeningTag().$text.$this->getClosingTag();
@@ -129,7 +129,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'] );
 		}
@@ -141,10 +141,10 @@ 
 	 */
 	protected function buildFormInput()
 	{
-		if( !in_array( $this->args['type'], ['checkbox', 'radio'] )) {
+		if( !in_array( $this->args['type'], ['checkbox', 'radio'] ) ) {
 			return $this->buildFormLabel().$this->getOpeningTag();
 		}
-		return empty( $this->args['options'] )
+		return empty($this->args['options'])
 			? $this->buildFormInputChoice()
 			: $this->buildFormInputMultiChoice();
 	}
@@ -154,13 +154,13 @@ 
 	 */
 	protected function buildFormInputChoice()
 	{
-		$labelText = !empty( $this->args['text'] )
+		$labelText = !empty($this->args['text'])
 			? $this->args['text']
 			: $this->args['label'];
 		return $this->label( $this->getOpeningTag().' '.$labelText, [
 			'class' => 'glsr-'.$this->args['type'].'-label',
 			'for' => $this->args['id'],
-		]);
+		] );
 	}
 
 	/**
@@ -169,7 +169,7 @@ 
 	protected function buildFormInputMultiChoice()
 	{
 		if( $this->args['type'] == 'checkbox' ) {
-			$this->args['name'].= '[]';
+			$this->args['name'] .= '[]';
 		}
 		$options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) {
 			return $carry.$this->li( $this->{$this->args['type']}([
@@ -177,12 +177,12 @@ 
 				'name' => $this->args['name'],
 				'text' => $this->args['options'][$key],
 				'value' => $key,
-			]));
+			]) );
 		});
 		return $this->ul( $options, [
 			'class' => $this->args['class'],
 			'id' => $this->args['id'],
-		]);
+		] );
 	}
 
 	/**
@@ -190,11 +190,11 @@ 
 	 */
 	protected function buildFormLabel()
 	{
-		if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return;
-		return $this->label([
+		if( empty($this->args['label']) || $this->args['type'] == 'hidden' )return;
+		return $this->label( [
 			'for' => $this->args['id'],
 			'text' => $this->args['label'],
-		]);
+		] );
 	}
 
 	/**
@@ -211,11 +211,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,
-			]);
+			] );
 		});
 	}
 
@@ -233,10 +233,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();
@@ -256,7 +256,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()
@@ -271,13 +271,13 @@ 
 	 */
 	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];
 		}
 	}
@@ -301,7 +301,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 +14 added lines, -14 removed lines

@@ -23,7 +23,7 @@ 
 			: $this->$method( $id );
 		return glsr( Template::class )->build( 'pages/settings/'.$id, [
 			'context' => $context,
-		]);
+		] );
 	}
 
 	/**
@@ -31,7 +31,7 @@ 
 	 */
 	protected function getFieldDefault( array $field )
 	{
-		return isset( $field['default'] )
+		return isset($field['default'])
 			? $field['default']
 			: '';
 	}
@@ -42,7 +42,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 );
 	}
@@ -53,14 +53,14 @@ 
 	 */
 	protected function getTemplateContext( $id )
 	{
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
 		$rows = '';
 		foreach( $fields as $name => $field ) {
 			$field = wp_parse_args( $field, [
 				'is_setting' => true,
 				'name' => $name,
-			]);
-			$rows.= new Field( $this->normalize( $field ));
+			] );
+			$rows .= new Field( $this->normalize( $field ) );
 		}
 		return [
 			'rows' => $rows,
@@ -73,7 +73,7 @@ 
 	protected function getTemplateContextForTranslations()
 	{
 		$translations = glsr( Translator::class )->renderAll();
-		$class = empty( $translations )
+		$class = empty($translations)
 			? 'glsr-hidden'
 			: '';
 		return [
@@ -94,7 +94,7 @@ 
 			$path,
 			glsr( Helper::class )->getPathValue( $path, glsr()->defaults )
 		);
-		if( is_array( $expectedValue )) {
+		if( is_array( $expectedValue ) ) {
 			return !in_array( $optionValue, $expectedValue );
 		}
 		return $optionValue != $expectedValue;
@@ -116,13 +116,13 @@ 
 	 */
 	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];
-			$field['data-depends'] = json_encode([
+			$field['data-depends'] = json_encode( [
 				'name' => glsr( Helper::class )->convertPathToName( $path, OptionManager::databaseKey() ),
 				'value' => $expectedValue,
-			], JSON_HEX_APOS|JSON_HEX_QUOT );
+			], JSON_HEX_APOS | JSON_HEX_QUOT );
 			$field['is_hidden'] = $this->isFieldHidden( $path, $expectedValue );
 		}
 		return $field;
@@ -133,9 +133,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;
@@ -149,7 +149,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/Html/Field.php

Spacing +18 added lines, -18 removed lines

@@ -26,7 +26,7 @@ 
 			'is_valid' => true,
 			'is_widget' => false,
 			'path' => '',
-		]);
+		] );
 		$this->normalize();
 	}
 
@@ -45,7 +45,7 @@ 
 	{
 		if( !$this->field['is_valid'] )return;
 		if( $this->field['is_raw'] ) {
-			return glsr( Builder::class )->{$this->field['type']}( $this->field );
+			return glsr( Builder::class )->{$this->field['type']}($this->field);
 		}
 		if( !$this->field['is_setting'] ) {
 			return $this->buildField();
@@ -73,9 +73,9 @@ 
 			'context' => [
 				'class' => $this->getFieldClass(),
 				'errors' => $this->getFieldErrors(),
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -86,10 +86,10 @@ 
 		return glsr( Template::class )->build( 'partials/form/table-row', [
 			'context' => [
 				'class' => $this->getFieldClass(),
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
 				'label' => glsr( Builder::class )->label( $this->field['legend'], ['for' => $this->field['id']] ),
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -98,16 +98,16 @@ 
 	protected function buildSettingMultiField()
 	{
 		$dependsOn = $this->getFieldDependsOn();
-		unset( $this->field['data-depends'] );
+		unset($this->field['data-depends']);
 		return glsr( Template::class )->build( 'partials/form/table-row-multiple', [
 			'context' => [
 				'class' => $this->getFieldClass(),
 				'depends_on' => $dependsOn,
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
 				'label' => glsr( Builder::class )->label( $this->field['legend'], ['for' => $this->field['id']] ),
 				'legend' => $this->field['legend'],
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -116,13 +116,13 @@ 
 	protected function getFieldClass()
 	{
 		$classes = [];
-		if( !empty( $this->field['errors'] )) {
+		if( !empty($this->field['errors']) ) {
 			$classes[] = 'glsr-has-error';
 		}
 		if( $this->field['is_hidden'] ) {
 			$classes[] = 'hidden';
 		}
-		if( !empty( $this->field['required'] )) {
+		if( !empty($this->field['required']) ) {
 			$classes[] = 'glsr-required';
 		}
 		return implode( ' ', $classes );
@@ -133,7 +133,7 @@ 
 	 */
 	protected function getFieldDependsOn()
 	{
-		return !empty( $this->field['data-depends'] )
+		return !empty($this->field['data-depends'])
 			? $this->field['data-depends']
 			: '';
 	}
@@ -143,7 +143,7 @@ 
 	 */
 	protected function getFieldErrors()
 	{
-		if( empty( $this->field['errors'] ) || !is_array( $this->field['errors'] ))return;
+		if( empty($this->field['errors']) || !is_array( $this->field['errors'] ) )return;
 		$errors = array_reduce( $this->field['errors'], function( $carry, $error ) {
 			return $carry.glsr( Builder::class )->span( $error, ['class' => 'glsr-field-error'] );
 		});
@@ -170,13 +170,13 @@ 
 			'name', 'type',
 		];
 		foreach( $requiredValues as $value ) {
-			if( isset( $this->field[$value] ))continue;
+			if( isset($this->field[$value]) )continue;
 			$missingValues[] = $value;
 			$this->field['is_valid'] = false;
 		}
-		if( !empty( $missingValues )) {
+		if( !empty($missingValues) ) {
 			glsr_log()
-				->warning( 'Field is missing: '.implode( ', ', $missingValues ))
+				->warning( 'Field is missing: '.implode( ', ', $missingValues ) )
 				->info( $this->field );
 		}
 		return $this->field['is_valid'];
@@ -190,7 +190,7 @@ 
 		if( !$this->isFieldValid() )return;
 		$this->field['path'] = $this->field['name'];
 		$className = glsr( Helper::class )->buildClassName( $this->field['type'], __NAMESPACE__.'\Fields' );
-		if( class_exists( $className )) {
+		if( class_exists( $className ) ) {
 			$this->field = array_merge(
 				wp_parse_args( $this->field, $className::defaults() ),
 				$className::required()
@@ -205,7 +205,7 @@ 
 	 */
 	protected function normalizeFieldId()
 	{
-		if( isset( $this->field['id'] ) || $this->field['is_raw'] )return;
+		if( isset($this->field['id']) || $this->field['is_raw'] )return;
 		$this->field['id'] = glsr( Helper::class )->convertPathToId(
 			$this->field['path'],
 			$this->getFieldPrefix()

Braces +15 added lines, -5 removed lines

@@ -43,7 +43,9 @@ 
 	 */
 	public function build()
 	{
-		if( !$this->field['is_valid'] )return;
+		if( !$this->field['is_valid'] ) {
+			return;
+		}
 		if( $this->field['is_raw'] ) {
 			return glsr( Builder::class )->{$this->field['type']}( $this->field );
 		}
@@ -143,7 +145,9 @@ 
 	 */
 	protected function getFieldErrors()
 	{
-		if( empty( $this->field['errors'] ) || !is_array( $this->field['errors'] ))return;
+		if( empty( $this->field['errors'] ) || !is_array( $this->field['errors'] )) {
+			return;
+		}
 		$errors = array_reduce( $this->field['errors'], function( $carry, $error ) {
 			return $carry.glsr( Builder::class )->span( $error, ['class' => 'glsr-field-error'] );
 		});
@@ -170,7 +174,9 @@ 
 			'name', 'type',
 		];
 		foreach( $requiredValues as $value ) {
-			if( isset( $this->field[$value] ))continue;
+			if( isset( $this->field[$value] )) {
+				continue;
+			}
 			$missingValues[] = $value;
 			$this->field['is_valid'] = false;
 		}
@@ -187,7 +193,9 @@ 
 	 */
 	protected function normalize()
 	{
-		if( !$this->isFieldValid() )return;
+		if( !$this->isFieldValid() ) {
+			return;
+		}
 		$this->field['path'] = $this->field['name'];
 		$className = glsr( Helper::class )->buildClassName( $this->field['type'], __NAMESPACE__.'\Fields' );
 		if( class_exists( $className )) {
@@ -205,7 +213,9 @@ 
 	 */
 	protected function normalizeFieldId()
 	{
-		if( isset( $this->field['id'] ) || $this->field['is_raw'] )return;
+		if( isset( $this->field['id'] ) || $this->field['is_raw'] ) {
+			return;
+		}
 		$this->field['id'] = glsr( Helper::class )->convertPathToId(
 			$this->field['path'],
 			$this->getFieldPrefix()

plugin/Modules/Schema.php

Spacing +18 added lines, -18 removed lines

@@ -33,10 +33,10 @@ 
 		foreach( glsr( Database::class )->getReviews( $this->args )->reviews as $review ) {
 			$reviews[] = $this->buildReview( $review );
 		}
-		if( !empty( $reviews )) {
+		if( !empty($reviews) ) {
 			array_walk( $reviews, function( &$review ) {
-				unset( $review['@context'] );
-				unset( $review['itemReviewed'] );
+				unset($review['@context']);
+				unset($review['itemReviewed']);
 			});
 			$schema['review'] = $reviews;
 		}
@@ -50,20 +50,20 @@ 
 	public function buildReview( $review )
 	{
 		$schema = SchemaOrg::Review()
-			->doIf( !in_array( 'title', $this->args['hide'] ), function( ReviewSchema $schema ) use( $review ) {
+			->doIf( !in_array( 'title', $this->args['hide'] ), function( ReviewSchema $schema ) use($review) {
 				$schema->name( $review->title );
 			})
-			->doIf( !in_array( 'excerpt', $this->args['hide'] ), function( ReviewSchema $schema ) use( $review ) {
+			->doIf( !in_array( 'excerpt', $this->args['hide'] ), function( ReviewSchema $schema ) use($review) {
 				$schema->reviewBody( $review->content );
 			})
-			->datePublished(( new DateTime( $review->date ))->format( DateTime::ISO8601 ))
+			->datePublished( (new DateTime( $review->date ))->format( DateTime::ISO8601 ) )
 			->author( SchemaOrg::Person()
 				->name( $review->author )
 			)
 			->itemReviewed( $this->getSchemaType()
 				->name( $this->getThingName() )
 			);
-		if( !empty( $review->rating )) {
+		if( !empty($review->rating) ) {
 			$schema->reviewRating( SchemaOrg::Rating()
 				->ratingValue( $review->rating )
 				->bestRating( Rating::MAX_RATING )
@@ -79,7 +79,7 @@ 
 	 */
 	public function buildSummary( $args = null )
 	{
-		if( is_array( $args )) {
+		if( is_array( $args ) ) {
 			$this->args = $args;
 		}
 		$schema = $this->getSchemaType()
@@ -91,7 +91,7 @@ 
 			->image( $this->getThingImage() )
 			->url( $this->getThingUrl() );
 		$count = $this->getReviewCount();
-		if( !empty( $count )) {
+		if( !empty($count) ) {
 			$schema->aggregateRating( SchemaOrg::AggregateRating()
 				->ratingValue( $this->getRatingValue() )
 				->reviewCount( $count )
@@ -109,11 +109,11 @@ 
 	 */
 	public function render()
 	{
-		if( is_null( glsr()->schemas ))return;
+		if( is_null( glsr()->schemas ) )return;
 		printf( '<script type="application/ld+json">%s</script>', json_encode(
 			apply_filters( 'site-reviews/schema/all', glsr()->schemas ),
 			JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES
-		));
+		) );
 	}
 
 	/**
@@ -121,9 +121,9 @@ 
 	 */
 	public function store( array $schema )
 	{
-		$schemas = (array) glsr()->schemas;
+		$schemas = (array)glsr()->schemas;
 		$schemas[] = $schema;
-		glsr()->schemas = array_map( 'unserialize', array_unique( array_map( 'serialize', $schemas )));
+		glsr()->schemas = array_map( 'unserialize', array_unique( array_map( 'serialize', $schemas ) ) );
 	}
 
 	/**
@@ -148,7 +148,7 @@ 
 	 */
 	protected function getReviews( $force = false )
 	{
-		if( !isset( $this->reviews ) || $force ) {
+		if( !isset($this->reviews) || $force ) {
 			$args = wp_parse_args( ['count' => -1], $this->args );
 			$this->reviews = glsr( Database::class )->getReviews( $args )->reviews;
 		}
@@ -162,7 +162,7 @@ 
 	 */
 	protected function getSchemaOption( $option, $fallback )
 	{
-		if( $schemaOption = trim( (string)get_post_meta( intval( get_the_ID() ), 'schema_'.$option, true ))) {
+		if( $schemaOption = trim( (string)get_post_meta( intval( get_the_ID() ), 'schema_'.$option, true ) ) ) {
 			return $schemaOption;
 		}
 		$default = glsr( OptionManager::class )->get( 'settings.reviews.schema.'.$option.'.default', $fallback );
@@ -186,11 +186,11 @@ 
 		switch( $option ) {
 			case 'description':
 				$post = get_post();
-				if( !( $post instanceof WP_Post )) {
+				if( !($post instanceof WP_Post) ) {
 					return '';
 				}
-				$text = strip_shortcodes( wp_strip_all_tags( $post->post_excerpt ));
-				return wp_trim_words( $text, apply_filters( 'excerpt_length', 55 ));
+				$text = strip_shortcodes( wp_strip_all_tags( $post->post_excerpt ) );
+				return wp_trim_words( $text, apply_filters( 'excerpt_length', 55 ) );
 			case 'image':
 				return (string)get_the_post_thumbnail_url( null, 'large' );
 			case 'name':

Braces +6 added lines, -2 removed lines

@@ -109,7 +109,9 @@ 
 	 */
 	public function render()
 	{
-		if( is_null( glsr()->schemas ))return;
+		if( is_null( glsr()->schemas )) {
+			return;
+		}
 		printf( '<script type="application/ld+json">%s</script>', json_encode(
 			apply_filters( 'site-reviews/schema/all', glsr()->schemas ),
 			JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES
@@ -182,7 +184,9 @@ 
 		if( $value != $fallback ) {
 			return $value;
 		}
-		if( !is_single() && !is_page() )return;
+		if( !is_single() && !is_page() ) {
+			return;
+		}
 		switch( $option ) {
 			case 'description':
 				$post = get_post();

plugin/Modules/Validator.php

Spacing +17 added lines, -17 removed lines

@@ -73,7 +73,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;
@@ -88,13 +88,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( $attribute, $value, $parameters )) {
+		if( !$this->$method( $attribute, $value, $parameters ) ) {
 			$this->addFailure( $attribute, $rule, $parameters );
 		}
 	}
@@ -143,7 +143,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 );
@@ -158,11 +158,11 @@ 
 	 */
 	protected function getRule( $attribute, $rules )
 	{
-		if( !array_key_exists( $attribute, $this->rules ))return;
-		$rules = (array) $rules;
+		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];
 			}
 		}
@@ -180,7 +180,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 ) );
 	}
 
 	/**
@@ -259,11 +259,11 @@ 
 		$parameters = [];
 		// example: {rule}:{parameters}
 		if( strpos( $rule, ':' ) !== false ) {
-			list( $rule, $parameter ) = explode( ':', $rule, 2 );
+			list($rule, $parameter) = explode( ':', $rule, 2 );
 			// example: {parameter1,parameter2,...}
 			$parameters = $this->parseParameters( $rule, $parameter );
 		}
-		$rule = ucwords( str_replace( ['-', '_'], ' ', trim( $rule )));
+		$rule = ucwords( str_replace( ['-', '_'], ' ', trim( $rule ) ) );
 		$rule = str_replace( ' ', '', $rule );
 		return [$rule, $parameters];
 	}
@@ -290,7 +290,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 );
 	}
 
@@ -315,11 +315,11 @@ 
 			'regex' => __( 'The format is invalid.', 'site-reviews' ),
 			'required' => __( 'This field is required.', 'site-reviews' ),
 		];
-		$message = isset( $strings[$key] )
+		$message = isset($strings[$key])
 			? $strings[$key]
 			: false;
 		if( !$message )return;
-		if( method_exists( $this, $method = 'replace'.$rule )) {
+		if( method_exists( $this, $method = 'replace'.$rule ) ) {
 			$message = $this->$method( $message, $parameters );
 		}
 		return $message;

Braces +12 added lines, -4 removed lines

@@ -73,7 +73,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;
@@ -89,7 +91,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." );
@@ -158,7 +162,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 );
@@ -318,7 +324,9 @@ 
 		$message = isset( $strings[$key] )
 			? $strings[$key]
 			: false;
-		if( !$message )return;
+		if( !$message ) {
+			return;
+		}
 		if( method_exists( $this, $method = 'replace'.$rule )) {
 			$message = $this->$method( $message, $parameters );
 		}