Sugarcrm\UpgradeSpec\Data\ProviderChain

Complexity

Total Complexity

10

Size/Duplication

Total Lines	56
Duplicated Lines	28.57 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	2
Bugs	1	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	addProviders()	16	16	6
A	__call()	0	10	3

<?php

namespace Sugarcrm\UpgradeSpec\Data;

use Sugarcrm\UpgradeSpec\Data\Provider\ProviderInterface;

class ProviderChain
{
    /**
     * @var array
     */
    private $providers = [];

    /**
     * ProviderChain constructor.
     *
     * @param mixed $providers
     */
    public function __construct($providers)
    {
        $this->addProviders($providers);
    }

    /**
     * Adds providers to the chain
     *
     * @param mixed $providers
     */
    public function addProviders($providers)

This method seems to be duplicated in your project.

    {
        if (!is_array($providers) && !$providers instanceof \Traversable) {
            throw new \InvalidArgumentException(sprintf('Argument is not traversable: %s', $providers));
        }

        $providers = is_array($providers) ? $providers : iterator_to_array($providers);

Consider using a different name than the parameter $providers. This often makes code more readable.

        foreach ($providers as $provider) {
            if (!is_a($provider, ProviderInterface::class)) {
                throw new \InvalidArgumentException('ProviderChain expects ProviderInterface[]');
            }
        }

        $this->providers = array_merge($this->providers, $providers);
    }

    /**
     * @param $name
     * @param $arguments
     *
     * @return mixed
     */
    public function __call($name, $arguments)
    {
        foreach ($this->providers as $provider) {
            if (method_exists($provider, $name)) {
                return call_user_func_array([$provider, $name], $arguments);
            }
        }

        throw new \RuntimeException(sprintf('There is no provider with method: %s', $name));
    }
}