[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