Equip\Handler\DispatchHandler

Complexity

Total Complexity

8

Size/Duplication

Total Lines	82
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	__invoke()	0	22	2
A	dispatcher()	0	10	2
A	dispatch()	0	16	3

<?php

namespace Equip\Handler;

use FastRoute;
use FastRoute\Dispatcher;
use FastRoute\RouteCollector;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\ServerRequestInterface;
use Equip\Directory;
use Equip\Exception\HttpException;
use Equip\Handler\ActionHandler;

class DispatchHandler
{
    /**
     * @var Directory
     */
    private $directory;

    /**
     * @param Directory $directory
     */
    public function __construct(Directory $directory)
    {
        $this->directory = $directory;
    }

    /**
     * @inheritDoc
     */
    public function __invoke(
        ServerRequestInterface $request,
        ResponseInterface $response,
        callable $next
    ) {
        /**
         * @var $action Arbiter\Action
         */
        list($action, $args) = $this->dispatch(
            $this->dispatcher(),
            $request->getMethod(),
            $request->getUri()->getPath()
        );

        $request = $request->withAttribute(ActionHandler::ACTION_ATTRIBUTE, $action);

        foreach ($args as $key => $value) {
            $request = $request->withAttribute($key, $value);
        }

        return $next($request, $response);
    }

    /**
     * @return Dispatcher
     */
    protected function dispatcher()
    {
        return FastRoute\simpleDispatcher(function (RouteCollector $collector) {
            foreach ($this->directory as $request => $action) {
                // 'GET /foo' becomes ['GET', '/foo']

55% of this comment could be valid code. Did you maybe forget this after debugging?

                list($method, $path) = explode(' ', $request, 2);
                $collector->addRoute($method, $path, $action);
            }
        });
    }

    /**
     * @param Dispatcher $dispatcher
     * @param string $method
     * @param string $path
     *
     * @return [Action, $arguments]

The doc-type Action,">[Action, could not be parsed: Unknown type name "[" at position 0. [(view supported doc-types)

     *
     * @throws HttpNotFound
     * @throws HttpMethodNotAllowed
     */
    private function dispatch(Dispatcher $dispatcher, $method, $path)
    {
        $route = $dispatcher->dispatch($method, $path);

Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

        $status = array_shift($route);

        if (Dispatcher::FOUND === $status) {
            return $route;
        }

        if (Dispatcher::METHOD_NOT_ALLOWED === $status) {
            $allowed = array_shift($route);
            throw HttpException::methodNotAllowed($path, $method, $allowed);
        }

        throw HttpException::notFound($path);
    }
}