Symfony2 用户密码更改

Question

Good morning, i'm trying to do a password change form but without any success so far i've been doing research on this part and i didn't find any useful results on google so i came here :D

This is my controller action for password change

/**
 *
 * @Route("/user/change-password", name="change_password")
 * @Template()
 *
 */
public function changePasswordAction(Request $request) {
    $changePasswordModel = new ChangePassword();

    $form = $this->createForm(new ChangePasswordType(), $changePasswordModel);

    $form->handleRequest($request);

    if($form->isValid()) {
        $changePassword = $form->getData();

        $factory = $this->get('security.encoder_factory');
        $encoder = $factory->getEncoder($changePassword->getUser());

        var_dump($encoder);
        $password = $encoder->encodePassword($changePassword->getUser()->getPlainPassword(), $changePassword->getUser()->getSalt());

        $changePassword->getUser()->setPassword($password);

        $em->persist($changePassword->getUser());
        $em->flush();

        return $this->redirect($this->generateUrl('change_password_success'));
    }

    return array(
            'form' => $form->createView(),
        );
}

Password Change Model

/**
 * @Assert\Type(type="AppBundle\Entity\User")
 * @Assert\Valid()
 */
protected $user;

/**
 * @SecurityAssert\UserPassword(
 *     message = "Wrong value for your current password"
 * )
 */
 protected $password;

/**
 * @Assert\Length(
 *     min = 6,
 *     minMessage = "Password should by at least 6 chars long"
 * )
 */
 protected $plainPassword;

public function setUser(User $user)
{
    $this->user = $user;
}

public function getUser()
{
    return $this->user;
}

public function setPassword($password) {

    $this->password = $password;

    return $this;
}

public function getPassword() {

    return $this->password;
}

public function setPlainPassword($plainPassword) {

    $this->plainPassword = $plainPassword;

    return $this;
}

public function getPlainPassword() {

    return $this->plainPassword;
}

Password Type

public function buildForm(FormBuilderInterface $builder, array $options)
{
    $builder->add('password', 'password');
    $builder->add('plainPassword', 'repeated', array(
        'type' => 'password',
        'invalid_message' => 'The password fields must match.',
        'required' => true,
        'first_options'  => array('label' => 'Password'),
        'second_options' => array('label' => 'Repeat Password'),
    ));
    $builder->add('ChangePassword', 'submit');
}

My current error is :

No encoder has been configured for account "".
500 Internal Server Error - RuntimeException 

Forgot to add the security.yml

    encoders:
    AppBundle\Entity\User:
        algorithm: bcrypt
        cost: 15
# http://symfony.com/doc/current/book/security.html#where-do-users-come-from-user-providers
providers:
    in_memory:
        memory: ~
    db_provider:
        entity:
            class: AppBundle\Entity\User

Answer 1

With your current code you never set the actual user for which you want to change the password. Because of this, the following line fails, as null is passed as the argument:

 $factory->getEncoder($changePassword->getUser());

If you want to stick to your current implemenation, you only need to realize that ChangePassword is actually just a data transfer object which needs to be set up correctly. Parts of this setup will come through the form handling ( ChangePasswordType ) other parts have to be set explicitly by you. So, for example just add:

$changePasswordModel = new ChangePassword();
$changePasswordModel->setUser($this->getUser());

Personally I think this aproach is a good idea because it makes the code more semantic (or expressive if you will). Just remember to not assume to much about implicit magic on the side of symfony.