pryley / site-reviews

plugin/Helper.php

Spacing +22 added lines, -22 removed lines

@@ -16,7 +16,7 @@ 
 	{
 		$className = $this->camelCase( $name );
 		$path = ltrim( str_replace( __NAMESPACE__, '', $path ), '\\' );
-		return !empty( $path )
+		return !empty($path)
 			? __NAMESPACE__.'\\'.$path.'\\'.$className
 			: $className;
 	}
@@ -28,7 +28,7 @@ 
 	 */
 	public function buildMethodName( $name, $prefix = '' )
 	{
-		return lcfirst( $prefix.$this->buildClassName( $name ));
+		return lcfirst( $prefix.$this->buildClassName( $name ) );
 	}
 
 	/**
@@ -37,7 +37,7 @@ 
 	 */
 	public function buildPropertyName( $name )
 	{
-		return lcfirst( $this->buildClassName( $name ));
+		return lcfirst( $this->buildClassName( $name ) );
 	}
 
 	/**
@@ -46,7 +46,7 @@ 
 	 */
 	public function camelCase( $string )
 	{
-		$string = ucwords( str_replace( ['-', '_'], ' ', trim( $string )));
+		$string = ucwords( str_replace( ['-', '_'], ' ', trim( $string ) ) );
 		return str_replace( ' ', '', $string );
 	}
 
@@ -78,7 +78,7 @@ 
 	 */
 	public function convertPathToId( $path, $prefix = '' )
 	{
-		return str_replace( ['[', ']'], ['-', ''], $this->convertPathToName( $path, $prefix ));
+		return str_replace( ['[', ']'], ['-', ''], $this->convertPathToName( $path, $prefix ) );
 	}
 
 	/**
@@ -89,7 +89,7 @@ 
 	{
 		$levels = explode( '.', $path );
 		return array_reduce( $levels, function( $result, $value ) {
-			return $result.= '['.$value.']';
+			return $result .= '['.$value.']';
 		}, $prefix );
 	}
 
@@ -100,7 +100,7 @@ 
 	 */
 	public function convertStringToArray( $string, $callback = null )
 	{
-		$array = array_map( 'trim', explode( ',', $string ));
+		$array = array_map( 'trim', explode( ',', $string ) );
 		return $callback
 			? array_filter( $array, $callback )
 			: array_filter( $array );
@@ -112,7 +112,7 @@ 
 	 */
 	public function dashCase( $string )
 	{
-		return str_replace( '_', '-', $this->snakeCase( $string ));
+		return str_replace( '_', '-', $this->snakeCase( $string ) );
 	}
 
 	/**
@@ -134,11 +134,11 @@ 
 	 */
 	public function filterInput( $key, array $request = [] )
 	{
-		if( isset( $request[$key] )) {
+		if( isset($request[$key]) ) {
 			return $request[$key];
 		}
 		$variable = filter_input( INPUT_POST, $key );
-		if( is_null( $variable ) && isset( $_POST[$key] )) {
+		if( is_null( $variable ) && isset($_POST[$key]) ) {
 			$variable = $_POST[$key];
 		}
 		return $variable;
@@ -151,7 +151,7 @@ 
 	public function filterInputArray( $key )
 	{
 		$variable = filter_input( INPUT_POST, $key, FILTER_DEFAULT, FILTER_REQUIRE_ARRAY );
-		if( empty( $variable ) && !empty( $_POST[$key] ) && is_array( $_POST[$key] )) {
+		if( empty($variable) && !empty($_POST[$key]) && is_array( $_POST[$key] ) ) {
 			$variable = $_POST[$key];
 		}
 		return (array)$variable;
@@ -167,13 +167,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;
@@ -192,7 +192,7 @@ 
 				Whip::IPV4 => $cloudflareIps['v4'],
 				Whip::IPV6 => $cloudflareIps['v6'],
 			],
-		]))->getValidIpAddress();
+		] ))->getValidIpAddress();
 	}
 
 	/**
@@ -205,7 +205,7 @@ 
 	{
 		$keys = explode( '.', $path );
 		foreach( $keys as $key ) {
-			if( !isset( $values[$key] )) {
+			if( !isset($values[$key]) ) {
 				return $fallback;
 			}
 			$values = $values[$key];
@@ -219,7 +219,7 @@ 
 	 */
 	public function isIndexedArray( $array )
 	{
-		if( !is_array( $array )) {
+		if( !is_array( $array ) ) {
 			return false;
 		}
 		$current = 0;
@@ -238,7 +238,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 );
@@ -251,7 +251,7 @@ 
 	 */
 	public function prefixString( $string, $prefix = '' )
 	{
-		return $prefix.str_replace( $prefix, '', trim( $string ));
+		return $prefix.str_replace( $prefix, '', trim( $string ) );
 	}
 
 	/**
@@ -276,7 +276,7 @@ 
 	 */
 	public function removePrefix( $prefix, $text ) {
 		return 0 === strpos( $text, $prefix )
-			? substr( $text, strlen( $prefix ))
+			? substr( $text, strlen( $prefix ) )
 			: $text;
 	}
 
@@ -307,7 +307,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 = function_exists( 'mb_strtolower' )
@@ -324,6 +324,6 @@ 
 	 */
 	public function startsWith( $needle, $haystack )
 	{
-		return substr( $haystack, 0, strlen( $needle )) === $needle;
+		return substr( $haystack, 0, strlen( $needle ) ) === $needle;
 	}
 }

plugin/Modules/Validator.php

Spacing +14 added lines, -14 removed lines

@@ -74,7 +74,7 @@ 
 		foreach( $this->rules as $attribute => $rules ) {
 			foreach( $rules as $rule ) {
 				$this->validateAttribute( $attribute, $rule );
-				if( $this->shouldStopValidating( $attribute ))break;
+				if( $this->shouldStopValidating( $attribute ) )break;
 			}
 		}
 		return $this->errors;
@@ -89,13 +89,13 @@ 
 	 */
 	public function validateAttribute( $attribute, $rule )
 	{
-		list( $rule, $parameters ) = $this->parseRule( $rule );
+		list($rule, $parameters) = $this->parseRule( $rule );
 		if( $rule == '' )return;
 		$value = $this->getValue( $attribute );
-		if( !method_exists( $this, $method = 'validate'.$rule )) {
+		if( !method_exists( $this, $method = 'validate'.$rule ) ) {
 			throw new BadMethodCallException( "Method [$method] does not exist." );
 		}
-		if( !$this->$method( $value, $attribute, $parameters )) {
+		if( !$this->$method( $value, $attribute, $parameters ) ) {
 			$this->addFailure( $attribute, $rule, $parameters );
 		}
 	}
@@ -144,7 +144,7 @@ 
 	 */
 	protected function getMessage( $attribute, $rule, array $parameters )
 	{
-		if( in_array( $rule, $this->sizeRules )) {
+		if( in_array( $rule, $this->sizeRules ) ) {
 			return $this->getSizeMessage( $attribute, $rule, $parameters );
 		}
 		$lowerRule = glsr( Helper::class )->snakeCase( $rule );
@@ -159,11 +159,11 @@ 
 	 */
 	protected function getRule( $attribute, $rules )
 	{
-		if( !array_key_exists( $attribute, $this->rules ))return;
+		if( !array_key_exists( $attribute, $this->rules ) )return;
 		$rules = (array)$rules;
 		foreach( $this->rules[$attribute] as $rule ) {
-			list( $rule, $parameters ) = $this->parseRule( $rule );
-			if( in_array( $rule, $rules )) {
+			list($rule, $parameters) = $this->parseRule( $rule );
+			if( in_array( $rule, $rules ) ) {
 				return [$rule, $parameters];
 			}
 		}
@@ -181,7 +181,7 @@ 
 		if( is_numeric( $value ) && $hasNumeric ) {
 			return $value;
 		}
-		elseif( is_array( $value )) {
+		elseif( is_array( $value ) ) {
 			return count( $value );
 		}
 		return function_exists( 'mb_strlen' )
@@ -209,7 +209,7 @@ 
 	 */
 	protected function getValue( $attribute )
 	{
-		if( isset( $this->data[$attribute] )) {
+		if( isset($this->data[$attribute]) ) {
 			return $this->data[$attribute];
 		}
 	}
@@ -222,7 +222,7 @@ 
 	 */
 	protected function hasRule( $attribute, $rules )
 	{
-		return !is_null( $this->getRule( $attribute, $rules ));
+		return !is_null( $this->getRule( $attribute, $rules ) );
 	}
 
 	/**
@@ -260,7 +260,7 @@ 
 	{
 		$parameters = [];
 		if( strpos( $rule, ':' ) !== false ) {
-			list( $rule, $parameter ) = explode( ':', $rule, 2 );
+			list($rule, $parameter) = explode( ':', $rule, 2 );
 			$parameters = $this->parseParameters( $rule, $parameter );
 		}
 		$rule = glsr( Helper::class )->camelCase( $rule );
@@ -289,7 +289,7 @@ 
 	protected function shouldStopValidating( $attribute )
 	{
 		return $this->hasRule( $attribute, $this->implicitRules )
-			&& isset( $this->failedRules[$attribute] )
+			&& isset($this->failedRules[$attribute])
 			&& array_intersect( array_keys( $this->failedRules[$attribute] ), $this->implicitRules );
 	}
 
@@ -302,7 +302,7 @@ 
 	protected function translator( $key, array $parameters )
 	{
 		$strings = glsr( ValidationStringsDefaults::class )->defaults();
-		if( isset( $strings[$key] )) {
+		if( isset($strings[$key]) ) {
 			return $this->replace( $strings[$key], $parameters );
 		}
 		return "error";

plugin/Modules/System.php

Spacing +23 added lines, -23 removed lines

@@ -40,7 +40,7 @@ 
 			'setting' => 'Plugin Settings',
 			'reviews' => 'Review Counts',
 		];
-		$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( strtoupper( $details[$key] ), $systemDetails );
@@ -57,8 +57,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 ) );
 	}
 
 	/**
@@ -92,7 +92,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 ) ) );
 	}
 
 	/**
@@ -102,9 +102,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 );
 	}
 
@@ -113,7 +113,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() );
 	}
 
@@ -139,11 +139,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' ),
 		];
 	}
@@ -155,8 +155,8 @@ 
 	{
 		$counts = glsr( CountsManager::class )->getCounts();
 		$counts = glsr( Helper::class )->flattenArray( $counts );
- 		array_walk( $counts, function( &$ratings ) use( $counts ) {
-			if( !is_array( $ratings )) {
+ 		array_walk( $counts, function( &$ratings ) use($counts) {
+			if( !is_array( $ratings ) ) {
 				glsr_log()
 					->error( '$ratings is not an array, possibly due to incorrectly imported reviews.' )
 					->debug( $ratings )
@@ -195,8 +195,8 @@ 
 		ksort( $settings );
 		$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;
@@ -268,10 +268,10 @@ 
 			'WPE_APIKEY' => 'WP Engine',
 		];
 		foreach( $checks as $key => $value ) {
-			if( !$this->isWebhostCheckValid( $key ))continue;
+			if( !$this->isWebhostCheckValid( $key ) )continue;
 			return $value;
 		}
-		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ));
+		return implode( ',', array_filter( [DB_HOST, filter_input( INPUT_SERVER, 'SERVER_NAME' )] ) );
 	}
 
 	/**
@@ -292,8 +292,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;
 	}
 
@@ -304,7 +304,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 )
@@ -347,9 +347,9 @@ 
 		$keys = [
 			'licenses.', 'submissions.recaptcha.key', 'submissions.recaptcha.secret',
 		];
-		array_walk( $settings, function( &$value, $setting ) use( $keys ) {
+		array_walk( $settings, function( &$value, $setting ) use($keys) {
 			foreach( $keys as $key ) {
-				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty( $value ))continue;
+				if( !glsr( Helper::class )->startsWith( $key, $setting ) || empty($value) )continue;
 				$value = str_repeat( '•', 13 );
 				return;
 			}