Jun 27, 2024 · Jun 27, 2024 · Jun 27, 2024
diff --git a/src/Symfony/Component/DependencyInjection/Dumper/PhpDumper.php b/src/Symfony/Component/DependencyInjection/Dumper/PhpDumper.php
        $name = $this->getNextVariableName();
        $this->referenceVariables[$targetId] = new Variable($name);

        $reference = ContainerInterface::EXCEPTION_ON_INVALID_REFERENCE>= $behavior ? new Reference($targetId, $behavior) : null;
        $reference = ContainerInterface::EXCEPTION_ON_INVALID_REFERENCE!== $behavior ? new Reference($targetId, $behavior) : null;
        $code .= sprintf("        \$%s = %s;\n", $name, $this->getServiceCall($targetId, $reference));

        if (!$hasSelfRef || !$forConstructor) {
Original file line number	Diff line number	Diff line change
Expand Up		@@ -1009,7 +1009,7 @@ private function addInlineReference(string $id, Definition $definition, string $
		$name = $this->getNextVariableName();
		$this->referenceVariables[$targetId] = new Variable($name);

		$reference = ContainerInterface::EXCEPTION_ON_INVALID_REFERENCE>= $behavior ? new Reference($targetId, $behavior) : null;
		$reference = ContainerInterface::EXCEPTION_ON_INVALID_REFERENCE!== $behavior ? new Reference($targetId, $behavior) : null;
Copy link ContributorAuthor vtsykunJun 27, 2024 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Perhaps, there was just a typo here and the author meant`$behavior >= ContainerInterface::EXCEPTION_ON_INVALID_REFERENCE`. The bug was hidden for a long time because it can only be reproduced for "synthetic" services like`kernel` and in specific cases. But the other services replaced by null on optimization step before
		$code .= sprintf(" \$%s = %s;\n", $name, $this->getServiceCall($targetId, $reference));

		if (!$hasSelfRef \|\| !$forConstructor) {
Expand Down