Arp\LaminasDoctrine\Service\Connection\ConnectionFactory

Complexity

Total Complexity

5

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	26	4
A	__construct()	0	3	1

<?php

declare(strict_types=1);

namespace Arp\LaminasDoctrine\Service\Connection;

use Arp\LaminasDoctrine\Service\Exception\ConnectionFactoryException;

The type Arp\LaminasDoctrine\Service\Exception\ConnectionFactoryException was not found. Maybe you did not declare it correctly or list all dependencies?

use Doctrine\Common\EventManager;
use Doctrine\DBAL\Configuration;
use Doctrine\DBAL\Connection;
use Doctrine\DBAL\Driver\PDO\MySQL\Driver;
use Doctrine\DBAL\DriverManager;

/**
 * @author  Alex Patterson <alex.patterson.webdev@gmail.com>
 * @package Arp\LaminasDoctrine\Service\Connection
 */
final class ConnectionFactory implements ConnectionFactoryInterface
{
    /**
     * @var ConfigurationManager
     */
    private ConfigurationManager $configurationManager;

The type Arp\LaminasDoctrine\Service\Connection\ConfigurationManager was not found. Maybe you did not declare it correctly or list all dependencies?

    /**
     * @param ConfigurationManager $configurationManager
     */
    public function __construct(ConfigurationManager $configurationManager)
    {
        $this->configurationManager = $configurationManager;
    }

    /**
     * Create a new connection from the provided $config
     *
     * @param array                     $config
     * @param Configuration|string|null $configuration
     * @param EventManager|string|null  $eventManager
     *
     * @return Connection
     *
     * @throws ConnectionFactoryException
     */
    public function create(array $config, $configuration = null, $eventManager = null): Connection
    {
        $params = array_merge(
            [
                'driverClass'  => $config['driverClass'] ?? Driver::class,
                'wrapperClass' => $config['wrapperClass'] ?? null,
                'pdo'          => $config['pdo'] ?? null,
            ],
            $config['params'] ?? []
        );

        if (!empty($config['platform'])) {
            $platform = null;

The assignment to $platform is dead and can be removed.

        }

        try {
            if (is_string($configuration)) {
                $configuration = $this->configurationManager->getConfiguration($configuration);
            }

            return DriverManager::getConnection($params, $configuration, $eventManager);

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

        } catch (\Throwable $e) {
            throw new ConnectionFactoryException(
                sprintf('Failed to create new connection: %s', $e->getMessage()),
                $e->getCode(),
                $e
            );
        }
    }
}