Bigwhoop\Trumpet\Config\Params\DateParam

Complexity

Total Complexity

3

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	parse()	0	15	3

<?php declare(strict_types=1);

namespace Bigwhoop\Trumpet\Config\Params;

use Bigwhoop\Trumpet\Config\ConfigException;
use Bigwhoop\Trumpet\Config\Presentation;
use Bigwhoop\Trumpet\Config\Date;

final class DateParam implements Param
{
    /**
     * {@inheritdoc}
     */
    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?

    }
}