Downloading multiple files using PHP ZipArchive

Question

I am trying to randomly download some files.

I apologize because I posted it earlier but can someone explain what I am doing wrong in detail?

I can not seem to debug it as I know minimal php.

<?php 
$rootPath = realpath('./uploads');
//make zip file
$zip = new ZipArchive();
$zip->open('file.zip', ZipArchive::CREATE | ZipArchive::OVERWRITE);
$nfiles = glob($rootPath.'*.{aiff}', GLOB_BRACE);     
$files = array_rand($nfiles, 20);
foreach($files as $file)  {
    $pathtofile = $nfiles[$file];
    if (!$file->isDir())
    {
        // Get path for current file
        $filePath = $pathtofile->getRealPath();
        $relativePath = str_replace($rootPath, "", $file);
        // Add file to zip
        $zip->addFile($filePath, $relativePath);
    }
}
//-------------------------------------------//
$zip->close();
header('Content-Type: application/zip');
header('Content-disposition: attachment;
filename='.'generatedsounds.zip');
header('Content-Length: ' . filesize('file.zip'));
readfile('file.zip');
if(file_exists('file.zip'))
{
    unlink('file.zip');
}
?>

what is the problem? whats not working? what errors do you get? — cmorrissey, Nov 30 '15 at 16:35
What is your problem? What happens when you run the code? *Does* it run? — Hexaholic, Nov 30 '15 at 16:36
What I did is something such as this: `Download file`. Worked for me :) — Can O' Spam, Nov 30 '15 at 16:52
it just did this http://samplepackgenerator.com/phDownloader.php — Nick Garver, Nov 30 '15 at 17:45
Improved code formatting, improved tags, and improved title. — samlev, Dec 02 '15 at 16:51
The error that I'm seeing there is that you are running `Varnish` in front of your web site. Because you are trying to generate a new, random collection of files each time, you should tell varnish to 'pass' for that URL (i.e. don't attempt to cache the response). — samlev, Dec 02 '15 at 16:54

score 0 · Answer 1 · answered Dec 02 '15 at 16:59

You're calling ->isDir() method which is not defined, And you're calling ->getRealPath() which is also not defined.

And you're getting all files then selecting 20 random files from the array (why??)

While you are rewriting the headers, errors won't be shown in browser, try without rewriting headers which means it's kept as html header and test until you get a binary file output then add the headers back so you have a working code.

Downloading multiple files using PHP ZipArchive

1 Answers1