pryley / site-reviews

plugin/Controllers/SettingsController.php

Spacing +9 added lines, -9 removed lines

@@ -21,11 +21,11 @@ 
 			return $input;
 		}
 		$triggered = true;
-		if( !is_array( $input )) {
+		if( !is_array( $input ) ) {
 			$input = ['settings' => []];
 		}
 		if( key( $input ) == 'settings' ) {
-			glsr( Notice::class )->addSuccess( __( 'Settings updated.', 'site-reviews' ));
+			glsr( Notice::class )->addSuccess( __( 'Settings updated.', 'site-reviews' ) );
 		}
 		$options = array_replace_recursive( glsr( OptionManager::class )->all(), $input );
 		$options = $this->sanitizeSubmissions( $input, $options );
@@ -41,7 +41,7 @@ 
 	{
 		register_setting( Application::ID.'-settings', OptionManager::databaseKey(), [
 			'sanitize_callback' => [$this, 'callbackRegisterSettings'],
-		]);
+		] );
 	}
 
 	/**
@@ -49,9 +49,9 @@ 
 	 */
 	protected function sanitizeSubmissions( array $input, array $options )
 	{
-		if( isset( $input['settings']['submissions'] )) {
+		if( isset($input['settings']['submissions']) ) {
 			$inputForm = $input['settings']['submissions'];
-			$options['settings']['submissions']['required'] = isset( $inputForm['required'] )
+			$options['settings']['submissions']['required'] = isset($inputForm['required'])
 				? $inputForm['required']
 				: [];
 		}
@@ -63,13 +63,13 @@ 
 	 */
 	protected function sanitizeTranslations( array $input, array $options )
 	{
-		if( isset( $input['settings']['strings'] )) {
-			$options['settings']['strings'] = array_values( array_filter( $input['settings']['strings'] ));
+		if( isset($input['settings']['strings']) ) {
+			$options['settings']['strings'] = array_values( array_filter( $input['settings']['strings'] ) );
 			array_walk( $options['settings']['strings'], function( &$string ) {
-				if( isset( $string['s2'] )) {
+				if( isset($string['s2']) ) {
 					$string['s2'] = wp_strip_all_tags( $string['s2'] );
 				}
-				if( isset( $string['p2'] )) {
+				if( isset($string['p2']) ) {
 					$string['p2'] = wp_strip_all_tags( $string['p2'] );
 				}
 			});

plugin/Modules/System.php

Spacing +19 added lines, -19 removed lines

@@ -37,7 +37,7 @@ 
 			'inactive-plugin' => 'Inactive Plugins',
 			'setting' => 'Plugin Settings',
 		];
-		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use( $details ) {
+		$systemInfo = array_reduce( array_keys( $details ), function( $carry, $key ) use($details) {
 			$methodName = glsr( Helper::class )->buildMethodName( 'get-'.$key.'-details' );
 			if( method_exists( $this, $methodName ) && $systemDetails = $this->$methodName() ) {
 				return $carry.$this->implode( $details[$key], $systemDetails );
@@ -54,8 +54,8 @@ 
 	{
 		$plugins = get_plugins();
 		$activePlugins = (array)get_option( 'active_plugins', [] );
-		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ));
-		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
+		$inactive = array_diff_key( $plugins, array_flip( $activePlugins ) );
+		return $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
 	}
 
 	/**
@@ -79,7 +79,7 @@ 
 	public function getInactivePluginDetails()
 	{
 		$activePlugins = (array)get_option( 'active_plugins', [] );
-		return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins )));
+		return $this->normalizePluginList( array_diff_key( get_plugins(), array_flip( $activePlugins ) ) );
 	}
 
 	/**
@@ -89,9 +89,9 @@ 
 	{
 		$plugins = array_merge(
 			get_mu_plugins(),
-			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ))
+			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ) )
 		);
-		if( empty( $plugins ))return;
+		if( empty($plugins) )return;
 		return $this->normalizePluginList( $plugins );
 	}
 
@@ -100,7 +100,7 @@ 
 	 */
 	public function getMultisitePluginDetails()
 	{
-		if( !is_multisite() || empty( get_site_option( 'active_sitewide_plugins', [] )))return;
+		if( !is_multisite() || empty(get_site_option( 'active_sitewide_plugins', [] )) )return;
 		return $this->normalizePluginList( wp_get_active_network_plugins() );
 	}
 
@@ -122,11 +122,11 @@ 
 			'Max Input Vars' => ini_get( 'max_input_vars' ),
 			'Memory Limit' => ini_get( 'memory_limit' ),
 			'Post Max Size' => ini_get( 'post_max_size' ),
-			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' )),
-			'Session Name' => esc_html( ini_get( 'session.name' )),
-			'Session Save Path' => esc_html( ini_get( 'session.save_path' )),
-			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' )), true ),
-			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' )), true ),
+			'Session Cookie Path' => esc_html( ini_get( 'session.cookie_path' ) ),
+			'Session Name' => esc_html( ini_get( 'session.name' ) ),
+			'Session Save Path' => esc_html( ini_get( 'session.save_path' ) ),
+			'Session Use Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_cookies' ) ), true ),
+			'Session Use Only Cookies' => var_export( wp_validate_boolean( ini_get( 'session.use_only_cookies' ) ), true ),
 			'Upload Max Filesize' => ini_get( 'upload_max_filesize' ),
 		];
 	}
@@ -154,13 +154,13 @@ 
 		$settings = glsr( OptionManager::class )->get( 'settings', [] );
 		$settings = $helper->flattenArray( $settings, true );
 		foreach( ['submissions.recaptcha.key', 'submissions.recaptcha.secret'] as $key ) {
-			if( empty( $settings[$key] ))continue;
+			if( empty($settings[$key]) )continue;
 			$settings[$key] = str_repeat( '*', 10 );
 		}
 		$details = [];
 		foreach( $settings as $key => $value ) {
-			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ))continue;
-			$value = htmlspecialchars( trim( preg_replace('/\s\s+/', '\\n', $value )), ENT_QUOTES, 'UTF-8' );
+			if( $helper->startsWith( 'strings', $key ) && $helper->endsWith( 'id', $key ) )continue;
+			$value = htmlspecialchars( trim( preg_replace( '/\s\s+/', '\\n', $value ) ), ENT_QUOTES, 'UTF-8' );
 			$details[$key] = $value;
 		}
 		return $details;
@@ -240,7 +240,7 @@ 
 				return $value;
 			}
 		}
-		return implode( ',', array_filter( [DB_HOST, $serverName] ));
+		return implode( ',', array_filter( [DB_HOST, $serverName] ) );
 	}
 
 	/**
@@ -261,8 +261,8 @@ 
 	{
 		$plugins = get_plugins();
 		$activePlugins = (array)get_option( 'active_plugins', [] );
-		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins )));
-		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ));
+		$inactive = $this->normalizePluginList( array_diff_key( $plugins, array_flip( $activePlugins ) ) );
+		$active = $this->normalizePluginList( array_diff_key( $plugins, $inactive ) );
 		return $active + $inactive;
 	}
 
@@ -273,7 +273,7 @@ 
 	protected function implode( $title, array $details )
 	{
 		$strings = ['['.$title.']'];
-		$padding = max( array_map( 'strlen', array_keys( $details )) );
+		$padding = max( array_map( 'strlen', array_keys( $details ) ) );
 		$padding = max( [$padding, static::PAD] );
 		foreach( $details as $key => $value ) {
 			$strings[] = is_string( $key )

plugin/Modules/Html/Form.php

Spacing +5 added lines, -5 removed lines

@@ -23,7 +23,7 @@ 
 			: $this->$method( $id );
 		return glsr( Template::class )->build( 'pages/settings/'.$id, [
 			'context' => $context,
-		]);
+		] );
 	}
 
 	/**
@@ -41,7 +41,7 @@ 
 	protected function getSettingFields( $path )
 	{
 		$settings = glsr( DefaultsManager::class )->settings();
-		return array_filter( $settings, function( $key ) use( $path ) {
+		return array_filter( $settings, function( $key ) use($path) {
 			return glsr( Helper::class )->startsWith( $path, $key );
 		}, ARRAY_FILTER_USE_KEY );
 	}
@@ -52,11 +52,11 @@ 
 	 */
 	protected function getTemplateContext( $id )
 	{
-		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ));
+		$fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
 		$rows = '';
 		foreach( $fields as $name => $field ) {
 			$field = wp_parse_args( $field, ['name' => $name] );
-			$rows.= (new Field( $field ))->build();
+			$rows .= (new Field( $field ))->build();
 		}
 		return [
 			'rows' => $rows,
@@ -69,7 +69,7 @@ 
 	protected function getTemplateContextForTranslations()
 	{
 		$translations = glsr( Translator::class )->renderAll();
-		$class = empty( $translations )
+		$class = empty($translations)
 			? 'glsr-hidden'
 			: '';
 		return [

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/Controllers/AjaxController.php

Spacing +16 added lines, -16 removed lines

@@ -24,7 +24,7 @@ 
 	{
 		wp_send_json( glsr( Html::class )->renderPartial( 'link', [
 			'post_id' => $request['ID'],
-		]));
+		] ) );
 	}
 
 	/**
@@ -32,7 +32,7 @@ 
 	 */
 	public function routerChangeReviewStatus( array $request )
 	{
-		wp_send_json( $this->execute( new ChangeStatus( $request )));
+		wp_send_json( $this->execute( new ChangeStatus( $request ) ) );
 	}
 
 	/**
@@ -41,10 +41,10 @@ 
 	public function routerClearLog()
 	{
 		glsr( AdminController::class )->routerClearLog();
-		wp_send_json([
+		wp_send_json( [
 			'logger' => glsr( Logger::class )->get(),
 			'notices' => glsr( Notice::class )->get(),
-		]);
+		] );
 	}
 
 	/**
@@ -54,9 +54,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 = [
@@ -76,10 +76,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,
-		]);
+		] );
 	}
 
 	/**
@@ -87,7 +87,7 @@ 
 	 */
 	public function routerSearchTranslations( array $request )
 	{
-		if( empty( $request['exclude'] )) {
+		if( empty($request['exclude']) ) {
 			$request['exclude'] = [];
 		}
 		$results = glsr( Translator::class )
@@ -95,10 +95,10 @@ 
 			->exclude()
 			->exclude( $request['exclude'] )
 			->renderResults();
-		wp_send_json_success([
+		wp_send_json_success( [
 			'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>',
 			'items' => $results,
-		]);
+		] );
 	}
 
 	/**
@@ -108,11 +108,11 @@ 
 	{
 		$response = glsr( PublicController::class )->routerCreateReview( $request );
 		$session = glsr( Session::class );
-		wp_send_json([
+		wp_send_json( [
 			'errors' => $session->get( $request['form_id'].'-errors', false, true ),
 			'message' => $response,
 			'recaptcha' => $session->get( $request['form_id'].'-recaptcha', false, true ),
-		]);
+		] );
 	}
 
 	/**
@@ -120,9 +120,9 @@ 
 	 */
 	public function routerTogglePinned( array $request )
 	{
-		wp_send_json([
+		wp_send_json( [
 			'notices' => glsr( Notice::class )->get(),
-			'pinned' => $this->execute( new TogglePinned( $request )),
-		]);
+			'pinned' => $this->execute( new TogglePinned( $request ) ),
+		] );
 	}
 }

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

plugin/Router.php

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