Phprest\Service\Hateoas\Util

Complexity

Total Complexity

5

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Test Coverage

Coverage

96%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	serialize()	0	27	3
A	deserialize()	0	13	2

<?php

namespace Phprest\Service\Hateoas;

use JMS\Serializer\DeserializationContext;
use JMS\Serializer\SerializationContext;
use Negotiation\FormatNegotiator;
use Phprest\Exception;
use Phprest\Util\Mime;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;

trait Util
{
    use Mime;

    /**
     * @param mixed $value
     * @param Request $request
     * @param Response $response
     *
     * @throws Exception\NotAcceptable
     *
     * @return Response
     */
    protected function serialize($value, Request $request, Response $response)
    {
        $mimeProcResult = $this->processMime(
            (new FormatNegotiator())->getBest($request->headers->get('Accept', '*/*'))->getValue()
        );

        if ($mimeProcResult->mime === '*/*') {
            $mimeProcResult->mime = 'application/vnd.' . $mimeProcResult->vendor .
                '+json; version=' . $mimeProcResult->apiVersion;
            $mimeProcResult->format = 'json';
        }

        if (in_array($mimeProcResult->format, ['json', 'xml'])) {
            $response->setContent(
                $this->serviceHateoas()->serialize(
                    $value,
                    $mimeProcResult->format,
                    SerializationContext::create()->setVersion($mimeProcResult->apiVersion)

It seems like $mimeProcResult->apiVersion can also be of type string; however, parameter $version of JMS\Serializer\Context::setVersion() does only seem to accept integer, maybe add an additional type check?

                )
            );

            $response->headers->set('Content-Type', $mimeProcResult->mime);

            return $response;
        }

        throw new Exception\NotAcceptable(0, [$mimeProcResult->mime . ' is not supported']);
    }

    /**
     * @param string $type
     * @param Request $request
     *
     * @throws Exception\UnsupportedMediaType
     *
     * @return mixed
     */
    protected function deserialize($type, Request $request)
    {
        $mimeProcResult = $this->processMime($request->headers->get('Content-Type'));

        if (is_null($mimeProcResult->format)) {

The condition is_null($mimeProcResult->format) is always false.

            throw new Exception\UnsupportedMediaType();
        }

        return $this->serviceHateoas()->getSerializer()->deserialize(
            $request->getContent(),
            $type,
            $mimeProcResult->format,
            DeserializationContext::create()->setVersion($mimeProcResult->apiVersion)

It seems like $mimeProcResult->apiVersion can also be of type string; however, parameter $version of JMS\Serializer\Context::setVersion() does only seem to accept integer, maybe add an additional type check?

        );
    }

    /**
     * @return \Hateoas\Hateoas
     */
    abstract protected function serviceHateoas();
}