Sent answered messages are marked as unread

[motomac]

If I send a private message, after that I have a notification about unread message in BuddyBar. And it displays untill I open my inbox folder.

[apeatling]

Correct. It's called a notification of a new message. :)

[r-a-y]

Are you sure this isn't a defect?

I'm experiencing this on testbp.org.

I send a private message, why should my own sent message be counted as a notification?

[apeatling]

Just tested this, it does not happen for me. If you are sending the message to yourself then you will get a notification.

[r-a-y]

It happens when I reply to a private message.

[motomac]

Bug still takes place.

[motomac]

Bug still takes place in 1.2.4.

[pisanojm]

I've looked through the trac and wonder if this is related to the a problem I am having with Private messages @Rivh!etivit said there was a trac for this... We still have the problem with 1.2.5 and 3.0 here is the buddypress.org thread:
http://buddypress.org/community/groups/how-to-and-troubleshooting/forum/topic/private-messate-notification-ghost/

[nuprn1]

the issue only happens if you reply to a message thread (even under sent message - if recipient did not respond)

the problem is caused by get_recipients() array containing the user_id for the user who sent the reply. (due to the loop with the notification)

1) send message to a different user
2) view message thread under "Sent Message"
3) reply to message
4) click to the home page (notification then displays)

[r-a-y]

I just realized after attaching the above patch that there's a better way to block the sender from the recipient's list. Check out line 447-448 in bp-messages.php.

I'm going to look into this and provide another patch, unless someone does it before me ;)

[pisanojm]

can you provide new patch to test?

[pisanojm]

Ray, I've tried this and it appears to be working... I have a couple of users trying this... if you DON'T hear back from me here...consider it WORKING from my stand point... Great job!

[motomac]

I confirm. Patch works good.

[r-a-y]

Hey pisanojm and motomac, I got sidetracked!

The current patch works, but I would still like to provide a better patch as the current one does the check a little late in the game.

It should be made where Rich (@nuprn1) stated around line 425 in bp-messages.php or in BP_Messages_Thread::get_recipients() in bp-messages-classes.php.

[r-a-y]

Or maybe the current one works just fine? ;)

[pisanojm]

Ray,

Ok... possibly an issue with this.. after I put this patch on seems like now nobody get's notification or emails sent to them for notification... When I turned the patch off (i.e. back to the original) the ghosting started again, but people were being notified again...

Please check this and confirm this to be happening... I will try this again tonight in more detail.

[pisanojm]

Ray,

I've tried this extensively tonight and it DOES seem to be working correctly. I suspect, now, it was a false alarm by a USER on my site that didn't understand a couple of things. I checked it fairly throughly tonight A/B applied/unapplied and the result was consistant both in what it was supposed to fix and with regard to e-mails/notificaiton still be sent.

Sorry for the above comment. If no one else is suspecting the behavior I described above, I'd say it's good to go!

[johnjamesjacoby]

(In [3140]) Fixes #2068