return null when message with name is not set

xabbuh · xabbuh · commit 31b87204dcd7 · 2024-03-29T21:46:22.000+01:00
diff --git a/src/Symfony/Component/Console/Helper/ProgressBar.php b/src/Symfony/Component/Console/Helper/ProgressBar.php
@@ -171,7 +171,7 @@ public function setMessage(string $message, string $name = 'message')
 
     public function getMessage(string $name = 'message')
     {
-        return $this->messages[$name];
+        return $this->messages[$name] ?? null;
     }
 
     public function getStartTime(): int
diff --git a/src/Symfony/Component/Console/Tests/Helper/ProgressBarTest.php b/src/Symfony/Component/Console/Tests/Helper/ProgressBarTest.php
@@ -1173,4 +1173,11 @@ public function testMultiLineFormatIsFullyCorrectlyWithManuallyCleanup()
             stream_get_contents($output->getStream())
         );
     }
+
+    public function testGetNotSetMessage()
+    {
+        $progressBar = new ProgressBar($this->getOutputStream());
+
+        $this->assertNull($progressBar->getMessage());
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -171,7 +171,7 @@ public function setMessage(string $message, string $name = 'message')`
`171`	`171`
`172`	`172`	`public function getMessage(string $name = 'message')`
`173`	`173`	`{`
`174`		`- return $this->messages[$name];`
	`174`	`+ return $this->messages[$name] ?? null;`
`175`	`175`	`}`
`176`	`176`
`177`	`177`	`public function getStartTime(): int`
Original file line number	Diff line number	Diff line change
`@@ -1173,4 +1173,11 @@ public function testMultiLineFormatIsFullyCorrectlyWithManuallyCleanup()`
`1173`	`1173`	`stream_get_contents($output->getStream())`
`1174`	`1174`	`);`
`1175`	`1175`	`}`
	`1176`	`+`
	`1177`	`+ public function testGetNotSetMessage()`
	`1178`	`+ {`
	`1179`	`+ $progressBar = new ProgressBar($this->getOutputStream());`
	`1180`	`+`
	`1181`	`+ $this->assertNull($progressBar->getMessage());`
	`1182`	`+ }`
`1176`	`1183`	`}`