[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
------------------------------------------------+--------------------------
Changes (by boonebgorges):

 * owner:   => boonebgorges
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"7677"]:
 {{{
 #!CommitTicketReference repository="" revision="7677"
 Don't show notifications from inactive components in admin bar

 Inactive components don't have formatting callbacks registered, which
 means
 that those notifications will not be properly formatted. In addition, the
 target links of the notifications will 404, with the result that the
 notifications will not be marked as read when clicked.

 Fixes #5290

 Props imath
 }}}

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