Edit report at http://bugs.php.net/bug.php?id=53856&edit=1

 ID:                 53856
 Updated by:         paj...@php.net
 Reported by:        rquadl...@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] rquadl...@php.net

The supplied patch makes the path parameter mandatory.

------------------------------------------------------------------------
[2011-01-27 16:46:04] rquadl...@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

Reply via email to