[buddypress-trac] [BuddyPress] #2718: Swap function_exists() calls to bp_is_active()
buddypress-trac at lists.automattic.com
buddypress-trac at lists.automattic.com
Sun Nov 7 21:08:56 UTC 2010
#2718: Swap function_exists() calls to bp_is_active()
-------------------------+--------------------------------------------------
Reporter: DJPaul | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 1.3
Component: Core | Version:
Keywords: |
-------------------------+--------------------------------------------------
Description changed by DJPaul:
Old description:
New description:
Trunk currently has a mixture of function_exists() (on different function
names) and bp_is_active() checks to decide if a certain part of BuddyPress
is loaded. bp_is_active() was added earlier this year to eventually
replace the function_exists() checks, if I remember correctly.
We should standardise on one for BuddyPress 1.3, so that core is more
consistent. I suggest bp_is_active() as it prevents any third-party themes
or plugins checking for the presence of functions which could possibly be
removed or deprecated.
--
--
Ticket URL: <http://trac.buddypress.org/ticket/2718#comment:2>
BuddyPress <http://buddypress.org/>
BuddyPress
More information about the buddypress-trac
mailing list