Bug #74383 [Opn->Csd]: Wrong reflection on Phar::running

Edit report at https://bugs.php.net/bug.php?id=74383&edit=1 ID: 74383 Updated by: krakjoe@php.net Reported by: fabien dot villepinte at gmail dot com Summary: Wrong reflection on Phar::running -Status: Open +Status: Closed Type: Bug Package: PHAR related PHP Version: Irrelevant -Assigned To: +Assigned To: krakjoe Block user comment: N Private report: N New Comment: The fix for this bug has been committed. Snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. For Windows: http://windows.php.net/snapshots/ Thank you for the report, and for helping us make PHP better. Not sure why this didn't close automatically. Previous Comments: ------------------------------------------------------------------------ [2017-04-06 11:38:36] fabien dot villepinte at gmail dot com Description: ------------ The parameter of Phar::running is optional but it is described as required by reflection. Test script: --------------- <?php $rc = new ReflectionClass(Phar::class); $rm = $rc->getMethod("running"); echo $rm->getNumberOfParameters(); echo PHP_EOL; echo $rm->getNumberOfRequiredParameters(); echo PHP_EOL; echo (int) $rm->getParameters()[0]->isOptional(); Expected result: ---------------- 1 0 1 Actual result: -------------- 1 1 0 ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=74383&edit=1