- 🇳🇱Netherlands Nicasso
- Status changed to Needs review
9 months ago 4:00am 7 July 2024 - Merge request !9Issue #3292429 by Nicasso, uberengineer: Return proper AjaxResponse on ajaxSubmit → (Open) created by tr
- 🇺🇸United States tr Cascadia
I created a MR from the patch so that we can do automated testing on it.
- First commit to issue fork.
- 🇺🇸United States tr Cascadia
I think returning a form array is the preferred method in this case.
Regardless, if you return an AjaxResponse, it can't be an empty response - you have to ->addCommand() and provide the form array to the response.Bottom line, not a bug, since we are currently using the recommended method for this simple use-case, and needs work because returning an empty AjaxResponse will not work.
- 🇺🇸United States tr Cascadia
Needs tests.
"use" statements need to be alphabetical.
What about #wrapper ?