Changeset 5232

Timestamp:

12/23/2013 09:29:19 PM (11 years ago)

Author:

johnjamesjacoby

Message:

Introduce helper function for getting the BuddyPress component name, and filter new BuddyPress function to include it in registered components. Fixes #2495. Hat-tip imath. (trunk)

Location:

trunk/includes/extend/buddypress

Files:

• functions.php (modified) (1 diff)
• notifications.php (modified) (3 diffs)

trunk/includes/extend/buddypress/functions.php

@@ -13 +13 @@
 
 /** BuddyPress Helpers ********************************************************/
+
+/**
+ * Return bbPress's component name/ID ('forums' by default)
+ *
+ * This is used primarily for Notifications integration.
+ *
+ * @since bbPress (r5232)
+ * @return string
+ */
+function bbp_get_component_name() {
+
+    // Use existing ID
+    if ( !empty( bbpress()->extend->buddypress->id ) ) {
+        $retval = bbpress()->extend->buddypress->id;
+
+    // Use default
+    } else {
+        $retval = 'forums';
+    }
+
+    return apply_filters( 'bbp_get_component_name', $retval );
+}
 
 /**

trunk/includes/extend/buddypress/notifications.php

@@ -1 +1 @@
 <?php
+
+/**
+ * Filter registered notifications components, and add 'forums' to the queried
+ * 'component_name' array.
+ *
+ * @since bbPress (r5232)
+ *
+ * @see BP_Notifications_Notification::get()
+ * @param array $component_names
+ * @return array
+ */
+function bbp_filter_notifications_get_registered_components( $component_names = array() ) {
+
+    // Force $component_names to be an array
+    if ( ! is_array( $component_names ) ) {
+        $component_names = array();
+    }
+
+    // Add 'forums' component to registered components array
+    array_push( $component_names, bbp_get_component_name() );
+
+    // Return component's with 'forums' appended
+    return $component_names;
+}
+add_filter( 'bp_notifications_get_registered_components', 'bbp_filter_notifications_get_registered_components', 10 );
 
 /**
@@ -85 +110 @@
 
     // Get some reply information
-    $date_notified    = get_post( $reply_id )->post_date;
-    $item_id          = $topic_id;
-    $component_name   = 'forums';
-    $component_action = 'bbp_new_reply';
+    $args = array(
+        'user_id'          => $topic_author_id,
+        'item_id'          => $topic_id,
+        'component_name'   => bbp_get_component_name(),
+        'component_action' => 'bbp_new_reply',
+        'date_notified'    => get_post( $reply_id )->post_date,
+    );
 
-    // Notify the topic author if not the current reply author
-    if ( $author_id !== $topic_author_id ) {
-        bp_core_add_notification( $item_id, $topic_author_id, $component_name, $component_action, $secondary_item_id, $date_notified );
-    }
+    // Notify the topic author if not the current reply author
+    if ( $author_id !== $topic_author_id ) {
+        $args['secondary_item_id'] = $secondary_item_id ;
 
-    // Notify the immediate reply author if not the current reply author
-    if ( !empty( $reply_to ) && ( $author_id !== $reply_to_item_id ) ) {
-        bp_core_add_notification( $item_id, $topic_author_id, $component_name, $component_action, $reply_to_item_id, $date_notified );
-    }
+        bp_notifications_add_notification( $args );
+    }
+ 
+    // Notify the immediate reply author if not the current reply author
+    if ( !empty( $reply_to ) && ( $author_id !== $reply_to_item_id ) ) {
+        $args['secondary_item_id'] = $reply_to_item_id ;
+
+        bp_notifications_add_notification( $args );
+    }
 }
 add_action( 'bbp_new_reply', 'bbp_buddypress_add_notification', 10, 7 );
@@ -138 +170 @@
 
         // Attempt to clear notifications for the current user from this topic
-        $success = bp_core_mark_notifications_by_item_id( $user_id, $topic_id, 'forums', 'bbp_new_reply' );
+        $success = bp_notifications_mark_notifications_by_item_id( $user_id, $topic_id, bbp_get_component_name(), 'bbp_new_reply' );
 
         // Do additional subscriptions actions