[buddypress-trac] [BuddyPress] #3622:	bp_get_sitewide_activity_feed_link()
    buddypress-trac at lists.automattic.com 
    buddypress-trac at lists.automattic.com
       
    Sun Sep 25 20:22:54 UTC 2011
    
    
  
#3622: bp_get_sitewide_activity_feed_link()
-------------------------------------------------+-------------------------
 Reporter:  webraket                             |       Owner:
     Type:  defect                               |      Status:  new
 Priority:  normal                               |   Milestone:  1.5.1
Component:  Activity                             |     Version:  1.5
 Severity:  major                                |  Resolution:
 Keywords:  has-patch needs-testing reporter-    |
  feedback                                       |
-------------------------------------------------+-------------------------
Comment (by webraket):
 Replying to [comment:7 boonebgorges]:
 > The patch 3622.02 is attached to this ticket. If you're reading this in
 an email, click through to it.
 >
 > > Sorry, I don't trust bp_init at the moment.
 >
 > bp_init is itself hooked to init.
 http://buddypress.trac.wordpress.org/browser/trunk/bp-core/bp-core-
 hooks.php#L17
 >
 > > you can't add actions to wp_ajax{callback_function} from inside
 classes/functions hooked to bp_init.
 >
 > Load order was changed in 1.5, so you may be right. I suggest that you
 load your base class at bp_loaded. This is what the new BP Skeleton
 Component will recommend when it comes out in the upcoming days:
 https://github.com/boonebgorges/buddypress-skeleton-
 component/blob/1.5-compat/includes/bp-example-loader.php#L364
 bp_loaded is hooked to plugins_loaded, that seems a bit too early.
 I like to construct my base class at least after wordpress loaded the
 current user to make use of functions like is_super_admin() etc.
 I mostly hook my base classes (for BuddyPress only plugins) from bp_init,
 I expect this to still be fine just not for some functions anymore like
 wp_ajax (and probably some other functions).
-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/3622#comment:10>
BuddyPress <http://buddypress.org/>
BuddyPress
    
    
More information about the buddypress-trac
mailing list