Changeset 10715 for trunk/src/bp-notifications/bp-notifications-template.php

Timestamp:

04/21/2016 07:44:45 AM (9 years ago)

Author:

imath

Message:

Introduce screen notifications for activity comments and activity comment replies

Users will be notified about comments made on their activities and about replies made on their activity comments.

If there is only one notification of the above types, the WP Admin Bar bubble will link to the single activity. Clicking on this link will mark the notification as read.
If there are more than one, we use the notifications screen and an extra parameter to filter the notifications according to the type so that only these notifications are displayed. Clicking on the listed links will mark the notification as read.

Props dcavins, boonebgorges, DJPaul

Fixes #6057

File:

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

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

@@ -168 +168 @@
     } else {
         $user_id = bp_loggedin_user_id();
+    }
+
+    // Set the component action (by default false to get all actions)
+    $component_action = false;
+
+    if ( isset( $_REQUEST['type'] ) ) {
+        $component_action = sanitize_key( $_REQUEST['type'] );
+    }
+
+    // Set the search terms (by default an empty string to get all notifications)
+    $search_terms = '';
+
+    if ( isset( $_REQUEST['s'] ) ) {
+        $search_terms = stripslashes( $_REQUEST['s'] );
     }
 
@@ -176 +190 @@
         'secondary_item_id' => false,
         'component_name'    => bp_notifications_get_registered_components(),
-        'component_action'  => false,
+        'component_action'  => $component_action,
         'is_new'            => $is_new,
-        'search_terms'      => isset( $_REQUEST['s'] ) ? stripslashes( $_REQUEST['s'] ) : '',
+        'search_terms'      => $search_terms,
         'order_by'          => 'date_notified',
         'sort_order'        => 'DESC',