DoctrineCacheAdapter::get() - Code Metrics - Inspection of "Merge pull request #4 from b2pweb/doctrine-cache-a..." - b2pweb/bdf-prime - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 6692d3...4dd3a5 )

by Sébastien

created 2020-07-23 14:03 UTC

DoctrineCacheAdapter::get() A

↳ Parent: DoctrineCacheAdapter

Complexity

Conditions	2
Paths	2

Size

Total Lines	5
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	2
c	1
b	0
f	0
dl	0
loc	5
ccs	3
cts	3
cp	1
rs	10
cc	2
nc	2
nop	1
crap	2

<?php

namespace Bdf\Prime\Cache;

use Doctrine\Common\Cache\CacheProvider;

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


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


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

    {
        $this->driver = $driver;
    }

    /**

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

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

    }

    /**

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

    /**

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

    /**

     * {@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

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

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

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

    }
}


1		<?php
2
3		namespace Bdf\Prime\Cache;
4
5		use Doctrine\Common\Cache\CacheProvider;
6
7		/**
8		* Adapter for doctrine cache provider
9		*/
10		class DoctrineCacheAdapter implements CacheInterface
11		{
12		/**
13		* @var CacheProvider
14		*/
15		private $driver;
		0 ignored issues – show Coding Style introduced 2020-07-23 13:59 UTC by Report Bug Copy Issue Report Expected 1 blank line(s) before first member var; 0 found Loading history...
16
17		/**
18		* @var CacheProvider[]
19		*/
20		private $driverByNamespace = [];
21
22
23		/**
24		* DoctrineCacheAdapter constructor.
25		*
26		* @param CacheProvider $driver
27		*/
28	18	public function __construct(CacheProvider $driver)
		0 ignored issues – show Coding Style introduced 2020-07-23 13:59 UTC by Report Bug Copy Issue Report Expected 1 blank line before function; 2 found Loading history...
29		{
30	18	$this->driver = $driver;
31	18	}
32
33		/**
		0 ignored issues – show Coding Style introduced 2020-07-23 13:59 UTC by Report Bug Copy Issue Report Parameter `$key` should have a doc-comment as per coding-style. Loading history...
34		* {@inheritdoc}
35		*/
36	18	public function get(CacheKey $key)
37		{
38	18	$data = $this->namespace($key->namespace())->fetch($key->key());
39
40	18	return $data === false ? null : $data;
		0 ignored issues – show Coding Style introduced 2020-07-23 13:59 UTC by Report Bug Copy Issue Report Inline IF statements are not allowed Loading history... Coding Style introduced 2020-07-23 13:59 UTC by Report Bug Copy Issue Report Inline shorthand IF statement requires brackets around comparison Loading history...
41		}
42
43		/**
		0 ignored issues – show Coding Style introduced 2020-07-23 13:59 UTC by Report Bug Copy Issue Report Parameter `$key` should have a doc-comment as per coding-style. Loading history... Coding Style introduced 2020-07-23 13:59 UTC by Report Bug Copy Issue Report Parameter `$data` should have a doc-comment as per coding-style. Loading history...
44		* {@inheritdoc}
45		*/
46	17	public function set(CacheKey $key, $data)
47		{
48	17	$this->namespace($key->namespace())->save($key->key(), $data, $key->lifetime());
49	17	}
50
51		/**
		0 ignored issues – show Coding Style introduced 2020-07-23 13:59 UTC by Report Bug Copy Issue Report Parameter `$key` should have a doc-comment as per coding-style. Loading history...
52		* {@inheritdoc}
53		*/
54	1	public function delete(CacheKey $key)
55		{
56	1	$this->namespace($key->namespace())->delete($key->key());
57	1	}
58
59		/**
		0 ignored issues – show Coding Style introduced 2020-07-23 13:59 UTC by Report Bug Copy Issue Report Parameter `$namespace` should have a doc-comment as per coding-style. Loading history...
60		* {@inheritdoc}
61		*/
62	8	public function flush($namespace)
63		{
64	8	$this->namespace($namespace)->deleteAll();
65	8	}
66
67		/**
68		* {@inheritdoc}
69		*/
70	1	public function clear()
71		{
72	1	$this->driver->flushAll();
73
74	1	foreach ($this->driverByNamespace as $driver) {
75	1	$driver->flushAll();
76		}
77	1	}
78
79		/**
80		* Get the cache provider for the given namespace
81		*
82		* @param string $ns
83		*
84		* @return CacheProvider
85		*/
86	18	private function namespace(string $ns): CacheProvider
		0 ignored issues – show Coding Style introduced 2020-07-23 13:59 UTC by Report Bug Copy Issue Report Private method name "DoctrineCacheAdapter::namespace" must be prefixed with an underscore Loading history...
87		{
88	18	if (isset($this->driverByNamespace[$ns])) {
89	17	return $this->driverByNamespace[$ns];
90		}
91
92	18	$driver = clone $this->driver;
93	18	$driver->setNamespace($ns);
94
95	18	return $this->driverByNamespace[$ns] = $driver;
		0 ignored issues – show Coding Style introduced 2020-07-23 13:59 UTC by Report Bug Copy Issue Report Assignments must be the first block of code on a line Loading history...
96		}
97		}
98

b2pweb / bdf-prime

Push — master ( 6692d3...4dd3a5 )

DoctrineCacheAdapter::get() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like