Merge pull request #371 from ikwattro/alias

Configurable producer service alias

Author: skafandri

DependencyInjection/Configuration.php

@@ -90,6 +90,7 @@ protected function addProducers(ArrayNodeDefinition $node)
                             ->scalarNode('auto_setup_fabric')->defaultTrue()->end()
                             ->scalarNode('class')->defaultValue('%old_sound_rabbit_mq.producer.class%')->end()
                             ->scalarNode('enable_logger')->defaultFalse()->end()
+                            ->scalarNode('service_alias')->defaultValue(null)->end()
                         ->end()
                     ->end()
                 ->end()

DependencyInjection/OldSoundRabbitMqExtension.php

@@ -152,7 +152,12 @@ protected function loadProducers()
                     $this->injectLogger($definition);
                 }
 
-                $this->container->setDefinition(sprintf('old_sound_rabbit_mq.%s_producer', $key), $definition);
+                $producerServiceName = sprintf('old_sound_rabbit_mq.%s_producer', $key);
+
+                $this->container->setDefinition($producerServiceName, $definition);
+                if (null !== $producer['service_alias']) {
+                    $this->container->setAlias($producer['service_alias'], $producerServiceName);
+                }
             }
         } else {
             foreach ($this->config['producers'] as $key => $producer) {

README.md

@@ -117,6 +117,7 @@ old_sound_rabbit_mq:
         upload_picture:
             connection:       default
             exchange_options: {name: 'upload-picture', type: direct}
+            service_alias:    my_app_service # no alias by default
     consumers:
         upload_picture:
             connection:       default

Tests/DependencyInjection/Fixtures/test.yml

@@ -62,6 +62,21 @@ old_sound_rabbit_mq:
                 arguments:   null
                 ticket:      null
 
+        foo_producer_aliased:
+            class:           My\Foo\Producer
+            connection:      foo_connection
+            exchange_options:
+                name:        foo_exchange
+                type:        direct
+                passive:     true
+                durable:     false
+                auto_delete: true
+                internal:    true
+                nowait:      true
+                arguments:   null
+                ticket:      null
+            service_alias: foo_producer_alias
+
 
         default_producer:
             exchange_options:

Tests/DependencyInjection/OldSoundRabbitMqExtensionTest.php

@@ -264,6 +264,17 @@ public function testFooProducerDefinition()
         $this->assertEquals('My\Foo\Producer', $definition->getClass());
     }
 
+    /**
+     * @group alias
+     */
+    public function testAliasedFooProducerDefinition()
+    {
+        $container = $this->getContainer('test.yml');
+
+        $this->assertTrue($container->has('old_sound_rabbit_mq.foo_producer_producer'));
+        $this->assertTrue($container->has('foo_producer_alias'));
+    }
+
     public function testDefaultProducerDefinition()
     {
         $container = $this->getContainer('test.yml');