minor symfony#100 Added a note about client-side and server-side form validation (javiereguiluz)

javiereguiluz · javiereguiluz · commit be4e454a1532 · 2015-07-20T16:02:53.000+02:00
This PR was squashed before being merged into the master branch (closes symfony#100). Discussion ---------- Added a note about client-side and server-side form validation This fixes symfony#91. Commits ------- bc53c7c Added a note about client-side and server-side form validation
diff --git a/app/Resources/views/admin/blog/_form.html.twig b/app/Resources/views/admin/blog/_form.html.twig
@@ -1,3 +1,11 @@
+{#
+    By default, forms enable client-side validation. This means that you can't
+    test the server-side validation errors from the browser. To temporarily
+    disable this validation, add the 'novalidate' attribute:
+
+    {{ form_start(form, { attr: { novalidate: 'novalidate' } }) }}
+#}
+
 {{ form_start(form) }}
     {{ form_widget(form) }}
 
diff --git a/app/Resources/views/blog/comment_form.html.twig b/app/Resources/views/blog/comment_form.html.twig
@@ -1,3 +1,11 @@
+{#
+    By default, forms enable client-side validation. This means that you can't
+    test the server-side validation errors from the browser. To temporarily
+    disable this validation, add the 'novalidate' attribute:
+
+    {{ form_start(form, { method: ..., action: ..., attr: { novalidate: 'novalidate' } }) }}
+#}
+
 {{ form_start(form, { method: 'POST', action: path('comment_new', { 'postSlug': post.slug }) }) }}
     {#  instead of displaying form fields one by one, you can also display them
         all with their default options and styles just by calling to this function:
diff --git a/src/AppBundle/Form/CommentType.php b/src/AppBundle/Form/CommentType.php
@@ -32,6 +32,13 @@ class CommentType extends AbstractType
      */
     public function buildForm(FormBuilderInterface $builder, array $options)
     {
+        // By default, form fields include the 'required' attribute, which enables
+        // the client-side form validation. This means that you can't test the
+        // server-side validation errors from the browser. To temporarily disable
+        // this validation, set the 'required' attribute to 'false':
+        //
+        //     $builder->add('content', null, array('required' => false));
+
         $builder
             ->add('content')
         ;
diff --git a/src/AppBundle/Form/PostType.php b/src/AppBundle/Form/PostType.php
@@ -29,8 +29,16 @@ class PostType extends AbstractType
      */
     public function buildForm(FormBuilderInterface $builder, array $options)
     {
-        // for the full reference of options defined by each form field type
+        // For the full reference of options defined by each form field type
         // see http://symfony.com/doc/current/reference/forms/types.html
+
+        // By default, form fields include the 'required' attribute, which enables
+        // the client-side form validation. This means that you can't test the
+        // server-side validation errors from the browser. To temporarily disable
+        // this validation, set the 'required' attribute to 'false':
+        //
+        //     $builder->add('title', null, array('required' => false, ...));
+
         $builder
             ->add('title', null, array('label' => 'label.title'))
             ->add('summary', 'textarea', array('label' => 'label.summary'))

Original file line number	Diff line number	Diff line change
`@@ -32,6 +32,13 @@ class CommentType extends AbstractType`
`32`	`32`	`*/`
`33`	`33`	`public function buildForm(FormBuilderInterface $builder, array $options)`
`34`	`34`	`{`
	`35`	`+ // By default, form fields include the 'required' attribute, which enables`
	`36`	`+ // the client-side form validation. This means that you can't test the`
	`37`	`+ // server-side validation errors from the browser. To temporarily disable`
	`38`	`+ // this validation, set the 'required' attribute to 'false':`
	`39`	`+ //`
	`40`	`+ // $builder->add('content', null, array('required' => false));`
	`41`	`+`
`35`	`42`	`$builder`
`36`	`43`	`->add('content')`
`37`	`44`	`;`