RemotelyLiving\PHPDNS\Resolvers\Traits\Time

Complexity

Total Complexity

4

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setDateTimeImmutable()	0	3	1
A	getTimeStamp()	0	3	1
A	getNewDateTimeImmutable()	0	8	2

<?php

namespace RemotelyLiving\PHPDNS\Resolvers\Traits;

trait Time
{
    /**
     * @var \DateTimeImmutable|null
     */
    private $dateTimeImmutable = null;

    public function setDateTimeImmutable(\DateTimeImmutable $dateTimeImmutable): void
    {
        $this->dateTimeImmutable = $dateTimeImmutable;
    }

    public function getTimeStamp() : int
    {
        return $this->getNewDateTimeImmutable()->getTimestamp();
    }

    private function getNewDateTimeImmutable(): \DateTimeImmutable
    {
        if (!$this->dateTimeImmutable) {
            $this->dateTimeImmutable = new \DateTimeImmutable();
        }

        return /** @scrutinizer ignore-type */ $this->dateTimeImmutable

The expression return $this->dateTimeImmutable->setTimestamp(time()) could return the type false which is incompatible with the type-hinted return DateTimeImmutable. Consider adding an additional type-check to rule them out.

            ->setTimestamp(time());
    }
}