If I go to a page that has been redirected, the URL retains the original title instead of the redirected title. So if I enter VE and then refresh, I'm taken to the editing the original page instead of the redirect page.
For example, go here: https://en.wikipedia.org/wiki/Zedonk, then click VE, then refresh the page.
URL should be changed to: https://en.wikipedia.org/wiki/Zebroid?veaction=edit
Version: unspecified
Severity: minor
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Declined | matmarex | T69330 VisualEditor: On a redirected page, VE URL retains original searched title instead of redirect title | |||
| Resolved | matmarex | T37045 Use history.replaceState to rewrite redirect urls |
Rewriting the URL is a bit evil, but if we don't we have the issue of users having a URL they can't share/reload without the context changing.
Ideally MediaWiki's redirects should 307 rather than 200 so we don't have this problem… :-(
(In reply to James Forrester from comment #1)
Ideally MediaWiki's redirects should 307 rather than 200 so we don't have
this problem… :-(
They shouldn't because then you can't show the "Redirected from" message without hacks, but they probably should update the URL with JavaScript instead. Wanna file a bug or that? (I wouldn't be surprised if there was one already.)
(In reply to Bartosz Dziewoński from comment #2)
(In reply to James Forrester from comment #1)
Ideally MediaWiki's redirects should 307 rather than 200 so we don't have
this problem… :-(They shouldn't because then you can't show the "Redirected from" message
without hacks,
I'm aware of that minor impact on users, but that info is normally duplicated by editorial templates anyway on Wikimedia wikis, at least.
but they probably should update the URL with JavaScript instead.
Wanna file a bug or that? (I wouldn't be surprised if there was one already.)
Yes, that looks like bug 35045.
Change 143852 had a related patch set uploaded by Bartosz Dziewoński:
On redirect, actually rewrite page URL to the target one with JavaScript
That change has been merged and this should therefore be magically fixed. I'd appreciate if someone could verify.