StefanoTree\NestedSet\Adapter\Doctrine2DBAL

Complexity

Total Complexity

14

Size/Duplication

Total Lines	89
Duplicated Lines	0 %

Test Coverage

Coverage

97.3%

Importance

Changes	1
Bugs	1	Features	0

12 Methods

Rating	Name	Duplication	Size	Complexity
A	rollbackTransaction()	0	4	1
A	__construct()	0	4	1
A	commitTransaction()	0	4	1
A	isInTransaction()	0	3	2
A	executeSelectSQL()	0	5	1
A	getConnection()	0	3	1
A	beginTransaction()	0	4	1
A	getOptions()	0	3	1
A	canHandleNestedTransaction()	0	3	1
A	quoteIdentifier()	0	4	1
A	executeSQL()	0	4	1
A	executeInsertSQL()	0	10	2

<?php

declare(strict_types=1);

namespace StefanoTree\NestedSet\Adapter;

use Doctrine\DBAL\Connection as DbConnection;
use StefanoTree\NestedSet\Options;

class Doctrine2DBAL implements AdapterInterface
{
    private $connection;
    private $options;

    /**
     * @param Options      $options
     * @param DbConnection $connection
     */
    public function __construct(Options $options, DbConnection $connection)
    {
        $this->connection = $connection;
        $this->options = $options;
    }

    /**
     * @return DbConnection
     */
    private function getConnection(): DbConnection
    {
        return $this->connection;
    }

    /**
     * @return Options
     */
    public function getOptions(): Options
    {
        return $this->options;
    }

    public function beginTransaction(): void
    {
        $this->getConnection()
            ->beginTransaction();
    }

    public function commitTransaction(): void
    {
        $this->getConnection()
            ->commit();
    }

    public function rollbackTransaction(): void
    {
        $this->getConnection()
            ->rollBack();
    }

    public function isInTransaction(): bool
    {
        return 0 === $this->getConnection()->getTransactionNestingLevel() ? false : true;
    }

    public function canHandleNestedTransaction(): bool
    {
        return true;
    }

    public function quoteIdentifier(string $columnName): string
    {
        return $this->getConnection()
            ->quoteIdentifier($columnName);
    }

    public function executeInsertSQL(string $sql, array $params = array())
    {
        $options = $this->getOptions();
        $this->executeSQL($sql, $params);

        if (array_key_exists($options->getIdColumnName(), $params)) {
            return $params[$options->getIdColumnName()];
        } else {
            return $this->getConnection()

The expression return $this->getConnection()->lastInsertId($options->getSequenceName()) also could return the type false which is incompatible with the return type mandated by StefanoTree\NestedSet\Adapter\AdapterInterface::executeInsertSQL() of integer|string.

                ->lastInsertId($options->getSequenceName());
        }
    }

    public function executeSQL(string $sql, array $params = array()): void
    {
        $this->getConnection()
            ->executeQuery($sql, $params);
    }

    public function executeSelectSQL(string $sql, array $params = array()): array
    {
        return $this->getConnection()

The function Doctrine\DBAL\ForwardCompatibility\Result::fetchAll() has been deprecated: Use fetchAllNumeric(), fetchAllAssociative() or fetchFirstColumn() instead.

            ->executeQuery($sql, $params)
            ->fetchAll();
    }
}