WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
03/19/2012 05:12:44 PM (14 years ago)
Author:
nacin
Message:

Introduce new registration methods for custom headers and custom backgrounds. Backwards compatible, but old methods will be deprecated. see #20249. see #17242.

Custom header: Use add_theme_support('custom-header', $args) instead of add_custom_image_header(). Deprecates all use of constants.

  • HEADER_TEXTCOLOR is now (string) 'default-text-color'.
  • NO_HEADER_TEXT is nowi ! (bool) 'header-text'.
  • HEADER_IMAGE_WIDTH (and _HEIGHT) are now (int) 'width' and 'height'.
  • HEADER_IMAGE is now (string) 'default-image'.
  • The 3.4 arguments 'suggested-width' and 'suggested-height' are now just 'width' and 'height' (they are "suggested" when flex-width and flex-height are set).
  • Callback arguments for add_custom_image_header() can now be passed to add_theme_support().

Custom background: Use add_theme_support('custom-background, $args) instead of add_custom_background(). Deprecates all use of constants.

  • BACKGROUND_COLOR is now (string) 'default-color'.
  • BACKGROUND_IMAGE is now (string) 'default-image'.
  • Callback arguments for add_custom_background() can now be passed to add_theme_support().

Inheritance: add_theme_support() arguments for custom headers and custom backgrounds is a first-one-wins situation. This is not an unusual paradigm for theming as a child theme (which is included first) overrides a parent theme.

  • Once an argument is explicitly set, it cannot be overridden. You must hook in earlier and set it first.
  • Any argument that is not explicitly set before WP is loaded will inherit the default value for that argument.
  • It is therefore possible for a child theme to pass minimal arguments as long as the parent theme specifies others that may be necessary.
  • Allows for a child theme to alter callbacks for <head> and preview (previously, calling add_custom_image_header more than once broke things).
  • The just-in-time bits ensure that arguments fall back to default values, that the values of all constants are considered (such as one defined after an old add_custom_image_header call), and that all constants are defined (so as to be backwards compatible).

get_theme_support(): Introduce new second argument, which headers and backgrounds leverage to return an argument. current_theme_supports() already supported checking the truthiness of the argument.

  • For example, get_theme_support( 'custom-header', 'width' ) will return the width specified during registration.
  • If you had wanted the default image, use get_theme_support( 'custom-header', 'default-image' ) instead of HEADER_IMAGE.

Deprecate remove_custom_image_header(), remove_custom_background(). Use remove_theme_support('custom-header'), 'custom-background'.

Deprecate short-lived custom-header-uploads internal support; this is now (bool) 'uploads' for add_theme_support().

New 3.4 functions renamed or removed: Rename get_current_header_data() to get_custom_header(). Remove get_header_image_width() and _height() in favor of get_custom_header()->width and height.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/wp-admin/custom-header.php

    r20211 r20212  
    7272        $this->admin_header_callback = $admin_header_callback;
    7373        $this->admin_image_div_callback = $admin_image_div_callback;
     74
     75        add_action( 'admin_menu', array( $this, 'init' ) );
    7476    }
    7577
    … …  
    141143        $step = $this->step();
    142144
    143         if ( ( 1 == $step || 3 == $step ) && $this->header_text() )
     145        if ( ( 1 == $step || 3 == $step ) && current_theme_supports( 'custom-header', 'header-text' ) )
    144146            wp_enqueue_script('farbtastic');
    145147        elseif ( 2 == $step )
    … …  
    155157        $step = $this->step();
    156158
    157         if ( ( 1 == $step || 3 == $step ) && $this->header_text() )
     159        if ( ( 1 == $step || 3 == $step ) && current_theme_supports( 'custom-header', 'header-text' ) )
    158160            wp_enqueue_style('farbtastic');
    159161        elseif ( 2 == $step )
    … …  
    162164
    163165    /**
    164      * Check if header text is allowed
    165      *
    166      * @since 3.0.0
    167      */
    168     function header_text() {
    169         if ( defined( 'NO_HEADER_TEXT' ) && NO_HEADER_TEXT )
    170             return false;
    171 
    172         return true;
    173     }
    174 
    175     /**
    176166     * Execute custom header modification.
    177167     *
    … …  
    190180            check_admin_referer( 'custom-header-options', '_wpnonce-custom-header-options' );
    191181            $this->process_default_headers();
    192             $default = defined( 'HEADER_IMAGE' ) ? HEADER_IMAGE : '';
     182            $default = get_theme_support( 'custom-header', 'default-image' );
    193183            $default = sprintf( $default, get_template_directory_uri(), get_stylesheet_directory_uri() );
    194184            foreach ( $this->default_headers as $header => $details ) {
    … …  
    200190            set_theme_mod( 'header_image', $default );
    201191            if ( empty( $default_data['width'] ) )
    202                 $default_data['width'] = HEADER_IMAGE_WIDTH;
     192                $default_data['width'] = get_theme_support( 'custom-header', 'width' );
    203193            if ( empty( $default_data['height'] ) )
    204                 $default_data['height'] = HEADER_IMAGE_HEIGHT;
     194                $default_data['height'] = get_theme_support( 'custom-header', 'height' );
    205195            set_theme_mod( 'header_image_data', (object) $default_data );
    206196            return;
    … …  
    246236                    set_theme_mod( 'header_image', esc_url( $this->default_headers[$_POST['default-header']]['url'] ) );
    247237                    if ( empty( $this->default_headers[$_POST['default-header']]['width'] ) )
    248                         $this->default_headers[$_POST['default-header']]['width'] = HEADER_IMAGE_WIDTH;
     238                        $this->default_headers[$_POST['default-header']]['width'] = get_theme_support( 'custom-header', 'width' );
    249239                    if ( empty( $this->default_headers[$_POST['default-header']]['height'] ) )
    250                         $this->default_headers[$_POST['default-header']]['height'] = HEADER_IMAGE_HEIGHT;
     240                        $this->default_headers[$_POST['default-header']]['height'] = get_theme_support( 'custom-header', 'height' );
    251241                    set_theme_mod( 'header_image_data', (object) $this->default_headers[$_POST['default-header']] );
    252242                }
    … …  
    326316    function js() {
    327317        $step = $this->step();
    328         if ( ( 1 == $step || 3 == $step ) && $this->header_text() )
     318        if ( ( 1 == $step || 3 == $step ) && current_theme_supports( 'custom-header', 'header-text' ) )
    329319            $this->js_1();
    330320        elseif ( 2 == $step )
    … …  
    342332    var text_objects = ['#name', '#desc', '#text-color-row'];
    343333    var farbtastic;
    344     var default_color = '#<?php echo HEADER_TEXTCOLOR; ?>';
     334    var default_color = '#<?php echo get_theme_support( 'custom-header', 'default-text-color' ); ?>';
    345335    var old_color = null;
    346336
    … …  
    412402
    413403        farbtastic = jQuery.farbtastic('#color-picker', function(color) { pickColor(color); });
    414         <?php if ( $color = get_theme_mod('header_textcolor', HEADER_TEXTCOLOR) ) { ?>
     404        <?php if ( $color = get_header_textcolor() ) { ?>
    415405        pickColor('#<?php echo $color; ?>');
    416406        <?php } ?>
    417407
    418         <?php if ( 'blank' == get_theme_mod( 'header_textcolor', HEADER_TEXTCOLOR ) || '' == get_theme_mod('header_textcolor', HEADER_TEXTCOLOR) || ! $this->header_text() ) { ?>
     408        <?php if ( 'blank' == $color || '' == $color || ! current_theme_supports( 'custom-header', 'header-text' ) ) { ?>
    419409        toggle_text();
    420410        <?php } ?>
    … …  
    441431
    442432    jQuery(document).ready(function() {
    443         <?php
    444         $xinit = HEADER_IMAGE_WIDTH;
    445         $yinit = HEADER_IMAGE_HEIGHT;
    446         $header_support = get_theme_support( 'custom-header' );
    447         if ( !empty( $header_support[ 0 ][ 'suggested-width' ] ) )
    448             $xinit = $header_support[ 0 ][ 'suggested-width' ];
    449         if ( !empty( $header_support[ 0 ][ 'suggested-height' ] ) )
    450             $yinit = $header_support[ 0 ][ 'suggested-height' ];
    451         ?>
    452         var xinit = <?php echo absint( $xinit ); ?>;
    453         var yinit = <?php echo absint( $yinit ); ?>;
     433        var xinit = <?php echo absint( get_theme_support( 'custom-header', 'width' ) ); ?>;
     434        var yinit = <?php echo absint( get_theme_support( 'custom-header', 'height' ) ); ?>;
    454435        var ratio = xinit / yinit;
    455436        var ximg = jQuery('img#upload').width();
    … …  
    482463            if ( ! current_theme_supports( 'custom-header', 'flex-height' ) ) {
    483464            ?>
    484             maxHeight: <?php echo HEADER_IMAGE_HEIGHT; ?>,
     465            maxHeight: <?php echo get_theme_support( 'custom-header', 'height' ); ?>,
    485466            <?php
    486467            }
    487468            if ( ! current_theme_supports( 'custom-header', 'flex-width' ) ) {
    488469            ?>
    489             maxWidth: <?php echo HEADER_IMAGE_WIDTH; ?>,
     470            maxWidth: <?php echo get_theme_support( 'custom-header', 'width' ); ?>,
    490471            <?php
    491472            }
    … …  
    532513<tr valign="top">
    533514<th scope="row"><?php _e( 'Preview' ); ?></th>
    534 <td >
     515<td>
    535516    <?php if ( $this->admin_image_div_callback ) {
    536517      call_user_func( $this->admin_image_div_callback );
    537518    } else {
    538519    ?>
    539     <div id="headimg" style="background-image:url(<?php esc_url ( header_image() ) ?>);max-width:<?php echo get_header_image_width(); ?>px;height:<?php echo get_header_image_height(); ?>px;">
     520    <div id="headimg" style="background-image:url(<?php esc_url ( header_image() ) ?>);max-width:<?php echo get_custom_header()->width; ?>px;height:<?php echo get_custom_header()->height; ?>px;">
    540521        <?php
    541         if ( 'blank' == get_theme_mod('header_textcolor', HEADER_TEXTCOLOR) || '' == get_theme_mod('header_textcolor', HEADER_TEXTCOLOR) || ! $this->header_text() )
     522        $color = get_header_textcolor();
     523        if ( 'blank' == $color || '' == $color || ! current_theme_supports( 'custom-header', 'header-text' ) )
    542524            $style = ' style="display:none;"';
    543525        else
    544             $style = ' style="color:#' . get_theme_mod( 'header_textcolor', HEADER_TEXTCOLOR ) . ';"';
     526            $style = ' style="color:#' . $color . ';"';
    545527        ?>
    546528        <h1><a id="name"<?php echo $style; ?> onclick="return false;" href="<?php bloginfo('url'); ?>"><?php bloginfo( 'name' ); ?></a></h1>
    … …  
    550532</td>
    551533</tr>
    552 <?php if ( current_theme_supports( 'custom-header-uploads' ) ) : ?>
     534<?php if ( current_theme_supports( 'custom-header', 'uploads' ) ) : ?>
    553535<tr valign="top">
    554536<th scope="row"><?php _e( 'Upload Image' ); ?></th>
    … …  
    557539    <?php
    558540    if ( ! current_theme_supports( 'custom-header', 'flex-height' ) && ! current_theme_supports( 'custom-header', 'flex-width' ) ) {
    559         printf( __( 'Images of exactly <strong>%1$d &times; %2$d pixels</strong> will be used as-is.' ) . '<br />', HEADER_IMAGE_WIDTH, HEADER_IMAGE_HEIGHT );
     541        printf( __( 'Images of exactly <strong>%1$d &times; %2$d pixels</strong> will be used as-is.' ) . '<br />', get_theme_support( 'custom-header', 'width' ), get_theme_support( 'custom-header', 'height' ) );
    560542    } elseif ( current_theme_supports( 'custom-header', 'flex-height' ) ) {
    561543        if ( ! current_theme_supports( 'custom-header', 'flex-width' ) )
    562             printf( __( 'Images should be at least <strong>%1$d pixels</strong> wide.' ) . '<br />', HEADER_IMAGE_WIDTH );
     544            printf( __( 'Images should be at least <strong>%1$d pixels</strong> wide.' ) . ' ', get_theme_support( 'custom-header', 'width' ) );
    563545    } elseif ( current_theme_supports( 'custom-header', 'flex-width' ) ) {
    564546        if ( ! current_theme_supports( 'custom-header', 'flex-height' ) )
    565             printf( __( 'Images should be at least <strong>%1$d pixels</strong> tall.' ) . '<br />', HEADER_IMAGE_HEIGHT );
     547            printf( __( 'Images should be at least <strong>%1$d pixels</strong> tall.' ) . ' ', get_theme_support( 'custom-header', 'height' ) );
    566548    }
    567549    if ( current_theme_supports( 'custom-header', 'flex-height' ) || current_theme_supports( 'custom-header', 'flex-width' ) ) {
    568         $header_support = get_theme_support( 'custom-header' );
    569         if ( !empty( $header_support[ 0 ][ 'suggested-width' ] ) )
    570             printf( __( 'Suggested width is <strong>%1$d pixels</strong>.' ) . '<br />', absint( $header_support[ 0 ][ 'suggested-width' ] ) );
    571         if ( !empty( $header_support[ 0 ][ 'suggested-height' ] ) )
    572             printf( __( 'Suggested height is <strong>%1$d pixels</strong>.' ) . '<br />', absint( $header_support[ 0 ][ 'suggested-height' ] ) );
     550        if ( current_theme_supports( 'custom-header', 'width' ) )
     551            printf( __( 'Suggested width is <strong>%1$d pixels</strong>.' ) . ' ', get_theme_support( 'custom-header', 'width' ) );
     552        if ( current_theme_supports( 'custom-header', 'height' ) )
     553            printf( __( 'Suggested height is <strong>%1$d pixels</strong>.' ) . ' ', get_theme_support( 'custom-header', 'height' ) );
    573554    }
    574555    ?></p>
    … …  
    606587<th scope="row"><?php _e( 'Default Images' ); ?></th>
    607588<td>
    608 <?php if ( current_theme_supports( 'custom-header-uploads' ) ) : ?>
     589<?php if ( current_theme_supports( 'custom-header', 'uploads' ) ) : ?>
    609590    <p><?php _e( 'If you don&lsquo;t want to upload your own image, you can use one of these cool headers, or show a random one.' ) ?></p>
    610591<?php else: ?>
    … …  
    627608    <?php endif;
    628609
    629     if ( defined( 'HEADER_IMAGE' ) && '' != HEADER_IMAGE ) : ?>
     610    if ( current_theme_supports( 'custom-header', 'default-image' ) ) : ?>
    630611<tr valign="top">
    631612<th scope="row"><?php _e( 'Reset Image' ); ?></th>
    … …  
    638619</tbody>
    639620</table>
    640     <?php if ( $this->header_text() ) : ?>
     621    <?php if ( current_theme_supports( 'custom-header', 'header-text' ) ) : ?>
    641622<table class="form-table">
    642623<tbody>
    … …  
    645626<td>
    646627    <p>
    647     <?php $hidetext = get_theme_mod( 'header_textcolor', HEADER_TEXTCOLOR ); ?>
    648     <label><input type="radio" value="1" name="hidetext" id="hidetext"<?php checked( ( 'blank' == $hidetext || empty( $hidetext ) )  ? true : false ); ?> /> <?php _e( 'No' ); ?></label>
    649     <label><input type="radio" value="0" name="hidetext" id="showtext"<?php checked( ( 'blank' == $hidetext || empty( $hidetext ) ) ? false : true ); ?> /> <?php _e( 'Yes' ); ?></label>
     628    <?php
     629        $show_text = get_header_textcolor();
     630        if ( 'blank' == $show_text )
     631            $show_text = false;
     632        else
     633            $show_text = (bool) $show_text;
     634    ?>
     635    <label><input type="radio" value="1" name="hidetext" id="hidetext"<?php checked( ! $show_text ); ?> /> <?php _e( 'No' ); ?></label>
     636    <label><input type="radio" value="0" name="hidetext" id="showtext"<?php checked( $show_text ); ?> /> <?php _e( 'Yes' ); ?></label>
    650637    </p>
    651638</td>
    … …  
    656643<td>
    657644    <p>
    658         <input type="text" name="text-color" id="text-color" value="#<?php echo esc_attr( get_theme_mod( 'header_textcolor', HEADER_TEXTCOLOR ) ); ?>" />
     645        <input type="text" name="text-color" id="text-color" value="#<?php echo esc_attr( get_header_textcolor() ); ?>" />
    659646        <span class="description hide-if-js"><?php _e( 'If you want to hide header text, add <strong>#blank</strong> as text color.' );?></span>
    660647        <input type="button" class="button hide-if-no-js" value="<?php esc_attr_e( 'Select a Color' ); ?>" id="pickcolor" />
    … …  
    664651</tr>
    665652
    666     <?php if ( defined('HEADER_TEXTCOLOR') && get_theme_mod('header_textcolor') ) { ?>
     653    <?php if ( current_theme_supports( 'custom-header', 'default-text-color' ) && get_theme_mod( 'header_textcolor' ) ) { ?>
    667654<tr valign="top">
    668655<th scope="row"><?php _e('Reset Text Color'); ?></th>
    … …  
    695682    function step_2() {
    696683        check_admin_referer('custom-header-upload', '_wpnonce-custom-header-upload');
    697         if ( ! current_theme_supports( 'custom-header-uploads' ) )
     684        if ( ! current_theme_supports( 'custom-header', 'uploads' ) )
    698685            wp_die( __( 'Cheatin&#8217; uh?' ) );
    699686
    … …  
    723710        list($width, $height, $type, $attr) = getimagesize( $file );
    724711
    725         $header_support = get_theme_support( 'custom-header' );
    726712        $max_width = 0;
    727713        // For flex, limit size of image displayed to 1500px unless theme says otherwise
    … …  
    729715            $max_width = 1500;
    730716
    731         if ( !empty( $header_support[ 0 ][ 'max-width' ] ) )
    732             $max_width = max( $max_width, absint( $header_support[ 0 ][ 'max-width' ] ) );
    733 
    734         if ( defined( 'HEADER_IMAGE_WIDTH' ) )
    735             $max_width = max( $max_width, HEADER_IMAGE_WIDTH );
     717        if ( current_theme_supports( 'custom-header', 'max-width' ) )
     718            $max_width = max( $max_width, get_theme_support( 'custom-header', 'max-width' ) );
     719        $max_width = max( $max_width, get_theme_support( 'custom-header', 'width' ) );
     720
    736721        // If flexible height isn't supported and the image is the exact right size
    737         if ( ! current_theme_supports( 'custom-header', 'flex-height' ) && ! current_theme_supports( 'custom-header', 'flex-width' ) && $width == HEADER_IMAGE_WIDTH && $height == HEADER_IMAGE_HEIGHT ) {
     722        if ( ! current_theme_supports( 'custom-header', 'flex-height' ) && ! current_theme_supports( 'custom-header', 'flex-width' )
     723            && $width == get_theme_support( 'custom-header', 'width' ) && $height == get_theme_support( 'custom-header', 'height' ) )
     724        {
    738725            // Add the meta-data
    739726            wp_update_attachment_metadata( $id, wp_generate_attachment_metadata( $id, $file ) );
    … …  
    792779     */
    793780    function step_3() {
    794         check_admin_referer('custom-header-crop-image');
    795         if ( ! current_theme_supports( 'custom-header-uploads' ) )
     781        check_admin_referer( 'custom-header-crop-image' );
     782
     783        if ( ! current_theme_supports( 'custom-header', 'uploads' ) )
    796784            wp_die( __( 'Cheatin&#8217; uh?' ) );
    797785
    … …  
    806794        $original = get_attached_file($attachment_id);
    807795
    808         $header_support = get_theme_support( 'custom-header' );
     796
    809797        $max_width = 0;
    810798        // For flex, limit size of image displayed to 1500px unless theme says otherwise
    … …  
    812800            $max_width = 1500;
    813801
    814         if ( !empty( $header_support[ 0 ][ 'max-width' ] ) )
    815             $max_width = max( $max_width, absint( $header_support[ 0 ][ 'max-width' ] ) );
    816 
    817         if ( defined( 'HEADER_IMAGE_WIDTH' ) )
    818             $max_width = max( $max_width, HEADER_IMAGE_WIDTH );
     802        if ( current_theme_supports( 'custom-header', 'max-width' ) )
     803            $max_width = max( $max_width, get_theme_support( 'custom-header', 'max-width' ) );
     804        $max_width = max( $max_width, get_theme_support( 'custom-header', 'width' ) );
    819805
    820806        if ( ( current_theme_supports( 'custom-header', 'flex-height' ) && ! current_theme_supports( 'custom-header', 'flex-width' ) ) || $_POST['width'] > $max_width )
    … …  
    823809            $dst_height = absint( $_POST['height'] );
    824810        else
    825             $dst_height = HEADER_IMAGE_HEIGHT;
     811            $dst_height = get_theme_support( 'custom-header', 'height' );
    826812
    827813        if ( ( current_theme_supports( 'custom-header', 'flex-width' ) && ! current_theme_supports( 'custom-header', 'flex-height' ) ) || $_POST['width'] > $max_width )
    … …  
    830816            $dst_width = absint( $_POST['width'] );
    831817        else
    832             $dst_width = HEADER_IMAGE_WIDTH;
     818            $dst_width = get_theme_support( 'custom-header', 'width' );
    833819
    834820        $cropped = wp_crop_image( $attachment_id, (int) $_POST['x1'], (int) $_POST['y1'], (int) $_POST['width'], (int) $_POST['height'], $dst_width, $dst_height );
Note: See TracChangeset for help on using the changeset viewer.