[buddypress-trac] [BuddyPress] #4914: BP_Groups_Component->setup_globals should use $bp->site_options

buddypress-trac noreply at wordpress.org
Wed Apr 3 23:50:32 UTC 2013


#4914: BP_Groups_Component->setup_globals should use $bp->site_options
-------------------------+------------------
 Reporter:  wpdennis     |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  1.8
Component:  Groups       |     Version:
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |
-------------------------+------------------
Changes (by boonebgorges):

 * keywords:  dev-feedback => has-patch
 * milestone:  Awaiting Review => 1.8


Comment:

 This is definitely a problem.

 > bp_get_option() should check if the needed value is already loaded

 This is overly broad, and might cause name clashes.

 We should be doing something closer to your first suggestion: finding the
 spots where we use bp_get_option() to fetch a value that we've already
 cached in `site_options`.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4914#comment:1>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list