TL-46723 scssphp: Don't close a stream that failed to open

Author: codyfinegan

.github/workflows/ci.yml

@@ -13,7 +13,7 @@ jobs:
         strategy:
             fail-fast: false
             matrix:
-                php: [ '8.1', '8.2' ]
+                php: [ '8.2' ]
                 extensions: [ '' ]
                 os: [ubuntu-latest]
                 include:
@@ -58,48 +58,3 @@ jobs:
 
             -   name: Run tests
                 run: vendor/bin/phpunit --colors=always
-
-    static_analysis:
-        name: Static analysis
-        runs-on: ubuntu-latest
-
-        steps:
-            -   name: Checkout
-                uses: actions/checkout@v4
-
-            -   name: Setup PHP
-                uses: shivammathur/setup-php@v2
-                with:
-                    coverage: "none"
-                    php-version: "8.2"
-                    ini-file: development
-
-            -   name: Install dependencies
-                run: composer update --ansi --no-progress
-
-            -   name: Install phpstan
-                run: composer bin phpstan update --ansi --no-progress
-
-            -   name: Run phpstan
-                run: vendor-bin/phpstan/vendor/bin/phpstan analyse --ansi --no-progress
-
-    coding_styles:
-      name: Coding styles
-      runs-on: ubuntu-latest
-
-      steps:
-        -   name: Checkout
-            uses: actions/checkout@v4
-
-        -   name: Setup PHP
-            uses: shivammathur/setup-php@v2
-            with:
-              coverage: "none"
-              php-version: "8.2"
-              ini-file: development
-
-        -   name: Install dependencies
-            run: composer update --ansi --no-progress
-
-        -   name: Run phpcs
-            run: vendor/bin/phpcs --extensions=php bin src tests *.php

src/Logger/StreamLogger.php

@@ -37,7 +37,7 @@ public function __construct($stream, $closeOnDestruct = false)
      */
     public function __destruct()
     {
-        if ($this->closeOnDestruct) {
+        if ($this->closeOnDestruct && $this->stream) {
             fclose($this->stream);
         }
     }
@@ -47,6 +47,10 @@ public function __destruct()
      */
     public function warn($message, $deprecation = false)
     {
+        if (!$this->stream) {
+            return;
+        }
+
         $prefix = ($deprecation ? 'DEPRECATION ' : '') . 'WARNING: ';
 
         fwrite($this->stream, $prefix . $message . "\n\n");
@@ -57,6 +61,10 @@ public function warn($message, $deprecation = false)
      */
     public function debug($message)
     {
+        if (!$this->stream) {
+            return;
+        }
+
         fwrite($this->stream, $message . "\n");
     }
 }