kalanis\kw_mapper\Storage\Database\ConfigStorage

Complexity

Total Complexity

7

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getConfig()	0	6	2
A	getInstance()	0	6	2
A	addConfig()	0	3	1
A	__construct()	0	2	1
A	__clone()	0	2	1

<?php

namespace kalanis\kw_mapper\Storage\Database;


use kalanis\kw_mapper\MapperException;


/**
 * Class ConfigStorage
 * @package kalanis\kw_mapper\Storage\Database
 * Singleton to access configs across the mapper system
 */
class ConfigStorage
{
    protected static ?ConfigStorage $instance = null;
    /** @var Config[] */
    private array $configs = [];

    public static function getInstance(): self
    {
        if (empty(static::$instance)) {
            static::$instance = new self();
        }
        return static::$instance;

The expression return static::instance could return the type null which is incompatible with the type-hinted return kalanis\kw_mapper\Storage\Database\ConfigStorage. Consider adding an additional type-check to rule them out.

    }

    protected function __construct()
    {
    }

    /**
     * @codeCoverageIgnore why someone would run that?!
     */
    private function __clone()
    {
    }

    final public function addConfig(Config $config): void
    {
        $this->configs[$config->getSourceName()] = $config;
    }

    /**
     * @param string $sourceName
     * @throws MapperException
     * @return Config
     */
    final public function getConfig(string $sourceName): Config
    {
        if (empty($this->configs[$sourceName])) {
            throw new MapperException(sprintf('Unknown source *%s*', $sourceName));
        }
        return $this->configs[$sourceName];
    }
}