Throw on service:method factory notation in PHP-based DI configuration

vudaltsov · vudaltsov · commit 49fc6778ee6c · 2017-11-17T12:52:23.000+03:00
diff --git a/src/Symfony/Component/DependencyInjection/Loader/Configurator/Traits/FactoryTrait.php b/src/Symfony/Component/DependencyInjection/Loader/Configurator/Traits/FactoryTrait.php
@@ -11,6 +11,8 @@
 
 namespace Symfony\Component\DependencyInjection\Loader\Configurator\Traits;
 
+use Symfony\Component\DependencyInjection\Exception\InvalidArgumentException;
+
 trait FactoryTrait
 {
     /**
@@ -22,6 +24,12 @@ trait FactoryTrait
      */
     final public function factory($factory)
     {
+        if (is_string($factory) && 1 === substr_count($factory, ':')) {
+            $factoryParts = explode(':', $factory);
+
+            throw new InvalidArgumentException(sprintf('Invalid factory "%s": the `service:method` notation is not available when using PHP-based DI configuration. Use "[ref(\'%s\'), \'%s\']" instead.', $factory, $factoryParts[0], $factoryParts[1]));
+        }
+
         $this->definition->setFactory(static::processValue($factory, true));
 
         return $this;
diff --git a/src/Symfony/Component/DependencyInjection/Tests/Fixtures/config/factory_short_notation.php b/src/Symfony/Component/DependencyInjection/Tests/Fixtures/config/factory_short_notation.php
@@ -0,0 +1,9 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Loader\Configurator;
+
+return function (ContainerConfigurator $c) {
+    $c->services()
+        ->set('service', \stdClass::class)
+        ->factory('factory:method');
+};
diff --git a/src/Symfony/Component/DependencyInjection/Tests/Loader/PhpFileLoaderTest.php b/src/Symfony/Component/DependencyInjection/Tests/Loader/PhpFileLoaderTest.php
@@ -89,4 +89,17 @@ public function testAutoConfigureAndChildDefinitionNotAllowed()
         $loader->load($fixtures.'/config/services_autoconfigure_with_parent.php');
         $container->compile();
     }
+
+    /**
+     * @expectedException \Symfony\Component\DependencyInjection\Exception\InvalidArgumentException
+     * @expectedExceptionMessage Invalid factory "factory:method": the `service:method` notation is not available when using PHP-based DI configuration. Use "[ref('factory'), 'method']" instead.
+     */
+    public function testFactoryShortNotationNotAllowed()
+    {
+        $fixtures = realpath(__DIR__.'/../Fixtures');
+        $container = new ContainerBuilder();
+        $loader = new PhpFileLoader($container, new FileLocator());
+        $loader->load($fixtures.'/config/factory_short_notation.php');
+        $container->compile();
+    }
 }
