2017-02-22 17 views
0

Я связываюсь с 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"); 
+0

Большой вопрос и большое объяснение! Почему кто-то проголосовал бы за это отрицательно? – mburakergenc

ответ

4

Вы обновляете класс сущности (объект). Чтобы сохранить его в базе данных, вы должны использовать службу менеджера доктрины.

в контроллере вы делаете что-то вроде этого: doc

$em = $this->getDoctrine()->getManager(); 

// tells Doctrine you want to (eventually) save the Product (no queries yet) 
$em->persist($product); 

// actually executes the queries (i.e. the INSERT query) 
$em->flush(); 

, но вы не в контроллере, так что вы можете вводить услугу доктрины в конструкторе и сохраняться ваше лицо на мероприятии.

<?php 
namespace AppBundle\EventListener; 

use Symfony\Component\Security\Core\Event\AuthenticationEvent; 
use Symfony\Component\Security\Core\User\AdvancedUserInterface; 

class LoginListener { 
    protected $doctrine; 

    public function __construct($doctrine){ 
     $this->doctrine = $doctrine; 
    } 

    public function onSuccessfulLogin(AuthenticationEvent $event) 
    { 
     $user = $event->getAuthenticationToken()->getUser(); 
     if($user instanceof AdvancedUserInterface){ 
      $user->updateLastLogin(); 
      $this->doctrine->getManager()->persist($user); 
      $this->doctrine->getManager()->flush(); 
     } 
    } 

    public function onLoginError(AuthenticationEvent $event) 
    { 
     // Login error 
    } 
} 

и в своем определении сервиса:

services: 
    app.login_service: 
     class:  AppBundle\EventListener\LoginListener 
     arguments: [doctrine] 
+0

Спасибо @mdma, но я попробовал это, и он дает мне «вызов функции-члена getManager() на строку« error now » –

+1

попробуй больше, у тебя есть отличный ответ здесь! – LBA

+1

Альберто, как @LBA, я считаю, что это идеальный ответ. Убедитесь, что вы ввели свой код точно так, как показано в этом ответе. Похоже, у вас где-то есть опечатка. –

Смежные вопросы