Page MenuHomePhabricator

Mobileviewapi should correctly return the redirect target
Closed, ResolvedPublic

Description

The complete redirect target contains a fragment, if there is any, too, not only the target page title.

Copied from blocked task:

Steps to reproduce

  1. (I don't know, how long this page is there with this content, but you can test the behavior with https://test.wikipedia.org/wiki/Test clicking the link TestFragment2)
  2. Create a page with multiple sections
  3. Create a second page which is a redirect to the first page (including a section, so add a fragment to the redirect "#Fragment")
  4. Create a third page and add a link to the second page
  5. Click the link in the third page

You can test it on testwiki:
https://test.wikipedia.org/w/api.php?page=TestFragment2&action=mobileview

Returns "TestFragment" as the value of "redirected" property, but should instead return "TestFragment#Fragment"

Event Timeline

Florian claimed this task.
Florian raised the priority of this task from to Needs Triage.
Florian updated the task description. (Show Details)
Florian added a project: MobileFrontend.
Florian added subscribers: Aklapper, Florian.

Change 232859 had a related patch set uploaded (by Florianschmidtwelzow):
Return the redirect target with Fragment

https://gerrit.wikimedia.org/r/232859

Florian set Security to None.

Change 232859 merged by jenkins-bot:
Return the redirect target with Fragment

https://gerrit.wikimedia.org/r/232859

Jdlrobson subscribed.

I assume this is done now :)

Oops, yeah, you're right, thanks for merging! :)