Welcome Guest, Not a member yet? Register   Sign In
CodeIgniter Forums Using CodeIgniter Best Practices Multiple file upload, resize and store in different tables.
Multiple file upload, resize and store in different tables.
  • Offline HarrysR
    Member
  • ***
  • Posts: 87
    Threads: 29
    Joined: Aug 2018
    Reputation: 0
#1
11-04-2018, 05:29 PM

Hello...again!!

I've created this controller where i'm trying to upload multiple image files and i want to store them in two different tables (the first as the main image in "table_1" and the others as extra images in "table_2".

The thing is that i does not store the other ones and now i get an error like "Undefined variable $image" as you will see in the example.
Any ideas on that?!

Controller
Code:
 public function submit(){
   if ($this->session->userdata('logged_in') == true) {
     $user_id = $this->session->userdata('user_id');
     $data['user'] = $this->user_model->get_user_data($user_id);
   }
   $data['categories'] = $this->pet_model->get_pet_category();
   $data['sizes'] = $this->pet_model->get_pet_sizes();

   $this->form_validation->set_rules('petTitle', 'Title, 'required');
   $this->form_validation->set_rules('pet_category', 'Category', 'required');

   if($this->form_validation->run() === FALSE){
     echo validation_errors();
   } else {
     // Single
     $pet_entry_id = random_string('numeric', 6);

     $dir_exist = false; //Directory exist
     if ( !is_dir('assets/img/pets/' . $pet_entry_id) ) {
       mkdir('./assets/img/pets/' . $pet_entry_id, 0777, true);
       $dir_exist = true; // dir not exist
     }

     // Multiple
     if( !empty($_FILES['images']['name']) ){

       $countfiles = count($_FILES['images']);
       for($i=0; $i<1; $i++){
         if(!empty($_FILES['images']['name'][$i])){
           $_FILES['image']['name'] = $_FILES['images']['name'][$i];
           $_FILES['image']['type'] = $_FILES['images']['type'][$i];
           $_FILES['image']['tmp_name'] = $_FILES['images']['tmp_name'][$i];
           $_FILES['image']['error'] = $_FILES['images']['error'][$i];
           $_FILES['image']['size'] = $_FILES['images']['size'][$i];

           $config['upload_path'] = './assets/img/pets/' . $pet_entry_id;
           $config['encrypt_name'] = TRUE;
           $config['allowed_types'] = 'gif|jpg|jpeg|png';
           $config['max_size'] = '1310720';
           $config['max_width'] = '2048';
           $config['max_height'] = '2048';

           $this->load->library('upload', $config);

           if ( $this->upload->do_upload('image') ) {
             $image_data = $this->upload->data();
             $config['image_library'] = 'gd2';
             $config['source_image'] = $image_data['full_path'];
             $config['maintain_ratio'] = TRUE;
             $config['width'] = 650;
             $config['height'] = 650;

             $this->load->library('image_lib', $config);

             $image = $image_data['file_name'];
           }
         }
       }

       for($i=1; $i<$countfiles; $i++){
         if(!empty($_FILES['images']['name'][$i])){
           $_FILES['image']['name'] = $_FILES['images']['name'][$i];
           $_FILES['image']['type'] = $_FILES['images']['type'][$i];
           $_FILES['image']['tmp_name'] = $_FILES['images']['tmp_name'][$i];
           $_FILES['image']['error'] = $_FILES['images']['error'][$i];
           $_FILES['image']['size'] = $_FILES['images']['size'][$i];

           $config['upload_path'] = './assets/img/pets/' . $pet_entry_id;
           $config['encrypt_name'] = TRUE;
           $config['allowed_types'] = 'gif|jpg|jpeg|png';
           $config['max_size'] = '1310720';
           $config['max_width'] = '2048';
           $config['max_height'] = '2048';

           $this->load->library('upload', $config);

           if ( $this->upload->do_upload('image') ) {
             $image_data = $this->upload->data();
             $config['image_library'] = 'gd2';
             $config['source_image'] = $image_data['full_path'];
             $config['maintain_ratio'] = TRUE;
             $config['width'] = 650;
             $config['height'] = 650;

             $this->load->library('image_lib', $config);

             $m_image = $image_data['file_name'];
             $this->pet_model->insert_images($m_image, $pet_entry_id);
           }
         }
       }

     } else {
       if($dir_exist)
       rmdir('./assets/img/pets/' . $pet_entry_id);
       $image = 'default_petavatar.png';
     }

     $this->pet_model->create_pet($image, $pet_entry_id);
     redirect('pets');
   }

   $this->render_page('pets/submit', $data);
 }


File Input
Code:
<div class="row">
 <div class="col-md-12">
 <input class="form-control" type="file" name="images[]" id="images" multiple value="">
 </div>
</div>


"Insert images model
Code:
public function insert_images($m_image, $pet_entry_id){

$data['image_path'] = $m_image;
$data['pet_entry_id_img'] = $pet_entry_id;

return $this->db->insert('pet_images', $data);
}



P.S.: If you have any better (or maybe safer) way on how to achieve this feel free to recommend!!

Thank you!
//Life motto
if (sad() == true) {
     sad().stop();
     develop();
}
Reply
  • Offline jreklund
    Administrator
  • *******
  • Posts: 1,408
    Threads: 3
    Joined: Aug 2017
    Reputation: 42
#2
11-05-2018, 01:08 PM
(This post was last modified: 11-05-2018, 01:10 PM by jreklund.)

Personally I would rewrite the insert_images too and make use of insert_batch instead, so that you don't need to make a foreach loop.
PHP Code:
public function submit(){
   if ($this->session->userdata('logged_in') == true) {
     $user_id = $this->session->userdata('user_id');
     $data['user'] = $this->user_model->get_user_data($user_id);
   }
   $data['categories'] = $this->pet_model->get_pet_category();
   $data['sizes'] = $this->pet_model->get_pet_sizes();

   $this->form_validation->set_rules('petTitle', 'Title', 'required');
   $this->form_validation->set_rules('pet_category', 'Category', 'required');

   if($this->form_validation->run() === FALSE){
     echo validation_errors();
   } else {
     // Single
     $pet_entry_id = random_string('numeric', 6);

     $dir_exist = false; //Directory exist
     if ( !is_dir('assets/img/pets/' . $pet_entry_id) ) {
       mkdir('./assets/img/pets/' . $pet_entry_id, 0777, true);
       $dir_exist = true; // dir not exist
     }

     // Multiple
     $images = array();
     if( !empty($_FILES['images']['name']) ){
       $countfiles = count($_FILES['images']);
       for($i=0; $i<1; $i++){
         if(!empty($_FILES['images']['name'][$i])){
           $_FILES['image']['name'] = $_FILES['images']['name'][$i];
           $_FILES['image']['type'] = $_FILES['images']['type'][$i];
           $_FILES['image']['tmp_name'] = $_FILES['images']['tmp_name'][$i];
           $_FILES['image']['error'] = $_FILES['images']['error'][$i];
           $_FILES['image']['size'] = $_FILES['images']['size'][$i];

           $config['upload_path'] = './assets/img/pets/' . $pet_entry_id;
           $config['encrypt_name'] = TRUE;
           $config['allowed_types'] = 'gif|jpg|jpeg|png';
           $config['max_size'] = '1310720';
           $config['max_width'] = '2048';
           $config['max_height'] = '2048';

           $this->load->library('upload', $config);

           if ( $this->upload->do_upload('image') ) {
             $image_data = $this->upload->data();
             $config['image_library'] = 'gd2';
             $config['source_image'] = $image_data['full_path'];
             $config['maintain_ratio'] = TRUE;
             $config['width'] = 650;
             $config['height'] = 650;

             $this->load->library('image_lib', $config);

             $images[] = $image_data['file_name'];
           }
         }
       }
     } else {
       if($dir_exist)
       rmdir('./assets/img/pets/' . $pet_entry_id);
       $image = 'default_petavatar.png';
     }
     if( ! empty($images) )
     {
         $image = array_shift($images);
         $this->pet_model->create_pet($image, $pet_entry_id);
         if( ! empty($images) )
         {
             foreach($images AS $img)
             {
                $this->pet_model->insert_images($img, $pet_entry_id);
             }
         }
     }

     redirect('pets');
   }

   $this->render_page('pets/submit', $data);
 } 
Reply
  • Offline HarrysR
    Member
  • ***
  • Posts: 87
    Threads: 29
    Joined: Aug 2018
    Reputation: 0
#3
11-11-2018, 03:18 PM
(This post was last modified: 11-11-2018, 03:44 PM by HarrysR.)

(11-05-2018, 01:08 PM)jreklund Wrote: Personally I would rewrite the insert_images too and make use of insert_batch instead, so that you don't need to make a foreach loop.
PHP Code:
public function submit(){
   if ($this->session->userdata('logged_in') == true) {
     $user_id = $this->session->userdata('user_id');
     $data['user'] = $this->user_model->get_user_data($user_id);
   }
   $data['categories'] = $this->pet_model->get_pet_category();
   $data['sizes'] = $this->pet_model->get_pet_sizes();

   $this->form_validation->set_rules('petTitle', 'Title', 'required');
   $this->form_validation->set_rules('pet_category', 'Category', 'required');

   if($this->form_validation->run() === FALSE){
     echo validation_errors();
   } else {
     // Single
     $pet_entry_id = random_string('numeric', 6);

     $dir_exist = false; //Directory exist
     if ( !is_dir('assets/img/pets/' . $pet_entry_id) ) {
       mkdir('./assets/img/pets/' . $pet_entry_id, 0777, true);
       $dir_exist = true; // dir not exist
     }

     // Multiple
     $images = array();
     if( !empty($_FILES['images']['name']) ){
       $countfiles = count($_FILES['images']);
       for($i=0; $i<1; $i++){
         if(!empty($_FILES['images']['name'][$i])){
           $_FILES['image']['name'] = $_FILES['images']['name'][$i];
           $_FILES['image']['type'] = $_FILES['images']['type'][$i];
           $_FILES['image']['tmp_name'] = $_FILES['images']['tmp_name'][$i];
           $_FILES['image']['error'] = $_FILES['images']['error'][$i];
           $_FILES['image']['size'] = $_FILES['images']['size'][$i];

           $config['upload_path'] = './assets/img/pets/' . $pet_entry_id;
           $config['encrypt_name'] = TRUE;
           $config['allowed_types'] = 'gif|jpg|jpeg|png';
           $config['max_size'] = '1310720';
           $config['max_width'] = '2048';
           $config['max_height'] = '2048';

           $this->load->library('upload', $config);

           if ( $this->upload->do_upload('image') ) {
             $image_data = $this->upload->data();
             $config['image_library'] = 'gd2';
             $config['source_image'] = $image_data['full_path'];
             $config['maintain_ratio'] = TRUE;
             $config['width'] = 650;
             $config['height'] = 650;

             $this->load->library('image_lib', $config);

             $images[] = $image_data['file_name'];
           }
         }
       }
     } else {
       if($dir_exist)
       rmdir('./assets/img/pets/' . $pet_entry_id);
       $image = 'default_petavatar.png';
     }
     if( ! empty($images) )
     {
         $image = array_shift($images);
         $this->pet_model->create_pet($image, $pet_entry_id);
         if( ! empty($images) )
         {
             foreach($images AS $img)
             {
                $this->pet_model->insert_images($img, $pet_entry_id);
             }
         }
     }

     redirect('pets');
   }

   $this->render_page('pets/submit', $data);
 } 

Sorry for being late to answer but i was trying to insert it in my code and clean some other parts of the code.
One issue is that while it uploads the first image of the array, the others do not pass in the "foreach" loop and they are not uploaded in db neither in the folder..
Any ideas?!

[EDIT]: I'm just an idiot! I had the number "1" instead of the countfiles number in the "for" loop! It works perfectly! May i ask if you know any good plugins for previewing - deleting - editing images before uploading? I've searched all over the internet and foudn nothing.
//Life motto
if (sad() == true) {
     sad().stop();
     develop();
}
Reply
  • Offline jreklund
    Administrator
  • *******
  • Posts: 1,408
    Threads: 3
    Joined: Aug 2017
    Reputation: 42
#4
11-12-2018, 10:28 AM

I'm afraid I don't have any tips there. But it's possible. You can make a Drag and Drop uploader/previewer in Javascript, you don't need to upload them. But I don't know if it's possible to drop one image and later drop five more and not delete them. It should be possible to store them in a variable.

I'm using this design together with Dropzone JS.
https://stackoverflow.com/a/47178466

But I won't them to be automatically uploaded.
Reply
  • Offline HarrysR
    Member
  • ***
  • Posts: 87
    Threads: 29
    Joined: Aug 2018
    Reputation: 0
#5
11-12-2018, 01:55 PM

(11-12-2018, 10:28 AM)jreklund Wrote: I'm afraid I don't have any tips there. But it's possible. You can make a Drag and Drop uploader/previewer in Javascript, you don't need to upload them. But I don't know if it's possible to drop one image and later drop five more and not delete them. It should be possible to store them in a variable.

I'm using this design together with Dropzone JS.
https://stackoverflow.com/a/47178466

But I won't them to be automatically uploaded.

Thank you very much! I will take a look at it!  Smile
//Life motto
if (sad() == true) {
     sad().stop();
     develop();
}
Reply




Theme © iAndrew 2016 - Forum software by © MyBB