Wandu\Http\HttpServiceProvider

Complexity

Total Complexity

5

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	5

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	2

2 Methods

Rating	Name	Duplication	Size	Complexity
B	register()	0	23	4
A	boot()	0	3	1

<?php
namespace Wandu\Http;

use Predis\Client;
use SessionHandlerInterface;
use Wandu\DI\ContainerInterface;
use Wandu\DI\ServiceProviderInterface;
use Wandu\Http\Factory\ResponseFactory;
use Wandu\Http\Session\Handler\FileHandler;
use Wandu\Http\Session\Handler\GlobalHandler;
use Wandu\Http\Session\Handler\RedisHandler;
use Wandu\Http\Session\SessionFactory;
use function Wandu\Foundation\config;
use function Wandu\Foundation\path;

class HttpServiceProvider implements ServiceProviderInterface
{
    /**
     * {@inheritdoc}
     */
    public function register(ContainerInterface $app)
    {
        $app->closure(ResponseFactory::class, function () {
            return response(); // singleton
        });
        $app->closure(SessionFactory::class, function ($app) {
            return new SessionFactory($app[SessionHandlerInterface::class], [
                'timeout' => config('session.timeout', 3600),
                'name' => config('session.name', ini_get('session.name') ?: 'WdSessId'),
                'gc_frequency' => config('session.gc_frequency', 100),
            ]);
        });
        $app->closure(SessionHandlerInterface::class, function ($app) {
            switch (config('session.type')) {
                case 'file':
                    return new FileHandler(path(config('session.path', 'cache/sessions')));

It seems like \Wandu\Foundation\path(\Wandu\Foundation\config('session.path', 'cache/sessions')) targeting Wandu\Foundation\path() can also be of type array; however, Wandu\Http\Session\Handler\FileHandler::__construct() does only seem to accept string, maybe add an additional type check?

                case 'redis':
                    return new RedisHandler($app[Client::class], config('session.timeout', 3600));
                default:
                    return new GlobalHandler();
            }
        });
    }

    /**
     * {@inheritdoc}
     */
    public function boot(ContainerInterface $app)
    {
    }
}