ConfigStorage::addConfig() - Code Metrics - Inspection of "kw_mapper - changes for MSSQL and Oracle, separate..." - alex-kalanis/kw_mapper - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 5d8af1...e44bd6 )

by Petr

created 2025-05-11 16:15 UTC

ConfigStorage::addConfig() A

↳ Parent: ConfigStorage

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	1
c	1
b	0
f	0
dl	0
loc	3
ccs	2
cts	2
cp	1
rs	10
cc	1
nc	1
nop	1
crap	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;

    }

    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];
    }
}


1		<?php
2
3		namespace kalanis\kw_mapper\Storage\Database;
4
5
6		use kalanis\kw_mapper\MapperException;
7
8
9		/**
10		* Class ConfigStorage
11		* @package kalanis\kw_mapper\Storage\Database
12		* Singleton to access configs across the mapper system
13		*/
14		class ConfigStorage
15		{
16		protected static ?ConfigStorage $instance = null;
17		/** @var Config[] */
18		private array $configs = [];
19
20	67	public static function getInstance(): self
21		{
22	67	if (empty(static::$instance)) {
23	1	static::$instance = new self();
24		}
25	67	return static::$instance;
		0 ignored issues – show Bug Best Practice introduced 2024-04-01 13:31 UTC by Report Bug Copy Issue Report 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. Loading history...
26		}
27
28	1	protected function __construct()
29		{
30	1	}
31
32		/**
33		* @codeCoverageIgnore why someone would run that?!
34		*/
35		private function __clone()
36		{
37		}
38
39	64	final public function addConfig(Config $config): void
40		{
41	64	$this->configs[$config->getSourceName()] = $config;
42		}
43
44		/**
45		* @param string $sourceName
46		* @throws MapperException
47		* @return Config
48		*/
49	27	final public function getConfig(string $sourceName): Config
50		{
51	27	if (empty($this->configs[$sourceName])) {
52	1	throw new MapperException(sprintf('Unknown source %s', $sourceName));
53		}
54	26	return $this->configs[$sourceName];
55		}
56		}
57

alex-kalanis / kw_mapper

Push — master ( 5d8af1...e44bd6 )

ConfigStorage::addConfig() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like