Drobotik\Eav\Database\Connection

Complexity

Total Complexity

5

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Test Coverage

Coverage

81.82%

Importance

Changes	3
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	get()	0	10	3
A	reset()	0	3	1
A	getNativeConnection()	0	3	1

<?php
/**
 * This file is part of the eav package.
 * @author    Aleksandr Drobotik <drobotiksbox@gmail.com>
 * @copyright 2023 Aleksandr Drobotik
 * @license   https://opensource.org/license/mit  The MIT License
 */
declare(strict_types=1);

namespace Drobotik\Eav\Database;

use Doctrine\DBAL\Connection as DBALConnection;
use Doctrine\DBAL\DriverManager;
use Drobotik\Eav\Exception\ConnectionException;
use PDO;

class Connection
{
    protected static DBALConnection|null $conn = null;

    public static function reset() : void
    {
        self::$conn = null;
    }

    public static function get(array $params = null) : DBALConnection
    {
        if(!is_null($params)) {
            self::$conn = DriverManager::getConnection($params);
            return self::$conn;
        }
        if(!is_null(self::$conn)) {
            return self::$conn;
        }
        ConnectionException::undefined();

In this branch, the function will implicitly return null which is incompatible with the type-hinted return Doctrine\DBAL\Connection. Consider adding a return statement or allowing null as return value.

    }

    public static function getNativeConnection() : PDO
    {
        return Connection::$conn->getNativeConnection();

The method getNativeConnection() does not exist on null.

The expression return Drobotik\Eav\Database\Connection::conn->getNativeConnection() could return the type resource which is incompatible with the type-hinted return PDO. Consider adding an additional type-check to rule them out.

    }

}