fix: Fix the configuration of the extension when DBAL is not configured

Author: theofidry

src/DependencyInjection/LeagueOAuth2ServerExtension.php

@@ -95,6 +95,13 @@ public function getAlias(): string
      */
     public function prepend(ContainerBuilder $container)
     {
+        // If no doctrine connection is configured, the DBAL connection should
+        // be left alone as adding any configuration setting with no connection
+        // will result in an invalid configuration leading to a hard failure.
+        if (!self::hasDoctrineConnectionsConfigured($container->getExtensionConfig('doctrine'))) {
+            return;
+        }
+
         $container->prependExtensionConfig('doctrine', [
             'dbal' => [
                 'connections' => null,
@@ -115,6 +122,20 @@ public function process(ContainerBuilder $container)
         $this->assertRequiredBundlesAreEnabled($container);
     }
 
+    /**
+     * @param mixed[] $configs
+     */
+    private static function hasDoctrineConnectionsConfigured(array $configs): bool
+    {
+        foreach ($configs as $config) {
+            if (isset($config['dbal'])) {
+                return true;
+            }
+        }
+
+        return false;
+    }
+
     private function assertRequiredBundlesAreEnabled(ContainerBuilder $container): void
     {
         $requiredBundles = [
@@ -254,7 +275,7 @@ private function configurePersistence(LoaderInterface $loader, ContainerBuilder
         }
 
         $persistenceConfig = current($config['persistence']);
-        $persistenceMethod = key($config['persistence']);
+        $persistenceMethod = $this->getPersistenceMethod($config);
 
         switch ($persistenceMethod) {
             case 'in_memory':
@@ -271,6 +292,16 @@ private function configurePersistence(LoaderInterface $loader, ContainerBuilder
         }
     }
 
+    /**
+     * @param mixed[] $config
+     */
+    private function getPersistenceMethod(array $config): ?string
+    {
+        $persistenceMethod = key($config['persistence']);
+
+        return \is_string($persistenceMethod) ? $persistenceMethod : null;
+    }
+
     /**
      * @param mixed[] $config
      * @param mixed[] $persistenceConfig