Bug #71416 [Ana]: Reflection for ReflectionMethod::invoke considers second parameter as required

Edit report at https://bugs.php.net/bug.php?id=71416&edit=1 ID: 71416 Updated by: carusogabriel@php.net Reported by: ondrej at mirtes dot cz Summary: Reflection for ReflectionMethod::invoke considers second parameter as required Status: Analyzed Type: Bug Package: Reflection related Operating System: all -PHP Version: Irrelevant +PHP Version: Next major version -Assigned To: +Assigned To: carusogabriel Block user comment: N Private report: N New Comment: This is only taking place in our next major version, as discussed in https://github.com/php/php-src/pull/3443 Previous Comments: ------------------------------------------------------------------------ [2018-02-23 18:58:53] cmb@php.net Related To: Bug #74421 ------------------------------------------------------------------------ [2017-04-14 09:26:44] cmb@php.net The arginfo[1] is obviously wrong. We need ZEND_BEGIN_ARG_INFO_EX() here. [1] <https://github.com/php/php-src/blob/php-7.1.4/ext/reflection/php_reflection.c#L6421-L6424> ------------------------------------------------------------------------ [2016-12-22 17:51:01] rquadling@php.net Reopening. ------------------------------------------------------------------------ [2016-12-22 17:46:43] rquadling@php.net The reflection of the ReflectionMethod::invoke is certainly NOT in keeping with the documentation. Running: php --rc ReflectionMethod and then looking at the invoke() method shows ... Method [ <internal:Reflection> public method invoke ] { - Parameters [2] { Parameter #0 [ <required> $object ] Parameter #1 [ <required> $args ] } } Not HHVM. Not any PHP script. Just PHP. ------------------------------------------------------------------------ [2016-02-03 08:15:18] ondrej at mirtes dot cz I am not talking about HHVM at all. I am comparing PHP documentation and the current production behaviour to what PHP reflection reports. I haven't mentioned HHVM even once. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=71416 -- Edit this bug report at https://bugs.php.net/bug.php?id=71416&edit=1