Rade\Handler\RouteHandler::__construct()

Complexity

Conditions	20
Paths	1

Size

Total Lines	54
Code Lines	32

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	20
CRAP Score	20.0431

Importance

Changes	3
Bugs	0	Features	0

<?php
    public function __construct(Container $container)
    {
        $handlerResolver = static function ($handler, array $parameters) use ($container) {
            $request = $parameters[ServerRequestInterface::class] ?? null;

            if (\is_string($handler)) {
                if ($container->has($handler)) {
                    $handler = $container->get($handler);
                } elseif ($container->typed($handler)) {
                    $handler = $container->autowired($handler);
                } elseif (\class_exists($handler)) {
                    $handler = $container->getResolver()->resolveClass($handler);
                }
            } elseif (\is_array($handler) && \count($handler) === 2) {
                $handler[0] = \is_string($handler[0]) ? $container->get($handler[0]) : $handler[0];
            }

            $ref = !$handler instanceof RequestHandlerInterface ? Callback::toReflection($handler) : new \ReflectionMethod($handler, 'handle');

            if ($ref->getNumberOfParameters() > 0) {
                foreach (\class_implements($request) + (\class_parents($request) ?: []) as $psr7Interface) {
                    if (\Stringable::class === $psr7Interface) {
                        continue;
                    }
                    $parameters[$psr7Interface] = $request;
                }
                $parameters[\get_class($request)] = $request;

It seems like $request can also be of type null; however, parameter $object of get_class() does only seem to accept object, maybe add an additional type check?

                $args = $container->getResolver()->autowireArguments($ref, $parameters);
            }

            if ($container->has('events.dispatcher')) {
                $container->get('events.dispatcher')->dispatch($event = new RequestEvent($container, $request));

It seems like $request can also be of type null; however, parameter $request of Rade\Event\RequestEvent::__construct() does only seem to accept Psr\Http\Message\ServerRequestInterface, maybe add an additional type check?

                if ($event->hasResponse()) {
                    return $event->getResponse();
                }

                $request = $event->getRequest();
                $container->get('events.dispatcher')->dispatch($event = new ControllerEvent($container, $handler, $ref, $args ?? [], $request));
                [$handler, $ref, $args] = [$event->getController(), $event->getReflection(), $event->getArguments()];
            }

            if ($request instanceof Request && $container->has('request_stack')) {
                $container->get('request_stack')->push($request->getRequest());
            }

            if ($ref instanceof \ReflectionMethod) {
                return $ref->isStatic() ? $ref->invokeArgs(null, $args ?? []) : $ref->invokeArgs(!\is_object($handler) ? $handler[0] : $handler, $args ?? []);
            }

            return $ref->invokeArgs($args ?? []);
        };

        parent::__construct($container->get('psr17.factory'), $handlerResolver);

It seems like $container->get('psr17.factory') can also be of type array and mixed; however, parameter $responseFactory of Flight\Routing\Handlers\RouteHandler::__construct() does only seem to accept Psr\Http\Message\ResponseFactoryInterface, maybe add an additional type check?

    }