[buddypress-trac] [BuddyPress Trac] #7569: Bulk message actions (Mark as Read) do not affect corresponding notifications

[buddypress-trac]

#7569: Bulk message actions (Mark as Read) do not affect corresponding
notifications
-----------------------------------+------------------
 Reporter:  boonebgorges           |       Owner:
     Type:  defect (bug)           |      Status:  new
 Priority:  normal                 |   Milestone:  3.0
Component:  Messages               |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  has-patch 2nd-opinion  |
-----------------------------------+------------------
Changes (by boonebgorges):

 * keywords:   => has-patch 2nd-opinion


Comment:

 [attachment:7569.diff] fixes this by hooking directly to the
 `messages_thread_mark_as_read` action.

 I think, though I haven't tested, that this is a better and more general
 way of handling message notifications than the technique used in
 `bp_messages_screen_conversation_mark_notifications()`, and so perhaps
 could replace it. But for this, I'd like a second opinion from
 @johnjamesjacoby (who originally wrote this [7621]) or @r-a-y (who touched
 it last [8202]).

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7569#comment:1>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac