SwooleTW\Http\Server\Resetters\RebindRouterContainer::handle()

Complexity

Conditions	6
Paths	3

Size

Total Lines	33
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function handle(Container $app, Sandbox $sandbox)
    {
        if ($sandbox->isLaravel()) {
            $router = $app->make('router');
            $request = $sandbox->getRequest();
            $closure = function () use ($app, $request) {
                $this->container = $app;

The property container does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

                if (is_null($request)) {
                    return;
                }
                try {
                    $route = $this->routes->match($request);

The property routes does not exist on SwooleTW\Http\Server\Resetters\RebindRouterContainer. Did you maybe forget to declare it?

                    // clear resolved controller
                    if (property_exists($route, 'container')) {
                        $route->controller = null;
                    }
                    // rebind matched route's container
                    $route->setContainer($app);
                } catch (NotFoundHttpException $e) {
                    // do nothing
                }
            };

            $resetRouter = $closure->bindTo($router, $router);
            $resetRouter();
        } else {
            // lumen router only exists after lumen 5.5
            if (property_exists($app, 'router')) {
                $app->router->app = $app;
            }
        }

        return $app;
    }