Changeset 6310 for trunk/bp-core/admin/bp-core-slugs.php

Timestamp:

09/06/2012 04:24:22 AM (14 years ago)

Author:

johnjamesjacoby

Message:

Admin Audit:

• Introduce bp_is_network_activated() to tell if BuddyPress is active at a network level.
• Use bp_is_network_activated() in places where multiblog or single-site activation might mistakingly link to the incorrect site/network admin.
• Add phpdoc to bp_is_multiblog_mode() to further explain why it exists, and why you should not use it.
• Audit usage of network_admin_url() and admin_url(), and replace with bp_get_admin_url() where appropriate.
• Escape some admin URL usages; maybe more to do here.

File:

• trunk/bp-core/admin/bp-core-slugs.php (modified) (4 diffs)

trunk/bp-core/admin/bp-core-slugs.php

@@ -95 +95 @@
 
                         <td>
-                            <?php if ( !bp_is_root_blog() )
-                                switch_to_blog( bp_get_root_blog_id() ) ?>
+
+                            <?php if ( ! bp_is_root_blog() ) switch_to_blog( bp_get_root_blog_id() ); ?>
 
                             <?php echo wp_dropdown_pages( array(
@@ -114 +114 @@
                             <?php endif; ?>
 
-                            <?php if ( !bp_is_root_blog() )
-                                restore_current_blog() ?>
+                            <?php if ( ! bp_is_root_blog() ) restore_current_blog(); ?>
 
                         </td>
@@ -159 +158 @@
 
                         <td>
+                            
+                            <?php if ( ! bp_is_root_blog() ) switch_to_blog( bp_get_root_blog_id() ); ?>
+
                             <?php echo wp_dropdown_pages( array(
                                 'name'             => 'bp_pages[' . esc_attr( $name ) . ']',
@@ -174 +176 @@
 
                             <?php endif; ?>
+
+                            <?php if ( ! bp_is_root_blog() ) restore_current_blog(); ?>
 
                         </td>