[Console] do not make the getHelp() method smart

xabbuh · xabbuh · commit 169fad1b0b63 · 2015-09-28T17:40:44.000+02:00
diff --git a/src/Symfony/Component/Console/Command/Command.php b/src/Symfony/Component/Console/Command/Command.php
@@ -491,7 +491,7 @@ public function setHelp($help)
      */
     public function getHelp()
     {
-        return $this->help ?: $this->description;
+        return $this->help;
     }
 
     /**
@@ -513,7 +513,7 @@ public function getProcessedHelp()
             $_SERVER['PHP_SELF'].' '.$name,
         );
 
-        return str_replace($placeholders, $replacements, $this->getHelp());
+        return str_replace($placeholders, $replacements, $this->getHelp() ? $this->getHelp() : $this->getDescription());
     }
 
     /**
diff --git a/src/Symfony/Component/Console/Tests/Command/CommandTest.php b/src/Symfony/Component/Console/Tests/Command/CommandTest.php
@@ -132,7 +132,7 @@ public function testGetSetHelp()
         $this->assertEquals($command, $ret, '->setHelp() implements a fluent interface');
         $this->assertEquals('help1', $command->getHelp(), '->setHelp() sets the help');
         $command->setHelp('');
-        $this->assertEquals('description', $command->getHelp(), '->getHelp() fallback to the description');
+        $this->assertEquals('', $command->getHelp(), '->getHelp() does not fall back to the description');
     }
 
     public function testGetProcessedHelp()

Original file line number	Diff line number	Diff line change
`@@ -491,7 +491,7 @@ public function setHelp($help)`
`491`	`491`	`*/`
`492`	`492`	`public function getHelp()`
`493`	`493`	`{`
`494`		`- return $this->help ?: $this->description;`
	`494`	`+ return $this->help;`
`495`	`495`	`}`
`496`	`496`
`497`	`497`	`/**`
`@@ -513,7 +513,7 @@ public function getProcessedHelp()`
`513`	`513`	`$_SERVER['PHP_SELF'].' '.$name,`
`514`	`514`	`);`
`515`	`515`
`516`		`- return str_replace($placeholders, $replacements, $this->getHelp());`
	`516`	`+ return str_replace($placeholders, $replacements, $this->getHelp() ? $this->getHelp() : $this->getDescription());`
`517`	`517`	`}`
`518`	`518`
`519`	`519`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -132,7 +132,7 @@ public function testGetSetHelp()`
`132`	`132`	`$this->assertEquals($command, $ret, '->setHelp() implements a fluent interface');`
`133`	`133`	`$this->assertEquals('help1', $command->getHelp(), '->setHelp() sets the help');`
`134`	`134`	`$command->setHelp('');`
`135`		`- $this->assertEquals('description', $command->getHelp(), '->getHelp() fallback to the description');`
	`135`	`+ $this->assertEquals('', $command->getHelp(), '->getHelp() does not fall back to the description');`
`136`	`136`	`}`
`137`	`137`
`138`	`138`	`public function testGetProcessedHelp()`