Merge branch 'feature/isreference-bugfix-closure-return-by-reference'…

… of https://github.com/jrfnl/PHP_CodeSniffer
squizlabs · Jun 22, 2020 · 57a67b0 · 57a67b0
2 parents 3a1dc78 + 1ae9566
commit 57a67b0
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/src/Files/File.php b/src/Files/File.php
@@ -1937,6 +1937,7 @@ public function isReference($stackPtr)
         );
 
         if ($this->tokens[$tokenBefore]['code'] === T_FUNCTION
+            || $this->tokens[$tokenBefore]['code'] === T_CLOSURE
             || $this->tokens[$tokenBefore]['code'] === T_FN
         ) {
             // Function returns a reference.

diff --git a/tests/Core/File/IsReferenceTest.inc b/tests/Core/File/IsReferenceTest.inc
@@ -139,3 +139,6 @@ $closure = function() use (&$var){};
 
 /* testArrowFunctionReturnByReference */
 fn&($x) => $x;
+
+/* testClosureReturnByReference */
+$closure = function &($param) use ($value) {};
diff --git a/tests/Core/File/IsReferenceTest.php b/tests/Core/File/IsReferenceTest.php
@@ -228,6 +228,10 @@ public function dataIsReference()
                 '/* testArrowFunctionReturnByReference */',
                 true,
             ],
+            [
+                '/* testClosureReturnByReference */',
+                true,
+            ],
         ];
 
     }//end dataIsReference()