Bone\BoneDoctrine\Traits\HasDeletedAtDate

Complexity

Total Complexity

2

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getDeletedAt()	0	3	1
A	setDeletedAt()	0	4	1

<?php

declare(strict_types=1);

namespace Bone\BoneDoctrine\Traits;

use DateTimeImmutable;
use DateTimeInterface;
use DateTimeZone;
use Doctrine\ORM\Mapping as ORM;

#[ORM\HasLifecycleCallbacks]
trait HasDeletedAtDate
{
    #[ORM\Column(type: 'datetime_immutable', nullable:true)]
    private ?DateTimeImmutable $deletedAt = null;

    public function getDeletedAt(): DateTimeInterface
    {
        return $this->deletedAt;

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

    }

    #[ORM\PreUpdate]
    public function setDeletedAt(): void
    {
        $this->deletedAt = new DateTimeImmutable('now', new DateTimeZone('UTC'));
    }
}