Bugfix when using enviroment variables and dsn (#43)

* Do not resolve arguments on build

* Bugfix

* cs

Author: Nyholm

DependencyInjection/BMBackupManagerExtension.php

@@ -31,7 +31,6 @@ public function load(array $configs, ContainerBuilder $container)
         $config['storage'] = isset($config['storage']) ? $config['storage'] : [];
         $config['database'] = isset($config['database']) ? $config['database'] : [];
         $this->validateStorage($config['storage']);
-        $config['database'] = $this->parseDsn($config['database']);
 
         $managerIdMap = [
             'Local' => 'backup_manager.filesystems.local_filesystem',
@@ -83,28 +82,4 @@ private function validateStorage(array $config)
             }
         }
     }
-
-    /**
-     * If a DSN is configured, then let it override other database storages.
-     * @param array $config
-     *
-     * @param array
-     */
-    private function parseDsn(array $config)
-    {
-        foreach ($config as $key => $databaseConfig) {
-            if (isset($databaseConfig['dsn'])) {
-                $dsn = new DSN($databaseConfig['dsn']);
-                $config[$key]['type'] = $dsn->getProtocol();
-                $config[$key]['host'] = $dsn->getFirstHost();
-                $config[$key]['port'] = $dsn->getFirstPort();
-                $config[$key]['user'] = $dsn->getUsername();
-                $config[$key]['pass'] = $dsn->getPassword();
-                $config[$key]['database'] = $dsn->getDatabase();
-                unset($config[$key]['dsn']);
-            }
-        }
-
-        return $config;
-    }
 }

DependencyInjection/Configuration.php

@@ -64,10 +64,10 @@ public function getConfigTreeBuilder()
                             $valid = true;
                             foreach ($databases as $name => $d) {
                                 if (isset($d['dsn'])) {
-                                    // "Fake" "type" for now.
-                                    $d['type'] = substr($d['dsn'], 0, strpos($d['dsn'], ':'));
+                                    // We cannot resolve the DSN now. It might be a environment variable.
+                                    continue;
                                 }
-                                if (!isset($d['type'])) {
+                                if (empty($d['type'])) {
                                     throw new InvalidConfigurationException(sprintf('You must define a "type" or "dsn" for database "%s"', $name));
                                 }
                                 if ($d['type'] !== 'mysql') {

Factory/ConfigFactory.php

@@ -0,0 +1,37 @@
+<?php
+
+namespace BM\BackupManagerBundle\Factory;
+
+use BackupManager\Config\Config;
+use Nyholm\DSN;
+
+/**
+ * A factory class to resolve DSN
+ *
+ * @author Tobias Nyholm <tobias.nyholm@gmail.com>
+ */
+class ConfigFactory
+{
+    /**
+     * If a DSN is configured, then let it override other database storages.
+     * @param array $config
+     * @return Config
+     */
+    public static function createConfig(array $config)
+    {
+        foreach ($config as $key => $databaseConfig) {
+            if (isset($databaseConfig['dsn'])) {
+                $dsn = new DSN($databaseConfig['dsn']);
+                $config[$key]['type'] = $dsn->getProtocol();
+                $config[$key]['host'] = $dsn->getFirstHost();
+                $config[$key]['port'] = $dsn->getFirstPort();
+                $config[$key]['user'] = $dsn->getUsername();
+                $config[$key]['pass'] = $dsn->getPassword();
+                $config[$key]['database'] = $dsn->getDatabase();
+                unset($config[$key]['dsn']);
+            }
+        }
+
+        return new Config($config);
+    }
+}

Resources/config/services.yml

@@ -54,6 +54,7 @@ services:
     # Database
     backup_manager.config_database:
         class:  BackupManager\Config\Config
+        factory: 'BM\BackupManagerBundle\Factory\ConfigFactory::createConfig'
         arguments: [[]]
         public: false