Artack\Dsn\Field\DateField

Complexity

Total Complexity

4

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setDateTime()	0	7	2
A	__construct()	0	5	1
A	getDateTime()	0	3	1

<?php

declare(strict_types=1);

namespace Artack\Dsn\Field;

class DateField extends AbstractField
{
    private $dateTime;

    public function __construct(string $name, \DateTimeInterface $date)
    {
        parent::__construct($name, null);

        $this->setDateTime($date);
    }

    public function getDateTime(): \DateTimeImmutable
    {
        return $this->dateTime;

The expression return $this->dateTime could return the type boolean which is incompatible with the type-hinted return DateTimeImmutable. Consider adding an additional type-check to rule them out.

    }

    /**
     * If a DateTime instance is provided, it is converted to DateTimeImmutable.
     */
    public function setDateTime(\DateTimeInterface $dateTime): void
    {
        if ($dateTime instanceof \DateTime) {
            $immutable = new \DateTimeImmutable('@'.$dateTime->getTimestamp());
            $dateTime = $immutable->setTimezone($dateTime->getTimezone());
        }
        $this->dateTime = $dateTime;
    }
}