Changeset 38949 for trunk/src/wp-includes/class-wp-image-editor-imagick.php

Timestamp:

10/26/2016 07:27:51 AM (8 years ago)

Author:

mikeschroder

Message:

Media: Add support for rendering PDF thumbnails.

When support for PDFs is available, on upload,
render 'Thumbnail', 'Medium', 'Large', and 'Full' sizes of
the first page, and save them in attachment meta.

Use these renders within Add Media, Media Gallery and List views,
Attachment Details, Post/Attachment Edit screens, and Attachment pages.

Support available by default via Imagick -> ImageMagick -> Ghostscript,
but can be provided by any WP_Image_Editor that supports PDFs.

Props adamsilverstein, azaozz, celloexpressions, desrosj, dglingren, ericlewis, ipstenu, joemcgill, joyously, markoheijnen, melchoyce, mikeschroder, tomauger.
Fixes #31050.

File:

• trunk/src/wp-includes/class-wp-image-editor-imagick.php (modified) (2 diffs)

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

@@ -74 +74 @@
             'flipimage',
             'flopimage',
+            'readimage',
         );
 
@@ -145 +146 @@
 
         try {
-            $this->image = new Imagick( $this->file );
+            $this->image = new Imagick();
+            $file_parts = pathinfo( $this->file );
+
+            // By default, PDFs are rendered in a very low resolution.
+            // We want the thumbnail to be readable, so increase the rendering dpi.
+            if ( 'pdf' == strtolower( $file_parts['extension'] ) ) {
+                $this->image->setResolution( 128, 128 );
+            }
+
+            // Reading image after Imagick instantiation because `setResolution`
+            // only applies correctly before the image is read.
+            $this->image->readImage( $this->file );
 
             if ( ! $this->image->valid() )