The exception should be thrown if an object doesn't implement Traversable AND doesn't implement Countable, not when it doesn't implement Traversable but DOES implement Countable

Robin Kanters · fabpot · commit 8eea65e0d86c · 2016-02-17T07:12:58.000+01:00
diff --git a/src/Symfony/Component/Console/Question/Question.php b/src/Symfony/Component/Console/Question/Question.php
@@ -140,7 +140,7 @@ public function setAutocompleterValues($values)
         }
 
         if (null !== $values && !is_array($values)) {
-            if (!$values instanceof \Traversable || $values instanceof \Countable) {
+            if (!$values instanceof \Traversable || !$values instanceof \Countable) {
                 throw new \InvalidArgumentException('Autocompleter values can be either an array, `null` or an object implementing both `Countable` and `Traversable` interfaces.');
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,7 @@ public function setAutocompleterValues($values)`
`140`	`140`	`}`
`141`	`141`
`142`	`142`	`if (null !== $values && !is_array($values)) {`
`143`		`- if (!$values instanceof \Traversable \|\| $values instanceof \Countable) {`
	`143`	`+ if (!$values instanceof \Traversable \|\| !$values instanceof \Countable) {`
`144`	`144`	throw new \InvalidArgumentException('Autocompleter values can be either an array, `null` or an object implementing both `Countable` and `Traversable` interfaces.');
`145`	`145`	`}`
`146`	`146`	`}`