Yii Framework

Привет всем! Подскажите правильно ли я делаю.

Допустим есть модель Product и связанная с ней модель Image, то есть товар и изображения к нему.
С этими моделями я организовал CRUD. Но когда удаляешь товар, то изображения удаляются только в БД, а физически загруженные изображения не удаляются, хотя в модели Image прописан метод afterDelete() с удалением изображений. То есть получается, что удаление записей происходит только на уровне связей в БД, а чтобы данные связанных моделей удалялись с диска нужно это действие прописывать в коде самостоятельно?

Например, в контроллере ProductController метод actionDelete() должен быть в таком виде:

Код: Выделить всё

public function actionDelete($id)
    {
        $model = $this->findModel($id);

        if($model->images) {
            foreach ($model->images as $image) {
                $image->delete();
            }
        }

        $model->delete();

        return $this->redirect(['index']);
    }

Ну да. Можете эту штуку добавить в afterDelete() товара.

andrei.obuhovski писал(а):Ну да. Можете эту штуку добавить в afterDelete() товара.

По идее да, спасибо!

В модели Product добавил метод:

Код: Выделить всё

public function afterDelete()
    {
        if($this->images) {
            foreach ($this->images as $image) {
                $image->delete();
            }
        }

        parent::afterDelete();
    }

Но так не удаляет. Что делаю не так?

Смею предположить что у вас в БД стоит автоматическое удаление связных записей по внешнему ключу.
Как вариант, перенести эту штуку в beforeDelete()

А не правильнее будет это делать в beforeDelete() ?

Все верно! Спасибо!

Сделал так:

Код: Выделить всё

public function beforeDelete()
    {
        if (parent::beforeDelete()) {
            if($this->images) {
                foreach ($this->images as $image) {
                    $image->delete();
                }
            }
            return true;
        } else {
            return false;
        }
    }

Проверку if($this->images) можно убрать, т.к. там в любом случае вернется массив. И если он пустой будет то foreach просто не выполнится.

Yii Framework

Каскадное удаление данных

Каскадное удаление данных

Re: Каскадное удаление данных

Re: Каскадное удаление данных

Re: Каскадное удаление данных

Re: Каскадное удаление данных

Re: Каскадное удаление данных

Re: Каскадное удаление данных