Fix

vimeo · Jul 25, 2024 · 5b4cba4 · 5b4cba4
1 parent 2846b29
commit 5b4cba4
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/src/Psalm/Internal/PhpVisitor/Reflector/FunctionLikeNodeScanner.php b/src/Psalm/Internal/PhpVisitor/Reflector/FunctionLikeNodeScanner.php
@@ -126,8 +126,7 @@ public function start(
         PhpParser\Node\FunctionLike $stmt,
         bool $fake_method = false,
         PhpParser\Comment\Doc $doc_comment = null
-    )
-    {
+    ) {
         if ($stmt instanceof PhpParser\Node\Expr\Closure
             || $stmt instanceof PhpParser\Node\Expr\ArrowFunction
         ) {

diff --git a/src/Psalm/Plugin/EventHandler/Event/AddRemoveTaintsEvent.php b/src/Psalm/Plugin/EventHandler/Event/AddRemoveTaintsEvent.php
@@ -2,6 +2,7 @@
 
 namespace Psalm\Plugin\EventHandler\Event;
 
+use PhpParser\Node\ArrayItem;
 use PhpParser\Node\Expr;
 use Psalm\Codebase;
 use Psalm\Context;
@@ -17,10 +18,11 @@ final class AddRemoveTaintsEvent
     /**
      * Called after an expression has been checked
      *
+     * @param ArrayItem|Expr $expr
      * @internal
      */
     public function __construct(
-        Expr $expr,
+        $expr,
         Context $context,
         StatementsSource $statements_source,
         Codebase $codebase
@@ -31,7 +33,10 @@ public function __construct(
         $this->codebase = $codebase;
     }
 
-    public function getExpr(): Expr
+    /**
+     * @return ArrayItem|Expr
+     */
+    public function getExpr()
     {
         return $this->expr;
     }

diff --git a/tests/fixtures/SuicidalAutoloader/autoloader.php b/tests/fixtures/SuicidalAutoloader/autoloader.php
@@ -18,6 +18,7 @@
         'PHPUnit\Framework\DOMElement',
         'Stringable',
         'AllowDynamicProperties',
+        'PhpParser\Node\Stmt\Throw', // BCHelper for nikic/php-parser v4/5
 
         // https://github.com/symfony/symfony/pull/40203
         // these are actually functions, referenced as `if (!function_exists(u::class))`