Changeset 7713

Timestamp:

12/23/2013 06:28:04 PM (11 years ago)

Author:

johnjamesjacoby

Message:

In bp_notifications_get_registered_components(), loop through active components looking for notification_callback and only include components that actually touch Notifications. See #5300. (1.9 branch)

File:

• branches/1.9/bp-notifications/bp-notifications-functions.php (modified) (1 diff)

branches/1.9/bp-notifications/bp-notifications-functions.php

@@ -519 +519 @@
 function bp_notifications_get_registered_components() {
 
+    // Load BuddyPress
+    $bp = buddypress();
+
+    // Setup return value
+    $component_names = array();
+
     // Get the active components
-    $active_components = buddypress()->active_components;
-
-    // Get the component ID's from the active_components array
-    $component_names   = array_keys( $active_components );
+    $active_components = array_keys( $bp->active_components );
+
+    // Loop through components, look for callbacks, add to return value
+    foreach ( $active_components as $component ) {
+        if ( !empty( $bp->$component->notification_callback ) ) {
+            $component_names[] = $component;
+        }
+    }
 
     // Return active components with registered notifications callbacks
-    return apply_filters( 'bp_notifications_get_component_names', $component_names );
-}
+    return apply_filters( 'bp_notifications_get_component_names', $component_names, $active_components );
+}