tbmsg
tbmsg copied to clipboard
tzookb
Paginate
Is it possible to add a paginate function for messages in convos and for conversations?
This is something i ran into my self! I have been looking into it. Shouldn't be that hard to make but depends on how far you want to take it. Making a function like this should get people started:
public static function getMessages($convId, $start = 0, $perPage = 20) {
// SQL QUERY LIMIT = $perPage and OFFSET = $start.
}
This way you won't need to load all messages from the DB and should work great for ajax calls but the only problem with this is that you won't know how many pages there are in total. This also means you won't be able to make a Laravel Paginator object. I don't like working with the Paginator for messages and don't recommend it. But if that is what you want you could make something like:
$conv = TBMsg::getConversationMessages($convId, $userId,);
$conv->getAllMessages();
$conv->paginateMessages($perPage);
-----------------------
public function paginateMessages($perPage = 20) {
// $messages= all the messages
// $total = count() on all the $messages
// Paginator::make($collection->all(), $total, $perPage);
}
The problem here is that Pagenator::make() will use Paginator::getCurrentPage() to get the current page from the GET parameters. This will also make the package more dependent on laravel.
Raoul
yes your first option will be the option, as I dont want to add the laravel paginator now.
Ill implement it soon
just finished the big change with tests, so now it will be much safer to change stuff. I'll try to add it soon
Yes, this feature is very much required to move forward. I'd love to help out implementing this if you don't have the time.
Your package is awesome. One of the best chat systems for laravel available. Very flexible. Just a few issues else it's perfect.
Wish I could help but I'm a noob :-) On Dec 21, 2015 4:47 PM, "Tzook" [email protected] wrote:
not yet, hope to do it soon, or would be happy if you have suggestions.
— Reply to this email directly or view it on GitHub https://github.com/tzookb/tbmsg/issues/22#issuecomment-166274076.