Core\Renderers\JsonApiRenderer::render()

Complexity

Conditions	6
Paths	6

Size

Total Lines	26
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function render(ResponseInterface $response, array $data = [], $status = 200, $addHeaders = [])
    {
        $status = intval($status);
        $output = [
            'meta' => ['error' => true, 'status' => $status],
            'data' => $data,
        ];
        $output['meta']['error'] = ($status < 400) ? false : true;
        $json = json_encode($output, JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES);
        // Ensure that the json encoding passed successfully
        if ($json === false) {

The condition $json === false can never be true.

            throw new \RuntimeException(json_last_error_msg(), json_last_error());
        }
        $body = new Body(fopen('php://temp', 'r+'));

It seems like fopen('php://temp', 'r+') can also be of type false; however, parameter $stream of Slim\Http\Body::__construct() does only seem to accept resource, maybe add an additional type check?

        $body->write($json);
        $newResponse = $response->withBody($body);
        $newResponse = $newResponse->withStatus($status)
            ->withHeader('Content-Type', 'application/json;charset=utf-8');
        if (count($addHeaders)) {
            foreach ($addHeaders as $headerKey => $headerValue) {
                if (strtolower($headerKey) != 'content-type') {
                    $newResponse->withHeader($headerKey, $headerValue);
                }
            }
        }
        return $newResponse;
    }
}