lhs\Yii2SaveRelationsBehavior\SaveRelationsTrait::_prepareLoadData()

Complexity

Conditions	6
Paths	14

Size

Total Lines	28
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    private function _prepareLoadData(&$data, $formName) {
        $scope = $formName === null ? $this->formName() : $formName;

It seems like formName() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        foreach ($this->relations as $key => $value) {
            if (is_int($key)) {
                $relationName = $value;
            } else {
                $relationName = $key;
            }

            if(!isset($data[$scope][$relationName])) {
                continue;
            }

            $relation = $this->getRelation($relationName);

It seems like getRelation() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            if(!$relation) {
                continue;
            }

            $modelClass = $relation->modelClass;
            /** @var ActiveQuery $relationalModel */
            $relationalModel = new $modelClass;
            $keyName = $relationalModel->formName();

            $data[$keyName] = $data[$scope][$relationName];

            unset($data[$scope][$relationName]);
        }