Bigwhoop\Trumpet\Config\Params\DateParam::parse()

Complexity

Conditions	3
Paths	3

Size

Total Lines	15
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	3

Importance

Changes

0

<?php declare(strict_types=1);
    public function parse($value, Presentation $presentation)
    {
        if (is_int($value)) {
            // We don't use format 'U' as the TZ would be UTC
            $date = \DateTime::createFromFormat('Y-m-d', date('Y-m-d', $value));
        } elseif (preg_match('#\d{4}-\d{1,2}-\d{1,2}#', $value)) {
            $date = \DateTime::createFromFormat('Y-m-d', $value);
        } else {
            throw new ConfigException('Dates must be a string in the format YYYY-MM-DD.');
        }

        $date->setTime(0, 0, 0);

        $presentation->date = new Date($date);

It seems like $date can also be of type false; however, Bigwhoop\Trumpet\Config\Date::__construct() does only seem to accept object<DateTime>, did you maybe forget to handle an error condition?

    }
}