Doctrine Event Listeners

Hello again...

// src/Yoda/UserBundle/Doctrine/UserListener.php
// ...
use Yoda\UserBundle\Entity\User;
// ...
private function handleEvent(User $user)
{
$plainPassword = $user->getPlainPassword();
$encoder = $this->container->get('security.encoder_factory')->getEncoder($user);
$password = $encoder->encodePassword($plainPassword(), $user->getSalt());
$user->setPassword($password);
}

Remove brackets after $plainPassword please.

also it will be great if you could be more consistent and include RegisterController changes into the lesson. It is simple enough but may be useful for some students. So it maybe here just to be sure all on the same wave after the lesson.

Hey Taco!

You're right in your thinking that it would make sense and be clean, etc. BUT, there is a practical problem: we don't have access to the service (security.encoder_factory) that's needed to encode the password from inside setPlainPassword(). In fact, your entities never have access to services (this is by design - to keep your entities simple). That's why we do it in a listener. If we were doing something simpler (e.g. when you call `setEmail()`, maybe you want to call `setUsername()` to be this same value, for some reason - a silly example) then we would have done it this way.

Does that help?

# src/Yoda/UserBundle/Resources/config/services.yml
services:
doctrine.user_listener:
class: Yoda\UserBundle\Doctrine\UserListener
arugments: ["@security.encoder_factory"]
tags:
- { name: doctrine.event_listener, event: prePersist }

1. There is a small typo that results in an error:
arugments => arguments

2. Another mistake on this page is here:
// src/Yoda/UserBundle/DataFixtures/ORM/LoadUsers.php
// ...
public function load(ObjectManager $manager)
{
// ...
// $admin->setPassword($this->encodePassword($admin, 'waynepass'));
$user->setPlainPassword('waynepass');
}

For the admin ($user->setPlainPassword('waynepass');) should be changed to:
$admin->setPlainPassword('waynepass');