IdsChecker::validateIds() - Code Metrics - Inspection of "improve id checks" - studiomado/query-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — refactoring/additional-filters... ( 98d337...dd0af0 )

by Simone

created 2018-02-13 10:33 UTC

IdsChecker::validateIds() C

↳ Parent: IdsChecker

Complexity

Conditions	11
Paths	26

Size

Total Lines	53
Code Lines	27

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	27
CRAP Score	11.0055

Importance

Changes

Metric	Value
dl	0
loc	53
ccs	27
cts	28
cp	0.9643
rs	6.2926
c	0
b	0
f	0
cc	11
eloc	27
nc	26
nop	0
crap	11.0055

How to fix Long Method Complexity

<?php

namespace Mado\QueryBundle\Services;

use Mado\QueryBundle\Exceptions\ForbiddenContentException;
use Mado\QueryBundle\Objects\Filter;
use Psr\Log\LoggerInterface;

class IdsChecker
{
    private $idsMustBeSubset = true;

    private $filtering;

    private $additionalFilter;

    private $filterKey;

    private $finalFilterIds;

    private $logger;

    public function __construct()
    {
        $this->idsMustBeSubset = true;
    }

    public function setFiltering($filtering)
    {
        $this->filtering = $filtering;
    }

    public function setObjectFilter(Filter $additionalFilter)
    {
        $this->additionalFilter = $additionalFilter;
    }

    public function setFilterKey($filterKey)
    {
        $this->filterKey = $filterKey;
    }

    public function validateIds()
    {
        $rawFilteredIds = $this->additionalFilter->getIds();

        // guard
        $chiave = key($this->filtering);
        $chiaveEsplosa = explode('|' , $chiave);
        if (!isset($chiaveEsplosa[1]) || !in_array($chiaveEsplosa[1], ['list', 'nlist'])) {
            return;
        }

        // check
        foreach ($this->filtering as $key => $queryStringIds) {
            $querystringIds = explode(',', $queryStringIds);
            $additionalFiltersIds = explode(',', $this->additionalFilter->getIds());
            foreach ($querystringIds as $requestedId) {

                // list + list
                if ($this->additionalFilter->getOperator() == 'list') {
                    if (!in_array($requestedId, $additionalFiltersIds)) {
                        throw new ForbiddenContentException(
                            'Oops! Forbidden requested id ' . $requestedId
                            . ' is not available. '
                            . 'Available are ' . join(', ', $additionalFiltersIds) . ''
                        );
                    }
                }

                // list + nlist
                if ($this->additionalFilter->getOperator() == 'nlist') {
                    $queryStringOperator = explode('|', key($this->filtering));

                    if (array_intersect($querystringIds, $additionalFiltersIds) == []) {
                        $this->filterKey = str_replace('nlist', 'list', $this->additionalFilter->getRawFilter());
                        $rawFilteredIds = join(',', $querystringIds);
                        $this->idsMustBeSubset = false;
                    }
                }


                // nlist + list

                // nlist + nlist

            }
        }

        $this->finalFilterIds = $rawFilteredIds;

        if (true == $this->idsMustBeSubset) {

            foreach ($this->filtering as $key => $queryStringIds) {
                $querystringIds = explode(',', $queryStringIds);
                $additionalFiltersIds = explode(',', $this->additionalFilter->getIds());
                $this->finalFilterIds = join(',', array_intersect($querystringIds, $additionalFiltersIds));
            }
        }
    }

    public function getFilterKey()
    {
        return $this->filterKey;
    }

    public function getFinalFilterIds()
    {
        return $this->finalFilterIds;
    }

    public function setLogger(LoggerInterface $logger)
    {
        $this->logger = $logger;
    }

    public function log($message)
    {
        if ($this->logger) {
            $this->logger->critical($message);
        }
    }
}


1		<?php
2
3		namespace Mado\QueryBundle\Services;
4
5		use Mado\QueryBundle\Exceptions\ForbiddenContentException;
6		use Mado\QueryBundle\Objects\Filter;
7		use Psr\Log\LoggerInterface;
8
9		class IdsChecker
10		{
11		private $idsMustBeSubset = true;
12
13		private $filtering;
14
15		private $additionalFilter;
16
17		private $filterKey;
18
19		private $finalFilterIds;
20
21		private $logger;
22
23	5	public function __construct()
24		{
25	5	$this->idsMustBeSubset = true;
26	5	}
27
28	5	public function setFiltering($filtering)
29		{
30	5	$this->filtering = $filtering;
31	5	}
32
33	5	public function setObjectFilter(Filter $additionalFilter)
34		{
35	5	$this->additionalFilter = $additionalFilter;
36	5	}
37
38	1	public function setFilterKey($filterKey)
39		{
40	1	$this->filterKey = $filterKey;
41	1	}
42
43	5	public function validateIds()
44		{
45	5	$rawFilteredIds = $this->additionalFilter->getIds();
46
47		// guard
48	5	$chiave = key($this->filtering);
49	5	$chiaveEsplosa = explode('\|' , $chiave);
50	5	if (!isset($chiaveEsplosa[1]) \|\| !in_array($chiaveEsplosa[1], ['list', 'nlist'])) {
51		return;
52		}
53
54		// check
55	5	foreach ($this->filtering as $key => $queryStringIds) {
56	5	$querystringIds = explode(',', $queryStringIds);
57	5	$additionalFiltersIds = explode(',', $this->additionalFilter->getIds());
58	5	foreach ($querystringIds as $requestedId) {
59
60		// list + list
61	5	if ($this->additionalFilter->getOperator() == 'list') {
62	2	if (!in_array($requestedId, $additionalFiltersIds)) {
63	1	throw new ForbiddenContentException(
64	1	'Oops! Forbidden requested id ' . $requestedId
65	1	. ' is not available. '
66	1	. 'Available are ' . join(', ', $additionalFiltersIds) . ''
67		);
68		}
69		}
70
71		// list + nlist
72	4	if ($this->additionalFilter->getOperator() == 'nlist') {
73	3	$queryStringOperator = explode('\|', key($this->filtering));
		0 ignored issues – show Unused Code introduced 2018-02-08 14:40 UTC by Report Bug Copy Issue Report The assignment to `$queryStringOperator` is dead and can be removed. Loading history...
74	3	if (array_intersect($querystringIds, $additionalFiltersIds) == []) {
75	3	$this->filterKey = str_replace('nlist', 'list', $this->additionalFilter->getRawFilter());
76	3	$rawFilteredIds = join(',', $querystringIds);
77	4	$this->idsMustBeSubset = false;
78		}
79		}
80
81
82		// nlist + list
83
84		// nlist + nlist
85
86		}
87		}
88
89	4	$this->finalFilterIds = $rawFilteredIds;
90
91	4	if (true == $this->idsMustBeSubset) {
		0 ignored issues – show Coding Style Best Practice introduced 2018-02-08 14:40 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing two booleans. Considering using the strict comparison `===` instead. When comparing two booleans, it is generally considered safer to use the strict comparison operator. Loading history...
92	1	foreach ($this->filtering as $key => $queryStringIds) {
93	1	$querystringIds = explode(',', $queryStringIds);
94	1	$additionalFiltersIds = explode(',', $this->additionalFilter->getIds());
95	1	$this->finalFilterIds = join(',', array_intersect($querystringIds, $additionalFiltersIds));
96		}
97		}
98	4	}
99
100	2	public function getFilterKey()
101		{
102	2	return $this->filterKey;
103		}
104
105	2	public function getFinalFilterIds()
106		{
107	2	return $this->finalFilterIds;
108		}
109
110		public function setLogger(LoggerInterface $logger)
111		{
112		$this->logger = $logger;
113		}
114
115		public function log($message)
116		{
117		if ($this->logger) {
118		$this->logger->critical($message);
119		}
120		}
121		}
122

studiomado / query-bundle

GitHub Access Token became invalid

Push — refactoring/additional-filters... ( 98d337...dd0af0 )

IdsChecker::validateIds() C

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like