[wp-trac] [WordPress Trac] #14759: Improve the way oEmbed deals with caching
WordPress Trac
noreply at wordpress.org
Tue Nov 26 12:24:37 UTC 2013
#14759: Improve the way oEmbed deals with caching
--------------------------+-----------------------------
Reporter: Viper007Bond | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Future Release
Component: Embeds | Version: 3.0.1
Severity: normal | Resolution:
Keywords: |
--------------------------+-----------------------------
Comment (by sanchothefat):
Replying to [comment:27 Denis-de-Bernardy]:
> Replying to [comment:26 rmccue]:
> > `add_option()` takes an autoload parameter that can be set to `'no'`
instead, plus this stores it per-site rather than network-wide. Seems
better than files to me. :)
>
> But then you're adding extra queries. And storing this kind of stuff in
wp_options isn't so great an idea — recall Magpie.
>
> Btw, what's wrong with the idea I suggested earlier, i.e. on WP upgrade,
delete all cached oEmbeds? Not by service: all of them. No ifs, no buts.
Problem solved if we do that, no?
How would you go about caching within WP_oEmbed? Same as in Lee's patch
and passing in `$usecache` as false in the context of WP_Embed?
--
Ticket URL: <http://core.trac.wordpress.org/ticket/14759#comment:29>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list