WordPress.org

Make WordPress Core

Ticket #20921: 20921.4.diff

File 20921.4.diff, 9.7 KB (added by nacin, 5 years ago)

Moves send_origin_headers() above the edit_theme_options cap check.

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

     
    7272         * 
    7373         * @since 3.4.0 
    7474         */ 
    75         private function wp_die( $ajax_message, $message ) { 
     75        protected function wp_die( $ajax_message, $message = null ) { 
    7676                if ( $this->doing_ajax() ) 
    7777                        wp_die( $ajax_message ); 
    7878 
     79                if ( ! $message ) 
     80                        $message = __( 'Cheatin’ uh?' ); 
     81 
    7982                wp_die( $message ); 
    8083        } 
    8184 
    … …  
    98101         * @since 3.4.0 
    99102         */ 
    100103        public function setup_theme() { 
     104                send_origin_headers(); 
     105 
    101106                if ( is_admin() && ! $this->doing_ajax() ) 
    102107                    auth_redirect(); 
    103                 elseif ( $this->doing_ajax() && ! is_user_logged_in()) 
    104                     wp_die( 0 ); 
     108                elseif ( $this->doing_ajax() && ! is_user_logged_in() ) 
     109                    $this->wp_die( 0 ); 
    105110 
    106                 send_origin_headers(); 
     111                show_admin_bar( false ); 
    107112 
     113                if ( ! current_user_can( 'edit_theme_options' ) ) 
     114                        $this->wp_die( -1 ); 
     115 
    108116                $this->original_stylesheet = get_stylesheet(); 
    109117 
    110118                $this->theme = wp_get_theme( isset( $_REQUEST['theme'] ) ? $_REQUEST['theme'] : null ); 
    111119 
    112                 // You can't preview a theme if it doesn't exist, or if it is not allowed (unless active). 
    113                 if ( ! $this->theme->exists() ) 
    114                         $this->wp_die( -1, __( 'Cheatin’ uh?' ) ); 
     120                if ( $this->is_theme_active() ) { 
     121                        // Once the theme is loaded, we'll validate it. 
     122                        add_action( 'after_setup_theme', array( $this, 'after_setup_theme' ) ); 
     123                } else { 
     124                        if ( ! current_user_can( 'switch_themes' ) ) 
     125                                $this->wp_die( -1 ); 
    115126 
    116                 if ( $this->theme->get_stylesheet() != get_stylesheet() && ( ! $this->theme()->is_allowed() || ! current_user_can( 'switch_themes' ) ) ) 
    117                         $this->wp_die( -1, __( 'Cheatin’ uh?' ) ); 
     127                        // If the theme isn't active, you can't preview it if it is not allowed or has errors. 
     128                        if ( $this->theme()->errors() ) 
     129                                $this->wp_die( -1 ); 
    118130 
    119                 if ( ! current_user_can( 'edit_theme_options' ) ) 
    120                         $this->wp_die( -1, __( 'Cheatin’ uh?' ) ); 
     131                        if ( ! $this->theme()->is_allowed() ) 
     132                                $this->wp_die( -1 ); 
     133                } 
    121134 
    122135                $this->start_previewing_theme(); 
    123                 show_admin_bar( false ); 
    124136        } 
    125137 
     138        function after_setup_theme() { 
     139                if ( ! $this->doing_ajax() && ! validate_current_theme() ) { 
     140                        wp_redirect( 'themes.php?broken=true' ); 
     141                        exit; 
     142                } 
     143        } 
     144 
    126145        /** 
    127146         * Start previewing the selected theme. 
    128147         * 
    … …  
    137156 
    138157                $this->previewing = true; 
    139158 
    140                 add_filter( 'template', array( $this, 'get_template' ) ); 
    141                 add_filter( 'stylesheet', array( $this, 'get_stylesheet' ) ); 
    142                 add_filter( 'pre_option_current_theme', array( $this, 'current_theme' ) ); 
     159                if ( ! $this->is_theme_active() ) { 
     160                        add_filter( 'template', array( $this, 'get_template' ) ); 
     161                        add_filter( 'stylesheet', array( $this, 'get_stylesheet' ) ); 
     162                        add_filter( 'pre_option_current_theme', array( $this, 'current_theme' ) ); 
     163         
     164                        // @link: http://core.trac.wordpress.org/ticket/20027 
     165                        add_filter( 'pre_option_stylesheet', array( $this, 'get_stylesheet' ) ); 
     166                        add_filter( 'pre_option_template', array( $this, 'get_template' ) ); 
     167         
     168                        // Handle custom theme roots. 
     169                        add_filter( 'pre_option_stylesheet_root', array( $this, 'get_stylesheet_root' ) ); 
     170                        add_filter( 'pre_option_template_root', array( $this, 'get_template_root' ) ); 
     171                } 
    143172 
    144                 // @link: http://core.trac.wordpress.org/ticket/20027 
    145                 add_filter( 'pre_option_stylesheet', array( $this, 'get_stylesheet' ) ); 
    146                 add_filter( 'pre_option_template', array( $this, 'get_template' ) ); 
    147  
    148                 // Handle custom theme roots. 
    149                 add_filter( 'pre_option_stylesheet_root', array( $this, 'get_stylesheet_root' ) ); 
    150                 add_filter( 'pre_option_template_root', array( $this, 'get_template_root' ) ); 
    151  
    152173                do_action( 'start_previewing_theme', $this ); 
    153174        } 
    154175 
    … …  
    165186 
    166187                $this->previewing = false; 
    167188 
    168                 remove_filter( 'template', array( $this, 'get_template' ) ); 
    169                 remove_filter( 'stylesheet', array( $this, 'get_stylesheet' ) ); 
    170                 remove_filter( 'pre_option_current_theme', array( $this, 'current_theme' ) ); 
     189                if ( ! $this->is_theme_active() ) { 
     190                        remove_filter( 'template', array( $this, 'get_template' ) ); 
     191                        remove_filter( 'stylesheet', array( $this, 'get_stylesheet' ) ); 
     192                        remove_filter( 'pre_option_current_theme', array( $this, 'current_theme' ) ); 
     193         
     194                        // @link: http://core.trac.wordpress.org/ticket/20027 
     195                        remove_filter( 'pre_option_stylesheet', array( $this, 'get_stylesheet' ) ); 
     196                        remove_filter( 'pre_option_template', array( $this, 'get_template' ) ); 
     197         
     198                        // Handle custom theme roots. 
     199                        remove_filter( 'pre_option_stylesheet_root', array( $this, 'get_stylesheet_root' ) ); 
     200                        remove_filter( 'pre_option_template_root', array( $this, 'get_template_root' ) ); 
     201                } 
    171202 
    172                 // @link: http://core.trac.wordpress.org/ticket/20027 
    173                 remove_filter( 'pre_option_stylesheet', array( $this, 'get_stylesheet' ) ); 
    174                 remove_filter( 'pre_option_template', array( $this, 'get_template' ) ); 
    175  
    176                 // Handle custom theme roots. 
    177                 remove_filter( 'pre_option_stylesheet_root', array( $this, 'get_stylesheet_root' ) ); 
    178                 remove_filter( 'pre_option_template_root', array( $this, 'get_template_root' ) ); 
    179  
    180203                do_action( 'stop_previewing_theme', $this ); 
    181204        } 
    182205 
    … …  
    389412         * @return string Template name. 
    390413         */ 
    391414        public function get_template() { 
    392                 return $this->theme->get_template(); 
     415                return $this->theme()->get_template(); 
    393416        } 
    394417 
    395418        /** 
    … …  
    400423         * @return string Stylesheet name. 
    401424         */ 
    402425        public function get_stylesheet() { 
    403                 return $this->theme->get_stylesheet(); 
     426                return $this->theme()->get_stylesheet(); 
    404427        } 
    405428 
    406429        /** 
    … …  
    433456         * @return string Theme name. 
    434457         */ 
    435458        public function current_theme( $current_theme ) { 
    436                 return $this->theme->display('Name'); 
     459                return $this->theme()->display('Name'); 
    437460        } 
    438461 
    439462        /** 
    … …  
    448471                check_ajax_referer( 'customize_controls-' . $this->get_stylesheet(), 'nonce' ); 
    449472 
    450473                // Do we have to switch themes? 
    451                 if ( $this->get_stylesheet() != $this->original_stylesheet ) { 
     474                if ( ! $this->is_theme_active() ) { 
    452475                        // Temporarily stop previewing the theme to allow switch_themes() 
    453476                        // to operate properly. 
    454477                        $this->stop_previewing_theme(); 
    … …  
    462485                        $setting->save(); 
    463486                } 
    464487 
    465                 add_action( 'admin_notices', array( $this, '_save_feedback' ) ); 
    466  
    467488                die; 
    468489        } 
    469490 
    470491        /** 
    471          * Show an admin notice after settings are saved. 
    472          * 
    473          * @since 3.4.0 
    474          */ 
    475         public function _save_feedback() { 
    476                 ?> 
    477                 <div class="updated"><p><?php printf( __( 'Settings saved and theme activated. <a href="%s">Visit site</a>.' ), home_url( '/' ) ); ?></p></div> 
    478                 <?php 
    479         } 
    480  
    481         /** 
    482492         * Add a customize setting. 
    483493         * 
    484494         * @since 3.4.0 

  • wp-admin/customize.php

     
    161161                'url'      => array( 
    162162                        'preview'       => esc_url( $url ? $url : home_url( '/' ) ), 
    163163                        'parent'        => esc_url( admin_url() ), 
    164                         'activated'     => esc_url( admin_url( 'themes.php?activated=true' ) ), 
     164                        'activated'     => admin_url( 'themes.php?activated=true&previewed' ), 
    165165                        'ajax'          => esc_url( admin_url( 'admin-ajax.php', 'relative' ) ), 
    166166                        'allowed'       => array_map( 'esc_url', $allowed_urls ), 
    167167                        'isCrossDomain' => $cross_domain, 

  • wp-admin/themes.php

     
    9292require_once('./admin-header.php'); 
    9393?> 
    9494 
    95 <?php if ( ! validate_current_theme() ) : ?> 
    96 <div id="message1" class="updated"><p><?php _e('The active theme is broken. Reverting to the default theme.'); ?></p></div> 
    97 <?php elseif ( isset($_GET['activated']) ) : 
    98                 if ( isset($wp_registered_sidebars) && count( (array) $wp_registered_sidebars ) && current_user_can('edit_theme_options') ) { ?> 
    99 <div id="message2" class="updated"><p><?php printf( __('New theme activated. This theme supports widgets, please visit the <a href="%s">widgets settings</a> screen to configure them.'), admin_url( 'widgets.php' ) ); ?></p></div><?php 
    100                 } else { ?> 
    101 <div id="message2" class="updated"><p><?php printf( __( 'New theme activated. <a href="%s">Visit site</a>' ), home_url( '/' ) ); ?></p></div><?php 
    102                 } 
    103         elseif ( isset($_GET['deleted']) ) : ?> 
    104 <div id="message3" class="updated"><p><?php _e('Theme deleted.') ?></p></div> 
    105 <?php endif; ?> 
    106  
    10795<div class="wrap"><?php 
    10896screen_icon(); 
    10997if ( ! is_multisite() && current_user_can( 'install_themes' ) ) : ?> 
    … …  
    114102<?php endif; ?> 
    115103</h2> 
    116104<?php 
     105if ( ! validate_current_theme() || isset( $_GET['broken'] ) ) : ?> 
     106<div id="message1" class="updated"><p><?php _e('The active theme is broken. Reverting to the default theme.'); ?></p></div> 
     107<?php elseif ( isset($_GET['activated']) ) : 
     108                if ( isset( $_GET['previewed'] ) ) { ?> 
     109                <div id="message2" class="updated"><p><?php printf( __( 'Settings saved and theme activated. <a href="%s">Visit site</a>.' ), home_url( '/' ) ); ?></p></div> 
     110                <?php } elseif ( isset($wp_registered_sidebars) && count( (array) $wp_registered_sidebars ) && current_user_can('edit_theme_options') ) { ?> 
     111<div id="message2" class="updated"><p><?php printf( __('New theme activated. This theme supports widgets, please visit the <a href="%s">widgets settings</a> screen to configure them.'), admin_url( 'widgets.php' ) ); ?></p></div><?php 
     112                } else { ?> 
     113<div id="message2" class="updated"><p><?php printf( __( 'New theme activated. <a href="%s">Visit site</a>' ), home_url( '/' ) ); ?></p></div><?php 
     114                } 
     115        elseif ( isset($_GET['deleted']) ) : ?> 
     116<div id="message3" class="updated"><p><?php _e('Theme deleted.') ?></p></div> 
     117<?php 
     118endif; 
    117119 
    118120$ct = wp_get_theme(); 
    119121$screenshot = $ct->get_screenshot();