Fix class function parameter bug

preprocess · Oct 22, 2018 · c595bba · c595bba
1 parent 2a6f9d6
commit c595bba
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/source/functions.php b/source/functions.php
@@ -4,7 +4,7 @@
 
 use Yay\Ast;
 
-function closure($ast) {
+function compile($ast, $isClosure = true) {
     if (function_exists("\\Amp\\resolve")) {
         $append = new Ast("amp1");
         $append->push(new Ast());
@@ -40,7 +40,7 @@ function closure($ast) {
             continue;
         }
 
-        if (isset($defined[$name]) || isset($pushed[$name])) {
+        if (($isClosure && isset($defined[$name])) || isset($pushed[$name])) {
             continue;
         }
 

diff --git a/source/macros.yay b/source/macros.yay
@@ -41,7 +41,7 @@ $(macro :recursion) {
         _() as amp2
     ))
 } >> function($ast) {
-    return \Pre\Async\closure($ast);
+    return \Pre\Async\compile($ast, $isClosure = false);
 } >> {$$(trim(
     $(scope ? {
         $$(functionModifiers($(functionModifiers))) function $(function)($(functionArguments ... (, ) { $$(functionArgument($(functionArgument))) })): \Amp\Promise
@@ -88,7 +88,7 @@ $(macro :recursion) {
         _() as amp2
     ))
 } >> function($ast) {
-    return \Pre\Async\closure($ast);
+    return \Pre\Async\compile($ast, $isClosure = true);
 } >> {$$(trim(
     $(scope ? {
         [$(scope ... (, ) { $(var) = $(var) ?? null}), "fn" => function ($(functionArguments ... (, ) { $$(functionArgument($(functionArgument))) })) use ($(scope ... (, ) { &$(var) })): \Amp\Promise {

diff --git a/tests/specs/async.spec b/tests/specs/async.spec
@@ -23,9 +23,9 @@ class AsyncClass
         await \Amp\File\get("path/to/file");
     }
 
-    async public function second()
+    async public function second($terminator = "!")
     {
-        return "boo!";
+        return "boo{$terminator}";
     }
 
     async public static function complex(): string
@@ -45,10 +45,10 @@ class AsyncClass
         });
     }
 
-    public function second(): \Amp\Promise
+    public function second($terminator = "!"): \Amp\Promise
     {
-        return \Amp\call(function () {
-            return "boo!";
+        return \Amp\call(function () use (&$terminator) {
+            return "boo{$terminator}";
             yield;
         });
     }