Error Formatting for Twitter Bootstrap

Hey Junaid Farooq!

So... do you mean that form validation IS working, but you don't see the bootstrap styles? Or something different? If so, if you view the HTML source, do you see classes like form-group and form-control in your HTML, or not?

Cheers!

Hey weaverryan

Thanks for replying. I meant my server side form validation is not working. No errors are shown. For example if i disable the front end validation and then try to submit an empty form, it doesn't validate the form from the server side instead it tries to insert the data into the database which then throws a db exception. I hope you get what i mean. Cheers!

Hey Junaid,

What validation constraints do you use? And where do you declare it? Because, by default, Symfony does not add some validation on the server side for empty fields, you need to add it by your own.

Cheers!

Hey victor

Thanks for replying. I am using the constraints in the User entity. Following is an example of sych for Username and email:

class User implements AdvancedUserInterface, \Serializable
..................
/**
* @var string
*
* @ORM\Column(name="username", type="string", length=255, unique=true)
* @Assert\NotBlank(message="Username cannot be empty")
* @Assert\Length(min=3, minMessage="Username should be at least 3 characters!")
*/
private $username;

/**
* @var string
*
* @ORM\Column(name="email", type="string", length=255, unique=true)
* @Assert\NotBlank(message="Email cannot be empty")
* @Assert\Email
*
*/
private $email;
...................

Cheers!

Hey Junaid,

Hm, looks legitimate for me if you use the next namespace in User entity:
use Symfony\Component\Validator\Constraints as Assert;

So it really should work. Do you use "$form->isSubmitted() && $form->isValid()" in your controller? Well, try to clear the cache first and if you still have this problem, can you show your form type you use and a part of your controller which relates to this form?

Cheers!

Hey victor

Again Thank you for your prompt response. Back to your response. yes I am using the namespaces required in the User Entity. here are all of them:

use Doctrine\ORM\Mapping as ORM;
use Gedmo\Mapping\Annotation as Gedmo;
use Symfony\Component\Security\Core\User\AdvancedUserInterface;
use Symfony\Bridge\Doctrine\Validator\Constraints\UniqueEntity;
use Symfony\Component\Validator\Constraints as Assert;
use Doctrine\Common\Collections\ArrayCollection;

//...........
class User implements AdvancedUserInterface, \Serializable
//...........

To your question: Yes i am using "$form->isSubmitted().....".Here is the whole registerAction():

// ...........
class RegisterController extends Controller
// .........................
public function registerAction(Request $request)
    {
        $form = $this->createForm(new RegisterFormType());

        $form->handleRequest($request);
        if ($form->isSubmitted() && $form->isValid()) {
            $data = $form->getData();

            $user = new User();
            $user->setUsername($data['username']);
            $user->setEmail($data['email']);
            $user->setPlainPassword($data['plainPassword']);

            $em = $this->getDoctrine()->getManager();
            $em->persist($user);
            $em->flush();

            $request->getSession()
                ->getFlashBag()
                ->add('success', 'User Registered Successfully!');

            $this->authenticateUser($user);

            $url = $this->generateUrl('event');
            return $this->redirect($url);

        }

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

Here is my RegisterFormType code:

namespace Yoda\UserBundle\Form;

use Symfony\Component\Form\AbstractType;
use Symfony\Component\Form\FormBuilderInterface;

class RegisterFormType extends AbstractType
{
    /**
     * @param FormBuilderInterface $builder
     * @param array $options
     */
    public function buildForm(FormBuilderInterface $builder, array $options)
    {
        $builder->add('username', 'text', array(
                'required'  =>  true
                ))
            ->add('email', 'email', array(
                'required'  =>  true
            ))
            ->add('plainPassword', 'repeated', array(
                'type'  =>  'password',
                'required'  =>  true,
                'first_options'  => array('label' => 'Password'),
                'second_options' => array('label' => 'Repeat Password'),
            ));
    }

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

}
// ................

Please have a look and thank you once again for reaching out. Cheers!

Hey Junaid,

Hm, I see a few problems in your code which could cause this validation problems, so let me tweak your code to the best practice and if you're wondering about any changes I made - just ask another question ;) For simplicity, if I only change things of add new ones, so if I'm not sowing something in my code - this means it's the same as in yours.

So, first of all your need to bind RegisterFormType to the entity:

class RegisterFormType extends AbstractType
{
    // ...
    public function configureOptions(OptionsResolver $resolver)
    {
        $resolver->setDefaults([
            'data_class' => User::class,
        ]);
    }
}

public function registerAction(Request $request)
{
    $user = new User();
    $form = $this->createForm(RegisterFormType::class, $user);

    $form->handleRequest($request);
    if ($form->isSubmitted() && $form->isValid()) {
        $em = $this->getDoctrine()->getManager();
        $em->persist($user);
        $em->flush();

        $request->getSession()
            ->getFlashBag()
            ->add('success', 'User Registered Successfully!');

        $this->authenticateUser($user);

        return $this->redirectToRoute('event');
    }

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

So, it should work now, let me know if it does not help. I suppose the problem is that your don't have this configureOptions() method, so validator does not see your validation constrains ;) Also, don't forget to clear the cache after those changes.

Cheers!