Context Navigation

← Previous Change
Next Change →

import.php

Timestamp:

07/17/2016 03:31:29 PM (10 years ago)

Author:

ocean90

Message:

Import: Enhance accessibility on the Import screen.

Remove title attributes.
Show "Install Now" and "Details" links if the importer isn't installed yet.
Show a "Run Importer" link if the importer is installed. It also handles activation if the plugin isn't activated.
Add aria-label attributes to each link.
Unify the importer descriptions to make them independent from the plugin state. The API was changed in [meta3690].
Adjust JavaScript callbacks for ajaxified importer installs.

Props afercia, swissspidy, ocean90.
See #24766.
Fixes #35191.

File:

: 1 edited

trunk/src/wp-admin/import.php (modified) (4 diffs)

Legend:

: Unmodified
: Added
: Removed

trunk/src/wp-admin/import.php

-              r37998
+              r38075
 );
+if ( current_user_can( 'install_plugins' ) )
+if ( current_user_can( 'install_plugins' ) ) {
+        // List of popular importer plugins from the WordPress.org API.
         $popular_importers = wp_get_popular_importers();
+else
+        $popular_importers = array();
+} else {
+        $popular_importers = array();
+}
 // Detect and redirect invalid importers like 'movabletype', which is registered as 'mt'
 …
 <?php
+// Registered (already installed) importers. They're stored in the global $wp_importers.
 $importers = get_importers();
 …
         if ( isset( $importers[ $pop_data['importer-id'] ] ) )
                 continue;
+        // Fill the array of registered (already installed) importers with data of the popular importers from the WordPress.org API.
         $importers[ $pop_data['importer-id'] ] = array( $pop_data['name'], $pop_data['description'], 'install' => $pop_data['plugin-slug'] );
+}
 …
 <table class="widefat importers striped">
+<?php
+        foreach ($importers as $importer_id => $data) {
+                $action = '';
+        <?php
+        foreach ( $importers as $importer_id => $data ) {
+                $plugin_slug = $action = '';
+                $is_plugin_installed = false;
                 if ( isset( $data['install'] ) ) {
                         $plugin_slug = $data['install'];
                         if ( file_exists( WP_PLUGIN_DIR . '/' . $plugin_slug ) ) {
                                 // Looks like Importer is installed, But not active
+                                // Looks like an importer is installed, but not active.
                                 $plugins = get_plugins( '/' . $plugin_slug );
                                 if ( !empty($plugins) ) {
                                         $keys = array_keys($plugins);
+                                if ( ! empty( $plugins ) ) {
+                                        $keys = array_keys( $plugins );
                                         $plugin_file = $plugin_slug . '/' . $keys[0];
+                                        $action = '<a href="' . esc_url(wp_nonce_url(admin_url('plugins.php?action=activate&plugin=' . $plugin_file . '&from=import'), 'activate-plugin_' . $plugin_file)) .
+                                                                                        '"title="' . esc_attr__('Activate importer') . '"">' . $data[0] . '</a>';
+                                        $url = wp_nonce_url( add_query_arg( array(
+                                                'action' => 'activate',
+                                                'plugin' => $plugin_file,
+                                                'from'   => 'import',
+                                        ), admin_url( 'plugins.php' ) ), 'activate-plugin_' . $plugin_file );
+                                        $action = sprintf(
+                                                '<a href="%s" aria-label="%s">%s</a>',
+                                                esc_url( $url ),
+                                                /* translators: %s: Importer name */
+                                                esc_attr( sprintf( __( 'Run %s' ), $data[0] ) ),
+                                                __( 'Run Importer' )
+                                        );
+                                        $is_plugin_installed = true;
+                                }
+                        }
+                        if ( empty($action) ) {
+                        if ( empty( $action ) ) {
                                 if ( is_main_site() ) {
+                                        $action = '<a href="' . esc_url( network_admin_url( 'plugin-install.php?tab=plugin-information&plugin=' . $plugin_slug .
+                                                                                '&from=import&TB_iframe=true&width=600&height=550' ) ) . '" class="thickbox open-plugin-details-modal" title="' .
+                                                                                esc_attr__('Install importer') . '">' . $data[0] . '</a>';
+                                        $url = wp_nonce_url( add_query_arg( array(
+                                                'action' => 'install-plugin',
+                                                'plugin' => $plugin_slug,
+                                                'from'   => 'import',
+                                        ), self_admin_url( 'update.php' ) ), 'install-plugin_' . $plugin_slug );
+                                        $action = sprintf(
+                                                '<a href="%1$s" class="install-now" data-slug="%2$s" data-name="%3$s" aria-label="%4$s">%5$s</a>',
+                                                esc_url( $url ),
+                                                esc_attr( $plugin_slug ),
+                                                esc_attr( $data[0] ),
+                                                /* translators: %s: Importer name */
+                                                esc_attr( sprintf( __( 'Install %s' ), $data[0] ) ),
+                                                __( 'Install Now' )
+                                        );
                                 } else {
+                                        $action = $data[0];
+                                        $data[1] = sprintf( __( 'This importer is not installed. Please install importers from <a href="%s">the main site</a>.' ), get_admin_url( $current_site->blog_id, 'import.php' ) );
+                                        $action = sprintf(
+                                                /* translators: URL to wp-admin/import.php */
+                                                __( 'This importer is not installed. Please install importers from <a href="%s">the main site</a>.' ),
+                                                get_admin_url( get_current_network_id(), 'import.php' )
+                                        );
+                                }
+                        }
                 } else {
+                        $action = "<a href='" . esc_url( "admin.php?import=$importer_id" ) . "' title='" . esc_attr( wptexturize( strip_tags( $data[1] ) ) ) ."'>{$data[0]}</a>";
+                        $url = add_query_arg( array(
+                                'import' => $importer_id,
+                        ), self_admin_url( 'admin.php' ) );
+                        $action = sprintf(
+                                '<a href="%1$s" aria-label="%2$s">%3$s</a>',
+                                esc_url( $url ),
+                                /* translators: %s: Importer name */
+                                esc_attr( sprintf( __( 'Run %s' ), $data[0] ) ),
+                                __( 'Run Importer' )
+                        );
+                        $is_plugin_installed = true;
+                }
+                if ( ! $is_plugin_installed && is_main_site() ) {
+                        $url = add_query_arg( array(
+                                'tab'       => 'plugin-information',
+                                'plugin'    => $plugin_slug,
+                                'from'      => 'import',
+                                'TB_iframe' => 'true',
+                                'width'     => 600,
+                                'height'    => 550,
+                        ), network_admin_url( 'plugin-install.php' ) );
+                        $action .= sprintf(
+                                ' | <a href="%1$s" class="thickbox open-plugin-details-modal" aria-label="%2$s">%3$s</a>',
+                                esc_url( $url ),
+                                /* translators: %s: Importer name */
+                                esc_attr( sprintf( __( 'More information about %s' ), $data[0] ) ),
+                                __( 'Details' )
+                        );
+                }
                 echo "
+                        <tr>
+                                <td class='import-system row-title'>$action</td>
+                                <td class='desc'>{$data[1]}</td>
+                        <tr class='importer-item'>
+                                <td class='import-system'>
+                                        <span class='importer-title'>{$data[0]}</span>
+                                        <span class='importer-action'>{$action}</span>
+                                </td>
+                                <td class='desc'>
+                                        <span class='importer-desc'>{$data[1]}</span>
+                                </td>
                         </tr>";
+        }
+?>
+        ?>
 </table>
 <?php

Note: See TracChangeset for help on using the changeset viewer.

Make WordPress Core

Context Navigation

Changeset 38075 for trunk/src/wp-admin/import.php

Legend:

trunk/src/wp-admin/import.php

Download in other formats: