Notifcations for new replies do not get marked as read

[konnektiv]

When clicking on the notification for a new topic reply the notification is not marked as read an therefore does not disappear from the displayed notifications. This is due to the notifications are updated by item_id which is set as the topic_id, but are created with the reply_id as the notification item_id. The attached patch modifies the notification component action to include the topic_id and then marks all notifications as read with the same component action. This has the added benefit that if a user is notified of replies to different topics the user created these are shown as different notifications for each topic. Clicking on each notification opens the correct topic instead of just the first one as it is now.

[johnjamesjacoby]

This looks pretty good!

[johnjamesjacoby]

In 7183:

BuddyPress: add topic ID to bbp_new_reply notification type.

This commit adds more context to otherwise vague topic reply notifications, and also more accurately marks notifications as read when clicking on the adminbar menu to view them.

In trunk for 2.7.0. See #3341.

Props konnektiv.

[johnjamesjacoby]

In 7184:

BuddyPress: add topic ID to bbp_new_reply notification type.

This commit adds more context to otherwise vague topic reply notifications, and also more accurately marks notifications as read when clicking on the adminbar menu to view them.

In branches/2.6 for 2.6.7. Fixes #3341.

Props konnektiv.