diff options
| author | Nikita Popov <nikita.ppv@gmail.com> | 2018-10-02 18:19:53 +0200 |
|---|---|---|
| committer | Nikita Popov <nikita.ppv@gmail.com> | 2018-10-02 18:19:53 +0200 |
| commit | f0647edd88ca2a2d4e15b4019a3694ef1dbe33f9 (patch) | |
| tree | 09ba2154e490506b959fd0abeac8d8c3103b3b82 | |
| parent | d2477b284b73adb12fb9694d144855a468fe8825 (diff) | |
| download | php-git-f0647edd88ca2a2d4e15b4019a3694ef1dbe33f9.tar.gz | |
Fixed bug #66430
| -rw-r--r-- | NEWS | 2 | ||||
| -rw-r--r-- | ext/reflection/php_reflection.c | 10 | ||||
| -rw-r--r-- | ext/reflection/tests/bug66430.phpt | 42 |
3 files changed, 54 insertions, 0 deletions
@@ -10,6 +10,8 @@ PHP NEWS - Reflection: . Fixed bug #76936 (Objects cannot access their private attributes while handling reflection errors). (Nikita) + . Fixed bug #66430 (ReflectionFunction::invoke does not invoke closure with + object scope). (Nikita) 11 Oct 2018, PHP 7.2.11 diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c index 3bd31e48a8..03ae873aa2 100644 --- a/ext/reflection/php_reflection.c +++ b/ext/reflection/php_reflection.c @@ -1897,6 +1897,11 @@ ZEND_METHOD(reflection_function, invoke) fcc.called_scope = NULL; fcc.object = NULL; + if (!Z_ISUNDEF(intern->obj)) { + Z_OBJ_HT(intern->obj)->get_closure( + &intern->obj, &fcc.called_scope, &fcc.function_handler, &fcc.object); + } + result = zend_call_function(&fci, &fcc); if (result == FAILURE) { @@ -1958,6 +1963,11 @@ ZEND_METHOD(reflection_function, invokeArgs) fcc.called_scope = NULL; fcc.object = NULL; + if (!Z_ISUNDEF(intern->obj)) { + Z_OBJ_HT(intern->obj)->get_closure( + &intern->obj, &fcc.called_scope, &fcc.function_handler, &fcc.object); + } + result = zend_call_function(&fci, &fcc); for (i = 0; i < argc; i++) { diff --git a/ext/reflection/tests/bug66430.phpt b/ext/reflection/tests/bug66430.phpt new file mode 100644 index 0000000000..6e35a0389e --- /dev/null +++ b/ext/reflection/tests/bug66430.phpt @@ -0,0 +1,42 @@ +--TEST-- +Bug #66430: ReflectionFunction::invoke does not invoke closure with object scope +--FILE-- +<?php + +class Alpha { + public $message = "Valid representation"; + + public function bravo() { + return $this->message; + } +} + +$alpha = new Alpha(); + +echo "alpha.bravo: ", $alpha->bravo().PHP_EOL; + +$reflection = new ReflectionObject($alpha); + +$method = $reflection->getMethod("bravo"); +$closure = $method->getClosure($alpha); + +$reflectionC = new ReflectionFunction($closure); + +echo "reflection of alpha.bravo: ", $method->invoke($alpha).PHP_EOL; +echo "closure of alpha.bravo: ", $closure().PHP_EOL; +echo "call_user_func of closure: ", call_user_func($closure).PHP_EOL; +echo PHP_EOL; +echo "closure cl of c(alpha.bravo): ", get_class($reflectionC->getClosureThis()).PHP_EOL; +echo "scope cl of c(alpha.bravo): ", $reflectionC->getClosureScopeClass()->getName().PHP_EOL; +echo "reflection of c(alpha.bravo): ", $reflectionC->invoke().PHP_EOL; + +?> +--EXPECT-- +alpha.bravo: Valid representation +reflection of alpha.bravo: Valid representation +closure of alpha.bravo: Valid representation +call_user_func of closure: Valid representation + +closure cl of c(alpha.bravo): Alpha +scope cl of c(alpha.bravo): Alpha +reflection of c(alpha.bravo): Valid representation |