Bug #53856 [Opn->Asn]: zipArchive::addPattern requires path parameter

Edit report at http://bugs.php.net/bug.php?id=53856&edit=1 ID: 53856 Updated by: pajoye@php.net Reported by: rquadling@php.net Summary: zipArchive::addPattern requires path parameter -Status: Open +Status: Assigned Type: Bug Package: Zip Related Operating System: Windows XP SP3 PHP Version: 5.3.5 -Assigned To: +Assigned To: pajoye Block user comment: N Private report: N New Comment: It should be optional, making it mandatory is not correct. Previous Comments: ------------------------------------------------------------------------ [2011-01-27 16:55:46] rquadling@php.net The supplied patch makes the path parameter mandatory. ------------------------------------------------------------------------ [2011-01-27 16:46:04] rquadling@php.net 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