Changeset 4794 for trunk/bp-core/admin/bp-core-update.php

Timestamp:

07/22/2011 01:45:08 PM (14 years ago)

Author:

boonebgorges

Message:

Cleans up require()s in BP installation process. Fixes #3369. See #3367

File:

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

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

@@ -17 +17 @@
         global $bp;
 
-        // Ensure that we have access to some utility functions
-        require( BP_PLUGIN_DIR . '/bp-core/bp-core-functions.php' );
+        // Ensure that we have access to some utility functions. Must use require_once()
+        // because BP Core is loaded during incremental upgrades
+        require_once( BP_PLUGIN_DIR . '/bp-core/bp-core-functions.php' );
 
         // Get current DB version
@@ -585 +586 @@
             return false;
 
-        require( ABSPATH . WPINC . '/plugin.php' );
         $installed_plugins = get_plugins();
         $installed_themes  = get_themes();
@@ -1019 +1019 @@
 
             // Load BP and hook the admin menu, so that the redirect is successful
-            require( WP_PLUGIN_DIR . '/buddypress/bp-core/bp-core-loader.php' );
             require( WP_PLUGIN_DIR . '/buddypress/bp-core/admin/bp-core-admin.php' );
             bp_core_add_admin_menu();
@@ -1261 +1260 @@
  * @return array $page_ids
  */
-function bp_core_update_get_page_meta() {           
-    require( BP_PLUGIN_DIR . '/bp-core/bp-core-functions.php' );
-    
+function bp_core_update_get_page_meta() {
     if ( !$page_ids = bp_get_option( 'bp-pages' ) )
         $page_ids = array();