Obblm\Core\Entity\Player

Complexity

Total Complexity

17

Size/Duplication

Total Lines	130
Duplicated Lines	0 %

Test Coverage

Coverage

27.5%

Importance

Changes	1
Bugs	0	Features	0

14 Methods

Rating	Name	Duplication	Size	Complexity
A	setNumber()	0	5	1
A	getPosition()	0	3	1
A	setPosition()	0	5	1
A	addVersion()	0	8	2
A	getId()	0	3	1
A	removeVersion()	0	11	3
A	setStarPlayer()	0	5	1
A	getVersions()	0	3	1
A	getStarPlayer()	0	3	1
A	__construct()	0	3	1
A	getNumber()	0	3	1
A	isStarPlayer()	0	3	1
A	getTeam()	0	3	1
A	setTeam()	0	5	1

<?php

namespace Obblm\Core\Entity;

use Obblm\Core\Entity\Traits\DeadAndFireTrait;
use Obblm\Core\Entity\Traits\NameTrait;
use Obblm\Core\Repository\PlayerRepository;
use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\Common\Collections\Collection;
use Doctrine\ORM\Mapping as ORM;

/**
 * @ORM\Entity(repositoryClass=PlayerRepository::class)
 * @ORM\Table(name="obblm_team_player")
 */
class Player
{
    /**
     * @ORM\Id()
     * @ORM\GeneratedValue()
     * @ORM\Column(type="integer")
     */
    private $id;

    use NameTrait, DeadAndFireTrait;

    /**
     * @ORM\ManyToOne(targetEntity=Team::class, inversedBy="players")
     * @ORM\JoinColumn(nullable=false)
     */
    private $team;

    /**
     * @ORM\Column(type="integer")
     */
    private $number;

    /**
     * @ORM\Column(type="string", length=50)
     */
    private $position;

    /**
     * @ORM\Column(type="string", length=255)
     */
    private $starPlayer = false;

    /**
     * @ORM\OneToMany(targetEntity=PlayerVersion::class, mappedBy="player", orphanRemoval=true, cascade={"remove"})
     * @ORM\OrderBy({"id"="DESC"})
     */
    private $versions;

    public function __construct()
    {
        $this->versions = new ArrayCollection();
    }

    public function getId(): ?int
    {
        return $this->id;
    }

    public function getTeam(): ?Team
    {
        return $this->team;
    }

    public function setTeam(?Team $team): self
    {
        $this->team = $team;

        return $this;
    }

    public function getNumber(): ?int
    {
        return $this->number;
    }

    public function setNumber(int $number): self
    {
        $this->number = $number;

        return $this;
    }

    public function getPosition(): ?string
    {
        return $this->position;
    }

    public function setPosition(?string $position): self
    {
        $this->position = $position;

        return $this;
    }

    public function getStarPlayer(): ?bool
    {
        return $this->starPlayer;

The expression return $this->starPlayer returns the type string which is incompatible with the type-hinted return boolean|null.

    }

    public function isStarPlayer(): ?bool
    {
        return $this->starPlayer;

The expression return $this->starPlayer returns the type string which is incompatible with the type-hinted return boolean|null.

    }

    public function setStarPlayer(bool $starPlayer): self
    {
        $this->starPlayer = $starPlayer;

The property $starPlayer was declared of type string, but $starPlayer is of type boolean. Maybe add a type cast?

        return $this;
    }

    /**
     * @return Collection|PlayerVersion[]
     */
    public function getVersions(): Collection
    {
        return $this->versions;
    }

    public function addVersion(PlayerVersion $version): self
    {
        if (!$this->versions->contains($version)) {
            $this->versions[] = $version;
            $version->setPlayer($this);
        }

        return $this;
    }

    public function removeVersion(PlayerVersion $version): self
    {
        if ($this->versions->contains($version)) {
            $this->versions->removeElement($version);
            // set the owning side to null (unless already changed)
            if ($version->getPlayer() === $this) {
                $version->setPlayer(null);
            }
        }

        return $this;
    }
}