Почему не правильно сохраняются данные в базу Yii2?

Question

[Владимир]

Доброго времени суток! Есть таблица с полем "from_form" tiny_int(1). Вот экшин:

public function actionView($id)
{
    $model = $this->findModel($id);
    if ($model->from_form > 2) {
        $new_model = Mails::findOne($id);
        $new_model->from_form = $model->from_form - 2;
        $new_model->save();
    }

    return $this->render('view', [
        'model' => $model,
    ]);
}

Я хочу что бы когда открывается запись шла проверка и если true то отнять от этого числа 2 и записать в базу. Изначально в базе есть поле from_form со значением 4. То есть по логике при открытии должно быть 4 - 2 = 2 и 2 записать в базу. Но почему то записывается 3-а. Даже если я пишу $new_model->from_form = 2; всё равно записывается 3-а. Подскажите что я делаю не так? Если нужна ещё какая-то информация пишите в комментарии буду дополнять. Заранее благодарен!

Answer 1

[Дмитрий]

Добрый день.
Перенесите эти вычисления в модель, в метод beforeSave(), и немного измените строку с вычислением
$new_model->from_form = ($model->from_form - 2);

P.S. Приблизительно так должно быть:

public function beforeSave($insert)
{
	if(parent::beforeSave($insert)){
		if((self::getOldAttribute('from_form') > 2){
		    $this->from_form = ($this->from_form - 2)
        }
		return true;
	}
	return false;
}

Comments

[Владимир]

slo_nik мне нужно когда я буду открывать это письмо через actionView, то данные автоматически проверились и перезаписались, а я боюсь что если я запишу его в метод beforeSave() то значения 4 уже не будет а сразу 2. Это у меня письма (с формы на сайте), которые приходят с сайта, я их перед отправкой сохраняю в БД и в поле from_form записываю с какой формы пришло сообщение. У меня есть две формы, если первая форма то я записываю 1, а если вторая то 2. Например форму отправили со второй формы то я записываю в поле значение 4. А потом проверяю если там 4 то это из второй формы новое сообщение. И хочу при открытии этого письма поменять значение на 2. Я дошел до такого решения что бы не создавать ещё одно поле для того что бы обозначать что письмо новое. Может вы подскажите какое то другое решение? Или как быть? Но мне хотелось бы разобратся почему это не работает...

[Дмитрий]

Владимир, Вы сначала сделайте как я написал, а дальше будет видно. Этот метод работает перед сохранением в базу. Проверяет с ранее сохранённым значением.

[Владимир]

slo_nik а что посоветуете написать в контроллере? Мне же надо как то вызвать сохранение.

[Дмитрий]

Что писать в модели я уже написал выше, только добавить сохранение в базу надо.
После строки $this->from_form = ($this->from_form - 2); сохраняем
$this->save()
Но это в том случае, если у Вас новая запись. Если уже есть эта запись, то достаточно обновить один атрибут при помощи updateAttribute()
В контролере

public function actionView($id)
{
    return $this->render('view', [
        'model' => $model,
    ]);
}

[Владимир]

slo_nik, спасибо вам большое) Задача решилась) Не на столько я умен что бы вас понять о чем вы говорили в последнем комментарии но функция updateAttributes() спасла моё положение) Сделал вот так:

public function actionView($id)
{
    $model = $this->findModel($id);
    if ($model->from_form > 2) {
        $model->updateAttributes(['from_form' => $model->from_form - 2]);
    }

    return $this->render('view', [
        'model' => $model,
    ]);
}

Спасибо Вам ещё раз!)

[Дмитрий]

Владимир, а что там понимать. Просто выполнить ctrl+c -> ctrl->v очень сложно?