[DependencyInjection] Fix ternary in AutowireCallable attribute

alamirault · alamirault · commit f4792bcbf014 · 2024-06-13T19:55:40.000+02:00
diff --git a/src/Symfony/Component/DependencyInjection/Attribute/AutowireCallable.php b/src/Symfony/Component/DependencyInjection/Attribute/AutowireCallable.php
@@ -42,7 +42,7 @@ public function __construct(
 
     public function buildDefinition(mixed $value, ?string $type, \ReflectionParameter $parameter): Definition
     {
-        return (new Definition($type = \is_string($this->lazy) ? $this->lazy : ($type ?: 'Closure')))
+        return (new Definition($type = \is_array($this->lazy) ? current($this->lazy) : ($type ?: 'Closure')))
             ->setFactory(['Closure', 'fromCallable'])
             ->setArguments([\is_array($value) ? $value + [1 => '__invoke'] : $value])
             ->setLazy($this->lazy || 'Closure' !== $type && 'callable' !== (string) $parameter->getType());

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ public function __construct(`
`42`	`42`
`43`	`43`	`public function buildDefinition(mixed $value, ?string $type, \ReflectionParameter $parameter): Definition`
`44`	`44`	`{`
`45`		`- return (new Definition($type = \is_string($this->lazy) ? $this->lazy : ($type ?: 'Closure')))`
	`45`	`+ return (new Definition($type = \is_array($this->lazy) ? current($this->lazy) : ($type ?: 'Closure')))`
`46`	`46`	`->setFactory(['Closure', 'fromCallable'])`
`47`	`47`	`->setArguments([\is_array($value) ? $value + [1 => '__invoke'] : $value])`
`48`	`48`	`->setLazy($this->lazy \|\| 'Closure' !== $type && 'callable' !== (string) $parameter->getType());`