fix: satisfy PHPStan 2.2.3

- Remove redundant output-buffer level checks after ob_start()
- Narrow output-buffer helper return types

Signed-off-by: memleakd <121398829+memleakd@users.noreply.github.com>

Author: memleakd

system/Common.php

@@ -119,7 +119,7 @@ function clean_path(string $path): string
      *
      *  > command('migrate:create SomeMigration');
      *
-     * @return false|string
+     * @return string
      */
     function command(string $command)
     {

tests/system/CommonFunctionsSendTest.php

@@ -58,9 +58,7 @@ public function testRedirectResponseCookiesSent(): void
         // send it
         ob_start();
         $response->send();
-        if (ob_get_level() > 0) {
-            ob_end_clean();
-        }
+        ob_end_clean();
 
         // and what actually got sent?
         $this->assertHeaderEmitted('Set-Cookie: foo=onething;');

tests/system/HTTP/DownloadResponseTest.php

@@ -377,9 +377,7 @@ public function testRealOutput(): void
         // send it
         ob_start();
         $response->send();
-        if (ob_get_level() > 0) {
-            ob_end_clean();
-        }
+        ob_end_clean();
 
         // and what actually got sent?
         $this->assertHeaderEmitted('Content-Length: ' . filesize(__FILE__));

tests/system/HTTP/ResponseSendTest.php

@@ -67,9 +67,7 @@ public function testHeadersMissingDate(): void
         // send it
         ob_start();
         $response->send();
-        if (ob_get_level() > 0) {
-            ob_end_clean();
-        }
+        ob_end_clean();
 
         // and what actually got sent?
         $this->assertHeaderEmitted('Date:');
@@ -102,9 +100,7 @@ public function testHeadersWithCSP(): void
         // send it
         ob_start();
         $response->send();
-        if (ob_get_level() > 0) {
-            ob_end_clean();
-        }
+        ob_end_clean();
 
         // and what actually got sent?; test both ways
         $this->assertHeaderEmitted('Content-Security-Policy:');
@@ -140,9 +136,7 @@ public function testRedirectResponseCookies(): void
         // send it
         ob_start();
         $response->send();
-        if (ob_get_level() > 0) {
-            ob_end_clean();
-        }
+        ob_end_clean();
 
         // and what actually got sent?
         $this->assertHeaderEmitted('Set-Cookie: foo=bar;');
@@ -213,9 +207,7 @@ public function testHeaderOverride(): void
         header('Access-Control-Expose-Headers: Content-Encoding');
         header('Allow: GET, POST');
         $response->send();
-        if (ob_get_level() > 0) {
-            ob_end_clean();
-        }
+        ob_end_clean();
 
         // single header
         $this->assertHeaderEmitted('Vary: Accept-Encoding');

tests/system/Test/BootstrapFCPATHTest.php

@@ -100,7 +100,7 @@ private function fileContents(): string
         return $fileContents . 'echo FCPATH;' . PHP_EOL;
     }
 
-    private function readOutput(string $file): false|string
+    private function readOutput(string $file): string
     {
         ob_start();
         system('php -f ' . $file);

tests/system/Test/TestCaseEmissionsTest.php

@@ -62,9 +62,7 @@ public function testHeadersEmitted(): void
         // send it
         ob_start();
         $response->send();
-        if (ob_get_level() > 0) {
-            ob_end_clean();
-        }
+        ob_end_clean();
 
         // and what actually got sent?; test both ways
         $this->assertHeaderEmitted('Set-Cookie: foo=bar;');
@@ -90,9 +88,7 @@ public function testHeadersNotEmitted(): void
         // send it
         ob_start();
         $response->send(); // what really was sent
-        if (ob_get_level() > 0) {
-            ob_end_clean();
-        }
+        ob_end_clean();
 
         $this->assertHeaderNotEmitted('Set-Cookie: pop=corn', true);
     }