Changeset 6310 for trunk/bp-forums/bp-forums-admin.php

Timestamp:

09/06/2012 04:24:22 AM (12 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-forums/bp-forums-admin.php (modified) (3 diffs)

trunk/bp-forums/bp-forums-admin.php

@@ -4 +4 @@
 
 function bp_forums_add_admin_menu() {
-    global $bp;
 
     if ( !is_super_admin() )
@@ -23 +22 @@
  */
 function bp_forums_bbpress_admin() {
-    global $bp;
 
     // The text and URL of the Site Wide Forums button differs depending on whether bbPress
@@ -110 +108 @@
 
 function bp_forums_bbpress_install_wizard() {
-    $post_url                 = network_admin_url( 'admin.php?page=bb-forums-setup' );
+    $post_url                 = bp_get_admin_url( 'admin.php?page=bb-forums-setup' );
     $bbpress_plugin_is_active = false;