symfony
diff --git a/‎src/Symfony/Component/Console/DependencyInjection/AddConsoleCommandPass.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Console/DependencyInjection/AddConsoleCommandPass.php
+1-1Lines changed: 1 addition & 1 deletion b/‎src/Symfony/Component/Console/DependencyInjection/AddConsoleCommandPass.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Console/DependencyInjection/AddConsoleCommandPass.php
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Symfony/Component/Console/Tests/DependencyInjection/AddConsoleCommandPassTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Console/Tests/DependencyInjection/AddConsoleCommandPassTest.php
+29Lines changed: 29 additions & 0 deletions b/‎src/Symfony/Component/Console/Tests/DependencyInjection/AddConsoleCommandPassTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Console/Tests/DependencyInjection/AddConsoleCommandPassTest.php
+29Lines changed: 29 additions & 0 deletions
@@ -55,7 +55,7 @@ public function process(ContainerBuilder $container)
                 if (!$r->isSubclassOf(Command::class)) {
                     throw new InvalidArgumentException(sprintf('The service "%s" tagged "%s" must be a subclass of "%s".', $id, $this->commandTag, Command::class));
                 }
-                $commandName = $class::getDefaultName();
+                $commandName = $class::getDefaultName() !== null ? str_replace('%', '%%', $class::getDefaultName()) : null;
             }
 
             if (null === $commandName) {
 
@@ -118,6 +118,30 @@ public function visibilityProvider()
         ];
     }
 
+    public function testEscapesDefaultFromPhp()
+    {
+        $container = new ContainerBuilder();
+        $container
+            ->register('to-escape', EscapedDefaultsFromPhpCommand::class)
+            ->addTag('console.command')
+        ;
+
+        $pass = new AddConsoleCommandPass();
+        $pass->process($container);
+
+        $commandLoader = $container->getDefinition('console.command_loader');
+        $commandLocator = $container->getDefinition((string) $commandLoader->getArgument(0));
+
+        $this->assertSame(ContainerCommandLoader::class, $commandLoader->getClass());
+        $this->assertSame(['%%cmd%%' => 'to-escape'], $commandLoader->getArgument(1));
+        $this->assertEquals([['to-escape' => new ServiceClosureArgument(new TypedReference('to-escape', EscapedDefaultsFromPhpCommand::class))]], $commandLocator->getArguments());
+        $this->assertSame([], $container->getParameter('console.command.ids'));
+
+        $command = $container->get('console.command_loader')->get('%%cmd%%');
+
+        $this->assertSame('%cmd%', $command->getName());
+    }
+
     public function testProcessThrowAnExceptionIfTheServiceIsAbstract()
     {
         $this->expectException(\InvalidArgumentException::class);
@@ -250,3 +274,8 @@ class NamedCommand extends Command
 {
     protected static $defaultName = 'default';
 }
+
+class EscapedDefaultsFromPhpCommand extends Command
+{
+    protected static $defaultName = '%cmd%';
+}
Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ public function process(ContainerBuilder $container)`
`55`	`55`	`if (!$r->isSubclassOf(Command::class)) {`
`56`	`56`	`throw new InvalidArgumentException(sprintf('The service "%s" tagged "%s" must be a subclass of "%s".', $id, $this->commandTag, Command::class));`
`57`	`57`	`}`
`58`		`- $commandName = $class::getDefaultName();`
	`58`	`+ $commandName = $class::getDefaultName() !== null ? str_replace('%', '%%', $class::getDefaultName()) : null;`
`59`	`59`	`}`
`60`	`60`
`61`	`61`	`if (null === $commandName) {`