Yii2 AR. Как правильно создать 50 моделей в событии afterSave другой модели?

Question

[Mick Coder]

Здравствуйте! В AR эсть события afterSave которое срабатывает после сохранения записи модели. Мне нужно в этом событии сгенерировать приблизительно 50 записей другой модели и сохранить их в соответсвующу таблицу.

class Reservation extends \yii\db\ActiveRecord
public function afterSave($insert, $changedAttributes)
    {

	    if (parent::afterSave($insert, $changedAttributes)) {
	    	for ($i=0; $i < 50; $i++) {    
                   $modelRegCleanings = new RegularCleanings();
                   $modelRegCleanings->id_reservation = $this->id_reservation;
                   $modelRegCleanings->notes = $changedAttributes['notes'];
                   $modelRegCleanings->save();
               }
	        return true;
	    } else {
	        return false;
	    }
    }

Проблема в том, что после сохранения 1 записи модели Reservation должен произойти редирект на view новосозданной записи

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

        if ($model->load(Yii::$app->request->post()) && $model->save()) {
        	return $this->redirect(['view', 'id' => $model->id_reservation]);   
        } else {
        	return $this->render('create', [
        			'model' => $model,
        			]);
        }
    }

Как сделать так, чтоб перед редиректом выводился попап с уведомлениям, что при создании даной записи было создано 50 записей другой модели? И в правильно ли месте я делаю эту генерацию? Спасибо!

Comments

[Mick Coder]

Моё мнения что создания нового обьекта модели 50 раз в одном екшне - это много.)

Answer 1

[Сергей Протько]

Я без понятия почему вам нужно создать 50 записей... ну да несуть. Делайте где хотите. Правда по хорошему я бы вообще это все в сервисе делал который бы управлял этой логикой. Ну или вынес бы генерацию в статический метод целевой модели... все от бизнес логики зависит.

По поводу уведомления - flash messages.

Comments

[Mick Coder]

Мне нужно для каждой резервации комнаты, генерить записи на регулярную уборку. Т.е с создания резервации брать даты начала и конца резервации и вычислять даты регулярной уборки (каждую неделю).
Например если резервируют комнату на год, то нужно создать 52 записи Регулярных уборок.
Например через метод $modelRegularCleanings->generateRegularCleanings($lockStart, $lockFinish, $id_reservation);
Но я с вами согласен что всё это делать в методе afterSave() плохо.. А в yii2 можно как то с помощью сервиром перехватывать событие afterSave() для конкретной модели? Спасибо за ответ!) На других форумах меня не понимали, что я хочу сделать в стиле best practicies) только учусь и не хочеться прывыкать к костилям)

[Mick Coder]

А в статическом методе целевой модели RegularCleanings можно как то генерировать и сохранять записи той же модели RegularCleanings?
$modelRegularCleanings::generateRegularCleanings($lockStart, $lockFinish, $id_reservation);

[Сергей Протько]

Mick Coder: Ну тип того. Только будет RegularCleanings::generateRegularCleaning. Хотя сейчас подумал - по идее все же не стоит так делать. Эта логика больше относится Reservation.

[Сергей Протько]

Mick Coder: ну как бы... если исключений не повыкидывалось значит все сохранилось и можно дальше делать. Можно еще обернуть все в транзакцию...

Я с Yii не работаю уже давно. И иногда даже кажется что best practice и active record не сильно сочитаются.

[Mick Coder]

Можно как то yi2 с Doctrine подружить)
За подсказку з session flash отдельное спасибо!
А не много ли будет 50 обьектов создавать?

[Сергей Протько]

Mick Coder: Думаю можно, но не знаю как. Надо смотреть. В любом случае, просто взять доктрину не выйдет - лучше попробуйте с другим фреймворком где нет такого уровня связанности. И не на текущих проектах.

50 объектов - не так много. Все зависит от бизнес логики. Я бы расписание хранил как один объект.

[Mick Coder]

Спасибо, Сергей! Очень помогли!)

Answer 2

[Максим Колокольников]

При таком виде у Вас будет минимум 51 запрос в базу на вставку...
Лучше подготовьте модели заранее и потом с помощью batchInsert вставьте всё одним запросом.

Comments

[Mick Coder]

batchInsert() с yii\db\QueryBuilder ? Т.е. даже не придётся много моделей создавать.. попробую так сделать. Спасибо