hipanel\modules\finance\logic\TariffManager

Complexity

Total Complexity

5

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	setModel()	0	4	1
A	findBaseModel()	0	10	2
A	getType()	0	4	1

<?php

namespace hipanel\modules\finance\logic;

use hipanel\modules\finance\models\Tariff;
use yii\base\InvalidConfigException;
use yii\web\ForbiddenHttpException;

abstract class TariffManager
{
    /**
     * @var string The type used to find base tariff
     */
    protected $type;

    /**
     * @var Tariff
     */
    public $baseModel;

    /**
     * @var Tariff
     */
    public $model;

    public function __construct($model = null)
    {
        $this->findBaseModel();
        $this->setModel($model);
    }

    protected function setModel($model = null)
    {
        throw new InvalidConfigException("Method findModel must be implemented");
    }

    protected function findBaseModel()
    {
        $availableTariffs = Tariff::perform('GetAvailableInfo', ['type' => $this->type], true);
        if (empty($availableTariffs)) {
            throw new ForbiddenHttpException('No available domain tariffs found');
        }

        $query = Tariff::find()->joinWith('resources')->prepare();

Are you sure the assignment to $query is correct as \hipanel\modules\finance\models\Tariff::find()->joinWith('resources')->prepare() (which targets hiqdev\hiart\ActiveQuery::prepare()) seems to always return null.

        $this->baseModel = reset($query->populate($availableTariffs));

$query->populate($availableTariffs) cannot be passed to reset() as the parameter $array expects a reference.

The method populate cannot be called on $query (of type null).

    }

    public function getType()
    {
        return $this->type;
    }
}