hipanel\modules\finance\forms\AbstractTariffForm

Complexity

Total Complexity

25

Size/Duplication

Total Lines	193
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

0%

Importance

Changes	4
Bugs	0	Features	0

19 Methods

Rating	Name	Duplication	Size	Complexity
A	ensureTariff()	0	8	2
A	ensureScenario()	0	6	2
A	init()	0	4	1
A	initTariff()	0	6	1
A	setDefaultTariff()	0	6	1
A	rules()	0	9	1
A	fields()	0	6	1
A	attributes()	0	8	1
A	getResources()	0	4	1
A	setResources()	0	4	1
A	getResourceTypes()	0	4	1
A	attributeLabels()	0	9	1
A	load()	0	4	1
B	selectBaseTariff()	0	24	4
A	getBaseTariffsList()	0	7	1
A	insert()	0	4	1
A	update()	0	4	1
A	getTariff()	0	4	1
A	setTariff()	0	13	2

<?php

namespace hipanel\modules\finance\forms;

use hipanel\modules\finance\models\Tariff;
use Yii;
use yii\base\InvalidConfigException;
use yii\helpers\ArrayHelper;

abstract class AbstractTariffForm extends \yii\base\Model
{
    /**
     * @var int Tariff ID
     */
    public $id;

    /**
     * @var string Tariff name
     */
    public $name;

    /**
     * @var int Parent tariff ID
     */
    public $parent_id;
    /** @var Tariff */
    public $baseTariff;
    /** @var Tariff[] */
    public $baseTariffs;
    /** @var Tariff */
    protected $tariff;
    /** @var Resource[] */
    protected $_resources;

    public function init()
    {
        $this->initTariff();
    }

    protected function initTariff()
    {
        $this->selectBaseTariff();
        $this->ensureTariff();
        $this->ensureScenario();
    }

    protected function ensureTariff()
    {
        if ($this->getTariff() instanceof Tariff) {
            return true;
        }

        return $this->setDefaultTariff();
    }

    protected function ensureScenario()
    {
        foreach ($this->tariff->resources as $resource) {
            $resource->scenario = $this->scenario;
        }
    }

    protected function setDefaultTariff()
    {
        $this->setTariff($this->baseTariff);

        return true;
    }

    /** @inheritdoc */
    public function rules()
    {
        return [
            [['name'], 'required', 'on' => ['create', 'update']],
            [['parent_id', 'id'], 'integer', 'on' => ['create', 'update']],
            [['parent_id'], 'required', 'on' => ['create']],
            [['id'], 'required', 'on' => ['update']],
        ];
    }

    /** @inheritdoc */
    public function fields()
    {
        return ArrayHelper::merge(array_combine($this->attributes(), $this->attributes()), [
            'resources' => '_resources'
        ]);
    }

    /** @inheritdoc */
    public function attributes()
    {
        return [
            'id',
            'parent_id',
            'name',
        ];
    }

    public function getResources()
    {
        return $this->_resources;
    }

    /**
     * @param array $resources
     * @throws InvalidConfigException when not implemented
     */
    public function setResources($resources)
    {
        throw new InvalidConfigException('Method "setResources" must be implemented');
    }

    public function getResourceTypes()
    {
        return reset($this->baseTariff->resources)->getTypes();
    }

    public function attributeLabels()
    {
        return [
            'parent_id' => Yii::t('hipanel/finance/tariff', 'Parent tariff'),
            'name' => Yii::t('hipanel/finance/tariff', 'Name'),
            'label' => Yii::t('hipanel/finance/tariff', 'Label'),
            'note' => Yii::t('hipanel', 'Note'),
        ];
    }

    /**
     * @param array $data to be loaded
     * @return bool
     * @throws InvalidConfigException when not implemented
     */
    public function load($data)
    {
        throw new InvalidConfigException("Method load must be implemented");
    }

    public function selectBaseTariff()
    {
        if (!isset($this->parent_id)) {
            if (isset($this->tariff)) {
                $this->parent_id = $this->tariff->parent_id;

The property parent_id does not exist on object<hipanel\modules\finance\models\Tariff>. Since you implemented __set, maybe consider adding a @property annotation.

            } else {
                $this->parent_id = ArrayHelper::getValue(reset($this->baseTariffs), 'id');

It seems like reset($this->baseTariffs) targeting reset() can also be of type false; however, yii\helpers\BaseArrayHelper::getValue() does only seem to accept array|object, did you maybe forget to handle an error condition?

            }
        }

        $filtered = array_filter($this->baseTariffs, function ($model) {
            return $model->id == $this->parent_id;
        });

        if (count($filtered) !== 1) {
            Yii::error('Found ' . count($filtered) . ' base tariffs. Must be exactly one');
            return false;
        }

        $this->baseTariff = reset($filtered);
        $this->parent_id = $this->baseTariff->id;

        return true;
    }

    public function getBaseTariffsList()
    {
        return array_combine(
            ArrayHelper::getColumn($this->baseTariffs, 'id'),
            ArrayHelper::getColumn($this->baseTariffs, 'name')
        );
    }

    public function insert($runValidation = true, $attributes = null, $options = [])

The parameter $runValidation is not used and could be removed.

The parameter $attributes is not used and could be removed.

The parameter $options is not used and could be removed.

    {
        throw new InvalidConfigException("Method load must be implemented");
    }

    public function update($runValidation = true, $attributes = null, $options = [])

The parameter $runValidation is not used and could be removed.

The parameter $attributes is not used and could be removed.

The parameter $options is not used and could be removed.

    {
        throw new InvalidConfigException("Method load must be implemented");
    }

    /**
     * @return Tariff
     */
    public function getTariff()
    {
        return $this->tariff;
    }

    public function setTariff($tariff)
    {
        if ($tariff === null) {
            return false;
        }

        $this->tariff = $tariff;

        $this->id = $tariff->id;
        $this->name = $tariff->name;

        return true;
    }
}