[buddypress-trac] [BuddyPress Trac] #6748: extract() function is still used widely in BuddyPress

[buddypress-trac]

#6748: extract() function is still used widely in BuddyPress
--------------------------------------+----------------------
 Reporter:  slaFFik                   |       Owner:
     Type:  defect (bug)              |      Status:  closed
 Priority:  normal                    |   Milestone:
Component:  Tools - Code Improvement  |     Version:
 Severity:  normal                    |  Resolution:  invalid
 Keywords:                            |
--------------------------------------+----------------------

Comment (by boonebgorges):

 > It is something we're moving away from when we make changes, but I don't
 think we are intending to refactor old code like this "just because".

 One exception I'll make is this: if you're removing `extract()` use from a
 function that currently has insufficient unit test coverage, and your
 patch includes tests :) Otherwise, let's continue to remove `extract()` as
 we make other improvements to the functions in question.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6748#comment:2>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac