Assign unique key to each rule in generated model

[mohdqasim98]

The rules generated in the model should be such that each rule must have its a unique string key,
so that it can be easily overridden in the child model.
Example :

public function rules()
{
    return [
        'bioString'            => ['bio', 'string'],
        'timeZoneValidation'   => ['timezone', 'validateTimeZone'],
        'publicEmailPattern'   => ['public_email', 'email'],
        'gravatarEmailPattern' => ['gravatar_email', 'email'],
    ];
}

So that in the child class , it can be easily overridden like:

public function rules()
{
	$rules = parent::rules();
	$rules['bioString'] = ['bio', 'required'];
}

[keshavchauha]

some time i also facing problem when override the child model rules so please define unique string key creating model,

[schmunk42]

How would we handle combined properties like required?

[uldisn]

Real rules:

            [['regcode', 'name'], 'required'],
            [['type_id', 'reg_partnet_company_id'], 'integer'],
            [['office_address', 'status'], 'string'],
            [['regcode'], 'string', 'max' => 11],
            [['sepa'], 'string', 'max' => 20],
            [['name', 'address'], 'string', 'max' => 255],
            [['reg_partnet_company_id'], 'exist', 'skipOnError' => true, 'targetClass' => D3cCompany::className(), 'targetAttribute' => ['reg_partnet_company_id' => 'id']],
            [['country_id', 'comunication_language_id'], 'string', 'max' => 3],
            [['country_id'], 'exist', 'skipOnError' => true, 'targetClass' => \yii2d3\d3persons\models\ClCountries::className(), 'targetAttribute' => ['country_id' => 'id']],
            [['comunication_language_id'], 'exist', 'skipOnError' => true, 'targetClass' => \yii2d3\d3persons\models\ClLanguage::className(), 'targetAttribute' => ['comunication_language_id' => 'id']],
            [['type_id'], 'exist', 'skipOnError' => true, 'targetClass' => \yii2d3\d3persons\models\D3cType::className(), 'targetAttribute' => ['type_id' => 'id']],
            ['status', 'in', 'range' => [
                    self::STATUS_PARTNER,
                    self::STATUS_AUTH,
                    self::STATUS_CLOSED
                ]
            ]

with id:

            'required' => [['regcode', 'name'], 'required'],
            'integer' => [['type_id', 'reg_partnet_company_id'], 'integer'],
            'string' => [['office_address', 'status'], 'string'],
            'stringMax11' => [['regcode'], 'string', 'max' => 11],
            'stringMax20' => [['sepa'], 'string', 'max' => 20],
            'stringMax255' => [['name', 'address'], 'string', 'max' => 255],
            'regParnerCompanyIdExist' => [['reg_partnet_company_id'], 'exist', 'skipOnError' => true, 'targetClass' => D3cCompany::className(), 'targetAttribute' => ['reg_partnet_company_id' => 'id']],
            'stringMax2' => [['country_id', 'comunication_language_id'], 'string', 'max' => 3],
            'countryIdExist' => [['country_id'], 'exist', 'skipOnError' => true, 'targetClass' => \yii2d3\d3persons\models\ClCountries::className(), 'targetAttribute' => ['country_id' => 'id']],
            'statusRange' => ['status', 'in', 'range' => [
                    self::STATUS_PARTNER,
                    self::STATUS_AUTH,
                    self::STATUS_CLOSED
                ]
            ]

[schmunk42]

This looks tricky, but - as always - PRs welcome.