hipanel\modules\finance\logic\AbstractTariffManager

Complexity

Total Complexity

5

Size/Duplication

Total Lines	50
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	createForm()	0	4	1
A	findBaseModel()	0	10	2
A	getType()	0	4	1

<?php

namespace hipanel\modules\finance\logic;

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

abstract class AbstractTariffManager
{
    /**
     * @var Tariff
     */
    public $baseTariff;

    /**
     * @var AbstractTariffForm
     */
    public $form;

    /**
     * @var string The type used to find base tariff
     */
    protected $type;

    public function __construct($tariff = null)
    {
        $this->findBaseModel();
        $this->createForm($tariff);
    }

    /**
     * Fills [[form]] property with a proper
     *
     * @param Tariff $tariff
     * @throws InvalidConfigException
     */
    protected function createForm($tariff = null)
    {
        throw new InvalidConfigException("Method createForm 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->baseTariff = 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;
    }
}