- Issue created by @dww
- First commit to issue fork.
- Status changed to Needs review
8 months ago 11:03pm 26 March 2024 - Status changed to Needs work
8 months ago 2:06pm 27 March 2024
Near the end of
🐛
States API doesn't work with multiple select fields
Needs review
, we tried to get JavascriptStatesTest::doMultipleSelectTriggerTests() to not just verify that changing the trigger gets the desired results, but also if you restore the trigger to the initial state, all the elements should go back to the initial state, too. Sadly, there seems to be some weirdness in the intersection between item FormAPI elements, isVisible(), chromedriver, and perhaps $trigger->setValue([]) which is causing some of the assertions to fail. However, manually inspecting the test HTML output, and manual testing, both indicate that the states code is really working. Since #1149078 is a 12 year old bug, it'd be sad to hold up committing the fix even longer while sorting out the "perfect" tests. Hence, this follow-up.
Needs work
11.0 🔥