pryley / site-reviews

plugin/Modules/Html/Partials/StarRating.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
 	 */
 	public function build( array $args = [] )
 	{
-		require_once( ABSPATH.'wp-admin/includes/template.php' );
+		require_once(ABSPATH.'wp-admin/includes/template.php');
 		ob_start();
 		wp_star_rating( $args );
 		$stars = ob_get_clean();

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;
 		}
@@ -140,10 +140,10 @@ 
 	{
 		glsr_log()->error( $error )->debug( $request );
 		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,
-		]);
+		] );
 	}
 }

views/partials/editor/review.php

Spacing +3 added lines, -3 removed lines

@@ -1,14 +1,14 @@ 
 <?php defined( 'WPINC' ) || die; ?>
 
 <div id="titlediv">
-	<input type="text" id="title" value="<?= $post->post_title ? esc_attr( $post->post_title ) : sprintf( '(%s)', __( 'no title', 'site-reviews' )); ?>" readonly>
+	<input type="text" id="title" value="<?= $post->post_title ? esc_attr( $post->post_title ) : sprintf( '(%s)', __( 'no title', 'site-reviews' ) ); ?>" readonly>
 </div>
 
 <div id="contentdiv">
 	<textarea readonly><?= esc_attr( $post->post_content ); ?></textarea>
 </div>
 
-<?php if( empty( $response ))return; ?>
+<?php if( empty($response) )return; ?>
 
 <div class="postbox glsr-response-postbox">
 	<button type="button" class="handlediv" aria-expanded="true">
@@ -19,6 +19,6 @@ 
 		<span><?= __( 'Public Response', 'site-reviews' ); ?></span>
 	</h2>
 	<div class="inside">
-		<?= wpautop( esc_attr( $response )); ?>
+		<?= wpautop( esc_attr( $response ) ); ?>
 	</div>
 </div>

plugin/Modules/Polylang.php

Spacing +6 added lines, -6 removed lines

@@ -13,14 +13,14 @@ 
 	public function getPost( $postId )
 	{
 		$postId = trim( $postId );
-		if( empty( $postId ) || !is_numeric( $postId ))return;
+		if( empty($postId) || !is_numeric( $postId ) )return;
 		if( $this->isEnabled() ) {
-			$polylangPostId = pll_get_post( $postId, pll_get_post_language( get_the_ID() ));
+			$polylangPostId = pll_get_post( $postId, pll_get_post_language( get_the_ID() ) );
 		}
-		if( !empty( $polylangPostId )) {
+		if( !empty($polylangPostId) ) {
 			$postId = $polylangPostId;
 		}
-		return get_post( intval( $postId ));
+		return get_post( intval( $postId ) );
 	}
 
 	/**
@@ -35,7 +35,7 @@ 
 		foreach( $this->cleanIds( $postIds ) as $postId ) {
 			$newPostIds = array_merge(
 				$newPostIds,
-				array_values( pll_get_post_translations( $postId ))
+				array_values( pll_get_post_translations( $postId ) )
 			);
 		}
 		return $this->cleanIds( $newPostIds );
@@ -75,7 +75,7 @@ 
 	 */
 	protected function cleanIds( array $postIds )
 	{
-		return array_filter( array_unique( $postIds ));
+		return array_filter( array_unique( $postIds ) );
 	}
 }
 

plugin/Controllers/Controller.php

Spacing +5 added lines, -5 removed lines

@@ -15,7 +15,7 @@ 
 	 */
 	public function download( $filename, $content )
 	{
-		if( !current_user_can( glsr()->constant( 'CAPABILITY' )))return;
+		if( !current_user_can( glsr()->constant( 'CAPABILITY' ) ) )return;
 		nocache_headers();
 		header( 'Content-Type: text/plain' );
 		header( 'Content-Disposition: attachment; filename="'.$filename.'"' );
@@ -30,8 +30,8 @@ 
 	 */
 	public function execute( $command )
 	{
-		$handlerClass = str_replace( 'Commands', 'Handlers', get_class( $command ));
-		if( !class_exists( $handlerClass )) {
+		$handlerClass = str_replace( 'Commands', 'Handlers', get_class( $command ) );
+		if( !class_exists( $handlerClass ) ) {
 			throw new InvalidArgumentException( 'Handler '.$handlerClass.' not found.' );
 		}
 		try {
@@ -39,7 +39,7 @@ 
 		}
 		catch( Exception $e ) {
 			status_header( 400 );
-			glsr( Notice::class )->addError( new WP_Error( 'site_reviews_error', $e->getMessage() ));
+			glsr( Notice::class )->addError( new WP_Error( 'site_reviews_error', $e->getMessage() ) );
 			glsr_log()->error( $e->getMessage() );
 		}
 	}
@@ -49,7 +49,7 @@ 
 	 */
 	protected function getPostId()
 	{
-		return intval( filter_input( INPUT_GET, 'post' ));
+		return intval( filter_input( INPUT_GET, 'post' ) );
 	}
 
 	/**

plugin/Database/QueryBuilder.php

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,

plugin/Modules/Translator.php

Spacing +10 added lines, -10 removed lines

@@ -18,7 +18,7 @@ 
 	{
 		return $this->translate( $translation, $domain, [
 			'single' => $text,
-		]);
+		] );
 	}
 
 	/**
@@ -33,7 +33,7 @@ 
 		return $this->translate( $translation, $domain, [
 			'context' => $context,
 			'single' => $text,
-		]);
+		] );
 	}
 
 	/**
@@ -50,7 +50,7 @@ 
 			'number' => $number,
 			'plural' => $plural,
 			'single' => $single,
-		]);
+		] );
 	}
 
 	/**
@@ -69,7 +69,7 @@ 
 			'number' => $number,
 			'plural' => $plural,
 			'single' => $single,
-		]);
+		] );
 	}
 
 	/**
@@ -80,12 +80,12 @@ 
 	public function translate( $original, $domain, array $args )
 	{
 		$domains = apply_filters( 'site-reviews/translator/domains', [Application::ID] );
-		if( !in_array( $domain, $domains )) {
+		if( !in_array( $domain, $domains ) ) {
 			return $original;
 		}
 		$args = $this->normalizeTranslationArgs( $args );
 		$strings = $this->getTranslationStrings( $args['single'], $args['plural'] );
-		if( empty( $strings )) {
+		if( empty($strings) ) {
 			return $original;
 		}
 		$string = current( $strings );
@@ -101,7 +101,7 @@ 
 	 */
 	protected function getTranslationStrings( $single, $plural )
 	{
-		return array_filter( glsr( Translation::class )->translations(), function( $string ) use( $single, $plural ) {
+		return array_filter( glsr( Translation::class )->translations(), function( $string ) use($single, $plural) {
 			return $string['s1'] == html_entity_decode( $single, ENT_COMPAT, 'UTF-8' )
 				&& $string['p1'] == html_entity_decode( $plural, ENT_COMPAT, 'UTF-8' );
 		});
@@ -127,10 +127,10 @@ 
 	 */
 	protected function translatePlural( $domain, array $string, array $args )
 	{
-		if( !empty( $string['s2'] )) {
+		if( !empty($string['s2']) ) {
 			$args['single'] = $string['s2'];
 		}
-		if( !empty( $string['p2'] )) {
+		if( !empty($string['p2']) ) {
 			$args['plural'] = $string['p2'];
 		}
 		return get_translations_for_domain( $domain )->translate_plural(
@@ -147,7 +147,7 @@ 
 	 */
 	protected function translateSingle( $domain, array $string, array $args )
 	{
-		if( !empty( $string['s2'] )) {
+		if( !empty($string['s2']) ) {
 			$args['single'] = $string['s2'];
 		}
 		return get_translations_for_domain( $domain )->translate(

plugin/Modules/Html/Fields/Field.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 	 */
 	public function build()
 	{
-		glsr_log()->error( 'Build method is not implemented for '.get_class( $this ));
+		glsr_log()->error( 'Build method is not implemented for '.get_class( $this ) );
 	}
 
 	/**

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 )
@@ -91,8 +91,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;
 	}
@@ -102,7 +102,7 @@ 
 	 */
 	public function getClosingTag()
 	{
-		if( empty( $this->tag ))return;
+		if( empty($this->tag) )return;
 		return '</'.$this->tag.'>';
 	}
 
@@ -111,8 +111,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 ).'>';
 	}
 
@@ -121,10 +121,10 @@ 
 	 */
 	public function getTag()
 	{
-		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();
@@ -135,8 +135,8 @@ 
 	 */
 	public function raw( array $field )
 	{
-		unset( $field['label'] );
-		return $this->{$field['type']}( $field );
+		unset($field['label']);
+		return $this->{$field['type']}($field);
 	}
 
 	/**
@@ -145,7 +145,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 );
@@ -156,7 +156,7 @@ 
 	 */
 	protected function buildDefaultTag( $text = '' )
 	{
-		if( empty( $text )) {
+		if( empty($text) ) {
 			$text = $this->args['text'];
 		}
 		return $this->getOpeningTag().$text.$this->getClosingTag();
@@ -167,7 +167,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'] );
 		}
@@ -179,13 +179,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();
 	}
@@ -195,19 +195,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>',
-		]);
+		] );
 	}
 
 	/**
@@ -216,22 +216,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'],
-		]);
+		] );
 	}
 
 	/**
@@ -239,13 +239,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'],
-		]));
+		] ) );
 	}
 
 	/**
@@ -262,11 +262,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,
-			]);
+			] );
 		});
 	}
 
@@ -302,7 +302,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()
@@ -317,16 +317,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;
 		}
 	}
@@ -350,7 +350,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';
 		}