Symfony2 - Cannot set password form field to type "password"

Question

I have build a form class to generate a form that will allow a user to type in a new password twice in order to change it.

code:

<?php

namespace UserBundle\Form\Type;

use Symfony\Component\Form\AbstractType;
use Symfony\Component\Form\FormBuilder;
use Symfony\Component\Form\FormError;
use Symfony\Component\Form\CallbackValidator;

class PasswordType extends AbstractType
{
    public function buildForm(FormBuilder $builder, array $options)
    {
        $builder->add('password', null);
        $builder->add('confirmPassword', null, array('label' => 'Confirm Password', 'property_path' => false));
        $builder->addValidator(new CallbackValidator(function($form)
            {
                if($form['confirmPassword']->getData() != $form['password']->getData()) {
                    $form['confirmPassword']->addError(new FormError('Passwords must match.'));
                }
            }));
    }

    public function getDefaultOptions(array $options)
    {
        return array(
            'data_class' => 'UserBundle\Entity\User',
        );
    }

    public function getName()
    {
        return 'password';
    }
}

Now, this class works pretty well, BUT my issues is that when I set the first password field to type "password" I get this error:

Circular reference detected in the "password" type (defined in class "UserBundle\Form\Type\PasswordType").

And I cannot leave it set to "null" as it will use a normal text input field, which is not ideal.

Any ideas folks?

meze · Accepted Answer · 2012-02-14 12:21:13Z

4

You defined PasswordType form type field. Then you added a field of Password type inside it. That caused another call to PasswordType::buildForm() to add another field of this type and it goes forever.

The solution is to rename your password type to something like 'userpassword' or you can use repeated field type instead. It will add two fields and do comparison validation of thier values.

edited Feb 14, 2012 at 12:21

answered Feb 14, 2012 at 11:35

meze

15.1k4 gold badges49 silver badges53 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Mr Pablo Over a year ago

The value "password" in the first option of $builder->add() is the field name, not it's type.

meze Over a year ago

@MrPablo i know. "set the first password field to type "password"" so you set the second argument which is its type.

Martin · Accepted Answer · 2012-02-14 12:07:02Z

1

You can use repeated field type instead of your solution, see http://symfony.com/doc/2.0/reference/forms/types/repeated.html

answered Feb 14, 2012 at 12:07

Martin

87710 silver badges10 bronze badges

4 Comments

Mr Pablo Over a year ago

Looked at using repeat, but there are currently issues with the form errors it produces, so I am staying away form it for now :) I figured out the answer, but I cannot post it for 8 hours, but here it is: The issue seems to be from using "password" as the getName() value. I changed this to "pass" and now I can use "password" to set the type, i.e. $builder->add('password', 'password');

meze Over a year ago

@MrPablo that's what i told you in my answer ;p

Theodore R. Smith Over a year ago

@MrPablo That's exactly what meze said ;o

Mr Pablo Over a year ago

Sorry, but it had nothing to do with the $builder->add() part. Meze said I had defined the field type as password, but if you check, it is set to "null". The issues was getName(). Using getName() as "password" was causing the problem, I changed that and THEN I changed $builder->add('password', null) to $builder->add('password', 'password') and it worked. Meze was wrong.

Mr Pablo · Accepted Answer · 2012-02-15 10:36:25Z

0

The issue seems to be from using "password" as the getName() value.

Before:

public function getName()
{
    return 'password';
}

After:

public function getName()
{
    return 'changePassword'; // basically anything other than 'password'
}

I changed this to "pass" and now I can use "password" to set the type, i.e.

$builder->add('password', 'password');

answered Feb 15, 2012 at 10:36

Mr Pablo

4,1869 gold badges56 silver badges111 bronze badges

Collectives™ on Stack Overflow

Symfony2 - Cannot set password form field to type "password"

3 Answers 3

2 Comments

4 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

4 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related