Last modified: 2014-07-17 00:05:31 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T45509, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 43509 - Special:Whatlinkshere pagination broken
Special:Whatlinkshere pagination broken
Status: PATCH_TO_REVIEW
Product: MediaWiki
Classification: Unclassified
Special pages (Other open bugs)
1.21.x
All All
: High normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-12-29 23:07 UTC by Jarry1250
Modified: 2014-07-17 00:05 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Jarry1250 2012-12-29 23:07:31 UTC
The (prev) links rely on having a &back= id from which they can be filled. This works neatly if you want to go back one level, but try and go back two in a row and the whole thing falls apart.

For example:, take https://en.wikipedia.org/w/index.php?title=Special:WhatLinksHere/Abacus . Now go forward (next) two times, and then back twice. In fact, you get trapped on page 2, because now there's no &back= id to use in the generation of a (prev) link. 

WLH should probably copy other special pages and do "(prev)" using a dir=prev parameter.
Comment 1 Jarry1250 2012-12-30 17:10:49 UTC
Patch: https://gerrit.wikimedia.org/r/#/c/41386/
Comment 2 Andre Klapper 2013-05-16 16:03:44 UTC
Patch in Gerrit still waiting for review. :(
Comment 3 Gerrit Notification Bot 2013-09-12 19:03:10 UTC
Change 41386 had a related patch set uploaded by Nemo bis:
(bug #43509) Special:Whatlinkshere pagination broken

https://gerrit.wikimedia.org/r/41386
Comment 4 Greg Grossmeier 2014-06-24 17:52:29 UTC
Hi Jarry: Are you still available to move your patch forward?

Nemo: Do you think you can help move it forward?

Reseting assignee for now and adding the gerrit reviewers from the patch as a CC here.
Comment 5 Derk-Jan Hartman 2014-07-15 09:25:49 UTC
The problem here is that you need a test instance with a lot of pages, that link to 1 page. Not many reviewers will have that.
Comment 6 Bawolff (Brian Wolff) 2014-07-17 00:05:31 UTC
(In reply to Derk-Jan Hartman from comment #5)
> The problem here is that you need a test instance with a lot of pages, that
> link to 1 page. Not many reviewers will have that.

Doing &limit=3 will help with that...

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links