Changeset 10816 for trunk/src/bp-notifications/bp-notifications-functions.php

Timestamp:

05/27/2016 05:56:01 AM (9 years ago)

Author:

r-a-y

Message:

Notifications: Ensure moderators are able to manage a user's notifications.

Previously, due to hardcoded logged-in user checks, it was not possible
for moderators to manage a displayed user's notifications.

This commit adds a user ID parameter to all appropriate notification
template tag functions and sets some of those functions to default to the
displayed user ID where necessary. This will allow moderators to manage
a displayed user's notifications as intended.

Props r-a-y, VibeThemes.

Fixes #7023.

File:

• trunk/src/bp-notifications/bp-notifications-functions.php (modified) (2 diffs)

trunk/src/bp-notifications/bp-notifications-functions.php

@@ -99 +99 @@
  */
 function bp_notifications_delete_notification( $id ) {
-    if ( ! bp_notifications_check_notification_access( bp_loggedin_user_id(), $id ) ) {
+    if ( ! bp_notifications_check_notification_access( bp_displayed_user_id(), $id ) ) {
         return false;
     }
@@ -118 +118 @@
  */
 function bp_notifications_mark_notification( $id, $is_new = false ) {
-    if ( ! bp_notifications_check_notification_access( bp_loggedin_user_id(), $id ) ) {
+    if ( ! bp_notifications_check_notification_access( bp_displayed_user_id(), $id ) ) {
         return false;
     }