[buddypress-trac] [BuddyPress Trac] #7140: Class autoloading needs to check if the component is active before including the class
buddypress-trac
noreply at wordpress.org
Fri Jun 24 18:52:40 UTC 2016
#7140: Class autoloading needs to check if the component is active before
including the class
--------------------------+--------------------
Reporter: r-a-y | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 2.6.1
Component: Core | Version: 2.6.0
Severity: major | Resolution:
Keywords: has-patch |
--------------------------+--------------------
Comment (by r-a-y):
Using parenthesis would make sense if we were checking multiple conditions
for one clause.
For example:
`( true === bp_is_active( 'members' ) || true === bp_is_active( 'groups' )
)`
But since the patch is only checking single conditions, I don't think it
is necessary.
I'm kind of agnostic either way, so whatever everyone thinks is best is
okay with me.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7140#comment:3>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list