Current time: 02-07-2012, 05:32 AM Hello There, Guest! (LoginRegister)


 
Thread Rating:
  • 0 Votes - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
[F] /me bug in PM [R] [C-Chris]
03-11-2009, 02:24 AM
Post: #1
[F] /me bug in PM [R] [C-Chris]
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?

Clicktotweet.me
Find all posts by this user
03-11-2009, 02:31 AM
Post: #2
RE: /me bug in PM
I can confirm this.
Visit this user's website Find all posts by this user
03-11-2009, 03:26 AM
Post: #3
RE: /me bug in PM
Yeah, this is definitely reproducible.

Elite-Area / Last.FM Profile / PSN Name: XDaciD / Minecraft Map Progress: Coming Soon!
Visit this user's website Find all posts by this user
03-11-2009, 01:15 PM
Post: #4
RE: /me bug in PM [R]
Possible fix:
open private.php, find:
PHP Code:
            $pm['message'] = preg_replace('#^/me (.*)$#im'"* ".$pm['quotename']." \\1"$pm['message']); 
add after:
PHP Code:
            $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?

Creativity is a drug I cannot live without.
[Image: igG319dTu71gT.png]
Visit this user's website Find all posts by this user
03-11-2009, 03:55 PM (This post was last modified: 03-11-2009 03:57 PM by Imad Jomaa.)
Post: #5
RE: /me bug in PM [R]
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?

Clicktotweet.me
Find all posts by this user
03-15-2009, 11:52 PM
Post: #6
RE: /me bug in PM [R]
The correct patch is to apply dvb's fix but replace instead of add after.
Visit this user's website Find all posts by this user
03-15-2009, 11:52 PM
Post: #7
[F] /me bug in PM [R]
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
Visit this user's website Find all posts by this user


Forum Jump:


User(s) browsing this thread: 1 Guest(s)

Contact Us | MyBB | Return to Top | Return to Content | Lite (Archive) Mode | RSS Syndication