no issue - fix configuration regression when 'connections' is being u…

…sed as a string
makinacorpus · Nov 21, 2024 · 782cc63 · 782cc63
1 parent 101205e
commit 782cc63
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/docs/content/getting-started/installation.md b/docs/content/getting-started/installation.md
@@ -58,7 +58,6 @@ Then, copy the default configuration file from the vendor directory:
 ```sh
 cd your_project_dir
 cp vendor/makinacorpus/db-tools-bundle/config/db_tools.standalone.sample.yaml db_tools.config.yaml
-cp config/anonymizations.sample.yaml db_tools.anonymization.yaml
 ```
 
 Update these files to your needs. The only required parameter is `connections` in which you

diff --git a/src/Bridge/Symfony/DependencyInjection/DbToolsConfiguration.php b/src/Bridge/Symfony/DependencyInjection/DbToolsConfiguration.php
@@ -117,14 +117,16 @@ public function getConfigTreeBuilder(): TreeBuilder
         $connectionsNode = $rootNode
             ->children()
                 ->arrayNode('connections')
+                    ->beforeNormalization()->ifString()->then(function ($v) { return ['default' => $v]; })->end()
                     ->useAttributeAsKey('name')
                     ->arrayPrototype()
-                    ->children()
-                        ->scalarNode('url')
-                            ->defaultNull()
+                        ->beforeNormalization()->ifString()->then(function ($v) { return ['url' => $v]; })->end()
+                        ->children()
+                            ->scalarNode('url')
+                                ->defaultNull()
+                            ->end()
                         ->end()
-                    ->end()
-                    // Do not close arrayNode() we use it below.
+                        // Do not close arrayNode() we use it below.
         ;
         \assert($connectionsNode instanceof ArrayNodeDefinition);
         $this->addConnectionConfigTreeBuilder($connectionsNode);