WordPress.org

Make WordPress Core

Changeset 27869


Ignore:
Timestamp:
03/31/2014 05:15:39 AM (7 years ago)
Author:
wonderboymusic
Message:

Cleanup up the display, escaping, and handling of ID3 data for media. Rename wp_get_relevant_id3_keys() to wp_get_attachment_id3_keys().

Props nacin.
See #27574.

Location:
trunk/src
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/includes/media.php

    r27864 r27869  
    26622662    <div class="wp_attachment_details edit-form-section">
    26632663        <p>
    2664             <label for="attachment_caption"><strong><?php _e( 'Caption' ); ?></strong><?php
    2665             if ( preg_match( '#^(audio|video)/#', $post->post_mime_type ) ): ?>: <?php
    2666                 _e( "Custom label for item in a playlist. If empty, the file's available data is used." );
    2667             endif ?></label><br />
     2664            <label for="attachment_caption"><strong><?php _e( 'Caption' ); ?></strong></label><br />
    26682665            <textarea class="widefat" name="excerpt" id="attachment_caption"><?php echo $post->post_excerpt; ?></textarea>
    26692666        </p>
     
    26892686
    26902687    <label for="content"><strong><?php _e( 'Description' ); ?></strong><?php
    2691     if ( preg_match( '#^(audio|video)/#', $post->post_mime_type ) ): ?>: <?php
    2692         _e( 'Displayed on attachment pages.' );
    2693     endif ?></label>
     2688    if ( preg_match( '#^(audio|video)/#', $post->post_mime_type ) ) {
     2689        echo ': ' . __( 'Displayed on attachment pages.' );
     2690    } ?></label>
    26942691    <?php wp_editor( $post->post_content, 'attachment_content', $editor_args ); ?>
    26952692
     
    27292726    <div class="misc-pub-section misc-pub-filetype">
    27302727        <?php _e( 'File type:' ); ?> <strong><?php
    2731             if ( preg_match( '/^.*?\.(\w+)$/', get_attached_file( $post->ID ), $matches ) )
     2728            if ( preg_match( '/^.*?\.(\w+)$/', get_attached_file( $post->ID ), $matches ) ) {
    27322729                echo esc_html( strtoupper( $matches[1] ) );
    2733             else
     2730                list( $mime_type ) = explode( '/', $post->post_mime_type );
     2731                if ( $mime_type !== 'image' && ! empty( $meta['mime_type'] ) ) {
     2732                    if ( $meta['mime_type'] !== "$mime_type/" . strtolower( $matches[1] ) ) {
     2733                        echo ' (' . $meta['mime_type'] . ')';
     2734                    }
     2735                }
     2736            } else {
    27342737                echo strtoupper( str_replace( 'image/', '', $post->post_mime_type ) );
     2738            }
    27352739        ?></strong>
    27362740    </div>
     
    27522756        endif;
    27532757
    2754     if ( preg_match( '#^(audio|video)/#', $post->post_mime_type ) ):
     2758    if ( preg_match( '#^(audio|video)/#', $post->post_mime_type ) ) {
    27552759
    27562760        /**
     
    27622766         * @since 3.7.0
    27632767         *
    2764          * @param array $fields {
    2765          *     An array of the attachment metadata keys and labels.
    2766          *
    2767          *     @type string $mime_type        Label to be shown before the field mime_type.
    2768          *     @type string $year             Label to be shown before the field year.
    2769          *     @type string $genre            Label to be shown before the field genre.
    2770          *     @type string $length_formatted Label to be shown before the field length_formatted.
    2771          * }
     2768         * @param array $fields An array of the attachment metadata keys and labels.
    27722769         */
    27732770        $fields = apply_filters( 'media_submitbox_misc_sections', array(
    2774             'mime_type'        => __( 'Mime-type:' ),
    27752771            'length_formatted' => __( 'Length:' ),
     2772            'bitrate'          => __( 'Bitrate:' ),
    27762773        ) );
    27772774
    2778         foreach ( $fields as $key => $label ):
    2779             if ( ! empty( $meta[$key] ) ) : ?>
     2775        foreach ( $fields as $key => $label ) {
     2776            if ( empty( $meta[ $key ] ) ) {
     2777                continue;
     2778            }
     2779    ?>
    27802780        <div class="misc-pub-section misc-pub-mime-meta misc-pub-<?php echo sanitize_html_class( $key ); ?>">
    2781             <?php echo $label ?> <strong><?php echo esc_html( $meta[$key] ); ?></strong>
    2782         </div>
    2783     <?php
    2784             endif;
    2785         endforeach;
    2786 
    2787         if ( ! empty( $meta['bitrate'] ) ) : ?>
    2788         <div class="misc-pub-section misc-pub-bitrate">
    2789             <?php _e( 'Bitrate:' ); ?> <strong><?php
    2790                 echo round( $meta['bitrate'] / 1000 ), 'kb/s';
    2791 
    2792                 if ( ! empty( $meta['bitrate_mode'] ) )
    2793                     echo ' ', strtoupper( $meta['bitrate_mode'] );
    2794 
     2781            <?php echo $label ?> <strong><?php
     2782                switch ( $key ) {
     2783                    case 'bitrate' :
     2784                        echo round( $meta['bitrate'] / 1000 ) . 'kb/s';
     2785                        if ( ! empty( $meta['bitrate_mode'] ) ) {
     2786                            echo ' ' . strtoupper( esc_html( $meta['bitrate_mode'] ) );
     2787                        }
     2788                        break;
     2789                    default:
     2790                        echo esc_html( $meta[ $key ] );
     2791                        break;
     2792                }
    27952793            ?></strong>
    27962794        </div>
    27972795    <?php
    2798         endif;
     2796        }
    27992797
    28002798        /**
     
    28062804         * @since 3.7.0
    28072805         *
    2808          * @param array $fields {
    2809          *     An array of the attachment metadata keys and labels.
    2810          *
    2811          *     @type string $dataformat Label to be shown before the field dataformat.
    2812          *     @type string $codec      Label to be shown before the field codec.
    2813          * }
     2806         * @param array $fields An array of the attachment metadata keys and labels.
    28142807         */
    28152808        $audio_fields = apply_filters( 'audio_submitbox_misc_sections', array(
     
    28182811        ) );
    28192812
    2820         foreach ( $audio_fields as $key => $label ):
    2821             if ( ! empty( $meta['audio'][$key] ) ) : ?>
     2813        foreach ( $audio_fields as $key => $label ) {
     2814            if ( empty( $meta['audio'][ $key ] ) ) {
     2815                continue;
     2816            }
     2817    ?>
    28222818        <div class="misc-pub-section misc-pub-audio misc-pub-<?php echo sanitize_html_class( $key ); ?>">
    28232819            <?php echo $label; ?> <strong><?php echo esc_html( $meta['audio'][$key] ); ?></strong>
    28242820        </div>
    28252821    <?php
    2826             endif;
    2827         endforeach;
    2828 
    2829     endif;
     2822        }
     2823
     2824    }
    28302825
    28312826    if ( $media_dims ) : ?>
  • trunk/src/wp-admin/includes/meta-boxes.php

    r27864 r27869  
    10721072    }
    10731073
    1074     foreach ( wp_get_relevant_id3_keys( $post ) as $key => $label ): ?>
     1074    foreach ( wp_get_attachment_id3_keys( $post, 'edit' ) as $key => $label ) : ?>
    10751075    <p>
    1076         <label for="title"><?php echo $label ?></label>
    1077         <input type="text" name="id3_<?php echo esc_attr( $key ) ?>" id="id3_<?php echo esc_attr( $key ) ?>" class="widefat" value="<?php
     1076        <label for="title"><?php echo $label ?></label><br />
     1077        <input type="text" name="id3_<?php echo esc_attr( $key ) ?>" id="id3_<?php echo esc_attr( $key ) ?>" class="large-text" value="<?php
    10781078            if ( ! empty( $meta[ $key ] ) ) {
    10791079                echo esc_attr( $meta[ $key ] );
     
    10811081        ?>" />
    10821082    </p>
    1083     <?php endforeach;
    1084 }
     1083    <?php
     1084    endforeach;
     1085}
  • trunk/src/wp-admin/includes/post.php

    r27866 r27869  
    244244        }
    245245
    246         foreach ( wp_get_relevant_id3_keys( $post ) as $key => $label ) {
     246        foreach ( wp_get_attachment_id3_keys( $post, 'edit' ) as $key => $label ) {
    247247            if ( isset( $post_data[ 'id3_' . $key ] ) ) {
    248248                $id3data[ $key ] = sanitize_text_field( wp_unslash( $post_data[ 'id3_' . $key ] ) );
  • trunk/src/wp-includes/media.php

    r27865 r27869  
    10201020    <# } #>
    10211021    <div class="wp-playlist-caption">
    1022         <span class="wp-playlist-item-meta wp-playlist-item-title">&#8220;{{{ data.title }}}&#8221;</span>
     1022        <span class="wp-playlist-item-meta wp-playlist-item-title">&#8220;{{ data.title }}&#8221;</span>
    10231023        <# if ( data.meta.album ) { #><span class="wp-playlist-item-meta wp-playlist-item-album">{{ data.meta.album }}</span><# } #>
    10241024        <# if ( data.meta.artist ) { #><span class="wp-playlist-item-meta wp-playlist-item-artist">{{ data.meta.artist }}</span><# } #>
     
    10301030            {{ data.index ? ( data.index + '. ' ) : '' }}
    10311031            <# if ( data.caption ) { #>
    1032                 {{{ data.caption }}}
     1032                {{ data.caption }}
    10331033            <# } else { #>
    1034                 <span class="wp-playlist-item-title">&#8220;{{{ data.title }}}&#8221;</span>
     1034                <span class="wp-playlist-item-title">&#8220;{{ data.title }}&#8221;</span>
    10351035                <# if ( data.artists && data.meta.artist ) { #>
    10361036                <span class="wp-playlist-item-artist"> &mdash; {{ data.meta.artist }}</span>
     
    10391039        </div>
    10401040        <# if ( data.meta.length_formatted ) { #>
    1041         <div class="wp-playlist-item-length">{{{ data.meta.length_formatted }}}</div>
     1041        <div class="wp-playlist-item-length">{{ data.meta.length_formatted }}</div>
    10421042        <# } #>
    10431043    </div>
     
    12031203        if ( ! empty( $meta ) ) {
    12041204
    1205             foreach ( wp_get_relevant_id3_keys( $attachment ) as $key => $label ) {
     1205            foreach ( wp_get_attachment_id3_keys( $attachment ) as $key => $label ) {
    12061206                if ( ! empty( $meta[ $key ] ) ) {
    12071207                    $track['meta'][ $key ] = $meta[ $key ];
     
    13131313
    13141314/**
    1315  * Return useful keys to use to lookup data from an attachment's stored metadata
     1315 * Return useful keys to use to lookup data from an attachment's stored metadata.
    13161316 *
    13171317 * @since 3.9.0
    13181318 *
    1319  * @param WP_Post $post The post in question, provided for context.
     1319 * @param WP_Post $attachment The attachment in question, provided for context.
     1320 * @param string  $context    The context. Accepts 'edit', 'display'. Default 'display'.
    13201321 * @return array
    13211322 */
    1322 function wp_get_relevant_id3_keys( $post ) {
     1323function wp_get_attachment_id3_keys( $attachment, $context = 'display' ) {
    13231324    $fields = array(
    13241325        'artist' => __( 'Artist' ),
    13251326        'album' => __( 'Album' ),
    1326         'genre' => __( 'Genre' ),
    1327         'year' => __( 'Year' ),
    1328         'length_formatted' => __( 'Formatted Length' )
    13291327    );
     1328
     1329    if ( 'display' === $context ) {
     1330        $fields['genre']            = __( 'Genre' );
     1331        $fields['year']             = __( 'Year' );
     1332        $fields['length_formatted'] = _x( 'Length', 'video or audio' );
     1333    }
    13301334
    13311335    /**
     
    13341338     * @since 3.9.0
    13351339     *
    1336      * @param array   $fields Key/value pairs of field keys to labels.
    1337      * @param WP_Post $post   Post object.
     1340     * @param array   $fields     Key/value pairs of field keys to labels.
     1341     * @param WP_Post $attachment Attachment object.
    13381342     */
    1339     return apply_filters( 'wp_get_relevant_id3_keys', $fields, $post );
     1343    return apply_filters( 'wp_get_attachment_id3_keys', $fields, $attachment, $context );
    13401344}
    13411345/**
     
    23342338
    23352339        $response['meta'] = array();
    2336         foreach ( wp_get_relevant_id3_keys( $attachment ) as $key => $label ) {
     2340        foreach ( wp_get_attachment_id3_keys( $attachment ) as $key => $label ) {
    23372341            if ( ! empty( $meta[ $key ] ) ) {
    23382342                $response['meta'][ $key ] = $meta[ $key ];
Note: See TracChangeset for help on using the changeset viewer.