[wp-trac] [WordPress Trac] #14740: 'taxonomy_template' Filter Name Clash

WordPress Trac wp-trac at lists.automattic.com
Wed Dec 1 23:39:35 UTC 2010


#14740: 'taxonomy_template' Filter Name Clash
-----------------------------+----------------------------------------------
 Reporter:  CharlesClarkson  |        Owner:          
     Type:  defect (bug)     |       Status:  reopened
 Priority:  normal           |    Milestone:  3.1     
Component:  Taxonomy         |      Version:  3.0     
 Severity:  normal           |   Resolution:          
 Keywords:  has-patch        |  
-----------------------------+----------------------------------------------

Comment(by nacin):

 For one, I haven't a clue what `%l` is. (It's an L, not a 1.) It's not a
 standard sprintf() notation, and I'm just for the first time discovering
 wp_sprintf(). I won't even get into how disjointed that function looks.

 Template tags are about ease of use. That also means deciding on new
 function arguments (which you chose) versus wp_parse_args (which you used
 in your example) versus a filter (which would probably be better here,
 because it's such a narrow use case).

 It's not like this template tag has another 15 arguments. It has one,
 $post. Doesn't get easier to use than that :-)

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


More information about the wp-trac mailing list