Custom header from theme A used when previewing theme B

[Viper007Bond]

As reported by Matt.

If you're using theme A and set a custom header via the standardized API (i.e. a background image and text color) and then preview theme B on the theme switcher, the settings from theme A are used. Once you actually switch themes though, it works fine and uses the settings from theme B.

The problem stems from get_current_theme() returning theme A even when previewing theme B.

function get_theme_mod($name, $default = false) {
        $theme = get_current_theme();

        $mods = get_option("mods_$theme");

        if ( isset($mods[$name]) )
                return apply_filters( "theme_mod_$name", $mods[$name] );

        return apply_filters( "theme_mod_$name", sprintf($default, get_template_directory_uri(), get_stylesheet_directory_uri()) );
}

Still working on a solution to this (i.e. telling WordPress it's using theme B during the preview or something).

This patch seems to do the trick. It hacks the theme header mods option to return the mod settings for the theme being displayed rather than the actual current theme.

Previous patch wasn't properly returning mod options for theme being previewed, now it does

Cool.

Could you point at a theme that uses this for testing purposes.

I used the "ambiru" and "benevolence" themes from the WordPress.com theme SVN for testing purposes.

Just did a quick test of this - not enough time to investigate further but it looked like the preview was still showing the image text colour from the active theme.

Ooooh, I think I just realized what the problem is. The theme you're previewing doesn't have a custom header, does it?

If not, that means get_option('mod_PreviewTheme') returns false. That makes the filter not kick into effect and abort the get_option('mod_CurrentTheme').

Looks like rather than showing the previewed theme's custom header when previewing it, we'll have to display just the default version of the theme at all themes.

Always display the default header when previewing a theme

(In [11061]) Always display the default header when previewing a theme. Props Viper007Bond. fixes #9152