I'm trying to reverse the messages so the newest one can be at the bottom, but I think either the pagination or something else is interferring :
public function view(Request $request)
{
$userId = $request->user()->id;
$conversations = Conversation::where(function ($query) use ($userId) {
$query->where('participant1_id', $userId)
->orWhere('participant2_id', $userId);
})
->with(['participant1:id,name', 'participant2:id,name'])
->with(['messages' => function ($query) {
$query->latest()->take(20);
}])
->latest()
->paginate(10);
$conversations->getCollection()->transform(function ($conversation) {
$conversation->messages = $conversation->messages->reverse()->values();
$conversation->messages->transform(function ($message) use ($conversation) {
$message->participant = ($message->user_id === $conversation->participant1_id)
? $conversation->participant1
: $conversation->participant2;
return $message;
});
return $conversation;
});
return Inertia::render('ConversationsPage', ['conversations' => $conversations]);
}
Even with the reverse()->values being called on the messages of each conversation I'm still getting the newest messages above the older ones. I can easily fix this in my react front end , but I'm wondering what it is that I'm missing here.
Not sure about the reverse, but you cannot limit the number of messages by conversation like this. You will get a total of 20 messages across all conversations.
In Laravel 11 there is a technique.
See this; https://ashallendesign.co.uk/blog/limit-eager-loaded-relationships
Most likely issue is that your modified conversation is not making it back out from the closure
Maybe after using getCollection() you have to use setCollection to reattach the modified data
@Snapey What I did instead was $query->orderBy('created_at', 'asc')->take(-20); I changed the order to ascending and took the last 20, but I will investigate your earlier comment about limiting the messages to 20 tomorrow, I will make a factory for some conversations and messages to see the behavior and if it indeed limits it to 20 for all conversations combined I will fix it. Thanks!
public function view(Request $request)
{
$userId = $request->user()->id;
$conversations = Conversation::where(function ($query) use ($userId) {
$query->where('participant1_id', $userId)
->orWhere('participant2_id', $userId);
})
->with(['participant1:id,name', 'participant2:id,name'])
->with(['messages' => function ($query) {
$query->orderBy('created_at', 'desc')->limit(20);
}])
->latest()
->paginate(10);
$conversations->getCollection()->transform(function ($conversation) {
$conversation->messages->transform(function ($message) use ($conversation) {
$message->participant = ($message->user_id === $conversation->participant1_id)
? $conversation->participant1
: $conversation->participant2;
return $message;
});
return $conversation;
});
return Inertia::render('ConversationsPage', ['conversations' => $conversations]);
}
I've decided that this would be the last version of the method, and I would just reverse the content in the front-end react component, however I'm not experiencing the issue @snapey described, I'm getting 20 messages per conversation and not 20 in total, so everything seems to be working fine.
Please or to participate in this conversation.