Я связываюсь с Symfony, это мое первое приложение, и я пытаюсь понять всю окружающую среду. Это очень простой вопрос, но после долгого серфинга в Интернете я не понял, как это сделать.Обновление данных пользователя после входа в систему Symfony 3
У меня есть форма входа, работающая очень хорошо с слушателем, чтобы обрабатывать успех или ошибку, все работает хорошо, но проблема возникает, когда я пытаюсь обновить последнюю дату доступа для пользователя.
У меня есть объект пользователя (взятый из документации):
<?php
namespace AppBundle\Entity;
use Doctrine\ORM\Mapping as ORM;
use Symfony\Component\Security\Core\User\AdvancedUserInterface;
/**
* @ORM\Table(name="app_users")
* @ORM\Entity(repositoryClass="AppBundle\Repository\UserRepository")
*/
class User implements AdvancedUserInterface, \Serializable
{
/**
* @ORM\Column(type="integer")
* @ORM\Id
* @ORM\GeneratedValue(strategy="AUTO")
*/
private $id;
/**
* @ORM\Column(type="string", length=25, unique=true)
*/
private $username;
/**
* @ORM\Column(type="string", length=64)
*/
private $password;
/**
* @ORM\Column(type="string", length=60, unique=true)
*/
private $email;
/**
* @ORM\Column(name="is_active", type="boolean")
*/
private $isActive;
/**
* @ORM\Column(name="salt", type="string", length=100)
*/
private $salt;
/**
* @ORM\Column(type="datetime", nullable=true)
*/
private $lastLogin = null;
/**
* @ORM\Column(type="datetime", nullable=true)
*/
private $createdAt = null;
public function __construct()
{
$this->isActive = true;
// may not be needed, see section on salt below
// $this->salt = md5(uniqid(null, true));
}
public function getUsername()
{
return $this->username;
}
public function setSalt($salt)
{
$this->salt = $salt;
}
public function getSalt()
{
// you *may* need a real salt depending on your encoder
// see section on salt below
return null;
}
public function getPassword()
{
return $this->password;
}
public function getRoles()
{
return array('ROLE_USER');
}
public function eraseCredentials()
{
}
public function isAccountNonExpired()
{
return true;
}
public function isAccountNonLocked()
{
return true;
}
public function isCredentialsNonExpired()
{
return true;
}
public function isEnabled()
{
return $this->isActive;
}
/** @see \Serializable::serialize() */
public function serialize()
{
return serialize(array(
$this->id,
$this->username,
$this->password,
$this->isActive,
$this->createdAt,
$this->lastLogin,
// see section on salt below
// $this->salt,
));
}
/** @see \Serializable::unserialize() */
public function unserialize($serialized)
{
list (
$this->id,
$this->username,
$this->password,
$this->isActive,
$this->createdAt,
$this->lastLogin,
// see section on salt below
// $this->salt
) = unserialize($serialized);
}
/**
* Get id
*
* @return integer
*/
public function getId()
{
return $this->id;
}
/**
* Set username
*
* @param string $username
*
* @return User
*/
public function setUsername($username)
{
$this->username = $username;
return $this;
}
/**
* Set password
*
* @param string $password
*
* @return User
*/
public function setPassword($password)
{
$this->password = $password;
return $this;
}
/**
* Set email
*
* @param string $email
*
* @return User
*/
public function setEmail($email)
{
$this->email = $email;
return $this;
}
/**
* Get email
*
* @return string
*/
public function getEmail()
{
return $this->email;
}
/**
* Set isActive
*
* @param boolean $isActive
*
* @return User
*/
public function setIsActive($isActive)
{
$this->isActive = $isActive;
return $this;
}
/**
* Get isActive
*
* @return boolean
*/
public function getIsActive()
{
return $this->isActive;
}
/**
* Updates last user login date
*
*/
public function updateLastLogin()
{
$this->lastAccess = new \DateTime("now");
}
public function getLastLogin()
{
return $this->lastLogin->format('Y-m-d H:i:s');
}
public function getCreatedAt()
{
return $this->createdAt->format('Y-m-d H:i:s');
}
}
Когда пользователь входит в систему я называю updateLastLogin
метод:
<?php
namespace AppBundle\EventListener;
use Symfony\Component\Security\Core\Event\AuthenticationEvent;
use Symfony\Component\Security\Core\User\AdvancedUserInterface;
class LoginListener {
public function __construct(){
}
public function onSuccessfulLogin(AuthenticationEvent $event)
{
$user = $event->getAuthenticationToken()->getUser();
if($user instanceof AdvancedUserInterface){
$user->updateLastLogin();
}
}
public function onLoginError(AuthenticationEvent $event)
{
// Login error
}
}
updateLastLogin
код достигается, но это Безразлично» t получить обновление в базе данных. Может ли кто-нибудь сказать мне, что я делаю неправильно?
Спасибо заранее
@mdma ответ приведет меня к правильному решению, его решение, которое я получил ошибку Call to a member function getManager() on string
. Чтобы решить эту проблему, я просто ввел EntityManager
в конструкторе слушателя и добавил недостающий вызов сохранения данных в базе данных:
<?php
namespace AppBundle\EventListener;
use Doctrine\ORM\EntityManager;
use Symfony\Component\Security\Core\Event\AuthenticationEvent;
use Symfony\Component\Security\Core\User\AdvancedUserInterface;
class LoginListener {
protected $entityManager;
public function __construct(EntityManager $entityManager){
$this->entityManager = $entityManager;
}
public function onSuccessfulLogin(AuthenticationEvent $event)
{
$user = $event->getAuthenticationToken()->getUser();
if($user instanceof AdvancedUserInterface){
$user->updateLastLogin();
$this->entityManager->persist($user);
$this->entityManager->flush();
}
}
public function onLoginError(AuthenticationEvent $event)
{
// Login error
}
}
И в файле services.yml
я добавил его в качестве аргумента:
app.login_service:
class: AppBundle\EventListener\LoginListener
arguments: ["@doctrine.orm.entity_manager"]
tags:
- { name: kernel.event_listener, event: security.authentication.success, method: onSuccessfulLogin }
Теперь он работает так, как ожидалось.
Обратите внимание, что существует опечатка в классе пользователей при обновлении даты входа, я использовал переменную, которая не существует. Вместо $this->lastAccess = new \DateTime("now");
это должно быть:
$this->lastLogin = new \DateTime("now");
Большой вопрос и большое объяснение! Почему кто-то проголосовал бы за это отрицательно? – mburakergenc