Quantum\Libraries\Cache\CacheManager

Complexity

Total Complexity

3

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getHandler()	0	17	3

<?php

/**
 * Quantum PHP Framework
 *
 * An open source software development framework for PHP
 *
 * @package Quantum
 * @author Arman Ag. <arman.ag@softberg.org>
 * @copyright Copyright (c) 2018 Softberg LLC (https://softberg.org)
 * @link http://quantum.softberg.org/
 * @since 2.8.0
 */

namespace Quantum\Libraries\Cache;

use Psr\SimpleCache\CacheInterface;
use Quantum\Loader\Setup;

/**
 * Class AuthManager
 * @package Quantum\Libraries\Cache
 */
class CacheManager
{

    const DRIVERS = [
        'file',
        'memcache',
        'redis'
    ];

    /**
     *  Get Handler
     * @return CacheInterface
     * @throws Exception
     */
    public static function getHandler()
    {
        if (!config()->has('cache')) {
            config()->import(new Setup('Config', 'cache'));
        }

        $cacheDriver = config()->get('cache.current');

        if (!in_array($cacheDriver, self::DRIVERS)) {
            throw new \Exception();
        }

        $cacheAdapterClass = __NAMESPACE__ . '\\Adapters\\' . ucfirst($cacheDriver) . 'Cache';

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

        $cacheAdapter = new $cacheAdapterClass(config()->get('cache.' . $cacheDriver . '.params'));
        
        return new Cache($cacheAdapter);

The expression return new Quantum\Libraries\Cache\Cache($cacheAdapter) returns the type Quantum\Libraries\Cache\Cache which is incompatible with the documented return type Psr\SimpleCache\CacheInterface.

    }

}