hipanel\modules\finance\models\CertificatePrice

Complexity

Total Complexity

18

Size/Duplication

Total Lines	165
Duplicated Lines	19.39 %

Coupling/Cohesion

Components	3
Dependencies	9

Test Coverage

Coverage

0%

Importance

Changes

0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	getPeriods()	7	7	1
A	getAvailablePeriods()	11	11	3
A	__construct()	0	6	1
A	tableName()	0	4	1
A	behaviors()	0	33	3
A	rules()	0	21	1
A	isTypeCorrect()	0	4	1
A	getPriceForPeriod()	8	8	2
A	hasPriceForPeriod()	0	4	1
A	validatePrices()	6	14	3
A	getTypes()	0	7	1

<?php

namespace hipanel\modules\finance\models;

use Money\Currency;

This use statement conflicts with another class in this namespace, hipanel\modules\finance\models\Currency.

use Money\Formatter\DecimalMoneyFormatter;
use Money\Money;
use Money\Currencies\ISOCurrencies;
use Money\Parser\DecimalMoneyParser;
use Yii;
use hipanel\base\ModelTrait;
use yii\behaviors\AttributeTypecastBehavior;
use yii\validators\NumberValidator;

/**
 * @property array $sums
 */
class CertificatePrice extends Price
{
    use ModelTrait;

    const TYPE_CERT_PURCHASE = 'certificate,certificate_purchase';
    const TYPE_CERT_RENEWAL = 'certificate,certificate_renewal';

    /**
     * @var DecimalMoneyFormatter
     */
    private $moneyFormatter;

    /**
     * @var DecimalMoneyParser
     */
    private $moneyParser;

    public function __construct(array $config = [])
    {
        parent::__construct($config);
        $this->moneyFormatter = new DecimalMoneyFormatter(new ISOCurrencies());
        $this->moneyParser = new DecimalMoneyParser(new ISOCurrencies());
    }

    public static function tableName()
    {
        return 'price';
    }

    public function behaviors()
    {
        return [
            'typecastAfterFind' => [
                'class' => AttributeTypecastBehavior::class,
                'attributeTypes' => [
                    'sums' => function ($sums) {
                        foreach ($sums as $key => $value) {
                            $sums[$key] = new Money($value, new Currency(strtoupper($this->currency)));
                        }
                        return $sums;
                    }],
                'typecastAfterFind' => true,
                'typecastAfterValidate' => false,
                'typecastBeforeSave' => false,
            ],
            'typecastBeforeSave' => [
                'class' => AttributeTypecastBehavior::class,
                'attributeTypes' => [
                    'sums' => function ($sums) {
                        foreach ($sums as $key => $value) {
                            $sums[$key] = $this->moneyParser
                                ->parse($value, strtoupper($this->currency))
                                ->getAmount();
                        }
                        return $sums;
                    }],
                'typecastAfterFind' => false,
                'typecastAfterValidate' => false,
                'typecastBeforeSave' => true,
            ],
        ];
    }


    public function rules()
    {
        $rules = parent::rules();
        $rules['create-required'] = [
            ['object_id'],
            'required',
            'on' => ['create', 'update'],
            'when' => function ($model) {
                /** @var self $model */
                return $model->isTypeCorrect();

The method isTypeCorrect does not exist on object<hipanel\modules\finance\models\Calculation>? Since you implemented __call, maybe consider adding a @method annotation.

            },
        ];
        $rules[] = [['certificateType'], 'safe'];
        $rules[] = [['sums'], 'validatePrices', 'on' => ['create', 'update']];
        $rules[] = [['sums'], 'number', 'min' => 0, 'when' => function () {
            return false;
        }];
        $rules[] = [['sums'], 'each', 'rule' => ['number', 'min' => 0]];

        return $rules;
    }

    public function isTypeCorrect()
    {
        return isset($this->getTypes()[$this->type]);
    }

    /**
     * @return array
     */
    public static function getPeriods()

This method seems to be duplicated in your project.

    {
        return [
            1 => Yii::t('hipanel:finance:tariff', '{n, plural, one{# year} other{# years}}', ['n' => 1]),
            2 => Yii::t('hipanel:finance:tariff', '{n, plural, one{# year} other{# years}}', ['n' => 2]),
        ];
    }

    /**
     * @return array
     */
    public function getAvailablePeriods()

This method seems to be duplicated in your project.

    {
        $periods = [];
        foreach ([1, 2] as $period) {
            if ($this->hasPriceForPeriod($period)) {
                $periods[$period] = Yii::t('hipanel:finance:tariff', '{n, plural, one{# year} other{# years}}', ['n' => $period]);
            }
        }

        return $periods;
    }

    /**
     * @param int $period
     * @return null|string
     */
    public function getPriceForPeriod(int $period)

This method seems to be duplicated in your project.

    {
        if (!$this->hasPriceForPeriod($period)) {
            return null;
        }

        return $this->moneyFormatter->format($this->sums[$period]);
    }

    /**
     * @param int $period
     * @return bool
     */
    public function hasPriceForPeriod(int $period)
    {
        return !empty($this->sums[$period]);
    }

    public function validatePrices(): bool
    {
        $periods = $this->getPeriods();
        $validator = new NumberValidator();

        foreach (array_keys($periods) as $period) {

This code seems to be duplicated across your project.

            $validation = $validator->validate($this->sums[$period]);
            if ($validation === false) {
                unset($this->sums[$period]);
            }
        }

        return true;
    }

    /**
     * @return array
     */
    public static function getTypes()
    {
        return [
            static::TYPE_CERT_PURCHASE => Yii::t('hipanel:finance:tariff', 'Purchase'),
            static::TYPE_CERT_RENEWAL => Yii::t('hipanel:finance:tariff', 'Renewal'),
        ];
    }
}