Merge pull request #152 from gsteel/iterable-validator-chain

Mark the ValidatorChain as iterable and final

composer.json

@@ -33,7 +33,7 @@
     "require": {
         "php": "^7.4 || ~8.0.0 || ~8.1.0",
         "laminas/laminas-servicemanager": "^3.12.0",
-        "laminas/laminas-stdlib": "^3.10"
+        "laminas/laminas-stdlib": "^3.13"
     },
     "require-dev": {
         "laminas/laminas-coding-standard": "^2.4.0",

src/ValidatorChain.php

@@ -3,9 +3,11 @@
 namespace Laminas\Validator;
 
 use Countable;
+use IteratorAggregate;
 use Laminas\ServiceManager\ServiceManager;
 use Laminas\Stdlib\PriorityQueue;
 use ReturnTypeWillChange;
+use Traversable;
 
 use function array_replace;
 use function assert;
@@ -16,10 +18,10 @@
 
 /**
  * @psalm-type QueueElement = array{instance: ValidatorInterface, breakChainOnFailure: bool}
+ * @implements IteratorAggregate<array-key, QueueElement>
+ * @final
  */
-class ValidatorChain implements
-    Countable,
-    ValidatorInterface
+class ValidatorChain implements Countable, IteratorAggregate, ValidatorInterface
 {
     /**
      * Default priority at which validators are added
@@ -248,7 +250,7 @@ public function isValid($value, $context = null)
     {
         $this->messages = [];
         $result         = true;
-        foreach ($this->validators as $element) {
+        foreach ($this as $element) {
             $validator = $element['instance'];
             assert($validator instanceof ValidatorInterface);
             if ($validator->isValid($value, $context)) {
@@ -331,4 +333,10 @@ public function __sleep()
     {
         return ['validators', 'messages'];
     }
+
+    /** @return Traversable<array-key, QueueElement> */
+    public function getIterator(): Traversable
+    {
+        return clone $this->validators;
+    }
 }