Author: colin.pitts
Description:
Special:Undelete displays the oldest deleted revision not the newest.
This behavior is both counter-intuitive and backwards from the existing behavior.
Version: 1.4.x
Severity: normal
Description
Description
Details
Details
- Reference
- bz1163
Event Timeline
Comment Actions
Created attachment 438
Patch against HEAD
The bug is caused by a string passed instead of an array of options to
selectrow.
Attached:
Comment Actions
Applied with slight modification (include the limit; if there are many we only want one here).
Fix in CVS HEAD and REL1_4, will appear in 1.4.3 release
(Note that deletion is a bit borked in HEAD at the moment.)
Comment Actions
(In reply to comment #3)
Applied with slight modification (include the limit; if there are many we only
want one here).
Note that this is unnecessary, Database::selectRow specifies LIMIT 1
automatically ("select [one] row").
Comment Actions
Ok, time to sleep before any more check-ins. :)
Removed the redundant clause. (It's harmless as it's overridden in selectRow(), not appended, but let's not
needlessly multiply non-parsimonious elements.)