[buddypress-trac] [BuddyPress] #5290: Getting notifications for a user should check for active components

[buddypress-trac]

#5290: Getting notifications for a user should check for active components
------------------------------------------------+--------------------------
 Reporter:  imath                               |       Owner:
     Type:  defect (bug)                        |  boonebgorges
 Priority:  normal                              |      Status:  closed
Component:  Notifications                       |   Milestone:  1.9
 Severity:  normal                              |     Version:  1.8.1
 Keywords:  has-patch 2nd-opinion dev-feedback  |  Resolution:  fixed
------------------------------------------------+--------------------------

Comment (by johnjamesjacoby):

 In [changeset:"7687"]:
 {{{
 #!CommitTicketReference repository="" revision="7687"
 Notifications:

 Make `buddypress()->active_components` the default value for the
 `component_name` argument.

 This is necessary because notifications are persistent in the database,
 and active components register callbacks for generating output based on
 the queried results. If notifications exist for components that are
 inactive, no callback will be registered, and the output will be empty. By
 only querying for active components, we can be sure no empty notifications
 will be displayed to the user.

 Reverts r7677. See #5290. Fixes #5300. (1.9 branch)
 }}}

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5290#comment:7>
BuddyPress <http://buddypress.org/>
BuddyPress