[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