[buddypress-trac] [BuddyPress] #4081: Failed activity pages due to bad logic in bp_is_current_component

[buddypress-trac at lists.automattic.com]

#4081: Failed activity pages due to bad logic in bp_is_current_component
-------------------------------------+---------------------
 Reporter:  chrisbliss18             |       Owner:
     Type:  defect (bug)             |      Status:  closed
 Priority:  normal                   |   Milestone:  1.6
Component:  Core                     |     Version:  1.5.4
 Severity:  normal                   |  Resolution:  fixed
 Keywords:  has-patch needs-testing  |
-------------------------------------+---------------------
Changes (by boonebgorges):

 * status:  new => closed
 * resolution:   => fixed


Comment:

 (In [6096]) Ensures consistent typing when core components register
 themselves in active array

 When core components register themselves in the active_components array
 using
 a boolean, it causes problems with the array_search() logic in
 bp_is_current_component(). By using the int 1 instead, we avoid false
 positives
 from array_search(), and thus from bp_is_current_component().

 Fixes #4081.

 Props chrisbliss18, cnorris23

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4081#comment:8>
BuddyPress <http://buddypress.org/>
BuddyPress