Whoops\Handler\JsonResponseHandler

Complexity

Total Complexity

7

Size/Duplication

Total Lines	73
Duplicated Lines	12.33 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

80%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	setJsonApi()	0	5	1
A	addTraceToOutput()	9	9	2
A	handle()	0	24	3
A	contentType()	0	4	1

<?php
/**
 * Whoops - php errors for cool kids
 * @author Filipe Dobreira <http://github.com/filp>
 */

namespace Whoops\Handler;

use Whoops\Exception\Formatter;

/**
 * Catches an exception and converts it to a JSON
 * response. Additionally can also return exception
 * frames for consumption by an API.
 */
class JsonResponseHandler extends Handler
{
    /**
     * @var bool
     */
    private $returnFrames = false;

    /**
     * @var bool
     */
    private $jsonApi = false;

    /**
     * Returns errors[[]] instead of error[] to be in compliance with the json:api spec
     * @param bool $jsonApi Default is false
     * @return static
     */
    public function setJsonApi($jsonApi = false)
    {
        $this->jsonApi = (bool) $jsonApi;
        return $this;
    }

    /**
     * @param  bool|null  $returnFrames
     * @return bool|static
     */
    public function addTraceToOutput($returnFrames = null)

This method seems to be duplicated in your project.

    {
        if (func_num_args() == 0) {
            return $this->returnFrames;
        }

        $this->returnFrames = (bool) $returnFrames;
        return $this;
    }

    /**
     * @return int
     */
    public function handle()
    {
        if ($this->jsonApi === true) {
            $response = [
                'errors' => [
                    Formatter::formatExceptionAsDataArray(
                        $this->getInspector(),
                        $this->addTraceToOutput()

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

                    ),
                ]
            ];
        } else {
            $response = [
                'error' => Formatter::formatExceptionAsDataArray(
                    $this->getInspector(),
                    $this->addTraceToOutput()

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

                ),
            ];
        }

        echo json_encode($response, defined('JSON_PARTIAL_OUTPUT_ON_ERROR') ? JSON_PARTIAL_OUTPUT_ON_ERROR : 0);

        return Handler::QUIT;
    }

    /**
     * @return string
     */
    public function contentType()
    {
        return 'application/json';
    }
}