Edit report at http://bugs.php.net/bug.php?id=53166&edit=1
ID: 53166 Patch added by: rquadl...@php.net Reported by: znarkus at gmail dot com Summary: ZipArchive::addFile() is missing parameters in docs Status: Assigned Type: Bug Package: Zip Related PHP Version: 5.3.3 Assigned To: pajoye Block user comment: N Private report: N New Comment: The following patch has been added/updated: Patch Name: php_zip.c__Reflection__Documentation Revision: 1291730797 URL: http://bugs.php.net/patch-display.php?bug=53166&patch=php_zip.c__Reflection__Documentation&revision=1291730797 Previous Comments: ------------------------------------------------------------------------ [2010-11-18 12:39:53] rquadl...@php.net New patch against 5.3 and trunk. ------------------------------------------------------------------------ [2010-10-27 15:52:34] rquadl...@php.net I've added a patch to this bug which I feel fixes the reflection/prototype issues in php_zip.c (branch/5.3) If it could be reviewed and committed, then I can get the documentation in agreement with this. ------------------------------------------------------------------------ [2010-10-26 18:30:40] paj...@php.net Richard, also please use either the bug tracker or a direct mail, but not both. I will look at these as soon as possible (but toooo soon). ------------------------------------------------------------------------ [2010-10-26 18:25:45] phi...@php.net Assigning to Pierre, he should help investigate this. ------------------------------------------------------------------------ [2010-10-26 18:18:08] rquadl...@php.net There would also be an issue with ZipArchive::setArchiveComment, ZipArchive::getCommentIndex, ZipArchive::getCommentName, ZipArchive::getNameIndex(), ZipArchive::getFromName() and ZipArchive::getFromIndex(). None of these methods documented prototypes tie up with the actual parameters being reflected by the extension. Also, in some cases, the reflection doesn't tie up with the argument parsing in the C code. Whilst, on the surface, this looks like a documentation bug, I think that the underlying code should be fixed and then accurate documentation can be written from that. There are also missing function ZipArchive::addGlob() and ZipArchive::addPattern() are missing. ------------------------------------------------------------------------ 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 http://bugs.php.net/bug.php?id=53166 -- Edit this bug report at http://bugs.php.net/bug.php?id=53166&edit=1
