[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