Last modified: 2011-03-13 18:04:25 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 T13541, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 11541 - change rv{end|start}id to rv{min|max}id
change rv{end|start}id to rv{min|max}id
Status: RESOLVED WONTFIX
Product: MediaWiki
Classification: Unclassified
API (Other open bugs)
unspecified
All All
: Lowest enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-10-02 18:41 UTC by Carl Fürstenberg
Modified: 2011-03-13 18:04 UTC (History)
2 users (show)

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


Attachments

Description Carl Fürstenberg 2007-10-02 18:41:34 UTC
as it's not directly understandable that rvstartid and rvendid is related to the enumeration, and not the revisions per se, I would suggest changing it to rvminid and rvmaxid that is probably more understandable for most people.
Comment 1 Roan Kattouw 2007-10-02 20:17:45 UTC
Something similar is the case with list=recentchanges: rcstart is *later* than rcend, because listing STARTS at the newest revision (if rcdir=older). There is logic in this, and we should document this at [1]. Note my bold additions at [2], which aim to clarify this a little.

Recommend WONTFIX.

[1] http://www.mediawiki.org/wiki/API:Query_-_Lists#recentchanges_.2F_rc_.28done.29
[2] http://www.mediawiki.org/wiki/API:Query_-_Lists#deletedrevs_.2F_dr
Comment 2 Daniel Cannon (AmiDaniel) 2007-10-26 03:40:27 UTC
For the sake of not introducing further incompatibilities, I agree with the WONTFIX. It may have been a bad choice of the parameter names, but I think it's best we stick them so as not to break all of the various tools working off of this query. (The other option that will satisfy both desires is to add a redundant rvminid and rvmaxid, but this is a PITA.)

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


Navigation
Links