The current implementation in RenderElement::preRenderAjaxForm() destroys any url options when a Url object is passed for form ajax. This was the root cause of probably 40+ hours of ctools debugging and a huge thanks to dawehner for ultimately sorting it. If a Url object is generated with options in place, we should respect that and not require they be restated in ['#ajax']['options'] manually.
Instead of overwriting the Url objects options so whatever is set in ['#ajax']['options'] (which could be empty) simply iterate over the options array and set individual options which will preserve options passed with the Url object.
Provide patch
Write tests
Get it done
None
Should not affect existing core implementations, and might smooth the way for some contrib modules as they begin implementing work here.
None.
Needs work
9.5
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.
Used to track the progress of issues reviewed by the Drupal Needs Review Queue Initiative.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
The tests in #4 and steps to reproduce in #14