lhs\Yii2SaveRelationsBehavior\SaveRelationsTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	isTransactional()	0	7	2
A	load()	0	7	3

<?php

namespace lhs\Yii2SaveRelationsBehavior;

trait SaveRelationsTrait
{

    /**
     * Populates the relations with input data.
     */
    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->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;
    }

    /**
     * Auto start transaction if model has relations
     */
    public function isTransactional($operation)
    {
        if ($this->hasProperty('relations')) {

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

            return count($this->relations) > 0;
        }

        return parent::isTransactional($operation);
    }

}