pryley / site-reviews

plugin/Database.php

Spacing +29 added lines, -29 removed lines

@@ -34,11 +34,11 @@ 
 			'post_type' => Application::POST_TYPE,
 		];
 		if( $review['review_type'] == 'local' && (
-			glsr( OptionManager::class )->get( 'settings.general.require.approval' ) == 'yes' || $command->blacklisted )) {
+			glsr( OptionManager::class )->get( 'settings.general.require.approval' ) == 'yes' || $command->blacklisted ) ) {
 			$post['post_status'] = 'pending';
 		}
 		$postId = wp_insert_post( $post, true );
-		if( is_wp_error( $postId )) {
+		if( is_wp_error( $postId ) ) {
 			glsr_log()->error( $postId->get_error_message() );
 			return false;
 		}
@@ -56,7 +56,7 @@ 
 	 */
 	public function deleteReview( $metaReviewId )
 	{
-		if( $postId = $this->getReviewPostId( $metaReviewId )) {
+		if( $postId = $this->getReviewPostId( $metaReviewId ) ) {
 			wp_delete_post( $postId, true );
 		}
 	}
@@ -67,7 +67,7 @@ 
 	 */
 	public function getReview( $post )
 	{
-		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE )return;
+		if( !($post instanceof WP_Post) || $post->post_type != Application::POST_TYPE )return;
 		$review = $this->getReviewMeta( $post->ID );
 		$modified = $this->isReviewModified( $post, $review );
 		$review->content = $post->post_content;
@@ -89,13 +89,13 @@ 
 	{
 		$metaKey = $this->normalizeMetaKey( $metaKey );
 		if( !$metaKey ) {
-			return (array) wp_count_posts( Application::POST_TYPE );
+			return (array)wp_count_posts( Application::POST_TYPE );
 		}
 		$counts = glsr( Cache::class )->getReviewCountsFor( $metaKey );
 		if( !$metaValue ) {
 			return $counts;
 		}
-		return isset( $counts[$metaValue] )
+		return isset($counts[$metaValue])
 			? $counts[$metaValue]
 			: 0;
 	}
@@ -116,9 +116,9 @@ 
 	public function getReviewMeta( $postId )
 	{
 		$meta = get_post_type( $postId ) == Application::POST_TYPE
-			? array_map( 'array_shift', (array) get_post_meta( $postId ))
+			? array_map( 'array_shift', (array)get_post_meta( $postId ) )
 			: [];
-		return (object) $this->normalizeMeta( array_filter( $meta, 'strlen' ));
+		return (object)$this->normalizeMeta( array_filter( $meta, 'strlen' ) );
 	}
 
 	/**
@@ -147,7 +147,7 @@ 
 		$paged = glsr( QueryBuilder::class )->getPaged(
 			wp_validate_boolean( $args['pagination'] )
 		);
-		$reviews = new WP_Query([
+		$reviews = new WP_Query( [
 			'meta_key' => 'pinned',
 			'meta_query' => $metaQuery,
 			'offset' => $args['offset'],
@@ -160,8 +160,8 @@ 
 			'post_type' => Application::POST_TYPE,
 			'posts_per_page' => $args['count'] ? $args['count'] : -1,
 			'tax_query' => $taxQuery,
-		]);
-		return (object) [
+		] );
+		return (object)[
 			'results' => array_map( [$this, 'getReview'], $reviews->posts ),
 			'max_num_pages' => $reviews->max_num_pages,
 		];
@@ -175,7 +175,7 @@ 
 	public function getReviewsMeta( $keys, $status = 'publish' )
 	{
 		$keys = array_map( [$this, 'normalizeMetaKey'], (array)$keys );
-		if( $status == 'all' || empty( $status )) {
+		if( $status == 'all' || empty($status) ) {
 			$status = get_post_stati( ['exclude_from_search' => false] );
 		}
 		return glsr( SqlQueries::class )->getReviewsMeta( $keys, $status );
@@ -191,10 +191,10 @@ 
 			'fields' => 'id=>name',
 			'hide_empty' => false,
 			'taxonomy' => Application::TAXONOMY,
-		]);
-		unset( $args['count'] ); //we don't want a term count
+		] );
+		unset($args['count']); //we don't want a term count
 		$terms = get_terms( $args );
-		if( is_wp_error( $terms )) {
+		if( is_wp_error( $terms ) ) {
 			glsr_log()->error( $terms->get_error_message() );
 			return [];
 		}
@@ -206,7 +206,7 @@ 
 	 */
 	public function normalizeMeta( array $meta )
 	{
-		if( empty( $meta )) {
+		if( empty($meta) ) {
 			return [];
 		}
 		$defaults = wp_parse_args( $meta, [
@@ -214,7 +214,7 @@ 
 			'date' => '',
 			'review_id' => '',
 			'review_type' => '',
-		]);
+		] );
 		return glsr( CreateReviewDefaults::class )->merge( $defaults, true );
 	}
 
@@ -225,7 +225,7 @@ 
 	public function normalizeMetaKey( $metaKey )
 	{
 		$metaKey = strtolower( $metaKey );
-		if( in_array( $metaKey, ['id', 'type'] )) {
+		if( in_array( $metaKey, ['id', 'type'] ) ) {
 			$metaKey = 'review_'.$metaKey;
 		}
 		return $metaKey;
@@ -238,10 +238,10 @@ 
 	public function normalizeTerms( $termIds )
 	{
 		$terms = [];
-		$termIds = array_map( 'trim', explode( ',', $termIds ));
+		$termIds = array_map( 'trim', explode( ',', $termIds ) );
 		foreach( $termIds as $termId ) {
 			$term = term_exists( $termId, Application::TAXONOMY );
-			if( !isset( $term['term_id'] ))continue;
+			if( !isset($term['term_id']) )continue;
 			$terms[] = intval( $term['term_id'] );
 		}
 		return $terms;
@@ -254,15 +254,15 @@ 
 	public function revertReview( $postId )
 	{
 		$post = get_post( $postId );
-		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE )return;
+		if( !($post instanceof WP_Post) || $post->post_type != Application::POST_TYPE )return;
 		delete_post_meta( $post->ID, '_edit_last' );
-		$result = wp_update_post([
+		$result = wp_update_post( [
 			'ID' => $post->ID,
 			'post_content' => get_post_meta( $post->ID, 'content', true ),
 			'post_date' => get_post_meta( $post->ID, 'date', true ),
 			'post_title' => get_post_meta( $post->ID, 'title', true ),
-		]);
-		if( is_wp_error( $result )) {
+		] );
+		if( is_wp_error( $result ) ) {
 			glsr_log()->error( $result->get_error_message() );
 		}
 	}
@@ -277,7 +277,7 @@ 
 			'post_status' => 'publish',
 			'post_type' => 'any',
 		];
-		if( is_numeric( $searchTerm )) {
+		if( is_numeric( $searchTerm ) ) {
 			$args['post__in'] = [$searchTerm];
 		}
 		else {
@@ -296,9 +296,9 @@ 
 			ob_start();
 			glsr()->render( 'edit/search-result', [
 				'ID' => get_the_ID(),
-				'permalink' => esc_url( (string) get_permalink() ),
+				'permalink' => esc_url( (string)get_permalink() ),
 				'title' => esc_attr( get_the_title() ),
-			]);
+			] );
 			$results .= ob_get_clean();
 		}
 		wp_reset_postdata();
@@ -313,9 +313,9 @@ 
 	public function setReviewMeta( $postId, $termIds )
 	{
 		$terms = $this->normalizeTerms( $termIds );
-		if( empty( $terms ))return;
+		if( empty($terms) )return;
 		$result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY );
-		if( is_wp_error( $result )) {
+		if( is_wp_error( $result ) ) {
 			glsr_log()->error( $result->get_error_message() );
 		}
 	}

Braces +15 added lines, -5 removed lines

@@ -67,7 +67,9 @@ 
 	 */
 	public function getReview( $post )
 	{
-		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE )return;
+		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE ) {
+			return;
+		}
 		$review = $this->getReviewMeta( $post->ID );
 		$modified = $this->isReviewModified( $post, $review );
 		$review->content = $post->post_content;
@@ -241,7 +243,9 @@ 
 		$termIds = array_map( 'trim', explode( ',', $termIds ));
 		foreach( $termIds as $termId ) {
 			$term = term_exists( $termId, Application::TAXONOMY );
-			if( !isset( $term['term_id'] ))continue;
+			if( !isset( $term['term_id'] )) {
+				continue;
+			}
 			$terms[] = intval( $term['term_id'] );
 		}
 		return $terms;
@@ -254,7 +258,9 @@ 
 	public function revertReview( $postId )
 	{
 		$post = get_post( $postId );
-		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE )return;
+		if( !( $post instanceof WP_Post ) || $post->post_type != Application::POST_TYPE ) {
+			return;
+		}
 		delete_post_meta( $post->ID, '_edit_last' );
 		$result = wp_update_post([
 			'ID' => $post->ID,
@@ -289,7 +295,9 @@ 
 		add_filter( 'posts_search', [$queryBuilder, 'filterSearchByTitle'], 500, 2 );
 		$search = new WP_Query( $args );
 		remove_filter( 'posts_search', [$queryBuilder, 'filterSearchByTitle'], 500 );
-		if( !$search->have_posts() )return;
+		if( !$search->have_posts() ) {
+			return;
+		}
 		$results = '';
 		while( $search->have_posts() ) {
 			$search->the_post();
@@ -313,7 +321,9 @@ 
 	public function setReviewMeta( $postId, $termIds )
 	{
 		$terms = $this->normalizeTerms( $termIds );
-		if( empty( $terms ))return;
+		if( empty( $terms )) {
+			return;
+		}
 		$result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY );
 		if( is_wp_error( $result )) {
 			glsr_log()->error( $result->get_error_message() );

plugin/Controllers/AjaxController.php

Spacing +16 added lines, -16 removed lines

@@ -23,7 +23,7 @@ 
 	{
 		wp_send_json( glsr( Html::class )->renderPartial( 'link', [
 			'post_id' => $request['ID'],
-		]));
+		] ) );
 	}
 
 	/**
@@ -31,7 +31,7 @@ 
 	 */
 	public function changeReviewStatus( array $request )
 	{
-		wp_send_json( $this->execute( new ChangeStatus( $request )));
+		wp_send_json( $this->execute( new ChangeStatus( $request ) ) );
 	}
 
 	/**
@@ -40,10 +40,10 @@ 
 	public function clearLog()
 	{
 		glsr( AdminController::class )->routerClearLog();
-		wp_send_json([
+		wp_send_json( [
 			'logger' => glsr( Logger::class )->get(),
 			'notices' => glsr( Notice::class )->get(),
-		]);
+		] );
 	}
 
 	/**
@@ -53,9 +53,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 = [
@@ -75,10 +75,10 @@ 
 	public function searchPosts( array $request )
 	{
 		$results = glsr_db()->searchPosts( $request['search'] );
-		wp_send_json_success([
+		wp_send_json_success( [
 			'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>',
 			'items' => $results,
-		]);
+		] );
 	}
 
 	/**
@@ -86,7 +86,7 @@ 
 	 */
 	public function searchTranslations( array $request )
 	{
-		if( empty( $request['exclude'] )) {
+		if( empty($request['exclude']) ) {
 			$request['exclude'] = [];
 		}
 		$results = glsr( Translator::class )
@@ -94,10 +94,10 @@ 
 			->exclude()
 			->exclude( $request['exclude'] )
 			->renderResults();
-		wp_send_json_success([
+		wp_send_json_success( [
 			'empty' => '<div>'.__( 'Nothing found.', 'site-reviews' ).'</div>',
 			'items' => $results,
-		]);
+		] );
 	}
 
 	/**
@@ -107,11 +107,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 ),
-		]);
+		] );
 	}
 
 	/**
@@ -119,9 +119,9 @@ 
 	 */
 	public function togglePinned( 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/Controllers/PublicController.php

Spacing +4 added lines, -4 removed lines

@@ -44,10 +44,10 @@ 
 		$vars[] = Application::PAGED_QUERY_VAR;
 		// dirty hack to fix a form submission with a field that has "name" as name
 		if( filter_input( INPUT_POST, 'action' ) == 'submit-review'
-			&& !is_null( filter_input( INPUT_POST, 'gotcha' ))) {
+			&& !is_null( filter_input( INPUT_POST, 'gotcha' ) ) ) {
 			$index = array_search( 'name', $vars, true );
 			if( false !== $index ) {
-				unset( $vars[$index] );
+				unset($vars[$index]);
 			}
 		}
 		return $vars;
@@ -59,10 +59,10 @@ 
 	public function postCreateReview( array $request )
 	{
 		$validated = glsr( ValidateReview::class )->validate( $request );
-		if( !empty( $validated->error )) {
+		if( !empty($validated->error) ) {
 			return $validated->request;
 		}
 		if( $validated->recaptchaIsUnset )return;
-		return $this->execute( new CreateReview( $validated->request ));
+		return $this->execute( new CreateReview( $validated->request ) );
 	}
 }

Braces +3 added lines, -1 removed lines

@@ -62,7 +62,9 @@
 		if( !empty( $validated->error )) {
 			return $validated->request;
 		}
-		if( $validated->recaptchaIsUnset )return;
+		if( $validated->recaptchaIsUnset ) {
+			return;
+		}
 		return $this->execute( new CreateReview( $validated->request ));
 	}
 }

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 );
@@ -43,7 +43,7 @@ 
 		foreach( $settings as $setting ) {
 			register_setting( Application::ID.'-'.$setting, OptionManager::databaseKey(), [
 				'sanitize_callback' => [$this, 'callbackRegisterSettings'],
-			]);
+			] );
 		}
 		glsr( Settings::class )->register();
 	}
@@ -53,9 +53,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']
 				: [];
 		}
@@ -67,13 +67,13 @@ 
 	 */
 	protected function sanitizeTranslations( array $input, array $options )
 	{
-		if( isset( $input['settings']['translations'] )) {
-			$options['settings']['translations'] = array_values( array_filter( $input['settings']['translations'] ));
+		if( isset($input['settings']['translations']) ) {
+			$options['settings']['translations'] = array_values( array_filter( $input['settings']['translations'] ) );
 			array_walk( $options['settings']['translations'], 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/Blacklist.php

Spacing +9 added lines, -9 removed lines

@@ -12,14 +12,14 @@ 
 	 */
 	public function isBlacklisted( CreateReview $review )
 	{
-		$target = implode( "\n", array_filter([
+		$target = implode( "\n", array_filter( [
 			$review->author,
 			$review->content,
 			$review->email,
 			$review->ipAddress,
 			$review->title,
-		]));
-		return (bool) apply_filters( 'site-reviews/blacklist/is-blacklisted',
+		] ) );
+		return (bool)apply_filters( 'site-reviews/blacklist/is-blacklisted',
 			$this->check( $target ),
 			$review
 		);
@@ -31,16 +31,16 @@ 
 	 */
 	protected function check( $target )
 	{
-		$blacklist = trim( glsr( OptionManager::class )->get( 'settings.reviews-form.blacklist.entries' ));
-		if( empty( $blacklist )) {
+		$blacklist = trim( glsr( OptionManager::class )->get( 'settings.reviews-form.blacklist.entries' ) );
+		if( empty($blacklist) ) {
 			return false;
 		}
 		$lines = explode( "\n", $blacklist );
-		foreach( (array) $lines as $line ) {
+		foreach( (array)$lines as $line ) {
 			$line = trim( $line );
-			if( empty( $line ) || 256 < strlen( $line ))continue;
-			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ));
-			if( preg_match( $pattern, $target )) {
+			if( empty($line) || 256 < strlen( $line ) )continue;
+			$pattern = sprintf( '#%s#i', preg_quote( $line, '#' ) );
+			if( preg_match( $pattern, $target ) ) {
 				return true;
 			}
 		}

plugin/Database/Cache.php

Spacing +2 added lines, -2 removed lines

@@ -17,11 +17,11 @@
 			$ipAddresses = array_fill_keys( ['v4', 'v6'], [] );
 			foreach( array_keys( $ipAddresses ) as $version ) {
 				$response = wp_remote_get( 'https://www.cloudflare.com/ips-'.$version );
-				if( is_wp_error( $response )) {
+				if( is_wp_error( $response ) ) {
 					glsr_log()->error( $response->get_error_message() );
 					continue;
 				}
-				$ipAddresses[$version] = array_filter( explode( PHP_EOL, wp_remote_retrieve_body( $response )));
+				$ipAddresses[$version] = array_filter( explode( PHP_EOL, wp_remote_retrieve_body( $response ) ) );
 			}
 			wp_cache_set( Application::ID, $ipAddresses, '_cloudflare_ips' );
 		}

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'] );
 		switch( $request['action'] ) {
 			case 'clear-log':
@@ -42,13 +42,13 @@ 
 	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 );
 		}
 		$controller = glsr( AjaxController::class );
 		$method = glsr( Helper::class )->buildMethodName( $request['action'] );
-		if( is_callable( [$controller, $method] )) {
+		if( is_callable( [$controller, $method] ) ) {
 			call_user_func( [$controller, $method], $request );
 		}
 		else {
@@ -62,9 +62,9 @@ 
 	 */
 	public function routePublicPostRequest()
 	{
-		switch( filter_input( INPUT_POST, 'action' )) {
+		switch( filter_input( INPUT_POST, 'action' ) ) {
 			case 'submit-review':
-				glsr( PublicController::class )->routerCreateReview( $this->normalize( $_POST ));
+				glsr( PublicController::class )->routerCreateReview( $this->normalize( $_POST ) );
 				break;
 		}
 	}
@@ -74,7 +74,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
 	}
@@ -112,14 +112,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();
 		}