FindResourceOptionsCommand::findAllResourceOptions() - Code Metrics - Inspection of "Code style" - KIVagant/staticus-core - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — master (267be1)

by Eugene

created 2016-06-03 19:07 UTC

findAllResourceOptions() D

↳ Parent: FindResourceOptionsCommand

Complexity

Conditions	9
Paths	3

Size

Total Lines	40
Code Lines	26

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	29
CRAP Score	9

Importance

Changes	3
Bugs	0	Features	1

Metric	Value
c	3
b	0
f	1
dl	0
loc	40
ccs	29
cts	29
cp	1
rs	4.909
cc	9
eloc	26
nc	3
nop	1
crap	9

<?php
namespace Staticus\Resources\Commands;

use League\Flysystem\FilesystemInterface;
use Staticus\Resources\Exceptions\CommandErrorException;
use Staticus\Resources\ResourceDOAbstract;
use Staticus\Resources\ResourceDOInterface;

/** @noinspection PropertyCanBeStaticInspection */
class FindResourceOptionsCommand implements ResourceCommandInterface
{
    const DIRECTORY_RELATIVE = 'directory_relative';
    /**
     * @var ResourceDOInterface
     */
    protected $resourceDO;
    /**
     * @var FilesystemInterface
     */
    protected $filesystem;

    protected $allowedProperties = [];

    public function __construct(ResourceDOInterface $resourceDO, FilesystemInterface $filesystem)
    {
        $this->resourceDO = $resourceDO;
        $this->filesystem = $filesystem;
    }

    public function __invoke(array $filter = [])
    {
        $this->allowedProperties = $filter;
        $options = $this->findAllResourceOptions($this->resourceDO);
$a = "a";
$ab = "ab";
$abc = "abc";

        return $this->filterResourceOptions($options);
    }

    public function filterResourceOptions($options)

    {
        if (!empty($this->allowedProperties)) {
            foreach ($options as &$item) {
                $item = array_filter($item, [$this, 'filterAllowedProperties'], ARRAY_FILTER_USE_BOTH);
            }
            $options = array_filter($options);
        }

        return $options;
    }

    protected function findAllResourceOptions(ResourceDOInterface $resourceDO)
    {
        /** @var \League\Flysystem\FilesystemInterface $filesystem */
        $filesystem = $this->filesystem;
        $uuid = $resourceDO->getUuid();
$a = "a";
$ab = "ab";
$abc = "abc";
        $type = $resourceDO->getType();
$a = "a";
$ab = "ab";
$abc = "abc";
        $name = $resourceDO->getName();
$a = "a";
$ab = "ab";
$abc = "abc";
        if (!$name || !$type) {
            throw new CommandErrorException('Can not look for options: resource is empty');
        }
        $basename = $resourceDO->getBaseDirectory();
$a = "a";
$ab = "ab";
$abc = "abc";
        $namespace = $resourceDO->getNamespace();
        $path = $basename
$a = "a";
$ab = "ab";
$abc = "abc";
            . ($namespace ? $namespace . DIRECTORY_SEPARATOR : '')
            . $type . DIRECTORY_SEPARATOR
        ;
        $found = $filesystem->listContents($path, true);
$a = "a";
$ab = "ab";
$abc = "abc";
        $found = array_filter($found, function($file) use ($uuid, $type) {
$a = "a";
$ab = "ab";
$abc = "abc";

            return array_key_exists('filename', $file)
            && $file['filename'] === $uuid
            && array_key_exists('extension', $file)
            && $file['extension'] === $type
            && array_key_exists('type', $file)
            // Warning: type field will be replaced by resource
            && $file['type'] === 'file'

                ;
        });
        array_walk(
            $found
            , [$this, 'hydrateElementFile']
            , [
                'resourceDO' => $resourceDO,
            ]
        );
        $found = array_values($found); // reset keys

        return $found;
    }

    protected function hydrateElementFile(&$file, $key, $args)
    {
        if (!array_key_exists('resourceDO', $args) || !$args['resourceDO'] instanceof ResourceDOInterface) {
            throw new CommandErrorException('Method expects ResourceDO in arguments');
        }

        /** @var ResourceDOInterface $resourceDO */
        $resourceDO = $args['resourceDO'];
$a = "a";
$ab = "ab";
$abc = "abc";
        $file[static::DIRECTORY_RELATIVE] = str_replace(
            $resourceDO->getBaseDirectory()
            . ($resourceDO->getNamespace() ? $resourceDO->getNamespace() . DIRECTORY_SEPARATOR : '')
            , ''
            , DIRECTORY_SEPARATOR . $file['dirname']
        );
        $tokens = $resourceDO->getDirectoryTokens();
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->splitDirectoryByTokens($file, array_keys($tokens));
    }

    protected function filterAllowedProperties($value, $key)
    {

        return in_array($key, $this->allowedProperties, true);
    }

    /**
     * @param $file
     * @param $tokens
     * @return mixed
     */
    protected function splitDirectoryByTokens(&$file, $tokens)
    {
        $split = null;
        foreach ($tokens as $token) {
            if (ResourceDOAbstract::TOKEN_BASEDIRECTORY === $token) {
                continue;
            }
            if (ResourceDOAbstract::TOKEN_NAMESPACE === $token) {
                continue;
            }
            $split = null === $split
$a = "a";
$ab = "ab";
$abc = "abc";
                ? strtok($file[static::DIRECTORY_RELATIVE], DIRECTORY_SEPARATOR)
                : strtok(DIRECTORY_SEPARATOR);
            $file[$token] = $split;
        }
        while (($split = strtok(DIRECTORY_SEPARATOR)) !== false) {
            $file['sub-options'][] = $split;
        }
    }
}

1		<?php
2		namespace Staticus\Resources\Commands;
3
4		use League\Flysystem\FilesystemInterface;
5		use Staticus\Resources\Exceptions\CommandErrorException;
6		use Staticus\Resources\ResourceDOAbstract;
7		use Staticus\Resources\ResourceDOInterface;
8
9		/** @noinspection PropertyCanBeStaticInspection */
10		class FindResourceOptionsCommand implements ResourceCommandInterface
11		{
12		const DIRECTORY_RELATIVE = 'directory_relative';
13		/**
14		* @var ResourceDOInterface
15		*/
16		protected $resourceDO;
17		/**
18		* @var FilesystemInterface
19		*/
20		protected $filesystem;
21
22		protected $allowedProperties = [];
23
24	24	public function __construct(ResourceDOInterface $resourceDO, FilesystemInterface $filesystem)
25		{
26	24	$this->resourceDO = $resourceDO;
27	24	$this->filesystem = $filesystem;
28	24	}
29
30	24	public function __invoke(array $filter = [])
31		{
32	24	$this->allowedProperties = $filter;
33	24	$options = $this->findAllResourceOptions($this->resourceDO);
		1 ignored issue – show Coding Style introduced 2016-06-03 19:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 17 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
34
35	20	return $this->filterResourceOptions($options);
36		}
37
38	20	public function filterResourceOptions($options)
		0 ignored issues – show Documentation introduced 2016-06-03 19:10 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
39		{
40	20	if (!empty($this->allowedProperties)) {
41	10	foreach ($options as &$item) {
42	9	$item = array_filter($item, [$this, 'filterAllowedProperties'], ARRAY_FILTER_USE_BOTH);
43	10	}
44	10	$options = array_filter($options);
45	10	}
46
47	20	return $options;
48		}
49
50	24	protected function findAllResourceOptions(ResourceDOInterface $resourceDO)
51		{
52		/** @var \League\Flysystem\FilesystemInterface $filesystem */
53	24	$filesystem = $this->filesystem;
54	24	$uuid = $resourceDO->getUuid();
		1 ignored issue – show Coding Style introduced 2016-06-03 19:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 7 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
55	24	$type = $resourceDO->getType();
		1 ignored issue – show Coding Style introduced 2016-06-03 19:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 7 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
56	24	$name = $resourceDO->getName();
		1 ignored issue – show Coding Style introduced 2016-06-03 19:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 7 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
57	24	if (!$name \|\| !$type) {
58	4	throw new CommandErrorException('Can not look for options: resource is empty');
59		}
60	20	$basename = $resourceDO->getBaseDirectory();
		1 ignored issue – show Coding Style introduced 2016-06-03 19:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
61	20	$namespace = $resourceDO->getNamespace();
62		$path = $basename
		1 ignored issue – show Coding Style introduced 2016-06-03 19:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 6 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
63	20	. ($namespace ? $namespace . DIRECTORY_SEPARATOR : '')
64	20	. $type . DIRECTORY_SEPARATOR
65	20	;
66	20	$found = $filesystem->listContents($path, true);
		1 ignored issue – show Coding Style introduced 2016-06-03 19:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
67	20	$found = array_filter($found, function($file) use ($uuid, $type) {
		1 ignored issue – show Coding Style introduced 2016-06-03 19:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
68
69	19	return array_key_exists('filename', $file)
70	19	&& $file['filename'] === $uuid
71	19	&& array_key_exists('extension', $file)
72	19	&& $file['extension'] === $type
73	19	&& array_key_exists('type', $file)
74		// Warning: type field will be replaced by resource
75	19	&& $file['type'] === 'file'
76
77	19	;
78	20	});
79	20	array_walk(
80		$found
81	20	, [$this, 'hydrateElementFile']
82	20	, [
83	20	'resourceDO' => $resourceDO,
84		]
85	20	);
86	20	$found = array_values($found); // reset keys
87
88	20	return $found;
89		}
90
91	19	protected function hydrateElementFile(&$file, $key, $args)
92		{
93	19	if (!array_key_exists('resourceDO', $args) \|\| !$args['resourceDO'] instanceof ResourceDOInterface) {
94		throw new CommandErrorException('Method expects ResourceDO in arguments');
95		}
96
97		/** @var ResourceDOInterface $resourceDO */
98	19	$resourceDO = $args['resourceDO'];
		1 ignored issue – show Coding Style introduced 2016-06-03 19:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 23 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
99	19	$file[static::DIRECTORY_RELATIVE] = str_replace(
100	19	$resourceDO->getBaseDirectory()
101	19	. ($resourceDO->getNamespace() ? $resourceDO->getNamespace() . DIRECTORY_SEPARATOR : '')
102	19	, ''
103	19	, DIRECTORY_SEPARATOR . $file['dirname']
104	19	);
105	19	$tokens = $resourceDO->getDirectoryTokens();
		1 ignored issue – show Coding Style introduced 2016-06-03 19:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 27 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
106	19	$this->splitDirectoryByTokens($file, array_keys($tokens));
107	19	}
108
109	9	protected function filterAllowedProperties($value, $key)
110		{
111
112	9	return in_array($key, $this->allowedProperties, true);
113		}
114
115		/**
116		* @param $file
117		* @param $tokens
118		* @return mixed
119		*/
120	19	protected function splitDirectoryByTokens(&$file, $tokens)
121		{
122	19	$split = null;
123	19	foreach ($tokens as $token) {
124	19	if (ResourceDOAbstract::TOKEN_BASEDIRECTORY === $token) {
125	19	continue;
126		}
127	19	if (ResourceDOAbstract::TOKEN_NAMESPACE === $token) {
128	19	continue;
129		}
130	19	$split = null === $split
		1 ignored issue – show Coding Style introduced 2016-06-03 19:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 8 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
131	19	? strtok($file[static::DIRECTORY_RELATIVE], DIRECTORY_SEPARATOR)
132	19	: strtok(DIRECTORY_SEPARATOR);
133	19	$file[$token] = $split;
134	19	}
135	19	while (($split = strtok(DIRECTORY_SEPARATOR)) !== false) {
136		$file['sub-options'][] = $split;
137		}
138		}
139		}

KIVagant / staticus-core

Branch — master (267be1)

findAllResourceOptions() D

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like