GeminiLabs\SiteReviews\Handlers\CreateReview

Complexity

Total Complexity

6

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getReferer()	0	11	3
A	handle()	0	15	3

<?php

namespace GeminiLabs\SiteReviews\Handlers;

use GeminiLabs\SiteReviews\Commands\CreateReview as Command;
use GeminiLabs\SiteReviews\Database\ReviewManager;
use GeminiLabs\SiteReviews\Modules\Notification;
use GeminiLabs\SiteReviews\Modules\Session;

class CreateReview
{
	/**
	 * @return void|string
	 */
	public function handle( Command $command )
	{
		$review = glsr( ReviewManager::class )->create( $command );
		if( !$review ) {
			glsr( Session::class )->set( $command->form_id.'errors', [] );
			glsr( Session::class )->set( $command->form_id.'message', __( 'Your review could not be submitted and the error has been logged. Please notify the site admin.', 'site-reviews' ));
			return;
		}
		glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ));
		glsr( Notification::class )->send( $review );
		do_action( 'site-reviews/local/review/submitted', null, $command ); // @compat
		do_action( 'site-reviews/review/submitted', $review );
		if( $command->ajax_request )return;
		wp_safe_redirect( $this->getReferer( $command ));
		exit;

Using exit here is not recommended.

	}

	/**
	 * @return string
	 */
	protected function getReferer( Command $command )
	{
		$referer = trim( get_post_meta( $command->post_id, 'redirect_to', true ));

It seems like get_post_meta($command->post_id, 'redirect_to', true) can also be of type false; however, parameter $str of trim() does only seem to accept string, maybe add an additional type check?

		if( empty( $referer )) {
			$referer = $command->referer;
		}
		if( empty( $referer )) {
			glsr_log()->warning( 'The form referer ($_SERVER[REQUEST_URI]) was empty.' )->info( $command );
			$referer = home_url();
		}
		return $referer;
	}
}