Ikechukwukalu\Dynamicdatabaseconfig\Middleware\EnvDatabaseConfig::handle()

Complexity

Conditions	2
Paths	2

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

<?php
    public function handle(Request $request, Closure $next, string $database, string $name, null|string $postFix = null)
    {
        $request->merge([config('dynamicdatabaseconfig.connection_name', '_db_connection') => $name]);

        if (!$postFix) {
            $postFix = session(config('dynamicdatabaseconfig.session_postfix'));
        }

        $newConfig = $this->setNewEnvConfig($database, $postFix);

It seems like $postFix can also be of type Illuminate\Session\SessionManager and Illuminate\Session\Store; however, parameter $postFix of Ikechukwukalu\Dynamicdatabaseconfig\Middleware\EnvDatabaseConfig::setNewEnvConfig() does only seem to accept string, maybe add an additional type check?

        $this->addNewConfig($database, $name, $newConfig);
        $this->createDatabase($database, $newConfig['database']);

        return $next($request);
    }