Rest_Api_Handler::isRestRequest() - Code Metrics - Rarst/wps - Measure and Improve Code Quality continuously with Scrutinizer

Rest_Api_Handler::isRestRequest() A
last analyzed 2020-03-25 09:57 UTC

↳ Parent: Rest_Api_Handler

Complexity

Conditions	5
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	13
rs	9.5222
c	0
b	0
f	0
cc	5
nc	3
nop	0

<?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 for REST API.
 */
class Rest_Api_Handler extends JsonResponseHandler {

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

		if ( defined( 'REST_REQUEST' ) && REST_REQUEST ) {
			return true;
		}

		// This is dirty, but no better way to detect before parse_request.
		if ( ! empty( $_SERVER['REQUEST_URI'] ) && false !== stripos( $_SERVER['REQUEST_URI'], rest_get_url_prefix() ) ) {
			return true;
		}

		return false;
	}

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

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

		$data     = Formatter::formatExceptionAsDataArray( $this->getInspector(), $this->addTraceToOutput() );

		$response = array(
			'code'    => $data['type'],
			'message' => $data['message'],
			'data'    => $data,
		);

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

		echo wp_json_encode( $response, JSON_PRETTY_PRINT );

		return Handler::QUIT;
	}
}


1			<?php
2			namespace Rarst\wps;
3
4			use Whoops\Exception\Formatter;
5			use Whoops\Handler\Handler;
6			use Whoops\Handler\JsonResponseHandler;
7			use Whoops\Util\Misc;
8
9			/**
10			* WordPress-specific version of Json handler for REST API.
11			*/
12			class Rest_Api_Handler extends JsonResponseHandler {
13
14			/**
15			* @return bool
16			*/
17			private function isRestRequest() {
18
19			if ( defined( 'REST_REQUEST' ) && REST_REQUEST ) {
20			return true;
21			}
22
23			// This is dirty, but no better way to detect before parse_request.
24			if ( ! empty( $_SERVER['REQUEST_URI'] ) && false !== stripos( $_SERVER['REQUEST_URI'], rest_get_url_prefix() ) ) {
25			return true;
26			}
27
28			return false;
29			}
30
31			/**
32			* @return int
33			*/
34			public function handle() {
35
36			if ( ! $this->isRestRequest() ) {
37			return Handler::DONE;
38			}
39
40			$data = Formatter::formatExceptionAsDataArray( $this->getInspector(), $this->addTraceToOutput() );
			0 ignored issues – show Bug introduced 2018-02-05 18:04 UTC by Report Bug Copy Issue Report It seems like `$this->addTraceToOutput()` targeting `Whoops\Handler\JsonRespo...ler::addTraceToOutput()` can also be of type `this<Rarst\wps\Rest_Api_Handler>`; however, `Whoops\Exception\Formatt...tExceptionAsDataArray()` does only seem to accept `boolean`, maybe add an additional type check? This check looks at variables that are passed out again to other methods. If the outgoing method call has stricter type requirements than the method itself, an issue is raised. An additional type check may prevent trouble. Loading history...
41			$response = array(
42			'code' => $data['type'],
43			'message' => $data['message'],
44			'data' => $data,
45			);
46
47			if ( Misc::canSendHeaders() ) {
48			status_header( 500 );
49			header( 'Content-Type: application/json; charset=' . get_option( 'blog_charset' ) );
50			}
51
52			echo wp_json_encode( $response, JSON_PRETTY_PRINT );
53
54			return Handler::QUIT;
55			}
56			}
57

Rarst / wps

Rest_Api_Handler::isRestRequest() A last analyzed 2020-03-25 09:57 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

Rest_Api_Handler::isRestRequest() A
last analyzed 2020-03-25 09:57 UTC