Merge remote branch 'kriswallsmith/form/is-valid-read-only'

* kriswallsmith/form/is-valid-read-only:
  [Form] fixed isValid() on readOnly forms that have children

src/Symfony/Component/Form/Form.php

@@ -679,10 +679,12 @@ class Form implements \IteratorAggregate, FormInterface
             return false;
         }
 
-        foreach ($this->children as $child) {
-            if (!$child->isValid()) {
+        if (!$this->readOnly) {
+            foreach ($this->children as $child) {
+                if (!$child->isValid()) {
 
-                return false;
+                    return false;
+                }
             }
         }
 

tests/Symfony/Tests/Component/Form/FormTest.php

@@ -306,6 +306,22 @@ class FormTest extends \PHPUnit_Framework_TestCase
         $this->assertTrue($form->isValid());
     }
 
+    public function testValidIfBoundAndReadOnlyWithChildren()
+    {
+        $this->factory->expects($this->once())
+            ->method('createNamedBuilder')
+            ->with('text', 'name', null, array())
+            ->will($this->returnValue($this->getBuilder('name')));
+
+        $form = $this->getBuilder('person')
+            ->setReadOnly(true)
+            ->add('name', 'text')
+            ->getForm();
+        $form->bind(array('name' => 'Jacques Doe'));
+
+        $this->assertTrue($form->isValid());
+    }
+
     public function testNotValidIfNotBound()
     {
         $this->assertFalse($this->form->isValid());