Loading previous messages and listing last messages doesn't work

I have the same problem. Maybe it's ajax issue, I don't know. I send a message. It doesn't seem to be gone, but when I refresh the page, it appears in the stream.

I have prepared a patch that fixes this problem.
I removed the function call in the method and replaced it with the $response[] = $message; construct.
I hope this helps.

Hi k.prozorov,
Thanks for patch but it didn't work.
Unfortunately both 2 problems still exist.

I confirm that the patch in #6 fixed this issue for me.

1. I cannot reproduce the overlay issue. Not sure which overlay it is though, because I cannot see it while clicking on the "Load Previous" link.
2. After applying this patch, the messages list started getting refreshed properly. If an AJAX refresh interval is set, it loads new messages after that interval, otherwise it loads instantly after form submission.

Can confirm patch #6 does not fix the issue.

This is a javascript issue, which is breaking not only the load previous link, but all other links on the page no longer work when viewing the thread.

I can confirm that you can send 25 messages just fine, but once the "Load Previous" link appears, you cannot submit the next one.

Whichever link you click on a link or button, the page submits to "/private-message/ajax-callback/get_old_messages? token=W2RZzcxI6isICC5_MZYFCantGuhyx_slkOhHC8zUul8&threadid=123456&messageid=NaN" which has an e.preventdefault() which prevents the page from submitting/ loading.

At some point the javascript functions were updated to use `() =>` notation.

However, using that broke the ability to use `$(this)` inside them. See https://www.freecodecamp.org/news/the-difference-between-arrow-functions... for a review of arrow functions.

I've replaced the `() =>` with `function()` and that worked for me. Will create a patch / MR shortly.

Added patch that fixes the previous message issue. Please review!

Will this be backported to private_message 3.0.0?

@SocialNicheGuru I doubt it. Ideally they will tag a new release when they merge the MR

#MR111 worked for me. +1 for RTBC