Application.php

@@ -111,10 +111,7 @@ public function setDispatcher(EventDispatcherInterface $dispatcher): void
         $this->dispatcher = $dispatcher;
     }
 
-    /**
-     * @return void
-     */
-    public function setCommandLoader(CommandLoaderInterface $commandLoader)
+    public function setCommandLoader(CommandLoaderInterface $commandLoader): void
     {
         $this->commandLoader = $commandLoader;
     }
@@ -128,10 +125,7 @@ public function getSignalRegistry(): SignalRegistry
         return $this->signalRegistry;
     }
 
-    /**
-     * @return void
-     */
-    public function setSignalsToDispatchEvent(int ...$signalsToDispatchEvent)
+    public function setSignalsToDispatchEvent(int ...$signalsToDispatchEvent): void
     {
         $this->signalsToDispatchEvent = $signalsToDispatchEvent;
     }
@@ -224,7 +218,7 @@ public function run(InputInterface $input = null, OutputInterface $output = null
      *
      * @return int 0 if everything went fine, or an error code
      */
-    public function doRun(InputInterface $input, OutputInterface $output)
+    public function doRun(InputInterface $input, OutputInterface $output): int
     {
         if (true === $input->hasParameterOption(['--version', '-V'], true)) {
             $output->writeln($this->getLongVersion());
@@ -327,17 +321,11 @@ public function doRun(InputInterface $input, OutputInterface $output)
         return $exitCode;
     }
 
-    /**
-     * @return void
-     */
-    public function reset()
+    public function reset(): void
     {
     }
 
-    /**
-     * @return void
-     */
-    public function setHelperSet(HelperSet $helperSet)
+    public function setHelperSet(HelperSet $helperSet): void
     {
         $this->helperSet = $helperSet;
     }
@@ -350,10 +338,7 @@ public function getHelperSet(): HelperSet
         return $this->helperSet ??= $this->getDefaultHelperSet();
     }
 
-    /**
-     * @return void
-     */
-    public function setDefinition(InputDefinition $definition)
+    public function setDefinition(InputDefinition $definition): void
     {
         $this->definition = $definition;
     }
@@ -423,10 +408,8 @@ public function areExceptionsCaught(): bool
 
     /**
      * Sets whether to catch exceptions or not during commands execution.
-     *
-     * @return void
      */
-    public function setCatchExceptions(bool $boolean)
+    public function setCatchExceptions(bool $boolean): void
     {
         $this->catchExceptions = $boolean;
     }
@@ -449,10 +432,8 @@ public function isAutoExitEnabled(): bool
 
     /**
      * Sets whether to automatically exit after a command execution or not.
-     *
-     * @return void
      */
-    public function setAutoExit(bool $boolean)
+    public function setAutoExit(bool $boolean): void
     {
         $this->autoExit = $boolean;
     }
@@ -467,10 +448,8 @@ public function getName(): string
 
     /**
      * Sets the application name.
-     *
-     * @return void
      */
-    public function setName(string $name)
+    public function setName(string $name): void
     {
         $this->name = $name;
     }
@@ -485,20 +464,16 @@ public function getVersion(): string
 
     /**
      * Sets the application version.
-     *
-     * @return void
      */
-    public function setVersion(string $version)
+    public function setVersion(string $version): void
     {
         $this->version = $version;
     }
 
     /**
      * Returns the long version of the application.
-     *
-     * @return string
      */
-    public function getLongVersion()
+    public function getLongVersion(): string
     {
         if ('UNKNOWN' !== $this->getName()) {
             if ('UNKNOWN' !== $this->getVersion()) {
@@ -525,10 +500,8 @@ public function register(string $name): Command
      * If a Command is not enabled it will not be added.
      *
      * @param Command[] $commands An array of commands
-     *
-     * @return void
      */
-    public function addCommands(array $commands)
+    public function addCommands(array $commands): void
     {
         foreach ($commands as $command) {
             $this->add($command);
@@ -540,10 +513,8 @@ public function addCommands(array $commands)
      *
      * If a command with the same name already exists, it will be overridden.
      * If the command is not enabled it will not be added.
-     *
-     * @return Command|null
      */
-    public function add(Command $command)
+    public function add(Command $command): ?Command
     {
         $this->init();
 
@@ -576,11 +547,9 @@ public function add(Command $command)
     /**
      * Returns a registered command by name or alias.
      *
-     * @return Command
-     *
      * @throws CommandNotFoundException When given command name does not exist
      */
-    public function get(string $name)
+    public function get(string $name): Command
     {
         $this->init();
 
@@ -683,11 +652,9 @@ public function findNamespace(string $namespace): string
      * Contrary to get, this command tries to find the best
      * match if you give it an abbreviation of a name or alias.
      *
-     * @return Command
-     *
      * @throws CommandNotFoundException When command name is incorrect or ambiguous
      */
-    public function find(string $name)
+    public function find(string $name): Command
     {
         $this->init();
 
@@ -795,7 +762,7 @@ public function find(string $name)
      *
      * @return Command[]
      */
-    public function all(string $namespace = null)
+    public function all(string $namespace = null): array
     {
         $this->init();
 
@@ -936,10 +903,8 @@ protected function doRenderThrowable(\Throwable $e, OutputInterface $output): vo
 
     /**
      * Configures the input and output instances based on the user arguments and options.
-     *
-     * @return void
      */
-    protected function configureIO(InputInterface $input, OutputInterface $output)
+    protected function configureIO(InputInterface $input, OutputInterface $output): void
     {
         if (true === $input->hasParameterOption(['--ansi'], true)) {
             $output->setDecorated(true);
@@ -1004,7 +969,7 @@ protected function configureIO(InputInterface $input, OutputInterface $output)
      *
      * @return int 0 if everything went fine, or an error code
      */
-    protected function doRunCommand(Command $command, InputInterface $input, OutputInterface $output)
+    protected function doRunCommand(Command $command, InputInterface $input, OutputInterface $output): int
     {
         foreach ($command->getHelperSet() as $helper) {
             if ($helper instanceof InputAwareInterface) {
@@ -1038,11 +1003,6 @@ protected function doRunCommand(Command $command, InputInterface $input, OutputI
                         // If the command is signalable, we call the handleSignal() method
                         if (\in_array($signal, $commandSignals, true)) {
                             $exitCode = $command->handleSignal($signal, $exitCode);
-                            // BC layer for Symfony <= 5
-                            if (null === $exitCode) {
-                                trigger_deprecation('symfony/console', '6.3', 'Not returning an exit code from "%s::handleSignal()" is deprecated, return "false" to keep the command running or "0" to exit successfully.', get_debug_type($command));
-                                $exitCode = 0;
-                            }
                         }
 
                         if (false !== $exitCode) {
@@ -1060,14 +1020,7 @@ protected function doRunCommand(Command $command, InputInterface $input, OutputI
 
             foreach ($commandSignals as $signal) {
                 $this->signalRegistry->register($signal, function (int $signal) use ($command): void {
-                    $exitCode = $command->handleSignal($signal);
-                    // BC layer for Symfony <= 5
-                    if (null === $exitCode) {
-                        trigger_deprecation('symfony/console', '6.3', 'Not returning an exit code from "%s::handleSignal()" is deprecated, return "false" to keep the command running or "0" to exit successfully.', get_debug_type($command));
-                        $exitCode = 0;
-                    }
-
-                    if (false !== $exitCode) {
+                    if (false !== $exitCode = $command->handleSignal($signal)) {
                         exit($exitCode);
                     }
                 });

CHANGELOG.md

@@ -1,6 +1,14 @@
 CHANGELOG
 =========
 
+7.0
+---
+
+ * Add method `__toString()` to `InputInterface`
+ * Remove `Command::$defaultName` and `Command::$defaultDescription`, use the `AsCommand` attribute instead
+ * Require explicit argument when calling `*Command::setApplication()`, `*FormatterStyle::setForeground/setBackground()`, `Helper::setHelpSet()`, `Input*::setDefault()` and `Question::setAutocompleterCallback/setValidator()`
+ * Remove `StringInput::REGEX_STRING`
+
 6.4
 ---