pryley / site-reviews

plugin/Handlers/RegisterTinymcePopups.php

Spacing +3 added lines, -3 removed lines

@@ -14,14 +14,14 @@
 	{
 		foreach( $command->popups as $slug => $label ) {
 			$buttonClass = glsr( Helper::class )->buildClassName( $slug.'-popup', 'Shortcodes' );
-			if( !class_exists( $buttonClass )) {
-				glsr_log()->error( sprintf( 'Class missing (%s)', $buttonClass ));
+			if( !class_exists( $buttonClass ) ) {
+				glsr_log()->error( sprintf( 'Class missing (%s)', $buttonClass ) );
 				continue;
 			}
 			$shortcode = glsr( $buttonClass )->register( $slug, [
 				'label' => $label,
 				'title' => $label,
-			]);
+			] );
 			glsr()->mceShortcodes[$slug] = $shortcode->properties;
 		}
 	}

plugin/Handlers/EnqueuePublicAssets.php

Braces +9 added lines, -3 removed lines

@@ -55,7 +55,9 @@ 
 	 */
 	public function enqueuePolyfillService()
 	{
-		if( !apply_filters( 'site-reviews/assets/polyfill', true ))return;
+		if( !apply_filters( 'site-reviews/assets/polyfill', true )) {
+			return;
+		}
 		wp_enqueue_script( Application::ID.'/polyfill', add_query_arg([
 			'features' => 'CustomEvent,Element.prototype.closest,Element.prototype.dataset,Event,MutationObserver',
 			'flags' => 'gated',
@@ -70,7 +72,9 @@ 
 		// wpforms-recaptcha
 		// google-recaptcha
 		// nf-google-recaptcha
-		if( !glsr( OptionManager::class )->isRecaptchaEnabled() )return;
+		if( !glsr( OptionManager::class )->isRecaptchaEnabled() ) {
+			return;
+		}
 		$language = apply_filters( 'site-reviews/recaptcha/language', get_locale() );
 		wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg([
 			'hl' => $language,
@@ -84,7 +88,9 @@ 
 	public function inlineStyles()
 	{
 		$inlineStylesheetPath = glsr()->path( 'assets/styles/inline-styles.css' );
-		if( !apply_filters( 'site-reviews/assets/css', true ))return;
+		if( !apply_filters( 'site-reviews/assets/css', true )) {
+			return;
+		}
 		if( !file_exists( $inlineStylesheetPath )) {
 			glsr_log()->error( 'Inline stylesheet is missing: '.$inlineStylesheetPath );
 			return;

Spacing +12 added lines, -12 removed lines

@@ -27,7 +27,7 @@ 
 	 */
 	public function enqueueAssets()
 	{
-		if( apply_filters( 'site-reviews/assets/css', true )) {
+		if( apply_filters( 'site-reviews/assets/css', true ) ) {
 			wp_enqueue_style(
 				Application::ID,
 				$this->getStylesheet(),
@@ -35,7 +35,7 @@ 
 				glsr()->version
 			);
 		}
-		if( apply_filters( 'site-reviews/assets/js', true )) {
+		if( apply_filters( 'site-reviews/assets/js', true ) ) {
 			$dependencies = apply_filters( 'site-reviews/assets/polyfill', true )
 				? [Application::ID.'/polyfill']
 				: [];
@@ -55,11 +55,11 @@ 
 	 */
 	public function enqueuePolyfillService()
 	{
-		if( !apply_filters( 'site-reviews/assets/polyfill', true ))return;
-		wp_enqueue_script( Application::ID.'/polyfill', add_query_arg([
+		if( !apply_filters( 'site-reviews/assets/polyfill', true ) )return;
+		wp_enqueue_script( Application::ID.'/polyfill', add_query_arg( [
 			'features' => 'CustomEvent,Element.prototype.closest,Element.prototype.dataset,Event,MutationObserver',
 			'flags' => 'gated',
-		], 'https://cdn.polyfill.io/v2/polyfill.min.js' ));
+		], 'https://cdn.polyfill.io/v2/polyfill.min.js' ) );
 	}
 
 	/**
@@ -72,10 +72,10 @@ 
 		// nf-google-recaptcha
 		if( !glsr( OptionManager::class )->isRecaptchaEnabled() )return;
 		$language = apply_filters( 'site-reviews/recaptcha/language', get_locale() );
-		wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg([
+		wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg( [
 			'hl' => $language,
 			'render' => 'explicit',
-		], 'https://www.google.com/recaptcha/api.js' ));
+		], 'https://www.google.com/recaptcha/api.js' ) );
 	}
 
 	/**
@@ -101,8 +101,8 @@ 
 	public function inlineStyles()
 	{
 		$inlineStylesheetPath = glsr()->path( 'assets/styles/inline-styles.css' );
-		if( !apply_filters( 'site-reviews/assets/css', true ))return;
-		if( !file_exists( $inlineStylesheetPath )) {
+		if( !apply_filters( 'site-reviews/assets/css', true ) )return;
+		if( !file_exists( $inlineStylesheetPath ) ) {
 			glsr_log()->error( 'Inline stylesheet is missing: '.$inlineStylesheetPath );
 			return;
 		}
@@ -122,7 +122,7 @@ 
 	{
 		$script = 'window.hasOwnProperty("GLSR")||(window.GLSR={});';
 		foreach( $variables as $key => $value ) {
-			$script.= sprintf( 'GLSR.%s=%s;', $key, json_encode( $value, JSON_UNESCAPED_SLASHES|JSON_UNESCAPED_UNICODE ));
+			$script .= sprintf( 'GLSR.%s=%s;', $key, json_encode( $value, JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE ) );
 		}
 		$pattern = '/\"([^ \-\"]+)\"(:[{\[\"])/'; // removes unnecessary quotes surrounding object keys
 		$optimizedScript = preg_replace( $pattern, '$1$2', $script );
@@ -134,7 +134,7 @@ 
 	 */
 	protected function getFixedSelectorsForPagination()
 	{
-		$selectors = ['#wpadminbar','.site-navigation-fixed'];
+		$selectors = ['#wpadminbar', '.site-navigation-fixed'];
 		return apply_filters( 'site-reviews/enqueue/public/localize/ajax-pagination', $selectors );
 	}
 
@@ -144,7 +144,7 @@ 
 	protected function getStylesheet()
 	{
 		$currentStyle = glsr( Style::class )->style;
-		return file_exists( glsr()->path( 'assets/styles/custom/'.$currentStyle.'.css' ))
+		return file_exists( glsr()->path( 'assets/styles/custom/'.$currentStyle.'.css' ) )
 			? glsr()->url( 'assets/styles/custom/'.$currentStyle.'.css' )
 			: glsr()->url( 'assets/styles/'.Application::ID.'.css' );
 	}

plugin/Router.php

Braces +15 added lines, -5 removed lines

@@ -17,7 +17,9 @@ 
 	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 );
 	}
@@ -39,10 +41,16 @@ 
 	 */
 	public function routePublicPostRequest()
 	{
-		if( is_admin() )return;
+		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 );
 	}
 
@@ -51,7 +59,9 @@ 
 	 */
 	protected function checkAjaxNonce( array $request )
 	{
-		if( !is_user_logged_in() )return;
+		if( !is_user_logged_in() ) {
+			return;
+		}
 		if( !isset( $request['_nonce'] )) {
 			$this->sendAjaxError( 'request is missing a nonce', $request );
 		}

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,
-		]);
+		] );
 	}
 }

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

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>

Braces +4 added lines, -1 removed lines

@@ -8,7 +8,10 @@
 	<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">

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

Braces +3 added lines, -1 removed lines

@@ -13,7 +13,9 @@
 	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() ));
 		}

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' ) );
 	}
 
 	/**

Braces +3 added lines, -1 removed lines

@@ -15,7 +15,9 @@
 	 */
 	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.'"' );

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 ) );
 	}
 
 	/**