SDIS62\Core\Ops\Entity\Evenement

Complexity

Total Complexity

5

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	4
Bugs	2	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	12	3
A	getDate()	0	4	1
A	getDescription()	0	4	1

<?php

namespace SDIS62\Core\Ops\Entity;

use Datetime;
use SDIS62\Core\Common\Entity\IdentityTrait;

class Evenement
{
    use IdentityTrait;

    /**
     * Date de l'évenement.
     *
     * @var Datetime|null
     */
    protected $date;

    /**
     * Description de l'évenement.
     *
     * @var string
     */
    protected $description;

    /**
     * Ajout d'un evenement à une intervention.
     *
     * @param SDIS62\Core\Ops\Entity\Intervention $intervention

There is no parameter named $intervention. Was it maybe removed?

     * @param string                              $description
     * @param Datetime|string|null                $date         Optionnel
     */
    public function __construct($description, $date = null)
    {
        if ($date instanceof Datetime) {
            $this->date = $date;
        } elseif ($date === null) {
            $this->date = new Datetime('NOW');
        } else {
            $this->date = DateTime::createFromFormat('d-m-Y H:i:s', (string) $date);

It seems like \Datetime::createFromFormat('d-m-Y H:i:s', (string) $date) can also be of type false. However, the property $date is declared as type object<DateTime>|null. Maybe add an additional type check?

        }

        $this->description = $description;
    }

    /**
     * Get the value of Date de l'évenement.
     *
     * @return Datetime|null
     */
    public function getDate()
    {
        return $this->date;
    }

    /**
     * Get the value of Description de l'évenement.
     *
     * @return string
     */
    public function getDescription()
    {
        return $this->description;
    }
}