Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-2234

PCLZIP under Win32 doesn't work OK


    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 1.4.2
    • Fix Version/s: None
    • Component/s: General
    • Labels:
    • Environment:
    • Affected Branches:



      I've received some reports in http://moodle.org/mod/forum/discuss.php?d=15169 about problems with pclzip under Win32.

      I've tested backups attached there and the problem really exists. Sure!

      Then I've gone to http://www.phpconcept.net/pclzip/index.en.php and I've seen that there is a new (2.3) version. Looking at its changes I've seen that some interesting bugs have been resolved! Cool (perhaps Petr sent his patch to that guys?).

      So I've installed the new library under MacOS X and Win32 and I've done these tests.

      In each OS, both using internal and external zip, I've created a backup file. Total: 4 files (attached to this bug).

      In each OS, both using internal and external unzip, I've restored the four courses. Total: 16 restores executed!

      Results: ALL restores were SUCCESSFUL!!!

      I've unzipped the four courses with a lot of utilities (StuffIt, BuiltIn XP, BuitIn MacOSX) and the files are uncompressed without problems at all!

      So, to finish completely the tests, it would be VERY interesting to make this:

      1.-Test 4 files under a linux OS (using the new pclzip lib, of course).

      2.-Generate 2 more files under linux, one uisng external executables and other using internal pclzip. It would be desirable to export the same course than in the other 4 files.

      3.-Test that 2 new files under Win32 and MacOS X (although something let's me think that they'll work!)

      If all the tests finish OK, my +99999 vote to incorporate pclzip 2.3 to MOODLE_14_STABLE and MOODLE_HEAD.

      It could be the long awaited end of this headache!!


        Gliffy Diagrams




              • Votes:
                0 Vote for this issue
                0 Start watching this issue


                • Created: