pryley / site-reviews

plugin/Database/DefaultsManager.php

Spacing +3 added lines, -3 removed lines

@@ -13,11 +13,11 @@ 
 	public function defaults()
 	{
 		$settings = $this->settings();
-		$defaults = array_combine( array_keys( $settings ), array_column( $settings, 'default' ));
+		$defaults = array_combine( array_keys( $settings ), array_column( $settings, 'default' ) );
 		return wp_parse_args( $defaults, [
 			'version' => '',
 			'version_upgraded_from' => '',
-		]);
+		] );
 	}
 
 	/**
@@ -57,7 +57,7 @@ 
 	protected function normalize( array $settings )
 	{
 		array_walk( $settings, function( &$setting ) {
-			if( isset( $setting['default'] ))return;
+			if( isset($setting['default']) )return;
 			$setting['default'] = '';
 		});
 		return $settings;

plugin/Controllers/Controller.php

Spacing +5 added lines, -5 removed lines

@@ -15,7 +15,7 @@ 
 	 */
 	public function download( $filename, $content )
 	{
-		if( !current_user_can( Application::CAPABILITY ))return;
+		if( !current_user_can( Application::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,6 +49,6 @@ 
 	 */
 	protected function getPostId()
 	{
-		return intval( filter_input( INPUT_GET, 'post' ));
+		return intval( filter_input( INPUT_GET, 'post' ) );
 	}
 }

plugin/Router.php

Braces +6 added lines, -2 removed lines

@@ -16,7 +16,9 @@ 
 	public function routeAdminPostRequest()
 	{
 		$request = filter_input( INPUT_POST, Application::ID, FILTER_DEFAULT, FILTER_REQUIRE_ARRAY );
-		if( !isset( $request['action'] ))return;
+		if( !isset( $request['action'] )) {
+			return;
+		}
 		$this->checkNonce( $request['action'] );
 		$this->routeRequest( 'admin', $request['action'], $request );
 	}
@@ -71,7 +73,9 @@ 
 	public function routeWebhookRequest()
 	{
 		$request = filter_input( INPUT_GET, sprintf( '%s-hook', Application::ID ));
-		if( !$request )return;
+		if( !$request ) {
+			return;
+		}
 		// @todo manage webhook here
 	}
 

Spacing +8 added lines, -8 removed lines

@@ -16,7 +16,7 @@ 
 	public function routeAdminPostRequest()
 	{
 		$request = filter_input( INPUT_POST, Application::ID, FILTER_DEFAULT, FILTER_REQUIRE_ARRAY );
-		if( !isset( $request['action'] ))return;
+		if( !isset($request['action']) )return;
 		$this->checkNonce( $request['action'] );
 		$this->routeRequest( 'admin', $request['action'], $request );
 	}
@@ -27,7 +27,7 @@ 
 	public function routeAjaxRequest()
 	{
 		$request = $this->normalizeAjaxRequest();
-		if( !wp_verify_nonce( $request['nonce'], $request['action'] )) {
+		if( !wp_verify_nonce( $request['nonce'], $request['action'] ) ) {
 			glsr_log()->error( 'Nonce check failed for ajax request' )->info( $request );
 			wp_die( -1, 403 );
 		}
@@ -52,9 +52,9 @@ 
 	 */
 	public function routeRequest( $type, $action, array $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' );
-		if( is_callable( [$controller, $method] )) {
+		if( is_callable( [$controller, $method] ) ) {
 			call_user_func( [$controller, $method], $request );
 			return;
 		}
@@ -70,7 +70,7 @@ 
 	 */
 	public function routeWebhookRequest()
 	{
-		$request = filter_input( INPUT_GET, sprintf( '%s-hook', Application::ID ));
+		$request = filter_input( INPUT_GET, sprintf( '%s-hook', Application::ID ) );
 		if( !$request )return;
 		// @todo manage webhook here
 	}
@@ -109,14 +109,14 @@ 
 	protected function normalizeAjaxRequest()
 	{
 		$request = filter_input( INPUT_POST, 'request', FILTER_DEFAULT, FILTER_REQUIRE_ARRAY );
-		if( isset( $request[Application::ID]['action'] )) {
+		if( isset($request[Application::ID]['action']) ) {
 			$request = $request[Application::ID];
 		}
-		if( !isset( $request['action'] )) {
+		if( !isset($request['action']) ) {
 			glsr_log()->error( 'The AJAX request must include an action' )->info( $request );
 			wp_die();
 		}
-		if( !isset( $request['nonce'] )) {
+		if( !isset($request['nonce']) ) {
 			glsr_log()->error( 'The AJAX request must include a nonce' )->info( $request );
 			wp_die();
 		}

plugin/Database/OptionManager.php

Spacing +6 added lines, -6 removed lines

@@ -24,7 +24,7 @@ 
 	public function all()
 	{
 		$options = get_option( static::databaseKey(), [] );
-		if( !is_array( $options )) {
+		if( !is_array( $options ) ) {
 			delete_option( static::databaseKey() );
 			$options = ['settings' => []];
 		}
@@ -42,10 +42,10 @@ 
 		$options = $this->all();
 		$pointer = &$options;
 		foreach( $keys as $key ) {
-			if( !isset( $pointer[$key] ) || !is_array( $pointer[$key] ))continue;
+			if( !isset($pointer[$key]) || !is_array( $pointer[$key] ) )continue;
 			$pointer = &$pointer[$key];
 		}
-		unset( $pointer[$last] );
+		unset($pointer[$last]);
 		return $this->set( $options );
 	}
 
@@ -77,8 +77,8 @@ 
 			glsr( DefaultsManager::class )->defaults()
 		);
 		array_walk( $options, function( &$value ) {
-			if( !is_string( $value ))return;
-			$value = wp_kses( $value, wp_kses_allowed_html( 'post' ));
+			if( !is_string( $value ) )return;
+			$value = wp_kses( $value, wp_kses_allowed_html( 'post' ) );
 		});
 		return glsr( Helper::class )->convertDotNotationArray( $options );
 	}
@@ -90,7 +90,7 @@ 
 	 */
 	public function set( $pathOrOptions, $value = '' )
 	{
-		if( is_string( $pathOrOptions )) {
+		if( is_string( $pathOrOptions ) ) {
 			$pathOrOptions = glsr( Helper::class )->setPathValue( $pathOrOptions, $value, $this->all() );
 		}
 		return update_option( static::databaseKey(), (array)$pathOrOptions );

Braces +6 added lines, -2 removed lines

@@ -42,7 +42,9 @@ 
 		$options = $this->all();
 		$pointer = &$options;
 		foreach( $keys as $key ) {
-			if( !isset( $pointer[$key] ) || !is_array( $pointer[$key] ))continue;
+			if( !isset( $pointer[$key] ) || !is_array( $pointer[$key] )) {
+				continue;
+			}
 			$pointer = &$pointer[$key];
 		}
 		unset( $pointer[$last] );
@@ -77,7 +79,9 @@ 
 			glsr( DefaultsManager::class )->defaults()
 		);
 		array_walk( $options, function( &$value ) {
-			if( !is_string( $value ))return;
+			if( !is_string( $value )) {
+				return;
+			}
 			$value = wp_kses( $value, wp_kses_allowed_html( 'post' ));
 		});
 		return glsr( Helper::class )->convertDotNotationArray( $options );

plugin/Helper.php

Spacing +16 added lines, -16 removed lines

@@ -14,11 +14,11 @@ 
 	 */
 	public function buildClassName( $name, $path = '' )
 	{
-		$className = array_map( 'strtolower', (array)preg_split( '/[-_]/', $name ));
+		$className = array_map( 'strtolower', (array)preg_split( '/[-_]/', $name ) );
 		$className = array_map( 'ucfirst', $className );
 		$className = implode( '', $className );
 		$path = ltrim( str_replace( __NAMESPACE__, '', $path ), '\\' );
-		return !empty( $path )
+		return !empty($path)
 			? __NAMESPACE__.'\\'.$path.'\\'.$className
 			: $className;
 	}
@@ -30,7 +30,7 @@ 
 	 */
 	public function buildMethodName( $name, $prefix = '' )
 	{
-		return lcfirst( $prefix.$this->buildClassName( $name ));
+		return lcfirst( $prefix.$this->buildClassName( $name ) );
 	}
 
 	/**
@@ -39,7 +39,7 @@ 
 	 */
 	public function buildPropertyName( $name )
 	{
-		return lcfirst( $this->buildClassName( $name ));
+		return lcfirst( $this->buildClassName( $name ) );
 	}
 
 	/**
@@ -81,7 +81,7 @@ 
 	{
 		$levels = explode( '.', $path );
 		return array_reduce( $levels, function( $result, $value ) {
-			return $result.= '['.$value.']';
+			return $result .= '['.$value.']';
 		}, $prefix );
 	}
 
@@ -91,7 +91,7 @@ 
 	 */
 	public function dashCase( $string )
 	{
-		return str_replace( '_', '-', $this->snakeCase( $string ));
+		return str_replace( '_', '-', $this->snakeCase( $string ) );
 	}
 
 	/**
@@ -117,13 +117,13 @@ 
 		$result = [];
 		foreach( $array as $key => $value ) {
 			$newKey = ltrim( $prefix.'.'.$key, '.' );
-			if( $this->isIndexedFlatArray( $value )) {
+			if( $this->isIndexedFlatArray( $value ) ) {
 				if( $flattenValue ) {
 					$value = '['.implode( ', ', $value ).']';
 				}
 			}
-			else if( is_array( $value )) {
-				$result = array_merge( $result, $this->flattenArray( $value, $flattenValue, $newKey ));
+			else if( is_array( $value ) ) {
+				$result = array_merge( $result, $this->flattenArray( $value, $flattenValue, $newKey ) );
 				continue;
 			}
 			$result[$newKey] = $value;
@@ -142,7 +142,7 @@ 
 				Whip::IPV4 => $cloudflareIps['v4'],
 				Whip::IPV6 => $cloudflareIps['v6'],
 			],
-		]))->getValidIpAddress();
+		] ))->getValidIpAddress();
 	}
 
 	/**
@@ -155,7 +155,7 @@ 
 	{
 		$keys = explode( '.', $path );
 		foreach( $keys as $key ) {
-			if( !isset( $values[$key] )) {
+			if( !isset($values[$key]) ) {
 				return $fallback;
 			}
 			$values = $values[$key];
@@ -169,7 +169,7 @@ 
 	 */
 	public function isIndexedArray( $array )
 	{
-		if( !is_array( $array )) {
+		if( !is_array( $array ) ) {
 			return false;
 		}
 		$current = 0;
@@ -188,7 +188,7 @@ 
 	 */
 	public function isIndexedFlatArray( $array )
 	{
-		if( !is_array( $array ) || array_filter( $array, 'is_array' )) {
+		if( !is_array( $array ) || array_filter( $array, 'is_array' ) ) {
 			return false;
 		}
 		return $this->isIndexedArray( $array );
@@ -201,7 +201,7 @@ 
 	 */
 	public function prefixString( $string, $prefix = '' )
 	{
-		return $prefix.str_replace( $prefix, '', trim( $string ));
+		return $prefix.str_replace( $prefix, '', trim( $string ) );
 	}
 
 	/**
@@ -247,7 +247,7 @@ 
 	 */
 	public function snakeCase( $string )
 	{
-		if( !ctype_lower( $string )) {
+		if( !ctype_lower( $string ) ) {
 			$string = preg_replace( '/\s+/u', '', $string );
 			$string = preg_replace( '/(.)(?=[A-Z])/u', '$1_', $string );
 			$string = mb_strtolower( $string, 'UTF-8' );
@@ -262,6 +262,6 @@ 
 	 */
 	public function startsWith( $needle, $haystack )
 	{
-		return substr( $haystack, 0, strlen( $needle )) === $needle;
+		return substr( $haystack, 0, strlen( $needle ) ) === $needle;
 	}
 }

plugin/Controllers/AdminController.php

Spacing +24 added lines, -24 removed lines

@@ -34,7 +34,7 @@ 
 	{
 		$links[] = glsr( Builder::class )->a( __( 'Settings', 'site-reviews' ), [
 			'href' => admin_url( 'edit.php?post_type='.Application::POST_TYPE.'&page=settings' ),
-		]);
+		] );
 		return $links;
 	}
 
@@ -45,19 +45,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;
 	}
 
@@ -68,7 +68,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;
@@ -80,7 +80,7 @@ 
 	 */
 	public function registerPointers()
 	{
-		$command = new RegisterPointers([[
+		$command = new RegisterPointers( [[
 			'content' => __( 'You can pin exceptional reviews so that they are always shown first.', 'site-reviews' ),
 			'id' => 'glsr-pointer-pinned',
 			'position' => [
@@ -90,7 +90,7 @@ 
 			'screen' => Application::POST_TYPE,
 			'target' => '#misc-pub-pinned',
 			'title' => __( 'Pin Your Reviews', 'site-reviews' ),
-		]]);
+		]] );
 		$this->execute( $command );
 	}
 
@@ -100,11 +100,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 );
 	}
 
@@ -117,7 +117,7 @@ 
 		if( !$this->isReviewEditable( $post ) )return;
 		glsr()->render( 'partials/editor/review', [
 			'post' => $post,
-		]);
+		] );
 	}
 
 	/**
@@ -127,7 +127,7 @@ 
 	public function renderReviewNotice( WP_Post $post )
 	{
 		if( !$this->isReviewEditable( $post ) )return;
-		glsr( Notice::class )->addWarning( __( 'This review is read-only.', 'site-reviews' ));
+		glsr( Notice::class )->addWarning( __( 'This review is read-only.', 'site-reviews' ) );
 		glsr()->render( 'partials/editor/notice' );
 	}
 
@@ -140,13 +140,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,
-		]);
+		] );
 	}
 
 	/**
@@ -155,7 +155,7 @@ 
 	public function routerClearLog()
 	{
 		glsr( Logger::class )->clear();
-		glsr( Notice::class )->addSuccess( __( 'Log cleared.', 'site-reviews' ));
+		glsr( Notice::class )->addSuccess( __( 'Log cleared.', 'site-reviews' ) );
 	}
 
 	/**
@@ -189,17 +189,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' ) );
 	}
 
 	/**
@@ -217,7 +217,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];
 	}