[buddypress-trac] [BuddyPress] #3476: Change priority of bp_screens (and maybe bp_actions) hooks

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Sun Aug 14 14:12:57 UTC 2011


#3476: Change priority of bp_screens (and maybe bp_actions) hooks
--------------------------+------------------
 Reporter:  boonebgorges  |       Owner:
     Type:  defect        |      Status:  new
 Priority:  major         |   Milestone:  1.5
Component:  Core          |     Version:  1.5
 Severity:  major         |  Resolution:
 Keywords:  has-patch     |
--------------------------+------------------

Comment (by boonebgorges):

 Thanks for the feedback, DJPaul. I've been thinking about this some more
 (in particular, about your a propos question "will it break anything
 else") and I'm pretty convinced that it won't. The only possibility is
 this: someone has hooked a function to 'wp' with priority 3 or 4 (which we
 don't do in BP, so it'd have to be another plugin); on the current setup,
 that function would never fire on a BP page, while with this change, it
 *will* fire. It's very unlikely that a BP-related plugin would ever have
 done this (the plugin would never have worked!). And if it is happening
 with a non-BP plugin, I would say that this change actually *fixes* a
 problem, rather than causes one.

 I'm going to make the change and mark the ticket closed. Let's keep an eye
 (and an ear) out for issues that might be related to screen function load
 order.

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


More information about the buddypress-trac mailing list