[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)                        |      Status:  new
 Priority:  normal                              |   Milestone:  2.0
Component:  Notifications                       |     Version:  1.8.1
 Severity:  normal                              |  Resolution:
 Keywords:  has-patch 2nd-opinion dev-feedback  |
------------------------------------------------+--------------------
Changes (by imath):

 * keywords:  has-patch 2nd-opinion => has-patch 2nd-opinion dev-feedback


Comment:

 Replying to [comment:1 boonebgorges]:
 > Let's look at this for 2.0.

 Boone, i'm sorry to insist, but it's a regression. In 1.8.1, if the
 component is not active, then the notification is not added in the WP
 Admin Bar. I'm sorry i should have checked in 1.8.1 before :(
 see http://buddypress.trac.wordpress.org/browser/tags/1.8.1/bp-members/bp-
 members-notifications.php#L107

 Although risk is very weak that an administrator deactivate a component
 once his BuddyPress is set, it's there. So maybe 2.0 might be a bit late,
 what do you think of keeping an eye on this and maybe add it in to a minor
 version if feedbacks about this trouble are sent by users ?

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