[buddypress-trac] [BuddyPress Trac] #6045: $bp->site_options should not be referenced directly

[buddypress-trac]

#6045: $bp->site_options should not be referenced directly
--------------------------+------------------
 Reporter:  boonebgorges  |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  2.2
Component:  Core          |     Version:
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |
--------------------------+------------------

Comment (by boonebgorges):

 > is our site_options cache necessary anymore (other than for backwards
 compatibility?)

 I'll have to look into this a little bit more. In the case of options
 fetched on the current site, they are cached persistently, and are also
 loaded into the cache with a single query ("alloptions") for sites that
 don't have a persistent cache. Site options don't work like this - they're
 not prefetched en masse, but are individually cached in the 'site-options'
 group. So removing this stuff would probably increase by a handful the
 number of database queries on sites without persistent caching. I'd have
 to study it a bit more to see what happens on different kinds of setups
 (like on a non-root blog).

 If we could tear this stuff out, I'd be delighted to do so, but I don't
 want to place too harsh a penalty on sites with no persistent cache
 backend.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6045#comment:5>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac