Changeset 39187

Timestamp:

11/09/2016 09:04:04 PM (8 years ago)

Author:

mikeschroder

Message:

Media: Only load first PDF page for thumbnails.

To improve performance, directly load the first page of uploaded PDFs
to reduce the total clock time necessary to generate thumbnails.

Props dglingren, lukecavanagh, helen, johnbillion, mikeschroder.
See #38522.

File:

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

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

@@ -148 +148 @@
             $this->image = new Imagick();
             $file_parts = pathinfo( $this->file );
+            $filename = $this->file;
 
             // By default, PDFs are rendered in a very low resolution.
@@ -153 +154 @@
             if ( 'pdf' == strtolower( $file_parts['extension'] ) ) {
                 $this->image->setResolution( 128, 128 );
+
+                // Only load the first page.
+                $filename .= '[0]';
             }
 
             // Reading image after Imagick instantiation because `setResolution`
             // only applies correctly before the image is read.
-            $this->image->readImage( $this->file );
+            $this->image->readImage( $filename );
 
             if ( ! $this->image->valid() )