Laravel Query Sending Different Result For Same Problem

Question

Hi I am trying to create a one on one messaging system on LARAVEL. It was working all fine until for some users it started showing different result then expected. And it happens only for some users.. What is wrong with this query

$id =$receiver->id;
$messages = Message::where(function ($query) use ($id) {
        $query->where('user_id', '=', Auth::user()->id)
              ->where('receiver_id', '=', $id);
        })->orWhere(function ($query) use ($id) {
            $query->where('user_id', '=', $id)
                ->where('receiver_id', '=', Auth::user()->id);
        })->get();

After I return $messages the result is like this...

Working Result: Messages are coming sequentially..

In View It shows like this..

Same Query Bad Result

In the view you can see date are not aligned in order

I really can't figure out what went wrong if you can help I would really appreciate..

what issue .? you are gettiing

Kamlesh Paul
– Kamlesh Paul

2020-09-24 09:10:51 +00:00
Commented Sep 24, 2020 at 9:10 — Kamlesh Paul
– Kamlesh Paul, Commented Sep 24, 2020 at 9:10

STA · Accepted Answer · 2020-09-24 09:33:39Z

0

Your date is casting as UTC with ISO-8601 format, but I think date is not related with your issue, if you are not ordering with timestamp.
I suggest you to use orderBy with id, it can solve your issue easily :

$messages = Message::where(function ($query) use ($id) {
        $query->where('user_id', '=', Auth::user()->id)
              ->where('receiver_id', '=', $id);
        })->orWhere(function ($query) use ($id) {
            $query->where('user_id', '=', $id)
                ->where('receiver_id', '=', Auth::user()->id);
        })
        ->orderBy('id', 'ASC')
        ->get();

edited Sep 24, 2020 at 9:33

answered Sep 24, 2020 at 9:17

STA

35.3k9 gold badges49 silver badges62 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

user8232211 Over a year ago

Thank you so much, I really did not think about ordering as it was giving correct result from the beginning.. As I populate messages from top to bottom instead of DESC I used ASC and it worked. Thank you for your help. If you can edit and make ASC i would mark it correct.

STA Over a year ago

Opps, sorry. I edited DESC as ASC. Nice to hear that it helped

Collectives™ on Stack Overflow

Laravel Query Sending Different Result For Same Problem

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related