stdClass on controller

[eluser]Sein Kraft[/eluser]

Fatal error: Cannot use object of type stdClass as array in E:\Server\htdocs\KL6G56PD8P\system\application\controllers\gallery.php on line 112

I get this error from the next controller.

Code:

function submit()
    {
    $config = $this->m_core->load_config();    

    $this->form_validation->set_rules('title', 'Title', 'trim|required|min_length[1]|max_length[25]');
    $this->form_validation->set_rules('description', 'Description', 'trim|required|min_length[1]');
    
    $this->form_validation->set_error_delimiters('<div class="error">', '</div>');
    
    if ($this->form_validation->run() == FALSE)
        {
        $config = $this->m_core->load_config();
    
        $data = array(
           'head_title' => $config->c_name,
           'head_slogan' => $config->c_slogan
                    );
        $this->load->view('page/header', $data);
        
        $this->load->view('gallery/submit');
        $this->load->view('page/sidebar');
    
    $latest_users = $this->m_core->latests_users();
    $latest_journals = $this->m_core->latests_journals();
    $latest_images = $this->m_core->latests_images();
    
    $attributes_u = array(
                
                'id'    => 'latests_users_list'
                );
    $attributes_j = array(
                
                'id'    => 'latests_journals_list'
                );
    $attributes_i = array(
                
                'id'    => 'latests_images_list'
                );

    $data = array(
                'latest_users' => ul($latest_users, $attributes_u),
                'latest_journals' => ul($latest_journals, $attributes_j),
                'latest_images' => ul($latest_images, $attributes_i)
                );
    
    $this->load->view('page/footer', $data);
        }
        else
        {
    switch ($_POST['type'])
            {
                case "Image":
                $config['upload_path'] = './files/temp/';
                $config['allowed_types'] = 'gif|jpg|png';
                $config['destination_path'] = './files/original/';
                $config['resized_path'] = './files/images/';
                $config['thumb_path'] = './files/thumbs/';
                
                $config['max_size']    = '1024';                
                break;
                
                case "Text":
                $config['upload_path'] = './files/temp/';
                $config['allowed_types'] = 'txt|rtf';
                $config['destination_path'] = './files/text/';
                
                $config['max_size']    = '1024';
                break;
                
                case "Audio":
                $config['upload_path'] = './files/temp/';
                $config['allowed_types'] = 'mp3';
                $config['destination_path'] = './files/audio/';
                
                $config['max_size']    = '10.240';
                break;
            }
            
    $config['overwrite'] = TRUE;
    $config['max_width']  = '2000';
    $config['max_height']  = '2000';
            
    $this->load->library('upload', $config);
            
    $this->upload->do_upload();
    
    $data = array('upload_data' => $this->upload->data());
    $upload_data = $this->upload->data();
    
    $user = $this->m_user->get_user($this->session->userdata('s_id'));
    
    $file_data['id'] = strtolower(random_string('alpha', 10));
    $file_data['user'] = $user->u_id;
    $file_data['realname'] = $upload_data['file_name'];
    
    $file_data['newname'] = $file_data['user'].'.'.$file_data['id'].'.'.underscore($_POST['title']);
    $file_data['ext'] = $upload_data['file_ext'];
    
    $temp_file = $config['upload_path'].$file_data['realname'];
    $new_file = $config['destination_path'].$file_data['newname'].$file_data['ext'];
    
    copy($temp_file , $new_file);
    unlink($temp_file);
    
    switch ($_POST['type'])
        {
            case "Image":
            //START CREATION OF RESIZED IMAGE
            $config['image_library'] = 'gd2';
            $config['source_image'] = $new_file;
            $config['new_image'] = $config['resized_path'].$file_data['newname'].$file_data['ext'];
            $config['maintain_ratio'] = TRUE;
            $config['width'] = 500;
            $config['height'] = 500;
            
            $this->load->library('image_lib', $config);
            
            $this->image_lib->resize();
            $this->image_lib->clear();
            //ENDS CREATION OF RESIZED IMAGE
            
            //START CREATION OF THUMBNAIL
            $config_thumb['image_library'] = 'gd2';
            $config_thumb['source_image'] = $new_file;
            $config_thumb['new_image'] = $config['thumb_path'].$file_data['newname'].$file_data['ext'];
            $config_thumb['maintain_ratio'] = TRUE;
            $config_thumb['width'] = 150;
            $config_thumb['height'] = 150;
            
            $this->image_lib->initialize($config_thumb);
            
            $this->image_lib->resize();
            $this->image_lib->clear();
            //ENDS CREATION OF THUMBNAIL
            break;
    }
            
    $this->load->model('m_gallery','', TRUE);
    $this->m_gallery->set_file($file_data);

    redirect('gallery/submission/'.$file_data['id']);
    }
    }

The line 112 is

Code:

$config['upload_path'] = './files/temp/';

Btw what do you think about the code?

[eluser]TheFuzzy0ne[/eluser]
The method you're calling on right at the top is returning an object, and you are trying to use it as an array.

You're accessing the $_POST array directly, which is advised against.

You call on $this->m_core->load_config() again when validation fails, which shouldn't be necessary.

Your code at first glance also seems to lack error checking, but I'm not entirely sure if it's necessary.

Other than that, you're code looks CI-exy.

[eluser]Sein Kraft[/eluser]
But it's not the $_POST who cause the error is the

Code:

$config['image_library'] = 'gd2';
$config['source_image'] = $new_file;
$config['new_image'] = $config['resized_path'].$file_data['newname'].$file_data['ext'];
$config['maintain_ratio'] = TRUE;
$config['width'] = 500;
$config['height'] = 500;
...

What do you mean with CI-exy?

[eluser]pistolPete[/eluser]
Rename the $config array and it will work:

Code:

switch ($_POST['type'])
{
    case "Image":
    $upload_config['upload_path'] = './files/temp/';
    ...
}

...

switch ($_POST['type'])
{
     case "Image":
    //START CREATION OF RESIZED IMAGE
     $resize_config['image_library'] = 'gd2';
    ...
}

Some additional suggestions:

1.) The second switch statement should be transformed into an if statement:

Code:

if($this->input->post('type') == 'Image')
{
   ...
}

2.) You can avoid copying/unlinking the file just to rename it:
Get the latest CI version from SVN and use this setting:

Code:

$upload_config['file_name'] = 'put_your_new_filename.here'

More information on that can be found here: forums/viewthread/98622/

[eluser]TheFuzzy0ne[/eluser]
The first sentence was addressing your problem, the rest were addressing your request "Btw what do you think about the code?".

CI-exy = sexy, but in a CI way.

[eluser]Sein Kraft[/eluser]
Thanks a lot you two.

Its working fine

Btw i've the latest version of CI but its less code with copying/unlinking.