[wp-trac] [WordPress Trac] #22369: get_transient() do not call delete_transient() when deleting one

WordPress Trac noreply at wordpress.org
Sat Nov 10 07:34:17 UTC 2012


#22369: get_transient() do not call delete_transient() when deleting one
--------------------------+------------------------------
 Reporter:  juliobox      |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  General       |     Version:  2.8
 Severity:  normal        |  Resolution:
 Keywords:  close         |
--------------------------+------------------------------
Changes (by nacin):

 * keywords:   => close
 * version:  trunk => 2.8


Comment:

 The transient isn't being "deleted" — it has simply expired and we've used
 the opportunity here to clean it up. (We could just as easily do it on
 cron or on upgrade — things we've considered.) A separate "expired
 transient" hook could be beneficial, but it wouldn't always work. That's
 because in environments with an object caching backend such as APC or
 Memcache, transient caching is completely offloaded there. Data in an
 external backend could get pushed out of cache at any time.

 I do think the `return false` should go through the existing
 `transient_$transient` filter, though. When using an external backend, the
 false from wp_cache_get() does go through the filter, so this should as
 well.

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


More information about the wp-trac mailing list