[wp-trac] [WordPress Trac] #21270: Use update_blog_option() when enabling/disabling themes per site in Multisite

WordPress Trac wp-trac at lists.automattic.com
Tue Jul 31 21:15:06 UTC 2012


#21270: Use update_blog_option() when enabling/disabling themes per site in
Multisite
--------------------------+-----------------------
 Reporter:  jeremyfelt    |       Owner:  ryan
     Type:  defect (bug)  |      Status:  reopened
 Priority:  normal        |   Milestone:  3.5
Component:  Cache         |     Version:
 Severity:  minor         |  Resolution:
 Keywords:  has-patch     |
--------------------------+-----------------------
Changes (by nacin):

 * status:  closed => reopened
 * resolution:  fixed =>


Comment:

 Re-opening this for some further discussion:
  * _blog_option() was never good, but these changes do highlight some
 problems because a switch_to_blog() clears the local cache. If you don't
 have a persistent backend, this means a single get_blog_option() wipes out
 most of your cache for the rest of the page.
  * We should remove all usage of _blog_option() in core.
  * We should deprecate _blog_option() in favor of "switch_to_blog(),
 get_option()" to ensure that people actually do realize that
 get_blog_option() is no less expensive.
  * We should improve our default caching backend to have a way to init it
 for a single blog, without losing the existing local cache.

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


More information about the wp-trac mailing list