Problem with multiple recipients at PMs

[Stefan-MyBB]

Send a PM to multiple recipients and then have a look to the usernames of the recipients (see attached image).
Sometimes the names of the recipients are shown and sometimes not. I have also sent a PM on another board, where no username was shown at all but only N/A ("keine Angabe").....

Original thread: Problem with multiple recipients at PMs

[Destroy666x]

Whoever will be fixing this, don't forget about these 2 (query optimisation and PM search sorting by sender bug): http://community.mybb.com/thread-168978-post-1150102.html#pid1150102 or feel free to move them to other issues.

[JN-Jones]

@mybb/developers Can somebody look at this? There seem to be fixes available which should be considered and created a PR with them.

[Eldenroot]

@Matslom - can you please check this?

[Eldenroot]

MyBB team, still valid?
@effone @euantorano

[Sama34]

I'm unsure if this was ever fixed but I couldn't replicate in a live board using the last MyBB version.

[euantorano]

I’m not sure either, I seem to recall seeing a patch at some point? If it can’t be replicated on a test board or the community forums, we can probably close this.

…

On 4 Jan 2019, at 22:48, Omar Gonzalez ***@***.***> wrote: I'm unsure if this was ever fixed but I couldn't replicate in a live board using the last MyBB version. <https://user-images.githubusercontent.com/1786584/50714985-d2f09400-102f-11e9-8252-23ab46662306.png> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1966 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAttcWK9N2oB5EtAYw5h0IDUbNvf_KZNks5u_9pKgaJpZM4ECK9f>.

[Sama34]

If Stefan solution was correct, it was applied in b01d134

However, I'm unable to guess if Destroyer's reports are still correct regarding the following lines:
https://github.com/mybb/mybb/blob/feature/private.php#L258-L362 (sorting by sender)
https://github.com/mybb/mybb/blob/feature/private.php#L325 (recipients redundant query)
https://github.com/mybb/mybb/blob/feature/private.php#L2189 (recipients redundant query)

If somebody can confirm such bugs to be fixed I think we can close this without issues.

[effone]

I have done this (or similar) patch already, but it was not this issue?
I am sure my patch got merged, it was a comma / space placing /parsing issue at frontend, select2...

[Eldenroot]

So what about queries mentioned above?

[Ben-MyBB]

I can't seem to replicate this so going to go ahead and close.