Kami\ApiCoreBundle\RequestProcessor\ProcessorResponse

Complexity

Total Complexity

8

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	toHttpResponse()	0	6	2
A	createResponse()	0	8	1
A	getContentTypeByFormat()	0	12	3
A	getData()	0	3	1
A	__construct()	0	5	1

<?php

namespace Kami\ApiCoreBundle\RequestProcessor;

use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;


class ProcessorResponse implements ResponseInterface
{
    /**
     * @var array
     */
    protected $data;

    /**
     * @var Request
     */
    protected $request;

    /**
     * @var int
     */
    protected $status;

    /**
     * @var bool
     */
    protected $isHttpReady = false;

    public function __construct(Request $request, array $data, $httpReady = false, $status = 200)

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

    {
        $this->request = $request;
        $this->data = $data;
        $this->isHttpReady = $httpReady;
    }

    public function toHttpResponse()
    {
        if (!$this->isHttpReady) {
           throw new ProcessingException('Response is not ready yet to be set as http');
        }
        $this->createResponse($this->request);
    }

    /**
     * @return mixed
     */
    public function getData()
    {
       return $this->data;
    }

    /**
     * @param Request $request
     *
     * @return Response
     */
    private function createResponse(Request $request)
    {
        $format = $request->attributes->get('_format');

        return new Response(
            $this->data['response_data'],
            $this->status,
            ['Content-type' => $this->getContentTypeByFormat($format)]
        );
    }

    /**
     * @param string $format
     * @return string
     */
    private function getContentTypeByFormat($format)
    {
        switch ($format) {
            case 'json':
                return 'application/json';
                break;

break is not strictly necessary here and could be removed.

            case 'xml':
                return 'application/xml';
                break;
            default:
                return 'text/plain';
                break;
        }
    }
}