hipanel\modules\server\helpers\consumptionPriceFormatter\MultiplePrice

Complexity

Total Complexity

3

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	showPrice()	0	8	2
A	__construct()	0	3	1

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

namespace hipanel\modules\server\helpers\consumptionPriceFormatter;

use hipanel\modules\server\models\Consumption;
use Yii;
use yii\helpers\Html;

class MultiplePrice implements PriceFormatterStrategy
{
    /**
     * @var string
     */
    private $delimiter;

    /**
     * @param string $delimiter
     */
    public function __construct(?string $delimiter = null)
    {
        $this->delimiter = $delimiter ?? Html::tag('br');
    }

    /**
     * @param Consumption $consumption
     * @throws \yii\base\InvalidConfigException
     * @return string
     */
    public function showPrice(Consumption $consumption): string
    {
        $prices = [];
        foreach ($consumption->prices as $currency => $amount) {

The property prices does not exist on hipanel\modules\server\models\Consumption. Since you implemented __get, consider adding a @property annotation.

            $prices[] = Yii::$app->formatter->asCurrency($amount, $currency);
        }

        return implode($this->delimiter, $prices);
    }
}