Dynamic\Salsify\Model\Service

Complexity

Total Complexity

5

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	15	5

<?php

namespace Dynamic\Salsify\Model;

use Dynamic\Salsify\Traits\Yieldable;
use SilverStripe\Core\Config\Config;
use SilverStripe\Core\Config\Configurable;
use SilverStripe\Core\Extensible;
use SilverStripe\Core\Injector\Injectable;

/**
 * Class Service
 * @package Dynamic\Salsify\Model
 *
 * @mixin Configurable
 * @mixin Extensible
 * @mixin Injectable
 * @mixin Yieldable
 */
abstract class Service
{
    use Configurable;
    use Extensible {
        defineMethods as extensibleDefineMethods;
    }
    use Injectable;
    use Yieldable;

    /**
     * @var string
     */
    protected $importerKey;

    /**
     * @var string
     */
    protected $serviceName;

    /**
     * Service constructor.
     * @param stirng $importerKey

The type Dynamic\Salsify\Model\stirng was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    public function __construct($importerKey)
    {
        $this->importerKey = $importerKey;
        $this->serviceName = static::class . '.' . $this->importerKey;

        $serviceConfig = Config::inst()->get($this->serviceName);
        foreach ($this->yieldKeyVal($serviceConfig) as $key => $value) {
            $this->config()->merge($key, $value);

It seems like $key can also be of type true; however, parameter $name of SilverStripe\Core\Config\Config_ForClass::merge() does only seem to accept string, maybe add an additional type check?

            if ($key === 'extensions') {
                if (!is_array($value)) {
                    throw new \Exception("Extensions for {$this->serviceName} is not an array");
                }
                foreach ($this->yieldSingle($value) as $extension) {
                    static::add_extension($extension);
                }
            }
        }
    }
}