Error: Call to a member function save() on a non-object

Question

public function upload ($data = null) {
    foreach ($data as $file) {
        $filename = $file['name'];
        $file_tmp_name = $file['tmp_name'];
        $dir = WWW_ROOT.'img'.DS.'uploads';
        $allowed = array('png', 'jpg', 'jpeg');

        if (!in_array(substr(strrchr($filename, '.'),1) , $allowed)) {
            throw new NotFoundException("Error Processing Request", 1);
        }
        elseif (is_uploaded_file($file_tmp_name)) {
            $new_filename = DS.STring::uuid().'-'.$filename;
            move_uploaded_file($file_tmp_name,$dir.$new_filename);

            if ($this->Upload->save($this->request->data)) {
            `$this->Session->setFlash('Data Saved!');
            }
        }
    }
}

And I got this error..

Call to a member function save() on a non-object

How to can I fix this?

Where is this function? inside a controller? if that, what is the name of controller? — Mohsenme, Nov 28 '14 at 22:12

Vivek S · Answer 1 · 2014-11-28T13:44:07.613

1

$model = ClassRegistry::init('Upload');

then use

 $model->save();

use

if (ClassRegistry::init('Upload')->save($this->request->data)) {

            }

edited Nov 28 '14 at 13:44

answered Nov 28 '14 at 13:23

Vivek S

2,010
1
13
15

Call to undefined method UploadComponent::loadModel() – Otori Takeo Nov 28 '14 at 13:26
is this a component action? – Vivek S Nov 28 '14 at 13:28
Yes, sorry I didn't mention it earlier. – Otori Takeo Nov 28 '14 at 13:29
Where will I put these? Sorry.. Newbie :/ – Otori Takeo Nov 28 '14 at 13:35
Call to a member function setFlash() on a non-object – Otori Takeo Nov 28 '14 at 14:30
return true . and use it in your controller. this is the proper cakephp way. – Vivek S Nov 28 '14 at 16:09

score 1 · Answer 2 · edited May 23 '17 at 11:57

1

Why didn't you pluralized the name of the controller and its filename (and also the table name), i.e. UploadsControllerl? It's a CakePHP convention:

Controller class names are plural, CamelCased, and end in Controller. PeopleController and LatestArticlesController are both examples of conventional controller names.

Edit: I thought your method was in a controller class. In a component, before using a model you need to register it as follows:

$model = ClassRegistry::init('Yourmodel');

so you should use:

$model = ClassRegistry::init('Upload');
$model->save($this->request->data);

see: Use a model in a component

edited May 23 '17 at 11:57

Community

1
1

answered Nov 29 '14 at 11:18

Mohsenme

1,012
10
20

That is not a controller class. It is a component. – Otori Takeo Nov 30 '14 at 04:38
Trying to get property of non-object – Otori Takeo Dec 04 '14 at 06:47
Are you sure the error is related to these lines? Did you change "$this->Upload->save()" with "$model->save()"? [ClassRegistry::init](http://api.cakephp.org/2.2/class-ClassRegistry.html#_init) is exactly for this purpose unless you don't have an Upload model class. – Mohsenme Dec 04 '14 at 10:48
I have a model, but this is the only code I use in model.. – Otori Takeo Dec 06 '14 at 07:06
The codes I used is from here so you can see the contents of the codes.. https://www.youtube.com/watch?v=scZeqDLxeqM – Otori Takeo Dec 06 '14 at 07:07

score 1 · Answer 3 · answered Dec 01 '14 at 11:49

1

use ClassRegistry if you are using model in component or in other modles if it has no association. use

ClassRegistry::init('Upload');

answered Dec 01 '14 at 11:49

mano_msk

51
1
9

score 0 · Answer 4 · answered Nov 29 '14 at 11:52

Please load the model in your controller function suppose you are work with users table then you load the User model in the function $this->loadmodel('User'); after then you can use the code

if($this->User->save($this->request->data)){
     your code......

}

Error: Call to a member function save() on a non-object

4 Answers4