[buddypress-trac] [BuddyPress] #3399: Audit usage of bp_is_current_component()
buddypress-trac at lists.automattic.com
buddypress-trac at lists.automattic.com
Wed Aug 31 01:14:17 UTC 2011
#3399: Audit usage of bp_is_current_component()
------------------------------+---------------------
Reporter: r-a-y | Owner:
Type: enhancement | Status: closed
Priority: normal | Milestone: 1.5
Component: Core | Version:
Severity: normal | Resolution: fixed
Keywords: has-patch commit |
------------------------------+---------------------
Comment (by johnjamesjacoby):
Note I switched the order of a few things not in this exact patch, putting
bp_is_user() ahead of the component checks, etc...
When doing multiple checks in succession, always put the check with the
least amount of processing first.
Otherwise, thanks as always for the great patch.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/3399#comment:10>
BuddyPress <http://buddypress.org/>
BuddyPress
More information about the buddypress-trac
mailing list