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/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/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/Filters.php

Spacing +21 added lines, -21 removed lines

@@ -38,27 +38,27 @@
 	 */
 	public function run()
 	{
-		add_filter( 'mce_external_plugins',                                    [$this->admin, 'filterTinymcePlugins'], 15 );
-		add_filter( 'plugin_action_links_'.$this->basename,                    [$this->admin, 'filterActionLinks'] );
-		add_filter( 'dashboard_glance_items',                                  [$this->admin, 'filterDashboardGlanceItems'] );
-		add_filter( 'wp_editor_settings',                                      [$this->editor, 'filterEditorSettings'] );
-		add_filter( 'the_editor',                                              [$this->editor, 'filterEditorTextarea'] );
-		add_filter( 'gettext',                                                 [$this->editor, 'filterPostStatusLabels'], 10, 3 );
-		add_filter( 'gettext_with_context',                                    [$this->editor, 'filterPostStatusLabelsWithContext'], 10, 4 );
-		add_filter( 'post_updated_messages',                                   [$this->editor, 'filterUpdateMessages'] );
-		add_filter( 'bulk_post_updated_messages',                              [$this->listtable, 'filterBulkUpdateMessages'], 10, 2 );
-		add_filter( 'manage_'.Application::POST_TYPE.'_posts_columns',         [$this->listtable, 'filterColumnsForPostType'] );
-		add_filter( 'post_date_column_status',                                 [$this->listtable, 'filterDateColumnStatus'], 10, 2 );
-		add_filter( 'default_hidden_columns',                                  [$this->listtable, 'filterDefaultHiddenColumns'], 10, 2 );
-		add_filter( 'display_post_states',                                     [$this->listtable, 'filterPostStates'], 10, 2 );
-		add_filter( 'post_row_actions',                                        [$this->listtable, 'filterRowActions'], 10, 2 );
+		add_filter( 'mce_external_plugins', [$this->admin, 'filterTinymcePlugins'], 15 );
+		add_filter( 'plugin_action_links_'.$this->basename, [$this->admin, 'filterActionLinks'] );
+		add_filter( 'dashboard_glance_items', [$this->admin, 'filterDashboardGlanceItems'] );
+		add_filter( 'wp_editor_settings', [$this->editor, 'filterEditorSettings'] );
+		add_filter( 'the_editor', [$this->editor, 'filterEditorTextarea'] );
+		add_filter( 'gettext', [$this->editor, 'filterPostStatusLabels'], 10, 3 );
+		add_filter( 'gettext_with_context', [$this->editor, 'filterPostStatusLabelsWithContext'], 10, 4 );
+		add_filter( 'post_updated_messages', [$this->editor, 'filterUpdateMessages'] );
+		add_filter( 'bulk_post_updated_messages', [$this->listtable, 'filterBulkUpdateMessages'], 10, 2 );
+		add_filter( 'manage_'.Application::POST_TYPE.'_posts_columns', [$this->listtable, 'filterColumnsForPostType'] );
+		add_filter( 'post_date_column_status', [$this->listtable, 'filterDateColumnStatus'], 10, 2 );
+		add_filter( 'default_hidden_columns', [$this->listtable, 'filterDefaultHiddenColumns'], 10, 2 );
+		add_filter( 'display_post_states', [$this->listtable, 'filterPostStates'], 10, 2 );
+		add_filter( 'post_row_actions', [$this->listtable, 'filterRowActions'], 10, 2 );
 		add_filter( 'manage_edit-'.Application::POST_TYPE.'_sortable_columns', [$this->listtable, 'filterSortableColumns'] );
-		add_filter( 'ngettext',                                                [$this->listtable, 'filterStatusText'], 10, 5 );
-		add_filter( 'script_loader_tag',                                       [$this->public, 'filterEnqueuedScripts'], 10, 2 );
-		add_filter( 'query_vars',                                              [$this->public, 'filterQueryVars'] );
-		add_filter( 'gettext',                                                 [$this->translator, 'filterGettext'], 10, 3 );
-		add_filter( 'gettext_with_context',                                    [$this->translator, 'filterGettextWithContext'], 10, 4 );
-		add_filter( 'ngettext',                                                [$this->translator, 'filterNgettext'], 10, 5 );
-		add_filter( 'ngettext_with_context',                                   [$this->translator, 'filterNgettextWithContext'], 10, 6 );
+		add_filter( 'ngettext', [$this->listtable, 'filterStatusText'], 10, 5 );
+		add_filter( 'script_loader_tag', [$this->public, 'filterEnqueuedScripts'], 10, 2 );
+		add_filter( 'query_vars', [$this->public, 'filterQueryVars'] );
+		add_filter( 'gettext', [$this->translator, 'filterGettext'], 10, 3 );
+		add_filter( 'gettext_with_context', [$this->translator, 'filterGettextWithContext'], 10, 4 );
+		add_filter( 'ngettext', [$this->translator, 'filterNgettext'], 10, 5 );
+		add_filter( 'ngettext_with_context', [$this->translator, 'filterNgettextWithContext'], 10, 6 );
 	}
 }

plugin/Handlers/ChangeStatus.php

Spacing +5 added lines, -5 removed lines

@@ -12,11 +12,11 @@ 
 	 */
 	public function handle( Command $command )
 	{
-		$postId = wp_update_post([
+		$postId = wp_update_post( [
 			'ID' => $command->id,
 			'post_status' => $command->status,
-		]);
-		if( is_wp_error( $postId )) {
+		] );
+		if( is_wp_error( $postId ) ) {
 			glsr_log()->error( $postId->get_error_message() );
 			return [];
 		}
@@ -37,7 +37,7 @@ 
 			'aria-label' => '“'.esc_attr( $title ).'” ('.__( 'Edit', 'site-reviews' ).')',
 			'class' => 'row-title',
 			'href' => get_edit_post_link( $postId ),
-		]);
+		] );
 	}
 
 	/**
@@ -47,7 +47,7 @@ 
 	protected function getPostState( $postId )
 	{
 		ob_start();
-		_post_states( get_post( $postId ));
+		_post_states( get_post( $postId ) );
 		return ob_get_clean();
 	}
 }

plugin/Commands/RegisterPostType.php

Spacing +7 added lines, -7 removed lines

@@ -30,13 +30,13 @@ 
 	{
 		foreach( $args as $key => $value ) {
 			$property = glsr( Helper::class )->buildPropertyName( $key );
-			if( !property_exists( $this, $property ))continue;
+			if( !property_exists( $this, $property ) )continue;
 			$this->$property = $value;
-			unset( $args[$key] );
+			unset($args[$key]);
 		}
 		$this->args = wp_parse_args( $args, [
 			'menu_name' => $this->plural,
-		]);
+		] );
 	}
 
 	/**
@@ -59,8 +59,8 @@ 
 			'singular_name' => $this->single,
 			'uploaded_to_this_item' => sprintf( _x( 'Uploaded to this %s', 'Uploaded to this Post', 'site-reviews' ), $this->single ),
 			'view_item' => sprintf( _x( 'View %s', 'View Post', 'site-reviews' ), $this->single ),
-		]);
-		unset( $this->args['menu_name'] );
+		] );
+		unset($this->args['menu_name']);
 	}
 
 	/**
@@ -69,12 +69,12 @@ 
 	protected function normalizeColumns()
 	{
 		$this->columns = ['cb' => ''] + $this->columns;
-		if( array_key_exists( 'category', $this->columns )) {
+		if( array_key_exists( 'category', $this->columns ) ) {
 			$keys = array_keys( $this->columns );
 			$keys[array_search( 'category', $keys )] = 'taxonomy-'.Application::TAXONOMY;
 			$this->columns = array_combine( $keys, $this->columns );
 		}
-		if( array_key_exists( 'pinned', $this->columns )) {
+		if( array_key_exists( 'pinned', $this->columns ) ) {
 			$this->columns['pinned'] = glsr( Builder::class )->span(
 				'<span>'.$this->columns['pinned'].'</span>',
 				['class' => 'pinned-icon']