pryley / site-reviews

plugin/Modules/Html/Template.php

Spacing +2 added lines, -2 removed lines

@@ -51,9 +51,9 @@
 	protected function normalize( array $data )
 	{
 		$arrayKeys = ['context', 'globals'];
-		$data = wp_parse_args( $data, array_fill_keys( $arrayKeys, [] ));
+		$data = wp_parse_args( $data, array_fill_keys( $arrayKeys, [] ) );
 		foreach( $arrayKeys as $key ) {
-			if( is_array( $data[$key] ))continue;
+			if( is_array( $data[$key] ) )continue;
 			$data[$key] = [];
 		}
 		return $data;

Braces +3 added lines, -1 removed lines

@@ -53,7 +53,9 @@
 		$arrayKeys = ['context', 'globals'];
 		$data = wp_parse_args( $data, array_fill_keys( $arrayKeys, [] ));
 		foreach( $arrayKeys as $key ) {
-			if( is_array( $data[$key] ))continue;
+			if( is_array( $data[$key] )) {
+				continue;
+			}
 			$data[$key] = [];
 		}
 		return $data;

plugin/Modules/Html/Fields/Honeypot.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 	{
 		$this->builder->args = wp_parse_args( $this->builder->args, [
 			'name' => $this->builder->args['text'],
-		]);
+		] );
 		$this->builder->tag = 'input';
 		$this->mergeFieldArgs();
 		return $this->builder->getOpeningTag();

plugin/Handlers/EnqueueAdminAssets.php

Spacing +5 added lines, -5 removed lines

@@ -44,7 +44,7 @@ 
 			glsr()->version,
 			true
 		);
-		if( !empty( $this->pointers )) {
+		if( !empty($this->pointers) ) {
 			wp_enqueue_style( 'wp-pointer' );
 			wp_enqueue_script( 'wp-pointer' );
 		}
@@ -84,7 +84,7 @@ 
 		$dependencies = apply_filters( 'site-reviews/enqueue/admin/dependencies', [] );
 		$dependencies = array_merge( $dependencies, [
 			'jquery', 'jquery-ui-sortable', 'underscore', 'wp-util',
-		]);
+		] );
 		return $dependencies;
 	}
 
@@ -114,7 +114,7 @@ 
 		$generatedPointers = [];
 		foreach( $pointers as $pointer ) {
 			if( $pointer['screen'] != glsr_current_screen()->id )continue;
-			if( in_array( $pointer['id'], $dismissedPointers ))continue;
+			if( in_array( $pointer['id'], $dismissedPointers ) )continue;
 			$generatedPointers[] = $this->generatePointer( $pointer );
 		}
 		$this->pointers = $generatedPointers;
@@ -126,7 +126,7 @@ 
 	protected function isCurrentScreen()
 	{
 		$screen = glsr_current_screen();
-		return $screen && ( $screen->post_type == Application::POST_TYPE
+		return $screen && ($screen->post_type == Application::POST_TYPE
 			|| $screen->base == 'post'
 			|| $screen->id == 'dashboard'
 			|| $screen->id == 'widgets'
@@ -140,7 +140,7 @@ 
 	{
 		$variables = [];
 		foreach( glsr()->mceShortcodes as $tag => $args ) {
-			if( empty( $args['required'] ))continue;
+			if( empty($args['required']) )continue;
 			$variables[$tag] = $args['required'];
 		}
 		return $variables;

Braces +12 added lines, -4 removed lines

@@ -36,7 +36,9 @@ 
 			[],
 			glsr()->version
 		);
-		if( !$this->isCurrentScreen() )return;
+		if( !$this->isCurrentScreen() ) {
+			return;
+		}
 		wp_enqueue_script(
 			Application::ID,
 			glsr()->url( 'assets/scripts/'.Application::ID.'-admin.js' ),
@@ -113,8 +115,12 @@ 
 		$dismissedPointers = explode( ',', (string)$dismissedPointers );
 		$generatedPointers = [];
 		foreach( $pointers as $pointer ) {
-			if( $pointer['screen'] != glsr_current_screen()->id )continue;
-			if( in_array( $pointer['id'], $dismissedPointers ))continue;
+			if( $pointer['screen'] != glsr_current_screen()->id ) {
+				continue;
+			}
+			if( in_array( $pointer['id'], $dismissedPointers )) {
+				continue;
+			}
 			$generatedPointers[] = $this->generatePointer( $pointer );
 		}
 		$this->pointers = $generatedPointers;
@@ -140,7 +146,9 @@ 
 	{
 		$variables = [];
 		foreach( glsr()->mceShortcodes as $tag => $args ) {
-			if( empty( $args['required'] ))continue;
+			if( empty( $args['required'] )) {
+				continue;
+			}
 			$variables[$tag] = $args['required'];
 		}
 		return $variables;

autoload.php

Spacing +3 added lines, -3 removed lines

@@ -2,7 +2,7 @@ 
 
 defined( 'WPINC' ) || die;
 
-require_once( ABSPATH.WPINC.'/class-phpass.php' );
+require_once(ABSPATH.WPINC.'/class-phpass.php');
 
 spl_autoload_register( function( $className ) {
 	$namespaces = [
@@ -16,8 +16,8 @@ 
 	foreach( $namespaces as $prefix => $baseDir ) {
 		$len = strlen( $prefix );
 		if( strncmp( $prefix, $className, $len ) !== 0 )continue;
-		$file = $baseDir.str_replace( '\\', '/', substr( $className, $len )).'.php';
-		if( !file_exists( $file ))continue;
+		$file = $baseDir.str_replace( '\\', '/', substr( $className, $len ) ).'.php';
+		if( !file_exists( $file ) )continue;
 		require $file;
 		break;
 	}

Braces +6 added lines, -2 removed lines

@@ -15,9 +15,13 @@
 	];
 	foreach( $namespaces as $prefix => $baseDir ) {
 		$len = strlen( $prefix );
-		if( strncmp( $prefix, $className, $len ) !== 0 )continue;
+		if( strncmp( $prefix, $className, $len ) !== 0 ) {
+			continue;
+		}
 		$file = $baseDir.str_replace( '\\', '/', substr( $className, $len )).'.php';
-		if( !file_exists( $file ))continue;
+		if( !file_exists( $file )) {
+			continue;
+		}
 		require $file;
 		break;
 	}

plugin/Controllers/PublicController.php

Braces +3 added lines, -1 removed lines

@@ -50,7 +50,9 @@
 	public function routerSubmitReview( array $request )
 	{
 		$validated = glsr( ValidateReview::class )->validate( $request );
-		if( !empty( $validated->error ) || $validated->recaptchaIsUnset )return;
+		if( !empty( $validated->error ) || $validated->recaptchaIsUnset ) {
+			return;
+		}
 		$this->execute( new CreateReview( $validated->request ));
 	}
 }

Spacing +4 added lines, -4 removed lines

@@ -32,10 +32,10 @@ 
 	public function filterEnqueuedScripts( $tag, $handle )
 	{
 		$scripts = [Application::ID.'/google-recaptcha'];
-		if( in_array( $handle, apply_filters( 'site-reviews/async-scripts', $scripts ))) {
+		if( in_array( $handle, apply_filters( 'site-reviews/async-scripts', $scripts ) ) ) {
 			$tag = str_replace( ' src=', ' async src=', $tag );
 		}
-		if( in_array( $handle, apply_filters( 'site-reviews/defer-scripts', $scripts ))) {
+		if( in_array( $handle, apply_filters( 'site-reviews/defer-scripts', $scripts ) ) ) {
 			$tag = str_replace( ' src=', ' defer src=', $tag );
 		}
 		return $tag;
@@ -85,7 +85,7 @@ 
 	public function routerSubmitReview( array $request )
 	{
 		$validated = glsr( ValidateReview::class )->validate( $request );
-		if( !empty( $validated->error ) || $validated->recaptchaIsUnset )return;
-		$this->execute( new CreateReview( $validated->request ));
+		if( !empty($validated->error) || $validated->recaptchaIsUnset )return;
+		$this->execute( new CreateReview( $validated->request ) );
 	}
 }

plugin/Modules/Html/Field.php

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()

Spacing +20 added lines, -20 removed lines

@@ -27,7 +27,7 @@ 
 			'is_valid' => true,
 			'is_widget' => false,
 			'path' => '',
-		]);
+		] );
 		$this->normalize();
 	}
 
@@ -46,7 +46,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();
@@ -75,15 +75,15 @@ 
 				'class' => $this->getFieldClass(),
 				'errors' => $this->getFieldErrors(),
 				'field' => glsr( Builder::class )->raw( $this->field ),
-				'label' => glsr( Builder::class )->label([
+				'label' => glsr( Builder::class )->label( [
 					'class' => 'glsr-'.$this->field['type'].'-label',
 					'for' => $this->field['id'],
 					'is_public' => $this->field['is_public'],
 					'text' => $this->field['label'].'<span></span>',
 					'type' => $this->field['type'],
-				]),
+				] ),
 			],
-		]);
+		] );
 		return apply_filters( 'site-reviews/rendered/field', $field, $this->field['type'], $this->field );
 	}
 
@@ -95,10 +95,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']] ),
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -107,16 +107,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'],
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -125,13 +125,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 );
@@ -142,7 +142,7 @@ 
 	 */
 	protected function getFieldDependsOn()
 	{
-		return !empty( $this->field['data-depends'] )
+		return !empty($this->field['data-depends'])
 			? $this->field['data-depends']
 			: '';
 	}
@@ -152,7 +152,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'] );
 		});
@@ -160,7 +160,7 @@ 
 			'context' => [
 				'errors' => $errors,
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -183,13 +183,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'];
@@ -203,7 +203,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()
@@ -218,7 +218,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()

views/partials/editor/metabox-categories.php

Spacing +3 added lines, -3 removed lines

@@ -16,11 +16,11 @@ 
 	<div id="<?= $tax_name; ?>-all" class="tabs-panel">
 		<input type="hidden" name="tax_input[<?= $tax_name; ?>][]" value='0' />
 		<ul id="<?= $tax_name; ?>checklist" data-wp-lists="list:<?= $tax_name; ?>" class="categorychecklist form-no-clear">
-			<?php wp_terms_checklist( $post->ID, array( 'taxonomy' => $tax_name, 'popular_cats' => $popular_ids )); ?>
+			<?php wp_terms_checklist( $post->ID, array( 'taxonomy' => $tax_name, 'popular_cats' => $popular_ids ) ); ?>
 		</ul>
 	</div>
 
-	<?php if( current_user_can( $taxonomy->cap->edit_terms )) : ?>
+	<?php if( current_user_can( $taxonomy->cap->edit_terms ) ) : ?>
 	<div id="<?= $tax_name; ?>-adder" class="wp-hidden-children">
 		<a id="<?= $tax_name; ?>-add-toggle" href="#<?= $tax_name; ?>-add" class="hide-if-no-js taxonomy-add-new">
 			<?= sprintf( '+ %s', $taxonomy->labels->add_new_item ); ?>
@@ -29,7 +29,7 @@ 
 			<label class="screen-reader-text" for="new<?= $tax_name; ?>"><?= $taxonomy->labels->add_new_item; ?></label>
 			<input type="text" name="new<?= $tax_name; ?>" id="new<?= $tax_name; ?>" class="form-required form-input-tip" value="<?= esc_attr( $taxonomy->labels->new_item_name ); ?>" aria-required="true"/>
 			<input type="button" id="<?= $tax_name; ?>-add-submit" data-wp-lists="add:<?= $tax_name; ?>checklist:<?= $tax_name; ?>-add" class="button category-add-submit" value="<?= esc_attr( $taxonomy->labels->add_new_item ); ?>" />
-			<?php wp_nonce_field( 'add-' . $tax_name, '_ajax_nonce-add-' . $tax_name, false ); ?>
+			<?php wp_nonce_field( 'add-'.$tax_name, '_ajax_nonce-add-'.$tax_name, false ); ?>
 			<span id="<?= $tax_name; ?>-ajax-response"></span>
 		</div>
 	</div>

views/partials/editor/review.php

Spacing +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?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">

plugin/Controllers/ListTableController/Columns.php

Braces +12 added lines, -4 removed lines

@@ -86,7 +86,9 @@ 
 	 */
 	public function renderFilters( $postType )
 	{
-		if( $postType !== Application::POST_TYPE )return;
+		if( $postType !== Application::POST_TYPE ) {
+			return;
+		}
 		if( !( $status = filter_input( INPUT_GET, 'post_status' ))) {
 			$status = 'publish';
 		}
@@ -103,7 +105,9 @@ 
 	 */
 	public function renderValues( $column, $postId )
 	{
-		if( glsr_current_screen()->post_type != Application::POST_TYPE )return;
+		if( glsr_current_screen()->post_type != Application::POST_TYPE ) {
+			return;
+		}
 		$method = glsr( Helper::class )->buildMethodName( $column, 'buildColumn' );
 		echo method_exists( $this, $method )
 			? call_user_func( [$this, $method], $postId )
@@ -116,7 +120,9 @@ 
 	 */
 	protected function renderFilterRatings( $ratings )
 	{
-		if( empty( $ratings ))return;
+		if( empty( $ratings )) {
+			return;
+		}
 		$ratings = array_flip( array_reverse( $ratings ));
 		array_walk( $ratings, function( &$value, $key ) {
 			$label = _n( '%s star', '%s stars', $key, 'site-reviews' );
@@ -139,7 +145,9 @@ 
 	 */
 	protected function renderFilterTypes( $types )
 	{
-		if( count( glsr()->reviewTypes ) < 2 )return;
+		if( count( glsr()->reviewTypes ) < 2 ) {
+			return;
+		}
 		echo glsr( Builder::class )->label( __( 'Filter by type', 'site-reviews' ), [
 			'class' => 'screen-reader-text',
 			'for' => 'review_type',

Spacing +15 added lines, -15 removed lines

@@ -22,7 +22,7 @@ 
 		if( $assignedPost instanceof WP_Post && $assignedPost->post_status == 'publish' ) {
 			$column = glsr( Builder::class )->a( get_the_title( $assignedPost->ID ), [
 				'href' => (string)get_the_permalink( $assignedPost->ID ),
-			]);
+			] );
 		}
 		return $column;
 	}
@@ -36,10 +36,10 @@ 
 		$pinned = get_post_meta( $postId, 'pinned', true )
 			? 'pinned '
 			: '';
-		return glsr( Builder::class )->i([
+		return glsr( Builder::class )->i( [
 			'class' => $pinned.'dashicons dashicons-sticky',
 			'data-id' => $postId,
-		]);
+		] );
 	}
 
 	/**
@@ -48,7 +48,7 @@ 
 	 */
 	public function buildColumnReviewer( $postId )
 	{
-		return strval( get_post_meta( $postId, 'author', true ));
+		return strval( get_post_meta( $postId, 'author', true ) );
 	}
 
 	/**
@@ -59,8 +59,8 @@ 
 	public function buildColumnRating( $postId )
 	{
 		return glsr( Partial::class )->build( 'star-rating', [
-			'rating' => intval( get_post_meta( $postId, 'rating', true )),
-		]);
+			'rating' => intval( get_post_meta( $postId, 'rating', true ) ),
+		] );
 	}
 
 	/**
@@ -82,7 +82,7 @@ 
 	public function renderFilters( $postType )
 	{
 		if( $postType !== Application::POST_TYPE )return;
-		if( !( $status = filter_input( INPUT_GET, 'post_status' ))) {
+		if( !($status = filter_input( INPUT_GET, 'post_status' )) ) {
 			$status = 'publish';
 		}
 		$ratings = glsr( Database::class )->getReviewsMeta( 'rating', $status );
@@ -111,8 +111,8 @@ 
 	 */
 	protected function renderFilterRatings( $ratings )
 	{
-		if( empty( $ratings ))return;
-		$ratings = array_flip( array_reverse( $ratings ));
+		if( empty($ratings) )return;
+		$ratings = array_flip( array_reverse( $ratings ) );
 		array_walk( $ratings, function( &$value, $key ) {
 			$label = _n( '%s star', '%s stars', $key, 'site-reviews' );
 			$value = sprintf( $label, $key );
@@ -120,12 +120,12 @@ 
 		echo glsr( Builder::class )->label( __( 'Filter by rating', 'site-reviews' ), [
 			'class' => 'screen-reader-text',
 			'for' => 'rating',
-		]);
-		echo glsr( Builder::class )->select([
+		] );
+		echo glsr( Builder::class )->select( [
 			'name' => 'rating',
 			'options' => ['' => __( 'All ratings', 'site-reviews' )] + $ratings,
 			'value' => filter_input( INPUT_GET, 'rating' ),
-		]);
+		] );
 	}
 
 	/**
@@ -138,11 +138,11 @@ 
 		echo glsr( Builder::class )->label( __( 'Filter by type', 'site-reviews' ), [
 			'class' => 'screen-reader-text',
 			'for' => 'review_type',
-		]);
-		echo glsr( Builder::class )->select([
+		] );
+		echo glsr( Builder::class )->select( [
 			'name' => 'review_type',
 			'options' => ['' => __( 'All types', 'site-reviews' )] + glsr()->reviewTypes,
 			'value' => filter_input( INPUT_GET, 'review_type' ),
-		]);
+		] );
 	}
 }