SDIS62\Core\Ops\Entity\Engagement\PompierEngagement

Complexity

Total Complexity

5

Size/Duplication

Total Lines	85
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	5
Bugs	2	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	12	1
A	getPompier()	0	4	1
A	getSpecialitesEngagees()	0	8	2
A	getMateriel()	0	4	1

<?php

namespace SDIS62\Core\Ops\Entity\Engagement;

use SDIS62\Core\Ops\Entity\Engagement;
use SDIS62\Core\Ops\Entity\Intervention;
use SDIS62\Core\Ops\Entity\Materiel;
use SDIS62\Core\Ops\Entity\Pompier;

class PompierEngagement extends Engagement
{
    /**
     * Type.
     *
     * @var string
     */
    protected $type = 'pompier';

    /**
     * Pompier associé.
     *
     * @var SDIS62\Core\Ops\Entity\Pompier
     */
    protected $pompier;

    /**
     * Matériel engagé.
     *
     * @var SDIS62\Core\Ops\Entity\Materiel
     */
    protected $materiel;

    /**
     * Spécialités engagées.
     *
     * @var array
     */
    protected $specialites_engagees;

    /**
     * Ajout d'un engagement de type pompier à une intervention.
     *
     * @param SDIS62\Core\Ops\Entity\Intervention $intervention
     * @param SDIS62\Core\Ops\Entity\Materiel     $materiel
     * @param SDIS62\Core\Ops\Entity\Pompier      $pompier
     * @param array                               $specialites_engagees
     */
    public function __construct(Intervention $intervention, Materiel $materiel, Pompier $pompier, array $specialites_engagees = [])
    {
        $this->pompier = $pompier;

It seems like $pompier of type object<SDIS62\Core\Ops\Entity\Pompier> is incompatible with the declared type object<SDIS62\Core\Ops\Entity\Engagement\SDIS62\Core\Ops\Entity\Pompier> of property $pompier.

        $this->pompier->addEngagement($this);

        $this->materiel = $materiel;

It seems like $materiel of type object<SDIS62\Core\Ops\Entity\Materiel> is incompatible with the declared type object<SDIS62\Core\Ops\Entity\Engagement\SDIS62\Core\Ops\Entity\Materiel> of property $materiel.

        $this->materiel->addEngagement($this);

        $this->specialites_engagees = $specialites_engagees;

        parent::__construct($intervention, $materiel);

The call to Engagement::__construct() has too many arguments starting with $materiel.

    }

    /**
     * Get the value of Pompier associé.
     *
     * @return SDIS62\Core\Ops\Entity\Pompier
     */
    public function getPompier()
    {
        return $this->pompier;
    }

    /**
     * Spécialitées engagées.
     *
     * @return array
     */
    public function getSpecialitesEngagees()
    {
        if (!($this->getPompier() instanceof Pompier\SpecialistePompier)) {
            return [];
        }

        return array_intersect($this->getPompier()->getSpecialites(), $this->specialites_engagees);
    }

    /**
     * Get the value of Matériel engagé.
     *
     * @return SDIS62\Core\Ops\Entity\Materiel
     */
    public function getMateriel()
    {
        return $this->materiel;
    }
}