[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