ansaurus

Question

Answer 1

+4 A:

humm

faced the exact similar stuff 2 days ago but did it the old fashioned way hope it helps. try this ..

function tester(){

 $this->load->library('upload');  // NOTE: always load the library outside the loop

 foreach($i=0; $i<count($_FILES['filename']['name']); $i++)
 {

   $_FILES['userfile']['name']    = $_FILES['filename']['name'][$i];
   $_FILES['userfile']['type']    = $_FILES['filename']['type'][$i];
   $_FILES['userfile']['tmp_name'] = $_FILES['filename']['tmp_name'][$i];
   $_FILES['userfile']['error']       = $_FILES['filename']['error'][$i];
   $_FILES['userfile']['size']    = $_FILES['filename']['size'][$i];

   $config['file_name']     = 'test_'.$j;
   $config['upload_path']   = './public/uploads/';
   $config['allowed_types'] = 'jpg|jpeg|gif|png';
   $config['max_size']      = '0';
   $config['overwrite']     = FALSE;

  $this->upload->initialize($config);

  if($this->upload->do_upload())
  {
    $error += 0;
  }else{
    $error += 1;
  }
 }

 if($error > 0){ return FALSE; }else{ return TRUE; }

}

that what did it for me, hope it will work for you too :)

thephpx 2009-12-15 16:19:01

this had been giving me a headache all morning, I modded your version for myself and it works great!

Bala Clark 2010-02-24 12:19:33

This is pretty good. There are some syntax errors, but with a bit of tweaking it works.replace foreach with for. replace $j with $i

k00k 2010-07-26 23:18:31

Yes, useful answer. Just not sure what $error += 0 does.

JannieT 2010-07-27 04:54:23

Answer 2

+1 A:

As far as I know, this happens in codeigniter 1.7.2 when encrypt_name preference is set.

Please check if that is the case... and you are using that version...

here is the relevant bug and workaround:
http://codeigniter.com/bug%5Ftracker/bug/9689/

It would help if you posted your controller class code though....

pǝlɐɥʞ 2009-12-15 20:54:26

Answer 3

A:

I am a bit confused by your answer thephpx. Won't restructuring $_FILES like that kill off the subarrays once you ahve run it once?

At any rate, I was trying OP's approach by adding the extra index to the a few of the upload library's lines.

I made it work by putting the for loop (over all files) in the upload controller, and the extra index $k as a parameter in the do_upload function in the library.

      $this->load->library('upload');
  for ($k = 0; $k < count($_FILES['userfile']['name']); $k++) {
   $this->upload->initialize($upload); //must reinitialize to get rid of your bug ( i had it as well)
   if (!$this->upload->do_upload('userfile',$k)) {
    $this->load->view('upload/image_form', $data + array('error'=>$this->upload->display_errors()));
   }
   $udata[$k] = $this->upload->data(); //gradually build up upload->data()
  }

clux 2009-12-25 22:45:19

"Won't restructuring $_FILES like that kill off the subarrays once you ahve run it once?"No. He is only modifying the $_FILES['userfile'] array (not his initial $_FILES['filename'] array. As the do_upload() is being run inside the loop, the $_FILES['userfile'] array being rest every time doesn't matter.

Bala Clark 2010-02-24 12:21:17

Answer 4

+1 A:

Hey,

I solved this exact problem by amending the Upload.php files located in the library folder.

Comment out line 935:

$filename = $this->file_name;

Vishal 2010-03-24 17:14:36

Answer 5

+1 A:

For me it was enough to recall $this->upload->initialize($config); every time I called $this->upload->do_upload()

Mihai MATEI 2010-05-04 11:38:02

This worked for me. For some reason it was retaining the config data from the previous call to do_upload. Calling initialize() forces it to update the config data.

Oliver 2010-06-18 12:51:40

Answer 6

A:

heading derken neyi gastediyon ?

osmanq 2010-07-30 12:06:51

ansaurus

tags:

views:

answers:

codeigniter multiple file upload

heading derken neyi gastediyon ?

related questions