Bdf\Prime\Connection\Factory\MasterSlaveConnectionFactory

Complexity

Total Complexity

3

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	support()	0	3	1
A	create()	0	11	1
A	__construct()	0	3	1

<?php

namespace Bdf\Prime\Connection\Factory;

use Bdf\Prime\Connection\ConnectionInterface;
use Bdf\Prime\Connection\MasterSlaveConnection;
use Doctrine\DBAL\Configuration;

/**
 * MasterSlaveConnectionLoader
 */
class MasterSlaveConnectionFactory implements ConnectionFactoryInterface
{
    /**
     * The delegated connectionFactory
     *
     * @var ConnectionFactoryInterface
     */
    private $connectionFactory;

Expected 1 blank line(s) before first member var; 0 found

    /**
     * Set default configuration
     *
     * @param ConnectionFactoryInterface $connectionFactory
     */
    public function __construct(ConnectionFactoryInterface $connectionFactory)
    {
        $this->connectionFactory = $connectionFactory;
    }

    /**

Parameter $connectionName should have a doc-comment as per coding-style.

Parameter $parameters should have a doc-comment as per coding-style.

Parameter $config should have a doc-comment as per coding-style.

     * {@inheritDoc}
     */
    public function create(string $connectionName, array $parameters, Configuration $config): ConnectionInterface
    {
        $masterParameters = $parameters;
        unset($masterParameters['read']);
        unset($masterParameters['wrapperClass']);
        $readParameters = array_merge($masterParameters, $parameters['read']);

        $parameters['read'] = $this->connectionFactory->create($connectionName.'.read', $readParameters, $config);
        $parameters['wrapperClass'] = $parameters['wrapperClass'] ?? MasterSlaveConnection::class;

Operation must be bracketed

        return $this->connectionFactory->create($connectionName, $parameters, $config);
    }

    /**

Parameter $connectionName should have a doc-comment as per coding-style.

Parameter $parameters should have a doc-comment as per coding-style.

     * {@inheritDoc}
     */
    public function support(string $connectionName, array $parameters): bool

The method parameter $connectionName is never used

    {
        return isset($parameters['read']);
    }
}