feature #141 fix: Fix the configuration of the extension when DBAL is not configured (theofidry)

This PR was squashed before being merged into the 1.x-dev branch.

Discussion
----------

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

Currently if you happen to not configure DoctrineDBAL, the extension will append the types to the DBAL configuration resulting in an invalid config, which will crash the booting of the Symfony app.

A typical scenario where DBAL may not be configured is if you an an environment with in-memory tests, where DBAL is not used at all.

Commits
-------

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

Author: chalasr

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