Changeset 6785 for trunk/src/includes/core/abstraction.php

Timestamp:

02/19/2018 07:00:55 AM (7 years ago)

Author:

johnjamesjacoby

Message:

Converter: remove WP_SETUP_CONFIG constant setting from converter process.

This introduces a helper database class to avoid directly connecting to the external database. Instead, we'll attempt to control that connection, and provide feedback to the user in the event a connection cannot be made. The WP_SETUP_CONFIG was causing calls to the options API to fail, resulting in broken calculations and invalid offsets.

This commit includes some general load order clean-up, which also fixes a regression causing fatal errors when attempting to upgrade converted user passwords from the old platform. It also fixes the condition where boundaries being converted would display beyond their maximums.

Fixes #3191.

File:

• trunk/src/includes/core/abstraction.php (modified) (1 diff)

trunk/src/includes/core/abstraction.php

@@ -60 +60 @@
     if ( empty( $bbp_admin->converter ) ) {
 
-        // Require the converter classes
+        // Require the converter files
+        require_once $bbp_admin->admin_dir . 'tools/converter.php';
+        require_once $bbp_admin->admin_dir . 'classes/class-bbp-converter.php';
+        require_once $bbp_admin->admin_dir . 'classes/class-bbp-converter-db.php';
         require_once $bbp_admin->admin_dir . 'classes/class-bbp-converter-base.php';
-        require_once $bbp_admin->admin_dir . 'classes/class-bbp-converter.php';
 
         // Setup