Rarst\wps\Admin_Ajax_Handler

Complexity

Total Complexity

5

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	isAjaxRequest()	0	4	2
A	handle()	0	19	3

<?php
namespace Rarst\wps;

use Whoops\Exception\Formatter;
use Whoops\Handler\Handler;
use Whoops\Handler\JsonResponseHandler;
use Whoops\Util\Misc;

/**
 * WordPress-specific version of Json handler.
 */
class Admin_Ajax_Handler extends JsonResponseHandler {

	/**
	 * @return bool
	 */
	private function isAjaxRequest() {

		return defined( 'DOING_AJAX' ) && DOING_AJAX;
	}

	/**
	 * @return int
	 */
	public function handle() {

		if ( ! $this->isAjaxRequest() ) {
			return Handler::DONE;
		}

		$response = array(
			'success' => false,
			'data'    => Formatter::formatExceptionAsDataArray( $this->getInspector(), $this->addTraceToOutput() ),

It seems like $this->addTraceToOutput() targeting Whoops\Handler\JsonResponseHandler::addTraceToOutput() can also be of type this<Rarst\wps\Admin_Ajax_Handler>; however, Whoops\Exception\Formatter::formatExceptionAsDataArray() does only seem to accept boolean, maybe add an additional type check?

		);

		if ( Misc::canSendHeaders() ) {
			header( 'Content-Type: application/json; charset=' . get_option( 'blog_charset' ) );
		}

		echo wp_json_encode( $response, JSON_PRETTY_PRINT );

		return Handler::QUIT;
	}
}