WordPress 4.5
[autoinstalls/wordpress.git] / wp-admin / includes / class-wp-upgrader-skins.php
index 8fcee68f3cd96533f20fac5a6347d7fba1fb83a2..c9d32cebe8c5fa6cd93e85afdae8f02519a9d5af 100644 (file)
@@ -20,41 +20,76 @@ class WP_Upgrader_Skin {
        public $done_header = false;
        public $done_footer = false;
        public $result = false;
+       public $options = array();
 
+       /**
+        *
+        * @param array $args
+        */
        public function __construct($args = array()) {
                $defaults = array( 'url' => '', 'nonce' => '', 'title' => '', 'context' => false );
                $this->options = wp_parse_args($args, $defaults);
        }
 
+       /**
+        * @param WP_Upgrader $upgrader
+        */
        public function set_upgrader(&$upgrader) {
                if ( is_object($upgrader) )
                        $this->upgrader =& $upgrader;
                $this->add_strings();
        }
 
+       /**
+        * @access public
+        */
        public function add_strings() {
        }
 
+       /**
+        *
+        * @param string|false|WP_Error $result
+        */
        public function set_result($result) {
                $this->result = $result;
        }
 
-       public function request_filesystem_credentials($error = false) {
+       /**
+        *
+        * @param bool   $error
+        * @param string $context
+        * @param bool   $allow_relaxed_file_ownership
+        * @return type
+        */
+       public function request_filesystem_credentials( $error = false, $context = false, $allow_relaxed_file_ownership = false ) {
                $url = $this->options['url'];
-               $context = $this->options['context'];
-               if ( !empty($this->options['nonce']) )
+               if ( ! $context ) {
+                       $context = $this->options['context'];
+               }
+               if ( !empty($this->options['nonce']) ) {
                        $url = wp_nonce_url($url, $this->options['nonce']);
-               return request_filesystem_credentials($url, '', $error, $context); //Possible to bring inline, Leaving as is for now.
+               }
+
+               $extra_fields = array();
+
+               return request_filesystem_credentials( $url, '', $error, $context, $extra_fields, $allow_relaxed_file_ownership );
        }
 
+       /**
+        * @access public
+        */
        public function header() {
                if ( $this->done_header ) {
                        return;
                }
                $this->done_header = true;
                echo '<div class="wrap">';
-               echo '<h2>' . $this->options['title'] . '</h2>';
+               echo '<h1>' . $this->options['title'] . '</h1>';
        }
+
+       /**
+        * @access public
+        */
        public function footer() {
                if ( $this->done_footer ) {
                        return;
@@ -63,6 +98,10 @@ class WP_Upgrader_Skin {
                echo '</div>';
        }
 
+       /**
+        *
+        * @param string|WP_Error $errors
+        */
        public function error($errors) {
                if ( ! $this->done_header )
                        $this->header();
@@ -78,6 +117,10 @@ class WP_Upgrader_Skin {
                }
        }
 
+       /**
+        *
+        * @param string $string
+        */
        public function feedback($string) {
                if ( isset( $this->upgrader->strings[$string] ) )
                        $string = $this->upgrader->strings[$string];
@@ -95,7 +138,15 @@ class WP_Upgrader_Skin {
                        return;
                show_message($string);
        }
+
+       /**
+        * @access public
+        */
        public function before() {}
+
+       /**
+        * @access public
+        */
        public function after() {}
 
        /**
@@ -127,6 +178,16 @@ class WP_Upgrader_Skin {
                                </script>';
                }
        }
+
+       /**
+        * @access public
+        */
+       public function bulk_header() {}
+
+       /**
+        * @access public
+        */
+       public function bulk_footer() {}
 }
 
 /**
@@ -141,7 +202,11 @@ class Plugin_Upgrader_Skin extends WP_Upgrader_Skin {
        public $plugin_active = false;
        public $plugin_network_active = false;
 
-       public function __construct($args = array()) {
+       /**
+        *
+        * @param array $args
+        */
+       public function __construct( $args = array() ) {
                $defaults = array( 'url' => '', 'plugin' => '', 'nonce' => '', 'title' => __('Update Plugin') );
                $args = wp_parse_args($args, $defaults);
 
@@ -153,17 +218,21 @@ class Plugin_Upgrader_Skin extends WP_Upgrader_Skin {
                parent::__construct($args);
        }
 
+       /**
+        * @access public
+        */
        public function after() {
                $this->plugin = $this->upgrader->plugin_info();
                if ( !empty($this->plugin) && !is_wp_error($this->result) && $this->plugin_active ){
-                       echo '<iframe style="border:0;overflow:hidden" width="100%" height="170px" src="' . wp_nonce_url('update.php?action=activate-plugin&networkwide=' . $this->plugin_network_active . '&plugin=' . urlencode( $this->plugin ), 'activate-plugin_' . $this->plugin) .'"></iframe>';
+                       // Currently used only when JS is off for a single plugin update?
+                       echo '<iframe title="' . esc_attr__( 'Update progress' ) . '" style="border:0;overflow:hidden" width="100%" height="170" src="' . wp_nonce_url( 'update.php?action=activate-plugin&networkwide=' . $this->plugin_network_active . '&plugin=' . urlencode( $this->plugin ), 'activate-plugin_' . $this->plugin ) . '"></iframe>';
                }
 
                $this->decrement_update_count( 'plugin' );
 
                $update_actions =  array(
-                       'activate_plugin' => '<a href="' . wp_nonce_url('plugins.php?action=activate&amp;plugin=' . urlencode( $this->plugin ), 'activate-plugin_' . $this->plugin) . '" title="' . esc_attr__('Activate this plugin') . '" target="_parent">' . __('Activate Plugin') . '</a>',
-                       'plugins_page' => '<a href="' . self_admin_url('plugins.php') . '" title="' . esc_attr__('Go to plugins page') . '" target="_parent">' . __('Return to Plugins page') . '</a>'
+                       'activate_plugin' => '<a href="' . wp_nonce_url( 'plugins.php?action=activate&amp;plugin=' . urlencode( $this->plugin ), 'activate-plugin_' . $this->plugin) . '" target="_parent">' . __( 'Activate Plugin' ) . '</a>',
+                       'plugins_page' => '<a href="' . self_admin_url( 'plugins.php' ) . '" target="_parent">' . __( 'Return to Plugins page' ) . '</a>'
                );
                if ( $this->plugin_active || ! $this->result || is_wp_error( $this->result ) || ! current_user_can( 'activate_plugins' ) )
                        unset( $update_actions['activate_plugin'] );
@@ -192,8 +261,15 @@ class Plugin_Upgrader_Skin extends WP_Upgrader_Skin {
  */
 class Bulk_Upgrader_Skin extends WP_Upgrader_Skin {
        public $in_loop = false;
+       /**
+        * @var string|false
+        */
        public $error = false;
 
+       /**
+        *
+        * @param array $args
+        */
        public function __construct($args = array()) {
                $defaults = array( 'url' => '', 'nonce' => '' );
                $args = wp_parse_args($args, $defaults);
@@ -201,14 +277,23 @@ class Bulk_Upgrader_Skin extends WP_Upgrader_Skin {
                parent::__construct($args);
        }
 
+       /**
+        * @access public
+        */
        public function add_strings() {
                $this->upgrader->strings['skin_upgrade_start'] = __('The update process is starting. This process may take a while on some hosts, so please be patient.');
-               $this->upgrader->strings['skin_update_failed_error'] = __('An error occurred while updating %1$s: <strong>%2$s</strong>');
+               /* translators: 1: Title of an update, 2: Error message */
+               $this->upgrader->strings['skin_update_failed_error'] = __('An error occurred while updating %1$s: %2$s');
+               /* translators: 1: Title of an update */
                $this->upgrader->strings['skin_update_failed'] = __('The update of %1$s failed.');
-               $this->upgrader->strings['skin_update_successful'] = __('%1$s updated successfully.').' <a onclick="%2$s" href="#" class="hide-if-no-js"><span>'.__('Show Details').'</span><span class="hidden">'.__('Hide Details').'</span>.</a>';
+               /* translators: 1: Title of an update */
+               $this->upgrader->strings['skin_update_successful'] = __( '%1$s updated successfully.' ) . ' <a onclick="%2$s" href="#" class="hide-if-no-js"><span>' . __( 'Show Details' ) . '</span><span class="hidden">' . __( 'Hide Details' ) . '</span></a>';
                $this->upgrader->strings['skin_upgrade_end'] = __('All updates have been completed.');
        }
 
+       /**
+        * @param string $string
+        */
        public function feedback($string) {
                if ( isset( $this->upgrader->strings[$string] ) )
                        $string = $this->upgrader->strings[$string];
@@ -230,13 +315,24 @@ class Bulk_Upgrader_Skin extends WP_Upgrader_Skin {
                        echo "<p>$string</p>\n";
        }
 
+       /**
+        * @access public
+        */
        public function header() {
                // Nothing, This will be displayed within a iframe.
        }
 
+       /**
+        * @access public
+        */
        public function footer() {
                // Nothing, This will be displayed within a iframe.
        }
+
+       /**
+        *
+        * @param string|WP_Error $error
+        */
        public function error($error) {
                if ( is_string($error) && isset( $this->upgrader->strings[$error] ) )
                        $this->error = $this->upgrader->strings[$error];
@@ -254,29 +350,44 @@ class Bulk_Upgrader_Skin extends WP_Upgrader_Skin {
                echo '<script type="text/javascript">jQuery(\'.waiting-' . esc_js($this->upgrader->update_current) . '\').hide();</script>';
        }
 
+       /**
+        * @access public
+        */
        public function bulk_header() {
                $this->feedback('skin_upgrade_start');
        }
 
+       /**
+        * @access public
+        */
        public function bulk_footer() {
                $this->feedback('skin_upgrade_end');
        }
 
+       /**
+        *
+        * @param string $title
+        */
        public function before($title = '') {
                $this->in_loop = true;
-               printf( '<h4>' . $this->upgrader->strings['skin_before_update_header'] . ' <span class="spinner waiting-' . $this->upgrader->update_current . '"></span></h4>',  $title, $this->upgrader->update_current, $this->upgrader->update_count);
+               printf( '<h2>' . $this->upgrader->strings['skin_before_update_header'] . ' <span class="spinner waiting-' . $this->upgrader->update_current . '"></span></h2>', $title, $this->upgrader->update_current, $this->upgrader->update_count );
                echo '<script type="text/javascript">jQuery(\'.waiting-' . esc_js($this->upgrader->update_current) . '\').css("display", "inline-block");</script>';
                echo '<div class="update-messages hide-if-js" id="progress-' . esc_attr($this->upgrader->update_current) . '"><p>';
                $this->flush_output();
        }
 
+       /**
+        *
+        * @param string $title
+        */
        public function after($title = '') {
                echo '</p></div>';
                if ( $this->error || ! $this->result ) {
-                       if ( $this->error )
-                               echo '<div class="error"><p>' . sprintf($this->upgrader->strings['skin_update_failed_error'], $title, $this->error) . '</p></div>';
-                       else
+                       if ( $this->error ) {
+                               echo '<div class="error"><p>' . sprintf($this->upgrader->strings['skin_update_failed_error'], $title, '<strong>' . $this->error . '</strong>' ) . '</p></div>';
+                       } else {
                                echo '<div class="error"><p>' . sprintf($this->upgrader->strings['skin_update_failed'], $title) . '</p></div>';
+                       }
 
                        echo '<script type="text/javascript">jQuery(\'#progress-' . esc_js($this->upgrader->update_current) . '\').show();</script>';
                }
@@ -290,11 +401,17 @@ class Bulk_Upgrader_Skin extends WP_Upgrader_Skin {
                $this->flush_output();
        }
 
+       /**
+        * @access public
+        */
        public function reset() {
                $this->in_loop = false;
                $this->error = false;
        }
 
+       /**
+        * @access public
+        */
        public function flush_output() {
                wp_ob_end_flush_all();
                flush();
@@ -304,28 +421,36 @@ class Bulk_Upgrader_Skin extends WP_Upgrader_Skin {
 class Bulk_Plugin_Upgrader_Skin extends Bulk_Upgrader_Skin {
        public $plugin_info = array(); // Plugin_Upgrader::bulk() will fill this in.
 
-       public function __construct($args = array()) {
-               parent::__construct($args);
-       }
-
        public function add_strings() {
                parent::add_strings();
                $this->upgrader->strings['skin_before_update_header'] = __('Updating Plugin %1$s (%2$d/%3$d)');
        }
 
+       /**
+        *
+        * @param string $title
+        */
        public function before($title = '') {
                parent::before($this->plugin_info['Title']);
        }
 
+       /**
+        *
+        * @param string $title
+        */
        public function after($title = '') {
                parent::after($this->plugin_info['Title']);
                $this->decrement_update_count( 'plugin' );
        }
+
+       /**
+        * @access public
+        */
        public function bulk_footer() {
                parent::bulk_footer();
                $update_actions =  array(
-                       'plugins_page' => '<a href="' . self_admin_url('plugins.php') . '" title="' . esc_attr__('Go to plugins page') . '" target="_parent">' . __('Return to Plugins page') . '</a>',
-                       'updates_page' => '<a href="' . self_admin_url('update-core.php') . '" title="' . esc_attr__('Go to WordPress Updates page') . '" target="_parent">' . __('Return to WordPress Updates') . '</a>'
+                       'plugins_page' => '<a href="' . self_admin_url( 'plugins.php' ) . '" target="_parent">' . __( 'Return to Plugins page' ) . '</a>',
+                       'updates_page' => '<a href="' . self_admin_url( 'update-core.php' ) . '" target="_parent">' . __( 'Return to WordPress Updates page' ) . '</a>'
                );
                if ( ! current_user_can( 'activate_plugins' ) )
                        unset( $update_actions['plugins_page'] );
@@ -348,29 +473,36 @@ class Bulk_Plugin_Upgrader_Skin extends Bulk_Upgrader_Skin {
 class Bulk_Theme_Upgrader_Skin extends Bulk_Upgrader_Skin {
        public $theme_info = array(); // Theme_Upgrader::bulk() will fill this in.
 
-       public function __construct($args = array()) {
-               parent::__construct($args);
-       }
-
        public function add_strings() {
                parent::add_strings();
                $this->upgrader->strings['skin_before_update_header'] = __('Updating Theme %1$s (%2$d/%3$d)');
        }
 
+       /**
+        *
+        * @param string $title
+        */
        public function before($title = '') {
                parent::before( $this->theme_info->display('Name') );
        }
 
+       /**
+        *
+        * @param string $title
+        */
        public function after($title = '') {
                parent::after( $this->theme_info->display('Name') );
                $this->decrement_update_count( 'theme' );
        }
 
+       /**
+        * @access public
+        */
        public function bulk_footer() {
                parent::bulk_footer();
                $update_actions =  array(
-                       'themes_page' => '<a href="' . self_admin_url('themes.php') . '" title="' . esc_attr__('Go to themes page') . '" target="_parent">' . __('Return to Themes page') . '</a>',
-                       'updates_page' => '<a href="' . self_admin_url('update-core.php') . '" title="' . esc_attr__('Go to WordPress Updates page') . '" target="_parent">' . __('Return to WordPress Updates') . '</a>'
+                       'themes_page' => '<a href="' . self_admin_url( 'themes.php' ) . '" target="_parent">' . __( 'Return to Themes page' ) . '</a>',
+                       'updates_page' => '<a href="' . self_admin_url( 'update-core.php' ) . '" target="_parent">' . __( 'Return to WordPress Updates page' ) . '</a>'
                );
                if ( ! current_user_can( 'switch_themes' ) && ! current_user_can( 'edit_theme_options' ) )
                        unset( $update_actions['themes_page'] );
@@ -401,6 +533,10 @@ class Plugin_Installer_Skin extends WP_Upgrader_Skin {
        public $api;
        public $type;
 
+       /**
+        *
+        * @param array $args
+        */
        public function __construct($args = array()) {
                $defaults = array( 'type' => 'web', 'url' => '', 'plugin' => '', 'nonce' => '', 'title' => '' );
                $args = wp_parse_args($args, $defaults);
@@ -411,13 +547,18 @@ class Plugin_Installer_Skin extends WP_Upgrader_Skin {
                parent::__construct($args);
        }
 
+       /**
+        * @access public
+        */
        public function before() {
                if ( !empty($this->api) )
                        $this->upgrader->strings['process_success'] = sprintf( __('Successfully installed the plugin <strong>%s %s</strong>.'), $this->api->name, $this->api->version);
        }
 
+       /**
+        * @access public
+        */
        public function after() {
-
                $plugin_file = $this->upgrader->plugin_info();
 
                $install_actions = array();
@@ -425,21 +566,22 @@ class Plugin_Installer_Skin extends WP_Upgrader_Skin {
                $from = isset($_GET['from']) ? wp_unslash( $_GET['from'] ) : 'plugins';
 
                if ( 'import' == $from )
-                       $install_actions['activate_plugin'] = '<a href="' . wp_nonce_url('plugins.php?action=activate&amp;from=import&amp;plugin=' . urlencode( $plugin_file ), 'activate-plugin_' . $plugin_file) . '" title="' . esc_attr__('Activate this plugin') . '" target="_parent">' . __('Activate Plugin &amp; Run Importer') . '</a>';
+                       $install_actions['activate_plugin'] = '<a href="' . wp_nonce_url( 'plugins.php?action=activate&amp;from=import&amp;plugin=' . urlencode( $plugin_file ), 'activate-plugin_' . $plugin_file ) . '" target="_parent">' . __( 'Activate Plugin &amp; Run Importer' ) . '</a>';
                else
-                       $install_actions['activate_plugin'] = '<a href="' . wp_nonce_url('plugins.php?action=activate&amp;plugin=' . urlencode( $plugin_file ), 'activate-plugin_' . $plugin_file) . '" title="' . esc_attr__('Activate this plugin') . '" target="_parent">' . __('Activate Plugin') . '</a>';
+                       $install_actions['activate_plugin'] = '<a href="' . wp_nonce_url( 'plugins.php?action=activate&amp;plugin=' . urlencode( $plugin_file ), 'activate-plugin_' . $plugin_file ) . '" target="_parent">' . __( 'Activate Plugin' ) . '</a>';
 
                if ( is_multisite() && current_user_can( 'manage_network_plugins' ) ) {
-                       $install_actions['network_activate'] = '<a href="' . wp_nonce_url('plugins.php?action=activate&amp;networkwide=1&amp;plugin=' . urlencode( $plugin_file ), 'activate-plugin_' . $plugin_file) . '" title="' . esc_attr__('Activate this plugin for all sites in this network') . '" target="_parent">' . __('Network Activate') . '</a>';
+                       $install_actions['network_activate'] = '<a href="' . wp_nonce_url( 'plugins.php?action=activate&amp;networkwide=1&amp;plugin=' . urlencode( $plugin_file ), 'activate-plugin_' . $plugin_file ) . '" target="_parent">' . __( 'Network Activate' ) . '</a>';
                        unset( $install_actions['activate_plugin'] );
                }
 
-               if ( 'import' == $from )
-                       $install_actions['importers_page'] = '<a href="' . admin_url('import.php') . '" title="' . esc_attr__('Return to Importers') . '" target="_parent">' . __('Return to Importers') . '</a>';
-               else if ( $this->type == 'web' )
-                       $install_actions['plugins_page'] = '<a href="' . self_admin_url('plugin-install.php') . '" title="' . esc_attr__('Return to Plugin Installer') . '" target="_parent">' . __('Return to Plugin Installer') . '</a>';
-               else
-                       $install_actions['plugins_page'] = '<a href="' . self_admin_url('plugins.php') . '" title="' . esc_attr__('Return to Plugins page') . '" target="_parent">' . __('Return to Plugins page') . '</a>';
+               if ( 'import' == $from ) {
+                       $install_actions['importers_page'] = '<a href="' . admin_url( 'import.php' ) . '" target="_parent">' . __( 'Return to Importers' ) . '</a>';
+               } elseif ( $this->type == 'web' ) {
+                       $install_actions['plugins_page'] = '<a href="' . self_admin_url( 'plugin-install.php' ) . '" target="_parent">' . __( 'Return to Plugin Installer' ) . '</a>';
+               } else {
+                       $install_actions['plugins_page'] = '<a href="' . self_admin_url( 'plugins.php' ) . '" target="_parent">' . __( 'Return to Plugins page' ) . '</a>';
+               }
 
                if ( ! $this->result || is_wp_error($this->result) ) {
                        unset( $install_actions['activate_plugin'], $install_actions['network_activate'] );
@@ -476,6 +618,10 @@ class Theme_Installer_Skin extends WP_Upgrader_Skin {
        public $api;
        public $type;
 
+       /**
+        *
+        * @param array $args
+        */
        public function __construct($args = array()) {
                $defaults = array( 'type' => 'web', 'url' => '', 'theme' => '', 'nonce' => '', 'title' => '' );
                $args = wp_parse_args($args, $defaults);
@@ -486,11 +632,17 @@ class Theme_Installer_Skin extends WP_Upgrader_Skin {
                parent::__construct($args);
        }
 
+       /**
+        * @access public
+        */
        public function before() {
                if ( !empty($this->api) )
                        $this->upgrader->strings['process_success'] = sprintf( $this->upgrader->strings['process_success_specific'], $this->api->name, $this->api->version);
        }
 
+       /**
+        * @access public
+        */
        public function after() {
                if ( empty($this->upgrader->result['destination_name']) )
                        return;
@@ -503,12 +655,6 @@ class Theme_Installer_Skin extends WP_Upgrader_Skin {
                $stylesheet = $this->upgrader->result['destination_name'];
                $template   = $theme_info->get_template();
 
-               $preview_link = add_query_arg( array(
-                       'preview'    => 1,
-                       'template'   => urlencode( $template ),
-                       'stylesheet' => urlencode( $stylesheet ),
-               ), trailingslashit( home_url() ) );
-
                $activate_link = add_query_arg( array(
                        'action'     => 'activate',
                        'template'   => urlencode( $template ),
@@ -517,19 +663,19 @@ class Theme_Installer_Skin extends WP_Upgrader_Skin {
                $activate_link = wp_nonce_url( $activate_link, 'switch-theme_' . $stylesheet );
 
                $install_actions = array();
-               $install_actions['preview']  = '<a href="' . esc_url( $preview_link ) . '" class="hide-if-customize" title="' . esc_attr( sprintf( __('Preview &#8220;%s&#8221;'), $name ) ) . '">' . __('Preview') . '</a>';
+
                if ( current_user_can( 'edit_theme_options' ) && current_user_can( 'customize' ) ) {
-                       $install_actions['preview'] .= '<a href="' . wp_customize_url( $stylesheet ) . '" class="hide-if-no-customize load-customize" title="' . esc_attr( sprintf( __('Preview &#8220;%s&#8221;'), $name ) ) . '">' . __('Live Preview') . '</a>';
+                       $install_actions['preview'] = '<a href="' . wp_customize_url( $stylesheet ) . '" class="hide-if-no-customize load-customize"><span aria-hidden="true">' . __( 'Live Preview' ) . '</span><span class="screen-reader-text">' . sprintf( __( 'Live Preview &#8220;%s&#8221;' ), $name ) . '</span></a>';
                }
-               $install_actions['activate'] = '<a href="' . esc_url( $activate_link ) . '" class="activatelink" title="' . esc_attr( sprintf( __('Activate &#8220;%s&#8221;'), $name ) ) . '">' . __('Activate') . '</a>';
+               $install_actions['activate'] = '<a href="' . esc_url( $activate_link ) . '" class="activatelink"><span aria-hidden="true">' . __( 'Activate' ) . '</span><span class="screen-reader-text">' . sprintf( __( 'Activate &#8220;%s&#8221;' ), $name ) . '</span></a>';
 
                if ( is_network_admin() && current_user_can( 'manage_network_themes' ) )
-                       $install_actions['network_enable'] = '<a href="' . esc_url( wp_nonce_url( 'themes.php?action=enable&amp;theme=' . urlencode( $stylesheet ), 'enable-theme_' . $stylesheet ) ) . '" title="' . esc_attr__( 'Enable this theme for all sites in this network' ) . '" target="_parent">' . __( 'Network Enable' ) . '</a>';
+                       $install_actions['network_enable'] = '<a href="' . esc_url( wp_nonce_url( 'themes.php?action=enable&amp;theme=' . urlencode( $stylesheet ), 'enable-theme_' . $stylesheet ) ) . '" target="_parent">' . __( 'Network Enable' ) . '</a>';
 
                if ( $this->type == 'web' )
-                       $install_actions['themes_page'] = '<a href="' . self_admin_url('theme-install.php') . '" title="' . esc_attr__('Return to Theme Installer') . '" target="_parent">' . __('Return to Theme Installer') . '</a>';
+                       $install_actions['themes_page'] = '<a href="' . self_admin_url( 'theme-install.php' ) . '" target="_parent">' . __( 'Return to Theme Installer' ) . '</a>';
                elseif ( current_user_can( 'switch_themes' ) || current_user_can( 'edit_theme_options' ) )
-                       $install_actions['themes_page'] = '<a href="' . self_admin_url('themes.php') . '" title="' . esc_attr__('Themes page') . '" target="_parent">' . __('Return to Themes page') . '</a>';
+                       $install_actions['themes_page'] = '<a href="' . self_admin_url( 'themes.php' ) . '" target="_parent">' . __( 'Return to Themes page' ) . '</a>';
 
                if ( ! $this->result || is_wp_error($this->result) || is_network_admin() || ! current_user_can( 'switch_themes' ) )
                        unset( $install_actions['activate'], $install_actions['preview'] );
@@ -560,6 +706,10 @@ class Theme_Installer_Skin extends WP_Upgrader_Skin {
 class Theme_Upgrader_Skin extends WP_Upgrader_Skin {
        public $theme = '';
 
+       /**
+        *
+        * @param array $args
+        */
        public function __construct($args = array()) {
                $defaults = array( 'url' => '', 'theme' => '', 'nonce' => '', 'title' => __('Update Theme') );
                $args = wp_parse_args($args, $defaults);
@@ -569,6 +719,9 @@ class Theme_Upgrader_Skin extends WP_Upgrader_Skin {
                parent::__construct($args);
        }
 
+       /**
+        * @access public
+        */
        public function after() {
                $this->decrement_update_count( 'theme' );
 
@@ -578,12 +731,6 @@ class Theme_Upgrader_Skin extends WP_Upgrader_Skin {
                        $stylesheet = $this->upgrader->result['destination_name'];
                        $template   = $theme_info->get_template();
 
-                       $preview_link = add_query_arg( array(
-                               'preview'    => 1,
-                               'template'   => urlencode( $template ),
-                               'stylesheet' => urlencode( $stylesheet ),
-                       ), trailingslashit( home_url() ) );
-
                        $activate_link = add_query_arg( array(
                                'action'     => 'activate',
                                'template'   => urlencode( $template ),
@@ -593,21 +740,20 @@ class Theme_Upgrader_Skin extends WP_Upgrader_Skin {
 
                        if ( get_stylesheet() == $stylesheet ) {
                                if ( current_user_can( 'edit_theme_options' ) && current_user_can( 'customize' ) ) {
-                                       $update_actions['preview']  = '<a href="' . wp_customize_url( $stylesheet ) . '" class="hide-if-no-customize load-customize" title="' . esc_attr( sprintf( __('Customize &#8220;%s&#8221;'), $name ) ) . '">' . __('Customize') . '</a>';
+                                       $update_actions['preview']  = '<a href="' . wp_customize_url( $stylesheet ) . '" class="hide-if-no-customize load-customize"><span aria-hidden="true">' . __( 'Customize' ) . '</span><span class="screen-reader-text">' . sprintf( __( 'Customize &#8220;%s&#8221;' ), $name ) . '</span></a>';
                                }
                        } elseif ( current_user_can( 'switch_themes' ) ) {
-                               $update_actions['preview']  = '<a href="' . esc_url( $preview_link ) . '" class="hide-if-customize" title="' . esc_attr( sprintf( __('Preview &#8220;%s&#8221;'), $name ) ) . '">' . __('Preview') . '</a>';
                                if ( current_user_can( 'edit_theme_options' ) && current_user_can( 'customize' ) ) {
-                                       $update_actions['preview'] .= '<a href="' . wp_customize_url( $stylesheet ) . '" class="hide-if-no-customize load-customize" title="' . esc_attr( sprintf( __('Preview &#8220;%s&#8221;'), $name ) ) . '">' . __('Live Preview') . '</a>';
+                                       $update_actions['preview'] = '<a href="' . wp_customize_url( $stylesheet ) . '" class="hide-if-no-customize load-customize"><span aria-hidden="true">' . __( 'Live Preview' ) . '</span><span class="screen-reader-text">' . sprintf( __( 'Live Preview &#8220;%s&#8221;' ), $name ) . '</span></a>';
                                }
-                               $update_actions['activate'] = '<a href="' . esc_url( $activate_link ) . '" class="activatelink" title="' . esc_attr( sprintf( __('Activate &#8220;%s&#8221;'), $name ) ) . '">' . __('Activate') . '</a>';
+                               $update_actions['activate'] = '<a href="' . esc_url( $activate_link ) . '" class="activatelink"><span aria-hidden="true">' . __( 'Activate' ) . '</span><span class="screen-reader-text">' . sprintf( __( 'Activate &#8220;%s&#8221;' ), $name ) . '</span></a>';
                        }
 
                        if ( ! $this->result || is_wp_error( $this->result ) || is_network_admin() )
                                unset( $update_actions['preview'], $update_actions['activate'] );
                }
 
-               $update_actions['themes_page'] = '<a href="' . self_admin_url('themes.php') . '" title="' . esc_attr__('Return to Themes page') . '" target="_parent">' . __('Return to Themes page') . '</a>';
+               $update_actions['themes_page'] = '<a href="' . self_admin_url( 'themes.php' ) . '" target="_parent">' . __( 'Return to Themes page' ) . '</a>';
 
                /**
                 * Filter the list of action links available following a single theme update.
@@ -637,6 +783,10 @@ class Language_Pack_Upgrader_Skin extends WP_Upgrader_Skin {
        public $done_footer = false;
        public $display_footer_actions = true;
 
+       /**
+        *
+        * @param array $args
+        */
        public function __construct( $args = array() ) {
                $defaults = array( 'url' => '', 'nonce' => '', 'title' => __( 'Update Translations' ), 'skip_header_footer' => false );
                $args = wp_parse_args( $args, $defaults );
@@ -648,28 +798,41 @@ class Language_Pack_Upgrader_Skin extends WP_Upgrader_Skin {
                parent::__construct( $args );
        }
 
+       /**
+        * @access public
+        */
        public function before() {
                $name = $this->upgrader->get_name_for_update( $this->language_update );
 
                echo '<div class="update-messages lp-show-latest">';
 
-               printf( '<h4>' . __( 'Updating translations for %1$s (%2$s)&#8230;' ) . '</h4>', $name, $this->language_update->language );
+               printf( '<h2>' . __( 'Updating translations for %1$s (%2$s)&#8230;' ) . '</h2>', $name, $this->language_update->language );
        }
 
+       /**
+        *
+        * @param string|WP_Error $error
+        */
        public function error( $error ) {
                echo '<div class="lp-error">';
                parent::error( $error );
                echo '</div>';
        }
 
+       /**
+        * @access public
+        */
        public function after() {
                echo '</div>';
        }
 
+       /**
+        * @access public
+        */
        public function bulk_footer() {
                $this->decrement_update_count( 'translation' );
                $update_actions = array();
-               $update_actions['updates_page'] = '<a href="' . self_admin_url( 'update-core.php' ) . '" title="' . esc_attr__( 'Go to WordPress Updates page' ) . '" target="_parent">' . __( 'Return to WordPress Updates' ) . '</a>';
+               $update_actions['updates_page'] = '<a href="' . self_admin_url( 'update-core.php' ) . '" target="_parent">' . __( 'Return to WordPress Updates page' ) . '</a>';
 
                /**
                 * Filter the list of action links available following a translations update.
@@ -698,29 +861,45 @@ class Language_Pack_Upgrader_Skin extends WP_Upgrader_Skin {
 class Automatic_Upgrader_Skin extends WP_Upgrader_Skin {
        protected $messages = array();
 
-       public function request_filesystem_credentials( $error = false, $context = '' ) {
-               if ( $context )
+       /**
+        *
+        * @param bool   $error
+        * @param string $context
+        * @param bool   $allow_relaxed_file_ownership
+        * @return bool
+        */
+       public function request_filesystem_credentials( $error = false, $context = '', $allow_relaxed_file_ownership = false ) {
+               if ( $context ) {
                        $this->options['context'] = $context;
+               }
                // TODO: fix up request_filesystem_credentials(), or split it, to allow us to request a no-output version
                // This will output a credentials form in event of failure, We don't want that, so just hide with a buffer
                ob_start();
-               $result = parent::request_filesystem_credentials( $error );
+               $result = parent::request_filesystem_credentials( $error, $context, $allow_relaxed_file_ownership );
                ob_end_clean();
                return $result;
        }
 
+       /**
+        * @access public
+        *
+        * @return array
+        */
        public function get_upgrade_messages() {
                return $this->messages;
        }
 
+       /**
+        * @param string|array|WP_Error $data
+        */
        public function feedback( $data ) {
-               if ( is_wp_error( $data ) )
+               if ( is_wp_error( $data ) ) {
                        $string = $data->get_error_message();
-               else if ( is_array( $data ) )
+               } elseif ( is_array( $data ) ) {
                        return;
-               else
+               } else {
                        $string = $data;
-
+               }
                if ( ! empty( $this->upgrader->strings[ $string ] ) )
                        $string = $this->upgrader->strings[ $string ];
 
@@ -749,19 +928,19 @@ class Automatic_Upgrader_Skin extends WP_Upgrader_Skin {
                $this->messages[] = $string;
        }
 
+       /**
+        * @access public
+        */
        public function header() {
                ob_start();
        }
 
+       /**
+        * @access public
+        */
        public function footer() {
-               $output = ob_get_contents();
+               $output = ob_get_clean();
                if ( ! empty( $output ) )
                        $this->feedback( $output );
-               ob_end_clean();
        }
-
-       public function bulk_header() {}
-       public function bulk_footer() {}
-       public function before() {}
-       public function after() {}
 }