[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