X-Git-Url: https://git.rapsys.eu/userbundle/blobdiff_plain/bab59a4b88a081a7a27a53b4559d74e63b68db92..7c1cf77e5008890f143f00e0b7a931ef7d7a77fb:/Entity/User.php?ds=inline diff --git a/Entity/User.php b/Entity/User.php index c6a6cdf..7d274f9 100644 --- a/Entity/User.php +++ b/Entity/User.php @@ -1,9 +1,28 @@ - + * + * for the full copyright and license information, please view the license + * file that was distributed with this source code. + */ -// src/Rapsys/UserBundle/Entity/User.php namespace Rapsys\UserBundle\Entity; -class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterface, \Serializable { +use Doctrine\Common\Collections\ArrayCollection; +use Doctrine\ORM\Event\PreUpdateEventArgs; +use Symfony\Component\Security\Core\User\UserInterface; +use Symfony\Component\Security\Core\User\PasswordAuthenticatedUserInterface; + +use Rapsys\UserBundle\Entity\Civility; +use Rapsys\UserBundle\Entity\Group; + +/** + * User + */ +class User implements UserInterface, PasswordAuthenticatedUserInterface { /** * @var integer */ @@ -14,11 +33,6 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac */ protected $mail; - /** - * @var string - */ - protected $pseudonym; - /** * @var string */ @@ -39,6 +53,11 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac */ protected $active; + /** + * @var bool + */ + protected $disabled; + /** * @var \DateTime */ @@ -50,21 +69,41 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac protected $updated; /** - * @var \Rapsys\UserBundle\Entity\Title + * @var Civility */ - protected $title; + protected $civility; /** - * @var \Doctrine\Common\Collections\Collection + * @var ArrayCollection */ protected $groups; /** - * User constructor. - */ - public function __construct() { + * Constructor + * + * @param ?string $mail The user mail + */ + public function __construct(?string $mail = null) { + //With mail + if ($mail !== null && !empty($mail)) { + $this->mail = $mail; + $this->password = $mail; + //Without mail + } else { + $this->mail = ''; + $this->password = ''; + } + + //Set defaults + $this->forename = null; + $this->surname = null; $this->active = false; - $this->groups = new \Doctrine\Common\Collections\ArrayCollection(); + $this->disabled = false; + $this->created = new \DateTime('now'); + $this->updated = new \DateTime('now'); + + //Set collections + $this->groups = new ArrayCollection(); } /** @@ -72,19 +111,31 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac * * @return integer */ - public function getId() { + public function getId(): int { return $this->id; } /** * Set mail * - * @param string $mail - * + * @param ?string $mail * @return User */ - public function setMail($mail) { - $this->mail = $mail; + public function setMail(?string $mail): User { + //With mail + if ($mail !== null && !empty($mail)) { + //Set mail + $this->mail = $mail; + + //Without password + if (empty($this->password)) { + //Set mail as password + $this->password = $mail; + } + //Without mail + } else { + $this->mail = ''; + } return $this; } @@ -94,40 +145,18 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac * * @return string */ - public function getMail() { + public function getMail(): string { return $this->mail; } - /** - * Set pseudonym - * - * @param string $pseudonym - * - * @return User - */ - public function setPseudonym($pseudonym) { - $this->pseudonym = $pseudonym; - - return $this; - } - - /** - * Get pseudonym - * - * @return string - */ - public function getPseudonym() { - return $this->pseudonym; - } - /** * Set forename * - * @param string $forename + * @param ?string $forename * * @return User */ - public function setForename($forename) { + public function setForename(?string $forename): User { $this->forename = $forename; return $this; @@ -136,20 +165,20 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac /** * Get forename * - * @return string + * @return ?string */ - public function getForename() { + public function getForename(): ?string { return $this->forename; } /** * Set surname * - * @param string $surname + * @param ?string $surname * * @return User */ - public function setSurname($surname) { + public function setSurname(?string $surname): User { $this->surname = $surname; return $this; @@ -158,9 +187,9 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac /** * Get surname * - * @return string + * @return ?string */ - public function getSurname() { + public function getSurname(): ?string { return $this->surname; } @@ -171,8 +200,14 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac * * @return User */ - public function setPassword($password) { - $this->password = $password; + public function setPassword(?string $password): User { + //With password + if ($password !== null && !empty($password)) { + $this->password = $password; + //Without password + } else { + $this->password = ''; + } return $this; } @@ -180,9 +215,11 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac /** * Get password * + * {@inheritdoc} + * * @return string */ - public function getPassword() { + public function getPassword(): string { return $this->password; } @@ -193,7 +230,7 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac * * @return User */ - public function setActive($active) { + public function setActive(bool $active): User { $this->active = $active; return $this; @@ -204,10 +241,32 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac * * @return bool */ - public function getActive() { + public function getActive(): bool { return $this->active; } + /** + * Set disabled + * + * @param bool $disabled + * + * @return User + */ + public function setDisabled(bool $disabled): User { + $this->disabled = $disabled; + + return $this; + } + + /** + * Get disabled + * + * @return bool + */ + public function getDisabled(): bool { + return $this->disabled; + } + /** * Set created * @@ -215,7 +274,7 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac * * @return User */ - public function setCreated($created) { + public function setCreated(\DateTime $created): User { $this->created = $created; return $this; @@ -226,7 +285,7 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac * * @return \DateTime */ - public function getCreated() { + public function getCreated(): \DateTime { return $this->created; } @@ -237,7 +296,7 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac * * @return User */ - public function setUpdated($updated) { + public function setUpdated(\DateTime $updated): User { $this->updated = $updated; return $this; @@ -248,34 +307,34 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac * * @return \DateTime */ - public function getUpdated() { + public function getUpdated(): \DateTime { return $this->updated; } /** - * Set title + * Set civility */ - public function setTitle($title) { - $this->title = $title; + public function setCivility(Civility $civility): User { + $this->civility = $civility; return $this; } /** - * Get title + * Get civility */ - public function getTitle() { - return $this->title; + public function getCivility(): ?Civility { + return $this->civility; } /** * Add group * - * @param \Rapsys\UserBundle\Entity\Group $group + * @param Group $group * * @return User */ - public function addGroup(\Rapsys\UserBundle\Entity\Group $group) { + public function addGroup(Group $group) { $this->groups[] = $group; return $this; @@ -284,72 +343,184 @@ class User implements \Symfony\Component\Security\Core\User\AdvancedUserInterfac /** * Remove group * - * @param \Rapsys\UserBundle\Entity\Group $group + * @param Group $group */ - public function removeGroup(\Rapsys\UserBundle\Entity\Group $group) { + public function removeGroup(Group $group) { $this->groups->removeElement($group); } /** * Get groups * - * @return \Doctrine\Common\Collections\Collection + * @return ArrayCollection */ - public function getGroups() { + public function getGroups(): ArrayCollection { return $this->groups; } - public function getRoles() { - return $this->groups->toArray(); + /** + * {@inheritdoc} + */ + public function getRoles(): array { + //Get the unique roles list by id + return array_unique(array_reduce( + //Cast groups as array + $this->groups->toArray(), + //Reduce to an array of id => group tuples + function ($array, $group) { + $array[$group->getId()] = $group->getRole(); + return $array; + }, + //Init with empty array + //XXX: on registration, add each group present in rapsys_user.default.group array to user + //XXX: see vendor/rapsys/userbundle/Controller/DefaultController.php +450 + [] + )); + } + + /** + * {@inheritdoc} + */ + public function getRole(): ?string { + //Retrieve roles + $roles = $this->getRoles(); + + //With roles array empty + if ($roles === []) { + //Return null + return null; + } + + //Return the role with max id + //XXX: should be rewriten if it change in your configuration + return $roles[array_reduce( + array_keys($roles), + function($cur, $id) { + if ($cur === null || $id > $cur) { + return $id; + } + return $cur; + }, + null + )]; } - public function getSalt() { + /** + * {@inheritdoc} + */ + public function getSalt(): ?string { //No salt required with bcrypt return null; } - public function getUsername() { + /** + * {@inheritdoc} + */ + public function getUsername(): string { return $this->mail; } - public function eraseCredentials() { + /** + * {@inheritdoc} + */ + public function getUserIdentifier(): string { + return $this->mail; } - public function serialize() { - return serialize(array( + /** + * {@inheritdoc} + */ + public function eraseCredentials(): void {} + + /** + * {@inheritdoc} + */ + public function __serialize(): array { + return [ $this->id, $this->mail, + $this->forename, + $this->surname, $this->password, $this->active, + $this->disabled, $this->created, $this->updated - )); + ]; } - public function unserialize($serialized) { + /** + * {@inheritdoc} + */ + public function __unserialize(array $data): void { list( $this->id, $this->mail, + $this->forename, + $this->surname, $this->password, $this->active, + $this->disabled, $this->created, $this->updated - ) = unserialize($serialized); + ) = $data; } - public function isAccountNonExpired() { - return true; + /** + * Check if account is activated + * + * It was from deprecated AdvancedUserInterface, see if it's used anymore + * + * @see vendor/symfony/security-core/User/AdvancedUserInterface.php + */ + public function isActivated(): bool { + return $this->active; } - public function isAccountNonLocked() { - return true; + /** + * Check if account is disabled + * + * It was from deprecated AdvancedUserInterface, see if it's used anymore + * + * @see vendor/symfony/security-core/User/AdvancedUserInterface.php + */ + public function isDisabled(): bool { + return $this->disabled; } - public function isCredentialsNonExpired() { - return true; + /** + * {@inheritdoc} + */ + public function preUpdate(PreUpdateEventArgs $eventArgs) { + //Check that we have an user instance + if (($user = $eventArgs->getEntity()) instanceof User) { + //Set updated value + $user->setUpdated(new \DateTime('now')); + } } - public function isEnabled() { - return $this->active; + /** + * Returns a recipient name of the user + * + * @return string + */ + public function getRecipientName(): string { + //Without forename and surname + if (empty($this->forename) && empty($this->surname)) { + //Return recipient name from mail + return ucwords(trim(preg_replace('/[^a-zA-Z]+/', ' ', current(explode('@', $this->mail))))); + } + + //Return recipient name from forename and surname + return implode(' ', [$this->forename, $this->surname]); + } + + /** + * Returns a string representation of the user + * + * @return string + */ + public function __toString(): string { + return $this->civility.' '.$this->forename.' '.$this->surname; } }