[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