Whoops\Handler\XmlResponseHandler

Complexity

Total Complexity

11

Size/Duplication

Total Lines	91
Duplicated Lines	9.89 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

88.24%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	addTraceToOutput()	9	9	2
A	handle()	0	13	1
A	contentType()	0	4	1
A	addDataToNode()	0	24	5
A	toXml()	0	8	2

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

namespace Whoops\Handler;

use SimpleXMLElement;
use Whoops\Exception\Formatter;

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

    /**
     * @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()
    {
        $response = [
            'error' => Formatter::formatExceptionAsDataArray(
                $this->getInspector(),
                $this->addTraceToOutput()

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

            ),
        ];

        echo self::toXml($response);

        return Handler::QUIT;
    }

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

    /**
     * @param  SimpleXMLElement  $node Node to append data to, will be modified in place
     * @param  array|\Traversable $data
     * @return SimpleXMLElement  The modified node, for chaining
     */
    private static function addDataToNode(\SimpleXMLElement $node, $data)
    {
        assert(is_array($data) || $data instanceof Traversable);

The class Whoops\Handler\Traversable does not exist. Did you forget a USE statement, or did you not list all dependencies?

        foreach ($data as $key => $value) {
            if (is_numeric($key)) {
                // Convert the key to a valid string
                $key = "unknownNode_". (string) $key;
            }

            // Delete any char not allowed in XML element names
            $key = preg_replace('/[^a-z0-9\-\_\.\:]/i', '', $key);

            if (is_array($value)) {
                $child = $node->addChild($key);
                self::addDataToNode($child, $value);
            } else {
                $value = str_replace('&', '&', print_r($value, true));
                $node->addChild($key, $value);
            }
        }

        return $node;
    }

    /**
     * The main function for converting to an XML document.
     *
     * @param  array|\Traversable $data
     * @return string            XML
     */
    private static function toXml($data)
    {
        assert(is_array($data) || $data instanceof Traversable);

The class Whoops\Handler\Traversable does not exist. Did you forget a USE statement, or did you not list all dependencies?

        $node = simplexml_load_string("<?xml version='1.0' encoding='utf-8'?><root />");

        return self::addDataToNode($node, $data)->asXML();
    }
}