Skip to content

Commit 43e74fe

Browse files
committed
fix(autocomplete): max_results option is not passed to parent
1 parent 240028f commit 43e74fe

File tree

5 files changed

+8
-3
lines changed

5 files changed

+8
-3
lines changed

src/Autocomplete/CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,9 @@
11
# CHANGELOG
22

3+
## Unreleased
4+
5+
- Fix issue where `max_results` was not passed as a Stimulus value (#538)
6+
37
## 2.5.0
48

59
- Automatic pagination support added: if the query would return more results

src/Autocomplete/src/Form/AutocompleteEntityTypeSubscriber.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ public function preSetData(FormEvent $event)
4343
// pass to AutocompleteChoiceTypeExtension
4444
$options['autocomplete'] = true;
4545
$options['autocomplete_url'] = $this->autocompleteUrl;
46-
unset($options['searchable_fields'], $options['security'], $options['filter_query'], $options['max_results']);
46+
unset($options['searchable_fields'], $options['security'], $options['filter_query']);
4747

4848
$form->add('autocomplete', EntityType::class, $options);
4949
}

src/Autocomplete/tests/Fixtures/Form/CategoryAutocompleteType.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ public function configureOptions(OptionsResolver $resolver)
4141
'attr' => [
4242
'data-controller' => 'custom-autocomplete',
4343
],
44-
'max_results' => 5,
44+
'max_results' => 25,
4545
'min_characters' => 2,
4646
]);
4747
}

src/Autocomplete/tests/Functional/AutocompleteFormRenderingTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ public function testFieldsRenderWithStimulusController()
3333
->assertElementAttributeContains('#product_category_autocomplete', 'data-controller', 'custom-autocomplete symfony--ux-autocomplete--autocomplete')
3434
->assertElementAttributeContains('#product_category_autocomplete', 'data-symfony--ux-autocomplete--autocomplete-url-value', '/test/autocomplete/category_autocomplete_type')
3535
->assertElementAttributeContains('#product_category_autocomplete', 'data-symfony--ux-autocomplete--autocomplete-min-characters-value', '2')
36+
->assertElementAttributeContains('#product_category_autocomplete', 'data-symfony--ux-autocomplete--autocomplete-max-results-value', '25')
3637

3738
->assertElementAttributeContains('#product_portionSize', 'data-controller', 'symfony--ux-autocomplete--autocomplete')
3839
->assertElementAttributeContains('#product_tags', 'data-controller', 'symfony--ux-autocomplete--autocomplete')

src/Autocomplete/tests/Functional/FieldAutocompleterTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ public function testItCheckMaxResultsOption(): void
8686
->throwExceptions()
8787
->get('/test/autocomplete/category_autocomplete_type?query=foo')
8888
->assertSuccessful()
89-
->assertJsonMatches('length(results)', 5)
89+
->assertJsonMatches('length(results)', 25)
9090
;
9191
}
9292

0 commit comments

Comments
 (0)