Consider using the PHP ZIP extension to extract Zip's if available

[dd32]

WordPress should use the PHP Zip extension if available to decompress zip files, It should be more efficient than a PHP-based uncompress routine..

​http://au.php.net/manual/en/intro.zip.php

[peaceablewhale]

+1. The extension is automatically installed in PHP Windows releases since 5.3.0.

[dd32]

If someone else doesnt implement this, i'll get it done before next release.. I've got other uses for that code :)

[peaceablewhale]

Then leaving this to you should be the best. The "needs-patch" keyboard is no longer needed :)

[Denis-de-Bernardy]

see also #10913

[dd32]

I've just written up a version which uses the ZipArchive class (Only available with the PHP Zip extension loaded).

When running locally on my system(extracting the wordpress 2.9.1 package via WP_Filesystem_Direct), the output of ZipArchive and PclZip appear to match exactly.

Both take approximately the same ammount of time locally.

ZipArchive uses on peak average, 2.25MB of memory. PclZip uses on peak average 11.70MB of memory (approximately the same size as the Compressed file & uncompressed files).

Code used for testing:

WP_Filesystem( array('hostname' => '', 'username' => '', 'password' => '') );

add_filter('unzip_file_use_ziparchive', 'return_true');
function return_false() { return false; } // Use PclZip
function return_true() { return true; } // Use ZipArchive

timer_start();

var_dump( memory_get_peak_usage(true) );

var_dump( unzip_file(ABSPATH . 'wordpress-2.9.1.zip', ABSPATH . 'test' . time()) )

timer_stop(1, 5);

var_dump( memory_get_peak_usage(true) );

[dd32]

(In [13005]) Introduce ZipArchive version of unzip_file(), More efficient on memory usage for supporting hosts. See #10403

[dd32]

(In [13006]) Better error checking for ZipArchive extraction. See #10403

[dd32]

(In [13007]) PHPdoc for _unzip_file_pclzip & _unzip_file_ziparchive. See #10403

[westi]

(In [13015]) Make _unzip_file_ziparchive() PHP4 safe. See #10403.

[dd32]

Closing a fixed. Open new tickets if any issues arrise from this.