Mailcode\CurrencyTrait

Complexity

Total Complexity

6

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	validateSyntax_check_currency()	0	21	4
A	isCurrencyPresent()	0	3	1
A	getCurrencyToken()	0	3	1

<?php
/**
 * File containing the trait {@see \Mailcode\Traits\Commands\Validation\CurrencyTrait}.
 *
 * @package Mailcode
 * @subpackage Validation
 * @see \Mailcode\Traits\Commands\Validation\CurrencyTrait
 */

declare(strict_types=1);

namespace Mailcode;

/**
 * @package Mailcode
 * @subpackage Validation
 * @author Olaf Böcker <olaf.boecker@ionos.com>
 *
 * @see CurrencyInterface
 */
trait CurrencyTrait
{
    private ?Mailcode_Parser_Statement_Tokenizer_Token $currency = null;

    /**
     * @throws Mailcode_Exception
     */
    protected function validateSyntax_check_currency(): void
    {
        $token = $this
            ->requireParams()

It seems like requireParams() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            ->getInfo()
            ->getTokenByParamName(CurrencyInterface::CURRENCY_PARAMETER_NAME);

        if ($token === null) {
            return;
        }

        if (!$token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Variable &&
            !$token instanceof Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral) {
            $this->validationResult->makeError(
                t('Invalid currency token:') . ' ' . t('Expected a variable or a string.'),
                CurrencyInterface::VALIDATION_CURRENCY_WRONG_TYPE
            );
            return;
        }

        $this->currency = $token;
    }

    public function isCurrencyPresent(): bool
    {
        return $this->getCurrencyToken() !== null;
    }

    public function getCurrencyToken(): ?Mailcode_Parser_Statement_Tokenizer_Token
    {
        return $this->currency;
    }
}