Ticket #20921: 20921.diff

wp-includes/class-wp-customize-manager.php

@@ -72 +72 @@
          *
          * @since 3.4.0
          */
-        private function wp_die( $ajax_message, $message ) {
+        protected function wp_die( $ajax_message, $message = null ) {
                 if ( $this->doing_ajax() )
                         wp_die( $ajax_message );
 
+                if ( ! $message )
+                        $message = __( 'Cheatin’ uh?' );
+
                 wp_die( $message );
         }
 
@@ -100 +103 @@
         public function setup_theme() {
                 if ( is_admin() && ! $this->doing_ajax() )
                     auth_redirect();
-                elseif ( $this->doing_ajax() && ! is_user_logged_in())
-                    wp_die( 0 );
+                elseif ( $this->doing_ajax() && ! is_user_logged_in() )
+                    $this->wp_die( 0 );
 
                 send_origin_headers();
 
@@ -109 +112 @@
 
                 $this->theme = wp_get_theme( isset( $_REQUEST['theme'] ) ? $_REQUEST['theme'] : null );
 
-                // You can't preview a theme if it doesn't exist, or if it is not allowed (unless active).
+                if ( $this->is_theme_active() && ! $this->doing_ajax() && ! validate_current_theme() ) {
+                        wp_redirect( 'themes.php?broken=true' );
+                        exit;
+                }
+
                 if ( ! $this->theme->exists() )
-                        $this->wp_die( -1, __( 'Cheatin’ uh?' ) );
+                        $this->wp_die( -1 );
 
-                if ( $this->theme->get_stylesheet() != get_stylesheet() && ( ! $this->theme()->is_allowed() || ! current_user_can( 'switch_themes' ) ) )
-                        $this->wp_die( -1, __( 'Cheatin’ uh?' ) );
+                // If the theme isn't active, you can't preview it if it is not allowed or has errors.
+                if ( ! $this->is_theme_active() ) {
+                        if ( ! current_user_can( 'switch_themes' ) )
+                                $this->wp_die( -1 );
 
+                        if ( $this->theme()->errors() )
+                                $this->wp_die( -1 );
+
+                        if ( ! $this->theme()->is_allowed() )
+                                $this->wp_die( -1 );
+                }
+
                 if ( ! current_user_can( 'edit_theme_options' ) )
-                        $this->wp_die( -1, __( 'Cheatin’ uh?' ) );
+                        $this->wp_die( -1 );
 
                 $this->start_previewing_theme();
                 show_admin_bar( false );
@@ -137 +153 @@
 
                 $this->previewing = true;
 
-                add_filter( 'template', array( $this, 'get_template' ) );
-                add_filter( 'stylesheet', array( $this, 'get_stylesheet' ) );
-                add_filter( 'pre_option_current_theme', array( $this, 'current_theme' ) );
+                if ( ! $this->is_theme_active() ) {
+                        add_filter( 'template', array( $this, 'get_template' ) );
+                        add_filter( 'stylesheet', array( $this, 'get_stylesheet' ) );
+                        add_filter( 'pre_option_current_theme', array( $this, 'current_theme' ) );
+        
+                        // @link: http://core.trac.wordpress.org/ticket/20027
+                        add_filter( 'pre_option_stylesheet', array( $this, 'get_stylesheet' ) );
+                        add_filter( 'pre_option_template', array( $this, 'get_template' ) );
+        
+                        // Handle custom theme roots.
+                        add_filter( 'pre_option_stylesheet_root', array( $this, 'get_stylesheet_root' ) );
+                        add_filter( 'pre_option_template_root', array( $this, 'get_template_root' ) );
+                }
 
-                // @link: http://core.trac.wordpress.org/ticket/20027
-                add_filter( 'pre_option_stylesheet', array( $this, 'get_stylesheet' ) );
-                add_filter( 'pre_option_template', array( $this, 'get_template' ) );
-
-                // Handle custom theme roots.
-                add_filter( 'pre_option_stylesheet_root', array( $this, 'get_stylesheet_root' ) );
-                add_filter( 'pre_option_template_root', array( $this, 'get_template_root' ) );
-
                 do_action( 'start_previewing_theme', $this );
         }
 
@@ -165 +183 @@
 
                 $this->previewing = false;
 
-                remove_filter( 'template', array( $this, 'get_template' ) );
-                remove_filter( 'stylesheet', array( $this, 'get_stylesheet' ) );
-                remove_filter( 'pre_option_current_theme', array( $this, 'current_theme' ) );
+                if ( ! $this->is_theme_active() ) {
+                        remove_filter( 'template', array( $this, 'get_template' ) );
+                        remove_filter( 'stylesheet', array( $this, 'get_stylesheet' ) );
+                        remove_filter( 'pre_option_current_theme', array( $this, 'current_theme' ) );
+        
+                        // @link: http://core.trac.wordpress.org/ticket/20027
+                        remove_filter( 'pre_option_stylesheet', array( $this, 'get_stylesheet' ) );
+                        remove_filter( 'pre_option_template', array( $this, 'get_template' ) );
+        
+                        // Handle custom theme roots.
+                        remove_filter( 'pre_option_stylesheet_root', array( $this, 'get_stylesheet_root' ) );
+                        remove_filter( 'pre_option_template_root', array( $this, 'get_template_root' ) );
+                }
 
-                // @link: http://core.trac.wordpress.org/ticket/20027
-                remove_filter( 'pre_option_stylesheet', array( $this, 'get_stylesheet' ) );
-                remove_filter( 'pre_option_template', array( $this, 'get_template' ) );
-
-                // Handle custom theme roots.
-                remove_filter( 'pre_option_stylesheet_root', array( $this, 'get_stylesheet_root' ) );
-                remove_filter( 'pre_option_template_root', array( $this, 'get_template_root' ) );
-
                 do_action( 'stop_previewing_theme', $this );
         }
 
@@ -448 +468 @@
                 check_ajax_referer( 'customize_controls-' . $this->get_stylesheet(), 'nonce' );
 
                 // Do we have to switch themes?
-                if ( $this->get_stylesheet() != $this->original_stylesheet ) {
+                if ( ! $this->is_theme_active() ) {
                         // Temporarily stop previewing the theme to allow switch_themes()
                         // to operate properly.
                         $this->stop_previewing_theme();

wp-admin/themes.php

@@ -92 +92 @@
 require_once('./admin-header.php');
 ?>
 
-<?php if ( ! validate_current_theme() ) : ?>
+<?php if ( ! validate_current_theme() || isset( $_GET['broken'] ) ) : ?>
 <div id="message1" class="updated"><p><?php _e('The active theme is broken. Reverting to the default theme.'); ?></p></div>
 <?php elseif ( isset($_GET['activated']) ) :
                 if ( isset($wp_registered_sidebars) && count( (array) $wp_registered_sidebars ) && current_user_can('edit_theme_options') ) { ?>