minor symfony#39990 [DoctrineBridge] Fix class resolution in Doctrine EventListenerPass (jderusse)

nicolas-grekas · nicolas-grekas · commit 190f089fee2b · 2021-01-27T09:30:24.000+01:00
This PR was merged into the 4.4 branch. Discussion ---------- [DoctrineBridge] Fix class resolution in Doctrine EventListenerPass | Q | A | ------------- | --- | Branch? | 4.x | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | fix # | License | MIT | Doc PR | - Commits ------- 08b2da7 Fix class resolution in Doctrine EventListenerPass
diff --git a/src/Symfony/Bridge/Doctrine/DependencyInjection/CompilerPass/RegisterEventListenersAndSubscribersPass.php b/src/Symfony/Bridge/Doctrine/DependencyInjection/CompilerPass/RegisterEventListenersAndSubscribersPass.php
@@ -85,7 +85,7 @@ private function addTaggedSubscribers(ContainerBuilder $container, array &$liste
 
                 if (!isset($managerDefs[$con])) {
                     $managerDef = $parentDef = $this->getEventManagerDef($container, $con);
-                    while ($parentDef instanceof ChildDefinition) {
+                    while (!$parentDef->getClass() && $parentDef instanceof ChildDefinition) {
                         $parentDef = $container->findDefinition($parentDef->getParent());
                     }
                     $managerClass = $container->getParameterBag()->resolveValue($parentDef->getClass());

Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ private function addTaggedSubscribers(ContainerBuilder $container, array &$liste`
`85`	`85`
`86`	`86`	`if (!isset($managerDefs[$con])) {`
`87`	`87`	`$managerDef = $parentDef = $this->getEventManagerDef($container, $con);`
`88`		`- while ($parentDef instanceof ChildDefinition) {`
	`88`	`+ while (!$parentDef->getClass() && $parentDef instanceof ChildDefinition) {`
`89`	`89`	`$parentDef = $container->findDefinition($parentDef->getParent());`
`90`	`90`	`}`
`91`	`91`	`$managerClass = $container->getParameterBag()->resolveValue($parentDef->getClass());`