SectionMatcher::matchSection() - Code Metrics - Inspection of "Allow composite matching using and and or" - kaliop-uk/ezmigrationbundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 8c5037...eed80c )

by Gaetano

created 2017-02-04 01:45 UTC

SectionMatcher::matchSection() C

↳ Parent: SectionMatcher

Complexity

Conditions	11
Paths	19

Size

Total Lines	33
Code Lines	20

Duplication

Lines	33
Ratio	100 %

Importance

Changes

Metric	Value
dl	33
loc	33
c	0
b	0
f	0
rs	5.2653
cc	11
eloc	20
nc	19
nop	1

How to fix Complexity

<?php

namespace Kaliop\eZMigrationBundle\Core\Matcher;

use eZ\Publish\API\Repository\Values\Content\Section;
use Kaliop\eZMigrationBundle\API\Collection\SectionCollection;
use Kaliop\eZMigrationBundle\API\KeyMatcherInterface;

class SectionMatcher extends RepositoryMatcher implements KeyMatcherInterface

{
    use FlexibleKeyMatcherTrait;

    const MATCH_SECTION_ID = 'section_id';
    const MATCH_SECTION_IDENTIFIER = 'section_identifier';

    protected $allowedConditions = array(
        self::MATCH_ALL, self::MATCH_AND, self::MATCH_OR, self::MATCH_NOT,
        self::MATCH_SECTION_ID, self::MATCH_SECTION_IDENTIFIER,
        // aliases
        'id', 'identifier'
    );
    protected $returns = 'Role';

    /**
     * @param array $conditions key: condition, value: int / string / int[] / string[]
     * @return SectionCollection
     */
    public function match(array $conditions)
    {
        return $this->matchSection($conditions);
    }

    /**
     * @param array $conditions key: condition, value: int / string / int[] / string[]
     * @return SectionCollection
     */
    public function matchSection(array $conditions)
    {
        $this->validateConditions($conditions);

        foreach ($conditions as $key => $values) {

            if (!is_array($values)) {
                $values = array($values);
            }

            switch ($key) {
                case 'id':
                case self::MATCH_SECTION_ID:
                   return new SectionCollection($this->findSectionsById($values));

                case 'identifier':
                case self::MATCH_SECTION_IDENTIFIER:
                    return new SectionCollection($this->findSectionsByIdentifier($values));

                case self::MATCH_ALL:
                    return new SectionCollection($this->findAllSections());

                case self::MATCH_AND:
                    return $this->matchAnd($values);


                case self::MATCH_OR:
                    return $this->matchOr($values);


                case self::MATCH_NOT:
                    return new SectionCollection(array_diff_key($this->findAllSections(), $this->matchSection($values)->getArrayCopy()));
            }
        }
    }

    protected function getConditionsFromKey($key)
    {
        if (is_int($key) || ctype_digit($key)) {
            return array(self::MATCH_SECTION_ID => $key);
        }
        return array(self::MATCH_SECTION_IDENTIFIER => $key);
    }

    /**
     * @param int[] $sectionIds
     * @return Section[]
     */
    protected function findSectionsById(array $sectionIds)
    {
        $sections = [];

        foreach ($sectionIds as $sectionId) {
            // return unique contents
            $section = $this->repository->getSectionService()->loadSection($sectionId);
            $sections[$section->id] = $section;
        }

        return $sections;
    }

    /**
     * @param string[] $sectionIdentifiers
     * @return Section[]
     */
    protected function findSectionsByIdentifier(array $sectionIdentifiers)
    {
        $sections = [];

        foreach ($sectionIdentifiers as $sectionIdentifier) {
            // return unique contents
            $section = $this->repository->getSectionService()->loadSectionByIdentifier($sectionIdentifier);
            $sections[$section->id] = $section;
        }

        return $sections;
    }

    /**
     * @return Section[]
     */
    protected function findAllSections()
    {
        $sections = [];

        foreach ($this->repository->getSectionService()->loadSections() as $section) {
            // return unique contents
            $sections[$section->id] = $section;
        }

        return $sections;
    }
}


1		<?php
2
3		namespace Kaliop\eZMigrationBundle\Core\Matcher;
4
5		use eZ\Publish\API\Repository\Values\Content\Section;
6		use Kaliop\eZMigrationBundle\API\Collection\SectionCollection;
7		use Kaliop\eZMigrationBundle\API\KeyMatcherInterface;
8
9	View Code Duplication	class SectionMatcher extends RepositoryMatcher implements KeyMatcherInterface
		0 ignored issues – show Duplication introduced 2016-09-25 22:10 UTC by Report Bug Copy Issue Report This class seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
10		{
11		use FlexibleKeyMatcherTrait;
12
13		const MATCH_SECTION_ID = 'section_id';
14		const MATCH_SECTION_IDENTIFIER = 'section_identifier';
15
16		protected $allowedConditions = array(
17		self::MATCH_ALL, self::MATCH_AND, self::MATCH_OR, self::MATCH_NOT,
18		self::MATCH_SECTION_ID, self::MATCH_SECTION_IDENTIFIER,
19		// aliases
20		'id', 'identifier'
21		);
22		protected $returns = 'Role';
23
24		/**
25		* @param array $conditions key: condition, value: int / string / int[] / string[]
26		* @return SectionCollection
27		*/
28		public function match(array $conditions)
29		{
30		return $this->matchSection($conditions);
31		}
32
33		/**
34		* @param array $conditions key: condition, value: int / string / int[] / string[]
35		* @return SectionCollection
36		*/
37		public function matchSection(array $conditions)
38		{
39		$this->validateConditions($conditions);
40
41		foreach ($conditions as $key => $values) {
42
43		if (!is_array($values)) {
44		$values = array($values);
45		}
46
47		switch ($key) {
48		case 'id':
49		case self::MATCH_SECTION_ID:
50		return new SectionCollection($this->findSectionsById($values));
51
52		case 'identifier':
53		case self::MATCH_SECTION_IDENTIFIER:
54		return new SectionCollection($this->findSectionsByIdentifier($values));
55
56		case self::MATCH_ALL:
57		return new SectionCollection($this->findAllSections());
58
59		case self::MATCH_AND:
60		return $this->matchAnd($values);
		0 ignored issues – show Bug Compatibility introduced 2017-02-04 01:52 UTC by Report Bug Copy Issue Report The expression `$this->matchAnd($values);` of type `object\|array` adds the type `array` to the return on line 60 which is incompatible with the return type documented by `Kaliop\eZMigrationBundle...onMatcher::matchSection` of type `Kaliop\eZMigrationBundle...\SectionCollection\|null`. Loading history...
61
62		case self::MATCH_OR:
63		return $this->matchOr($values);
		0 ignored issues – show Bug Compatibility introduced 2017-02-04 01:52 UTC by Report Bug Copy Issue Report The expression `$this->matchOr($values);` of type `object\|array` adds the type `array` to the return on line 63 which is incompatible with the return type documented by `Kaliop\eZMigrationBundle...onMatcher::matchSection` of type `Kaliop\eZMigrationBundle...\SectionCollection\|null`. Loading history...
64
65		case self::MATCH_NOT:
66		return new SectionCollection(array_diff_key($this->findAllSections(), $this->matchSection($values)->getArrayCopy()));
67		}
68		}
69		}
70
71		protected function getConditionsFromKey($key)
72		{
73		if (is_int($key) \|\| ctype_digit($key)) {
74		return array(self::MATCH_SECTION_ID => $key);
75		}
76		return array(self::MATCH_SECTION_IDENTIFIER => $key);
77		}
78
79		/**
80		* @param int[] $sectionIds
81		* @return Section[]
82		*/
83		protected function findSectionsById(array $sectionIds)
84		{
85		$sections = [];
86
87		foreach ($sectionIds as $sectionId) {
88		// return unique contents
89		$section = $this->repository->getSectionService()->loadSection($sectionId);
90		$sections[$section->id] = $section;
91		}
92
93		return $sections;
94		}
95
96		/**
97		* @param string[] $sectionIdentifiers
98		* @return Section[]
99		*/
100		protected function findSectionsByIdentifier(array $sectionIdentifiers)
101		{
102		$sections = [];
103
104		foreach ($sectionIdentifiers as $sectionIdentifier) {
105		// return unique contents
106		$section = $this->repository->getSectionService()->loadSectionByIdentifier($sectionIdentifier);
107		$sections[$section->id] = $section;
108		}
109
110		return $sections;
111		}
112
113		/**
114		* @return Section[]
115		*/
116		protected function findAllSections()
117		{
118		$sections = [];
119
120		foreach ($this->repository->getSectionService()->loadSections() as $section) {
121		// return unique contents
122		$sections[$section->id] = $section;
123		}
124
125		return $sections;
126		}
127		}
128

kaliop-uk / ezmigrationbundle

Push — master ( 8c5037...eed80c )

SectionMatcher::matchSection() C

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like