[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