[Form] Fixed strict "data_class" check to work with instances of \ArrayAccess

webmozart · webmozart · commit 82c221a1dd2b · 2012-05-22T18:24:59.000+02:00
diff --git a/src/Symfony/Component/Form/Form.php b/src/Symfony/Component/Form/Form.php
@@ -340,12 +340,8 @@ public function setData($appData)
         if (!empty($clientData)) {
             $dataClass = $this->config->getDataClass();
 
-            if (null === $dataClass && is_object($clientData)) {
-                $expectedType = 'scalar';
-
-                if (count($this->children) > 0 && $this->config->getDataMapper()) {
-                    $expectedType = 'array';
-                }
+            if (null === $dataClass && is_object($clientData) && !$clientData instanceof \ArrayAccess) {
+                $expectedType = 'scalar, array or an instance of \ArrayAccess';
 
                 throw new FormException(
                     'The form\'s client data is expected to be of type ' . $expectedType . ', ' .
diff --git a/src/Symfony/Component/Form/Tests/FormTest.php b/src/Symfony/Component/Form/Tests/FormTest.php
@@ -1249,6 +1249,21 @@ public function testClientDataMustNotBeObjectIfDataClassIsNull()
         $form->setData('foo');
     }
 
+    public function testClientDataMayBeArrayAccessIfDataClassIsNull()
+    {
+        $arrayAccess = $this->getMock('\ArrayAccess');
+        $config = new FormConfig('name', null, $this->dispatcher);
+        $config->appendClientTransformer(new FixedDataTransformer(array(
+            '' => '',
+            'foo' => $arrayAccess,
+        )));
+        $form = new Form($config);
+
+        $form->setData('foo');
+
+        $this->assertSame($arrayAccess, $form->getClientData());
+    }
+
     /**
      * @expectedException Symfony\Component\Form\Exception\FormException
      */
