Pcelta\Doctrine\Cache\Factory\RedisFactory

Complexity

Total Complexity

6

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	1	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getModuleName()	0	4	1
A	getRedisAdapter()	0	4	1
A	decorateWithConnectable()	0	10	1
A	isValidConfig()	0	13	3

<?php

namespace Pcelta\Doctrine\Cache\Factory;

use Doctrine\Common\Cache\CacheProvider;
use Pcelta\Doctrine\Cache\Entity\Config;

class RedisFactory extends AbstractFactory
{
    const MODULE_NAME = 'redis';

    /**
     * @return string
     */
    public function getModuleName()
    {
        return self::MODULE_NAME;
    }

    /**
     * @return \Redis
     */
    protected function getRedisAdapter()
    {
        return new \Redis();
    }

    /**
     * @param CacheProvider $cacheProvider
     *
     * @return CacheProvider
     */
    protected function decorateWithConnectable(CacheProvider $cacheProvider)
    {
        $redis = $this->getRedisAdapter();
        $settings = $this->config->getSettings();
        $redis->connect($settings['host'], $settings['port']);

        $cacheProvider->setRedis($redis);

It seems like you code against a specific sub-type and not the parent class Doctrine\Common\Cache\CacheProvider as the method setRedis() does only exist in the following sub-classes of Doctrine\Common\Cache\CacheProvider: Doctrine\Common\Cache\RedisCache. Maybe you want to instanceof check for one of these explicitly?

        return $cacheProvider;
    }

    /**
     * @param Config $config
     *
     * @return bool
     */
    protected function isValidConfig(Config $config)
    {
        $settings = $config->getSettings();
        if (!isset($settings['host'])) {
            return false;
        }

        if (!isset($settings['port'])) {
            return false;
        }

        return true;
    }
}