Container: detects circular reference for parameters

Author: dg

src/DI/Container.php

@@ -40,7 +40,7 @@ class Container
 	/** @var object[]  service name => instance */
 	private $instances = [];
 
-	/** @var array circular reference detector */
+	/** @var array<string, true> circular reference detector */
 	private $creating;
 
 	/** @var array<string, string|\Closure> */
@@ -66,7 +66,9 @@ public function getParameters(): array
 	public function getParameter($key)
 	{
 		if (!array_key_exists($key, $this->parameters)) {
-			$this->parameters[$key] = $this->getDynamicParameter($key);
+			$this->parameters[$key] = $this->preventDeadLock("%$key%", function () use ($key) {
+				return $this->getDynamicParameter($key);
+			});
 		}
 		return $this->parameters[$key];
 	}
@@ -219,28 +221,21 @@ public function createService(string $name, array $args = []): object
 	{
 		$name = $this->aliases[$name] ?? $name;
 		$method = self::getMethodName($name);
-		$cb = $this->methods[$method] ?? null;
-		if (isset($this->creating[$name])) {
-			throw new Nette\InvalidStateException(sprintf('Circular reference detected for services: %s.', implode(', ', array_keys($this->creating))));
-
-		} elseif ($cb === null) {
+		$callback = $this->methods[$method] ?? null;
+		if ($callback === null) {
 			throw new MissingServiceException(sprintf("Service '%s' not found.", $name));
 		}
 
-		try {
-			$this->creating[$name] = true;
-			$service = $cb instanceof \Closure
-				? $cb(...$args)
+		$service = $this->preventDeadLock($name, function () use ($callback, $args, $method) {
+			return $callback instanceof \Closure
+				? $callback(...$args)
 				: $this->$method(...$args);
-
-		} finally {
-			unset($this->creating[$name]);
-		}
+		});
 
 		if (!is_object($service)) {
 			throw new Nette\UnexpectedValueException(sprintf(
 				"Unable to create service '$name', value returned by %s is not object.",
-				$cb instanceof \Closure ? 'closure' : "method $method()"
+				$callback instanceof \Closure ? 'closure' : "method $method()"
 			));
 		}
 
@@ -326,6 +321,20 @@ public function findByTag(string $tag): array
 	}
 
 
+	private function preventDeadLock(string $key, \Closure $callback)
+	{
+		if (isset($this->creating[$key])) {
+			throw new Nette\InvalidStateException(sprintf('Circular reference detected for: %s.', implode(', ', array_keys($this->creating))));
+		}
+		try {
+			$this->creating[$key] = true;
+			return $callback();
+		} finally {
+			unset($this->creating[$key]);
+		}
+	}
+
+
 	/********************* autowiring ****************d*g**/
 
 

tests/DI/Compiler.dynamicParameters.phpt

@@ -140,5 +140,19 @@ test('Reference as parameter', function () {
 	');
 	Assert::exception(function () use ($container) {
 		$container->getService('one');
-	}, Nette\InvalidStateException::class, 'Circular reference detected for services: one.');
+	}, Nette\InvalidStateException::class, 'Circular reference detected for: one, %dynamic%.');
+});
+
+
+test('Circula references', function () {
+	$compiler = new DI\Compiler;
+	$compiler->setDynamicParameterNames(['one', 'two']);
+	$container = createContainer($compiler, '
+	parameters:
+		one: %two%
+		two: %one%
+	');
+	Assert::exception(function () use ($container) {
+		$container->getParameter('one');
+	}, Nette\InvalidStateException::class, 'Circular reference detected for: %one%, %two%.');
 });

tests/DI/Container.circular.phpt

@@ -32,4 +32,4 @@ $container = new MyContainer;
 
 Assert::exception(function () use ($container) {
 	$container->getService('one');
-}, Nette\InvalidStateException::class, 'Circular reference detected for services: one, two.');
+}, Nette\InvalidStateException::class, 'Circular reference detected for: one, two.');