Mailcode\Traits\Commands\Validation\TimezoneTrait::validateSyntax_check_timezone()

Complexity

Conditions	3
Paths	3

Size

Total Lines	25
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    protected function validateSyntax_check_timezone(): void
    {
        // first, check if we have an explicit timezone
        $tokens = $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()
            ->getStringLiterals();

        if (count($tokens) > 1) {
            $this->timezoneString = $tokens[1]->getText();
            return;
        }

        // then, check if a variable is used for timezone
        $variables = $this->requireParams()
            ->getInfo()
            ->getVariables();

        if (count($variables) > 1) {
            $this->timezoneVariable = $variables[1]->getFullName();
            return;
        }

        // neither explicit timezone nor variable present, so use nothing
        $this->timezoneString= null;
        $this->timezoneVariable = null;
    }