symfony · chalasr · Jan 11, 2025 · Jan 9, 2025
diff --git a/UPGRADE-7.3.md b/UPGRADE-7.3.md
@@ -8,6 +8,28 @@ Read more about this in the [Symfony documentation](https://symfony.com/doc/7.3/

 If you're upgrading from a version below 7.1, follow the [7.2 upgrade guide](UPGRADE-7.2.md) first.

+Console
+-------
+
+ * Omitting parameter types in callables configured via `Command::setCode` method is deprecated
+
+   *Before*
+   ```php
+   $command->setCode(function ($input, $output) {
+       // ...
+   });
+   ```
+
+   *After*
+   ```php
+   use Symfony\Component\Console\Input\InputInterface;
+   use Symfony\Component\Console\Output\OutputInterface;
+
+   $command->setCode(function (InputInterface $input, OutputInterface $output) {
+       // ...
+   });
+   ```
+
 FrameworkBundle
 ---------------


@@ -4,8 +4,8 @@ CHANGELOG
 7.3
 ---

-* Add support for invokable commands
-* Add `#[Argument]` and `#[Option]` attributes to define input arguments and options for invokable commands
+ * Add support for invokable commands and add `#[Argument]` and `#[Option]` attributes to define input arguments and options
+ * Deprecate not declaring the parameter type in callable commands defined through `setCode` method

 7.2
 ---

@@ -328,7 +328,7 @@ public function setCode(callable $code): static
            $code = $code(...);
        }

-        $this->code = new InvokableCommand($this, $code);
+        $this->code = new InvokableCommand($this, $code, triggerDeprecations: true);

        return $this;
    }

@@ -35,6 +35,7 @@ class InvokableCommand
    public function __construct(
        private readonly Command $command,
        private readonly \Closure $code,
+        private readonly bool $triggerDeprecations = false,
    ) {
        $this->reflection = new \ReflectionFunction($code);
    }
@@ -47,10 +48,13 @@ public function __invoke(InputInterface $input, OutputInterface $output): int
        $statusCode = ($this->code)(...$this->getParameters($input, $output));

        if (null !== $statusCode && !\is_int($statusCode)) {
-            // throw new LogicException(\sprintf('The command "%s" must return either void or an integer value in the "%s" method, but "%s" was returned.', $this->command->getName(), $this->reflection->getName(), get_debug_type($statusCode)));
-            trigger_deprecation('symfony/console', '7.3', \sprintf('Returning a non-integer value from the command "%s" is deprecated and will throw an exception in PHP 8.0.', $this->command->getName()));
+            if ($this->triggerDeprecations) {
+                trigger_deprecation('symfony/console', '7.3', \sprintf('Returning a non-integer value from the command "%s" is deprecated and will throw an exception in PHP 8.0.', $this->command->getName()));

-            return 0;
+                return 0;
+            }
+
+            throw new LogicException(\sprintf('The command "%s" must return either void or an integer value in the "%s" method, but "%s" was returned.', $this->command->getName(), $this->reflection->getName(), get_debug_type($statusCode)));
        }

        return $statusCode ?? 0;
@@ -92,10 +96,13 @@ private function getParameters(InputInterface $input, OutputInterface $output):
            $type = $parameter->getType();

            if (!$type instanceof \ReflectionNamedType) {
-                // throw new LogicException(\sprintf('The parameter "$%s" must have a named type. Untyped, Union or Intersection types are not supported.', $parameter->getName()));
-                trigger_deprecation('symfony/console', '7.3', \sprintf('Omitting the type declaration for the parameter "$%s" is deprecated and will throw an exception in PHP 8.0.', $parameter->getName()));
+                if ($this->triggerDeprecations) {
+                    trigger_deprecation('symfony/console', '7.3', \sprintf('Omitting the type declaration for the parameter "$%s" is deprecated and will throw an exception in PHP 8.0.', $parameter->getName()));

-                continue;
+                    continue;
+                }
+
+                throw new LogicException(\sprintf('The parameter "$%s" must have a named type. Untyped, Union or Intersection types are not supported.', $parameter->getName()));
            }

            $parameters[] = match ($type->getName()) {