Edit report at http://bugs.php.net/bug.php?id=53856&edit=1
ID: 53856 Updated by: [email protected] Reported by: [email protected] Summary: zipArchive::addPattern requires path parameter Status: Open Type: Bug Package: Zip Related Operating System: Windows XP SP3 PHP Version: 5.3.5 Block user comment: N Private report: N New Comment: The supplied patch makes the path parameter mandatory. Previous Comments: ------------------------------------------------------------------------ [2011-01-27 16:46:04] [email protected] Description: ------------ php --rf zipArchive::addPattern tells us that the path is optional. This is confirmed by the parameter parsing. But, at least on windows, if no path is supplied, with a pattern that will include everything, no files are added. Setting the path to the current directory (for example) will include all files in that directory (using the same regex). Test script: --------------- <?php $zip = new ZipArchive; if (True === $zip->open('./TestAddPattern.zip', ZIPARCHIVE::CREATE)) { echo 'Add all files with no path param', PHP_EOL, print_r($zip->addPattern('/.*/'), True), PHP_EOL, 'Number of files : ', $zip->numFiles, PHP_EOL, 'Add all files in current path param', PHP_EOL, print_r($zip->addPattern('/.*/', '.'), True), PHP_EOL, 'Number of files : ', $zip->numFiles, PHP_EOL; $zip->close(); } Expected result: ---------------- Add all files with no path param Array ( [0] => .\25.BAT ... [88] => .\upxit.bat ) Number of files : 89 Add all files in current path param Array ( [0] => .\25.BAT ... [88] => .\upxit.bat ) Number of files : 178 Actual result: -------------- Add all files with no path param Number of files : 0 Add all files in current path param Array ( [0] => .\25.BAT ... [88] => .\upxit.bat ) Number of files : 89 ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/bug.php?id=53856&edit=1
