Changeset 9698 for trunk/src/bp-members/bp-members-screens.php

Timestamp:

04/05/2015 06:18:13 PM (10 years ago)

Author:

boonebgorges

Message:

Introduce bp_core_get_root_option() and use throughout BP.

Previously, we referenced the $bp->site_options array directly. This causes
problems in cases where these options may be referenced before the array is
initially populated. bp_core_get_root_option() will fetch the requested
value from the array if it's been populated, and will populate it if it has not.

Fixes #6045.

File:

• trunk/src/bp-members/bp-members-screens.php (modified) (2 diffs)

trunk/src/bp-members/bp-members-screens.php

@@ -157 +157 @@
         // Finally, let's check the blog details, if the user wants a blog and blog creation is enabled
         if ( isset( $_POST['signup_with_blog'] ) ) {
-            $active_signup = $bp->site_options['registration'];
+            $active_signup = bp_core_get_root_option( 'registration' );
 
             if ( 'blog' == $active_signup || 'all' == $active_signup ) {
@@ -197 +197 @@
 
             // No errors! Let's register those deets.
-            $active_signup = !empty( $bp->site_options['registration'] ) ? $bp->site_options['registration'] : '';
+            $active_signup = bp_core_get_root_option( 'registration' );
 
             if ( 'none' != $active_signup ) {