Ticket #27573: 27573.diff

src/wp-admin/includes/image.php

@@ -134 +134 @@
         }
 
         if ( $support && ! empty( $metadata['image']['data'] ) ) {
-                $ext = '.jpg';
-                switch ( $metadata['image']['mime'] ) {
-                case 'image/gif':
-                        $ext = '.gif';
-                        break;
-                case 'image/png':
-                        $ext = '.png';
-                        break;
+                // check for existing cover
+                $hash = md5( $metadata['image']['data'] );
+                global $wpdb;
+                $sql = $wpdb->prepare(
+                        "SELECT ID FROM $wpdb->posts WHERE post_type = 'attachment' AND post_mime_type = %s AND post_content_filtered = %s",
+                        $metadata['image']['mime'],
+                        $hash
+                );
+                $exists = $wpdb->get_var( $sql );
+                if ( $exists ) {
+                        update_post_meta( $attachment_id, '_thumbnail_id', $exists );
+                } else {
+                        $ext = '.jpg';
+                        switch ( $metadata['image']['mime'] ) {
+                        case 'image/gif':
+                                $ext = '.gif';
+                                break;
+                        case 'image/png':
+                                $ext = '.png';
+                                break;
+                        }
+                        $basename = str_replace( '.', '-', basename( $file ) ) . '-image' . $ext;
+                        $uploaded = wp_upload_bits( $basename, '', $metadata['image']['data'] );
+                        if ( false === $uploaded['error'] ) {
+                                $attachment = array(
+                                        'post_mime_type' => $metadata['image']['mime'],
+                                        'post_type' => 'attachment',
+                                        'post_content' => '',
+                                        'post_content_filtered' => $hash
+                                );
+                                $sub_attachment_id = wp_insert_attachment( $attachment, $uploaded['file'] );
+                                $attach_data = wp_generate_attachment_metadata( $sub_attachment_id, $uploaded['file'] );
+                                wp_update_attachment_metadata( $sub_attachment_id, $attach_data );
+                                update_post_meta( $attachment_id, '_thumbnail_id', $sub_attachment_id );
+                        }
                 }
-                $basename = str_replace( '.', '-', basename( $file ) ) . '-image' . $ext;
-                $uploaded = wp_upload_bits( $basename, '', $metadata['image']['data'] );
-                if ( false === $uploaded['error'] ) {
-                        $attachment = array(
-                                'post_mime_type' => $metadata['image']['mime'],
-                                'post_type' => 'attachment',
-                                'post_content' => '',
-                        );
-                        $sub_attachment_id = wp_insert_attachment( $attachment, $uploaded['file'] );
-                        $attach_data = wp_generate_attachment_metadata( $sub_attachment_id, $uploaded['file'] );
-                        wp_update_attachment_metadata( $sub_attachment_id, $attach_data );
-                        update_post_meta( $attachment_id, '_thumbnail_id', $sub_attachment_id );
-                }
         }
 
         // remove the blob of binary data from the array