lhs\Yii2SaveRelationsBehavior\SaveRelationsTrait

Complexity

Total Complexity

9

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	_prepareLoadData()	0	28	6
A	load()	0	9	3

<?php

namespace lhs\Yii2SaveRelationsBehavior;

/**
 * Trait SaveRelationsTrait
 * @package lhs\Yii2SaveRelationsBehavior
 *
 * @mixin SaveRelationsBehavior
 */
trait SaveRelationsTrait
{

    public function load($data, $formName = null)
    {
        $loaded = parent::load($data, $formName);
        if ($loaded && $this->hasMethod('loadRelations')) {

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

            $this->_prepareLoadData($data);

            $this->loadRelations($data);

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

        }
        return $loaded;
    }

    /**
     * @param $data
     */
    private function _prepareLoadData(&$data) {
        $scope = $formName === null ? $this->formName() : $formName;

The variable $formName seems to be never defined.

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]);
        }
    }
}