Detect cyclic dependency using reflection abstract factory

src/AbstractFactory/ReflectionBasedAbstractFactory.php

@@ -10,8 +10,11 @@
 use Psr\Container\ContainerInterface;
 use ReflectionClass;
 use ReflectionParameter;
+use ReflectionProperty;
+use Zend\ServiceManager\Exception\InvalidServiceException;
 use Zend\ServiceManager\Exception\ServiceNotFoundException;
 use Zend\ServiceManager\Factory\AbstractFactoryInterface;
+use Zend\ServiceManager\ServiceManager;
 
 use function array_map;
 use function class_exists;
@@ -197,8 +200,10 @@ private function resolveParameterWithConfigService(ContainerInterface $container
      * @param ContainerInterface $container
      * @param string $requestedName
      * @return mixed
+     * @throws \Zend\ServiceManager\Exception\InvalidServiceException
      * @throws ServiceNotFoundException If type-hinted parameter cannot be
      *   resolved to a service in the container.
+     * @throws \ReflectionException
      */
     private function resolveParameter(ReflectionParameter $parameter, ContainerInterface $container, $requestedName)
     {
@@ -223,6 +228,19 @@ private function resolveParameter(ReflectionParameter $parameter, ContainerInter
         $type = $this->aliases[$type] ?? $type;
 
         if ($container->has($type)) {
+            if ($container instanceof ServiceManager) {
+                $aliasProperty = new ReflectionProperty($container, 'aliases');
+                $aliasProperty->setAccessible(true);
+                $serviceManagerAliases = $aliasProperty->getValue($container);
+                $unaliasedService = array_search($requestedName, $serviceManagerAliases, true);
+
+                if (false !== $unaliasedService && $unaliasedService === $type) {
+                    throw new InvalidServiceException(sprintf(
+                        'Unable to create service "%s"; a cyclic dependency was detected',
+                        $unaliasedService
+                    ));
+                }
+            }
             return $container->get($type);
         }
 

test/AbstractFactory/ReflectionBasedAbstractFactoryTest.php

@@ -11,7 +11,10 @@
 use PHPUnit\Framework\TestCase;
 use Psr\Container\ContainerInterface;
 use Zend\ServiceManager\AbstractFactory\ReflectionBasedAbstractFactory;
+use Zend\ServiceManager\Exception\InvalidServiceException;
 use Zend\ServiceManager\Exception\ServiceNotFoundException;
+use Zend\ServiceManager\ServiceManager;
+use ZendTest\ServiceManager\AbstractFactory\TestAsset\SampleInterface;
 
 use function sprintf;
 
@@ -173,4 +176,25 @@ public function testFactoryWillUseDefaultValueForTypeHintedArgument()
         $this->assertInstanceOf(TestAsset\ClassWithTypehintedDefaultValue::class, $instance);
         $this->assertNull($instance->value);
     }
+
+    public function testFactoryThrowsExceptionWhenCyclicDependencyDetectedForDecorators(): void
+    {
+        $serviceManager = new ServiceManager();
+        $serviceManager->setAlias(
+            TestAsset\SampleInterface::class,
+            TestAsset\DecoratorWithCyclicDependencyOnInterface::class
+        );
+        $serviceManager->addAbstractFactory(ReflectionBasedAbstractFactory::class);
+        $serviceManager->setFactory(
+            TestAsset\DecoratorWithCyclicDependencyOnInterface::class,
+            ReflectionBasedAbstractFactory::class
+        );
+
+        $this->expectException(InvalidServiceException::class);
+        $this->expectExceptionMessage(sprintf(
+            'Unable to create service "%s"; a cyclic dependency was detected',
+            TestAsset\SampleInterface::class
+        ));
+        $serviceManager->get(SampleInterface::class);
+    }
 }

test/AbstractFactory/TestAsset/DecoratorWithCyclicDependencyOnInterface.php

@@ -0,0 +1,15 @@
+<?php
+/**
+ * @link      http://github.com/zendframework/zend-servicemanager for the canonical source repository
+ * @copyright Copyright (c) 2016 Zend Technologies USA Inc. (http://www.zend.com)
+ * @license   http://framework.zend.com/license/new-bsd New BSD License
+ */
+
+namespace ZendTest\ServiceManager\AbstractFactory\TestAsset;
+
+final class DecoratorWithCyclicDependencyOnInterface implements SampleInterface
+{
+    public function __construct(SampleInterface $wrapped)
+    {
+    }
+}