Changeset 6601 for trunk/src/includes/admin/tools/converter.php

Timestamp:

06/23/2017 04:21:58 AM (9 years ago)

Author:

johnjamesjacoby

Message:

Admin/Converter: Lazy load admin & converter as needed.

• Move admin class into its own file
• Remove admin.php
• Introduce _setup_ functions to make loading them on the fly easier
• Simplify converter logic for smoother starting/stopping
• Improved UI for timer/status updates
• Remove double-duty variables from BBP_Converter and response data
• Switch from text-only response value to JSON object for improved flexibility

This allows the converter to return more data, and makes it easier to work with that data.

Todo:

• Error responses
• Check that starts are bumping correctly inside of steps
• Better utilize JSON responses

File:

• trunk/src/includes/admin/tools/converter.php (modified) (2 diffs)

trunk/src/includes/admin/tools/converter.php

@@ -24 +24 @@
     // Default
     $files  = array();
-    $path   = bbpress()->admin->admin_dir . 'converters/';
+    $path   = bbp_setup_converter()->converters_dir;
     $curdir = opendir( $path );
 
@@ -67 +67 @@
     // Create a new converter object if it's found
     if ( isset( $converters[ $platform ] ) ) {
+
+        // Include & create the converter
         require_once $converters[ $platform ];
-        return new $platform;
+        if ( class_exists( $platform ) ) {
+            return new $platform;
+        }
     }