Fix #882 - allow STDOUT and STDERR as constants

vimeo · Jul 10, 2018 · 0ef71a4 · 0ef71a4
1 parent 66f6d4b
commit 0ef71a4
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/Psalm/Checker/Statements/Expression/Fetch/ConstFetchChecker.php b/src/Psalm/Checker/Statements/Expression/Fetch/ConstFetchChecker.php
@@ -236,6 +236,13 @@ public static function getGlobalConstType(
         $fq_const_name,
         $const_name
     ) {
+        if ($const_name === 'STDERR'
+            || $const_name === 'STDOUT'
+            || $const_name === 'STDIN'
+        ) {
+            return Type::getResource();
+        }
+
         $predefined_constants = $codebase->config->getPredefinedConstants();
 
         if (isset($predefined_constants[$fq_const_name ?: $const_name])) {

diff --git a/tests/ConstantTest.php b/tests/ConstantTest.php
@@ -215,6 +215,12 @@ class B {
                 '<?php
                     if ("phpdbg" === \PHP_SAPI) {}',
             ],
+            'stdinout' => [
+                '<?php
+                    echo fread(STDIN, 100);
+                    fwrite(STDOUT, "asd");
+                    fwrite(STDERR, "zcx");'
+            ],
         ];
     }