[buddypress-trac] [BuddyPress Trac] #6329: Notifications not clearing for deleted messages if they have not been read first.

[buddypress-trac]

#6329: Notifications not clearing for deleted messages if they have not been read
first.
---------------------------------------+---------------------------
 Reporter:  mcpeanut                   |       Owner:  boonebgorges
     Type:  defect (bug)               |      Status:  closed
 Priority:  high                       |   Milestone:  2.3
Component:  Component - Notifications  |     Version:  2.2.1
 Severity:  major                      |  Resolution:  fixed
 Keywords:  has-patch                  |
---------------------------------------+---------------------------
Changes (by boonebgorges):

 * owner:   => boonebgorges
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"9744"]:
 {{{
 #!CommitTicketReference repository="" revision="9744"
 Ensure that corresponding notifications are deleted when a message thread
 is deleted.

 This means deleting the notifications for each message in the thread, for
 each
 recipient on the thread. Previously, only a subset of these were being
 deleted.

 This changeset removes a test that used a dummy thread_id, a testing
 technique that doesn't work with the new looping technique used for
 notification deletion. The new unit test is a proper replacement.

 Props pareshradadiya for an initial patch.
 Fixes #6329.
 }}}

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