[wp-trac] [WordPress Trac] #40510: REST API: Post Revisions: Adding support for pagination
WordPress Trac
noreply at wordpress.org
Fri Apr 21 08:14:07 UTC 2017
#40510: REST API: Post Revisions: Adding support for pagination
--------------------------+-----------------------------
Reporter: benoitperson | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: REST API | Version: 4.7
Severity: normal | Keywords:
Focuses: rest-api |
--------------------------+-----------------------------
The post revisions REST endpoint currently has no support for pagination.
While it works well for most usages with fairly low counts of revisions
(Wordpress.com limits the number at 25 for instance), on posts with a
revision count in the hundreds (or even thousands), this can quickly yield
unreasonably large (Mb+ of pure text) and long (several seconds on my
"test" instances) responses.
The API already supports pagination for a vast majority of ressources
(posts, pages, categories, tags, users) and is normalised around using 2
integer parameters: `page` (default: `1`)and `per_page` (default: `10`).
I wonder if there is any reason why pagination was never implemented on
that endpoint? Is it something potentially worth working on? Considering
some of the past issues around high memory usage for the revisions page of
wp-admin [0][1], it seems like getting a way to only load (for instance)
the most recent revisions would be an interesting win to provide a
smoother experience with revisions.
[0] https://core.trac.wordpress.org/ticket/34560
[1] https://core.trac.wordpress.org/ticket/24958
PS: To provide some (more) context, there is an ongoing effort to support
revisions in Calypso: https://github.com/Automattic/wp-calypso/pull/12733
from which this enhancement request emerged.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/40510>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list