Surfnet\StepupMiddleware\ManagementBundle\Service\DBALConnectionHelper

Complexity

Total Complexity

12

Size/Duplication

Total Lines	65
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	beginTransaction()	0	4	2
A	__construct()	0	9	3
A	commit()	0	4	2
A	rollBack()	0	4	2
A	getConnection()	0	11	3

<?php

/**
 * Copyright 2016 SURFnet B.V.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

Missing @link tag in file comment

namespace Surfnet\StepupMiddleware\ManagementBundle\Service;

use Doctrine\DBAL\Connection;
use Surfnet\StepupMiddleware\ManagementBundle\Exception\InvalidArgumentException;
use Surfnet\StepupMiddleware\ManagementBundle\Exception\UnknownDBALConnectionException;

class DBALConnectionHelper

Missing doc comment for class DBALConnectionHelper

{
    /**
     * @var Connection[]
     */
    private array $connections;

    /**
     * @param Connection[] $connections
     */
    public function __construct(array $connections)
    {
        foreach ($connections as $connection) {
            if (!$connection instanceof Connection) {
                throw InvalidArgumentException::invalidType(Connection::class, 'connection', $connection);
            }
        }

        $this->connections = $connections;
    }

    /**
     * Start transaction on each connection
     */

Missing @return tag in function comment

    public function beginTransaction(): void
    {
        foreach ($this->connections as $connection) {
            $connection->beginTransaction();
        }
    }

    /**
     * Commit transaction on each connection
     */

Missing @return tag in function comment

    public function commit(): void
    {
        foreach ($this->connections as $connection) {
            $connection->commit();
        }
    }

    /**
     * Roll back the transaction on each connection
     */

Missing @return tag in function comment

    public function rollBack(): void
    {
        foreach ($this->connections as $connection) {
            $connection->rollBack();
        }
    }

    /**

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

     * @return Connection
     */
    public function getConnection(string $connectionName): Connection
    {
        if (!is_string($connectionName)) {

The condition is_string($connectionName) is always true.

            throw InvalidArgumentException::invalidType('string', 'connectionName', $connectionName);
        }

        if (!array_key_exists($connectionName, $this->connections)) {
            throw new UnknownDBALConnectionException($connectionName);
        }

        return $this->connections[$connectionName];
    }
}