Mailcode\RegionTrait::validateSyntax_check_region()

Complexity

Conditions	4
Paths	3

Size

Total Lines	21
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    protected function validateSyntax_check_region(): 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(RegionInterface::REGION_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 region token:') . ' ' . t('Expected a variable or a string.'),
                RegionInterface::VALIDATION_REGION_WRONG_TYPE
            );
            return;
        }

        $this->region = $token;
    }