CpuResourceDecorator::displayPrepaidAmount() - Code Metrics - hiqdev/hipanel-module-finance - Measure and Improve Code Quality continuously with Scrutinizer

CpuResourceDecorator::displayPrepaidAmount() A
last analyzed 2020-11-24 16:48 UTC

↳ Parent: CpuResourceDecorator

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	9
ccs	0
cts	8
cp	0
rs	9.9666
cc	2
nc	2
nop	0
crap	6

<?php
/**
 * Finance module for HiPanel
 *
 * @link      https://github.com/hiqdev/hipanel-module-finance
 * @package   hipanel-module-finance
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2015-2019, HiQDev (http://hiqdev.com/)
 */

namespace hipanel\modules\finance\models\decorators\server;

use Yii;

class CpuResourceDecorator extends AbstractServerResourceDecorator
{
    const UNIT_CORE = 0;
    const UNIT_MHZ = 1;

    public function displayTitle()
    {
        return Yii::t('hipanel:server:order', 'CPU');
    }

    public function displayPrepaidAmount()
    {
        if ($this->getCpuUnit() === self::UNIT_CORE) {
            return Yii::t('hipanel:server:order', '{0, plural, one{# core} other{# cores}}',
                $this->getPrepaidQuantity());
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        }

        return Yii::t('hipanel:server:order', '{0} MHz', Yii::$app->formatter->asInteger($this->getPrepaidQuantity()));
    }

    public function getPrepaidQuantity()
    {
        if ($this->getCpuUnit() === self::UNIT_CORE) {
            preg_match('/((\d+) cores?)$/i', $this->resource->part->partno, $matches);
        } else {
            preg_match('/((\d+) MHz)$/i', $this->resource->part->partno, $matches);
        }

        return $matches[2] === null ? 0 : $matches[2];
    }

    private function getCpuUnit()
    {
        if (strpos($this->resource->part->partno, 'core') !== false) {
            return self::UNIT_CORE;
        }

        return self::UNIT_MHZ;
    }
}


1			<?php
2			/**
3			* Finance module for HiPanel
4			*
5			* @link https://github.com/hiqdev/hipanel-module-finance
6			* @package hipanel-module-finance
7			* @license BSD-3-Clause
8			* @copyright Copyright (c) 2015-2019, HiQDev (http://hiqdev.com/)
9			*/
10
11			namespace hipanel\modules\finance\models\decorators\server;
12
13			use Yii;
14
15			class CpuResourceDecorator extends AbstractServerResourceDecorator
16			{
17			const UNIT_CORE = 0;
18			const UNIT_MHZ = 1;
19
20			public function displayTitle()
21			{
22			return Yii::t('hipanel:server:order', 'CPU');
23			}
24
25			public function displayPrepaidAmount()
26			{
27			if ($this->getCpuUnit() === self::UNIT_CORE) {
28			return Yii::t('hipanel:server:order', '{0, plural, one{# core} other{# cores}}',
29			$this->getPrepaidQuantity());
			0 ignored issues – show Documentation introduced 2016-09-08 16:30 UTC by Report Bug Copy Issue Report `$this->getPrepaidQuantity()` is of type `integer\|string`, but the function expects a `array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
30			}
31
32			return Yii::t('hipanel:server:order', '{0} MHz', Yii::$app->formatter->asInteger($this->getPrepaidQuantity()));
33			}
34
35			public function getPrepaidQuantity()
36			{
37			if ($this->getCpuUnit() === self::UNIT_CORE) {
38			preg_match('/((\d+) cores?)$/i', $this->resource->part->partno, $matches);
39			} else {
40			preg_match('/((\d+) MHz)$/i', $this->resource->part->partno, $matches);
41			}
42
43			return $matches[2] === null ? 0 : $matches[2];
44			}
45
46			private function getCpuUnit()
47			{
48			if (strpos($this->resource->part->partno, 'core') !== false) {
49			return self::UNIT_CORE;
50			}
51
52			return self::UNIT_MHZ;
53			}
54			}
55

hiqdev / hipanel-module-finance

CpuResourceDecorator::displayPrepaidAmount() A last analyzed 2020-11-24 16:48 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

CpuResourceDecorator::displayPrepaidAmount() A
last analyzed 2020-11-24 16:48 UTC