beforeSave() - как правильно?

[porcelanosa]

Все ли тут правильно? Смущает параметр $insert - из документации, я не совсем поняла, в какой ветке срабатывает update? Там где return false?
Мне надо что бы нижеследующий код срабатывал всегда.

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

public function beforeSave($insert)
    {
        if (parent::beforeSave($insert)) {
            $this->publish_date = strtotime(Yii::$app->request->post('publish_date'));
            return true;
        } else {
            return false;
        }
    }

У меня не срабатывает.
Правда у меня и при попытке изменить в контроллере значение publish_date перед сохранением не работает.

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

    public function actionUpdate($id)
    {
        $model = $this->findModel($id);
        $model->publish_date = strtotime(Yii::$app->request->post('publish_date'));
        if ($model->load(Yii::$app->request->post()) && $model->save()) {
            return $this->redirect(['update', 'id' => $model->id]);
        } else {
            return $this->render('update', [
                'model' => $model,
            ]);
        }
    }

Поле publish_date - помечено как безопасное.
В чем может быть дело?

[zelenin]

Все ли тут правильно? Смущает параметр $insert - из документации, я не совсем поняла, в какой ветке срабатывает update? Там где return false?

это вообще не относится к делу. если false, то save не работает

Мне надо что бы нижеследующий код срабатывал всегда.

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

public function beforeSave($insert)
    {
        if (parent::beforeSave($insert)) {
            $this->publish_date = strtotime(Yii::$app->request->post('publish_date'));
            return true;
        } else {
            return false;
        }
    } 

не надо request использовать в модели.

Поле publish_date - помечено как безопасное.
В чем может быть дело?

да кто же его знает. xdebug и минуту времени и у вас есть ответ.

[kilimandjaro2]

Мне надо что бы нижеследующий код срабатывал всегда., тогда не делайте if (parent::beforeSave($insert)) { пишите сразу что делать, он срабатывает всегда

[zelenin]

Мне надо что бы нижеследующий код срабатывал всегда., тогда не делайте if (parent::beforeSave($insert)) { пишите сразу что делать, он срабатывает всегда

правильно она пишет, только лучше делать так:

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

$this->publish_date = strtotime(Yii::$app->request->post('publish_date'));
return parent::beforeSave($insert);
 

[porcelanosa]

не надо request использовать в модели.

А что же там можно использовать для получения $_POST ?

[porcelanosa]

Поняла, как сделать.

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

public function beforeSave($insert)
    {
        if (parent::beforeSave($insert)) {
            $this->publish_date= strtotime($this->publish_date);
            return parent::beforeSave($insert);
        } else {
            return false;
        }
    }

Теперь работает.
Получается массив request->post() обнуляется что ли? После использования в $model->load() ?

[porcelanosa]

И какие-то проблемы в использовании TIMESTAMP - не сохраняет если поле в таблице этого типа.
Если INT - то все ОК.
Но ведь strtotime как раз и конвертирует в UNIX TIMESTAMP
В чем может быть дело?

[zelenin]

не надо request использовать в модели.

А что же там можно использовать для получения $_POST ?

нет, вообще модель должна быть отделена от внешних данных. внешние данные вы обрабатываете в контроллере, а дальше уходят уже чистые данные.

[zelenin]

Поняла, как сделать.

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

public function beforeSave($insert)
    {
        if (parent::beforeSave($insert)) {
            $this->publish_date= strtotime($this->publish_date);
            return parent::beforeSave($insert);
        } else {
            return false;
        }
    } 

Теперь работает.

но это не имеет смысла.
вы два раза вызываете parent::beforeSave, причем второй отдает тот же true или false (в вашем случае true, т.к. первый раз сработало). То есть эта запись идентична вашей первой и моей, но мой вариант более лаконичный.

Получается массив request->post() обнуляется что ли? После использования в $model->load() ?

не должен. Но вы можете не задавать этот вопрос, а поставить два вар дампа.

[zelenin]

И какие-то проблемы в использовании TIMESTAMP - не сохраняет если поле в таблице этого типа.
Если INT - то все ОК.
Но ведь strtotime как раз и конвертирует в UNIX TIMESTAMP
В чем может быть дело?

ну что вы какие-то простейшие вопросы задаете. возьмите да вар дампом посмотрите, что он вам возвращает.
посомтрите getErrors() если не сохраняетс модель.
И вообще, не используйте strtotime() - унифицируйте ваш опыт работы со временем в помощью DateTime. Он универсален в отличии от strtotime

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

DateTime::createFromFormat($format, $value)->toUnixTime(); или (new DateTime($value))->toUnixTime();

[porcelanosa]

Да спасибо.

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

$this->publish_date = Yii::$app->formatter->asTimestamp($this->publish_date); 

или вот так вот

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

(new DateTime($this->publish_date))->getTimestamp(); 

Результат одинаковый.
Но проблема, возможно в MySQL
Вот такой запрос, выполняется без ошибок:

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

UPDATE  `news` SET  `publish_date` =  '1448841600' WHERE  `id` =7

Значение publish_date взято из var_damp
Но при этом значение не изменяется.
Схема БД
При этом created_date и modified_date через TimestampBehavior меняется нормально.

[zelenin]

"created_date и modified_date через TimestampBehavior меняется нормально"
покажите скрин. на нем будет видна ошибка

[Йож]

А чем не устраивает http://www.yiiframework.com/doc-2.0/yii ... avior.html ?

[porcelanosa]

А чем не устраивает http://www.yiiframework.com/doc-2.0/yii ... avior.html ?

Потому что оно срабатывает по insert и по update - а нужен независимый механизм установки даты.

Publish_date можно менять когда угодно

[zelenin]

Потому что оно срабатывает по insert и по update

оно сработает на любом AR-событии, которое укажите.