Fix null param to ucfirst() fatal error when DB information is missin…

…g. (#5918)
drush-ops · Mar 24, 2024 · ac2817a · ac2817a
1 parent c7e98c2
commit ac2817a
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/Sql/SqlBase.php b/src/Sql/SqlBase.php
@@ -119,9 +119,9 @@ public static function create(array $options = []): ?SqlBase
     public static function getInstance($db_spec, $options): ?self
     {
         $driver = $db_spec['driver'];
-        $class_name = 'Drush\Sql\Sql' . ucfirst($driver);
-        if (class_exists($class_name)) {
-            $instance = method_exists($class_name, 'make') ? $class_name::make($db_spec, $options) : new $class_name($db_spec, $options);
+        $class_name = !empty($driver) ? 'Drush\Sql\Sql' . ucfirst($driver) : null;
+        if ($class_name && class_exists($class_name)) {
+            $instance = new $class_name($db_spec, $options);
             // Inject config
             $instance->setConfig(Drush::config());
             return $instance;