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

buddypress-trac noreply at wordpress.org
Tue Dec 17 17:44:21 UTC 2013


#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


More information about the buddypress-trac mailing list