[wp-trac] Re: [WordPress Trac] #9004: More filtering in get_terms() and wp_generate_tag_cloud()

WordPress Trac wp-trac at lists.automattic.com
Tue Apr 21 19:06:48 GMT 2009


#9004: More filtering in get_terms() and wp_generate_tag_cloud()
--------------------------+-------------------------------------------------
 Reporter:  jhodgdon      |       Owner:  ryan                       
     Type:  defect (bug)  |      Status:  new                        
 Priority:  normal        |   Milestone:  2.8                        
Component:  Taxonomy      |     Version:  2.7                        
 Severity:  normal        |    Keywords:  taxonomy, filter, has-patch
--------------------------+-------------------------------------------------

Comment(by filosofo):

 Replying to [comment:7 jhodgdon]:
 > I'm not sure why it's better to pass an array, really...

 The PHP array functions are quicker and potentially more robust in
 filtering the fields than would be a regex replace.  Most people using
 that filter would probably split the string into an array, do their
 business, and join it back together as a string, anyways.  Other comma-
 delimited things-to-be-filtered are handled similarly throughout WP.  I'm
 just suggesting that if we're going to introduce a new filter it should be
 consistent with other filters and convenient for developers, but yeah, it
 doesn't really matter that much.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/9004#comment:8>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list