pryley / site-reviews

plugin/Database.php

Spacing +32 added lines, -32 removed lines

@@ -35,7 +35,7 @@ 
 			'post_type' => Application::POST_TYPE,
 		];
 		$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;
 		}
@@ -51,7 +51,7 @@ 
 	 */
 	public function deleteReview( $metaReviewId )
 	{
-		if( $postId = $this->getReviewPostId( $metaReviewId )) {
+		if( $postId = $this->getReviewPostId( $metaReviewId ) ) {
 			wp_delete_post( $postId, true );
 		}
 	}
@@ -64,12 +64,12 @@ 
 	public function getAssignedToPost( $post, $assignedTo = '' )
 	{
 		$post = get_post( $post );
-		if( !( $post instanceof WP_Post ))return;
-		if( empty( $assignedTo )) {
+		if( !($post instanceof WP_Post) )return;
+		if( empty($assignedTo) ) {
 			$assignedTo = get_post_meta( $post->ID, 'assigned_to', true );
 		}
 		$assignedPost = get_post( $assignedTo );
-		if( !empty( $assignedTo )
+		if( !empty($assignedTo)
 			&& $assignedPost instanceof WP_Post
 			&& $assignedPost->ID != $post->ID ) {
 			return $assignedPost;
@@ -82,7 +82,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;
@@ -104,13 +104,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;
 	}
@@ -131,9 +131,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' ) );
 	}
 
 	/**
@@ -162,7 +162,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'],
@@ -175,8 +175,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,
 		];
@@ -190,7 +190,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 );
@@ -206,10 +206,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 [];
 		}
@@ -221,7 +221,7 @@ 
 	 */
 	public function normalizeMeta( array $meta )
 	{
-		if( empty( $meta )) {
+		if( empty($meta) ) {
 			return [];
 		}
 		$defaults = wp_parse_args( $meta, [
@@ -229,7 +229,7 @@ 
 			'date' => '',
 			'review_id' => '',
 			'review_type' => '',
-		]);
+		] );
 		return glsr( CreateReviewDefaults::class )->restrict( $defaults );
 	}
 
@@ -240,7 +240,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;
@@ -253,10 +253,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;
@@ -269,15 +269,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() );
 		}
 	}
@@ -292,7 +292,7 @@ 
 			'post_status' => 'publish',
 			'post_type' => 'any',
 		];
-		if( is_numeric( $searchTerm )) {
+		if( is_numeric( $searchTerm ) ) {
 			$args['post__in'] = [$searchTerm];
 		}
 		else {
@@ -311,9 +311,9 @@ 
 			ob_start();
 			glsr()->render( 'partials/editor/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();
@@ -331,9 +331,9 @@ 
 			update_post_meta( $postId, $metaKey, $metaValue );
 		}
 		$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() );
 		}
 	}
@@ -345,7 +345,7 @@ 
 	protected function getNewPostStatus( array $review, $isBlacklisted )
 	{
 		$requireApprovalOption = glsr( OptionManager::class )->get( 'settings.general.require.approval' );
-		return $review['review_type'] == 'local' && ( $requireApprovalOption == 'yes' || $isBlacklisted )
+		return $review['review_type'] == 'local' && ($requireApprovalOption == 'yes' || $isBlacklisted)
 			? 'pending'
 			: 'publish';
 	}

Braces +18 added lines, -6 removed lines

@@ -64,7 +64,9 @@ 
 	public function getAssignedToPost( $post, $assignedTo = '' )
 	{
 		$post = get_post( $post );
-		if( !( $post instanceof WP_Post ))return;
+		if( !( $post instanceof WP_Post )) {
+			return;
+		}
 		if( empty( $assignedTo )) {
 			$assignedTo = get_post_meta( $post->ID, 'assigned_to', true );
 		}
@@ -82,7 +84,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;
@@ -256,7 +260,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;
@@ -269,7 +275,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,
@@ -304,7 +312,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();
@@ -331,7 +341,9 @@ 
 			update_post_meta( $postId, $metaKey, $metaValue );
 		}
 		$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/PublicController.php

Spacing +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@
 	public function routerSubmitReview( array $request )
 	{
 		$validated = glsr( ValidateReview::class )->validate( $request );
-		if( !empty( $validated->error ) || $validated->recaptchaIsUnset )return;
-		$this->execute( new CreateReview( $validated->request ));
+		if( !empty($validated->error) || $validated->recaptchaIsUnset )return;
+		$this->execute( new CreateReview( $validated->request ) );
 	}
 }

Braces +3 added lines, -1 removed lines

@@ -50,7 +50,9 @@
 	public function routerSubmitReview( array $request )
 	{
 		$validated = glsr( ValidateReview::class )->validate( $request );
-		if( !empty( $validated->error ) || $validated->recaptchaIsUnset )return;
+		if( !empty( $validated->error ) || $validated->recaptchaIsUnset ) {
+			return;
+		}
 		$this->execute( new CreateReview( $validated->request ));
 	}
 }

plugin/Database/SqlQueries.php

Spacing +15 added lines, -15 removed lines

@@ -23,11 +23,11 @@ 
 	 */
 	public function deleteAllSessions( $sessionCookiePrefix )
 	{
-		return $this->db->query("
+		return $this->db->query( "
 			DELETE
 			FROM {$this->db->options}
 			WHERE option_name LIKE '{$sessionCookiePrefix}_%'
-		");
+		" );
 	}
 
 	/**
@@ -36,11 +36,11 @@ 
 	 */
 	public function deleteExpiredSessions( $expiredSessions )
 	{
-		return $this->db->query("
+		return $this->db->query( "
 			DELETE
 			FROM {$this->db->options}
 			WHERE option_name IN ('{$expiredSessions}')
-		");
+		" );
 	}
 
 	/**
@@ -49,13 +49,13 @@ 
 	 */
 	public function getReviewCounts( $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 = '{$this->postType}' AND m.meta_key = '{$metaKey}'
 			GROUP BY name
-		");
+		" );
 	}
 
 	/**
@@ -65,13 +65,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}
-		");
+		" );
 	}
 
 	/**
@@ -80,14 +80,14 @@ 
 	 */
 	public function getReviewPostId( $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 pm ON p.ID = pm.post_id
 			WHERE p.post_type = '{$this->postType}'
 			AND pm.meta_key = 'review_id'
 			AND pm.meta_value = '{$metaReviewId}'
-		");
+		" );
 		return intval( $postId );
 	}
 
@@ -97,7 +97,7 @@ 
 	 */
 	public function getReviewIdsByType( $metaReviewType )
 	{
-		$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
@@ -106,8 +106,8 @@ 
 			AND m1.meta_key = 'review_id'
 			AND m2.meta_key = 'review_type'
 			AND m2.meta_value = '{$metaReviewType}'
-		");
-		return array_keys( array_flip( $query ));
+		" );
+		return array_keys( array_flip( $query ) );
 	}
 
 	/**
@@ -120,13 +120,13 @@ 
 		$queryBuilder = glsr( QueryBuilder::class );
 		$keys = $queryBuilder->buildSqlOr( $keys, "pm.meta_key = '%s'" );
 		$status = $queryBuilder->buildSqlOr( $status, "p.post_status = '%s'" );
-		return $this->db->get_col("
+		return $this->db->get_col( "
 			SELECT DISTINCT pm.meta_value FROM {$this->db->postmeta} pm
 			LEFT JOIN {$this->db->posts} p ON p.ID = pm.post_id
 			WHERE p.post_type = '{$this->postType}'
 			AND ({$keys})
 			AND ({$status})
 			ORDER BY pm.meta_value
-		");
+		" );
 	}
 }

plugin/Modules/Html/Partials/FormResults.php

Spacing +3 added lines, -3 removed lines

@@ -19,10 +19,10 @@ 
 	public function build( array $args = [] )
 	{
 		$this->errors = $args['errors'];
-		if( empty( $args['message'] ))return;
+		if( empty($args['message']) )return;
 		return glsr( Builder::class )->div( wpautop( $args['message'] ), [
 			'class' => $this->getClass(),
-		]);
+		] );
 	}
 
 	/**
@@ -30,7 +30,7 @@ 
 	 */
 	protected function getClass()
 	{
-		$errorClass = !empty( $this->errors )
+		$errorClass = !empty($this->errors)
 			? 'glsr-has-errors'
 			: '';
 		return trim( 'glsr-form-messages '.$errorClass );

Braces +3 added lines, -1 removed lines

@@ -19,7 +19,9 @@
 	public function build( array $args = [] )
 	{
 		$this->errors = $args['errors'];
-		if( empty( $args['message'] ))return;
+		if( empty( $args['message'] )) {
+			return;
+		}
 		return glsr( Builder::class )->div( wpautop( $args['message'] ), [
 			'class' => $this->getClass(),
 		]);

plugin/Modules/Html/Partials/SiteReviewsForm.php

Braces +9 added lines, -3 removed lines

@@ -152,7 +152,9 @@ 
 	 */
 	protected function normalizeFieldErrors( Field &$field )
 	{
-		if( !array_key_exists( $field->field['path'], $this->errors ))return;
+		if( !array_key_exists( $field->field['path'], $this->errors )) {
+			return;
+		}
 		$field->field['errors'] = $this->errors[$field->field['path']];
 	}
 
@@ -161,7 +163,9 @@ 
 	 */
 	protected function normalizeFieldRequired( Field &$field )
 	{
-		if( !in_array( $field->field['path'], $this->required ))return;
+		if( !in_array( $field->field['path'], $this->required )) {
+			return;
+		}
 		$field->field['required'] = true;
 	}
 
@@ -183,7 +187,9 @@ 
 	 */
 	protected function normalizeFieldValue( Field &$field )
 	{
-		if( !array_key_exists( $field->field['path'], $this->values ))return;
+		if( !array_key_exists( $field->field['path'], $this->values )) {
+			return;
+		}
 		if( in_array( $field->field['type'], ['radio', 'checkbox'] )) {
 			$field->field['checked'] = $field->field['value'] == $this->values[$field->field['path']];
 		}

Spacing +24 added lines, -24 removed lines

@@ -57,7 +57,7 @@ 
 				'submit_button' => $this->buildSubmitButton().$this->buildRecaptcha(),
 			],
 			'fields' => $this->getFields(),
-		]);
+		] );
 	}
 
 	/**
@@ -67,7 +67,7 @@ 
 	{
 		$integration = glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.integration' );
 		$recaptchaMethod = glsr( Helper::class )->buildMethodName( $integration, 'getRecaptcha' );
-		if( method_exists( $this, $recaptchaMethod )) {
+		if( method_exists( $this, $recaptchaMethod ) ) {
 			return $this->$recaptchaMethod();
 		}
 	}
@@ -80,7 +80,7 @@ 
 		return glsr( Partial::class )->build( 'form-results', [
 			'errors' => $this->errors,
 			'message' => $this->message,
-		]);
+		] );
 	}
 
 	/**
@@ -90,7 +90,7 @@ 
 	{
 		return glsr( Builder::class )->button( '<span></span>'.__( 'Submit your review', 'site-reviews' ), [
 			'type' => 'submit',
-		]);
+		] );
 	}
 
 	/**
@@ -110,7 +110,7 @@ 
 		$fields = array_merge(
 			$this->getHiddenFields(),
 			[$this->getHoneypotField()],
-			$this->normalizeFields( glsr( Form::class )->getFields( 'submission-form' ))
+			$this->normalizeFields( glsr( Form::class )->getFields( 'submission-form' ) )
 		);
 		return $fields;
 	}
@@ -123,27 +123,27 @@ 
 		$fields = [[
 			'name' => 'action',
 			'value' => 'submit-review',
-		],[
+		], [
 			'name' => 'assign_to',
 			'value' => $this->args['assign_to'],
-		],[
+		], [
 			'name' => 'category',
 			'value' => $this->args['category'],
-		],[
+		], [
 			'name' => 'excluded',
 			'value' => $this->args['excluded'], // @todo should default to "[]"
-		],[
+		], [
 			'name' => 'form_id',
 			'value' => $this->args['id'],
-		],[
+		], [
 			'name' => 'nonce',
 			'value' => wp_create_nonce( 'submit-review' ),
-		],[
+		], [
 			'name' => 'referer',
-			'value' => wp_unslash( filter_input( INPUT_SERVER, 'REQUEST_URI' )),
+			'value' => wp_unslash( filter_input( INPUT_SERVER, 'REQUEST_URI' ) ),
 		]];
 		return array_map( function( $field ) {
-			return new Field( wp_parse_args( $field, ['type' => 'hidden'] ));
+			return new Field( wp_parse_args( $field, ['type' => 'hidden'] ) );
 		}, $fields );
 	}
 
@@ -152,10 +152,10 @@ 
 	 */
 	protected function getHoneypotField()
 	{
-		return new Field([
+		return new Field( [
 			'name' => 'gotcha',
 			'type' => 'honeypot',
-		]);
+		] );
 	}
 
 	/**
@@ -163,12 +163,12 @@ 
 	 */
 	protected function getRecaptchaCustom()
 	{
-		return glsr( Builder::class )->div([
+		return glsr( Builder::class )->div( [
 			'class' => 'glsr-recaptcha-holder',
-			'data-badge' => sanitize_text_field( glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.position' )),
-			'data-sitekey' => sanitize_text_field( glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.key' )),
+			'data-badge' => sanitize_text_field( glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.position' ) ),
+			'data-sitekey' => sanitize_text_field( glsr( OptionManager::class )->get( 'settings.submissions.recaptcha.key' ) ),
 			'data-size' => 'invisible',
-		]);
+		] );
 	}
 
 	/**
@@ -181,7 +181,7 @@ 
 		$html = ob_get_clean();
 		return glsr( Builder::class )->div( $html, [
 			'class' => 'glsr-recaptcha-holder',
-		]);
+		] );
 	}
 
 	/**
@@ -189,7 +189,7 @@ 
 	 */
 	protected function normalizeFieldErrors( Field &$field )
 	{
-		if( !array_key_exists( $field->field['path'], $this->errors ))return;
+		if( !array_key_exists( $field->field['path'], $this->errors ) )return;
 		$field->field['errors'] = $this->errors[$field->field['path']];
 	}
 
@@ -198,7 +198,7 @@ 
 	 */
 	protected function normalizeFieldRequired( Field &$field )
 	{
-		if( !in_array( $field->field['path'], $this->required ))return;
+		if( !in_array( $field->field['path'], $this->required ) )return;
 		$field->field['required'] = true;
 	}
 
@@ -220,8 +220,8 @@ 
 	 */
 	protected function normalizeFieldValue( Field &$field )
 	{
-		if( !array_key_exists( $field->field['path'], $this->values ))return;
-		if( in_array( $field->field['type'], ['radio', 'checkbox'] )) {
+		if( !array_key_exists( $field->field['path'], $this->values ) )return;
+		if( in_array( $field->field['type'], ['radio', 'checkbox'] ) ) {
 			$field->field['checked'] = $field->field['value'] == $this->values[$field->field['path']];
 		}
 		else {

plugin/Modules/Html/Settings.php

Spacing +14 added lines, -14 removed lines

@@ -23,7 +23,7 @@ 
 			: $this->$method( $id );
 		return glsr( Template::class )->build( 'pages/settings/'.$id, [
 			'context' => $context,
-		]);
+		] );
 	}
 
 	/**
@@ -31,7 +31,7 @@ 
 	 */
 	protected function getFieldDefault( array $field )
 	{
-		return isset( $field['default'] )
+		return isset($field['default'])
 			? $field['default']
 			: '';
 	}
@@ -42,7 +42,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 );
 	}
@@ -53,14 +53,14 @@ 
 	 */
 	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, [
 				'is_setting' => true,
 				'name' => $name,
-			]);
-			$rows.= new Field( $this->normalize( $field ));
+			] );
+			$rows .= new Field( $this->normalize( $field ) );
 		}
 		return [
 			'rows' => $rows,
@@ -73,7 +73,7 @@ 
 	protected function getTemplateContextForTranslations()
 	{
 		$translations = glsr( Translator::class )->renderAll();
-		$class = empty( $translations )
+		$class = empty($translations)
 			? 'glsr-hidden'
 			: '';
 		return [
@@ -94,7 +94,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;
@@ -116,13 +116,13 @@ 
 	 */
 	protected function normalizeDependsOn( array $field )
 	{
-		if( !empty( $field['depends_on'] ) && is_array( $field['depends_on'] )) {
+		if( !empty($field['depends_on']) && is_array( $field['depends_on'] ) ) {
 			$path = key( $field['depends_on'] );
 			$expectedValue = $field['depends_on'][$path];
-			$field['data-depends'] = json_encode([
+			$field['data-depends'] = json_encode( [
 				'name' => glsr( Helper::class )->convertPathToName( $path, OptionManager::databaseKey() ),
 				'value' => $expectedValue,
-			], JSON_HEX_APOS|JSON_HEX_QUOT );
+			], JSON_HEX_APOS | JSON_HEX_QUOT );
 			$field['is_hidden'] = $this->isFieldHidden( $path, $expectedValue );
 		}
 		return $field;
@@ -133,9 +133,9 @@ 
 	 */
 	protected function normalizeLabelAndLegend( array $field )
 	{
-		if( !empty( $field['label'] )) {
+		if( !empty($field['label']) ) {
 			$field['legend'] = $field['label'];
-			unset( $field['label'] );
+			unset($field['label']);
 		}
 		else {
 			$field['is_valid'] = false;
@@ -149,7 +149,7 @@ 
 	 */
 	protected function normalizeValue( array $field )
 	{
-		if( !isset( $field['value'] )) {
+		if( !isset($field['value']) ) {
 			$field['value'] = glsr( OptionManager::class )->get(
 				$field['name'],
 				$this->getFieldDefault( $field )

plugin/Modules/Html/Field.php

Spacing +18 added lines, -18 removed lines

@@ -26,7 +26,7 @@ 
 			'is_valid' => true,
 			'is_widget' => false,
 			'path' => '',
-		]);
+		] );
 		$this->normalize();
 	}
 
@@ -45,7 +45,7 @@ 
 	{
 		if( !$this->field['is_valid'] )return;
 		if( $this->field['is_raw'] ) {
-			return glsr( Builder::class )->{$this->field['type']}( $this->field );
+			return glsr( Builder::class )->{$this->field['type']}($this->field);
 		}
 		if( !$this->field['is_setting'] ) {
 			return $this->buildField();
@@ -73,9 +73,9 @@ 
 			'context' => [
 				'class' => $this->getFieldClass(),
 				'errors' => $this->getFieldErrors(),
-				'field' => glsr( Builder::class )->{$this->field['type']}( $this->field ),
+				'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -86,10 +86,10 @@ 
 		return glsr( Template::class )->build( 'partials/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']] ),
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -98,16 +98,16 @@ 
 	protected function buildSettingMultiField()
 	{
 		$dependsOn = $this->getFieldDependsOn();
-		unset( $this->field['data-depends'] );
+		unset($this->field['data-depends']);
 		return glsr( Template::class )->build( 'partials/form/table-row-multiple', [
 			'context' => [
 				'class' => $this->getFieldClass(),
 				'depends_on' => $dependsOn,
-				'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'],
 			],
-		]);
+		] );
 	}
 
 	/**
@@ -116,13 +116,13 @@ 
 	protected function getFieldClass()
 	{
 		$classes = [];
-		if( !empty( $this->field['errors'] )) {
+		if( !empty($this->field['errors']) ) {
 			$classes[] = 'glsr-has-error';
 		}
 		if( $this->field['is_hidden'] ) {
 			$classes[] = 'hidden';
 		}
-		if( !empty( $this->field['required'] )) {
+		if( !empty($this->field['required']) ) {
 			$classes[] = 'glsr-required';
 		}
 		return implode( ' ', $classes );
@@ -133,7 +133,7 @@ 
 	 */
 	protected function getFieldDependsOn()
 	{
-		return !empty( $this->field['data-depends'] )
+		return !empty($this->field['data-depends'])
 			? $this->field['data-depends']
 			: '';
 	}
@@ -143,7 +143,7 @@ 
 	 */
 	protected function getFieldErrors()
 	{
-		if( empty( $this->field['errors'] ) || !is_array( $this->field['errors'] ))return;
+		if( empty($this->field['errors']) || !is_array( $this->field['errors'] ) )return;
 		$errors = array_reduce( $this->field['errors'], function( $carry, $error ) {
 			return $carry.glsr( Builder::class )->span( $error, ['class' => 'glsr-field-error'] );
 		});
@@ -170,13 +170,13 @@ 
 			'name', 'type',
 		];
 		foreach( $requiredValues as $value ) {
-			if( isset( $this->field[$value] ))continue;
+			if( isset($this->field[$value]) )continue;
 			$missingValues[] = $value;
 			$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 $this->field['is_valid'];
@@ -190,7 +190,7 @@ 
 		if( !$this->isFieldValid() )return;
 		$this->field['path'] = $this->field['name'];
 		$className = glsr( Helper::class )->buildClassName( $this->field['type'], __NAMESPACE__.'\Fields' );
-		if( class_exists( $className )) {
+		if( class_exists( $className ) ) {
 			$this->field = array_merge(
 				wp_parse_args( $this->field, $className::defaults() ),
 				$className::required()
@@ -205,7 +205,7 @@ 
 	 */
 	protected function normalizeFieldId()
 	{
-		if( isset( $this->field['id'] ) || $this->field['is_raw'] )return;
+		if( isset($this->field['id']) || $this->field['is_raw'] )return;
 		$this->field['id'] = glsr( Helper::class )->convertPathToId(
 			$this->field['path'],
 			$this->getFieldPrefix()

Braces +15 added lines, -5 removed lines

@@ -43,7 +43,9 @@ 
 	 */
 	public function build()
 	{
-		if( !$this->field['is_valid'] )return;
+		if( !$this->field['is_valid'] ) {
+			return;
+		}
 		if( $this->field['is_raw'] ) {
 			return glsr( Builder::class )->{$this->field['type']}( $this->field );
 		}
@@ -143,7 +145,9 @@ 
 	 */
 	protected function getFieldErrors()
 	{
-		if( empty( $this->field['errors'] ) || !is_array( $this->field['errors'] ))return;
+		if( empty( $this->field['errors'] ) || !is_array( $this->field['errors'] )) {
+			return;
+		}
 		$errors = array_reduce( $this->field['errors'], function( $carry, $error ) {
 			return $carry.glsr( Builder::class )->span( $error, ['class' => 'glsr-field-error'] );
 		});
@@ -170,7 +174,9 @@ 
 			'name', 'type',
 		];
 		foreach( $requiredValues as $value ) {
-			if( isset( $this->field[$value] ))continue;
+			if( isset( $this->field[$value] )) {
+				continue;
+			}
 			$missingValues[] = $value;
 			$this->field['is_valid'] = false;
 		}
@@ -187,7 +193,9 @@ 
 	 */
 	protected function normalize()
 	{
-		if( !$this->isFieldValid() )return;
+		if( !$this->isFieldValid() ) {
+			return;
+		}
 		$this->field['path'] = $this->field['name'];
 		$className = glsr( Helper::class )->buildClassName( $this->field['type'], __NAMESPACE__.'\Fields' );
 		if( class_exists( $className )) {
@@ -205,7 +213,9 @@ 
 	 */
 	protected function normalizeFieldId()
 	{
-		if( isset( $this->field['id'] ) || $this->field['is_raw'] )return;
+		if( isset( $this->field['id'] ) || $this->field['is_raw'] ) {
+			return;
+		}
 		$this->field['id'] = glsr( Helper::class )->convertPathToId(
 			$this->field['path'],
 			$this->getFieldPrefix()

plugin/Modules/Validator.php

Braces +12 added lines, -4 removed lines

@@ -73,7 +73,9 @@ 
 		foreach( $this->rules as $attribute => $rules ) {
 			foreach( $rules as $rule ) {
 				$this->validateAttribute( $attribute, $rule );
-				if( $this->shouldStopValidating( $attribute ))break;
+				if( $this->shouldStopValidating( $attribute )) {
+					break;
+				}
 			}
 		}
 		return $this->errors;
@@ -89,7 +91,9 @@ 
 	public function validateAttribute( $attribute, $rule )
 	{
 		list( $rule, $parameters ) = $this->parseRule( $rule );
-		if( $rule == '' )return;
+		if( $rule == '' ) {
+			return;
+		}
 		$value = $this->getValue( $attribute );
 		if( !method_exists( $this, $method = 'validate'.$rule )) {
 			throw new BadMethodCallException( "Method [$method] does not exist." );
@@ -158,7 +162,9 @@ 
 	 */
 	protected function getRule( $attribute, $rules )
 	{
-		if( !array_key_exists( $attribute, $this->rules ))return;
+		if( !array_key_exists( $attribute, $this->rules )) {
+			return;
+		}
 		$rules = (array) $rules;
 		foreach( $this->rules[$attribute] as $rule ) {
 			list( $rule, $parameters ) = $this->parseRule( $rule );
@@ -318,7 +324,9 @@ 
 		$message = isset( $strings[$key] )
 			? $strings[$key]
 			: false;
-		if( !$message )return;
+		if( !$message ) {
+			return;
+		}
 		if( method_exists( $this, $method = 'replace'.$rule )) {
 			$message = $this->$method( $message, $parameters );
 		}

Spacing +17 added lines, -17 removed lines

@@ -73,7 +73,7 @@ 
 		foreach( $this->rules as $attribute => $rules ) {
 			foreach( $rules as $rule ) {
 				$this->validateAttribute( $attribute, $rule );
-				if( $this->shouldStopValidating( $attribute ))break;
+				if( $this->shouldStopValidating( $attribute ) )break;
 			}
 		}
 		return $this->errors;
@@ -88,13 +88,13 @@ 
 	 */
 	public function validateAttribute( $attribute, $rule )
 	{
-		list( $rule, $parameters ) = $this->parseRule( $rule );
+		list($rule, $parameters) = $this->parseRule( $rule );
 		if( $rule == '' )return;
 		$value = $this->getValue( $attribute );
-		if( !method_exists( $this, $method = 'validate'.$rule )) {
+		if( !method_exists( $this, $method = 'validate'.$rule ) ) {
 			throw new BadMethodCallException( "Method [$method] does not exist." );
 		}
-		if( !$this->$method( $value, $attribute, $parameters )) {
+		if( !$this->$method( $value, $attribute, $parameters ) ) {
 			$this->addFailure( $attribute, $rule, $parameters );
 		}
 	}
@@ -143,7 +143,7 @@ 
 	 */
 	protected function getMessage( $attribute, $rule, array $parameters )
 	{
-		if( in_array( $rule, $this->sizeRules )) {
+		if( in_array( $rule, $this->sizeRules ) ) {
 			return $this->getSizeMessage( $attribute, $rule, $parameters );
 		}
 		$lowerRule = glsr( Helper::class )->snakeCase( $rule );
@@ -158,11 +158,11 @@ 
 	 */
 	protected function getRule( $attribute, $rules )
 	{
-		if( !array_key_exists( $attribute, $this->rules ))return;
-		$rules = (array) $rules;
+		if( !array_key_exists( $attribute, $this->rules ) )return;
+		$rules = (array)$rules;
 		foreach( $this->rules[$attribute] as $rule ) {
-			list( $rule, $parameters ) = $this->parseRule( $rule );
-			if( in_array( $rule, $rules )) {
+			list($rule, $parameters) = $this->parseRule( $rule );
+			if( in_array( $rule, $rules ) ) {
 				return [$rule, $parameters];
 			}
 		}
@@ -180,7 +180,7 @@ 
 		if( is_numeric( $value ) && $hasNumeric ) {
 			return $value;
 		}
-		elseif( is_array( $value )) {
+		elseif( is_array( $value ) ) {
 			return count( $value );
 		}
 		return mb_strlen( $value );
@@ -207,7 +207,7 @@ 
 	 */
 	protected function getValue( $attribute )
 	{
-		if( isset( $this->data[$attribute] )) {
+		if( isset($this->data[$attribute]) ) {
 			return $this->data[$attribute];
 		}
 	}
@@ -220,7 +220,7 @@ 
 	 */
 	protected function hasRule( $attribute, $rules )
 	{
-		return !is_null( $this->getRule( $attribute, $rules ));
+		return !is_null( $this->getRule( $attribute, $rules ) );
 	}
 
 	/**
@@ -259,11 +259,11 @@ 
 		$parameters = [];
 		// example: {rule}:{parameters}
 		if( strpos( $rule, ':' ) !== false ) {
-			list( $rule, $parameter ) = explode( ':', $rule, 2 );
+			list($rule, $parameter) = explode( ':', $rule, 2 );
 			// example: {parameter1,parameter2,...}
 			$parameters = $this->parseParameters( $rule, $parameter );
 		}
-		$rule = ucwords( str_replace( ['-', '_'], ' ', trim( $rule )));
+		$rule = ucwords( str_replace( ['-', '_'], ' ', trim( $rule ) ) );
 		$rule = str_replace( ' ', '', $rule );
 		return [$rule, $parameters];
 	}
@@ -290,7 +290,7 @@ 
 	protected function shouldStopValidating( $attribute )
 	{
 		return $this->hasRule( $attribute, $this->implicitRules )
-			&& isset( $this->failedRules[$attribute] )
+			&& isset($this->failedRules[$attribute])
 			&& array_intersect( array_keys( $this->failedRules[$attribute] ), $this->implicitRules );
 	}
 
@@ -315,11 +315,11 @@ 
 			'regex' => __( 'The format is invalid.', 'site-reviews' ),
 			'required' => __( 'This field is required.', 'site-reviews' ),
 		];
-		$message = isset( $strings[$key] )
+		$message = isset($strings[$key])
 			? $strings[$key]
 			: false;
 		if( !$message )return;
-		if( method_exists( $this, $method = 'replace'.$rule )) {
+		if( method_exists( $this, $method = 'replace'.$rule ) ) {
 			$message = $this->$method( $message, $parameters );
 		}
 		return $message;

plugin/Handlers/CreateReview.php

Braces +6 added lines, -2 removed lines

@@ -102,7 +102,9 @@ 
 	protected function sendNotification( $postId )
 	{
 		$notificationType = glsr( OptionManager::class )->get( 'settings.general.notification' );
-		if( !in_array( $notificationType, ['default','custom','webhook'] ))return;
+		if( !in_array( $notificationType, ['default','custom','webhook'] )) {
+			return;
+		}
 		$assignedToTitle = get_the_title( (int)$this->command->assigned_to );
 		$notificationSubject = _nx(
 			'New %s-star review',
@@ -150,7 +152,9 @@ 
 	 */
 	protected function sendWebhookNotification( array $args )
 	{
-		if( !( $endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' )))return;
+		if( !( $endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' ))) {
+			return;
+		}
 		$notification = $this->createWebhookNotification( $args );
 		$result = wp_remote_post( $endpoint, [
 			'blocking' => false,

Spacing +11 added lines, -11 removed lines

@@ -31,7 +31,7 @@ 
 		}
 		$this->sendNotification( $postId );
 		do_action( 'site-reviews/local/review/submitted', $postId, $command );
-		glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ));
+		glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ) );
 		if( $command->ajax_request ) {
 			glsr( Session::class )->clear();
 			return;
@@ -81,8 +81,8 @@ 
 		if( $this->command->author ) {
 			$fields[] = ['value' => trim( $this->command->author.$this->command->email.' - '.$this->command->ip_address )];
 		}
-		$fields[] = ['value' => sprintf( '<%s|%s>', $args['notification_link'], __( 'View Review', 'site-reviews' ))];
-		return json_encode([
+		$fields[] = ['value' => sprintf( '<%s|%s>', $args['notification_link'], __( 'View Review', 'site-reviews' ) )];
+		return json_encode( [
 			'icon_url' => glsr()->url( 'assets/img/icon.png' ),
 			'username' => glsr()->name,
 			'attachments' => [[
@@ -91,7 +91,7 @@ 
 				'fallback' => $this->createEmailNotification( $args )->read( 'plaintext' ),
 				'fields' => $fields,
 			]],
-		]);
+		] );
 	}
 
 	/**
@@ -101,12 +101,12 @@ 
 	protected function sendNotification( $postId )
 	{
 		$notificationType = glsr( OptionManager::class )->get( 'settings.general.notification' );
-		if( !in_array( $notificationType, ['default','custom','webhook'] ))return;
+		if( !in_array( $notificationType, ['default', 'custom', 'webhook'] ) )return;
 		$assignedToTitle = get_the_title( (int)$this->command->assigned_to );
 		$notificationSubject = _nx(
 			'New %s-star review',
 			'New %s-star review of: %s',
-			intval( empty( $assignedToTitle )),
+			intval( empty($assignedToTitle) ),
 			'The text is different depending on whether or not the review has been assigned to a post.',
 			'site-reviews'
 		);
@@ -115,7 +115,7 @@ 
 			sprintf( $notificationSubject, $this->command->rating, $assignedToTitle )
 		);
 		$args = [
-			'notification_link' => esc_url( admin_url( sprintf( 'post.php?post=%s&action=edit', $postId ))),
+			'notification_link' => esc_url( admin_url( sprintf( 'post.php?post=%s&action=edit', $postId ) ) ),
 			'notification_title' => $notificationTitle,
 			'notification_type' => $notificationType,
 		];
@@ -133,7 +133,7 @@ 
 		$args['recipient'] = $args['notification_type'] !== 'default'
 			? glsr( OptionManager::class )->get( 'settings.general.notification_email' )
 			: get_option( 'admin_email' );
-		if( empty( $args['recipient'] )) {
+		if( empty($args['recipient']) ) {
 			glsr_log()->error( 'Email notification was not sent: missing email, subject, or message.' );
 		}
 		else {
@@ -149,7 +149,7 @@ 
 	 */
 	protected function sendWebhookNotification( array $args )
 	{
-		if( !( $endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' )))return;
+		if( !($endpoint = glsr( OptionManager::class )->get( 'settings.general.webhook_url' )) )return;
 		$notification = $this->createWebhookNotification( $args );
 		$result = wp_remote_post( $endpoint, [
 			'blocking' => false,
@@ -160,8 +160,8 @@ 
 			'redirection' => 5,
 			'sslverify' => false,
 			'timeout' => 45,
-		]);
-		if( is_wp_error( $result )) {
+		] );
+		if( is_wp_error( $result ) ) {
 			glsr_log()->error( $result->get_error_message() );
 		}
 	}