Escape synopsis output

jschaedl · jschaedl · commit 96ed5b821a63 · 2021-05-23T17:28:21.000+02:00
diff --git a/src/Symfony/Component/Console/Application.php b/src/Symfony/Component/Console/Application.php
@@ -833,7 +833,7 @@ public function renderThrowable(\Throwable $e, OutputInterface $output): void
     private function finishRenderThrowableOrException(OutputInterface $output): void
     {
         if (null !== $this->runningCommand) {
-            $output->writeln(sprintf('<info>%s</info>', sprintf($this->runningCommand->getSynopsis(), $this->getName())), OutputInterface::VERBOSITY_QUIET);
+            $output->writeln(sprintf('<info>%s</info>', OutputFormatter::escape(sprintf($this->runningCommand->getSynopsis(), $this->getName()))), OutputInterface::VERBOSITY_QUIET);
             $output->writeln('', OutputInterface::VERBOSITY_QUIET);
         }
     }
diff --git a/src/Symfony/Component/Console/Tests/Command/CommandTest.php b/src/Symfony/Component/Console/Tests/Command/CommandTest.php
@@ -192,7 +192,8 @@ public function testGetSynopsis()
         $command = new \TestCommand();
         $command->addOption('foo');
         $command->addArgument('bar');
-        $this->assertEquals('namespace:name [--foo] [--] [<bar>]', $command->getSynopsis(), '->getSynopsis() returns the synopsis');
+        $command->addArgument('info');
+        $this->assertEquals('namespace:name [--foo] [--] [<bar> [<info>]]', $command->getSynopsis(), '->getSynopsis() returns the synopsis');
     }
 
     public function testAddGetUsages()

Original file line number	Diff line number	Diff line change
`@@ -833,7 +833,7 @@ public function renderThrowable(\Throwable $e, OutputInterface $output): void`
`833`	`833`	`private function finishRenderThrowableOrException(OutputInterface $output): void`
`834`	`834`	`{`
`835`	`835`	`if (null !== $this->runningCommand) {`
`836`		`- $output->writeln(sprintf('<info>%s</info>', sprintf($this->runningCommand->getSynopsis(), $this->getName())), OutputInterface::VERBOSITY_QUIET);`
	`836`	`+ $output->writeln(sprintf('<info>%s</info>', OutputFormatter::escape(sprintf($this->runningCommand->getSynopsis(), $this->getName()))), OutputInterface::VERBOSITY_QUIET);`
`837`	`837`	`$output->writeln('', OutputInterface::VERBOSITY_QUIET);`
`838`	`838`	`}`
`839`	`839`	`}`
Original file line number	Diff line number	Diff line change
`@@ -192,7 +192,8 @@ public function testGetSynopsis()`
`192`	`192`	`$command = new \TestCommand();`
`193`	`193`	`$command->addOption('foo');`
`194`	`194`	`$command->addArgument('bar');`
`195`		`- $this->assertEquals('namespace:name [--foo] [--] [<bar>]', $command->getSynopsis(), '->getSynopsis() returns the synopsis');`
	`195`	`+ $command->addArgument('info');`
	`196`	`+ $this->assertEquals('namespace:name [--foo] [--] [<bar> [<info>]]', $command->getSynopsis(), '->getSynopsis() returns the synopsis');`
`196`	`197`	`}`
`197`	`198`
`198`	`199`	`public function testAddGetUsages()`