SwooleTW\Http\LumenServiceProvider

Complexity

Total Complexity

4

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes	3
Bugs	2	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	registerManager()	0	7	1
A	loadConfigs()	0	4	1
A	pushAccessLogMiddleware()	0	3	1
A	bootWebsocketRoutes()	0	5	1

<?php

namespace SwooleTW\Http;

use SwooleTW\Http\Server\Manager;
use SwooleTW\Http\Middleware\AccessLog;

/**
 * @codeCoverageIgnore
 */
class LumenServiceProvider extends HttpServiceProvider
{
    /**
     * Load configurations.
     */
    protected function loadConfigs()
    {
        $this->app->configure('swoole_http');
        $this->app->configure('swoole_websocket');
    }

    /**
     * Register manager.
     *
     * @return void
     */
    protected function registerManager()
    {
        $this->app->singleton(Manager::class, function ($app) {
            return new Manager($app, 'lumen');
        });

        $this->app->alias(Manager::class, 'swoole.manager');
    }

    /**
     * Boot websocket routes.
     *
     * @return void
     */
    protected function bootWebsocketRoutes()
    {
        $this->app->router

Accessing router on the interface Illuminate\Contracts\Foundation\Application suggest that you code against a concrete implementation. How about adding an instanceof check?

            ->group(['namespace' => 'SwooleTW\Http\Controllers'], function ($router) {
                require __DIR__ . '/../routes/lumen_routes.php';
            });
    }

    /**
     * Register access log middleware to container.
     *
     * @return void
     */
    protected function pushAccessLogMiddleware()
    {
        $this->app->middleware(AccessLog::class);

SwooleTW\Http\Middleware\AccessLog::class of type string is incompatible with the type Closure|array expected by parameter $middleware of Laravel\Lumen\Application::middleware().

    }
}