HtImgModule\Imagine\Filter\Loader\Chain

Complexity

Total Complexity

6

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	load()	0	20	5

<?php
namespace HtImgModule\Imagine\Filter\Loader;

use HtImgModule\Exception;
use HtImgModule\Imagine\Filter\Chain as ChainFilter;
use Zend\ServiceManager\ServiceLocatorInterface;

class Chain implements LoaderInterface
{
    /**
     * @var ServiceLocatorInterface
     */
    protected $filterLoaders;

    /**
     * @param ServiceLocatorInterface $filterLoaders
     */
    public function __construct(ServiceLocatorInterface $filterLoaders)
    {
        $this->filterLoaders = $filterLoaders;
    }

    /**
     * {@inheritDoc}
     */
    public function load(array $options = [])
    {
        if (false == isset($options['filters']) || false == is_array($options['filters'])) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            throw new Exception\InvalidArgumentException('Expected filters key and type of array');
        }

        if (false == $options['filters']) {
            throw new Exception\InvalidArgumentException('At least one filter expected');
        }

        $filters = [];

        foreach ($options['filters'] as $loaderName => $loaderOptions) {
            $loader = $this->filterLoaders->get($loaderName);
            $filters[] = $loader->load($loaderOptions);
        }

        return new ChainFilter($filters);

    }
}