Antidot\Persistence\DBAL\Container\DBALConnectionFactory

Complexity

Total Complexity

2

Size/Duplication

Total Lines	16
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	1

1 Method

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	11	2

<?php

declare(strict_types=1);

namespace Antidot\Persistence\DBAL\Container;

use Doctrine\DBAL\Connection;
use Doctrine\DBAL\DriverManager;
use InvalidArgumentException;
use Psr\Container\ContainerInterface;

class DBALConnectionFactory
{
    public const DEFAULT_CONNECTION_NAME = 'default';
    public const MISSING_CONFIG_MESSAGE = 'Missing config for dbl connection inside the container.';

    public function __invoke(
        ContainerInterface $container,
        string $connectionName = self::DEFAULT_CONNECTION_NAME
    ): Connection {
        $connectionParams = $container->get('config')['dbal']['connections'][$connectionName] ?? null;

        if (false === is_array($connectionParams)) {
            throw new InvalidArgumentException(self::MISSING_CONFIG_MESSAGE);
        }

        return DriverManager::getConnection($connectionParams);

It seems like $connectionParams can also be of type null; however, parameter $params of Doctrine\DBAL\DriverManager::getConnection() does only seem to accept array, maybe add an additional type check?

    }
}