Yii2: safe validator on condition

Question

I am trying to use this validation rule in my model, but it's not working.

I mean it always remains safe even if select other option.

[['dhanwantri_bill_number'], 'safe', 
    'when' => function($model) {
        return $model->bill_type =='d';                 
    },
    'whenClient' => "function (attribute, value) {
                         return $('#opdtestbill-bill_type').val() == 'd';
                     }"
],

Am I doing anything wrong? is there any alternative solution to achieve the same.

Thanks.

Rule for bill_type is like

[['bill_type'], 'string', 'max' => 20],
[['bill_type','test_name','date'], 'required'],

Edit safe attribute public properties as per doc

$when - callable - A PHP callable whose return value determines whether this validator should be applied. yii\validators\Validator

$whenClient - string - A JavaScript function name whose return value determines whether this validator should be applied on the client side. yii\validators\Validator

Do you really want to use the 'safe' validator? Doc says SafeValidator serves as a dummy validator whose main purpose is to mark the attributes to be safe for massive assignment. So you want to disallow the assignment of dhanwantri_bill_number if bill_type is 'd'? — robsch
@robsch - No it's exactly the opposite - I want to allow the assignment if dhanwantri_bill_number if bill_type is 'd' and disallow in other cases. — Pawan
@topherkanyuga - No I have no other rules for this attribute. — Pawan

robsch robsch · Accepted Answer · 2015-04-01T09:16:12

As the 'safe' validator just tells that an attribute may be set by massive assignment, the approach is not appropiate. It just says that when you use the load() method the attribute can get a value. And if not marked as 'safe' it doesn't prevent setting a value with e.g. $model->dhanwantri_bill_number = 'asdf'. So it is not a proper solution.

More precisely: the 'safe' attribute does not have an effect when $model->validate() (which is usually called with $model->save()) gets called. It is only used when $model->load() is called. If you look into the source code of the SafeValidatior class you see that nothing happens with this validator. The validator doesn't do anything. It is just a marker (you may want to compare it to e.g. RequiredValidator). And with load() the 'when' expression is not used. So you can say 'safe' doesn't work with 'when'. The safe validator may get used when the rule gets evaluated but its validateAttribute() is empty so nothing happens in that point in time.

Besides the whenClient in your code doesn't make sense. What should happen here?

I guess there are several ways of realizing that. One idea would be to let the controller set the attributes not by load(), rather set them explicitely and check there if $model->dhanwantri_bill_number should be set or not. Or you could use load() and then revert the attribute after loading according to what $model->bill_type is set.

Or you could implement a setter method for dhanwantri_bill_number in your model and choose there if the attribute gets set or not. Maybe scenario dependent.

Yii2: safe validator on condition

2 Answers