Как запретить изменение некоторых данных в зависимости от статуса сущности?

Question

[hostadmin]

Есть модель выставленных счетов. Нужно, чтобы при определенных статусах счёта (или еще каких-то данных у конкретного счёта) нельзя было менять некоторые данные, например сумму счёта. При это реализовать это исключительно средствами самой модели.

Пытался это сделать через валидаторы, но они не видят "старых" данных.

Comments

[Дмитрий]

Вечер добрый.
А конкретно, при каких условиях, какие данные, как Вы это пытались реализовать?

[hostadmin]

Ну например, заходит пользователь в редактирование счёта, и пока его смотрел, другой пользователь отправил его в архив (т.е. поменял значение поля status, было 1 стало 9). Нужно, чтобы когда первый пользователь изменит сумму и нажмет "сохранить", ему система сообщила, что изменять сумму счёта нельзя (ну или хотя бы не дала сохранить само поле).

В модели используются сценарии, но довольно обобщено (типа: SCENARIO_USER и SCENARIO_ADMIN), и одинаковы по всему проекту, поэтому не очень хочется добавлять какие-то новые сценарии.

Как я уже писал, при валидации нет доступа к старым данным модели, поэтому сделать валидатор не получилось.

[hostadmin]

Ну еще, как одна идея, это тупо при проверке дёргать данные этой сущности из бд, типа:

['amount', function($attribute)
{
$old = Model::find($this->id);
if($model->{$attribute} != $this->{$attribute} && $model->status != Model::STATUS_NEW)
{
$this->addError($attribute, 'Нельзя менять сумму');
}
}
]

Answer 1

[hostadmin]

В общем, как это часто бывает, виной всему невнимательность. Оказалось, что AJAX отправлял данные на экшн, без указания id сущности, поэтому проверка была как для новой записи.
В итоге такая конструкция работает как нужно:

[['amount'], function($attribute){

                    if ( $this->$attribute != $this->getOldAttribute($attribute) ) {
                        $this->addError($attribute, \Yii::t('app', 'Нельзя изменять сумму счёта'));
                        return false;
                    }

                }, 'when'=>function($model, $attribute)
                {
                     return $model->getOldAttribute('status') != self::STATUS_IS_PENDING;
                }]

Answer 2

[xmoonlight]

Наследование + инкапсуляция.