Fixed determining the driver of the database

Author: freekmurze

CHANGELOG.md

@@ -2,6 +2,9 @@
 
 All Notable changes to `laravel-backup` will be documented in this file
 
+###2.8.1
+- Fixed determining the driver of the database.
+
 ###2.8.0
 - The temp backup file will now be explicitly deleted.
 

src/BackupHandlers/Database/DatabaseBackupHandler.php

@@ -17,21 +17,23 @@ public function __construct(DatabaseBuilder $databaseBuilder)
     /**
      * Get database configuration.
      *
-     * @param string $database
+     * @param string $connectionName
      *
      * @return mixed
      *
      * @throws Exception
      */
-    public function getDatabase($database = '')
+    public function getDatabase($connectionName = '')
     {
-        $database = $database ?: config('database.default');
+        $connectionName = $connectionName ?: config('database.default');
 
-        if ($database != 'mysql') {
+        $dbDriver = config("database.connections.{$connectionName}.driver");
+
+        if ($dbDriver != 'mysql') {
             throw new Exception('laravel-backup can only backup mysql databases');
         }
 
-        return $this->databaseBuilder->getDatabase(config('database.connections.'.$database));
+        return $this->databaseBuilder->getDatabase(config("database.connections.{$connectionName}"));
     }
 
     public function getDumpedDatabase()

tests/MySQLDatabaseTest.php

@@ -15,7 +15,13 @@ public function setUp()
         $this->console = m::mock('Spatie\Backup\Console');
 
         $this->database = new MySQLDatabase(
-            $this->console, 'testDatabase', 'testUser', 'password', 'localhost', '3306', '/var/run/mysqld/mysqld.sock'
+            $this->console,
+            'testDatabase',
+            'testUser',
+            'password',
+            'localhost',
+            '3306',
+            '/var/run/mysqld/mysqld.sock'
         );
     }