TariffManager::findBaseModel() - Code Metrics - Inspection of "Implemeted domain tariff management" - hiqdev/hipanel-module-finance - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 0012a2...099852 )

by Dmitry

created 2016-08-18 14:50 UTC

TariffManager::findBaseModel() A

↳ Parent: TariffManager

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	10
ccs	0
cts	9
cp	0
rs	9.4285
cc	2
eloc	6
nc	2
nop	0
crap	6

<?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();
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        $this->baseModel = reset($query->populate($availableTariffs));

    }

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


1			<?php
2
3			namespace hipanel\modules\finance\logic;
4
5			use hipanel\modules\finance\models\Tariff;
6			use yii\base\InvalidConfigException;
7			use yii\web\ForbiddenHttpException;
8
9			abstract class TariffManager
10			{
11			/**
12			* @var string The type used to find base tariff
13			*/
14			protected $type;
15
16			/**
17			* @var Tariff
18			*/
19			public $baseModel;
20
21			/**
22			* @var Tariff
23			*/
24			public $model;
25
26			public function __construct($model = null)
27			{
28			$this->findBaseModel();
29			$this->setModel($model);
30			}
31
32			protected function setModel($model = null)
33			{
34			throw new InvalidConfigException("Method findModel must be implemented");
35			}
36
37			protected function findBaseModel()
38			{
39			$availableTariffs = Tariff::perform('GetAvailableInfo', ['type' => $this->type], true);
40			if (empty($availableTariffs)) {
41			throw new ForbiddenHttpException('No available domain tariffs found');
42			}
43
44			$query = Tariff::find()->joinWith('resources')->prepare();
			0 ignored issues – show Bug introduced 2016-08-18 14:58 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$query` is correct as `\hipanel\modules\finance...'resources')->prepare()` (which targets `hiqdev\hiart\ActiveQuery::prepare()`) seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
45			$this->baseModel = reset($query->populate($availableTariffs));
			0 ignored issues – show Bug introduced 2016-08-18 14:58 UTC by Report Bug Copy Issue Report `$query->populate($availableTariffs)` cannot be passed to `reset()` as the parameter `$array` expects a reference. Loading history... Bug introduced 2016-08-18 14:58 UTC by Report Bug Copy Issue Report The method `populate` cannot be called on `$query` (of type `null`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
46			}
47
48			public function getType()
49			{
50			return $this->type;
51			}
52			}
53

hiqdev / hipanel-module-finance

Push — master ( 0012a2...099852 )

TariffManager::findBaseModel() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like