[wp-trac] [WordPress Trac] #15447: Cache Post Thumbnails

WordPress Trac wp-trac at lists.automattic.com
Sat Feb 12 12:49:40 UTC 2011


#15447: Cache Post Thumbnails
-----------------------------------+-----------------------------
 Reporter:  TheDeadMedic           |       Owner:  garyc40
     Type:  enhancement            |      Status:  assigned
 Priority:  normal                 |   Milestone:  Future Release
Component:  Post Thumbnails        |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  needs-patch 3.2-early  |
-----------------------------------+-----------------------------

Comment (by garyc40):

 Replying to [comment:12 TheDeadMedic]:
 > Perhaps caching should also depend on the query arg 'cache_results', and
 possibly a new query arg 'update_post_thumbnail_cache' (to match the
 pattern of the existing 'update_post_term_cache' and
 'update_post_meta_cache' args)?
 >
 > So by default (if thumbnail feature supported), thumbnails are cached
 for the main query (i.e. the request), but a theme author has the option
 to cache/not cache for their own loop(s)?

 I don't think that's a good idea. There are loops that don't output
 thumbnails at all, so caching thumbnails whenever a loop start seems
 wasteful.

 Ignore attachment:15447.diff for the time being, I just realized there's a
 bug with it (using permanent cache and modifying the thumbnail of a post
 does not properly invalidate the cache).

 Any suggestion on how to tell if {{{get_post_thumbnail()}}} is being
 called for the first time in a loop?

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


More information about the wp-trac mailing list