[buddypress-trac] [BuddyPress Trac] #6937: fatal error when trashing emails

[buddypress-trac]

#6937: fatal error when trashing emails
-------------------------------+--------------------
 Reporter:  DJPaul             |       Owner:
     Type:  defect (bug)       |      Status:  new
 Priority:  normal             |   Milestone:  2.5.1
Component:  Component - Blogs  |     Version:  2.5.0
 Severity:  normal             |  Resolution:
 Keywords:  has-patch          |
-------------------------------+--------------------

Comment (by r-a-y):

 > I keep on thinking only loading the bp-blogs/bp-blogs-activity.php if
 the Activity Component is active is a good improvement as it avoids
 loading a bunch of code that is not used anyway.

 Darn it!  This was my fault! :(

 Here's an alternate patch that moves the activity function calls from `bp-
 blogs-functions.php` to `bp-blogs-activity.php`.

 Benefits are no more `bp_is_active()` checks and code is hooked.  The code
 changes may be too extreme for v2.5.1, so I'm thinking we roll with
 imath's fix for v2.5.1 and look at my patch for v2.6.0 instead.

 To whomever commits, feel free to give me anti-props! :)

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6937#comment:12>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac