Changeset 7715

Timestamp:

12/23/2013 06:31:37 PM (13 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. (trunk)

File:

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

trunk/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_registered_components', $component_names, $active_components );
+}