Container::getControls() returns list instead of iterator with names (BC break)

Author: dg

src/Forms/Container.php

@@ -307,18 +307,12 @@ public function addComponent(
 
 
 	/**
-	 * Iterates over all form controls.
-	 * @return iterable<Control>
+	 * Retrieves the entire hierarchy of form controls including nested.
+	 * @return list<Control>
 	 */
 	public function getControls(): iterable
 	{
-		return Nette\Utils\Iterables::repeatable(function () {
-			foreach ($this->getComponentTree() as $component) {
-				if ($component instanceof Control) {
-					yield $component->getName() => $component;
-				}
-			}
-		});
+		return array_values(array_filter($this->getComponentTree(), fn($c) => $c instanceof Control));
 	}
 
 

tests/Forms/Container.getControls().phpt

@@ -13,22 +13,4 @@ $form->addContainer('cont')
 	->addText('name');
 
 $controls = $form->getControls();
-
-$names = $values = [];
-foreach ($controls as $name => $value) {
-	$names[] = $name;
-	$values[] = $value;
-}
-
-Assert::same(['name', 'age', 'name'], $names);
-Assert::same([$form['name'], $form['age'], $form['cont-name']], $values);
-
-// again
-$names = $values = [];
-foreach ($controls as $name => $value) {
-	$names[] = $name;
-	$values[] = $value;
-}
-
-Assert::same(['name', 'age', 'name'], $names);
-Assert::same([$form['name'], $form['age'], $form['cont-name']], $values);
+Assert::same([$form['name'], $form['age'], $form['cont-name']], $controls);