ICanBoogie\Routing\RouteDispatcher

Complexity

Total Complexity

21

Size/Duplication

Total Lines	224
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	10

Importance

Changes

0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	get_routes()	0	4	1
A	__construct()	0	4	1
A	__invoke()	0	23	3
A	normalize_path()	0	11	2
A	resolve_route()	0	4	1
A	alter_params()	0	5	1
A	alter_context()	0	5	1
A	dispatch()	0	15	2
A	respond()	0	20	3
A	rescue()	0	16	3
A	resolve_controller()	0	20	3

<?php

/*
 * This file is part of the ICanBoogie package.
 *
 * (c) Olivier Laviale <olivier.laviale@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace ICanBoogie\Routing;

use ICanBoogie\Accessor\AccessorTrait;
use ICanBoogie\HTTP\Dispatcher;
use ICanBoogie\HTTP\RedirectResponse;
use ICanBoogie\HTTP\Request;
use ICanBoogie\HTTP\Response;
use ICanBoogie\HTTP\Status;
use ICanBoogie\Routing\RouteDispatcher\BeforeDispatchEvent;
use ICanBoogie\Routing\RouteDispatcher\DispatchEvent;
use ICanBoogie\Routing\Route\RescueEvent;

/**
 * Dispatch requests among the defined routes.
 *
 * If a route matching the request is found, the `$route` and `$decontextualized_path`
 * properties are added to the {@link Request} instance. `$route` holds the {@link Route} instance,
 * `$decontextualized_path` holds the decontextualized path. The path is decontextualized using
 * the {@link decontextualize()} function.
 *
 * @property-read RouteCollection $routes
 */
class RouteDispatcher implements Dispatcher
{
	use AccessorTrait;

	/**
	 * Route collection.
	 *
	 * @var RouteCollection
	 */
	protected $routes;

	protected function get_routes()
	{
		return $this->routes;
	}

	/**
	 * @param RouteCollection|null $routes
	 */
	public function __construct(RouteCollection $routes = null)
	{
		$this->routes = $routes;
	}

	/**
	 * @param Request $request
	 *
	 * @return Response|null
	 */
	public function __invoke(Request $request)
	{
		$captured = [];
		$normalized_path = $this->normalize_path($request->normalized_path);
		$route = $this->resolve_route($request, $normalized_path, $captured);

		if (!$route)
		{
			return null;
		}

		if ($route->location)
		{
			return new RedirectResponse(contextualize($route->location), Status::FOUND);
		}

		$this->alter_params($route, $request, $captured);

		$request->context->route = $route;
		$request->decontextualized_path = $normalized_path;

The property decontextualized_path does not seem to exist. Did you mean context?

		return $this->dispatch($route, $request);
	}

	/**
	 * Normalizes request path.
	 *
	 * @param string $path
	 *
	 * @return string Decontextualized path with trimmed ending slash.
	 */
	protected function normalize_path($path)
	{
		$normalized_path = decontextualize($path);

		if ($normalized_path != '/')
		{
			$normalized_path = rtrim($normalized_path, '/');
		}

		return $normalized_path;
	}

	/**
	 * Resolves route from request.
	 *
	 * @param Request $request
	 * @param string $normalized_path
	 * @param array $captured
	 *
	 * @return false|Route|null
	 */
	protected function resolve_route(Request $request, $normalized_path, array &$captured)
	{
		return $this->routes->find($normalized_path, $captured, $request->method);
	}

	/**
	 * Alters request parameters.
	 *
	 * @param Route $route
	 * @param Request $request
	 * @param array $captured Parameters captured from the request's path.
	 */
	protected function alter_params(Route $route, Request $request, array $captured)

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

	{
		$request->path_params = $captured + $request->path_params;
		$request->params = $captured + $request->params;
	}

	/**
	 * Alters request context with route and controller.
	 *
	 * @param Request\Context $context
	 * @param Route $route
	 * @param callable $controller
	 */
	protected function alter_context(Request\Context $context, Route $route, callable $controller)
	{
		$context->route = $route;
		$context->controller = $controller;
	}

	/**
	 * Dispatches the route.
	 *
	 * @param Route $route
	 * @param Request $request
	 *
	 * @return Response|null
	 */
	protected function dispatch(Route $route, Request $request)
	{
		$response = null;

		new BeforeDispatchEvent($this, $route, $request, $response);

		if (!$response)
		{
			$response = $this->respond($route, $request);
		}

		new DispatchEvent($this, $route, $request, $response);

		return $response;
	}

	/**
	 * Returns a response for the route and request.
	 *
	 * If the controller's result is not `null` but is not in instance of {@link Response}, its
	 * result is wrapped in a {@link response} instance with the status code 200 and the
	 * `Content-Type` "text/html; charset=utf-8".
	 *
	 * @param Route $route
	 * @param Request $request
	 *
	 * @return Response|mixed
	 */
	protected function respond(Route $route, Request $request)
	{
		$controller = $this->resolve_controller($route->controller);
		$controller_args = [ $request ];

		$this->alter_context($request->context, $route, $controller);

		$response = $controller(...$controller_args);

		if ($response !== null && !$response instanceof Response)
		{
			$response = new Response($response, Status::OK, [

				'Content-Type' => 'text/html; charset=utf-8'

			]);
		}

		return $response;
	}

	/**
	 * Fires {@link \ICanBoogie\Routing\RouteDispatcher\RescueEvent} and returns the response provided
	 * by third parties. If no response was provided, the exception (or the exception provided by
	 * third parties) is re-thrown.
	 *
	 * @param \Exception $exception The exception to rescue.
	 * @param Request $request The request being dispatched.
	 *
	 * @throws \Exception if the exception cannot be rescued.
	 *
	 * @return Response
	 */
	public function rescue(\Exception $exception, Request $request)
	{
		if (isset($request->context->route))
		{
			$response = null;

			new RescueEvent($request->context->route, $exception, $request, $response);

			if ($response)
			{
				return $response;
			}
		}

		throw $exception;
	}

	/**
	 * @param callable|string $controller
	 *
	 * @return Controller
	 */
	protected function resolve_controller($controller)
	{
		if ($controller instanceof \Closure)
		{
			return new ClosureController($controller);
		}

		if (is_callable($controller))
		{
			return new ClosureController(function () use ($controller) {

				/* @var $this ClosureController */

				return $controller($this->request);

The property request does not exist on object<ICanBoogie\Routing\RouteDispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

			});
		}

		return new $controller;
	}
}