From babc811381142eedbce77e7f947e919e7aa4c92b Mon Sep 17 00:00:00 2001 From: Sabina Talipova Date: Mon, 4 Jul 2022 11:45:30 +1200 Subject: [PATCH] FIX Remove unexpected message --- src/Forms/GridField/GridFieldFilterHeader.php | 2 +- .../GridField/GridFieldFilterHeaderTest.php | 17 +++++++++++++++++ 2 files changed, 18 insertions(+), 1 deletion(-) diff --git a/src/Forms/GridField/GridFieldFilterHeader.php b/src/Forms/GridField/GridFieldFilterHeader.php index d0077fffd..43e781a9c 100755 --- a/src/Forms/GridField/GridFieldFilterHeader.php +++ b/src/Forms/GridField/GridFieldFilterHeader.php @@ -362,7 +362,7 @@ class GridFieldFilterHeader extends AbstractGridFieldComponent implements GridFi } foreach ($searchFields->getIterator() as $field) { - $field->addExtraClass('stacked'); + $field->addExtraClass('stacked no-change-track'); } $name = $gridField->Title ?: singleton($gridField->getModelClass())->i18n_plural_name(); diff --git a/tests/php/Forms/GridField/GridFieldFilterHeaderTest.php b/tests/php/Forms/GridField/GridFieldFilterHeaderTest.php index 2c721927d..e7bf6a3fb 100644 --- a/tests/php/Forms/GridField/GridFieldFilterHeaderTest.php +++ b/tests/php/Forms/GridField/GridFieldFilterHeaderTest.php @@ -139,4 +139,21 @@ class GridFieldFilterHeaderTest extends SapphireTest 'GridFieldFilterHeader::handleAction resets the gridstate filter when the user resets the search.' ); } + + public function testGetSearchForm() + { + $searchForm = $this->component->getSearchForm($this->gridField); + + $this->assertTrue($searchForm instanceof Form); + $this->assertEquals('Search__Name', $searchForm->fields[0]->Name); + $this->assertEquals('Search__City', $searchForm->fields[1]->Name); + $this->assertEquals('Search__Cheerleader__Hat__Colour', $searchForm->fields[2]->Name); + $this->assertEquals('TeamsSearchForm', $searchForm->Name); + $this->assertEquals('cms-search-form', $searchForm->extraClasses['cms-search-form']); + + foreach ($searchForm->fields as $field) { + $this->assertEquals('stacked', $field->extraClasses['stacked']); + $this->assertEquals('no-change-track', $field->extraClasses['no-change-track']); + } + } }