Title says it all.
This is happening because My account
points to /user, which performs a redirect to the currently logged in user's profile page. However, it does a redirect using Symfony's RedirectResponse, which is not aware of formats, it always returns a HTML response. But worse, its Location header (which matters most of all) points to a URL without setting the current _format/_wrapper_format query argument, so the result is:
/user?_wrapper_format=drupal_refreshlessLocation: /user/1 (note the absence of the _wrapper_format query argument)I wonder if there's a core issue for the lack of AJAX redirect support on the server side.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | interdiff.txt | 529 bytes | wim leers |
| #7 | 2704075-7.patch | 6.57 KB | wim leers |
| #6 | 2704075-work-around-attempt2-do-not-test.patch | 2.47 KB | wim leers |
| #5 | 2704075-work-around-attempt1-do-not-test.patch | 1.48 KB | wim leers |
| #4 | 2704075-4.patch | 4.1 KB | wim leers |
Comments
Comment #2
wim leersSeems like the actual root cause here is that the
UrlGeneratordoesn't take the current format into account…Comment #3
wim leersFiled a core issue and patch: #2714275: Redirects performed while a wrapper format is active loses the wrapper format, hence breaking AJAX, Dialogs, RefreshLess.
However… this won't actually fix everything. Because RefreshLess also expects additional query strings to be preserved:
refreshless_page_state,ajax_page_state, et cetera. Therefore, I'm not going to block this on that core issue, I'm just going to fix it in RefreshLess itself.(This must be resolved by the module that provides the wrapper format, which in this case is RefreshLess.)
So, stay tuned for a patch.
Comment #4
wim leersThis works :)
But… the URL stubbornly stays at the original target URL:
/user, not/user/345345. And AFAICT we unfortunately have no way of detecting what the redirected XHR's URL is… :/Comment #5
wim leersThe first work-around attempt: generate the absolute URL on the client side. This works mostly… except that it utterly fails as soon as it comes to path aliases… So, alas, not a viable option.
Comment #6
wim leersHere's a second attempt, this one works!
Comment #7
wim leersNow merging #6 with #4, for a complete solution.
Comment #9
wim leersA tiny docs change on commit.
Also, I think this is critical, because it's a very frequently used link, present by default in Drupal 8. Many people will report this over and over if we don't fix it.