Msgpack::get() - Code Metrics - Inspection of "react/cache 0.5" - WyriHaximus/reactphp-cache-msgpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 08982d...00de86 )

by Cees-Jan

created 2018-09-03 20:15 UTC

Msgpack::get() A

↳ Parent: Msgpack

Complexity

Conditions	3
Paths	1

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	3

Importance

Changes

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

<?php declare(strict_types=1);

namespace WyriHaximus\React\Cache;

use React\Cache\CacheInterface;
use React\Promise\PromiseInterface;

final class Msgpack implements CacheInterface
{
    /**
     * @var CacheInterface
     */
    private $cache;

    /**
     * @param CacheInterface $cache
     */
    public function __construct(CacheInterface $cache)
    {
        $this->cache = $cache;
    }

    /**
     * @param $key
     * @param  null             $default

     * @return PromiseInterface
     */
    public function get($key, $default = null)
    {
        return $this->cache->get($key, $default)->then(function ($result) use ($default) {
            if ($result === null || $result === $default) {
                return $result;
            }

            return msgpack_unpack($result);
        });
    }

    /**
     * @param  string           $key
     * @param  mixed            $value
     * @param  null             $ttl

     * @return PromiseInterface
     */
    public function set($key, $value, $ttl = null)
    {
        return $this->cache->set($key, msgpack_pack($value), $ttl);
    }

    /**
     * @param  string           $key
     * @return PromiseInterface
     */
    public function delete($key)
    {
        return $this->cache->delete($key);
    }
}


1		<?php declare(strict_types=1);
2
3		namespace WyriHaximus\React\Cache;
4
5		use React\Cache\CacheInterface;
6		use React\Promise\PromiseInterface;
7
8		final class Msgpack implements CacheInterface
9		{
10		/**
11		* @var CacheInterface
12		*/
13		private $cache;
14
15		/**
16		* @param CacheInterface $cache
17		*/
18	4	public function __construct(CacheInterface $cache)
19		{
20	4	$this->cache = $cache;
21	4	}
22
23		/**
24		* @param $key
25		* @param null $default
		0 ignored issues – show Documentation Bug introduced 2018-09-03 20:19 UTC by Report Bug Copy Issue Report Are you sure the doc-type for parameter `$default` is correct as it would always require `null` to be passed? Loading history...
26		* @return PromiseInterface
27		*/
28		public function get($key, $default = null)
29		{
30	2	return $this->cache->get($key, $default)->then(function ($result) use ($default) {
31	2	if ($result === null \|\| $result === $default) {
32	1	return $result;
33		}
34
35	1	return msgpack_unpack($result);
36	2	});
37		}
38
39		/**
40		* @param string $key
41		* @param mixed $value
42		* @param null $ttl
		0 ignored issues – show Documentation Bug introduced 2018-09-03 20:19 UTC by Report Bug Copy Issue Report Are you sure the doc-type for parameter `$ttl` is correct as it would always require `null` to be passed? Loading history...
43		* @return PromiseInterface
44		*/
45	1	public function set($key, $value, $ttl = null)
46		{
47	1	return $this->cache->set($key, msgpack_pack($value), $ttl);
48		}
49
50		/**
51		* @param string $key
52		* @return PromiseInterface
53		*/
54	1	public function delete($key)
55		{
56	1	return $this->cache->delete($key);
57		}
58		}
59

WyriHaximus / reactphp-cache-msgpack

GitHub Access Token became invalid

Push — master ( 08982d...00de86 )

Msgpack::get() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like