pryley / site-reviews

plugin/Router.php

Spacing +13 added lines, -13 removed lines

@@ -16,7 +16,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 );
 	}
@@ -40,8 +40,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 );
 	}
 
@@ -51,10 +51,10 @@ 
 	protected function checkAjaxNonce( array $request )
 	{
 		if( !is_user_logged_in() )return;
-		if( !isset( $request['nonce'] )) {
+		if( !isset($request['nonce']) ) {
 			$this->sendAjaxError( 'The request is missing a nonce', $request );
 		}
-		if( !wp_verify_nonce( $request['nonce'], $request['action'] )) {
+		if( !wp_verify_nonce( $request['nonce'], $request['action'] ) ) {
 			$this->sendAjaxError( 'The request failed the nonce check', $request, 403 );
 		}
 	}
@@ -64,10 +64,10 @@ 
 	 */
 	protected function checkAjaxRequest( array $request )
 	{
-		if( !isset( $request['action'] )) {
+		if( !isset($request['action']) ) {
 			$this->sendAjaxError( 'The request must include an action', $request );
 		}
-		if( empty( $request['ajax_request'] )) {
+		if( empty($request['ajax_request']) ) {
 			$this->sendAjaxError( 'The request is invalid', $request );
 		}
 	}
@@ -84,7 +84,7 @@ 
 			return glsr( Helper::class )->filterInputArray( Application::ID );
 		}
 		$request = glsr( Helper::class )->filterInputArray( 'request' );
-		if( empty( $request )) {
+		if( empty($request) ) {
 			$request = glsr( Helper::class )->filterInputArray( Application::ID );
 		}
 		return $this->normalizeRequest( $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' )->info( $request );
 			return false;
 		}
@@ -132,11 +132,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;
 		}
@@ -153,7 +153,7 @@ 
 	protected function sendAjaxError( $error, array $request, $statusCode = 400 )
 	{
 		glsr_log()->error( $error )->info( $request );
-		wp_send_json_error([
+		wp_send_json_error( [
 			'message' => __( 'The form could not be submitted. Please notify the site administrator.', 'site-reviews' ),
 			'error' => $error,
 		], $statusCode );

plugin/Controllers/AjaxController.php

Spacing +16 added lines, -16 removed lines

@@ -22,7 +22,7 @@ 
 	 */
 	public function routerChangeReviewStatus( array $request )
 	{
-		wp_send_json_success( $this->execute( new ChangeStatus( $request )));
+		wp_send_json_success( $this->execute( new ChangeStatus( $request ) ) );
 	}
 
 	/**
@@ -31,10 +31,10 @@ 
 	public function routerClearConsole()
 	{
 		glsr( AdminController::class )->routerClearConsole();
-		wp_send_json_success([
+		wp_send_json_success( [
 			'console' => glsr( Console::class )->get(),
 			'notices' => glsr( Notice::class )->get(),
-		]);
+		] );
 	}
 
 	/**
@@ -43,9 +43,9 @@ 
 	public function routerCountReviews()
 	{
 		glsr( AdminController::class )->routerCountReviews();
-		wp_send_json_success([
+		wp_send_json_success( [
 			'notices' => glsr( Notice::class )->get(),
-		]);
+		] );
 	}
 
 	/**
@@ -55,9 +55,9 @@ 
 	{
 		$shortcode = $request['shortcode'];
 		$response = false;
-		if( array_key_exists( $shortcode, glsr()->mceShortcodes )) {
+		if( array_key_exists( $shortcode, glsr()->mceShortcodes ) ) {
 			$data = glsr()->mceShortcodes[$shortcode];
-			if( !empty( $data['errors'] )) {
+			if( !empty($data['errors']) ) {
 				$data['btn_okay'] = [esc_html__( 'Okay', 'site-reviews' )];
 			}
 			$response = [
@@ -77,10 +77,10 @@ 
 	public function routerSearchPosts( array $request )
 	{
 		$results = glsr( Database::class )->searchPosts( $request['search'] );
-		wp_send_json_success([
+		wp_send_json_success( [
 			'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>',
 			'items' => $results,
-		]);
+		] );
 	}
 
 	/**
@@ -88,7 +88,7 @@ 
 	 */
 	public function routerSearchTranslations( array $request )
 	{
-		if( empty( $request['exclude'] )) {
+		if( empty($request['exclude']) ) {
 			$request['exclude'] = [];
 		}
 		$results = glsr( Translator::class )
@@ -96,10 +96,10 @@ 
 			->exclude()
 			->exclude( $request['exclude'] )
 			->renderResults();
-		wp_send_json_success([
+		wp_send_json_success( [
 			'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>',
 			'items' => $results,
-		]);
+		] );
 	}
 
 	/**
@@ -113,7 +113,7 @@ 
 			'message' => glsr( Session::class )->get( $request['form_id'].'message', '', true ),
 			'recaptcha' => glsr( Session::class )->get( $request['form_id'].'recaptcha', false, true ),
 		];
-		if( !wp_validate_boolean( $data['errors'] )) {
+		if( !wp_validate_boolean( $data['errors'] ) ) {
 			wp_send_json_success( $data );
 		}
 		wp_send_json_error( $data );
@@ -124,10 +124,10 @@ 
 	 */
 	public function routerTogglePinned( array $request )
 	{
-		$isPinned = $this->execute( new TogglePinned( $request ));
-		wp_send_json_success([
+		$isPinned = $this->execute( new TogglePinned( $request ) );
+		wp_send_json_success( [
 			'notices' => glsr( Notice::class )->get(),
 			'pinned' => $isPinned,
-		]);
+		] );
 	}
 }

plugin/Controllers/AdminController.php

Spacing +28 added lines, -28 removed lines

@@ -23,7 +23,7 @@ 
 	 */
 	public function enqueueAssets()
 	{
-		$command = new EnqueueAdminAssets([
+		$command = new EnqueueAdminAssets( [
 			'pointers' => [[
 				'content' => __( 'You can pin exceptional reviews so that they are always shown first.', 'site-reviews' ),
 				'id' => 'glsr-pointer-pinned',
@@ -35,7 +35,7 @@ 
 				'target' => '#misc-pub-pinned',
 				'title' => __( 'Pin Your Reviews', 'site-reviews' ),
 			]],
-		]);
+		] );
 		$this->execute( $command );
 	}
 
@@ -47,7 +47,7 @@ 
 	{
 		$links['settings'] = glsr( Builder::class )->a( __( 'Settings', 'site-reviews' ), [
 			'href' => admin_url( 'edit.php?post_type='.Application::POST_TYPE.'&page=settings' ),
-		]);
+		] );
 		return $links;
 	}
 
@@ -58,19 +58,19 @@ 
 	public function filterDashboardGlanceItems( array $items )
 	{
 		$postCount = wp_count_posts( Application::POST_TYPE );
-		if( empty( $postCount->publish )) {
+		if( empty($postCount->publish) ) {
 			return $items;
 		}
 		$text = _n( '%s Review', '%s Reviews', $postCount->publish, 'site-reviews' );
-		$text = sprintf( $text, number_format_i18n( $postCount->publish ));
+		$text = sprintf( $text, number_format_i18n( $postCount->publish ) );
 		$items[] = current_user_can( get_post_type_object( Application::POST_TYPE )->cap->edit_posts )
 			? glsr( Builder::class )->a( $text, [
 				'class' => 'glsr-review-count',
 				'href' => 'edit.php?post_type='.Application::POST_TYPE,
-			])
+			] )
 			: glsr( Builder::class )->span( $text, [
 				'class' => 'glsr-review-count',
-			]);
+			] );
 		return $items;
 	}
 
@@ -81,7 +81,7 @@ 
 	public function filterTinymcePlugins( array $plugins )
 	{
 		if( user_can_richedit()
-			&& ( current_user_can( 'edit_posts' ) || current_user_can( 'edit_pages' ))) {
+			&& (current_user_can( 'edit_posts' ) || current_user_can( 'edit_pages' )) ) {
 			$plugins['glsr_shortcode'] = glsr()->url( 'assets/scripts/mce-plugin.js' );
 		}
 		return $plugins;
@@ -93,11 +93,11 @@ 
 	 */
 	public function registerShortcodeButtons()
 	{
-		$command = new RegisterShortcodeButtons([
+		$command = new RegisterShortcodeButtons( [
 			'site_reviews' => esc_html__( 'Recent Reviews', 'site-reviews' ),
 			'site_reviews_form' => esc_html__( 'Submit a Review', 'site-reviews' ),
 			'site_reviews_summary' => esc_html__( 'Summary of Reviews', 'site-reviews' ),
-		]);
+		] );
 		$this->execute( $command );
 	}
 
@@ -107,10 +107,10 @@ 
 	 */
 	public function renderReviewEditor( WP_Post $post )
 	{
-		if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post ))return;
+		if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post ) )return;
 		glsr()->render( 'partials/editor/review', [
 			'post' => $post,
-		]);
+		] );
 	}
 
 	/**
@@ -119,13 +119,13 @@ 
 	 */
 	public function renderReviewNotice( WP_Post $post )
 	{
-		if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post ))return;
-		glsr( Notice::class )->addWarning( __( 'This review is read-only.', 'site-reviews' ));
+		if( !$this->isReviewPostType( $post ) || $this->isReviewEditable( $post ) )return;
+		glsr( Notice::class )->addWarning( __( 'This review is read-only.', 'site-reviews' ) );
 		glsr( Html::class )->renderTemplate( 'partials/editor/notice', [
 			'context' => [
 				'notices' => glsr( Notice::class )->get(),
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -137,13 +137,13 @@ 
 		if( glsr_current_screen()->base != 'post' )return;
 		$shortcodes = [];
 		foreach( glsr()->mceShortcodes as $shortcode => $values ) {
-			if( !apply_filters( sanitize_title( $shortcode ).'_condition', true ))continue;
+			if( !apply_filters( sanitize_title( $shortcode ).'_condition', true ) )continue;
 			$shortcodes[$shortcode] = $values;
 		}
-		if( empty( $shortcodes ))return;
+		if( empty($shortcodes) )return;
 		glsr()->render( 'partials/editor/tinymce', [
 			'shortcodes' => $shortcodes,
-		]);
+		] );
 	}
 
 	/**
@@ -152,7 +152,7 @@ 
 	public function routerClearConsole()
 	{
 		glsr( Console::class )->clear();
-		glsr( Notice::class )->addSuccess( __( 'Console cleared.', 'site-reviews' ));
+		glsr( Notice::class )->addSuccess( __( 'Console cleared.', 'site-reviews' ) );
 	}
 
 	/**
@@ -161,7 +161,7 @@ 
 	public function routerCountReviews()
 	{
 		// @todo calculate the review counts here
-		glsr( Notice::class )->addSuccess( __( 'Recalculated review counts.', 'site-reviews' ));
+		glsr( Notice::class )->addSuccess( __( 'Recalculated review counts.', 'site-reviews' ) );
 	}
 
 	/**
@@ -195,17 +195,17 @@ 
 	{
 		$file = $_FILES['import-file'];
 		if( $file['error'] !== UPLOAD_ERR_OK ) {
-			return glsr( Notice::class )->addError( $this->getUploadError( $file['error'] ));
+			return glsr( Notice::class )->addError( $this->getUploadError( $file['error'] ) );
 		}
-		if( $file['type'] !== 'application/json' || !glsr( Helper::class )->endsWith( '.json', $file['name'] )) {
-			return glsr( Notice::class )->addError( __( 'Please use a valid Site Reviews settings file.', 'site-reviews' ));
+		if( $file['type'] !== 'application/json' || !glsr( Helper::class )->endsWith( '.json', $file['name'] ) ) {
+			return glsr( Notice::class )->addError( __( 'Please use a valid Site Reviews settings file.', 'site-reviews' ) );
 		}
 		$settings = json_decode( file_get_contents( $file['tmp_name'] ), true );
-		if( empty( $settings )) {
-			return glsr( Notice::class )->addWarning( __( 'There were no settings found to import.', 'site-reviews' ));
+		if( empty($settings) ) {
+			return glsr( Notice::class )->addWarning( __( 'There were no settings found to import.', 'site-reviews' ) );
 		}
-		glsr( OptionManager::class )->set( glsr( OptionManager::class )->normalize( $settings ));
-		glsr( Notice::class )->addSuccess( __( 'Settings imported.', 'site-reviews' ));
+		glsr( OptionManager::class )->set( glsr( OptionManager::class )->normalize( $settings ) );
+		glsr( Notice::class )->addSuccess( __( 'Settings imported.', 'site-reviews' ) );
 	}
 
 	/**
@@ -223,7 +223,7 @@ 
 			UPLOAD_ERR_CANT_WRITE => __( 'Failed to write file to disk.', 'site-reviews' ),
 			UPLOAD_ERR_EXTENSION => __( 'A PHP extension stopped the file upload.', 'site-reviews' ),
 		];
-		return !isset( $errors[$errorCode] )
+		return !isset($errors[$errorCode])
 			? __( 'Unknown upload error.', 'site-reviews' )
 			: $errors[$errorCode];
 	}

plugin/Modules/Upgrader.php

Braces +3 added lines, -1 removed lines

@@ -16,7 +16,9 @@
 		natsort( $routines );
 		array_walk( $routines, function( $routine ) {
 			$version = str_replace( strtolower( __CLASS__ ).'_', '', $routine );
-			if( version_compare( glsr()->version, $version, '>=' ))return;
+			if( version_compare( glsr()->version, $version, '>=' )) {
+				return;
+			}
 			call_user_func( [$this, $routine] );
 		});
 		$this->updateVersion();