Page MenuHomePhabricator

Revision::fetchAllRevisions has LIMIT=1, thus not returning all revisions
Closed, ResolvedPublic

Description

Author: endlessoblivion

Description:
Revision::fetchAllRevisions (http://svn.wikimedia.org/doc/Revision_8php-source.html#l00205) does its work using Revision::fetchFromConds (http://svn.wikimedia.org/doc/Revision_8php-source.html#l00243), which has a fixed LIMIT=1 (http://svn.wikimedia.org/doc/Revision_8php-source.html#l00253), thus not returning all revisions, but only one of them.


Version: 1.16.x
Severity: minor

Details

Reference
bz18821

Event Timeline

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

The LIMIT=1 was added in r9211 in 2005...

Revision::fetchAllRevisions() appears to be unused, and well it should be unused as it would work very poorly on large pages. :)

Probably best fix is to kill the bad function.