roaresearch\yii2\formgenerator\roa\models\FieldRulePropertySearch

Complexity

Total Complexity

5

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	search()	0	18	3
A	rules()	0	5	1
A	slugBehaviorConfig()	0	5	1

<?php

namespace roaresearch\yii2\formgenerator\roa\models;

use roaresearch\yii2\roa\ResourceSearch;
use yii\data\ActiveDataProvider;
use yii\web\NotFoundHttpException;

/**
 * Contract to filter and sort collections of `Form` records.
 * @author Angel (Faryshta) Guevara <aguevara@alquimiadigital.mx>
 */
class FieldRulePropertySearch extends FieldRuleProperty implements
    ResourceSearch
{
    /**
     * @inhertidoc
     */
    protected function slugBehaviorConfig(): array
    {
        return [
            'idAttribute' => [],
            'resourceName' => 'property',
        ];
    }

    /**
     * @inhertidoc
     */
    public function rules()
    {
        return [
            [['rule_id'], 'required'],
            [['rule_id', 'created_by'], 'integer'],
        ];
    }

    /**
     * @inhertidoc
     */
    public function search(
        array $params,
        ?string $formName = ''
    ): ?ActiveDataProvider {
        $this->load($params, $formName);
        if (!$this->validate()) {
            return null;
        }
        if (null === $this->rule) {
            throw new NotFoundHttpException('Rule not found');
        }
        $this->rule->checkAccess($params);

The method checkAccess() does not exist on roaresearch\yii2\formgenerator\models\FieldRule. Since you implemented __call, consider adding a @method annotation.

        $class = get_parent_class();

        return new ActiveDataProvider([
            'query' => $class::find()->andFilterWhere([
                'rule_id' => $this->rule_id,
                'created_by' => $this->created_by,

The property created_by does not exist on roaresearch\yii2\formgenerator\roa\models\FieldRulePropertySearch. Since you implemented __get, consider adding a @property annotation.

            ]),
        ]);
    }
}