[wp-trac] [WordPress Trac] #4575: Add functions to return the last-modified timestamp of a category/tag
WordPress Trac
noreply at wordpress.org
Sun Jun 14 19:42:40 UTC 2015
#4575: Add functions to return the last-modified timestamp of a category/tag
--------------------------------------------+-----------------------
Reporter: delusions | Owner: jorbin
Type: enhancement | Status: reopened
Priority: normal | Milestone: 4.3
Component: Feeds | Version: 2.2.1
Severity: normal | Resolution:
Keywords: has-patch needs-testing commit | Focuses: template
--------------------------------------------+-----------------------
Changes (by nacin):
* status: closed => reopened
* resolution: fixed =>
Comment:
Something feels a bit off about this. I can't put my finger on it.
* The query for a comments feed looks kind of nasty. Is it actually
necessary for `/comments/feed/`? I don't think so. I think it's only
necessary for a comment feed for an individual post, which should be a lot
easier to query and wouldn't require the IN or MAX. Can a comment feed
other than `/comments/feed/` have comments from more than one post? Should
we even try to support that with an insane query that could be a slow
query against a huge DB table? I'm not sure.
* This doesn't patch atom feeds.
* What's the purpose of get_lastcommentmodified()? Assuming the atom feed
is also updated, it won't be used anymore. Seems like we might need to do
some tweaks.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/4575#comment:29>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list