zip help

[steviez]

Hi,

 

I am trying to zip up user files from a posted form, the form posts the file ids and then my script query's the database to get the file name. The only problem is that it only adds one file to the zip archive. Here is the code:

 

<?php

if(isset($_POST['file_option']) && $_POST['file_option'] == 'zip')
{
    $files_array = $_POST['image_file'];

 // Move image
 foreach ($files_array as $f) 
    {
        $r = sql_row("SELECT * FROM iv_uploads WHERE id = '".$f."'");
	$filess = './uploads/' . $r['image_name'];
	$files = array($filess);
	create_zip($files, $username . '_images.zip', false);
	header("Pragma: public");
                header("Expires: 0");
                header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
                header("Content-Type: application/force-download"); 
	header("Content-type: application/octet-stream");
	header("Content-Type: application/download"); 
                header("Content-disposition: attachment; filename=". $username . "_images.zip");
	header("Content-Transfer-Encoding: binary");
                header("Content-Length: ".filesize($username . "_images.zip"));
                readfile($username . "_images.zip");
}

?>

 

any ideas?

[Masna]

Why are you putting everything in a loop? All of your header declarations should be outside of the loop. The only thing in the loop should be the readfile for each file (and whatever else you need to add each file to the .zip).

[steviez]

Why are you putting everything in a loop? All of your header declarations should be outside of the loop. The only thing in the loop should be the readfile for each file (and whatever else you need to add each file to the .zip).

 

my mistake, moved them out of the loop and it still dont work...

[Masna]

Why are you putting everything in a loop? All of your header declarations should be outside of the loop. The only thing in the loop should be the readfile for each file (and whatever else you need to add each file to the .zip).

 

my mistake, moved them out of the loop and it still dont work...

 

Post your new code.

[steviez]

<?php

if(isset($_POST['file_option']) && $_POST['file_option'] == 'zip')
{
    $files_array = $_POST['image_file'];
   
    // Move image
    foreach ($files_array as $f) 
       {
           $r = sql_row("SELECT * FROM iv_uploads WHERE id = '".$f."'");
      $filess = './uploads/' . $r['image_name'];
      $files = array($filess);
}

create_zip($files, $username . '_images.zip', false);
      header("Pragma: public");
                header("Expires: 0");
                header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
                header("Content-Type: application/force-download"); 
      header("Content-type: application/octet-stream");
      header("Content-Type: application/download"); 
                header("Content-disposition: attachment; filename=". $username . "_images.zip");
      header("Content-Transfer-Encoding: binary");
                header("Content-Length: ".filesize($username . "_images.zip"));
                readfile($username . "_images.zip");

?>

[Masna]

Can we see the code for this mysterious create_zip function?