League\Route\Strategy\RequestResponseJsonStrategy

Complexity

Total Complexity

6

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	dispatch()	0	32	6

<?php

namespace League\Route\Strategy;

use ArrayObject;
use League\Route\Http\Exception as HttpException;
use League\Route\Route;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\ServerRequestInterface;
use RuntimeException;

class RequestResponseJsonStrategy extends AbstractStrategy implements StrategyInterface
{
    /**
     * {@inheritdoc}
     */
    public function dispatch(callable $controller, array $vars, Route $route = null)
    {
        $middleware = function (
            ServerRequestInterface $request, ResponseInterface $response, callable $next
        ) use (
            $controller, $vars
        ) {
            try {
                $result = call_user_func_array($controller, [$request, $response, $vars]);

                if (is_array($result) || $result instanceof ArrayObject) {
                    $body     = json_encode($result);
                    $response = $this->getResponse();

                    if ($response->getBody()->isWritable()) {
                        $response->getBody()->write($body);
                    }
                }

                if ($response instanceof ResponseInterface) {
                    $response = $response->withAddedHeader('content-type', 'application/json');
                    return $next($request, $response);
                }
            } catch (HttpException $e) {
                return $e->buildJsonResponse($this->getResponse());
            }

            throw new RuntimeException('Unable to build a json response from controller return value.');
        };

        return $route->getMiddlewareRunner()->run($middleware, $this->getRequest(), $this->getResponse());

It seems like $route is not always an object, but can also be of type null. Maybe add an additional type check?

    }
}