Skip to:
Content

BuddyPress.org

Opened 11 years ago

Closed 11 years ago

#6329 closed defect (bug) (fixed)

Notifications not clearing for deleted messages if they have not been read first.

Reported by: mcpeanut Owned by: boonebgorges
Priority: high Milestone: 2.3
Component: Toolbar & Notifications Version: 2.2.1
Severity: major Keywords: has-patch
Cc:

Description

This is also confirmed by Henrywright and danbp

Attachments (1)

6329.diff​ (969 bytes ) - added by pareshradadiya 11 years ago.
messages_thread_deleted_thread hook added to force delete corresponding notifications when message is deleted and some recipients remain for message

Download all attachments as: .zip

Change History (6)

#2 @DJPaul
11 years ago

  • Milestone Awaiting Review → 2.3

Oh, sadness. :(

Good catch, and thanks for bringing it here from the forum.

@pareshradadiya
11 years ago

messages_thread_deleted_thread hook added to force delete corresponding notifications when message is deleted and some recipients remain for message

#3 @pareshradadiya
11 years ago

  • Keywords has-patch added; needs-patch removed

#4 @boonebgorges
11 years ago

Thanks for the patch, pareshradadiya. I think we can do this without repurposing the hook name, though.

#5 @boonebgorges
11 years ago

  • Owner set to boonebgorges
  • Resolution → fixed
  • Status new → closed

In 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.

Note: See TracTickets for help on using tickets.