Page MenuHomePhabricator

VisualEditor: Refuse to load for pages which are redirects
Closed, ResolvedPublic

Description

In advance of fixing bug 47328 (which will probably be after the July release), we need to prevent the broken behaviour outlined by C. Scott in bug 47328 comment 1 - so just refuse to load on a page which is a redirect?


Version: unspecified
Severity: normal
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=47328

Details

Reference
bz47329

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 1:14 AM
bzimport set Reference to bz47329.

dayuloli wrote:

For me, if you tried to edit a redirect page, it loads with the redirected page.

(In reply to comment #1)

For me, if you tried to edit a redirect page, it loads with the redirected
page.

Yes, this is current behaviour from Parsoid that will be fixed by fixing bug 47328, but that will leave users with a very confusing blank page. That is why I made this bug. :-)

It seems like it would be trivial to check whether the page is a redirect and simply disable VisualEditor if so. I'm adding the "easy" keyword to this bug.

(In reply to comment #3)

It seems like it would be trivial to check whether the page is a redirect and
simply disable VisualEditor if so. I'm adding the "easy" keyword to this bug.

It gets a bit trickier if the page does not yet exist, though.

Related URL: https://gerrit.wikimedia.org/r/64275 (Gerrit Change I68836156c35c796aa2156c382992b0c105ae34cf)

Merged; will go out with wmf5.