bug #32970 [Messenger] Don't deep-merge senders configuration (chalasr)

This PR was merged into the 4.4 branch. Discussion ---------- [Messenger] Don't deep-merge senders configuration | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #32953 | License | MIT | Doc PR | - Commits ------- 20d19ac [Messenger] Don't deep merge routing.senders config option
symfony · Sep 16, 2019 · 7f81cb5 · 7f81cb5
2 parents e82fdbf + 20d19ac
commit 7f81cb5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md b/src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md
@@ -16,7 +16,8 @@ CHANGELOG
  * Overriding the methods `KernelTestCase::tearDown()` and `WebTestCase::tearDown()` without the `void` return-type is deprecated.
  * Added new `error_controller` configuration to handle system exceptions
  * Added sort option for `translation:update` command.
-
+ * [BC Break] The `framework.messenger.routing.senders` config key is not deep merged anymore.
+
 4.3.0
 -----
 

diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php
@@ -1162,6 +1162,7 @@ function ($a) {
                                 })
                             ->end()
                             ->prototype('array')
+                                ->performNoDeepMerging()
                                 ->children()
                                     ->arrayNode('senders')
                                         ->requiresAtLeastOneElement()