Skip to content

Commit

Permalink
Fix psalm/phan's complaints about the DBAL config
Browse files Browse the repository at this point in the history
  • Loading branch information
dshoreman committed Apr 20, 2021
1 parent cbd77e7 commit 9a7a208
Showing 1 changed file with 11 additions and 6 deletions.
17 changes: 11 additions & 6 deletions app/Database.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
use Doctrine\DBAL\Connection;
use Doctrine\DBAL\DriverManager;
use Doctrine\DBAL\Schema\AbstractSchemaManager;
use Illuminate\Contracts\Config\Repository;

class Database
{
Expand All @@ -16,12 +17,16 @@ private function connection(): Connection
return $this->connection;
}

$this->connection = DriverManager::getConnection([
'user' => config('database.dbal.user'),
'password' => config('database.dbal.password'),
'unix_socket' => config('database.connections.mysql.unix_socket'),
'driver' => 'pdo_mysql',
]);
$config = config();
assert($config instanceof Repository);
$socket = (string) $config->get('database.connections.mysql.unix_socket');

$this->connection = DriverManager::getConnection(
array_merge((array) config('database.dbal'), [
'driver' => 'pdo_mysql',
'unix_socket' => $socket,
]),
);

return $this->connection;
}
Expand Down

0 comments on commit 9a7a208

Please sign in to comment.