BUGFIX: Handle empty stdout when formatting logs

Sebobo · Sebobo · commit 7d576fe6e93b · 2024-04-24T09:58:44.000+02:00
diff --git a/Classes/Eel/ModuleHelper.php b/Classes/Eel/ModuleHelper.php
@@ -12,8 +12,12 @@
  */
 class ModuleHelper implements ProtectedContextAwareInterface
 {
-    public function formatStdOutput(string $stdOut): string
+    public function formatStdOutput(?string $stdOut): string
     {
+        if (!$stdOut) {
+            return '';
+        }
+
         $escapedString = $stdOut;
         $lines = array_reverse(array_filter(preg_split("/\r\n|\n|\r/", $escapedString)));
         $lineCount = count($lines);

Original file line number	Diff line number	Diff line change
`@@ -12,8 +12,12 @@`
`12`	`12`	`*/`
`13`	`13`	`class ModuleHelper implements ProtectedContextAwareInterface`
`14`	`14`	`{`
`15`		`- public function formatStdOutput(string $stdOut): string`
	`15`	`+ public function formatStdOutput(?string $stdOut): string`
`16`	`16`	`{`
	`17`	`+ if (!$stdOut) {`
	`18`	`+ return '';`
	`19`	`+ }`
	`20`	`+`
`17`	`21`	`$escapedString = $stdOut;`
`18`	`22`	`$lines = array_reverse(array_filter(preg_split("/\r\n\|\n\|\r/", $escapedString)));`
`19`	`23`	`$lineCount = count($lines);`