Changeset 22632 for trunk/wp-admin/import.php

Timestamp:

11/17/2012 07:20:04 AM (13 years ago)

Author:

nacin

Message:

Pull the list of popular importers from WordPress.org.

These are the importers we suggest on import.php, prompting the user to
install the relevant plugin for the import they want to go through.

If the API is inaccessible, it falls back to a hard-coded list that should
be kept sync'd with the API with each major version of WordPress. This API
enables us to add new importers between releases, as they are completed or
if services gain quick adoption. As a last resort, we can also temporarily
disable importers that are broken (due to API changes, for example).

The importer currently returns English strings (which are then run through
translate() for existing strings), but the locale is passed to the API,
allowing us to ship translated strings if we wish to be adventurous.

props dllh for the assist.
fixes #18977.

File:

• trunk/wp-admin/import.php (modified) (3 diffs)

trunk/wp-admin/import.php

@@ -30 +30 @@
 );
 
-$popular_importers = array();
-if ( current_user_can('install_plugins') )
-    $popular_importers = array(
-        'blogger' => array( __('Blogger'), __('Install the Blogger importer to import posts, comments, and users from a Blogger blog.'), 'install' ),
-        'wpcat2tag' => array(__('Categories and Tags Converter'), __('Install the category/tag converter to convert existing categories to tags or tags to categories, selectively.'), 'install', 'wp-cat2tag' ),
-        'livejournal' => array( __( 'LiveJournal' ), __( 'Install the LiveJournal importer to import posts from LiveJournal using their API.' ), 'install' ),
-        'movabletype' => array( __('Movable Type and TypePad'), __('Install the Movable Type importer to import posts and comments from a Movable Type or TypePad blog.'), 'install', 'mt' ),
-        'opml' => array( __('Blogroll'), __('Install the blogroll importer to import links in OPML format.'), 'install' ),
-        'rss' => array( __('RSS'), __('Install the RSS importer to import posts from an RSS feed.'), 'install' ),
-        'tumblr' => array( __('Tumblr'), __('Install the Tumblr importer to import posts & media from Tumblr using their API.'), 'install' ),
-        'wordpress' => array( 'WordPress', __('Install the WordPress importer to import posts, pages, comments, custom fields, categories, and tags from a WordPress export file.'), 'install' )
-    );
+if ( current_user_can( 'install_plugins' ) )
+    $popular_importers = wp_get_popular_importers();
+else
+    $popular_importers = array();
 
-if ( ! empty( $_GET['invalid'] ) && !empty($popular_importers[$_GET['invalid']][3]) ) {
-    wp_redirect( admin_url('import.php?import=' . $popular_importers[$_GET['invalid']][3]) );
-    exit;
+// Detect and redirect invalid importers like 'movabletype', which is registered as 'mt'
+if ( ! empty( $_GET['invalid'] ) && isset( $popular_importers[ $_GET['invalid'] ] ) ) {
+    $importer_id = $popular_importers[ $_GET['invalid'] ]['importer-id'];
+    if ( $importer_id != $_GET['invalid'] ) { // Prevent redirect loops.
+        wp_redirect( admin_url( 'admin.php?import=' . $importer_id ) );
+        exit;
+    }
+    unset( $importer_id );
 }
 
@@ -69 +66 @@
 // If a popular importer is not registered, create a dummy registration that links to the plugin installer.
 foreach ( $popular_importers as $pop_importer => $pop_data ) {
-    if ( isset( $importers[$pop_importer] ) )
+    if ( isset( $importers[ $pop_importer ] ) )
         continue;
-    if ( isset( $pop_data[3] ) && isset( $importers[ $pop_data[3] ] ) )
+    if ( isset( $importers[ $pop_data['importer-id'] ] ) )
         continue;
-
-    $importers[$pop_importer] = $popular_importers[$pop_importer];
+    $importers[ $pop_data['importer-id'] ] = array( $pop_data['name'], $pop_data['description'], 'install' => $pop_data['plugin-slug'] );
 }
 
-if ( empty($importers) ) {
-    echo '<p>'.__('No importers are available.').'</p>'; // TODO: make more helpful
+if ( empty( $importers ) ) {
+    echo '<p>' . __('No importers are available.') . '</p>'; // TODO: make more helpful
 } else {
-    uasort($importers, create_function('$a, $b', 'return strcmp($a[0], $b[0]);'));
+    uasort($importers, create_function('$a, $b', 'return strnatcasecmp($a[0], $b[0]);'));
 ?>
 <table class="widefat importers" cellspacing="0">
 
 <?php
-    $style = '';
-    foreach ($importers as $id => $data) {
-        $style = ('class="alternate"' == $style || 'class="alternate active"' == $style) ? '' : 'alternate';
+    $alt = '';
+    foreach ($importers as $importer_id => $data) {
         $action = '';
-        if ( 'install' == $data[2] ) {
-            $plugin_slug = $id . '-importer';
+        if ( isset( $data['install'] ) ) {
+            $plugin_slug = $data['install'];
             if ( file_exists( WP_PLUGIN_DIR . '/' . $plugin_slug ) ) {
                 // Looks like Importer is installed, But not active
@@ -112 +107 @@
             }
         } else {
-            $action = "<a href='" . esc_url("admin.php?import=$id") . "' title='" . esc_attr( wptexturize(strip_tags($data[1])) ) ."'>{$data[0]}</a>";
+            $action = "<a href='" . esc_url( "admin.php?import=$importer_id" ) . "' title='" . esc_attr( wptexturize( strip_tags( $data[1] ) ) ) ."'>{$data[0]}</a>";
         }
 
-        if ($style != '')
-            $style = 'class="'.$style.'"';
+        $alt = $alt ? '' : ' class="alternate"';
         echo "
-            <tr $style>
+            <tr$alt>
                 <td class='import-system row-title'>$action</td>
                 <td class='desc'>{$data[1]}</td>