Bdf\Prime\Cache\DoctrineCacheAdapter

Complexity

Total Complexity

10

Size/Duplication

Total Lines	86
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	delete()	0	3	1
A	__construct()	0	3	1
A	set()	0	3	1
A	get()	0	5	2
A	clear()	0	6	2
A	flush()	0	3	1
A	namespace()	0	10	2

<?php

namespace Bdf\Prime\Cache;

use Doctrine\Common\Cache\CacheProvider;

/**
 * Adapter for doctrine cache provider
 */
class DoctrineCacheAdapter implements CacheInterface
{
    /**
     * @var CacheProvider
     */
    private $driver;

Expected 1 blank line(s) before first member var; 0 found

    /**
     * @var CacheProvider[]
     */
    private $driverByNamespace = [];


    /**
     * DoctrineCacheAdapter constructor.
     *
     * @param CacheProvider $driver
     */
    public function __construct(CacheProvider $driver)

Expected 1 blank line before function; 2 found

    {
        $this->driver = $driver;
    }

    /**

Parameter $key should have a doc-comment as per coding-style.

     * {@inheritdoc}
     */
    public function get(CacheKey $key)
    {
        $data = $this->namespace($key->namespace())->fetch($key->key());

        return $data === false ? null : $data;

Inline IF statements are not allowed

Inline shorthand IF statement requires brackets around comparison

    }

    /**

Parameter $key should have a doc-comment as per coding-style.

Parameter $data should have a doc-comment as per coding-style.

     * {@inheritdoc}
     */
    public function set(CacheKey $key, $data)
    {
        $this->namespace($key->namespace())->save($key->key(), $data, $key->lifetime());
    }

    /**

Parameter $key should have a doc-comment as per coding-style.

     * {@inheritdoc}
     */
    public function delete(CacheKey $key)
    {
        $this->namespace($key->namespace())->delete($key->key());
    }

    /**

Parameter $namespace should have a doc-comment as per coding-style.

     * {@inheritdoc}
     */
    public function flush($namespace)
    {
        $this->namespace($namespace)->deleteAll();
    }

    /**
     * {@inheritdoc}
     */
    public function clear()
    {
        $this->driver->flushAll();

        foreach ($this->driverByNamespace as $driver) {
            $driver->flushAll();
        }
    }

    /**
     * Get the cache provider for the given namespace
     *
     * @param string $ns
     *
     * @return CacheProvider
     */
    private function namespace(string $ns): CacheProvider

Private method name "DoctrineCacheAdapter::namespace" must be prefixed with an underscore

    {
        if (isset($this->driverByNamespace[$ns])) {
            return $this->driverByNamespace[$ns];
        }

        $driver = clone $this->driver;
        $driver->setNamespace($ns);

        return $this->driverByNamespace[$ns] = $driver;

Assignments must be the first block of code on a line

    }
}