Page MenuHomePhabricator

name clash in Special:Allpages
Closed, ResolvedPublic

Description

Author: Michael.Keppler

Description:
The inversion feature on Allpages calculates the name of the output namespace as
variable "n" (line 261) which is used as loop counter at the same time resulting
in invalid XHTML and output containing only 1 link per row (instead of 3).


Version: 1.5.x
Severity: minor
URL: http://test.leuksman.com/index.php/Special:Allpages

Details

Reference
bz2101

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 8:29 PM
bzimport set Reference to bz2101.
bzimport added a subscriber: Unknown Object (MLST).

Michael.Keppler wrote:

simple variable renaming patch for HEAD

Attached:

avarab wrote:

Applied to HEAD with small changes.

gerritbot subscribed.

Change 284637 had a related patch set uploaded (by Xqt):
[T4101] Use starthexsortkey instead of startsortkey

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