Make the message_bus alias public

sroze · sroze · commit 51b6e9eb96ec · 2018-08-17T14:07:19.000+02:00
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php
@@ -1488,7 +1488,7 @@ private function registerMessengerConfiguration(array $config, ContainerBuilder
             $container->register($busId, MessageBus::class)->addArgument(array())->addTag('messenger.bus');
 
             if ($busId === $config['default_bus']) {
-                $container->setAlias('message_bus', $busId);
+                $container->setAlias('message_bus', $busId)->setPublic(true);
                 $container->setAlias(MessageBusInterface::class, $busId);
             }
         }
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/FrameworkExtensionTest.php b/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/FrameworkExtensionTest.php
@@ -528,6 +528,7 @@ public function testMessenger()
     {
         $container = $this->createContainerFromFile('messenger');
         $this->assertTrue($container->hasAlias('message_bus'));
+        $this->assertTrue($container->getAlias('message_bus')->isPublic());
         $this->assertFalse($container->hasDefinition('messenger.transport.amqp.factory'));
         $this->assertTrue($container->hasDefinition('messenger.transport_factory'));
         $this->assertSame(TransportFactory::class, $container->getDefinition('messenger.transport_factory')->getClass());

Original file line number	Diff line number	Diff line change
`@@ -1488,7 +1488,7 @@ private function registerMessengerConfiguration(array $config, ContainerBuilder`
`1488`	`1488`	`$container->register($busId, MessageBus::class)->addArgument(array())->addTag('messenger.bus');`
`1489`	`1489`
`1490`	`1490`	`if ($busId === $config['default_bus']) {`
`1491`		`- $container->setAlias('message_bus', $busId);`
	`1491`	`+ $container->setAlias('message_bus', $busId)->setPublic(true);`
`1492`	`1492`	`$container->setAlias(MessageBusInterface::class, $busId);`
`1493`	`1493`	`}`
`1494`	`1494`	`}`
Original file line number	Diff line number	Diff line change
`@@ -528,6 +528,7 @@ public function testMessenger()`
`528`	`528`	`{`
`529`	`529`	`$container = $this->createContainerFromFile('messenger');`
`530`	`530`	`$this->assertTrue($container->hasAlias('message_bus'));`
	`531`	`+ $this->assertTrue($container->getAlias('message_bus')->isPublic());`
`531`	`532`	`$this->assertFalse($container->hasDefinition('messenger.transport.amqp.factory'));`
`532`	`533`	`$this->assertTrue($container->hasDefinition('messenger.transport_factory'));`
`533`	`534`	`$this->assertSame(TransportFactory::class, $container->getDefinition('messenger.transport_factory')->getClass());`