[wp-trac] [WordPress Trac] #17134: Add support for the_slug() in template tags

WordPress Trac noreply at wordpress.org
Fri Feb 1 06:32:18 UTC 2013


#17134: Add support for the_slug() in template tags
-----------------------------+------------------------------
 Reporter:  tomauger         |       Owner:
     Type:  feature request  |      Status:  closed
 Priority:  normal           |   Milestone:  Awaiting Review
Component:  Template         |     Version:
 Severity:  normal           |  Resolution:  wontfix
 Keywords:  has-patch        |
-----------------------------+------------------------------
Changes (by iandunn):

 * keywords:  2nd-opinion has-patch => has-patch
 * status:  reopened => closed
 * resolution:   => wontfix


Comment:

 1. I see your points about filters; I've removed them from the patch.
 2. I don't personally like the idea of denying something to everyone
 because a few people will abuse or misunderstand it, but I understand your
 reasoning.
 3. That's also a good point, it'd have to be tested and possibly handled
 if this were to go anywhere.
 4. Same as !#2, anything can be misused, but I don't feel like that's a
 good reason not do it.  I don't think the_slug() is essential, though, so
 I removed that too.

 So, I don't really agree with all your reasons, but I do appreciate both
 of you taking the time to articulate why you're against it. I'll move on
 to other tickets and just use get_post( $id )->post_name in the future.
 That way still provides some abstraction in the (albeit unlikely) event
 that $post changes in the future.

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


More information about the wp-trac mailing list