Ticket #48853: 48853.1.diff

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

@@ -140 +140 @@
                 try {
                         $this->image    = new Imagick();
                         $file_extension = strtolower( pathinfo( $this->file, PATHINFO_EXTENSION ) );
-                        $filename       = $this->file;
 
                         if ( 'pdf' === $file_extension ) {
-                                $filename = $this->pdf_setup();
+                                $pdf_loaded = $this->pdf_load_source();
+
+                                if ( is_wp_error( $pdf_loaded ) ) {
+                                        return $pdf_loaded;
+                                }
+                        } else {
+                                $this->image->readImage( $this->file );
                         }
 
-                        // Reading image after Imagick instantiation because `setResolution`
-                        // only applies correctly before the image is read.
-                        $this->image->readImage( $filename );
-
                         if ( ! $this->image->valid() ) {
                                 return new WP_Error( 'invalid_image', __( 'File is not an image.' ), $this->file );
                         }
@@ -165 +166 @@
                 }
 
                 $updated_size = $this->update_size();
+
                 if ( is_wp_error( $updated_size ) ) {
                         return $updated_size;
                 }
@@ -794 +796 @@
                         // We want the thumbnail to be readable, so increase the rendering DPI.
                         $this->image->setResolution( 128, 128 );
 
+                        // Only load the first page.
+                        return $this->file . '[0]';
+                } catch ( Exception $e ) {
+                        return new WP_Error( 'pdf_setup_failed', $e->getMessage(), $this->file );
+                }
+        }
+
+        /**
+         * Load the image produced by Ghostscript.
+         *
+         * Includes a workaround for a bug in Ghostscript 8.7 that may prevent processing of some PDF files
+         * when `use-cropbox` is set.
+         *
+         * @since 5.3.1
+         *
+         * @return true|WP_error
+         */
+        protected function pdf_load_source() {
+                $filename = $this->pdf_setup();
+
+                if ( is_wp_error( $filename ) ) {
+                        return $filename;
+                }
+
+                try {
                         // When generating thumbnails from cropped PDF pages, Imagemagick uses the uncropped
                         // area (resulting in unnecessary whitespace) unless the following option is set.
                         $this->image->setOption( 'pdf:use-cropbox', true );
 
-                        // Only load the first page.
-                        return $this->file . '[0]';
+                        // Reading image after Imagick instantiation because `setResolution`
+                        // only applies correctly before the image is read.
+                        $this->image->readImage( $filename );
                 } catch ( Exception $e ) {
-                        return new WP_Error( 'pdf_setup_failed', $e->getMessage(), $this->file );
+                        // Attempt to run `gs` without the `use-cropbox` option. See #48853.
+                        $this->image->setOption( 'pdf:use-cropbox', false );
+
+                        $this->image->readImage( $filename );
                 }
+
+                return true;
         }
 
 }