Page MenuHomePhabricator

RecentChanges URL carries duplicate limit
Closed, ResolvedPublic

Description

Author: rl

Description:
When changing the number of changes to show in [1], the new limit gets tacked to
the URL
in addition to (rather than in place of) the previous limit [2].

[1] http://en.wikipedia.org/wiki/Special:Recentchanges
[2]
http://en.wikipedia.org/w/index.php?title=Special:Recentchanges&hideminor=1&hideliu=0&hidebots=1&hidepatrolled=0&limit=50&days=7&limit=100


Version: 1.4.x
Severity: minor
OS: Linux
Platform: PC

Details

Reference
bz1755

Event Timeline

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

wegge wrote:

Proposed patch against CVS HEAD

The problem is that the current limit is included in the array of the Hide/show
toggles.

Attached:

wegge wrote:

Patch against CVS REL1_4

Attached:

jeluf wrote:

Fixed in CVS HEAD, SpecialRecentchanges.php, rev 1.63