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

From: Date: Fri, 03 Feb 2017 09:00:41 +0000
Subject: Bug #74035 [Com]: getNumberOfRequiredParameters wrong for ReflectionClass::newInstance
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-207145@lists.php.net to get a copy of this message 
Edit report at https://bugs.php.net/bug.php?id=74035&edit=1

 ID:                 74035
 Comment by:         andrew dot nester dot dev at gmail dot com
 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
 Block user comment: N
 Private report:     N

 New Comment:

Just added PR with fix for the issue.


Previous Comments:
------------------------------------------------------------------------
[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

Thread (7 messages)

« previous php.bugs (#207145) next »