Merge pull request zendframework#3484 branch 'hotfix/array-to-string-…

…conversion'
HolgerW1 · Jan 21, 2013 · a219656 · a219656
2 parents d0321dd + 05c1703
commit a219656
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/library/Zend/Validator/AbstractValidator.php b/library/Zend/Validator/AbstractValidator.php
@@ -303,7 +303,7 @@ protected function createMessage($messageKey, $value)
         ) {
             $value = get_class($value) . ' object';
         } elseif (is_array($value)) {
-            $value = '[' . implode(', ', $value) . ']';
+            $value = var_export($value, 1);
         } else {
             $value = (string) $value;
         }

diff --git a/tests/ZendTest/Validator/AbstractTest.php b/tests/ZendTest/Validator/AbstractTest.php
@@ -10,6 +10,7 @@
 
 namespace ZendTest\Validator;
 
+use ReflectionMethod;
 use Zend\I18n\Translator\Translator;
 use Zend\Validator\AbstractValidator;
 
@@ -227,6 +228,17 @@ public function testDefaultTranslatorMethods()
         $this->assertTrue(AbstractValidator::hasDefaultTranslator());
     }
 
+    public function testMessageCreationWithNestedArrayValueDoesNotRaiseNotice()
+    {
+        $r = new ReflectionMethod($this->validator, 'createMessage');
+        $r->setAccessible(true);
+
+        $message = $r->invoke($this->validator, 'fooMessage', array('foo' => array('bar' => 'baz')));
+        $this->assertContains('foo', $message);
+        $this->assertContains('bar', $message);
+        $this->assertContains('baz', $message);
+    }
+
     /**
      * Ignores a raised PHP error when in effect, but throws a flag to indicate an error occurred
      *