Platform independence issues with the pclzip class

(In [7314]) Chase down some 64bit dragons. Ensure that we only ever create a 32bit number to compare to the 32bit magic number when searching a zip file for the end of the directory record. See #6236.

Commit has fixed the issues on my server setup :)

Cheers for your knowledge westi :P

Replying to DD32:

Commit has fixed the issues on my server setup :)

Cool. I will send this upstream

I'm currently getting this (in r7331):

Fatal error: Call to a member function exists() on a non-object in path.../wp-admin/includes/class-wp-filesystem-ftpsockets.php on line 110

Declaring the $wp_filesystem variable as a global in the find_base_dir() method seems to fix that error. But: 1) The update seems to be slow as molasses on my system, and 2) I'm still getting zero-byte files during the unzip stage.

Are the built-in zip functions being used, when available? My system is PHP 5.2.1 with the ZIP extension installed.

And I still think this either needs to use AJAX, or we should put some flush() calls at various key points of the upgrade. There needs to be some sort of user feedback for slow updates, or users are going to be punching the "stop" button in their browsers, thinking that something has stalled.

(In [7334]) Fix var name. see #6236

(In [7335]) Fix var name. see #6236

Flushing can be added to show_message() in wp-admin/update.php

Are the built-in zip functions being used, when available? My system is PHP 5.2.1 with the ZIP extension installed.

Not yet, And probably not in this release, but i'd like it to be used if available, but not many people have it enabled.

And I still think this either needs to use AJAX, or we should put some flush() calls at various key points of the upgrade. There needs to be some sort of user feedback for slow updates, or users are going to be punching the "stop" button in their browsers, thinking that something has stalled.

If the webserver has an output buffer running, then theres nothing that can force it to flush out the page until its fully loaded, I've come accross it on Dreamhost, even adding some flushes didnt help.

However, On other servers without a forced output buffer, then the messages are shown as they happen.

Ajax wouldnt be possible in this situation, It'd simply make the output appear in one hit, rather than in small chunks. Someone suggested using HTTP Continues to load the page with one message, then in the same script execution, load the next "page" (but its the same page just with extra messages), But i've not a clue how to do that.

2) I'm still getting zero-byte files during the unzip stage.

Looking at the debug output you sent me, and its writing files correctly, Or at least the test file is, But aparantly not extracting properly for some reason.. i'll send back some small changes and see what that gives.

Closing this as the original issue was fixed.

Please raise new tickets for any new issues.