FOS\RestBundle\Serializer\Normalizer\ExceptionHandler

Complexity

Total Complexity

15

Size/Duplication

Total Lines	151
Duplicated Lines	35.76 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

69.09%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getSubscribingMethods()	0	29	1
A	serializeToJson()	0	10	1
A	serializeErrorToJson()	0	10	1
A	serializeToXml()	27	27	4
A	serializeErrorToXml()	27	27	4
A	convertToArray()	0	4	1
A	convertThrowableToArray()	0	12	3

<?php

/*
 * This file is part of the FOSRestBundle package.
 *
 * (c) FriendsOfSymfony <http://friendsofsymfony.github.com/>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace FOS\RestBundle\Serializer\Normalizer;

use JMS\Serializer\Context;
use JMS\Serializer\GraphNavigatorInterface;
use JMS\Serializer\Handler\SubscribingHandlerInterface;
use JMS\Serializer\JsonSerializationVisitor;
use JMS\Serializer\XmlSerializationVisitor;

/**
 * @internal since 2.8
 */
class ExceptionHandler extends AbstractExceptionNormalizer implements SubscribingHandlerInterface
{
    /**
     * @return array
     */
    public static function getSubscribingMethods()
    {
        return [
            [
                'direction' => GraphNavigatorInterface::DIRECTION_SERIALIZATION,
                'format' => 'json',
                'type' => \Error::class,
                'method' => 'serializeErrorToJson',
            ],
            [
                'direction' => GraphNavigatorInterface::DIRECTION_SERIALIZATION,
                'format' => 'json',
                'type' => \Exception::class,
                'method' => 'serializeToJson',
            ],
            [
                'direction' => GraphNavigatorInterface::DIRECTION_SERIALIZATION,
                'format' => 'xml',
                'type' => \Error::class,
                'method' => 'serializeErrorToXml',
            ],
            [
                'direction' => GraphNavigatorInterface::DIRECTION_SERIALIZATION,
                'format' => 'xml',
                'type' => \Exception::class,
                'method' => 'serializeToXml',
            ],
        ];
    }

    /**
     * @param JsonSerializationVisitor $visitor
     * @param \Exception               $exception
     * @param array                    $type
     * @param Context                  $context
     *
     * @return array
     */
    public function serializeToJson(
        JsonSerializationVisitor $visitor,
        \Exception $exception,
        array $type,
        Context $context
    ) {
        $data = $this->convertToArray($exception, $context);

        return $visitor->visitArray($data, $type, $context);

The call to JsonSerializationVisitor::visitArray() has too many arguments starting with $context.

    }

    public function serializeErrorToJson(
        JsonSerializationVisitor $visitor,
        \Throwable $exception,
        array $type,
        Context $context
    ) {
        $data = $this->convertThrowableToArray($exception, $context);

        return $visitor->visitArray($data, $type, $context);

The call to JsonSerializationVisitor::visitArray() has too many arguments starting with $context.

    }

    /**
     * @param XmlSerializationVisitor $visitor
     * @param \Exception              $exception
     * @param array                   $type
     * @param Context                 $context
     */
    public function serializeToXml(

This method seems to be duplicated in your project.

        XmlSerializationVisitor $visitor,
        \Exception $exception,
        array $type,

The parameter $type is not used and could be removed.

        Context $context
    ) {
        $data = $this->convertToArray($exception, $context);

        $document = $visitor->getDocument(true);

The call to XmlSerializationVisitor::getDocument() has too many arguments starting with true.

        if (!$visitor->getCurrentNode()) {
            $visitor->createRoot();
        }

        foreach ($data as $key => $value) {
            $entryNode = $document->createElement($key);
            $visitor->getCurrentNode()->appendChild($entryNode);
            $visitor->setCurrentNode($entryNode);

            $node = $context->getNavigator()->accept($value, null, $context);

The call to GraphNavigatorInterface::accept() has too many arguments starting with $context.

            if (null !== $node) {
                $visitor->getCurrentNode()->appendChild($node);
            }

            $visitor->revertCurrentNode();
        }
    }

    public function serializeErrorToXml(

This method seems to be duplicated in your project.

        XmlSerializationVisitor $visitor,
        \Throwable $exception,
        array $type,

The parameter $type is not used and could be removed.

        Context $context
    ) {
        $data = $this->convertThrowableToArray($exception, $context);

        $document = $visitor->getDocument(true);

The call to XmlSerializationVisitor::getDocument() has too many arguments starting with true.

        if (!$visitor->getCurrentNode()) {
            $visitor->createRoot();
        }

        foreach ($data as $key => $value) {
            $entryNode = $document->createElement($key);
            $visitor->getCurrentNode()->appendChild($entryNode);
            $visitor->setCurrentNode($entryNode);

            $node = $context->getNavigator()->accept($value, null, $context);

The call to GraphNavigatorInterface::accept() has too many arguments starting with $context.

            if (null !== $node) {
                $visitor->getCurrentNode()->appendChild($node);
            }

            $visitor->revertCurrentNode();
        }
    }

    /**
     * @param \Exception $exception
     * @param Context    $context
     *
     * @return array
     */
    protected function convertToArray(\Exception $exception, Context $context)
    {
        return $this->convertThrowableToArray($exception, $context);
    }

    private function convertThrowableToArray(\Throwable $throwable, Context $context): array
    {
        $data = [];

        if ($context->hasAttribute('status_code')) {
            $data['code'] = $context->getAttribute('status_code');
        }

        $data['message'] = $this->getMessageFromThrowable($throwable, isset($statusCode) ? $statusCode : null);

The variable $statusCode seems to never exist, and therefore isset should always return false. Did you maybe rename this variable?

        return $data;
    }
}