Ray\DbalModule\DbalProvider

Complexity

Total Complexity

6

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

85%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	setContext()	0	4	1
A	init()	0	18	3
A	get()	0	6	1

<?php
/**
 * This file is part of the Ray.DbalModule package.
 *
 * @license http://opensource.org/licenses/MIT MIT
 */
namespace Ray\DbalModule;

use Doctrine\DBAL\DriverManager;
use Ray\DbalModule\Annotation\DbalConfig;
use Ray\Di\Di\PostConstruct;
use Ray\Di\InjectorInterface;
use Ray\Di\ProviderInterface;
use Ray\Di\SetContextInterface;

class DbalProvider implements ProviderInterface, SetContextInterface
{
    /**
     * @var InjectorInterface
     */
    private $injector;

    /**
     * @var string
     */
    private $context;

    /**
     * @var array
     */
    private $config;

    public function __construct(InjectorInterface $injector)
    {
        $this->injector = $injector;
    }

    public function setContext($context)
    {
        $this->context = $context;
    }

    /**
     * @PostConstruct
     */
    public function init()
    {
        $config = $this->injector->getInstance('', DbalConfig::class . $this->context);
        if (is_array($config)) {
            $this->config = $config;

            return;
        }
        if (is_string($config)) {
            $parsedConfig = [];
            parse_str($config, $parsedConfig);
            $this->config = $parsedConfig;

It seems like $parsedConfig can be null. However, the property $config is declared as array. Maybe change the type of the property to array|null or add a type check?

            return;
        }

        throw new \InvalidArgumentException('@DbalConfig');
    }

    /**
     * {@inheritdoc}
     */
    public function get()
    {
        $conn = DriverManager::getConnection($this->config);

        return $conn;
    }
}