Skip to content

Commit

Permalink
Fix phpstan v1.6.4
Browse files Browse the repository at this point in the history
  • Loading branch information
mvorisek committed May 5, 2022
1 parent 7ede464 commit 35a1c75
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions src/Persistence/Sql/Connection.php
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ abstract class Connection
/** @var DbalConnection */
protected $connection;

/** @var array<string, string> */
/** @var array<string, class-string<self>> */
protected static $connectionClassRegistry = [
'pdo_sqlite' => Sqlite\Connection::class,
'pdo_mysql' => Mysql\Connection::class,
Expand Down Expand Up @@ -155,6 +155,8 @@ public static function normalizeDsn($dsn, $user = null, $password = null)
*
* Can be used as:
* Connection::registerConnection(MySQL\Connection::class, 'pdo_mysql')
*
* @param class-string<self> $connectionClass
*/
public static function registerConnectionClass(string $connectionClass, string $driverName): void
{
Expand All @@ -163,6 +165,8 @@ public static function registerConnectionClass(string $connectionClass, string $

/**
* Resolves the connection class to use based on driver type.
*
* @return class-string<self>
*/
public static function resolveConnectionClass(string $driverName): string
{
Expand Down Expand Up @@ -206,12 +210,15 @@ public static function connect($dsn, $user = null, $password = null, $args = [])
], $args));
}

/**
* @return 'pdo_sqlite'|'pdo_mysql'|'pdo_pgsql'|'pdo_sqlsrv'|'pdo_oci'|'mysqli'|'oci8'
*/
private static function getDriverNameFromDbalDriverConnection(DbalDriverConnection $connection): string
{
$driver = $connection->getNativeConnection();

if ($driver instanceof \PDO) {
return 'pdo_' . $driver->getAttribute(\PDO::ATTR_DRIVER_NAME);
return 'pdo_' . $driver->getAttribute(\PDO::ATTR_DRIVER_NAME); // @phpstan-ignore-line
} elseif ($driver instanceof \mysqli) {
return 'mysqli';
} elseif (is_resource($driver) && get_resource_type($driver) === 'oci8 connection') {
Expand Down

0 comments on commit 35a1c75

Please sign in to comment.