zendframework/zendframework#6496 zendframework/zendframework#6386 - r…

…everting minor bc-break (`attach` method should keep a compatible signature)

src/ValidatorChain.php

@@ -102,27 +102,23 @@ public function plugin($name, array $options = null)
      * If $breakChainOnFailure is true, then if the validator fails, the next validator in the chain,
      * if one exists, will not be executed.
      *
-     * @param  callable|ValidatorInterface $callback A Validator implementation or valid PHP callback
-     * @param  bool                        $breakChainOnFailure
-     * @param  int                         $priority            Priority at which to enqueue validator; defaults to
+     * @param  ValidatorInterface $validator
+     * @param  bool               $breakChainOnFailure
+     * @param  int                $priority            Priority at which to enqueue validator; defaults to
      *                                                          1 (higher executes earlier)
      *
      * @throws Exception\InvalidArgumentException
      *
      * @return self
      */
-    public function attach($callback, $breakChainOnFailure = false, $priority = self::DEFAULT_PRIORITY)
-    {
-        if (! ($callback instanceof ValidatorInterface || is_callable($callback))) {
-            throw new Exception\InvalidArgumentException(sprintf(
-                'Expected a valid PHP callable or Zend\Validator\ValidatorInterface; "%s" received',
-                is_object($callback) ? get_class($callback) : gettype($callback)
-            ));
-        }
-
+    public function attach(
+        ValidatorInterface $validator,
+        $breakChainOnFailure = false,
+        $priority = self::DEFAULT_PRIORITY
+    ) {
         $this->validators->insert(
             array(
-                'instance'            => $callback,
+                'instance'            => $validator,
                 'breakChainOnFailure' => (bool) $breakChainOnFailure,
             ),
             $priority