pryley / site-reviews

plugin/Database/DefaultsManager.php

Braces +3 added lines, -1 removed lines

@@ -53,7 +53,9 @@
 	protected function normalize( array $settings )
 	{
 		array_walk( $settings, function( &$setting ) {
-			if( isset( $setting['default'] ))return;
+			if( isset( $setting['default'] )) {
+				return;
+			}
 			$setting['default'] = '';
 		});
 		return $settings;

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/Modules/Html/Field.php

Spacing +20 added lines, -20 removed lines

@@ -23,7 +23,7 @@ 
 			'is_multi' => false,
 			'is_valid' => true,
 			'path' => '',
-		]);
+		] );
 		$this->normalize();
 	}
 
@@ -40,10 +40,10 @@ 
 		return glsr( Template::class )->build( 'partials/settings/form-table-row', [
 			'context' => [
 				'class' => $this->getFieldClass(),
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
 				'label' => glsr( Builder::class )->label( $this->field['legend'], ['for' => $this->field['id']] ),
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -63,11 +63,11 @@ 
 			'context' => [
 				'class' => $this->getFieldClass(),
 				'depends_on' => $this->getFieldDependsOn(),
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
 				'label' => glsr( Builder::class )->label( $this->field['legend'], ['for' => $this->field['id']] ),
 				'legend' => $this->field['legend'],
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -85,7 +85,7 @@ 
 	 */
 	protected function getFieldDefault()
 	{
-		return isset( $this->field['default'] )
+		return isset($this->field['default'])
 			? $this->field['default']
 			: '';
 	}
@@ -95,7 +95,7 @@ 
 	 */
 	protected function getFieldDependsOn()
 	{
-		return !empty( $this->field['depends_on'] )
+		return !empty($this->field['depends_on'])
 			? $this->field['depends_on']
 			: '';
 	}
@@ -111,7 +111,7 @@ 
 			$path,
 			glsr( Helper::class )->getPathValue( $path, glsr()->defaults )
 		);
-		if( is_array( $expectedValue )) {
+		if( is_array( $expectedValue ) ) {
 			return !in_array( $optionValue, $expectedValue );
 		}
 		return $optionValue != $expectedValue;
@@ -128,13 +128,13 @@ 
 			'label', 'name', 'type',
 		];
 		foreach( $requiredValues as $value ) {
-			if( isset( $this->field[$value] ))continue;
+			if( isset($this->field[$value]) )continue;
 			$missingValues[] = $value;
 			$isValid = $this->field['is_valid'] = false;
 		}
-		if( !empty( $missingValues )) {
+		if( !empty($missingValues) ) {
 			glsr_log()
-				->warning( 'Field is missing: '.implode( ', ', $missingValues ))
+				->warning( 'Field is missing: '.implode( ', ', $missingValues ) )
 				->info( $this->field );
 		}
 		return $isValid;
@@ -149,7 +149,7 @@ 
 		$field = $this->field;
 		foreach( $field as $key => $value ) {
 			$methodName = glsr( Helper::class )->buildMethodName( $key, 'normalize' );
-			if( !method_exists( $this, $methodName ))continue;
+			if( !method_exists( $this, $methodName ) )continue;
 			$this->$methodName();
 		}
 		$this->normalizeFieldId();
@@ -162,13 +162,13 @@ 
 	 */
 	protected function normalizeDependsOn()
 	{
-		if( empty( $this->field['depends_on'] ) || !is_array( $this->field['depends_on'] ))return;
+		if( empty($this->field['depends_on']) || !is_array( $this->field['depends_on'] ) )return;
 		$path = key( $this->field['depends_on'] );
 		$value = $this->field['depends_on'][$path];
-		$this->field['depends_on'] = json_encode([
+		$this->field['depends_on'] = json_encode( [
 			'name' => glsr( Helper::class )->convertPathToName( $path, OptionManager::databaseKey() ),
 			'value' => $value,
-		], JSON_HEX_APOS|JSON_HEX_QUOT );
+		], JSON_HEX_APOS | JSON_HEX_QUOT );
 		$this->field['is_hidden'] = $this->isFieldHidden( $path, $value );
 	}
 
@@ -177,7 +177,7 @@ 
 	 */
 	protected function normalizeFieldId()
 	{
-		if( isset( $this->field['id'] ))return;
+		if( isset($this->field['id']) )return;
 		$this->field['id'] = glsr( Helper::class )->convertNameToId( $this->field['name'] );
 	}
 
@@ -187,10 +187,10 @@ 
 	protected function normalizeFieldType()
 	{
 		$className = glsr( Helper::class )->buildClassName( $this->field['type'], 'Modules\Html\Fields' );
-		if( class_exists( $className )) {
+		if( class_exists( $className ) ) {
 			$this->field = array_merge( $this->field, glsr( $className )->defaults() );
 		}
-		if( in_array( $this->field['type'], static::MULTI_FIELD_TYPES )) {
+		if( in_array( $this->field['type'], static::MULTI_FIELD_TYPES ) ) {
 			$this->field['is_multi'] = true;
 		}
 	}
@@ -200,7 +200,7 @@ 
 	 */
 	protected function normalizeFieldValue()
 	{
-		if( isset( $this->field['value'] ))return;
+		if( isset($this->field['value']) )return;
 		$this->field['value'] = glsr( OptionManager::class )->get(
 			$this->field['path'],
 			$this->getFieldDefault()
@@ -213,7 +213,7 @@ 
 	protected function normalizeLabel()
 	{
 		$this->field['legend'] = $this->field['label'];
-		unset( $this->field['label'] );
+		unset($this->field['label']);
 	}
 
 	/**

Braces +21 added lines, -7 removed lines

@@ -32,7 +32,9 @@ 
 	 */
 	public function build()
 	{
-		if( !$this->field['is_valid'] )return;
+		if( !$this->field['is_valid'] ) {
+			return;
+		}
 		if( $this->field['is_multi'] ) {
 			return $this->buildMultiField();
 		}
@@ -128,7 +130,9 @@ 
 			'label', 'name', 'type',
 		];
 		foreach( $requiredValues as $value ) {
-			if( isset( $this->field[$value] ))continue;
+			if( isset( $this->field[$value] )) {
+				continue;
+			}
 			$missingValues[] = $value;
 			$isValid = $this->field['is_valid'] = false;
 		}
@@ -145,11 +149,15 @@ 
 	 */
 	protected function normalize()
 	{
-		if( !$this->isFieldValid() )return;
+		if( !$this->isFieldValid() ) {
+			return;
+		}
 		$field = $this->field;
 		foreach( $field as $key => $value ) {
 			$methodName = glsr( Helper::class )->buildMethodName( $key, 'normalize' );
-			if( !method_exists( $this, $methodName ))continue;
+			if( !method_exists( $this, $methodName )) {
+				continue;
+			}
 			$this->$methodName();
 		}
 		$this->normalizeFieldId();
@@ -162,7 +170,9 @@ 
 	 */
 	protected function normalizeDependsOn()
 	{
-		if( empty( $this->field['depends_on'] ) || !is_array( $this->field['depends_on'] ))return;
+		if( empty( $this->field['depends_on'] ) || !is_array( $this->field['depends_on'] )) {
+			return;
+		}
 		$path = key( $this->field['depends_on'] );
 		$value = $this->field['depends_on'][$path];
 		$this->field['depends_on'] = json_encode([
@@ -177,7 +187,9 @@ 
 	 */
 	protected function normalizeFieldId()
 	{
-		if( isset( $this->field['id'] ))return;
+		if( isset( $this->field['id'] )) {
+			return;
+		}
 		$this->field['id'] = glsr( Helper::class )->convertNameToId( $this->field['name'] );
 	}
 
@@ -200,7 +212,9 @@ 
 	 */
 	protected function normalizeFieldValue()
 	{
-		if( isset( $this->field['value'] ))return;
+		if( isset( $this->field['value'] )) {
+			return;
+		}
 		$this->field['value'] = glsr( OptionManager::class )->get(
 			$this->field['path'],
 			$this->getFieldDefault()

plugin/Modules/Translator.php

Spacing +29 added lines, -29 removed lines

@@ -30,7 +30,7 @@ 
 	{
 		$translations = $this->getTranslations();
 		$entries = $this->filter( $translations, $this->entries() )->results();
-		array_walk( $translations, function( &$entry ) use( $entries ) {
+		array_walk( $translations, function( &$entry ) use($entries) {
 			$entry['desc'] = array_key_exists( $entry['id'], $entries )
 				? $this->getEntryString( $entries[$entry['id']], 'msgctxt' )
 				: '';
@@ -43,7 +43,7 @@ 
 	 */
 	public function entries()
 	{
-		if( !isset( $this->entries )) {
+		if( !isset($this->entries) ) {
 			try {
 				$potFile = glsr()->path( glsr()->languages.'/'.Application::ID.'.pot' );
 				$entries = $this->normalize( Parser::parseFile( $potFile )->getEntries() );
@@ -76,13 +76,13 @@ 
 	 */
 	public function filter( $filterWith = null, $entries = null, $intersect = true )
 	{
-		if( !is_array( $entries )) {
+		if( !is_array( $entries ) ) {
 			$entries = $this->results;
 		}
-		if( !is_array( $filterWith )) {
+		if( !is_array( $filterWith ) ) {
 			$filterWith = $this->getTranslations();
 		}
-		$keys = array_flip( array_column( $filterWith, 'id' ));
+		$keys = array_flip( array_column( $filterWith, 'id' ) );
 		$this->results = $intersect
 			? array_intersect_key( $entries, $keys )
 			: array_diff_key( $entries, $keys );
@@ -99,7 +99,7 @@ 
 	{
 		return $this->translate( $translation, $domain, [
 			'single' => $text,
-		]);
+		] );
 	}
 
 	/**
@@ -114,7 +114,7 @@ 
 		return $this->translate( $translation, $domain, [
 			'context' => $context,
 			'single' => $text,
-		]);
+		] );
 	}
 
 	/**
@@ -131,7 +131,7 @@ 
 			'number' => $number,
 			'plural' => $plural,
 			'single' => $single,
-		]);
+		] );
 	}
 
 	/**
@@ -150,7 +150,7 @@ 
 			'number' => $number,
 			'plural' => $plural,
 			'single' => $single,
-		]);
+		] );
 	}
 
 	/**
@@ -160,13 +160,13 @@ 
 	public function render( $template, array $entry )
 	{
 		$data = array_combine(
-			array_map( function( $key ) { return 'data.'.$key; }, array_keys( $entry )),
+			array_map( function( $key ) { return 'data.'.$key; }, array_keys( $entry ) ),
 			$entry
 		);
 		ob_start();
 		glsr( Html::class )->renderTemplate( 'partials/translations/'.$template, [
 			'context' => $data,
-		]);
+		] );
 		return ob_get_clean();
 	}
 
@@ -199,13 +199,13 @@ 
 				'p1' => $this->getEntryString( $entry, 'msgid_plural' ),
 				's1' => $this->getEntryString( $entry, 'msgid' ),
 			];
-			$text = !empty( $data['p1'] )
+			$text = !empty($data['p1'])
 				? sprintf( '%s | %s', $data['s1'], $data['p1'] )
 				: $data['s1'];
 			$rendered .= $this->render( 'result', [
 				'entry' => wp_json_encode( $data ),
 				'text' => wp_strip_all_tags( $text ),
-			]);
+			] );
 		}
 		if( $resetAfterRender ) {
 			$this->reset();
@@ -238,12 +238,12 @@ 
 	public function search( $needle = '' )
 	{
 		$this->reset();
-		$needle = trim( strtolower( $needle ));
+		$needle = trim( strtolower( $needle ) );
 		foreach( $this->entries() as $key => $entry ) {
-			$single = strtolower( $this->getEntryString( $entry, 'msgid' ));
-			$plural = strtolower( $this->getEntryString( $entry, 'msgid_plural' ));
+			$single = strtolower( $this->getEntryString( $entry, 'msgid' ) );
+			$plural = strtolower( $this->getEntryString( $entry, 'msgid_plural' ) );
 			if( strlen( $needle ) < static::SEARCH_THRESHOLD ) {
-				if( in_array( $needle, [$single, $plural] )) {
+				if( in_array( $needle, [$single, $plural] ) ) {
 					$this->results[$key] = $entry;
 				}
 			}
@@ -266,7 +266,7 @@ 
 		}
 		$args = $this->normalizeTranslationArgs( $args );
 		$strings = $this->getTranslationStrings( $args['single'], $args['plural'] );
-		if( empty( $strings )) {
+		if( empty($strings) ) {
 			return $original;
 		}
 		$string = current( $strings );
@@ -281,8 +281,8 @@ 
 	 */
 	protected function getEntryString( array $entry, $key )
 	{
-		return isset( $entry[$key] )
-			? implode( '', (array) $entry[$key] )
+		return isset($entry[$key])
+			? implode( '', (array)$entry[$key] )
 			: '';
 	}
 
@@ -292,8 +292,8 @@ 
 	protected function getTranslations()
 	{
 		$settings = glsr( OptionManager::class )->get( 'settings' );
-		return isset( $settings['strings'] )
-			? $this->normalizeSettings( (array) $settings['strings'] )
+		return isset($settings['strings'])
+			? $this->normalizeSettings( (array)$settings['strings'] )
 			: [];
 	}
 
@@ -304,7 +304,7 @@ 
 	 */
 	protected function getTranslationStrings( $single, $plural )
 	{
-		return array_filter( $this->getTranslations(), function( $string ) use( $single, $plural ) {
+		return array_filter( $this->getTranslations(), 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' );
 		});
@@ -318,7 +318,7 @@ 
 		$keys = [
 			'msgctxt', 'msgid', 'msgid_plural', 'msgstr', 'msgstr[0]', 'msgstr[1]',
 		];
-		array_walk( $entries, function( &$entry ) use( $keys ) {
+		array_walk( $entries, function( &$entry ) use($keys) {
 			foreach( $keys as $key ) {
 				$entry = $this->normalizeEntryString( $entry, $key );
 			}
@@ -332,7 +332,7 @@ 
 	 */
 	protected function normalizeEntryString( array $entry, $key )
 	{
-		if( isset( $entry[$key] )) {
+		if( isset($entry[$key]) ) {
 			$entry[$key] = $this->getEntryString( $entry, $key );
 		}
 		return $entry;
@@ -346,11 +346,11 @@ 
 		$defaultString = array_fill_keys( ['id', 's1', 's2', 'p1', 'p2'], '' );
 		$strings = array_filter( $strings, 'is_array' );
 		foreach( $strings as &$string ) {
-			$string['type'] = isset( $string['p1'] ) ? 'plural' : 'single';
+			$string['type'] = isset($string['p1']) ? 'plural' : 'single';
 			$string = wp_parse_args( $string, $defaultString );
 		}
 		return array_filter( $strings, function( $string ) {
-			return !empty( $string['id'] );
+			return !empty($string['id']);
 		});
 	}
 
@@ -374,7 +374,7 @@ 
 	 */
 	protected function translatePlural( $domain, array $string, array $args )
 	{
-		if( !empty( $string['p2'] )) {
+		if( !empty($string['p2']) ) {
 			$args['plural'] = $string['p2'];
 		}
 		return get_translations_for_domain( $domain )->translate_plural(
@@ -391,7 +391,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/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 )

Braces +12 added lines, -4 removed lines

@@ -91,7 +91,9 @@ 
 			get_mu_plugins(),
 			get_plugins( '/../'.basename( WPMU_PLUGIN_DIR ))
 		);
-		if( empty( $plugins ))return;
+		if( empty( $plugins )) {
+			return;
+		}
 		return $this->normalizePluginList( $plugins );
 	}
 
@@ -100,7 +102,9 @@ 
 	 */
 	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() );
 	}
 
@@ -154,12 +158,16 @@ 
 		$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;
+			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;
 		}

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/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, $key ) {
-			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, $key ) {
-			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/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/Router.php

Spacing +9 added lines, -9 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 );
 		}
@@ -53,14 +53,14 @@ 
 	 */
 	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;
 		}
 		$response = do_action( 'site-reviews/route/'.$type.'/request', $action, $request );
-		if( empty( $response )) {
+		if( empty($response) ) {
 			glsr_log( 'Unknown '.$type.' router request: '.$action );
 		}
 	}
@@ -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();
 		}

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
 	}