Page MenuHomePhabricator

Add pagination when displaying deleted revisions on Special:Undelete
Closed, ResolvedPublic

Description

When restoring a deleted page, it presents a form to put a reason for the undeletion, and below there's a list of *ALL* revisions that were deleted, with checkboxes to select each of them, without any pagination.

This presents problems with pages with lots (thousands) of deleted revisions. For example, deleting for merging histories, "accidentaly" delete a page, hide a specific revision, etc:

  • If you attempt to load the Special:Undelete for that page, you may get a timeout an OOM or your browser crash/eat all your memory.
  • If it even loads and you want to select lots of individual revisions to restore, you may hit the max_input_vars limit [1] (see bug 18687) and the selection won't be applied.

Pagination should be like page history.

Of course, pagination would be tricky for doing partial restoring, since on pagination you will lost the previous marked checkboxes. But at least the undelete page would load on pages with huge number of deleted revisions, and one could select revisions to restore in smaller batches.


[1] http://www.php.net/manual/info.configuration.php#ini.max-input-vars


Version: 1.21.x
Severity: enhancement

Details

Reference
bz44410

Related Objects

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 1:16 AM
bzimport set Reference to bz44410.
bzimport added a subscriber: Unknown Object (MLST).

(In reply to comment #1)

Dupe of bug 7996?

Indeed. I don't know how the search missed it

  • This bug has been marked as a duplicate of bug 7996 ***