MyBB Community Forums
[F] /me bug in PM [R] [C-Chris] - Printable Version

+- MyBB Community Forums (https://community.mybb.com)
+-- Forum: Community Archive (https://community.mybb.com/forum-106.html)
+--- Forum: Archived Forums (https://community.mybb.com/forum-143.html)
+---- Forum: Archived Development and Support (https://community.mybb.com/forum-155.html)
+----- Forum: Archived Bug Reports (https://community.mybb.com/forum-74.html)
+------ Forum: MyBB 1.4.4 (https://community.mybb.com/forum-110.html)
+------ Thread: [F] /me bug in PM [R] [C-Chris] (/thread-46492.html)



[F] /me bug in PM [R] [C-Chris] - Imad Jomaa - 03-11-2009

When you send a PM and include the /me, it works fine. However, when someone replies to you, which quotes your text, the /me username becomes the username of who replied. Shouldn't it just quote it and put it in plain text?

Reproduce:

1) Login to one account, send a PM containing a '/me' (without the quotes and some text).

2) Now login to the account that you sent the PM to, click on "reply" and send it.

3) Now login back to the first account and view the PM, you'll notice the '/me' username becomes the one who replied to your PM.

Can anyone reproduce?


RE: /me bug in PM - bobbit - 03-11-2009

I can confirm this.


RE: /me bug in PM - x_Stricken_x - 03-11-2009

Yeah, this is definitely reproducible.


RE: /me bug in PM [R] - dvb - 03-11-2009

Possible fix:
open private.php, find:
			$pm['message'] = preg_replace('#^/me (.*)$#im', "* ".$pm['quotename']." \\1", $pm['message']);
add after:
			$message = preg_replace('#^/me (.*)$#im', "* ".$pm['quotename']." \\1", $message);

I think the first line (with $pm['message']) is unnecessary but I'm not sure so I wrote "add after" but maybe it should be replaced with the second line?


RE: /me bug in PM [R] - Imad Jomaa - 03-11-2009

Your fix works. Adding after seems more appropriate as replacing it isn't going to fix the problem. The pm['message'] preg_replace should work though alone?


RE: /me bug in PM [R] - Ryan Gordon - 03-15-2009

The correct patch is to apply dvb's fix but replace instead of add after.


[F] /me bug in PM [R] - Ryan Gordon - 03-15-2009

Thank you for your bug report.

This bug has been fixed in our internal code repository. Please note that the problem will not be fixed here until these forums are updated.

With regards,
MyBB Group