Kaliop\eZMigrationBundle\Core\ReferenceResolver\ChainRegexpResolver

Complexity

Total Complexity

5

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	addResolver()	0	7	2
A	getRegexp()	0	11	3

<?php

namespace Kaliop\eZMigrationBundle\Core\ReferenceResolver;

use Kaliop\eZMigrationBundle\API\Exception\MigrationBundleException;
use Kaliop\eZMigrationBundle\API\ReferenceResolverInterface;

class ChainRegexpResolver extends ChainResolver implements RegexpBasedResolverInterface
{
    public function addResolver(ReferenceResolverInterface $resolver)
    {
        if (!$resolver instanceof RegexpBasedResolverInterface) {
            throw new MigrationBundleException("Can not add resolver of class " . get_class($resolver) . " to a chain regexp resolver");
        }

        parent::addResolver($resolver);
    }

    /**
     * NB: assumes that all the resolvers we chain use '/' as delimiter...
     * @return string
     */
    public function getRegexp()
    {
        $regexps = array();
        /** @var RegexpBasedResolverInterface $resolver */
        foreach ($this->resolvers as $resolver) {
            $regexp = substr($resolver->getRegexp(), 1, -1);

The method getRegexp() does not exist on Kaliop\eZMigrationBundle\API\ReferenceResolverInterface. It seems like you code against a sub-type of said class. However, the method does not exist in Kaliop\eZMigrationBundle\API\ReferenceResolverBagInterface or Kaliop\eZMigrationBundle\API\EmbeddedReferenceResolverBagInterface or Kaliop\eZMigrationBundle\Core\ReferenceResolver\ChainResolver. Are you sure you never get one of those?

            if ($regexp !== '') {
                $regexps[] = $regexp;
            }
        }
        return '/(' . implode(')|(', $regexps) . '))/';
    }
}