AppBundle\Entity\User

Complexity

Total Complexity

13

Size/Duplication

Total Lines	159
Duplicated Lines	0 %

Coupling/Cohesion

Components	3
Dependencies	3

Test Coverage

Coverage

80%

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	20	3
A	getRoles()	0	9	1
A	setVkToken()	0	4	1
A	updateToken()	0	4	1
A	getPassword()	0	1	1
A	getSalt()	0	1	1
A	eraseCredentials()	0	1	1
A	getUsername()	0	4	1
A	getToken()	0	4	1
A	getName()	0	4	1

<?php

namespace AppBundle\Entity;

use AppBundle\Entity\Infrasctucture\FormattedRegistrationDateTrait;
use AppBundle\Service\HashGenerator;
use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\ORM\Mapping as ORM;
use JMS\Serializer\Annotation as Serializer;
use JMS\Serializer\Annotation\Accessor;
use JMS\Serializer\Annotation\SerializedName;
use JMS\Serializer\Annotation\Type as SerializerType;
use Symfony\Component\Security\Core\User\UserInterface;

/**
 * User
 * @ORM\Table(name="users")
 * @ORM\Entity(repositoryClass="AppBundle\Entity\Repository\UserRepository")
 */
class User implements UserInterface
{

    use FormattedRegistrationDateTrait;

    const TOKEN_LENGTH = 32;

    /**
     * @var string
     * @ORM\Id
     * @ORM\Column(name="login", type="string", length=255, nullable=false)
     */
    private $login;

    /**
     * @var string
     * @ORM\Column(name="name", type="string", length=255, nullable=false, unique=true)
     */
    private $name;

    /**
     * @var string
     * @ORM\Column(name="email", type="string", length=255, nullable=true, unique=true)
     * @Serializer\Exclude()
     */
    private $email;

    /**
     * @var string
     * @ORM\Column(name="description", type="text", nullable=true)
     */
    private $description;

    /**
     * @var string
     * @ORM\Column(name="vkontakte_id", type="string", length=50, nullable=false, unique=true)
     * @Serializer\Exclude()
     */
    private $vkontakteId;

    /**
     * @var string
     * @ORM\Column(name="vk_token", type="string", length=85, nullable=false)
     * @Serializer\Exclude()
     */
    private $vkToken;

    /**
     * @var string
     * @ORM\Column(name="token", type="string", length=32, nullable=false, unique=true)
     * @Serializer\Exclude()
     */
    private $token;

    /**
     * @var \DateTime
     * @ORM\Column(name="registration_date", type="datetime", nullable=false)
     * @SerializedName("registration_date")
     * @Accessor(getter="getRegistrationDate")
     * @SerializerType("string")
     */
    private $registrationDate;

    /**
     * @var array
     * @ORM\Column(name="roles", type="simple_array", nullable=true)
     * @Serializer\Exclude()
     */
    private $roles;

    /**
     * @var Event[]|ArrayCollection
     * @ORM\OneToMany(targetEntity="AppBundle\Entity\Event", mappedBy="creator")
     * @SerializerType("array")
     */
    private $events;

    /**
     * @var Band[]|ArrayCollection
     * @ORM\OneToMany(targetEntity="AppBundle\Entity\Band", mappedBy="creator")
     * @SerializerType("array")
     */
    private $createdBands;

    public function __construct(
        string $login,
        string $name,
        int $vkontakteId,
        string $vkToken,
        string $email = null,
        string $description = null,
        string $token = null
    ) {
        $this->login = $login;
        $this->name = $name;
        $this->vkontakteId = $vkontakteId;

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

        $this->vkToken = $vkToken;
        $this->token = $token ?: HashGenerator::generate(self::TOKEN_LENGTH);
        $this->email = $email ?: null;
        $this->description = $description;
        $this->registrationDate = new \DateTime();
        $this->events = new ArrayCollection();
        $this->createdBands = new ArrayCollection();
    }

    /**
     * @return string
     */
    public function getLogin(): string
    {
        return $this->login;
    }

    /** {@inheritDoc} */
    public function getRoles(): array
    {
        return array_merge(
            $this->roles,
            [
                'ROLE_USER',
            ]
        );
    }

    public function setVkToken(string $vkToken)
    {
        $this->vkToken = $vkToken;
    }

    public function updateToken()
    {
        $this->token = HashGenerator::generate(self::TOKEN_LENGTH);
    }

    public function getToken(): string
    {
        return $this->token;
    }

    /** {@inheritDoc} */
    public function getPassword() {}

    /** {@inheritDoc} */
    public function getSalt() {}

    /** {@inheritDoc} */
    public function eraseCredentials() {}

    /** {@inheritDoc} */
    public function getUsername(): string
    {
        return $this->getLogin();
    }

    public function getName(): string
    {
        return $this->name;
    }
}