#6445 closed defect (bug) (fixed)
Notifications: Invalidate cache when updating a notification
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 2.3 | Priority: | normal |
Severity: | normal | Version: | 1.9 |
Component: | Toolbar & Notifications | Keywords: | has-patch |
Cc: |
Description
If you're on your notifications page and you click on the "Read" action link for a notification, the user's notifications cache isn't invalidated.
Problem is due to not pulling up the user ID before updating the notification.
I've attached a patch with unit tests.
The patch adds a new hook - 'bp_notification_before_update'
- which mirrors the other before_save
and before_delete
hooks and moves the cache clearing to bp-notifications-cache.php
.
Attachments (1)
Change History (6)
Note: See
TracTickets for help on using
tickets.
In 9887: