pryley / site-reviews

plugin/Modules/Validator.php

Braces +12 added lines, -4 removed lines

@@ -74,7 +74,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;
@@ -90,7 +92,9 @@ 
 	public function validateAttribute( $attribute, $rule )
 	{
 		list( $rule, $parameters ) = $this->parseRule( $rule );
-		if( $rule == '' )return;
+		if( $rule == '' ) {
+			return;
+		}
 		$value = $this->getValue( $attribute );
 		$this->validateRequired( $attribute, $value ) || in_array( $rule, $this->implicitRules );
 		if( !method_exists( $this, $method = 'validate'.$rule )) {
@@ -163,7 +167,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 );
@@ -314,7 +320,9 @@ 
 		$message = isset( $strings[$key] )
 			? $strings[$key]
 			: false;
-		if( !$message )return;
+		if( !$message ) {
+			return;
+		}
 		$message = str_replace( ':attribute', $attribute, $message );
 		if( method_exists( $this, $replacer = 'replace'.$rule )) {
 			$message = $this->$replacer( $message, $parameters );

Spacing +18 added lines, -18 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,14 +88,14 @@ 
 	 */
 	public function validateAttribute( $attribute, $rule )
 	{
-		list( $rule, $parameters ) = $this->parseRule( $rule );
+		list($rule, $parameters) = $this->parseRule( $rule );
 		if( $rule == '' )return;
 		$value = $this->getValue( $attribute );
 		$this->validateRequired( $attribute, $value ) || in_array( $rule, $this->implicitRules );
-		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 );
 		}
 	}
@@ -110,7 +110,7 @@ 
 	{
 		$message = $this->getMessage( $attribute, $rule, $parameters );
 		$this->errors[$attribute]['errors'][] = $message;
-		if( !isset( $this->errors[$attribute]['value'] )) {
+		if( !isset($this->errors[$attribute]['value']) ) {
 			$this->errors[$attribute]['value'] = $this->getValue( $attribute );
 		}
 	}
@@ -147,7 +147,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 );
@@ -162,11 +162,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];
 			}
 		}
@@ -184,7 +184,7 @@ 
 		if( is_numeric( $value ) && $hasNumeric ) {
 			return $value;
 		}
-		elseif( is_array( $value )) {
+		elseif( is_array( $value ) ) {
 			return count( $value );
 		}
 		return mb_strlen( $value );
@@ -211,7 +211,7 @@ 
 	 */
 	protected function getValue( $attribute )
 	{
-		if( isset( $this->data[$attribute] )) {
+		if( isset($this->data[$attribute]) ) {
 			return $this->data[$attribute];
 		}
 	}
@@ -224,7 +224,7 @@ 
 	 */
 	protected function hasRule( $attribute, $rules )
 	{
-		return !is_null( $this->getRule( $attribute, $rules ));
+		return !is_null( $this->getRule( $attribute, $rules ) );
 	}
 
 	/**
@@ -264,11 +264,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];
 	}
@@ -296,7 +296,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 );
 	}
 
@@ -321,12 +321,12 @@ 
 			'regex' => _x( 'The :attribute format is invalid.', ':attribute is a placeholder and should not be translated.', 'site-reviews' ),
 			'required' => _x( 'The :attribute field is required.', ':attribute is a placeholder and should not be translated.', 'site-reviews' ),
 		];
-		$message = isset( $strings[$key] )
+		$message = isset($strings[$key])
 			? $strings[$key]
 			: false;
 		if( !$message )return;
 		$message = str_replace( ':attribute', $attribute, $message );
-		if( method_exists( $this, $replacer = 'replace'.$rule )) {
+		if( method_exists( $this, $replacer = 'replace'.$rule ) ) {
 			$message = $this->$replacer( $message, $parameters );
 		}
 		return $message;

plugin/Modules/Html/Partials/SiteReviewsForm.php

Spacing +3 added lines, -3 removed lines

@@ -14,14 +14,14 @@
 	{
 		$partial = glsr( Html::class )->buildTemplate( 'templates/reviews-form', [
 			'globals' => $args,
-		]);
+		] );
 		$form = glsr( Builder::class )->form( $partial, [
 			'class' => 'glsr-form',
 			'id' => $args['id'],
 			'method' => 'post',
-		]);
+		] );
 		return glsr( Builder::class )->div( $form, [
 			'class' => 'glsr-form-wrap '.$args['class'],
-		]);
+		] );
 	}
 }

plugin/Modules/Html/Partials/StarRating.php

Spacing +4 added lines, -4 removed lines

@@ -15,11 +15,11 @@
 	public function build( $name, array $args = [] )
 	{
 		$roundedRating = floor( round( $args['rating'], 1 ) * 2 ) / 2;
-		$percentage = glsr( Builder::class )->span([
-			'style' => 'width:'.( $roundedRating / Rating::MAX_RATING * 100 ).'%;',
-		]);
+		$percentage = glsr( Builder::class )->span( [
+			'style' => 'width:'.($roundedRating / Rating::MAX_RATING * 100).'%;',
+		] );
 		return glsr( Builder::class )->span( $percentage, [
 			'class' => 'glsr-star-rating',
-		]);
+		] );
 	}
 }

plugin/Modules/Html/Partial.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 	public function build( $partialPath, array $args = [] )
 	{
 		$className = glsr( Helper::class )->buildClassName( $partialPath, 'Modules\Html\Partials' );
-		if( !class_exists( $className )) {
+		if( !class_exists( $className ) ) {
 			glsr_log()->error( 'Partial missing: '.$className );
 			return;
 		}

plugin/Modules/Logger.php

Braces +3 added lines, -1 removed lines

@@ -190,7 +190,9 @@
 		if( $index === false
 			|| !isset( $caller[$index+2]['class'] )
 			|| !isset( $caller[$index+2]['function'] )
-		)return;
+		) {
+			return;
+		}
 		return sprintf( '[%s()->%s:%s] ',
 			$caller[$index+2]['class'],
 			$caller[$index+2]['function'],

Spacing +12 added lines, -12 removed lines

@@ -108,7 +108,7 @@ 
 	 */
 	public function get()
 	{
-		return empty( $this->log )
+		return empty($this->log)
 			? __( 'Log is empty', 'site-reviews' )
 			: $this->log;
 	}
@@ -134,9 +134,9 @@ 
 	{
 		$constants = (new ReflectionClass( __CLASS__ ))->getConstants();
 		$constants = (array)apply_filters( 'site-reviews/log-levels', $constants );
-		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;
@@ -186,15 +186,15 @@ 
 	protected function getDebugInformation()
 	{
 		$caller = debug_backtrace( 0, 6 );
-		$index = array_search( 'log', array_column( $caller, 'function' ));
+		$index = array_search( 'log', array_column( $caller, 'function' ) );
 		if( $index === false
-			|| !isset( $caller[$index+2]['class'] )
-			|| !isset( $caller[$index+2]['function'] )
+			|| !isset($caller[$index + 2]['class'])
+			|| !isset($caller[$index + 2]['function'])
 		)return;
 		return sprintf( '[%s()->%s:%s] ',
-			$caller[$index+2]['class'],
-			$caller[$index+2]['function'],
-			$caller[$index+1]['line']
+			$caller[$index + 2]['class'],
+			$caller[$index + 2]['function'],
+			$caller[$index + 1]['line']
 		);
 	}
 
@@ -206,7 +206,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 = [];
@@ -234,7 +234,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;
@@ -250,7 +250,7 @@ 
 			$this->clear();
 			file_put_contents(
 				$this->file,
-				$this->buildLogEntry( 'info', __( 'Log has been automatically reset (512 KB max size)', 'site-reviews' ))
+				$this->buildLogEntry( 'info', __( 'Log has been automatically reset (512 KB max size)', 'site-reviews' ) )
 			);
 		}
 	}

plugin/Commands/TogglePinned.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 	public function __construct( $input )
 	{
 		$this->id = $input['id'];
-		$this->pinned = isset( $input['pinned'] )
+		$this->pinned = isset($input['pinned'])
 			? wp_validate_boolean( $input['pinned'] )
 			: null;
 	}

plugin/Database/SqlQueries.php

Spacing +9 added lines, -9 removed lines

@@ -23,13 +23,13 @@ 
 	 */
 	public function getReviewCounts( $metaKey )
 	{
-		return (array) $this->db->get_results("
+		return (array)$this->db->get_results( "
 			SELECT m.meta_value AS name, COUNT(*) num_posts
 			FROM {$this->db->posts} AS p
 			INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
 			WHERE p.post_type = '{$this->postType}' AND m.meta_key = '{$metaKey}'
 			GROUP BY name
-		");
+		" );
 	}
 
 	/**
@@ -38,14 +38,14 @@ 
 	 */
 	public function getReviewPostId( $metaReviewId )
 	{
-		$postId = $this->db->get_var("
+		$postId = $this->db->get_var( "
 			SELECT p.ID
 			FROM {$this->db->posts} AS p
 			INNER JOIN {$this->db->postmeta} AS pm ON p.ID = pm.post_id
 			WHERE p.post_type = '{$this->postType}'
 			AND pm.meta_key = 'review_id'
 			AND pm.meta_value = '{$metaReviewId}'
-		");
+		" );
 		return intval( $postId );
 	}
 
@@ -55,7 +55,7 @@ 
 	 */
 	public function getReviewIdsByType( $metaReviewType )
 	{
-		$query = $this->db->get_col("
+		$query = $this->db->get_col( "
 			SELECT m1.meta_value AS review_id
 			FROM {$this->db->posts} AS p
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -64,8 +64,8 @@ 
 			AND m1.meta_key = 'review_id'
 			AND m2.meta_key = 'review_type'
 			AND m2.meta_value = '{$metaReviewType}'
-		");
-		return array_keys( array_flip( $query ));
+		" );
+		return array_keys( array_flip( $query ) );
 	}
 
 	/**
@@ -78,13 +78,13 @@ 
 		$queryBuilder = glsr( QueryBuilder::class );
 		$keys = $queryBuilder->buildSqlOr( $keys, "pm.meta_key = '%s'" );
 		$status = $queryBuilder->buildSqlOr( $status, "p.post_status = '%s'" );
-		return $this->db->get_col("
+		return $this->db->get_col( "
 			SELECT DISTINCT pm.meta_value FROM {$this->db->postmeta} pm
 			LEFT JOIN {$this->db->posts} p ON p.ID = pm.post_id
 			WHERE p.post_type = '{$this->postType}'
 			AND ({$keys})
 			AND ({$status})
 			ORDER BY pm.meta_value
-		");
+		" );
 	}
 }

plugin/Defaults/CreateReviewDefaults.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 			'author' => '',
 			'avatar' => '',
 			'content' => '',
-			'date' => get_date_from_gmt( gmdate( 'Y-m-d H:i:s' )),
+			'date' => get_date_from_gmt( gmdate( 'Y-m-d H:i:s' ) ),
 			'email' => '',
 			'ip_address' => '',
 			'pinned' => false,

activate.php

Spacing +10 added lines, -10 removed lines

@@ -32,8 +32,8 @@ 
 	 */
 	public static function isPhpValid( $version = '' )
 	{
-		if( !empty( $version )) {
-			static::normalize( array( 'php' => $version ));
+		if( !empty($version) ) {
+			static::normalize( array( 'php' => $version ) );
 		}
 		return !version_compare( PHP_VERSION, static::$versions->php, '<' );
 	}
@@ -43,7 +43,7 @@ 
 	 */
 	public static function isValid( array $args = array() )
 	{
-		if( !empty( $args )) {
+		if( !empty($args) ) {
 			static::normalize( $args );
 		}
 		return static::isPhpValid() && static::isWpValid();
@@ -56,8 +56,8 @@ 
 	public static function isWpValid( $version = '' )
 	{
 		global $wp_version;
-		if( !empty( $version )) {
-			static::normalize( array( 'wordpress' => $version ));
+		if( !empty($version) ) {
+			static::normalize( array( 'wordpress' => $version ) );
 		}
 		return !version_compare( $wp_version, static::$versions->wordpress, '<' );
 	}
@@ -68,14 +68,14 @@ 
 	 */
 	public static function shouldDeactivate( $file, array $args = array() )
 	{
-		if( empty( static::$instance )) {
+		if( empty(static::$instance) ) {
 			static::$file = realpath( $file );
 			static::$instance = new static;
 			static::$versions = static::normalize( $args );
 		}
 		if( !static::isValid() ) {
-			add_action( 'activated_plugin', array( static::$instance, 'deactivate' ));
-			add_action( 'admin_notices', array( static::$instance, 'deactivate' ));
+			add_action( 'activated_plugin', array( static::$instance, 'deactivate' ) );
+			add_action( 'admin_notices', array( static::$instance, 'deactivate' ) );
 			return true;
 		}
 		return false;
@@ -105,7 +105,7 @@ 
 		return (object)wp_parse_args( $args, array(
 			'php' => static::MIN_PHP_VERSION,
 			'wordpress' => static::MIN_WORDPRESS_VERSION,
-		));
+		) );
 	}
 
 	/**
@@ -117,7 +117,7 @@ 
 			filter_input( INPUT_GET, 'plugin_status' ),
 			filter_input( INPUT_GET, 'paged' ),
 			filter_input( INPUT_GET, 's' )
-		)));
+		) ) );
 		exit;
 	}
 

Braces +3 added lines, -1 removed lines

@@ -87,7 +87,9 @@
 	 */
 	public function deactivate( $plugin )
 	{
-		if( static::isValid() )return;
+		if( static::isValid() ) {
+			return;
+		}
 		$pluginSlug = plugin_basename( static::$file );
 		if( $plugin == $pluginSlug ) {
 			$this->redirect(); //exit