Edit report at http://bugs.php.net/bug.php?id=53166&edit=1
ID: 53166 Patch added by: [email protected] 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] [email protected] New patch against 5.3 and trunk. ------------------------------------------------------------------------ [2010-10-27 15:52:34] [email protected] 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] [email protected] 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] [email protected] Assigning to Pierre, he should help investigate this. ------------------------------------------------------------------------ [2010-10-26 18:18:08] [email protected] 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
