Page MenuHomePhabricator

Patch: relative oldids (prev, next, cur) for raw pages
Closed, ResolvedPublic

Description

Author: lupin.wp

Description:
This patch adds a new parameter to action=raw, direction, which can be prev,
next or cur.

I'm implementing previews of diffs using client-side javascript diffing, and
this will be most useful.


Version: unspecified
Severity: enhancement

Details

Reference
bz4838

Event Timeline

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

lupin.wp wrote:

proposed patch

Attached:

Wouldn't recommend inventing yet another parameter for this.
If it's truly necessary, better to remain consistent with existing parameters.

lupin.wp wrote:

Ah, ok. Which existing parameters would you recommend I follow?

lupin.wp wrote:

changed direction to diff

Since this is intended to grab revisions referred to with the diff= parameter
in existing links, I've renamed the direction parameter to diff.

This seems a little illogical to me, but it's an existing parameter.

attachment dd ignored as obsolete

lupin.wp wrote:

Wait, isn't direction already used for this purpose? For example,

http://en.wikipedia.org/w/index.php?title=Wikipedia:Wikipedians&direction=next&oldid=1

I think my original patch makes more sense in view of this.

robchur wrote:

Committed to HEAD more or less intact. :)

Marking as FIXED since there is no explanation for the reopening.