pryley / site-reviews

deprecated.php

Spacing +16 added lines, -16 removed lines

@@ -4,7 +4,7 @@ 
 
 // Database/ReviewManager.php
 add_action( 'site-reviews/review/created', function( $review, $command ) {
-	if( has_action( 'site-reviews/local/review/create' )) {
+	if( has_action( 'site-reviews/local/review/create' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/local/review/create" hook has been deprecated. Please use the "site-reviews/create/review" hook instead.';
 		do_action( 'site-reviews/local/review/create', (array)get_post( $review->ID ), (array)$review, $review->ID );
 	}
@@ -12,18 +12,18 @@ 
 
 // Handlers/CreateReview.php
 add_action( 'site-reviews/review/submitted', function( $review ) {
-	if( has_action( 'site-reviews/local/review/submitted' )) {
+	if( has_action( 'site-reviews/local/review/submitted' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/local/review/submitted" hook has been deprecated. Please use the "site-reviews/review/submitted" hook instead.';
 		do_action( 'site-reviews/local/review/submitted', null, $review );
 	}
-	if( has_filter( 'site-reviews/local/review/submitted/message' )) {
+	if( has_filter( 'site-reviews/local/review/submitted/message' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/local/review/submitted/message" hook has been deprecated.';
 	}
 });
 
 // Database/ReviewManager.php
 add_filter( 'site-reviews/create/review-values', function( $values, $command ) {
-	if( has_filter( 'site-reviews/local/review' )) {
+	if( has_filter( 'site-reviews/local/review' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/local/review" hook has been deprecated. Please use the "site-reviews/create/review-values" hook instead.';
 		return apply_filters( 'site-reviews/local/review', $values, $command );
 	}
@@ -32,7 +32,7 @@ 
 
 // Handlers/EnqueuePublicAssets.php
 add_filter( 'site-reviews/enqueue/public/localize', function( $variables ) {
-	if( has_filter( 'site-reviews/enqueue/localize' )) {
+	if( has_filter( 'site-reviews/enqueue/localize' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/enqueue/localize" hook has been deprecated. Please use the "site-reviews/enqueue/public/localize" hook instead.';
 		return apply_filters( 'site-reviews/enqueue/localize', $variables );
 	}
@@ -41,7 +41,7 @@ 
 
 // Modules/Rating.php
 add_filter( 'site-reviews/rating/average', function( $average ) {
-	if( has_filter( 'site-reviews/average/rating' )) {
+	if( has_filter( 'site-reviews/average/rating' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/average/rating" hook has been deprecated. Please use the "site-reviews/rating/average" hook instead.';
 	}
 	return $average;
@@ -49,7 +49,7 @@ 
 
 // Modules/Rating.php
 add_filter( 'site-reviews/rating/ranking', function( $ranking ) {
-	if( has_filter( 'site-reviews/bayesian/ranking' )) {
+	if( has_filter( 'site-reviews/bayesian/ranking' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/bayesian/ranking" hook has been deprecated. Please use the "site-reviews/rating/ranking" hook instead.';
 	}
 	return $ranking;
@@ -57,13 +57,13 @@ 
 
 // Modules/Html/Partials/SiteReviews.php
 add_filter( 'site-reviews/review/build/after', function( $renderedFields ) {
-	if( has_filter( 'site-reviews/rendered/field' )) {
+	if( has_filter( 'site-reviews/rendered/field' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/rendered/field" hook has been deprecated. Please use the "site-reviews/review/build/after" hook instead.';
 	}
-	if( has_filter( 'site-reviews/reviews/review/text' )) {
+	if( has_filter( 'site-reviews/reviews/review/text' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/reviews/review/text" hook has been deprecated. Please use the "site-reviews/review/build/after" hook instead.';
 	}
-	if( has_filter( 'site-reviews/reviews/review/title' )) {
+	if( has_filter( 'site-reviews/reviews/review/title' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/reviews/review/title" hook has been deprecated. Please use the "site-reviews/review/build/after" hook instead.';
 	}
 	return $renderedFields;
@@ -71,26 +71,26 @@ 
 
 // Modules/Html/Partials/SiteReviews.php
 add_filter( 'site-reviews/review/build/before', function( $review ) {
-	if( has_filter( 'site-reviews/rendered/review' )) {
+	if( has_filter( 'site-reviews/rendered/review' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/rendered/review" hook has been deprecated. Please either use a custom "review.php" template (refer to the documentation), or use the "site-reviews/review/build/after" hook instead.';
 	}
-	if( has_filter( 'site-reviews/rendered/review/meta/order' )) {
+	if( has_filter( 'site-reviews/rendered/review/meta/order' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/rendered/review/meta/order" hook has been deprecated. Please use a custom "review.php" template instead (refer to the documentation).';
 	}
-	if( has_filter( 'site-reviews/rendered/review/order' )) {
+	if( has_filter( 'site-reviews/rendered/review/order' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/rendered/review/order" hook has been deprecated. Please use a custom "review.php" template instead (refer to the documentation).';
 	}
-	if( has_filter( 'site-reviews/rendered/review-form/login-register' )) {
+	if( has_filter( 'site-reviews/rendered/review-form/login-register' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/rendered/review-form/login-register" hook has been deprecated. Please use a custom "login-register.php" template instead (refer to the documentation).';
 	}
-	if( has_filter( 'site-reviews/reviews/navigation_links' )) {
+	if( has_filter( 'site-reviews/reviews/navigation_links' ) ) {
 		glsr()->deprecated[] = 'The "site-reviews/reviews/navigation_links" hook has been deprecated. Please use a custom "pagination.php" template instead (refer to the documentation).';
 	}
 	return $review;
 });
 
 add_action( 'wp_footer', function() {
-	$notices = array_keys( array_flip( glsr()->deprecated ));
+	$notices = array_keys( array_flip( glsr()->deprecated ) );
 	foreach( $notices as $notice ) {
 		glsr_log()->notice( $notice );
 	}

plugin/Modules/System.php

Spacing +21 added lines, -21 removed lines

@@ -38,7 +38,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 );
@@ -55,8 +55,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 ) );
 	}
 
 	/**
@@ -80,7 +80,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 ) ) );
 	}
 
 	/**
@@ -90,9 +90,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 );
 	}
 
@@ -101,7 +101,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() );
 	}
 
@@ -127,11 +127,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' ),
 		];
 	}
@@ -176,8 +176,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;
@@ -249,10 +249,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' )] ) );
 	}
 
 	/**
@@ -273,8 +273,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;
 	}
 
@@ -285,7 +285,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 )
@@ -328,9 +328,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;
 			}

helpers.php

Braces +16 added lines, -8 removed lines

@@ -4,7 +4,8 @@ 
 /**
  * @return mixed
  */
-function glsr( $alias = null ) {
+function glsr( $alias = null )
+{
 	$app = \GeminiLabs\SiteReviews\Application::load();
 	return !empty( $alias )
 		? $app->make( $alias )
@@ -14,7 +15,8 @@ 
 /**
  * @return \WP_Screen|object
  */
-function glsr_current_screen() {
+function glsr_current_screen()
+{
 	if( function_exists( 'get_current_screen' )) {
 		$screen = get_current_screen();
 	}
@@ -27,7 +29,8 @@ 
  * @param mixed ...$vars
  * @return void
  */
-function glsr_debug( ...$vars ) {
+function glsr_debug( ...$vars )
+{
 	if( count( $vars ) == 1 ) {
 		$value = htmlspecialchars( print_r( $vars[0], true ), ENT_QUOTES, 'UTF-8' );
 		printf( '<div class="glsr-debug"><pre>%s</pre></div>', $value );
@@ -46,14 +49,16 @@ 
  * @param mixed $fallback
  * @return string|array
  */
-function glsr_get_option( $path = '', $fallback = '' ) {
+function glsr_get_option( $path = '', $fallback = '' )
+{
 	return glsr( 'Database\OptionManager' )->get( 'settings.'.$path, $fallback );
 }
 
 /**
  * @return array
  */
-function glsr_get_options() {
+function glsr_get_options()
+{
 	return glsr( 'Database\OptionManager' )->get( 'settings' );
 }
 
@@ -61,7 +66,8 @@ 
  * @param int $post_id
  * @return void|\GeminiLabs\SiteReviews\Review
  */
-function glsr_get_review( $post_id ) {
+function glsr_get_review( $post_id )
+{
 	$post = get_post( $post_id );
 	if( $post instanceof WP_Post ) {
 		return glsr( 'Database\ReviewManager' )->single( $post );
@@ -72,14 +78,16 @@ 
  * @return array
  * @todo document change of $reviews->reviews to $reviews->results
  */
-function glsr_get_reviews( array $args = array() ) {
+function glsr_get_reviews( array $args = array() )
+{
 	return glsr( 'Database\ReviewManager' )->get( $args );
 }
 
 /**
  * @return \GeminiLabs\SiteReviews\Modules\Console
  */
-function glsr_log() {
+function glsr_log()
+{
 	$args = func_get_args();
 	$context = isset( $args[1] )
 		? $args[1]

compatibility.php

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
  */
 add_filter( 'sbp_exclude_defer_scripts', function( $scriptHandles ) {
 	$scriptHandles[] = 'site-reviews/google-recaptcha';
-	return array_keys( array_flip( $scriptHandles ));
+	return array_keys( array_flip( $scriptHandles ) );
 });
 
 /**
@@ -28,8 +28,8 @@ 
  * @see ??
  */
 add_filter( 'sf_edit_query_args', function( $query ) {
-	if( !empty( $query['meta_key'] ) && $query['meta_key'] == '_glsr_ranking' ) {
-		unset( $query['meta_key'] );
+	if( !empty($query['meta_key']) && $query['meta_key'] == '_glsr_ranking' ) {
+		unset($query['meta_key']);
 		$query['meta_query'] = [
 			'relation' => 'OR',
 			['key' => '_glsr_ranking', 'compare' => 'EXISTS'],