[buddypress-trac] [BuddyPress] #3447: Audit use of extract( $r, EXTR_SKIP ) (was: new 'hide empty profile fields' feature)

[buddypress-trac at lists.automattic.com]

#3447: Audit use of extract( $r, EXTR_SKIP )
------------------------+------------------
 Reporter:  dontdream   |       Owner:
     Type:  defect      |      Status:  new
 Priority:  normal      |   Milestone:  1.5
Component:  Core        |     Version:  1.5
 Severity:  normal      |  Resolution:
 Keywords:              |
------------------------+------------------
Changes (by boonebgorges):

 * type:  enhancement => defect
 * component:  XProfile => Core
 * milestone:  Awaiting Review => 1.5


Comment:

 Yes, good catch! To be extra safe, we should probably use EXTR_OVERWRITE
 in addition to different variable names when setting up function defaults,
 eg $hide_empty_fields_default

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