WordPress.org
Get WordPress

Make WordPress Core

Ticket #31050: 31050.3.diff

File 31050.3.diff, 9.9 KB (added by kirasong, 8 years ago)

Work around black thumbnail issue. Generate more sizes; use medium by default in gallery and large by default in attachment details. Lower rendering DPI to 128.

  • src/wp-admin/includes/image.php 

    diff --git src/wp-admin/includes/image.php src/wp-admin/includes/image.php
index 84e4c4c..de2a6eb 100644
    function wp_generate_attachment_metadata( $attachment_id, $file ) { 
    7676
    7777        $metadata = array();
    7878        $support = false;
    79         if ( preg_match('!^image/!', get_post_mime_type( $attachment )) && file_is_displayable_image($file) ) {
     79        $mime_type = get_post_mime_type( $attachment );
     80
     81        /**
     82         * Filter attachment mime-types that support thumbnail fallback.
     83         *
     84         * @since 4.7.0
     85         *
     86         * @param array $mime_types An array of mime-types.
     87         */
     88        $attachment_fallback_mimetypes = apply_filters( 'attachment_fallback_mimetypes', array( 'application/pdf' ) );
     89
     90        if ( preg_match( '!^image/!', $mime_type ) && file_is_displayable_image( $file ) ) {
    8091                $imagesize = getimagesize( $file );
    8192                $metadata['width'] = $imagesize[0];
    8293                $metadata['height'] = $imagesize[1];
    … … function wp_generate_attachment_metadata( $attachment_id, $file ) { 
    201212                        }
    202213                }
    203214        }
     215        // Check to see if we support a fallback thumbnail for this mime-type.
     216        else if ( in_array( $mime_type, $attachment_fallback_mimetypes ) ) {
     217                $editor = wp_get_image_editor( $file );
     218                $sizes = array(
     219                        'thumbnail' => array(
     220                                'width'  => get_option( "thumbnail_size_w" ),
     221                                'height' => get_option( "thumbnail_size_h" ),
     222                                'crop'   => 0,
     223                        ),
     224                        'medium' => array(
     225                                'width'  => get_option( "medium_size_w" ),
     226                                'height' => get_option( "medium_size_h" ),
     227                        ),
     228                        'large' => array(
     229                                'width'  => get_option( "large_size_w" ),
     230                                'height' => get_option( "large_size_h" ),
     231                        ),
     232                );
     233
     234                if ( ! is_wp_error( $editor ) ) { // No support for this type of file
     235                        $uploaded = $editor->save( $file, 'image/jpeg' );
     236                        unset( $editor );
     237
     238                        // Resize based on the full size image, rather than the source.
     239                        if ( ! is_wp_error( $uploaded ) ) {
     240                                $editor = wp_get_image_editor( $uploaded['path'] );
     241                                unset( $uploaded['path'] );
     242
     243                                if ( ! is_wp_error( $editor ) ) {
     244                                        $metadata['sizes'] = $editor->multi_resize( $sizes );
     245                                        $metadata['sizes']['full'] = $uploaded;
     246                                }
     247                        }
     248                }
     249        }
    204250
    205251        // Remove the blob of binary data from the array.
    206252        if ( $metadata ) {

  • src/wp-admin/includes/media.php 

    diff --git src/wp-admin/includes/media.php src/wp-admin/includes/media.php
index 1ccb998..c79e312 100644
    function edit_form_image_editor( $post ) { 
    27662766
    27672767                echo wp_video_shortcode( $attr );
    27682768
    2769         else :
     2769        elseif ( isset( $thumb_url[0] ) ):
     2770
     2771                ?>
     2772                <div class="wp_attachment_image wp-clearfix" id="media-head-<?php echo $attachment_id; ?>">
     2773                        <p id="thumbnail-head-<?php echo $attachment_id; ?>">
     2774                                <img class="thumbnail" src="<?php echo set_url_scheme( $thumb_url[0] ); ?>" style="max-width:100%" alt="" />
     2775                        </p>
     2776                </div>
     2777                <?php
     2778
     2779        else:
    27702780
    27712781                /**
    27722782                 * Fires when an attachment type can't be rendered in the edit form.

  • src/wp-includes/class-wp-image-editor-imagick.php 

    diff --git src/wp-includes/class-wp-image-editor-imagick.php src/wp-includes/class-wp-image-editor-imagick.php
index 82b872d..fb56eb1 100644
    class WP_Image_Editor_Imagick extends WP_Image_Editor { 
    7373                        'rotateimage',
    7474                        'flipimage',
    7575                        'flopimage',
     76                        'readimage',
    7677                );
    7778
    7879                // Now, test for deep requirements within Imagick.
    … … class WP_Image_Editor_Imagick extends WP_Image_Editor { 
    144145                wp_raise_memory_limit( 'image' );
    145146
    146147                try {
    147                         $this->image = new Imagick( $this->file );
     148                        $this->image = new Imagick();
     149                        $file_parts = pathinfo( $this->file );
     150
     151                        // By default, PDFs are rendered in a very low resolution.
     152                        // We want the thumbnail to be readable, so increase the rendering dpi.
     153                        if ( 'pdf' == strtolower( $file_parts['extension'] ) ) {
     154                                $this->image->setResolution( 128, 128 );
     155                        }
     156
     157                        // Reading image after Imagick instantiation because `setResolution`
     158                        // only applies correctly before the image is read.
     159                        $this->image->readImage( $this->file );
    148160
    149161                        if ( ! $this->image->valid() )
    150162                                return new WP_Error( 'invalid_image', __('File is not an image.'), $this->file);

  • src/wp-includes/media-template.php 

    diff --git src/wp-includes/media-template.php src/wp-includes/media-template.php
index c669921..95fbb9a 100644
    function wp_print_media_templates() { 
    290290                        <div class="thumbnail thumbnail-{{ data.type }}">
    291291                                <# if ( data.uploading ) { #>
    292292                                        <div class="media-progress-bar"><div></div></div>
    293                                 <# } else if ( 'image' === data.type && data.sizes && data.sizes.large ) { #>
     293                                <# } else if ( data.sizes && data.sizes.large ) { #>
    294294                                        <img class="details-image" src="{{ data.sizes.large.url }}" draggable="false" alt="" />
    295                                 <# } else if ( 'image' === data.type && data.sizes && data.sizes.full ) { #>
     295                                <# } else if ( data.sizes && data.sizes.full ) { #>
    296296                                        <img class="details-image" src="{{ data.sizes.full.url }}" draggable="false" alt="" />
    297297                                <# } else if ( -1 === jQuery.inArray( data.type, [ 'audio', 'video' ] ) ) { #>
    298298                                        <img class="details-image icon" src="{{ data.icon }}" draggable="false" alt="" />
    … … function wp_print_media_templates() { 
    454454                                        <div class="centered">
    455455                                                <# if ( data.image && data.image.src && data.image.src !== data.icon ) { #>
    456456                                                        <img src="{{ data.image.src }}" class="thumbnail" draggable="false" alt="" />
     457                                                <# } else if ( data.sizes && data.sizes.medium ) { #>
     458                                                        <img src="{{ data.sizes.medium.url }}" class="thumbnail" draggable="false" alt="" />
    457459                                                <# } else { #>
    458460                                                        <img src="{{ data.icon }}" class="icon" draggable="false" alt="" />
    459461                                                <# } #>

  • src/wp-includes/media.php 

    diff --git src/wp-includes/media.php src/wp-includes/media.php
index f6c5f0f..58cbabf 100644
    function image_hwstring( $width, $height ) { 
    183183 *                     the image is an intermediate size. False on failure.
    184184 */
    185185function image_downsize( $id, $size = 'medium' ) {
    186 
    187         if ( !wp_attachment_is_image($id) )
    188                 return false;
     186        $is_image = wp_attachment_is_image( $id );
    189187
    190188        /**
    191189         * Filters whether to preempt the output of image_downsize().
    … … function image_downsize( $id, $size = 'medium' ) { 
    210208        $is_intermediate = false;
    211209        $img_url_basename = wp_basename($img_url);
    212210
     211        // If the file isn't an image, attempt to replace its URL with a rendered image from its meta.
     212        if ( ! $is_image ) {
     213                if ( ! empty( $meta['sizes'] ) ) {
     214                        $img_url = str_replace( $img_url_basename, $meta['sizes']['full']['file'], $img_url );
     215                        $img_url_basename = $meta['sizes']['full']['file'];
     216                        $width = $meta['sizes']['full']['width'];
     217                        $height = $meta['sizes']['full']['height'];
     218                } else {
     219                        return false;
     220                }
     221        }
     222
    213223        // try for a new style intermediate size
    214224        if ( $intermediate = image_get_intermediate_size($id, $size) ) {
    215225                $img_url = str_replace($img_url_basename, $intermediate['file'], $img_url);
    … … function image_get_intermediate_size( $post_id, $size = 'thumbnail' ) { 
    685695        if ( is_array( $size ) ) {
    686696                $candidates = array();
    687697
     698                if ( ! isset( $imagedata['file'] ) && isset( $imagedata['sizes']['full'] ) ) {
     699                        $imagedata['height'] = $imagedata['sizes']['full']['height'];
     700                        $imagedata['width']  = $imagedata['sizes']['full']['width'];
     701                }
     702
    688703                foreach ( $imagedata['sizes'] as $_size => $data ) {
    689704                        // If there's an exact match to an existing image size, short circuit.
    690705                        if ( $data['width'] == $size[0] && $data['height'] == $size[1] ) {
    … … function image_get_intermediate_size( $post_id, $size = 'thumbnail' ) { 
    738753        }
    739754
    740755        // include the full filesystem path of the intermediate file
    741         if ( empty($data['path']) && !empty($data['file']) ) {
     756        if ( empty( $data['path'] ) && ! empty( $data['file'] ) && ! empty( $imagedata['file'] ) ) {
    742757                $file_url = wp_get_attachment_url($post_id);
    743758                $data['path'] = path_join( dirname($imagedata['file']), $data['file'] );
    744759                $data['url'] = path_join( dirname($file_url), $data['file'] );
    … … function wp_prepare_attachment_for_js( $attachment ) { 
    31233138        if ( current_user_can( 'delete_post', $attachment->ID ) )
    31243139                $response['nonces']['delete'] = wp_create_nonce( 'delete-post_' . $attachment->ID );
    31253140
    3126         if ( $meta && 'image' === $type ) {
     3141        if ( $meta && ! empty( $meta['sizes'] ) ) {
    31273142                $sizes = array();
    31283143
    31293144                /** This filter is documented in wp-admin/includes/media.php */
    … … function wp_prepare_attachment_for_js( $attachment ) { 
    31713186                        }
    31723187                }
    31733188
    3174                 $sizes['full'] = array( 'url' => $attachment_url );
     3189                if ( 'image' === $type ) {
     3190                        $sizes['full'] = array( 'url' => $attachment_url );
     3191
     3192                        if ( isset( $meta['height'], $meta['width'] ) ) {
     3193                                $sizes['full']['height'] = $meta['height'];
     3194                                $sizes['full']['width'] = $meta['width'];
     3195                                $sizes['full']['orientation'] = $meta['height'] > $meta['width'] ? 'portrait' : 'landscape';
     3196                        }
    31753197
    3176                 if ( isset( $meta['height'], $meta['width'] ) ) {
    3177                         $sizes['full']['height'] = $meta['height'];
    3178                         $sizes['full']['width'] = $meta['width'];
    3179                         $sizes['full']['orientation'] = $meta['height'] > $meta['width'] ? 'portrait' : 'landscape';
     3198                        array_merge( $response, $sizes['full'] );
    31803199                }
     3200                else if ( $meta['sizes']['full']['file'] ) {
     3201                        $sizes['full'] = array(
     3202                                'url'         => $base_url . $meta['sizes']['full']['file'],
     3203                                'height'      => $meta['sizes']['full']['height'],
     3204                                'width'       => $meta['sizes']['full']['width'],
     3205                                'orientation' => $meta['sizes']['full']['height'] < $meta['sizes']['full']['width'] ? 'portrait' : 'landscape'
     3206                        );
     3207                }
     3208
     3209                $response = array_merge( $response, array( 'sizes' => $sizes ) );
     3210        }
    31813211
    3182                 $response = array_merge( $response, array( 'sizes' => $sizes ), $sizes['full'] );
    3183         } elseif ( $meta && 'video' === $type ) {
     3212        if ( $meta && 'video' === $type ) {
    31843213                if ( isset( $meta['width'] ) )
    31853214                        $response['width'] = (int) $meta['width'];
    31863215                if ( isset( $meta['height'] ) )