Changeset 46621

Timestamp:

10/30/2019 09:08:32 PM (7 years ago)

Author:

azaozz

Message:

Upload:

• Run the wp_generate_attachment_metadata filter at the end in wp_update_image_subsizes() when new metadata was generated and additional image sub-sizes were created.
• Add another arg in the wp_generate_attachment_metadata filter for additional context.
• Fix inline docs and ensure the new image meta is always saved before starting post-processing.

Fixes #48472 for trunk.

File:

• trunk/src/wp-admin/includes/image.php (modified) (5 diffs)

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

@@ -143 +143 @@
                 // If there is an uploaded file, make all sub-sizes and generate all of the attachment meta.
                 if ( ! empty( $image_file ) ) {
-                        return wp_create_image_subsizes( $image_file, $attachment_id );
+                        $image_meta = wp_create_image_subsizes( $image_file, $attachment_id );
                 } else {
                         return new WP_Error( 'invalid_attachment', __( 'The attached file cannot be found.' ) );
                 }
-        }
-
-        $missing_sizes = wp_get_missing_image_subsizes( $attachment_id );
-
-        if ( empty( $missing_sizes ) ) {
-                return $image_meta;
-        }
-
-        // This also updates the image meta.
-        return _wp_make_subsizes( $missing_sizes, $image_file, $image_meta, $attachment_id );
+        } else {
+                $missing_sizes = wp_get_missing_image_subsizes( $attachment_id );
+
+                if ( empty( $missing_sizes ) ) {
+                        return $image_meta;
+                }
+
+                // This also updates the image meta.
+                $image_meta = _wp_make_subsizes( $missing_sizes, $image_file, $image_meta, $attachment_id );
+        }
+
+        /** This filter is documented in wp-admin/includes/image.php */
+        $image_meta = apply_filters( 'wp_generate_attachment_metadata', $image_meta, $attachment_id, 'update' );
+
+        // Save the updated metadata.
+        wp_update_attachment_metadata( $attachment_id, $image_meta );
+
+        return $image_meta;
 }
 
@@ -276 +284 @@
                                 }
 
+                                // Initial save of the new metadata when the original image was scaled.
+                                // At this point the file was uploaded and moved to the uploads directory
+                                // but the image sub-sizes haven't been created yet and the `sizes` array is empty.
                                 wp_update_attachment_metadata( $attachment_id, $image_meta );
                         } else {
@@ -308 +319 @@
                                 }
 
+                                // Initial save of the new metadata when the original image was rotated.
                                 wp_update_attachment_metadata( $attachment_id, $image_meta );
                         } else {
@@ -313 +325 @@
                         }
                 }
+        } else {
+                // Initial save of the new metadata when the image was not scaled or rotated.
+                wp_update_attachment_metadata( $attachment_id, $image_meta );
         }
 
@@ -579 +594 @@
          * @since 2.1.0
          *
-         * @param array $metadata      An array of attachment meta data.
-         * @param int   $attachment_id Current attachment ID.
+         * @param array  $metadata      An array of attachment meta data.
+         * @param int    $attachment_id Current attachment ID.
+         * @param string $context       Additional context. Can be 'create' when metadata was initially created for new attachment
+         *                              or 'update' when the metadata was updated.
          */
-        return apply_filters( 'wp_generate_attachment_metadata', $metadata, $attachment_id );
+        return apply_filters( 'wp_generate_attachment_metadata', $metadata, $attachment_id, 'create' );
 }