Kaliop\eZMigrationBundle\Core\Helper\LimitationConverter

Complexity

Total Complexity

24

Size/Duplication

Total Lines	121
Duplicated Lines	19.83 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	16	2
B	getLimitationArrayWithIdentifiers()	12	38	10
C	resolveLimitationValue()	12	40	12

<?php

namespace Kaliop\eZMigrationBundle\Core\Helper;

use eZ\Publish\API\Repository\Values\User\Limitation;
use Kaliop\eZMigrationBundle\Core\Matcher\LocationMatcher;
use Kaliop\eZMigrationBundle\Core\Matcher\SectionMatcher;
use Kaliop\eZMigrationBundle\Core\Matcher\ContentTypeMatcher;

class LimitationConverter
{
    protected $locationMatcher;
    protected $sectionMatcher;
    protected $contentTypeMatcher;

    /** @var array  */
    protected $siteAccessList = array();

    public function __construct(
        array $siteAccessList,
        LocationMatcher $locationMatcher,
        SectionMatcher $sectionMatcher,
        ContentTypeMatcher $contentTypeMatcher
    )
    {
        $this->locationMatcher = $locationMatcher;
        $this->sectionMatcher = $sectionMatcher;
        $this->contentTypeMatcher = $contentTypeMatcher;

        foreach ($siteAccessList as $siteAccess) {
            $id = sprintf('%u', crc32($siteAccess));
            $this->siteAccessList[$id] = $siteAccess;
        }
    }

    /**
     * Used f.e. to generate a human-readable role definition: substitute entity ids with their identifiers/names
     *
     * @param Limitation $limitation
     * @return array keys: identifier, values
     */
    public function getLimitationArrayWithIdentifiers(Limitation $limitation)
    {
        $retValues = array();
        $values = $limitation->limitationValues;

        switch ($limitation->getIdentifier()) {
            case 'Section':
            case 'NewSection':

This code seems to be duplicated across your project.

                foreach ($values as $value) {
                    $section = $this->sectionMatcher->matchOneByKey($value);
                    $retValues[] = $section->identifier;
                }
                break;
            case 'SiteAccess':
                foreach ($values as $value) {
                    $name = "No/unknown SiteAccess Found";
                    if (array_key_exists($value, $this->siteAccessList)) {
                        $name = $this->siteAccessList[$value];
                    }
                    $retValues[] = $name;
                }
                break;
            case 'ParentClass':
            case 'Class':

This code seems to be duplicated across your project.

                foreach ($values as $value) {
                    $contentType = $this->contentTypeMatcher->matchOneByKey($value);
                    $retValues[] = $contentType->identifier;
                }
                break;
            default:
                $retValues = $values;
        }

        return array(
            'identifier' => $limitation->getIdentifier(),
            'values' => $retValues
        );
    }

    /**
     * Converts human readable limitation values to their numeric counterparts we get the array in 2 parts so the
     * referencing can be completed on it before it's sent to this method.
     *
     * @param $limitationIdentifier
     * @param array $values
     * @return array
     * @throws \Exception
     */
    public function resolveLimitationValue($limitationIdentifier, array $values)
    {
        $retValues = array();
        switch ($limitationIdentifier) {
            // q: is it worth doing this for nodes? does it make a translation at all?
            case 'Node':
                foreach ($values as $value) {
                    $location = $this->locationMatcher->matchOneByKey($value);
                    $retValues[] = $location->id;
                }
                break;
            case 'Section':
            case 'NewSection':

This code seems to be duplicated across your project.

                foreach ($values as $value) {
                    $section = $this->sectionMatcher->matchOneByKey($value);
                    $retValues[] = $section->id;
                }
                break;
            case 'SiteAccess':
                $siteAccesses = array_flip($this->siteAccessList);
                foreach ($values as $value) {
                    if (!isset($siteAccesses[$value])) {
                        throw new \Exception("SiteAccess '$value' is not configured");
                    }
                    $retValues[] = (string)$siteAccesses[$value];
                }
                break;
            case 'ParentClass':
            case 'Class':

This code seems to be duplicated across your project.

                foreach ($values as $value) {
                    $contentType = $this->contentTypeMatcher->matchOneByKey($value);
                    $retValues[] = $contentType->id;
                }
                break;
            default:
                $retValues = $values;
        }

        return $retValues;
    }
}