ClassMethods Hydrator ignores invalid values

zendframework · May 31, 2012 · 30aa565 · 30aa565
commit 30aa565
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 1 deletion.
diff --git a/src/Hydrator/ClassMethods.php b/src/Hydrator/ClassMethods.php
@@ -120,7 +120,9 @@ public function hydrate(array $data, $object)
                 $property = preg_replace_callback('/(_[a-z])/', $transform, $property);
             }
             $method = 'set' . ucfirst($property);
-            $object->$method($value);
+            if (method_exists($object, $method)) {
+                $object->$method($value);
+            }
         }
         return $object;
     }

diff --git a/test/HydratorTest.php b/test/HydratorTest.php
@@ -37,6 +37,22 @@
  */
 class HydratorTest extends \PHPUnit_Framework_TestCase
 {
+
+    /**
+     * @var ClassMethodsCamelCase
+     */
+    protected $classMethodsCamelCase;
+
+    /**
+     * @var ClassMethodsCamelCaseMissing
+     */
+    protected $classMethodsCamelCaseMissing;
+
+    /**
+     * @var ClassMethodsUnderscore
+     */
+    protected $classMethodsUnderscore;
+
     public function setUp()
     {
         $this->classMethodsCamelCase = new ClassMethodsCamelCase();
@@ -93,4 +109,16 @@ public function testHydratorClassMethodsUnderscore()
         $this->assertEquals($test->getFooBar(), 'foo');
         $this->assertEquals($test->getFooBarBaz(), 'bar');
     }
+
+    public function testHydratorClassMethodsIgnoresInvalidValues()
+    {
+        $hydrator = new ClassMethods(false);
+        $data = array(
+            'foo_bar' => '1',
+            'foo_bar_baz' => '2',
+            'invalid' => 'value'
+        );
+        $test = $hydrator->hydrate($data, $this->classMethodsUnderscore);
+        $this->assertSame($this->classMethodsUnderscore, $test);
+    }
 }