kalanis\kw_mapper\Storage\Database\DatabaseSingleton

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	getInstance()	0	6	2
A	getFactory()	0	3	1
A	__construct()	0	2	1
A	getDatabase()	0	6	2
A	__clone()	0	2	1

<?php

namespace kalanis\kw_mapper\Storage\Database;


use kalanis\kw_mapper\MapperException;


/**
 * Class DatabaseSingleton
 * @package kalanis\kw_mapper\Storage\Database
 * Singleton to access databases across the mappers
 */
class DatabaseSingleton
{
    protected static ?DatabaseSingleton $instance = null;
    /** @var ADatabase[] */
    private array $database = [];

    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\DatabaseSingleton. Consider adding an additional type-check to rule them out.

    }

    protected function __construct()
    {
    }

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

    /**
     * @param Config $config
     * @throws MapperException
     * @return ADatabase
     */
    final public function getDatabase(Config $config): ADatabase
    {
        if (empty($this->database[$config->getDriver()])) {
            $this->database[$config->getDriver()] = $this->getFactory()->getDatabase($config);
        }
        return $this->database[$config->getDriver()];
    }

    protected function getFactory(): Factory
    {
        return Factory::getInstance();
    }
}