[buddypress-trac] [BuddyPress Trac] #8508: Paginate messages in a thread

[buddypress-trac]

#8508: Paginate messages in a thread
-------------------------+------------------------------
 Reporter:  espellcaste  |       Owner:  (none)
     Type:  enhancement  |      Status:  assigned
 Priority:  high         |   Milestone:  Awaiting Review
Component:  Messages     |     Version:  8.0.0
 Severity:  normal       |  Resolution:
 Keywords:  needs-patch  |
-------------------------+------------------------------
Description changed by espellcaste:

Old description:

> Currently, the `BP_Messages_Thread` class grabs all messages created for
> a thread. So if a thread has 1k messages, it tries to grab all of them
> and caches them, but there is no way to paginate those messages.
>
> I suggest we refactor that part of the class to not cache the messages
> but to set a default number of messages, like 10, and add a parameter to
> paginate those messa

New description:

 Currently, the `BP_Messages_Thread` class grabs all messages created for a
 thread. So if a thread has 1k messages, it tries to grab all of them and
 caches them, but there is no way to paginate those messages.

 I suggest we refactor that part of the class to not cache the messages but
 to set a default number of messages, like 10, and add a parameter to
 paginate the rest of the messages.

--

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/8508#comment:1>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac