symfony · olvlvl · Feb 25, 2018
diff --git a/src/Symfony/Component/Validator/Tests/Fixtures/Timer.php b/src/Symfony/Component/Validator/Tests/Fixtures/Timer.php
@@ -0,0 +1,13 @@
+<?php
+
+namespace Symfony\Component\Validator\Tests\Fixtures;
+
+class Timer
+{
+    private $duration;
+
+    public function __construct($duration)
+    {
+        $this->duration = $duration;
+    }
+}
diff --git a/src/Symfony/Component/Validator/Tests/Fixtures/TimerCollection.php b/src/Symfony/Component/Validator/Tests/Fixtures/TimerCollection.php
@@ -0,0 +1,27 @@
+<?php
+
+namespace Symfony\Component\Validator\Tests\Fixtures;
+
+class TimerCollection implements \IteratorAggregate
+{
+    /**
+     * @var array
+     */
+    private $timers;
+
+    /**
+     * @param Timer[] $timers
+     */
+    public function __construct(array $timers)
+    {
+        $this->timers = $timers;
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function getIterator()
+    {
+        return new \ArrayIterator($this->timers);
+    }
+}
diff --git a/src/Symfony/Component/Validator/Tests/LegacyExecutionContextTest.php b/src/Symfony/Component/Validator/Tests/LegacyExecutionContextTest.php
@@ -11,6 +11,7 @@
 
 namespace Symfony\Component\Validator\Tests;
 
+use Symfony\Component\Validator\Constraints\NotBlank;
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\Validator\Constraints\Collection;
 use Symfony\Component\Validator\Constraints\All;
@@ -19,6 +20,9 @@
 use Symfony\Component\Validator\ConstraintViolationList;
 use Symfony\Component\Validator\ExecutionContext;
 use Symfony\Component\Validator\Tests\Fixtures\ConstraintA;
+use Symfony\Component\Validator\Tests\Fixtures\Timer;
+use Symfony\Component\Validator\Tests\Fixtures\TimerCollection;
+use Symfony\Component\Validator\Validation;
 use Symfony\Component\Validator\ValidationVisitor;
 
 /**
@@ -327,6 +331,27 @@ public function testGetPropertyPathWithNestedCollectionsAndAllMixed()
 
         $this->assertEquals($expectedViolationPaths, $violationPaths);
     }
+
+    public function testGetPropertyPathWithNestedCollectionsAndAllMixedWithIteratorAggregate()
+    {
+        $data = new TimerCollection(array(
+            new Timer(null),
+            new Timer('PT10M'),
+            new Timer(null),
+        ));
+
+        $validator = Validation::createValidatorBuilder()->getValidator();
+        $metadata = $validator->getMetadataFor('Symfony\Component\Validator\Tests\Fixtures\Timer');
+        $metadata->addPropertyConstraint('duration', new NotBlank());
+
+        $violationPaths = array();
+
+        foreach ($validator->validate($data) as $violation) {
+            $violationPaths[] = $violation->getPropertyPath();
+        }
+
+        $this->assertSame(array('[0].duration', '[2].duration'), $violationPaths);
+    }
 }
 
 class ExecutionContextTest_TestClass

diff --git a/src/Symfony/Component/Validator/Validator/RecursiveContextualValidator.php b/src/Symfony/Component/Validator/Validator/RecursiveContextualValidator.php
@@ -748,9 +748,10 @@ private function validateGenericNode($value, $object, $cacheKey, MetadataInterfa
         // See validateClassNode()
         $cascadedGroups = null !== $cascadedGroups && count($cascadedGroups) > 0 ? $cascadedGroups : $groups;
 
+//        if (is_array($value) || $value instanceof \IteratorAggregate) {
         if (is_array($value)) {
-            // Arrays are always traversed, independent of the specified
-            // traversal strategy
+            // Arrays and instance of IteratorAggregate are always traversed, independent of the
+            // specified traversal strategy
             // (BC with Symfony < 2.5)
             $this->validateEachObjectIn(
                 $value,