[Cookbook] Make registration_form follow best practices

Author: xelaris

cookbook/doctrine/registration_form.rst

@@ -10,13 +10,13 @@ database. For example, you may want to create a registration form with some
 extra fields (like a "terms accepted" checkbox field) and embed the form
 that actually stores the account information.
 
-The simple User Model
----------------------
+The simple User Entity
+----------------------
 
 You have a simple ``User`` entity mapped to the database::
 
-    // src/Acme/AccountBundle/Entity/User.php
-    namespace Acme\AccountBundle\Entity;
+    // src/AppBundle/Entity/User.php
+    namespace AppBundle\Entity;
 
     use Doctrine\ORM\Mapping as ORM;
     use Symfony\Component\Validator\Constraints as Assert;
@@ -45,7 +45,7 @@ You have a simple ``User`` entity mapped to the database::
         /**
          * @ORM\Column(type="string", length=255)
          * @Assert\NotBlank()
-         * @Assert\Length(max = 4096)
+         * @Assert\Length(max=4096)
          */
         protected $plainPassword;
 
@@ -76,9 +76,9 @@ You have a simple ``User`` entity mapped to the database::
     }
 
 This ``User`` entity contains three fields and two of them (``email`` and
-``plainPassword``) should display on the form. The email property must be unique
-in the database, this is enforced by adding this validation at the top of
-the class.
+``plainPassword``) should be displayed by the form. The email property must be
+unique in the database, this is enforced by adding an ``@UniqueEntity``
+validation constraint at the top of the class.
 
 .. note::
 
@@ -101,13 +101,13 @@ the class.
     only place where you don't need to worry about this is your login form,
     since Symfony's Security component handles this for you.
 
-Create a Form for the Model
----------------------------
+Create a Form for the Entity
+----------------------------
 
-Next, create the form for the ``User`` model::
+Next, create the form for the ``User`` entity::
 
-    // src/Acme/AccountBundle/Form/Type/UserType.php
-    namespace Acme\AccountBundle\Form\Type;
+    // src/AppBundle/Form/Type/UserType.php
+    namespace AppBundle\Form\Type;
 
     use Symfony\Component\Form\AbstractType;
     use Symfony\Component\Form\FormBuilderInterface;
@@ -119,16 +119,16 @@ Next, create the form for the ``User`` model::
         {
             $builder->add('email', 'email');
             $builder->add('plainPassword', 'repeated', array(
-               'first_name'  => 'password',
-               'second_name' => 'confirm',
-               'type'        => 'password',
+                'first_name'  => 'password',
+                'second_name' => 'confirm',
+                'type'        => 'password',
             ));
         }
 
         public function setDefaultOptions(OptionsResolverInterface $resolver)
         {
             $resolver->setDefaults(array(
-                'data_class' => 'Acme\AccountBundle\Entity\User'
+                'data_class' => 'AppBundle\Entity\User'
             ));
         }
 
@@ -156,17 +156,17 @@ be stored in the database.
 
 Start by creating a simple class which represents the "registration"::
 
-    // src/Acme/AccountBundle/Form/Model/Registration.php
-    namespace Acme\AccountBundle\Form\Model;
+    // src/AppBundle/Form/Model/Registration.php
+    namespace AppBundle\Form\Model;
 
     use Symfony\Component\Validator\Constraints as Assert;
 
-    use Acme\AccountBundle\Entity\User;
+    use AppBundle\Entity\User;
 
     class Registration
     {
         /**
-         * @Assert\Type(type="Acme\AccountBundle\Entity\User")
+         * @Assert\Type(type="AppBundle\Entity\User")
          * @Assert\Valid()
          */
         protected $user;
@@ -200,8 +200,8 @@ Start by creating a simple class which represents the "registration"::
 
 Next, create the form for this ``Registration`` model::
 
-    // src/Acme/AccountBundle/Form/Type/RegistrationType.php
-    namespace Acme\AccountBundle\Form\Type;
+    // src/AppBundle/Form/Type/RegistrationType.php
+    namespace AppBundle\Form\Type;
 
     use Symfony\Component\Form\AbstractType;
     use Symfony\Component\Form\FormBuilderInterface;
@@ -211,11 +211,9 @@ Next, create the form for this ``Registration`` model::
         public function buildForm(FormBuilderInterface $builder, array $options)
         {
             $builder->add('user', new UserType());
-            $builder->add(
-                'terms',
-                'checkbox',
-                array('property_path' => 'termsAccepted')
-            );
+            $builder->add('terms', 'checkbox', array(
+                'property_path' => 'termsAccepted',
+            ));
             $builder->add('Register', 'submit');
         }
 
@@ -233,28 +231,43 @@ of the ``User`` class.
 Handling the Form Submission
 ----------------------------
 
-Next, you need a controller to handle the form. Start by creating a simple
-controller for displaying the registration form::
+Next, you need a controller to handle the form rendering and submission. If the
+form is submitted, the controller performs the validation and saves the data
+into the database::
 
-    // src/Acme/AccountBundle/Controller/AccountController.php
-    namespace Acme\AccountBundle\Controller;
+    // src/AppBundle/Controller/AccountController.php
+    namespace AppBundle\Controller;
 
     use Symfony\Bundle\FrameworkBundle\Controller\Controller;
+    use Symfony\Component\HttpFoundation\Request;
+    use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
 
-    use Acme\AccountBundle\Form\Type\RegistrationType;
-    use Acme\AccountBundle\Form\Model\Registration;
+    use AppBundle\Form\Type\RegistrationType;
+    use AppBundle\Form\Model\Registration;
 
     class AccountController extends Controller
     {
-        public function registerAction()
+        /**
+         * @Route("/register", name="account_register")
+         */
+        public function registerAction(Request $request)
         {
-            $registration = new Registration();
-            $form = $this->createForm(new RegistrationType(), $registration, array(
-                'action' => $this->generateUrl('account_create'),
-            ));
+            $form = $this->createForm(new RegistrationType(), new Registration());
+
+            $form->handleRequest($request);
+
+            if ($form->isSubmitted() && $form->isValid()) {
+                $registration = $form->getData();
+
+                $em = $this->getDoctrine()->getManager();
+                $em->persist($registration->getUser());
+                $em->flush();
+
+                return $this->redirect($this->generateUrl('homepage'));
+            }
 
             return $this->render(
-                'AcmeAccountBundle:Account:register.html.twig',
+                'account/register.html.twig',
                 array('form' => $form->createView())
             );
         }
@@ -264,90 +277,15 @@ And its template:
 
 .. code-block:: html+jinja
 
-    {# src/Acme/AccountBundle/Resources/views/Account/register.html.twig #}
+    {# app/Resources/views/account/register.html.twig #}
     {{ form(form) }}
 
-Next, create the controller which handles the form submission. This performs
-the validation and saves the data into the database::
-
-    use Symfony\Component\HttpFoundation\Request;
-    // ...
-
-    public function createAction(Request $request)
-    {
-        $em = $this->getDoctrine()->getManager();
-
-        $form = $this->createForm(new RegistrationType(), new Registration());
-
-        $form->handleRequest($request);
-
-        if ($form->isValid()) {
-            $registration = $form->getData();
-
-            $em->persist($registration->getUser());
-            $em->flush();
-
-            return $this->redirect(...);
-        }
-
-        return $this->render(
-            'AcmeAccountBundle:Account:register.html.twig',
-            array('form' => $form->createView())
-        );
-    }
-
 Add new Routes
 --------------
 
-Next, update your routes. If you're placing your routes inside your bundle
-(as shown here), don't forget to make sure that the routing file is being
-:ref:`imported <routing-include-external-resources>`.
-
-.. configuration-block::
-
-    .. code-block:: yaml
-
-        # src/Acme/AccountBundle/Resources/config/routing.yml
-        account_register:
-            path:     /register
-            defaults: { _controller: AcmeAccountBundle:Account:register }
-
-        account_create:
-            path:     /register/create
-            defaults: { _controller: AcmeAccountBundle:Account:create }
-
-    .. code-block:: xml
-
-        <!-- src/Acme/AccountBundle/Resources/config/routing.xml -->
-        <?xml version="1.0" encoding="UTF-8" ?>
-        <routes xmlns="http://symfony.com/schema/routing"
-            xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-            xsi:schemaLocation="http://symfony.com/schema/routing http://symfony.com/schema/routing/routing-1.0.xsd">
-
-            <route id="account_register" path="/register">
-                <default key="_controller">AcmeAccountBundle:Account:register</default>
-            </route>
-
-            <route id="account_create" path="/register/create">
-                <default key="_controller">AcmeAccountBundle:Account:create</default>
-            </route>
-        </routes>
-
-    .. code-block:: php
-
-        // src/Acme/AccountBundle/Resources/config/routing.php
-        use Symfony\Component\Routing\RouteCollection;
-        use Symfony\Component\Routing\Route;
-
-        $collection = new RouteCollection();
-        $collection->add('account_register', new Route('/register', array(
-            '_controller' => 'AcmeAccountBundle:Account:register',
-        )));
-        $collection->add('account_create', new Route('/register/create', array(
-            '_controller' => 'AcmeAccountBundle:Account:create',
-        )));
-
-        return $collection;
+Don't forget to make sure that the routes defined as annotations in your
+controller are :ref:`loaded <routing-include-external-resources>` by your main
+routing configuration file.
 
 Update your Database Schema
 ---------------------------