pryley / site-reviews

plugin/Modules/Html/ReviewsHtml.php

Spacing +6 added lines, -6 removed lines

@@ -31,8 +31,8 @@ 
 		$this->reviews = $reviews;
 		$this->navigation = glsr( Partial::class )->build( 'pagination', [
 			'total' => $maxPageCount,
-		]);
-		parent::__construct( $reviews, ArrayObject::STD_PROP_LIST|ArrayObject::ARRAY_AS_PROPS );
+		] );
+		parent::__construct( $reviews, ArrayObject::STD_PROP_LIST | ArrayObject::ARRAY_AS_PROPS );
 	}
 
 	/**
@@ -58,7 +58,7 @@ 
 				'navigation' => $this->getNavigation(),
 				'reviews' => $this->getReviews(),
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -73,7 +73,7 @@ 
 			$defaults[] = 'glsr-ajax-pagination';
 		}
 		$classes = explode( ' ', $this->args['class'] );
-		$classes = array_unique( array_merge( $defaults, $classes ));
+		$classes = array_unique( array_merge( $defaults, $classes ) );
 		return implode( ' ', $classes );
 	}
 
@@ -92,7 +92,7 @@ 
 	 */
 	protected function getReviews()
 	{
-		return empty( $this->reviews )
+		return empty($this->reviews)
 			? $this->getReviewsFallback()
 			: implode( PHP_EOL, $this->reviews );
 	}
@@ -102,7 +102,7 @@ 
 	 */
 	protected function getReviewsFallback()
 	{
-		if( empty( $this->args['fallback'] ) && glsr( OptionManager::class )->getBool( 'settings.reviews.fallback' )) {
+		if( empty($this->args['fallback']) && glsr( OptionManager::class )->getBool( 'settings.reviews.fallback' ) ) {
 			$this->args['fallback'] = __( 'There are no reviews yet. Be the first one to write one.', 'site-reviews' );
 		}
 		$fallback = '<p class="glsr-no-margins">'.$this->args['fallback'].'</p>';

plugin/Modules/Translation.php

Spacing +25 added lines, -25 removed lines

@@ -30,7 +30,7 @@ 
 	{
 		$translations = $this->translations();
 		$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) ) {
 			$potFile = glsr()->path( glsr()->languages.'/'.Application::ID.'.pot' );
 			$entries = $this->extractEntriesFromPotFile( $potFile );
 			$entries = apply_filters( 'site-reviews/translation/entries', $entries );
@@ -88,13 +88,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->translations();
 		}
-		$keys = array_flip( glsr_array_column( $filterWith, 'id' ));
+		$keys = array_flip( glsr_array_column( $filterWith, 'id' ) );
 		$this->results = $intersect
 			? array_intersect_key( $entries, $keys )
 			: array_diff_key( $entries, $keys );
@@ -108,17 +108,17 @@ 
 	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
 		);
 		$data['data.class'] = $data['data.error'] = '';
-		if( array_search( $entry['s1'], glsr_array_column( $this->entries(), 'msgid' )) === false ) {
+		if( array_search( $entry['s1'], glsr_array_column( $this->entries(), 'msgid' ) ) === false ) {
 			$data['data.class'] = 'is-invalid';
 			$data['data.error'] = __( 'This custom translation is no longer valid as the original text has been changed or removed.', 'site-reviews' );
 		}
 		return glsr( Template::class )->build( 'partials/translations/'.$template, [
 			'context' => $data,
-		]);
+		] );
 	}
 
 	/**
@@ -150,13 +150,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' => json_encode( $data, JSON_HEX_APOS|JSON_HEX_QUOT|JSON_HEX_TAG|JSON_UNESCAPED_SLASHES|JSON_UNESCAPED_UNICODE ),
+				'entry' => json_encode( $data, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE ),
 				'text' => wp_strip_all_tags( $text ),
-			]);
+			] );
 		}
 		if( $resetAfterRender ) {
 			$this->reset();
@@ -189,12 +189,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;
 				}
 			}
@@ -212,10 +212,10 @@ 
 	public function translations()
 	{
 		static $translations;
-		if( empty( $translations )) {
+		if( empty($translations) ) {
 			$settings = glsr( OptionManager::class )->get( 'settings' );
-			$translations = isset( $settings['strings'] )
-				? $this->normalizeSettings( (array) $settings['strings'] )
+			$translations = isset($settings['strings'])
+				? $this->normalizeSettings( (array)$settings['strings'] )
 				: [];
 		}
 		return $translations;
@@ -227,8 +227,8 @@ 
 	 */
 	protected function getEntryString( array $entry, $key )
 	{
-		return isset( $entry[$key] )
-			? implode( '', (array) $entry[$key] )
+		return isset($entry[$key])
+			? implode( '', (array)$entry[$key] )
 			: '';
 	}
 
@@ -240,9 +240,9 @@ 
 		$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 ) {
-				if( is_array( $entry )) {
+				if( is_array( $entry ) ) {
 					$entry = $this->normalizeEntryString( $entry, $key );
 					continue;
 				}
@@ -258,7 +258,7 @@ 
 	 */
 	protected function normalizeEntryString( array $entry, $key )
 	{
-		if( isset( $entry[$key] )) {
+		if( isset($entry[$key]) ) {
 			$entry[$key] = $this->getEntryString( $entry, $key );
 		}
 		return $entry;
@@ -272,11 +272,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']);
 		});
 	}
 }

plugin/Modules/Console.php

Spacing +14 added lines, -14 removed lines

@@ -110,7 +110,7 @@ 
 	 */
 	public function get()
 	{
-		return empty( $this->log )
+		return empty($this->log)
 			? __( 'Console is empty', 'site-reviews' )
 			: $this->log;
 	}
@@ -122,11 +122,11 @@ 
 	public function humanSize( $valueIfEmpty = null )
 	{
 		$bytes = $this->size();
-		if( empty( $bytes ) && is_string( $valueIfEmpty )) {
+		if( empty($bytes) && is_string( $valueIfEmpty ) ) {
 			return $valueIfEmpty;
 		}
-		$exponent = floor( log( max( $bytes, 1 ), 1024 ));
-		return round( $bytes / pow( 1024, $exponent ), 2 ).' '.['bytes','KB','MB','GB'][$exponent];
+		$exponent = floor( log( max( $bytes, 1 ), 1024 ) );
+		return round( $bytes / pow( 1024, $exponent ), 2 ).' '.['bytes', 'KB', 'MB', 'GB'][$exponent];
 	}
 
 	/**
@@ -149,9 +149,9 @@ 
 	public function log( $level, $message, array $context = [] )
 	{
 		$constants = (new ReflectionClass( __CLASS__ ))->getConstants();
-		if( in_array( $level, $constants, true )) {
+		if( in_array( $level, $constants, true ) ) {
 			$entry = $this->buildLogEntry( $level, $message, $context );
-			file_put_contents( $this->file, $entry, FILE_APPEND|LOCK_EX );
+			file_put_contents( $this->file, $entry, FILE_APPEND | LOCK_EX );
 			$this->reset();
 		}
 		return $this;
@@ -163,10 +163,10 @@ 
 	public function logOnce()
 	{
 		$single = glsr( Session::class )->get( 'glsr_log_once', null, true );
-		if( !is_array( $single ))return;
+		if( !is_array( $single ) )return;
 		foreach( $single as $message => $debug ) {
-			if( !empty( $debug )) {
-				$message.= PHP_EOL.print_r( $debug, 1 );
+			if( !empty($debug) ) {
+				$message .= PHP_EOL.print_r( $debug, 1 );
 			}
 			$this->log( static::RECURRING, $message );
 		}
@@ -191,10 +191,10 @@ 
 	public function once( $message, $debug = '' )
 	{
 		$once = glsr( Session::class )->get( 'glsr_log_once', [] );
-		if( !is_array( $once )) {
+		if( !is_array( $once ) ) {
 			$once = [];
 		}
-		if( !isset( $once[$message] )) {
+		if( !isset($once[$message]) ) {
 			$once[$message] = $debug;
 			glsr( Session::class )->set( 'glsr_log_once', $once );
 		}
@@ -256,7 +256,7 @@ 
 	 */
 	protected function interpolate( $message, $context = [] )
 	{
-		if( $this->isObjectOrArray( $message ) || !is_array( $context )) {
+		if( $this->isObjectOrArray( $message ) || !is_array( $context ) ) {
 			return print_r( $message, true );
 		}
 		$replace = [];
@@ -284,7 +284,7 @@ 
 		if( $value instanceof DateTime ) {
 			$value = $value->format( 'Y-m-d H:i:s' );
 		}
-		else if( $this->isObjectOrArray( $value )) {
+		else if( $this->isObjectOrArray( $value ) ) {
 			$value = json_encode( $value );
 		}
 		return (string)$value;
@@ -299,7 +299,7 @@ 
 			$this->clear();
 			file_put_contents(
 				$this->file,
-				$this->buildLogEntry( 'info', __( 'Console was automatically cleared (128 KB maximum size)', 'site-reviews' ))
+				$this->buildLogEntry( 'info', __( 'Console was automatically cleared (128 KB maximum size)', 'site-reviews' ) )
 			);
 		}
 	}

Braces +3 added lines, -1 removed lines

@@ -163,7 +163,9 @@
 	public function logOnce()
 	{
 		$single = glsr( Session::class )->get( 'glsr_log_once', null, true );
-		if( !is_array( $single ))return;
+		if( !is_array( $single )) {
+			return;
+		}
 		foreach( $single as $message => $debug ) {
 			if( !empty( $debug )) {
 				$message.= PHP_EOL.print_r( $debug, 1 );

plugin/Commands/CreateReview.php

Spacing +20 added lines, -20 removed lines

@@ -30,25 +30,25 @@ 
 	public function __construct( $input )
 	{
 		$this->request = $input;
-		$this->ajax_request = isset( $input['_ajax_request'] );
+		$this->ajax_request = isset($input['_ajax_request']);
 		$this->assigned_to = $this->getNumeric( 'assign_to' );
-		$this->author = sanitize_text_field( $this->getUser( 'name' ));
+		$this->author = sanitize_text_field( $this->getUser( 'name' ) );
 		$this->avatar = $this->getAvatar();
-		$this->blacklisted = isset( $input['blacklisted'] );
-		$this->category = sanitize_key( $this->get( 'category' ));
-		$this->content = sanitize_textarea_field( $this->get( 'content' ));
+		$this->blacklisted = isset($input['blacklisted']);
+		$this->category = sanitize_key( $this->get( 'category' ) );
+		$this->content = sanitize_textarea_field( $this->get( 'content' ) );
 		$this->custom = $this->getCustom();
 		$this->date = $this->getDate( 'date' );
-		$this->email = sanitize_email( $this->getUser( 'email' ));
-		$this->form_id = sanitize_key( $this->get( 'form_id' ));
+		$this->email = sanitize_email( $this->getUser( 'email' ) );
+		$this->form_id = sanitize_key( $this->get( 'form_id' ) );
 		$this->ip_address = $this->get( 'ip_address' );
-		$this->post_id = intval( $this->get( '_post_id' ));
-		$this->rating = intval( $this->get( 'rating' ));
+		$this->post_id = intval( $this->get( '_post_id' ) );
+		$this->rating = intval( $this->get( 'rating' ) );
 		$this->referer = $this->get( '_referer' );
-		$this->response = sanitize_textarea_field( $this->get( 'response' ));
-		$this->terms = !empty( $input['terms'] );
-		$this->title = sanitize_text_field( $this->get( 'title' ));
-		$this->url = esc_url_raw( $this->get( 'url' ));
+		$this->response = sanitize_textarea_field( $this->get( 'response' ) );
+		$this->terms = !empty($input['terms']);
+		$this->title = sanitize_text_field( $this->get( 'title' ) );
+		$this->url = esc_url_raw( $this->get( 'url' ) );
 	}
 
 	/**
@@ -57,7 +57,7 @@ 
 	 */
 	protected function get( $key )
 	{
-		return isset( $this->request[$key] )
+		return isset($this->request[$key])
 			? (string)$this->request[$key]
 			: '';
 	}
@@ -69,7 +69,7 @@ 
 	{
 		$avatar = $this->get( 'avatar' );
 		return !filter_var( $avatar, FILTER_VALIDATE_URL, FILTER_FLAG_PATH_REQUIRED )
-			? (string)get_avatar_url( $this->get( 'email' ))
+			? (string)get_avatar_url( $this->get( 'email' ) )
 			: $avatar;
 	}
 
@@ -86,7 +86,7 @@ 
 		$unset = apply_filters( 'site-reviews/create/unset-keys-from-custom', $unset );
 		$custom = $this->request;
 		foreach( $unset as $value ) {
-			unset( $custom[$value] );
+			unset($custom[$value]);
 		}
 		return $custom;
 	}
@@ -97,11 +97,11 @@ 
 	 */
 	protected function getDate( $key )
 	{
-		$date = strtotime( $this->get( $key ));
+		$date = strtotime( $this->get( $key ) );
 		if( $date === false ) {
 			$date = time();
 		}
-		return get_date_from_gmt( gmdate( 'Y-m-d H:i:s', $date ));
+		return get_date_from_gmt( gmdate( 'Y-m-d H:i:s', $date ) );
 	}
 
 	/**
@@ -111,13 +111,13 @@ 
 	protected function getUser( $key )
 	{
 		$value = $this->get( $key );
-		if( empty( $value )) {
+		if( empty($value) ) {
 			$user = wp_get_current_user();
 			$userValues = [
 				'email' => 'user_email',
 				'name' => 'display_name',
 			];
-			if( $user->exists() && array_key_exists( $key, $userValues )) {
+			if( $user->exists() && array_key_exists( $key, $userValues ) ) {
 				return $user->{$userValues[$key]};
 			}
 		}

plugin/Database/SqlQueries.php

Spacing +26 added lines, -26 removed lines

@@ -21,11 +21,11 @@ 
 	 */
 	public function deleteAllSessions( $sessionCookiePrefix )
 	{
-		return $this->db->query("
+		return $this->db->query( "
 			DELETE
 			FROM {$this->db->options}
 			WHERE option_name LIKE '{$sessionCookiePrefix}_%'
-		");
+		" );
 	}
 
 	/**
@@ -34,11 +34,11 @@ 
 	 */
 	public function deleteExpiredSessions( $expiredSessions )
 	{
-		return $this->db->query("
+		return $this->db->query( "
 			DELETE
 			FROM {$this->db->options}
 			WHERE option_name IN ('{$expiredSessions}')
-		");
+		" );
 	}
 
 	/**
@@ -48,13 +48,13 @@ 
 	 */
 	public function getExpiredSessions( $sessionCookiePrefix, $limit )
 	{
-		return $this->db->get_results("
+		return $this->db->get_results( "
 			SELECT option_name AS name, option_value AS expiration
 			FROM {$this->db->options}
 			WHERE option_name LIKE '{$sessionCookiePrefix}_expires_%'
 			ORDER BY option_value ASC
 			LIMIT 0, {$limit}
-		");
+		" );
 	}
 
 	/**
@@ -63,14 +63,14 @@ 
 	 */
 	public function getPostIdFromReviewId( $metaReviewId )
 	{
-		$postId = $this->db->get_var("
+		$postId = $this->db->get_var( "
 			SELECT p.ID
 			FROM {$this->db->posts} AS p
 			INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
 			WHERE p.post_type = '{Application::POST_TYPE}'
 			AND m.meta_key = 'review_id'
 			AND m.meta_value = '{$metaReviewId}'
-		");
+		" );
 		return intval( $postId );
 	}
 
@@ -81,7 +81,7 @@ 
 	 */
 	public function getReviewCounts( array $args, $lastPostId = 0, $limit = 500 )
 	{
-		return $this->db->get_results("
+		return $this->db->get_results( "
 			SELECT p.ID, m1.meta_value AS rating, m2.meta_value AS type
 			FROM {$this->db->posts} AS p
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -104,14 +104,14 @@ 
 	 */
 	public function getReviewCountsFor( $metaKey )
 	{
-		return (array) $this->db->get_results("
+		return (array)$this->db->get_results( "
 			SELECT m.meta_value AS name, COUNT(*) num_posts
 			FROM {$this->db->posts} AS p
 			INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
 			WHERE p.post_type = '{Application::POST_TYPE}'
 			AND m.meta_key = '{$metaKey}'
 			GROUP BY name
-		");
+		" );
 	}
 
 	/**
@@ -120,7 +120,7 @@ 
 	 */
 	public function getReviewIdsByType( $reviewType )
 	{
-		$query = $this->db->get_col("
+		$query = $this->db->get_col( "
 			SELECT m1.meta_value AS review_id
 			FROM {$this->db->posts} AS p
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -129,8 +129,8 @@ 
 			AND m1.meta_key = 'review_id'
 			AND m2.meta_key = 'review_type'
 			AND m2.meta_value = '{$reviewType}'
-		");
-		return array_keys( array_flip( $query ));
+		" );
+		return array_keys( array_flip( $query ) );
 	}
 
 	/**
@@ -141,9 +141,9 @@ 
 	public function getReviewRatingsFromIds( array $postIds, $greaterThanId = 0, $limit = 100 )
 	{
 		sort( $postIds );
-		$postIds = array_slice( $postIds, intval( array_search( $greaterThanId, $postIds )), $limit );
+		$postIds = array_slice( $postIds, intval( array_search( $greaterThanId, $postIds ) ), $limit );
 		$postIds = implode( ',', $postIds );
-		return $this->db->get_results("
+		return $this->db->get_results( "
 			SELECT p.ID, m.meta_value AS rating
 			FROM {$this->db->posts} AS p
 			INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
@@ -167,7 +167,7 @@ 
 		$queryBuilder = glsr( QueryBuilder::class );
 		$key = $queryBuilder->buildSqlOr( $key, "m.meta_key = '%s'" );
 		$status = $queryBuilder->buildSqlOr( $status, "p.post_status = '%s'" );
-		return $this->db->get_col("
+		return $this->db->get_col( "
 			SELECT DISTINCT m.meta_value
 			FROM {$this->db->postmeta} m
 			LEFT JOIN {$this->db->posts} p ON p.ID = m.post_id
@@ -175,7 +175,7 @@ 
 			AND ({$key})
 			AND ({$status})
 			ORDER BY m.meta_value
-		");
+		" );
 	}
 
 	/**
@@ -184,11 +184,11 @@ 
 	 */
 	protected function getAndForCounts( array $args, $and = '' )
 	{
-		if( !empty( $args['post_id'] )) {
-			$and.= "AND m3.meta_key = 'assigned_to' AND m3.meta_value = {$args['post_id']}";
+		if( !empty($args['post_id']) ) {
+			$and .= "AND m3.meta_key = 'assigned_to' AND m3.meta_value = {$args['post_id']}";
 		}
-		if( !empty( $args['term_taxonomy_id'] )) {
-			$and.= "AND tr.term_taxonomy_id = {$args['term_taxonomy_id']}";
+		if( !empty($args['term_taxonomy_id']) ) {
+			$and .= "AND tr.term_taxonomy_id = {$args['term_taxonomy_id']}";
 		}
 		return $and;
 	}
@@ -199,11 +199,11 @@ 
 	 */
 	protected function getInnerJoinForCounts( array $args, $innerJoin = '' )
 	{
-		if( !empty( $args['post_id'] )) {
-			$innerJoin.= "INNER JOIN {$this->db->postmeta} AS m3 ON p.ID = m3.post_id";
+		if( !empty($args['post_id']) ) {
+			$innerJoin .= "INNER JOIN {$this->db->postmeta} AS m3 ON p.ID = m3.post_id";
 		}
-		if( !empty( $args['term_taxonomy_id'] )) {
-			$innerJoin.= "INNER JOIN {$this->db->term_relationships} AS tr ON p.ID = tr.object_id";
+		if( !empty($args['term_taxonomy_id']) ) {
+			$innerJoin .= "INNER JOIN {$this->db->term_relationships} AS tr ON p.ID = tr.object_id";
 		}
 		return $innerJoin;
 	}