HasCacheStoreTrait::createCacheStore() - Code Metrics - Inspection of "Merge branch '0.9' into develop" - bytic/filesystem - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( dee4cd...d18a68 )

by Gabriel

created 2020-04-19 17:03 UTC

HasCacheStoreTrait::createCacheStore() A

↳ Parent: HasCacheStoreTrait

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	12
ccs	2
cts	2
cp	1
rs	9.8666
c	0
b	0
f	0
cc	1
nc	1
nop	1
crap	1

<?php

namespace Nip\Filesystem\FilesystemManager;

use League\Flysystem\Cached\CachedAdapter;
use League\Flysystem\Cached\Storage\Memory as MemoryStore;
use Nip\Utility\Arr;

/**
 * Trait HasCacheStoreTrait
 * @package Nip\Filesystem\FilesystemManager
 */
trait HasCacheStoreTrait
{
    /**
     * @param $adapter
     * @param $config
     */
    protected function checkForCacheNeeded(&$adapter, &$config)
    {
        $cache = Arr::pull($config, 'cache');
        if (!empty($cache)) {
            $adapter = $this->createCacheAdapter($adapter, $cache);
        }
    }

    /**
     * @param $adapter
     * @param $cache
     * @return CachedAdapter
     */
    protected function createCacheAdapter($adapter, $cache)
    {
        return new CachedAdapter($adapter, $this->createCacheStore($cache));
    }


    /**
     * Create a cache store instance.
     *
     * @param mixed $config
     * @return \League\Flysystem\Cached\CacheInterface|MemoryStore
     *
     * @throws \InvalidArgumentException
     */
    protected function createCacheStore($config)

    {
//        if ($config === true) {
        return new MemoryStore;
//        }
//
//        return new Cache(
//            $this->app['cache']->store($config['store']),
//            $config['prefix'] ?? 'flysystem',
//            $config['expire'] ?? null
//        );
    }
}


1		<?php
2
3		namespace Nip\Filesystem\FilesystemManager;
4
5		use League\Flysystem\Cached\CachedAdapter;
6		use League\Flysystem\Cached\Storage\Memory as MemoryStore;
7		use Nip\Utility\Arr;
8
9		/**
10		* Trait HasCacheStoreTrait
11		* @package Nip\Filesystem\FilesystemManager
12		*/
13		trait HasCacheStoreTrait
14		{
15		/**
16		* @param $adapter
17		* @param $config
18		*/
19	2	protected function checkForCacheNeeded(&$adapter, &$config)
20		{
21	2	$cache = Arr::pull($config, 'cache');
22	2	if (!empty($cache)) {
23	1	$adapter = $this->createCacheAdapter($adapter, $cache);
24		}
25	2	}
26
27		/**
28		* @param $adapter
29		* @param $cache
30		* @return CachedAdapter
31		*/
32	1	protected function createCacheAdapter($adapter, $cache)
33		{
34	1	return new CachedAdapter($adapter, $this->createCacheStore($cache));
35		}
36
37
38		/**
39		* Create a cache store instance.
40		*
41		* @param mixed $config
42		* @return \League\Flysystem\Cached\CacheInterface\|MemoryStore
43		*
44		* @throws \InvalidArgumentException
45		*/
46	1	protected function createCacheStore($config)
		0 ignored issues – show Unused Code introduced 2020-04-19 17:09 UTC by Report Bug Copy Issue Report The parameter `$config` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
47		{
48		// if ($config === true) {
49	1	return new MemoryStore;
50		// }
51		//
52		// return new Cache(
53		// $this->app['cache']->store($config['store']),
54		// $config['prefix'] ?? 'flysystem',
55		// $config['expire'] ?? null
56		// );
57		}
58		}
59

bytic / filesystem

Push — master ( dee4cd...d18a68 )

HasCacheStoreTrait::createCacheStore() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like