[buddypress-trac] [BuddyPress Trac] #7023: Administrator not able to delete user notifications

[buddypress-trac]

#7023: Administrator not able to delete user notifications
---------------------------------------+--------------------
 Reporter:  VibeThemes                 |       Owner:
     Type:  defect (bug)               |      Status:  new
 Priority:  normal                     |   Milestone:  2.6
Component:  Component - Notifications  |     Version:  2.5.0
 Severity:  normal                     |  Resolution:
 Keywords:  has-patch has-screenshots  |
---------------------------------------+--------------------
Changes (by r-a-y):

 * milestone:  Awaiting Review => 2.6


Comment:

 Thanks for the patch, @VibeThemes.

 However, we can't just change the user domain to the displayed user's as
 some devs might already be using those functions to want the logged-in
 user's notification links.

 Instead `01.patch` adds a `$user_id` parameter to all notification action
 links and uses the displayed user as the default there.

 Also for the check access mods, I modified
 `bp_notifications_delete_notification()` and
 `bp_notifications_mark_notification()` instead.

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