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;
 		}
-		else if( is_array( $value )) {
+		else if( is_array( $value ) ) {
 			return count( $value );
 		}
 		return function_exists( 'mb_strlen' )
@@ -209,7 +209,7 @@ 
 	 */
 	protected function getValue( $attribute )
 	{
-		if( isset( $this->data[$attribute] )) {
+		if( isset($this->data[$attribute]) ) {
 			return $this->data[$attribute];
 		}
 	}
@@ -222,7 +222,7 @@ 
 	 */
 	protected function hasRule( $attribute, $rules )
 	{
-		return !is_null( $this->getRule( $attribute, $rules ));
+		return !is_null( $this->getRule( $attribute, $rules ) );
 	}
 
 	/**
@@ -260,7 +260,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 );
@@ -289,7 +289,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 );
 	}
 
@@ -302,7 +302,7 @@ 
 	protected function translator( $key, array $parameters )
 	{
 		$strings = glsr( ValidationStringsDefaults::class )->defaults();
-		if( isset( $strings[$key] )) {
+		if( isset($strings[$key]) ) {
 			return $this->replace( $strings[$key], $parameters );
 		}
 		return "error";

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

site-reviews.php

Braces +3 added lines, -1 removed lines

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

Spacing +4 added lines, -4 removed lines

@@ -18,7 +18,7 @@ 
 
 defined( 'WPINC' ) || die;
 
-if( !class_exists( 'GL_Plugin_Check_v3' )) {
+if( !class_exists( 'GL_Plugin_Check_v3' ) ) {
 	require_once __DIR__.'/activate.php';
 }
 if( !(new GL_Plugin_Check_v3( __FILE__ ))->canProceed() )return;
@@ -29,7 +29,7 @@ 
 
 $app = new GeminiLabs\SiteReviews\Application;
 $app->make( 'Provider' )->register( $app );
-register_activation_hook( __FILE__, array( $app, 'activate' ));
-register_deactivation_hook( __FILE__, array( $app, 'deactivate' ));
-register_shutdown_function( array( $app, 'catchFatalError' ));
+register_activation_hook( __FILE__, array( $app, 'activate' ) );
+register_deactivation_hook( __FILE__, array( $app, 'deactivate' ) );
+register_shutdown_function( array( $app, 'catchFatalError' ) );
 $app->init();

plugin/Modules/Blacklist.php

Spacing +7 added lines, -7 removed lines

@@ -11,13 +11,13 @@ 
 	 */
 	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',
 			$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 ) {
 			$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;
 			}
 		}

Braces +3 added lines, -1 removed lines

@@ -37,7 +37,9 @@
 		$lines = explode( "\n", $blacklist );
 		foreach( (array)$lines as $line ) {
 			$line = trim( $line );
-			if( empty( $line ) || 256 < strlen( $line ))continue;
+			if( empty( $line ) || 256 < strlen( $line )) {
+				continue;
+			}
 			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ));
 			if( preg_match( $pattern, $target )) {
 				return true;

plugin/Modules/Html/ReviewHtml.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 	 */
 	public function __get( $key )
 	{
-		if( array_key_exists( $key, $this->values )) {
+		if( array_key_exists( $key, $this->values ) ) {
 			return $this->values[$key];
 		}
 		return '';

plugin/Review.php

Braces +6 added lines, -2 removed lines

@@ -69,7 +69,9 @@ 
 		$properties = glsr( CreateReviewDefaults::class )->restrict( array_merge( $defaults, $meta ));
 		$this->modified = $this->isModified( $properties );
 		array_walk( $properties, function( $value, $key ) {
-			if( !property_exists( $this, $key ) || isset( $this->$key ))return;
+			if( !property_exists( $this, $key ) || isset( $this->$key )) {
+				return;
+			}
 			$this->$key = maybe_unserialize( $value );
 		});
 	}
@@ -80,7 +82,9 @@ 
 	protected function setTermIds( WP_Post $post )
 	{
 		$this->term_ids = [];
-		if( !is_array( $terms = get_the_terms( $post, Application::TAXONOMY )))return;
+		if( !is_array( $terms = get_the_terms( $post, Application::TAXONOMY ))) {
+			return;
+		}
 		foreach( $terms as $term ) {
 			$this->term_ids[] = $term->term_id;
 		}

Spacing +6 added lines, -6 removed lines

@@ -46,10 +46,10 @@ 
 	 */
 	public function __get( $key )
 	{
-		if( property_exists( $this, $key )) {
+		if( property_exists( $this, $key ) ) {
 			return $this->$key;
 		}
-		if( is_array( $this->custom ) && array_key_exists( $key, $this->custom )) {
+		if( is_array( $this->custom ) && array_key_exists( $key, $this->custom ) ) {
 			return $this->custom[$key];
 		}
 		return '';
@@ -77,13 +77,13 @@ 
 			'review_type' => 'local',
 		];
 		$meta = array_filter(
-			array_map( 'array_shift', (array)get_post_meta( $post->ID )),
+			array_map( 'array_shift', (array)get_post_meta( $post->ID ) ),
 			'strlen'
 		);
-		$properties = glsr( CreateReviewDefaults::class )->restrict( array_merge( $defaults, $meta ));
+		$properties = glsr( CreateReviewDefaults::class )->restrict( array_merge( $defaults, $meta ) );
 		$this->modified = $this->isModified( $properties );
 		array_walk( $properties, function( $value, $key ) {
-			if( !property_exists( $this, $key ) || isset( $this->$key ))return;
+			if( !property_exists( $this, $key ) || isset($this->$key) )return;
 			$this->$key = maybe_unserialize( $value );
 		});
 	}
@@ -94,7 +94,7 @@ 
 	protected function setTermIds( WP_Post $post )
 	{
 		$this->term_ids = [];
-		if( !is_array( $terms = get_the_terms( $post, Application::TAXONOMY )))return;
+		if( !is_array( $terms = get_the_terms( $post, Application::TAXONOMY ) ) )return;
 		foreach( $terms as $term ) {
 			$this->term_ids[] = $term->term_id;
 		}

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 );
 		$hookName = (new ReflectionClass( $this ))->getShortName();
 		$hookName = str_replace( 'Defaults', '', $hookName );

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 );
 		$hookName = (new ReflectionClass( $this ))->getShortName();
 		$hookName = str_replace( 'Defaults', '', $hookName );

plugin/Modules/Console.php

Spacing +9 added lines, -9 removed lines

@@ -108,7 +108,7 @@ 
 	 */
 	public function get()
 	{
-		return empty( $this->log )
+		return empty($this->log)
 			? __( 'Console is empty', 'site-reviews' )
 			: $this->log;
 	}
@@ -120,11 +120,11 @@ 
 	public function humanSize( $valueIfEmpty = null )
 	{
 		$bytes = $this->size();
-		if( empty( $bytes ) && is_string( $valueIfEmpty )) {
+		if( empty($bytes) && is_string( $valueIfEmpty ) ) {
 			return $valueIfEmpty;
 		}
-		$exponent = floor( log( max( $bytes, 1 ), 1024 ));
-		return round( $bytes / pow( 1024, $exponent ), 2 ).' '.['bytes','KB','MB','GB'][$exponent];
+		$exponent = floor( log( max( $bytes, 1 ), 1024 ) );
+		return round( $bytes / pow( 1024, $exponent ), 2 ).' '.['bytes', 'KB', 'MB', 'GB'][$exponent];
 	}
 
 	/**
@@ -147,9 +147,9 @@ 
 	public function log( $level, $message, array $context = [] )
 	{
 		$constants = (new ReflectionClass( __CLASS__ ))->getConstants();
-		if( in_array( $level, $constants, true )) {
+		if( in_array( $level, $constants, true ) ) {
 			$entry = $this->buildLogEntry( $level, $message, $context );
-			file_put_contents( $this->file, $entry, FILE_APPEND|LOCK_EX );
+			file_put_contents( $this->file, $entry, FILE_APPEND | LOCK_EX );
 			$this->reset();
 		}
 		return $this;
@@ -222,7 +222,7 @@ 
 	 */
 	protected function interpolate( $message, $context = [] )
 	{
-		if( $this->isObjectOrArray( $message ) || !is_array( $context )) {
+		if( $this->isObjectOrArray( $message ) || !is_array( $context ) ) {
 			return print_r( $message, true );
 		}
 		$replace = [];
@@ -250,7 +250,7 @@ 
 		if( $value instanceof DateTime ) {
 			$value = $value->format( 'Y-m-d H:i:s' );
 		}
-		else if( $this->isObjectOrArray( $value )) {
+		else if( $this->isObjectOrArray( $value ) ) {
 			$value = json_encode( $value );
 		}
 		return (string)$value;
@@ -265,7 +265,7 @@ 
 			$this->clear();
 			file_put_contents(
 				$this->file,
-				$this->buildLogEntry( 'info', __( 'Console was automatically cleared (128 KB maximum size)', 'site-reviews' ))
+				$this->buildLogEntry( 'info', __( 'Console was automatically cleared (128 KB maximum size)', 'site-reviews' ) )
 			);
 		}
 	}

plugin/Database/QueryBuilder.php

Braces +21 added lines, -7 removed lines

@@ -17,9 +17,13 @@ 
 	{
 		$queries = [];
 		foreach( $keys as $key ) {
-			if( !array_key_exists( $key, $values ))continue;
+			if( !array_key_exists( $key, $values )) {
+				continue;
+			}
 			$methodName = glsr( Helper::class )->buildMethodName( $key, __FUNCTION__ );
-			if( !method_exists( $this, $methodName ))continue;
+			if( !method_exists( $this, $methodName )) {
+				continue;
+			}
 			$query = call_user_func( [$this, $methodName], $values[$key] );
 			if( is_array( $query )) {
 				$queries[] = $query;
@@ -36,7 +40,9 @@ 
 		$string = '';
 		$values = array_filter( $values );
 		foreach( $conditions as $key => $value ) {
-			if( !isset( $values[$key] ))continue;
+			if( !isset( $values[$key] )) {
+				continue;
+			}
 			$values[$key] = implode( ',', (array)$values[$key] );
 			$string .= strpos( $value, '%s' ) !== false
 				? sprintf( $value, strval( $values[$key] ))
@@ -110,7 +116,9 @@ 
 	 */
 	protected function buildQueryAssignedTo( $value )
 	{
-		if( empty( $value ))return;
+		if( empty( $value )) {
+			return;
+		}
 		$postIds = glsr( Helper::class )->convertStringToArray( $value, 'is_numeric' );
 		return [
 			'compare' => 'IN',
@@ -125,7 +133,9 @@ 
 	 */
 	protected function buildQueryCategory( $value )
 	{
-		if( empty( $value ))return;
+		if( empty( $value )) {
+			return;
+		}
 		return [
 			'field' => 'term_id',
 			'taxonomy' => Application::TAXONOMY,
@@ -139,7 +149,9 @@ 
 	 */
 	protected function buildQueryRating( $value )
 	{
-		if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 )))return;
+		if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 ))) {
+			return;
+		}
 		return [
 			'compare' => '>=',
 			'key' => 'rating',
@@ -153,7 +165,9 @@ 
 	 */
 	protected function buildQueryType( $value )
 	{
-		if( in_array( $value, ['','all'] ))return;
+		if( in_array( $value, ['','all'] )) {
+			return;
+		}
 		return [
 			'key' => 'review_type',
 			'value' => $value,

Spacing +15 added lines, -15 removed lines

@@ -17,11 +17,11 @@ 
 	{
 		$queries = [];
 		foreach( $keys as $key ) {
-			if( !array_key_exists( $key, $values ))continue;
+			if( !array_key_exists( $key, $values ) )continue;
 			$methodName = glsr( Helper::class )->buildMethodName( $key, __FUNCTION__ );
-			if( !method_exists( $this, $methodName ))continue;
+			if( !method_exists( $this, $methodName ) )continue;
 			$query = call_user_func( [$this, $methodName], $values[$key] );
-			if( is_array( $query )) {
+			if( is_array( $query ) ) {
 				$queries[] = $query;
 			}
 		}
@@ -36,10 +36,10 @@ 
 		$string = '';
 		$values = array_filter( $values );
 		foreach( $conditions as $key => $value ) {
-			if( !isset( $values[$key] ))continue;
+			if( !isset($values[$key]) )continue;
 			$values[$key] = implode( ',', (array)$values[$key] );
 			$string .= strpos( $value, '%s' ) !== false
-				? sprintf( $value, strval( $values[$key] ))
+				? sprintf( $value, strval( $values[$key] ) )
 				: $value;
 		}
 		return $string;
@@ -53,11 +53,11 @@ 
 	 */
 	public function buildSqlOr( $values, $sprintfFormat )
 	{
-		if( !is_array( $values )) {
+		if( !is_array( $values ) ) {
 			$values = explode( ',', $values );
 		}
-		$values = array_filter( array_map( 'trim', (array)$values ));
-		$values = array_map( function( $value ) use( $sprintfFormat ) {
+		$values = array_filter( array_map( 'trim', (array)$values ) );
+		$values = array_map( function( $value ) use($sprintfFormat) {
 			return sprintf( $sprintfFormat, $value );
 		}, $values );
 		return implode( ' OR ', $values );
@@ -72,11 +72,11 @@ 
 	 */
 	public function filterSearchByTitle( $search, WP_Query $query )
 	{
-		if( empty( $search ) || empty( $query->get( 'search_terms' ))) {
+		if( empty($search) || empty($query->get( 'search_terms' )) ) {
 			return $search;
 		}
 		global $wpdb;
-		$n = empty( $query->get( 'exact' ))
+		$n = empty($query->get( 'exact' ))
 			? '%'
 			: '';
 		$search = [];
@@ -100,7 +100,7 @@ 
 			? glsr()->constant( 'PAGED_QUERY_VAR' )
 			: 'page';
 		return $isEnabled
-			? max( 1, intval( get_query_var( $pagedQuery )))
+			? max( 1, intval( get_query_var( $pagedQuery ) ) )
 			: 1;
 	}
 
@@ -110,7 +110,7 @@ 
 	 */
 	protected function buildQueryAssignedTo( $value )
 	{
-		if( empty( $value ))return;
+		if( empty($value) )return;
 		$postIds = glsr( Helper::class )->convertStringToArray( $value, 'is_numeric' );
 		return [
 			'compare' => 'IN',
@@ -125,7 +125,7 @@ 
 	 */
 	protected function buildQueryCategory( $value )
 	{
-		if( empty( $value ))return;
+		if( empty($value) )return;
 		return [
 			'field' => 'term_id',
 			'taxonomy' => Application::TAXONOMY,
@@ -139,7 +139,7 @@ 
 	 */
 	protected function buildQueryRating( $value )
 	{
-		if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 )))return;
+		if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 ) ) )return;
 		return [
 			'compare' => '>=',
 			'key' => 'rating',
@@ -153,7 +153,7 @@ 
 	 */
 	protected function buildQueryType( $value )
 	{
-		if( in_array( $value, ['','all'] ))return;
+		if( in_array( $value, ['', 'all'] ) )return;
 		return [
 			'key' => 'review_type',
 			'value' => $value,