Bug #74035 [Opn]: getNumberOfRequiredParameters wrong for ReflectionClass::newInstance

Edit report at https://bugs.php.net/bug.php?id=74035&edit=1 ID: 74035 Updated by: carusogabriel@php.net Reported by: fabien dot villepinte at gmail dot com Summary: getNumberOfRequiredParameters wrong for ReflectionClass::newInstance Status: Open Type: Bug Package: Reflection related -PHP Version: 7.1.1 +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: ------------------------------------------------------------------------ [2017-04-24 11:19:12] fabien dot villepinte at gmail dot com I reopen this issue since the fix has been rollbacked (see https://bugs.php.net/bug.php?id=74292 ). ------------------------------------------------------------------------ [2017-02-12 21:04:58] nikic@php.net Automatic comment on behalf of nikita.ppv@gmail.com Revision: http://git.php.net/?p=php-src.git;a=commit;h=eb1373e509a69a48e90e6dd4c2dda1348e19b776 Log: Revert &quot;Fixed bug #74035&quot; ------------------------------------------------------------------------ [2017-02-03 17:30:57] nikic@php.net Automatic comment on behalf of andrew.nester.dev@gmail.com Revision: http://git.php.net/?p=php-src.git;a=commit;h=9ffc6ca62f53431a4b32b30cdda8180142f47cdb Log: Fixed bug #74035 ------------------------------------------------------------------------ [2017-02-03 09:00:40] andrew dot nester dot dev at gmail dot com Just added PR with fix for the issue. ------------------------------------------------------------------------ [2017-02-02 15:36:06] fabien dot villepinte at gmail dot com Description: ------------ The method ReflectionFunctionAbstract::getNumberOfRequiredParameters returns 1 for the method ReflectionClass::newInstance when it can be invoked with no parameter. Furthermore I think the documentation should be updated because it is unclear if a parameter is required or not : http://php.net/manual/en/reflectionclass.newinstance.php FYI, HHVM returns already the right result : https://3v4l.org/RPXQp Test script: --------------- <?php $r = new ReflectionClass(ReflectionClass::class); $m = $r->getMethod('newInstance'); echo $m->getNumberOfRequiredParameters(); // returns 1 instead of 0, KO Expected result: ---------------- 0 Actual result: -------------- 1 ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=74035&edit=1