minor symfony#61074 [DependencyInjection] CS fix (nicolas-grekas)

nicolas-grekas · nicolas-grekas · commit 0afec11492a2 · 2025-07-09T09:27:36.000+02:00
This PR was merged into the 7.3 branch. Discussion ---------- [DependencyInjection] CS fix | Q | A | ------------- | --- | Branch? | 7.3 | Bug fix? | no | New feature? | no | Deprecations? | no | Issues | - | License | MIT As spotted by `@stof`, my mistake, only 7.3. Commits ------- 725d861 [DependencyInjection] CS fix
diff --git a/src/Symfony/Component/DependencyInjection/Compiler/ResolveInstanceofConditionalsPass.php b/src/Symfony/Component/DependencyInjection/Compiler/ResolveInstanceofConditionalsPass.php
@@ -112,8 +112,8 @@ private function processDefinition(ContainerBuilder $container, string $id, Defi
                 $definition = substr_replace($definition, '53', 2, 2);
                 $definition = substr_replace($definition, 'Child', 44, 0);
             }
-            $definition = unserialize($definition);
             /** @var ChildDefinition $definition */
+            $definition = unserialize($definition);
             $definition->setParent($parent);
 
             if (null !== $shared && !isset($definition->getChanges()['shared'])) {

Original file line number	Diff line number	Diff line change
`@@ -112,8 +112,8 @@ private function processDefinition(ContainerBuilder $container, string $id, Defi`
`112`	`112`	`$definition = substr_replace($definition, '53', 2, 2);`
`113`	`113`	`$definition = substr_replace($definition, 'Child', 44, 0);`
`114`	`114`	`}`
`115`		`- $definition = unserialize($definition);`
`116`	`115`	`/** @var ChildDefinition $definition */`
	`116`	`+ $definition = unserialize($definition);`
`117`	`117`	`$definition->setParent($parent);`
`118`	`118`
`119`	`119`	`if (null !== $shared && !isset($definition->getChanges()['shared'])) {`