Hi All,

I'm creating a website that creates a zip file and forces user to download of the created file.
I'm having trouble using the header to read the file correctly.

Below is my code:
(Toggle Plain Text)

$zipLocation = "./filesFolder/$pageName.zip";

if(file_exists($zipLocation))
$flag = ZipArchive::OVERWRITE;
else
$flag = ZipArchive::CREATE;

$zip = new ZipArchive();
$zip->open($zipLocation, $flag);

$zip->addFile("$htmlLocation","$pageName.html";
$zip->addFile("$cssLocation","$pageName.css";
$zip->addFile("$jsLocation","$pageName.js";

$zip->close();
$zip = null;

print "zipped<br/>";
//header("location: $zipLocation";

header("Cache-Control: must-revalidate";
header("Content-type: application/force-download";
header("Content-Transfer-Encoding: binary";
header("Content-disposition: attachment; filename=$pageName.zip";
readfile($zipLocation);

The zipfile gets created correctly but when the download dialog appears, the file size is wrong, and when it downloads it cannot be opened.

If I use the commented line it works correctly, but I would rather do it this way using the readfile.

Can any one tell the reason behind it?

Set all headers before using print
Once you use print/echo/print_r etc. any header set thereafter is a joke