Ошибка «Call to a member function saveAs() on a non-object» в чем ошибка?

Question

[Михаил]

Всем здравствуйте! Подскажите пожалуйста из-за чего выходит ошибка "Call to a member function saveAs() on a non-object", данная ошибка указывает на то, в моем случае ошибка показывает, что при сохранении формы не прикреплен документ, с прикрепленным документов сохранение проходит нормально.

Контроллер

public function actionCreate()
{    
    $model = new Material();

    if ($model->load(Yii::$app->request->post())){
        $model->alias = str2url($res . ' ' . $model->name); 
        $model->docFile = UploadedFile::getInstance($model, 'docFile'); // Загрузка документа
        if ($model->validate()) {
            if ($model->save(false)) {
                $model->gallery = UploadedFile::getInstances($model, 'gallery');
                $model->uploadGallery();
            }
            if($model->path = $model->upload()) { // Присваиваем путь возвращаемый upload()
                if ($model->save(false)) {
                    // return $this->redirect(['view', 'id' => $model->id]);
                }
            }
            Yii::$app->session->setFlash('success', "Материал {$model->name} добавлен");
            return $this->redirect(['view', 'id' => $model->id]);
        }
    }
    return $this->render('create', [
        'model' => $model,
    ]);
}

Модель

public function rules()
{
    return [
        [['docFile'], 'file', 'skipOnEmpty' => true, 'extensions' => 'txt, pdf, djvu, doc, docx, xls, xlsx, ppt, pptx, zip, rar, 7z, tiff'],
    ];
}

public function upload()
{    
    if ($this->validate()) {
        $path = $this->processMkDir();
        $fileName = Inflector::slug(pathinfo($this->str2url($this->docFile->baseName), PATHINFO_FILENAME)) . '.' . $this->docFile->extension;
        $this->docFile->saveAs($path . $fileName);
        return $fileName;
    } else {
        return false;
    }
}

Comments

[Дмитрий]

Добрый вечер.
return $fileName; -> return true;
И проверьте, что у Вас в $fileName попадает, скорей всего в этом проблема.

[Михаил]

slo_nik, в $fileName передается имя загружаемого файла, которое храниться у меня в БД, к сожалению предложенное Вами решение не помогло разрешить проблему

[Дмитрий]

Михаил Балагура, проверяйте, что у Вас попадает параметром в saveAs().

[Дмитрий]

Михаил Балагура, да и запись нового имени в базу я бы сделал по-другому. Зачем в действии два раза вызывать $model->save()? Это лишнее.

[Дмитрий]

Михаил Балагура, UploadedFile::getInstance применяется для загрузки одного файла, а UploadedFile::getInstances для загрузки нескольких.
Ошибка выпадает когда Вы пытаетесь сохранить один файл или несколько?
Возможно необходимо использовать цикл.
Смотрите внимательней, что у Вас приходит в upload()?

[Михаил]

slo_nik, не спорю, возможно можно написать по другому, я пытался и сталкивался с другими ошибками, в данном случае сделал так, потому что в этом случае форма работала и сохраняла данные. В форме у меня два разных поля - одно поле для сохранения картинки/фотографии, а другая для сохранения документа, в предыдущих версиях которые я писал в контролере - у меня получалось, что я сохранял либо только картинку, либо документ, а в такой версии получается сохранить и фото и документ одновременно.

[Михаил]

slo_nik, в данном случае я сохраняю форму и ничего не прикрепляю к ней - ни документ, ни картинку, но ошибка указывает именно на функцию загрузки документа. По поводу загрузок - картинок может быть много, поэтому используется getInstances, а при загрузке документа - не более одного, поэтому getInstance

[Дмитрий]

Михаил Балагура, skipOnEmpty по умолчанию true, так что это бессмысленно.
Метод upload() используете и для загрузки документа и для загрузки файлов?
Повторю, использовать два раза save() не нужно, необходимо сделать так, чтобы использовалось один раз.

[Дмитрий]

Михаил Балагура, возвращать нужно не имя файла, а true.
Если поле пустое, то как Вы можете вернуть имя файла?

Answer 1

[grinat]

Вместо инстанса пришел null https://github.com/yiisoft/yii2/blob/master/framew...
Проверяй его на null

Answer 2

[Михаил]

grinat, Вы не могли бы указать или показать на моем примере как реализовать проверку, я так понимаю проверку надо сделать в функции upload в модели, но как ее правильно прописать не пойму, так как я пытался, то у меня пошли только еще новые ошибки связанные с неправильно написанной функцией.

Comments

[grinat]

if ($this->docFile !== null) {
$path = $this->processMkDir();
$fileName = Inflector::slug(pathinfo($this->str2url($this->docFile->baseName), PATHINFO_FILENAME)) . '.' . $this->docFile->extension;
$this->docFile->saveAs($path . $fileName);
return $fileName;
} else {
return false;
}